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

Feature/ghcr integration #217

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

just-at-uber
Copy link
Contributor

@just-at-uber just-at-uber commented Oct 29, 2020

Experimenting with integrating with ghcr for docker image uploads.

Some caveats:

  • currently the package is not visible from the repository. You will need to go to the maintainers packages and see it there. I am unsure if switching to org if this helps at all? My understanding is GitHub is aware of this and is working on a fix (see here)
  • requires a PAT token instead of GITHUB_TOKEN. github is aware of issue and working on integrating GITHUB_TOKEN (see here)
  • ideally needs to be setup with Uber Open Source organisation. Currently it is setup with my personal account, however if I leave the org, this pipeline will break. This may not be an issue if GITHUB_TOKEN is fixed.
  • by default the package will be private. Once a package is published you will need to enable to public for the first time only.

Screenshots

just-at-uber release
Screen Shot 2020-10-29 at 1 18 44 PM

just-at-uber packages
Screen Shot 2020-10-29 at 1 19 06 PM

just-at-uber and others added 10 commits January 6, 2020 11:44
* fixing breaking test

* fixing test to search for Type anywhere in the header

* fixing task list test

* fixing test

* fixed one test but broke another

* grouping tests to stop leaking state between tests

* moving tests to make them run more consistently

* making test less flakey

* changing test to make more stable

* making test more stable

* commenting out test. will be fixed as part of history-virtualize-scroll
* fixing breaking test

* fixing test to search for Type anywhere in the header

* fixing task list test

* fixing test

* fixed one test but broke another

* grouping tests to stop leaking state between tests

* moving tests to make them run more consistently

* making test less flakey

* changing test to make more stable

* making test more stable

* commenting out test. will be fixed as part of history-virtualize-scroll
@@ -22,6 +22,22 @@ jobs:
with:
username: ${{ secrets.CADENCE_WEB_DOCKERHUB_USERNAME }}
password: ${{ secrets.CADENCE_WEB_DOCKERHUB_TOKEN }}
repository: ubercadence/web
repository: justatuber/web
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Need to swap back to ubercadence/web

uses: docker/build-push-action@v1
with:
username: ${{ github.actor }}
password: ${{ secrets.CADENCE_WEB_GHCR_PAT }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Need to change back to GITHUB_TOKEN once github supports this.
See here for more information

username: ${{ github.actor }}
password: ${{ secrets.CADENCE_WEB_GHCR_PAT }}
registry: ghcr.io
repository: just-at-uber/cadence-web/web
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Need to update repository to uber/cadence-web/web once organization has been initialized with ghcr (see here).

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.

None yet

1 participant