Skip to content

Add priorityClassName#72

Merged
irby merged 3 commits into
Keyfactor:epic/AB#89767/priority-class-namefrom
slammajamma28:priority-class
Jul 1, 2026
Merged

Add priorityClassName#72
irby merged 3 commits into
Keyfactor:epic/AB#89767/priority-class-namefrom
slammajamma28:priority-class

Conversation

@slammajamma28

Copy link
Copy Markdown

Adding priorityClassName to the Deployment

@slammajamma28 slammajamma28 deleted the priority-class branch June 11, 2026 01:46
@slammajamma28 slammajamma28 restored the priority-class branch June 11, 2026 01:47
@slammajamma28 slammajamma28 reopened this Jun 23, 2026
@irby

irby commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Hey @slammajamma28 thank you for your contribution. I'm going to add this to our backlog queue to be reviewed and prioritized.

Can you also update the deploy chart's README.md to document this field on the spec?

Thanks!

@irby irby self-requested a review June 26, 2026 15:02
@slammajamma28

Copy link
Copy Markdown
Author

@irby I added priorityClassName to the README. I also added a couple other missing keys from the values file and then did a bit of rearranging so they are all in the same order as the values file. Hope that's ok!

@irby

irby commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

@slammajamma28 That looks great! Thank you for adding in the missing chart values. I'll start working to test this locally and prepare it to be merged into the next release, it may take about 1-2 weeks to do this.

@irby

irby commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

@slammajamma28 did some tests this afternoon and everything looks good. I'm going to create a pull request to merge in this changeset as well as updates to the CONTRIBUTING guide.

@irby irby changed the base branch from main to epic/AB#89767/priority-class-name July 1, 2026 18:34
@irby irby merged commit 9724e90 into Keyfactor:epic/AB#89767/priority-class-name Jul 1, 2026
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