Skip to content

Conversation

ianbabysre
Copy link
Contributor

@ianbabysre ianbabysre commented Dec 27, 2024

Hello everyone !

I've read the Opensource guide on how to make contribution and felt that a Pull Request was the correct way to implement this as an Issue was maybe a bit too much, i hope i did things correctly.

I'm opening a PR to add a feature to the deployment :

This feature adds the following

  • Possibility to set a priorityClassName to the application

Let me know if this is the correct way to do so.

Copy link

Click here to review and test in web IDE: Contribute

@ianbabysre
Copy link
Contributor Author

Also i've noticed that this would not add a priorityClassName to the DaemonSet pods, i'll be happy to help implement this feature if it's something that could be useful

Co-authored-by: Anatolii Bazko <abazko@redhat.com>
Copy link

Click here to review and test in web IDE: Contribute

Copy link
Contributor

@tolusha tolusha left a comment

Choose a reason for hiding this comment

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

Sounds good. Thank you for conrtibution.
Please update README.md as wel..

Copy link

Click here to review and test in web IDE: Contribute

@ianbabysre
Copy link
Contributor Author

@tolusha I updated the readme

@tolusha tolusha merged commit ca7f186 into che-incubator:main Feb 14, 2025
2 checks passed
@devspacesbuild
Copy link

Build 3.20 :: imagepuller_3.x/150: Console, Changes, Git Data

@devspacesbuild
Copy link

@devspacesbuild
Copy link

@devspacesbuild
Copy link

Build 3.20 :: get-sources-rhpkg-container-build_3.x/8889: FAILURE

imagepuller : 3.x :: Failed in 66700746 : BREW:BUILD/STATUS:UNKNOWN
FAILURE:; copied to quay

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.

3 participants