Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

[stable/joomla] Update to the recommended pvc patterns. #449

Merged
merged 11 commits into from
Jan 28, 2017

Conversation

AmandaCameron
Copy link
Contributor

This is a split-off of PR #408 to just contain the chart joomla.
If there's any commonality in changes nessary between this PR and the others that are being submitted at once, please mention them in #408.
Some things to note:

  • This initial PR will not be updating the Chart.yaml's version string, as I feel it should be done manually per-pr once each chart's otherwise acceptable.
  • An automated script created the splits and the PRs, on a per-folder basis under stable, after copying the relevent files from a backup of my local copy of Update all stable PVCs to use recommended pattern. #408

@k8s-ci-robot
Copy link
Contributor

Hi @AmandaCameron. Thanks for your PR.

I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with @k8s-bot ok to test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jan 20, 2017
@viglesiasce
Copy link
Contributor

@k8s-bot ok to test

@prydonius prydonius added the lgtm Indicates that a PR is ready to be merged. label Jan 28, 2017
@k8s-ci-robot
Copy link
Contributor

@AmandaCameron: The following test(s) failed:

Test name Commit Details Rerun command
Jenkins Charts e2e 4771270 link @k8s-bot e2e test this

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@prydonius prydonius merged commit 9d8ed51 into helm:master Jan 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants