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.

Sign up for GitHub

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

Jump to bottom

(RHEL-1087) Automation improvements and issue fixes #421

Merged
merged 3 commits into from
Jan 12, 2024
Merged

(RHEL-1087) Automation improvements and issue fixes #421

merged 3 commits into from
Jan 12, 2024

Conversation

jamacku
Copy link
Member

@jamacku jamacku commented Jan 12, 2024 •
edited by github-actions bot
Loading

  • Increase the cron interval to 45 minutes
  • Use composite GitHub Action to reduce maintenance cost
  • Add Z-streams to an array of supported versions
  • Don't fail CI when validation/linting fails; instead show everything in a sticky comment in the PR

Example of new status reporting in sticky comment

Screenshot from 2024-01-12 15-38-28

@jamacku
ci: use source-git-automation composite Action
79f6551 
This will allow us maintain the source-git automation in separate repo
and reduce the duplication of the code and noise in the systemd repo.

rhel-only

Related: RHEL-1087
@jamacku
ci: increase the cron interval to 45 minutes
ac58e27 
This should help us to avoid hitting the rate limit on the GitHub API.

rhel-only

Related: RHEL-1087
@jamacku
ci: add all Z-Stream versions to array of allowed versions
469e886 
rhel-only

Related: RHEL-1087
@jamacku jamacku added this to the RHEL-8.10 milestone Jan 12, 2024
@github-actions GitHub Actions
Copy link

Tracker - RHEL-1087

The following commits meet all requirements

commit upstream
79f6551 - ci: use source-git-automation composite Action rhel-only
ac58e27 - ci: increase the cron interval to 45 minutes rhel-only
469e886 - ci: add all Z-Stream versions to array of allowed versions rhel-only

@github-actions github-actions bot added pr/needs-ci Formerly needs-ci pr/needs-review Formerly needs-review labels Jan 12, 2024
@github-actions github-actions bot changed the title Automation improvements and issue fixes (RHEL-1087) Automation improvements and issue fixes Jan 12, 2024
msekletar
msekletar approved these changes Jan 12, 2024
View reviewed changes
Copy link
Member

@msekletar msekletar left a comment

Choose a reason for hiding this comment

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

Very superficially LGTM :)

@github-actions github-actions bot removed pr/needs-ci Formerly needs-ci pr/needs-review Formerly needs-review labels Jan 12, 2024
@jamacku jamacku merged commit 42e3739 into redhat-plumbers:main Jan 12, 2024
8 of 10 checks passed
@jamacku jamacku deleted the ci-source-git-auto branch January 12, 2024 15:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@msekletar msekletar msekletar approved these changes

Assignees
No one assigned
Labels
ci-waived released
Projects
None yet
Milestone
RHEL-8.10
Development

Successfully merging this pull request may close these issues.
2 participants
@jamacku @msekletar