Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable NFS service for "luna" #1204

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Conversation

jochenehret
Copy link
Contributor

@jochenehret jochenehret commented Sep 26, 2024

WHAT is this change about?

Enable the NFS service for the "luna"/"fresh" environment.

What customer problem is being addressed? Use customer persona to define the problem e.g. Alana is unable to...

Alana wants to have the NFS service validated.

Please provide any contextual information.

Related CATs PR: cloudfoundry/cf-acceptance-tests#1257
CATs config update: https://github.com/cloudfoundry/relint-envs/pull/46
⚠️ Prerequisite for this PR is a new nfs-volume-release with cloudfoundry/nfs-volume-release#1038

Has a cf-deployment including this change passed cf-acceptance-tests?

  • YES
  • NO

Does this PR introduce a breaking change? Please take a moment to read through the examples before answering the question.

  • YES - please choose the category from below. Feel free to provide additional details.
  • NO

How should this change be described in cf-deployment release notes?

N/A

Does this PR introduce a new BOSH release into the base cf-deployment.yml manifest or any ops-files?

  • YES - please specify
  • NO

Does this PR make a change to an experimental or GA'd feature/component?

  • experimental feature/component
  • GA'd feature/component

Please provide Acceptance Criteria for this change?

"fresh-deploy" job must be green.

What is the level of urgency for publishing this change?

  • Urgent - unblocks current or future work
  • Slightly Less than Urgent

@jochenehret
Copy link
Contributor Author

Pushing the nfs-broker won't work at the moment because the "credhub" security group must be bound to the "staging" phase (not only "running"):
https://github.com/cloudfoundry/nfs-volume-release/blob/170abdf236baeb89468dd76b11ecc90f87db4f5c/jobs/nfsbrokerpush/templates/deploy.sh.erb#L64

@jochenehret
Copy link
Contributor Author

Related CATs change: cloudfoundry/cf-acceptance-tests#1257

@jochenehret
Copy link
Contributor Author

⚠️ This PR first needs a new nfs-volume release with this fix: cloudfoundry/nfs-volume-release#1038

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants