Add priorityClassName#72
Conversation
|
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 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! |
|
@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. |
|
@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. |
Adding priorityClassName to the Deployment