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

Added CI to create PRs to bump to latest Ark timestamps #505

Open
wants to merge 59 commits into
base: main
Choose a base branch
from

Conversation

wtripp180901
Copy link
Contributor

Added a nightly workflow which will check the URLs defined in appliances_repo_timestamp_sources for newer timestamps, update appliances_repo_timestamps and create a PR if any timestamps have been updated

@wtripp180901 wtripp180901 requested a review from a team as a code owner December 18, 2024 18:01
Copy link
Collaborator

@sjpb sjpb left a comment

Choose a reason for hiding this comment

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

See discussion on slack re. simplifying things so we're not defining repo URLs in 3x places.

Also needs to
a) kick off an image build in the PR - and not run the stackhpc.yml workflow

  • Could do this by changing that workflow to not run on branch names like e.g. auto/ and setting that branch for the PR
    b) make a commit to the PR to change environments/.stackhpc/terraform/cluster* vars to the new image - then run the stackhpc workflow

OR

  • do the image build on the branch
  • update ../cluster* definition
  • THEN open the PR (which runs stackhpc.yml)

@wtripp180901 wtripp180901 changed the base branch from main to refactor/pulp-urls December 19, 2024 15:53
@wtripp180901 wtripp180901 force-pushed the feat/auto-bump-timestamps branch from c3a36d0 to 29ad8db Compare December 20, 2024 13:37
@wtripp180901 wtripp180901 force-pushed the feat/auto-bump-timestamps branch from 29ad8db to 333ac6e Compare December 20, 2024 13:42
@wtripp180901 wtripp180901 force-pushed the feat/auto-bump-timestamps branch 2 times, most recently from 56cb15f to 02e3de6 Compare December 20, 2024 14:06
@wtripp180901 wtripp180901 force-pushed the feat/auto-bump-timestamps branch from 02e3de6 to bf14c8a Compare December 20, 2024 14:09
@wtripp180901 wtripp180901 force-pushed the feat/auto-bump-timestamps branch from cbc518f to 0863a9a Compare December 20, 2024 15:07
@wtripp180901 wtripp180901 force-pushed the feat/auto-bump-timestamps branch 3 times, most recently from 66d8775 to 37eb61b Compare December 20, 2024 15:24
@wtripp180901 wtripp180901 force-pushed the feat/auto-bump-timestamps branch 3 times, most recently from 58446ac to 57e2d7f Compare December 20, 2024 16:03
@wtripp180901 wtripp180901 force-pushed the feat/auto-bump-timestamps branch from 38df97e to 726dba2 Compare January 6, 2025 12:22
@wtripp180901 wtripp180901 changed the base branch from main to feat/ceph-release-train January 7, 2025 15:51
Base automatically changed from feat/ceph-release-train to main January 8, 2025 10:13
@wtripp180901
Copy link
Contributor Author

wtripp180901 commented Jan 8, 2025

Todo:

Todo after review but before merging:

  • Change reference for is_callee action to main (will break the workflow here but will work once merged)
  • Remove on.pull_request and on.workflow_dispatch from update_timestamps workflow
  • Delete current auto/bump-timestamps branch (it won't contain the two changes above)

@stackhpc stackhpc deleted a comment from github-actions bot Jan 8, 2025
@wtripp180901
Copy link
Contributor Author

@wtripp180901 wtripp180901 force-pushed the feat/auto-bump-timestamps branch 6 times, most recently from e436d5d to 7e8fd62 Compare January 13, 2025 09:48
@wtripp180901 wtripp180901 force-pushed the feat/auto-bump-timestamps branch from 7e8fd62 to 9ff34cb Compare January 13, 2025 09:50
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