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

adds osclass chart #345

Merged
merged 8 commits into from Jan 26, 2017
Merged

adds osclass chart #345

merged 8 commits into from Jan 26, 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 27, 2016
@prydonius
Copy link
Member

@k8s-bot ok to test

@viglesiasce
Copy link
Contributor

@k8s-bot ok to test

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

Choose a reason for hiding this comment

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

Please update to 0.5.6 which has the fix for K8s 1.5.1

@sameersbn
Copy link
Contributor Author

  • updated mariadb chart version to 0.5.6
  • imagePullPolicy explicitly set to IfNotPresent

@sameersbn sameersbn force-pushed the osclass-chart branch 2 times, most recently from 9388ac5 to 2085c90 Compare January 6, 2017 18:08
@k8s-ci-robot
Copy link
Contributor

Jenkins Charts e2e failed for commit 9388ac55945a32ab90028a6b8d8f7ee067b6749a. 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.

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.

@viglesiasce
Copy link
Contributor

@k8s-bot e2e test this

@sameersbn sameersbn force-pushed the osclass-chart branch 2 times, most recently from 40ed267 to 11fc24e Compare January 10, 2017 10:17
@viglesiasce viglesiasce added code reviewed lgtm Indicates that a PR is ready to be merged. labels Jan 23, 2017
@prydonius prydonius merged commit b8d5095 into helm:master Jan 26, 2017
@sameersbn sameersbn deleted the osclass-chart branch June 8, 2017 05:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@viglesiasce viglesiasce viglesiasce approved these 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.

None yet
4 participants
@sameersbn @k8s-ci-robot @prydonius @viglesiasce