dotpcap / sharppcap

Official repository - Fully managed, cross platform (Windows, Mac, Linux) .NET library for capturing packets
1.31k stars 267 forks source link

Optimize appveyor CI #396

Closed kayoub5 closed 8 months ago

kayoub5 commented 2 years ago

So far the slowest CI we have is appveyor, it does not allow jobs to run in parallel

The hosts provided by appveyor are slow as well, it would reduce CI time a lot if we limit the scope of what appveyor run:

@chmorgan thoughts?

chmorgan commented 8 months ago

I'm open to whatever you'd like to do on this front.

On a related topic, are we dropping support for circleci? I seem to recall this but perhaps that was for PacketDotNet instead.

kayoub5 commented 8 months ago

We didn't drop circleci as far as I remember, the config files are still committed, it just stopped working (maybe when we switched the repo from your account to the dotpcap org)

I don't have enough access right on the CI websites to figure out what's going on.

On a side note, having access to at least re-run jobs on appveyor will help with the sporadically failing tests.

chmorgan commented 8 months ago

Alright, created a team in the org on GitHub and granted the team user access to my appveyor account. If you wanted to make an org appveyor account that might be better but not sure if that’s possible with the free appveyor account.

let me know if you get a chance to see if that gives you permission to rerun ci jobs etc. I can adjust here if you need admin access.

kayoub5 commented 8 months ago

Alright, created a team in the org on GitHub and granted the team user access to my appveyor account. If you wanted to make an org appveyor account that might be better but not sure if that’s possible with the free appveyor account.

let me know if you get a chance to see if that gives you permission to rerun ci jobs etc. I can adjust here if you need admin access.

I don't see any changes in my access to appveyor, the sharpcap project does not even show up in https://ci.appveyor.com/projects for me

chmorgan commented 8 months ago

Bah. No idea what might be up with it. Have you used appveyor enough to know where I should look?

On Sun, Dec 31, 2023 at 8:12 AM Ayoub Kaanich @.***> wrote:

Alright, created a team in the org on GitHub and granted the team user access to my appveyor account. If you wanted to make an org appveyor account that might be better but not sure if that’s possible with the free appveyor account.

let me know if you get a chance to see if that gives you permission to rerun ci jobs etc. I can adjust here if you need admin access.

I don't see any changes in my access to appveyor, the sharpcap project does not even show up in https://ci.appveyor.com/projects for me

— Reply to this email directly, view it on GitHub https://github.com/dotpcap/sharppcap/issues/396#issuecomment-1872945950, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAJH4AEOSR6IH2ZMOKQ46CDYMFQFHAVCNFSM5XGMZKA2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBXGI4TINJZGUYA . You are receiving this because you were mentioned.Message ID: @.***>

kayoub5 commented 8 months ago

did you select "user" or "administrator" in appvoyer team config page?

did you configure the webhook provided by appvoyer for team sync in github?

kayoub5 commented 8 months ago

maybe try to invite me by email instead of using github groups?

chmorgan commented 8 months ago

I did select 'user', and the web hook looks like its configured as the team didn't show up until I had done some additional step to provide access to appveyor.

User inherits admin permissions, which means users can run, view etc, everything admin can do. Invited you using email address just a minute ago. Lmk if this opens up the proper permissions.

kayoub5 commented 8 months ago

I got the invitation, but my account is still limited access wise, I can only see logs and download artifacts, nothing more. I think "User" is the wrong role since "Inherited" will grant me nothing if I don't belong to a role with higher access.

kayoub5 commented 8 months ago

maybe create a new "Maintainer" role with more access than "User" ?

chmorgan commented 8 months ago

alright, added maintainers with more permissions, you are correct that user has literally nothing checked off, and reassigned the GitHub team to maintainer. Can you check that? I was hoping to avoid needing per-user permissions when we have the GitHub team.

kayoub5 commented 8 months ago

What access did you add? because I don't see any changes.

chmorgan commented 8 months ago

image

image

chmorgan commented 8 months ago

Are you logging in with your GitHub account or your email address?

kayoub5 commented 8 months ago

Are you logging in with your GitHub account or your email address?

GitHub

chmorgan commented 8 months ago

Added 'maintainer' to your live email address too. Odd man.

kayoub5 commented 8 months ago

now works

chmorgan commented 8 months ago

No idea. I did hit the 'sync permissions with github' button when I updated the team to maintainer, maybe it didn't sync yet/correctly? Should I remove access to the live account so we can test or just keep this?

kayoub5 commented 8 months ago

fine by me to test again, if still doesn't not work, maybe drop a message to support.

chmorgan commented 8 months ago

Alright, removed your live user, let me know if things are still broken after you log in from GitHub.

kayoub5 commented 8 months ago

Now your account does not even show up in the account drop down for me.

chmorgan commented 8 months ago

Alright, re-added as a user.

chmorgan commented 8 months ago

@kayoub5 I saw dotnet approved appveyor integration. I didn't see a prompt or question about that here but maybe because I wasn't set as the organization in GitHub? Did you approve that?

kayoub5 commented 8 months ago

@kayoub5 I saw dotnet approved appveyor integration. I didn't see a prompt or question about that here but maybe because I wasn't set as the organization in GitHub? Did you approve that?

@PhyxionNL reviewed and approved it.

chmorgan commented 8 months ago

Ahh k. Maybe things will work correctly now without your separate account? heh

kayoub5 commented 8 months ago

it's the same account (same email and logged through GitHub button), it's just that appveyor is not detecting GitHub groups correctly.