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) ci: point C8S containers to the Vault #440

Merged
merged 1 commit into from
Nov 7, 2024
Merged

(RHEL-1087) ci: point C8S containers to the Vault #440

merged 1 commit into from
Nov 7, 2024

Conversation

mrc0mmand
Copy link
Member

@mrc0mmand mrc0mmand commented Jun 26, 2024 •
edited by github-actions bot
Loading

Temporarily point repos in C8S containers to the Vault (since C8S is EOL), until we figure out a proper solution.

Related: RHEL-1087

@github-actions github-actions bot changed the title ci: point C8S containers to the Vault (RHEL-1087) ci: point C8S containers to the Vault Jun 26, 2024
@github-actions github-actions bot added rhel-8.10.0 pr/needs-ci Formerly needs-ci pr/needs-review Formerly needs-review labels Jun 26, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 26, 2024 •
edited
Loading

Commit validation

Tracker - RHEL-1087

The following commits need an inspection

commit note
1706c74 - ci: point C8S containers to the Vault Missing upstream reference ‼️

Tracker validation

Success

🟢 Tracker RHEL-1087 has set desired product: CentOS Stream 8
🟢 Tracker RHEL-1087 has set desired component: systemd
🟢 Tracker RHEL-1087 has been approved
🟢 Tracker RHEL-1087 has set severity

Pull Request validation

Success

🟡 CI - Waived
🟢 Review - Reviewed by a member
🟢 Approval - Changes were approved

Auto Merge

Failed

🔴 Pull Request has unsupported target branch rhel-8.10.0, expected branches are: 'main,master'

Success

🟢 Pull Request is not marked as draft and it's not blocked by dont-merge label
🟢 Pull Request meet requirements, title has correct form
🟢 Pull Request meet requirements, mergeable is true
🟠 Pull Request doesn't meet requirements, mergeable_state is blocked

@mrc0mmand mrc0mmand mentioned this pull request Jun 26, 2024
Open
@github-actions github-actions bot removed the pr/needs-ci Formerly needs-ci label Jun 26, 2024
@github-actions github-actions bot added tracker/missing Formerly needs-bz and removed tracker/missing Formerly needs-bz labels Jul 17, 2024
@github-actions github-actions bot added tracker/missing Formerly needs-bz and removed tracker/missing Formerly needs-bz labels Sep 15, 2024
@mrc0mmand mrc0mmand mentioned this pull request Nov 6, 2024
Merged
@mrc0mmand
ci: point C8S containers to the Vault
1706c74 
Temporarily point repos in C8S containers to the Vault (since C8S is
EOL), until we figure out a _proper_ solution.

Related: RHEL-1087
@mrc0mmand mrc0mmand marked this pull request as ready for review November 6, 2024 14:46
@github-actions github-actions bot added pr/needs-ci Formerly needs-ci and removed tracker/missing-severity labels Nov 6, 2024
jamacku
jamacku approved these changes Nov 6, 2024
View reviewed changes
Copy link
Member

@jamacku jamacku left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions github-actions bot removed the pr/needs-review Formerly needs-review label Nov 6, 2024
@github-actions github-actions bot added pr/needs-manual-merge and removed pr/needs-ci Formerly needs-ci labels Nov 6, 2024
@jamacku jamacku merged commit 3aae107 into redhat-plumbers:rhel-8.10.0 Nov 7, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jamacku jamacku jamacku approved these changes

Assignees
No one assigned
Labels
ci-waived pr/needs-manual-merge released rhel-8.10.0
Projects
None yet
Milestone
No milestone
Development

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