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

odoo chart #313

Merged
merged 7 commits into from
Jan 19, 2017
Merged

odoo chart #313

merged 7 commits into from
Jan 19, 2017

Conversation

sameersbn
Copy link
Contributor

No description provided.

@k8s-ci-robot
Copy link
Contributor

Hi @sameersbn. 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.

If you have questions or suggestions related to this bot's behavior, please file an issue against the kubernetes/test-infra repository.

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

@k8s-bot ok to test

@viglesiasce
Copy link
Contributor

@k8s-bot ok to test

@k8s-ci-robot
Copy link
Contributor

Jenkins Charts e2e failed for commit 4fc31f4cfb27a04b06faf179e1be44aa6999003a. Full PR test history.

The magic incantation to run this job again is @k8s-bot e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

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.

viglesiasce
viglesiasce suggested changes Jan 4, 2017
View reviewed changes
stable/odoo/requirements.yaml
@@ -0,0 +1,4 @@
dependencies:
- name: postgresql
version: 0.2.1
Copy link
Contributor

Choose a reason for hiding this comment

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

Change this to point to 0.3.0 which has the fix for imagePullPolicy on the init container.

@sameersbn
Copy link
Contributor Author

Completed the suggested changes, however I would like this chart to be merged after #318

@viglesiasce viglesiasce added code reviewed lgtm Indicates that a PR is ready to be merged. UX reviewed and removed awaiting review labels Jan 17, 2017
@sameersbn sameersbn mentioned this pull request Jan 19, 2017
Merged
@prydonius prydonius merged commit 3e83da2 into helm:master Jan 19, 2017
@sameersbn sameersbn deleted the odoo-chart branch January 20, 2017 05:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@viglesiasce viglesiasce viglesiasce requested changes

Assignees
No one assigned
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. code reviewed lgtm Indicates that a PR is ready to be merged. UX reviewed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sameersbn @k8s-ci-robot @viglesiasce @prydonius