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

github-env -> github-env-dispatch #654

Merged
merged 2 commits into from Nov 13, 2020
Merged

github-env -> github-env-dispatch #654

merged 2 commits into from Nov 13, 2020

Conversation

@ipmsteven
Copy link
Contributor

@ipmsteven ipmsteven commented Nov 12, 2020

This PR partially_resolves the github/ghes-infrastructure#1371

The backup-utils depends on a concrete implementation of github-env when it is used in backup/restore.

In the newer version of GHES, a github-env-dispatch is introduced to replace github-env when a running github-unicorn container is not a hard requriement

slack conversation https://github.slack.com/archives/C01EMTVKNF5/p1605215456026100

@ipmsteven ipmsteven requested a review from tquinn86 Nov 13, 2020
@ipmsteven ipmsteven marked this pull request as ready for review Nov 13, 2020
@ipmsteven ipmsteven force-pushed the ipmsteven/fix-github-env branch 2 times, most recently from c7be1d2 to 26ad004 Nov 13, 2020
@ipmsteven ipmsteven requested a review from jianghao0718 Nov 13, 2020
Copy link
Contributor

@jianghao0718 jianghao0718 left a comment

LGTM!

Copy link
Contributor

@cainejette cainejette left a comment

Succinct solution 👍

@ipmsteven ipmsteven force-pushed the ipmsteven/fix-github-env branch from 26ad004 to 70c92bf Nov 13, 2020
@ipmsteven
Copy link
Contributor Author

@ipmsteven ipmsteven commented Nov 13, 2020

I created two branches on enterprise2 side to test this PR on old version of GHES

  • ipmsteven/test-backup-util-on-2.21
  • ipmsteven/test-backup-util-on-2.21

also triggered ci build for those branches

  • ghe ci ipmsteven/test-backup-util-on-2.21 --backup-utils
  • ghe ci ipmsteven/test-backup-util-on-2.22 --backup-utils
@ipmsteven ipmsteven merged commit 87fd6da into master Nov 13, 2020
2 checks passed
2 checks passed
build (ubuntu-latest)
Details
build (macos-latest)
Details
@ipmsteven ipmsteven deleted the ipmsteven/fix-github-env branch Nov 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.