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.

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

[pull] master from plexinc:master #24

Open
wants to merge 40 commits into
base: master
Choose a base branch
from

Conversation

pull[bot]
Copy link

@pull pull bot commented Jan 11, 2023

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

@CLAassistant
Copy link

CLAassistant commented Jan 11, 2023

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 14 committers have signed the CLA.

❌ krsmanovic
❌ gbooker
❌ MarshallAsch
❌ mxmeeple
❌ JoelBChapmanII
❌ charlesthomas
❌ cilindrox
❌ jpflouret
❌ d3adb5
❌ thomaspetit
❌ alphayax
❌ rdmcguire
❌ ajalewis
❌ CodeSmithian
You have signed the CLA already but the status is still pending? Let us recheck it.

@pull pull bot added the ⤵️ pull label Jan 11, 2023
mxmeeple and others added 27 commits February 9, 2023 14:10
* feat: adding initial helm chart for PMS

docs: update readme with updated value descriptions and added the support disclaimer

Co-authored-by: Gaston Festari <[email protected]>

* docs: add a contribution guide and link to it in the chart readme

---------

Co-authored-by: Gaston Festari <[email protected]>
* feat: add a github action workflow to package the helm chart
* feat: add a codeowners file for the helm charts
* feat: Add priorityClassName

* Change to quote func

Co-authored-by: Gaston Festari <[email protected]>

---------

Co-authored-by: Alexander Lewis <[email protected]>
Co-authored-by: Gaston Festari <[email protected]>
Helm argument is redundant during lint and fails the checks.
When providing the UID or GID as a number the chart fails to deploy, adding quotations to where extraEnv lands the extra environment variables fixes the issue (#94)

Updating method to quote the env values per PR conversation, updating chart version with a patch bump

Co-authored-by: Joel Chapman <[email protected]>
Allow setting annotations for the Pod template in the StatefulSet spec.
Bump the version of our Helm chart to 0.3.0 as a new feature was added.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.