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

fix: Incorrect chart values for user, group and memory limit #344

Merged
merged 1 commit into from
Sep 28, 2024

Conversation

gianmarco-mameli
Copy link
Contributor

Closes #343

Fixed chart values for user, group and memory limit

Copy link
Collaborator

@dirien dirien left a comment

Choose a reason for hiding this comment

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

Hey @gianmarco-mameli,

Thanks for the PR. Could you revert the runAsUser/Group values and we are good to go to merge.

charts/node-red/values.yaml Show resolved Hide resolved
charts/node-red/values.yaml Outdated Show resolved Hide resolved
charts/node-red/Chart.yaml Outdated Show resolved Hide resolved
@dirien
Copy link
Collaborator

dirien commented Sep 27, 2024

hi @gianmarco-mameli,

can you please run git commit --amend --signoff to approve the DCO please?

@gianmarco-mameli
Copy link
Contributor Author

Hi @dirien, no problem, I'm trying to solve the action on DCO but seems that your command, and the commands suggested from the DCO aren't working.
Tell me if I'm doing something wrong, thanks

@dirien
Copy link
Collaborator

dirien commented Sep 27, 2024

@gianmarco-mameli

I would say now rebase the main branch into this branch, then git commit --amend --signoff and then git push origin HEAD --force it. This should fix this.

@gianmarco-mameli gianmarco-mameli force-pushed the fix_incorrect_values branch 2 times, most recently from b1e0a3e to 58e0e3a Compare September 27, 2024 15:58
@gianmarco-mameli
Copy link
Contributor Author

I think it's still blocked, sorry for this mess

@dirien dirien force-pushed the fix_incorrect_values branch 4 times, most recently from e753974 to 442a60d Compare September 28, 2024 08:09
Signed-off-by: Gianmarco Mameli <[email protected]>
@dirien dirien force-pushed the fix_incorrect_values branch from 442a60d to bf5907b Compare September 28, 2024 08:10
@dirien
Copy link
Collaborator

dirien commented Sep 28, 2024

@gianmarco-mameli Fixed it.

@dirien dirien merged commit 43916a8 into SchwarzIT:main Sep 28, 2024
3 checks passed
@gianmarco-mameli gianmarco-mameli deleted the fix_incorrect_values branch September 29, 2024 20:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect chart values for user, group and memory limit
2 participants