Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Getting CLI working again #1267

Merged
merged 25 commits into from
Oct 4, 2024
Merged

Getting CLI working again #1267

merged 25 commits into from
Oct 4, 2024

Conversation

begonaguereca
Copy link
Contributor

@begonaguereca begonaguereca commented Sep 24, 2024

🟢 🟢 🟢

We have a couple of changes that happened in order to make this PR work:

  1. We moved our EMU orgs to point to one that Ocotoshift has control over:
    https://github.com/enterprises/fabrikam/sso

OnePass has been updated ✅

  1. Since we moved over to the new EMU org, there are lots of other existing "groups" there, so we had to fix a pagination bug. The issue was addressed in this PR and the changes were pulled into this branch.

  2. Dynamically naming the BBS tests fixes the following error:

FAILURE_REASON: Git source migration failed. Error message: A migration with the same target ssh url has already been queued.
  1. The Flaky BBS tests we have uncovered is a bug that the Git systems team is addressing here: https://github.com/github/repos/issues/12947#issuecomment-2391937931

My recommendation is that we move forward for now, catch the CI tests at the right time and let the Git systems team address that bug.

Closes: https://github.ghe.com/github/octoshift/issues/9379
Closes: https://github.ghe.com/github/octoshift/issues/9347
Closes: https://github.ghe.com/github/octoshift/issues/9373
Closes: https://github.ghe.com/github/octoshift/issues/9378

Copy link

github-actions bot commented Sep 24, 2024

Unit Test Results

811 tests   811 ✅  22s ⏱️
  1 suites    0 💤
  1 files      0 ❌

Results for commit b77dbf2.

♻️ This comment has been updated with latest results.

@begonaguereca begonaguereca changed the title TEST IGNORE Getting CLI working again Sep 26, 2024
@begonaguereca begonaguereca marked this pull request as ready for review September 26, 2024 20:54
Copy link

github-actions bot commented Oct 4, 2024

Code Coverage

Package Line Rate Branch Rate Complexity Health
bbs2gh 78% 73% 651
ado2gh 84% 78% 627
Octoshift 87% 76% 1276
gei 79% 70% 519
Summary 83% (6824 / 8177) 75% (1535 / 2046) 3073

Copy link
Collaborator

@ArinGhazarian ArinGhazarian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉 🥇

@begonaguereca begonaguereca merged commit fba738f into main Oct 4, 2024
30 checks passed
@begonaguereca begonaguereca deleted the confirming-ci branch October 4, 2024 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants