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

[stable/cloudserver]: add Cloudserver chart #9133

Merged
merged 1 commit into from
Dec 3, 2018

Conversation

giacomoguiulfo
Copy link
Collaborator

What this PR does / why we need it:

This PR adds a chart for Cloudserver. Cloudserver is an open-source Node.js implementation of the Amazon S3 protocol on the front-end and backend storage capabilities to multiple clouds, including Azure and Google.

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • DCO signed
  • Chart Version bumped
  • Variables are documented in the README.md

@helm-bot helm-bot added Contribution Allowed If the contributor has signed the DCO or the CNCF CLA (prior to the move to a DCO). size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 8, 2018
@k8s-ci-robot
Copy link
Contributor

Hi @giacomoguiulfo. Thanks for your PR.

I'm waiting for a helm member to verify that this patch is reasonable to test. If it is, they should reply with /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.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

@k8s-ci-robot k8s-ci-robot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Nov 8, 2018
@giacomoguiulfo giacomoguiulfo force-pushed the feature/add-cloudserver-chart branch from 62c00c8 to a96718f Compare November 8, 2018 22:50
@helm-bot helm-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 8, 2018
@giacomoguiulfo giacomoguiulfo force-pushed the feature/add-cloudserver-chart branch from a96718f to 31bee87 Compare November 8, 2018 22:53
@helm-bot helm-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 8, 2018
@giacomoguiulfo
Copy link
Collaborator Author

/assign @davidkarlsen

@giacomoguiulfo
Copy link
Collaborator Author

/assign @paulczar

@giacomoguiulfo
Copy link
Collaborator Author

/assign @foxish

@cpanato
Copy link
Member

cpanato commented Nov 9, 2018

/ok-to-test

@k8s-ci-robot k8s-ci-robot added ok-to-test and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Nov 9, 2018
@giacomoguiulfo giacomoguiulfo force-pushed the feature/add-cloudserver-chart branch from 31bee87 to 147c702 Compare November 9, 2018 21:46
@helm-bot helm-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 9, 2018
@giacomoguiulfo giacomoguiulfo force-pushed the feature/add-cloudserver-chart branch from 147c702 to 8a823c4 Compare November 9, 2018 22:33
@helm-bot helm-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 9, 2018
@giacomoguiulfo giacomoguiulfo force-pushed the feature/add-cloudserver-chart branch from 8a823c4 to 9739581 Compare November 9, 2018 22:55
@helm-bot helm-bot removed the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Nov 9, 2018
@helm-bot helm-bot added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Nov 9, 2018
@giacomoguiulfo giacomoguiulfo force-pushed the feature/add-cloudserver-chart branch from 12a29dd to 4852468 Compare November 9, 2018 23:18
@helm-bot helm-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 9, 2018
@giacomoguiulfo giacomoguiulfo force-pushed the feature/add-cloudserver-chart branch from 4852468 to 7ac25a1 Compare November 9, 2018 23:27
@helm-bot helm-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 9, 2018
@giacomoguiulfo giacomoguiulfo force-pushed the feature/add-cloudserver-chart branch from 7ac25a1 to bd3a7b8 Compare November 10, 2018 00:12
@helm-bot helm-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 10, 2018
@giacomoguiulfo giacomoguiulfo force-pushed the feature/add-cloudserver-chart branch from 874b482 to fe9c308 Compare November 10, 2018 00:18
@helm-bot helm-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 10, 2018
@giacomoguiulfo giacomoguiulfo force-pushed the feature/add-cloudserver-chart branch from fe9c308 to 4b7d6f8 Compare November 13, 2018 18:59
@helm-bot helm-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Nov 13, 2018
@giacomoguiulfo
Copy link
Collaborator Author

/assign @prydonius

Copy link
Member

@prydonius prydonius left a comment

Choose a reason for hiding this comment

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

/lgtm

name: cloudserver
version: 1.0.0
appVersion: 8.1.1
kubeVersion: "^1.8.0-0"
Copy link
Member

Choose a reason for hiding this comment

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

how does the -0 indicating a pre-release affect this semver range?

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: giacomoguiulfo, prydonius

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Dec 3, 2018
@k8s-ci-robot k8s-ci-robot merged commit f06383c into helm:master Dec 3, 2018
@giacomoguiulfo giacomoguiulfo deleted the feature/add-cloudserver-chart branch December 3, 2018 19:41
yuchaoran2011 pushed a commit to yuchaoran2011/charts that referenced this pull request Dec 11, 2018
Signed-off-by: Giacomo Guiulfo <[email protected]>
Signed-off-by: Chaoran Yu <[email protected]>
coreypobrien pushed a commit to FairwindsOps/helm-charts that referenced this pull request Dec 31, 2018
wgiddens pushed a commit to wgiddens/charts that referenced this pull request Jan 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. Contribution Allowed If the contributor has signed the DCO or the CNCF CLA (prior to the move to a DCO). lgtm Indicates that a PR is ready to be merged. ok-to-test size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants