Skip to content

Conversation

@scott-grimes
Copy link
Contributor

allow configuration of the operators resources

@DeyvsonL
Copy link

Do we have any expectations about when this will be merged?

@tkp75
Copy link

tkp75 commented Jun 22, 2022

Shouldn't the manager-upgrade.yaml have resources support too?

@benjaminjb
Copy link
Contributor

Could you
(a) rebase off devel
(b) PR this to merge into devel
(c) and add resources to the manager-upgrade.yaml as well? (thanks for pointing that out @tkp75 )

@benjaminjb benjaminjb linked an issue Oct 17, 2022 that may be closed by this pull request
@scott-grimes scott-grimes changed the base branch from main to devel October 17, 2022 22:26
@scott-grimes
Copy link
Contributor Author

done. feel free to edit further if needed

@benjaminjb
Copy link
Contributor

benjaminjb commented Oct 18, 2022

Tested and made some slight changes (oh and squashed into one commit), thanks for the PR -- can you deploy with Helm with these changes?

@benjaminjb benjaminjb merged commit aef0cda into CrunchyData:devel Oct 19, 2022
tjmoore4 pushed a commit to tjmoore4/postgres-operator-examples that referenced this pull request Dec 16, 2022
tjmoore4 pushed a commit to tjmoore4/postgres-operator-examples that referenced this pull request Dec 16, 2022
tjmoore4 pushed a commit that referenced this pull request Dec 21, 2022
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.

Resources for operator not configurable via helm

4 participants