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

[stable/nfs-server-provisioner] Upgrade nfs-provisioner to v2.3.0 #20468

Merged
merged 1 commit into from
Jan 31, 2020
Merged

[stable/nfs-server-provisioner] Upgrade nfs-provisioner to v2.3.0 #20468

merged 1 commit into from
Jan 31, 2020

Conversation

lurraca
Copy link
Contributor

@lurraca lurraca commented Jan 31, 2020

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

No

What this PR does / why we need it:

It upgrades the version of nfs-provisioner from v2.2.2 to v2.3.0. v2.3.0 contains valuable features including one that fixes the 'stale file handle' issue on certain environment dealing with persistent disks.

Which issue this PR fixes

There are no open issue in this repo related to this PR.

Checklist

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

  • DCO signed
  • Chart Version bumped
  • Title of the PR starts with chart name (e.g. [stable/mychartname])

Upgrade nfs-provisioner to v2.3.0
6aa3a61 
Signed-off-by: Luis Urraca <lurraca@pivotal.io>
@helm-bot helm-bot added the Contribution Allowed If the contributor has signed the DCO or the CNCF CLA (prior to the move to a DCO). label Jan 31, 2020
@helm-bot helm-bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jan 31, 2020
@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 Jan 31, 2020
@k8s-ci-robot
Copy link
Contributor

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

@lurraca
Copy link
Contributor Author

lurraca commented Jan 31, 2020

/assign @kiall

@naseemkullah
Copy link
Collaborator

/lgtm

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

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lurraca, naseemkullah

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 the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 31, 2020
@k8s-ci-robot k8s-ci-robot merged commit 15c75d5 into helm:master Jan 31, 2020
@xrow
Copy link

xrow commented Jan 31, 2020

Hi,

I think this commit is missing the ports in https://github.com/helm/charts/blob/master/stable/nfs-server-provisioner/templates/service.yaml. Maybe.

I git this error:

E0131 22:38:04.407784 1 controller.go:908] error syncing claim "4f34df36-447a-11ea-9fd5-00259056a256": failed to provision volume with StorageClass "standard": error getting NFS server IP for volume: service SERVICE_NAME=nfs-nfs-server-provisioner is not valid; check that it has for ports map[{111 TCP}:true {111 UDP}:true {662 TCP}:true {662 UDP}:true {875 TCP}:true {875 UDP}:true {2049 TCP}:true {2049 UDP}:true {20048 TCP}:true {20048 UDP}:true {32803 TCP}:true {32803 UDP}:true] exactly one endpoint, this pod's IP POD_IP=10.128.0.91

@naseemkullah
Copy link
Collaborator

Thanks @xrow , this should help #19260

sc2borges pushed a commit to sc2borges/charts that referenced this pull request Feb 3, 2020
@lurraca
Upgrade nfs-provisioner to v2.3.0 (helm#20468)
5bf0856 
Signed-off-by: Luis Urraca <lurraca@pivotal.io>
Signed-off-by: Sergio Borges <sergio.borges@safetyculture.io>
sc2borges pushed a commit to sc2borges/charts that referenced this pull request Feb 4, 2020
@lurraca
Upgrade nfs-provisioner to v2.3.0 (helm#20468)
2cadef9 
Signed-off-by: Luis Urraca <lurraca@pivotal.io>
Signed-off-by: Sergio Borges <sergio.borges@safetyculture.io>
MarcusSorealheis added a commit to MarcusSorealheis/charts that referenced this pull request Feb 4, 2020
@MarcusSorealheis
Merge remote-tracking branch 'upstream/master' into add_superset_main…
51c5fba 
…tainer

* upstream/master: (309 commits)
  [stable/moodle] Set MOODLE_SKIP_INSTALL environment variable via values.yaml config. (helm#20523)
  [stable/prometheus-redis-exporter] Update image for exporter (helm#19317)
  update alantis version (helm#20507)
  [stable/spring-cloud-data-flow] Update to new SCDF version 2.4.0 (helm#20517)
  [stable/prometheus-blackbox-exporter] add "allowIcmp" setting (helm#20409)
  Allow DNS egress to resolve hostnames (helm#20437)
  [stable/rabbitmq] Add PrometheusRule. (helm#20478)
  [stable/drupal] Release 6.2.4 updating components versions (helm#20491)
  [stable/metabase] interpolate liveness and readiness probes (helm#20475)
  [stable/grafana] Fix a typo in the Grafana chart notes (helm#20490)
  [stable/gocd] Quote ingress host (helm#20477)
  [stable/metabase] allow setting jetty options (helm#20456)
  [stable/pgadmin] upgrade to 4.17 and more (helm#20211)
  [stable/prometheus-operator] add apiserverConfig (helm#20453)
  [stable/graylog] Fix helm#20306, add `helm2Compatibility` flag to `values.yaml` that preserves specific labels installed by helm2 for backward compatibility. (helm#20464)
  Add annotation support to pushgateway service (helm#20351)
  Upgrade nfs-provisioner to v2.3.0 (helm#20468)
  [stable/sysdig] Upgrade to latest Sysdig Image (9.5.0) (helm#20471)
  [stable/odoo] Update PostgreSQL dependency to latest major (helm#20467)
  [stable/redis] Add PrometheusRule. (helm#20388)
  ...
@kvaps kvaps mentioned this pull request Feb 4, 2020
Merged
4 tasks
@naseemkullah
Copy link
Collaborator

Hi @lurraca could you please update the README values section with version 2.3.0?

mmingorance-dh pushed a commit to mmingorance-dh/charts that referenced this pull request Aug 28, 2020
@lurraca
Upgrade nfs-provisioner to v2.3.0 (helm#20468)
6ef0a0d 
Signed-off-by: Luis Urraca <lurraca@pivotal.io>
Signed-off-by: Miguel Mingorance <miguel.mingorance@deliveryhero.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kiall kiall Awaiting requested review from kiall

@naseemkullah naseemkullah Awaiting requested review from naseemkullah

Assignees

@kiall kiall

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. needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@lurraca @k8s-ci-robot @naseemkullah @xrow @kiall @helm-bot