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

fix: revert 1.6.0 3rd attempt #83

Merged
merged 7 commits into from
Dec 5, 2023
Merged

fix: revert 1.6.0 3rd attempt #83

merged 7 commits into from
Dec 5, 2023

Conversation

bassco
Copy link
Collaborator

@bassco bassco commented Dec 5, 2023

A separate testing repo was used to determine the issues with the current workflow.

  • Workflows cannot be trigger by events that are created with the GITHUB_TOKEN. A Personal Access Token must be used to overcome this.
  • Persisted credentials from the actions/checkout override the semantic-release ENV vars of GH_TOKEN or GITHUB_TOKEN, when set to a PAT, and also do not trigger workflows.
  • The [skip ci] commit message also blocks the workflow running on tags

The organisation has been setup for PATs and a SEM_REL_TOKEN was created in this repo as an action repository secret. The value of the token is from my personal access token;OLX_SEM_REL_PAT, expiring 30.09.2024, scoped to this repo only, with the following scopes for semantic-release to succeed.

scope access
metadata read
commit statuses read-write
contents read-write
issues read-write
pull requests read-write

fixes #11

This reverts commit 7046f63.
When the tagged commit is added to the repo, we need the docker tags workflow to
create the container images. If we set the ci to skip, then the workflow is not
triggered.
The GITHUB_TOKEN does not allow triggering of workflows. The persisted credentials
from the actions/checkout overrides the env tokens used by semantic release and
therefore these credentials must be purged.

This allows the PAT to create the new version commit and tag to allow the docker
workflow to create the docker images witht he correct release tag versions.
Allow the workflow to be cancelled by evaluating failure
Copy link

github-actions bot commented Dec 5, 2023

Hey there and thank you for opening this pull request! 👋🏼

You can pull a preview container image of dali with the below command:

docker pull ghcr.io/olxgroup-oss/dali/dali:revert-1.6.0-3rd-attempt

@github-actions github-actions bot added fix and removed fix labels Dec 5, 2023
@bassco bassco merged commit 828244e into master Dec 5, 2023
7 checks passed
@bassco bassco deleted the revert-1.6.0-3rd-attempt branch December 5, 2023 12:53
@bassco
Copy link
Collaborator Author

bassco commented Dec 5, 2023

This PR is included in version 1.6.0 🎉

1 similar comment
@bassco
Copy link
Collaborator Author

bassco commented Dec 5, 2023

This PR is included in version 1.6.0 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

build improvements
1 participant