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

Change openstack-service-critical to critical-infrastructure PriorityClass #7229

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Change openstack-service-critical to critical-infrastructure PriorityClass #7229

wants to merge 1 commit into from

Conversation

Nuckal777
Copy link
Contributor

Migrate old PriorityClass to the equivalent one in the new scheme. openstack-service-critical will stay in the metal cluster for now.

@Nuckal777
Change openstack-service-critical to critical-infrastructure Priority…
beff2ba 
…Class

Migrate old PriorityClass to the equivalent one in the new scheme.
openstack-service-critical will stay in the metal cluster for now.
galkindmitrii
galkindmitrii reviewed Oct 14, 2024
View reviewed changes
Copy link
Member

@galkindmitrii galkindmitrii left a comment

Choose a reason for hiding this comment

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

Don't we need to bump chart versions respectively?

@Nuckal777
Copy link
Contributor Author

Technically yes. I intended the change to be shipped, when the charts are changed anyway. In case the chart are updated exceptionally infrequent, I'm happy to follow-up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@galkindmitrii galkindmitrii galkindmitrii left review comments

@joker-at-work joker-at-work joker-at-work approved these changes

@auhlig auhlig Awaiting requested review from auhlig auhlig is a code owner

@defo89 defo89 Awaiting requested review from defo89 defo89 is a code owner

@fwiesel fwiesel Awaiting requested review from fwiesel fwiesel is a code owner

@Carthaca Carthaca Awaiting requested review from Carthaca Carthaca is a code owner

@bashar-alkhateeb bashar-alkhateeb Awaiting requested review from bashar-alkhateeb bashar-alkhateeb is a code owner

@businessbean businessbean Awaiting requested review from businessbean businessbean is a code owner

@s10 s10 Awaiting requested review from s10 s10 is a code owner

@stefanhipfel stefanhipfel Awaiting requested review from stefanhipfel stefanhipfel is a code owner

@IvoGoman IvoGoman Awaiting requested review from IvoGoman IvoGoman is a code owner

@occamshatchet occamshatchet Awaiting requested review from occamshatchet occamshatchet is a code owner

@BugRoger BugRoger Awaiting requested review from BugRoger BugRoger is a code owner

@databus23 databus23 Awaiting requested review from databus23 databus23 is a code owner

@dhoeller dhoeller Awaiting requested review from dhoeller dhoeller is a code owner

@edda edda Awaiting requested review from edda edda is a code owner

@geisslet geisslet Awaiting requested review from geisslet geisslet is a code owner

@artherd42 artherd42 Awaiting requested review from artherd42 artherd42 is a code owner

@martinpink martinpink Awaiting requested review from martinpink martinpink is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Nuckal777 @galkindmitrii @joker-at-work