Skip to content

Disable experimental/enable-iptables-logger for now #997

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

Merged
merged 1 commit into from
Oct 11, 2022

Conversation

jochenehret
Copy link
Contributor

@jochenehret jochenehret commented Oct 10, 2022

Please take a moment to review the questions before submitting the PR

WHAT is this change about?

The "experimental-deploy" is failing on Jammy because of cloudfoundry/silk-release#67 and cloudfoundry/bosh-linux-stemcell-builder#241.

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

Update to Jammy is blocked.

Please provide any contextual information.

cloudfoundry/silk-release#67
cloudfoundry/bosh-linux-stemcell-builder#241

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

Ops files changes in the following folders are considered as NON BREAKING CHANGES
./community, ./example-vars-files, ./test

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

Something brief that conveys the change and is written with the persona (Alana, Cody...) in mind. See previous release notes for examples.

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?

The "experimental-deploy" job should succeed:
https://app-runtime-deployments.ci.cloudfoundry.org/teams/main/pipelines/cf-deployment/jobs/experimental-deploy

What is the level of urgency for publishing this change?

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

Tag your pair, your PM, and/or team!

@jochenehret jochenehret requested a review from a team October 10, 2022 06:17
Copy link
Member

@ctlong ctlong left a comment

Choose a reason for hiding this comment

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

LGTM! Let's make sure to call out the issue in the release notes of the next cf-d release.

@jochenehret jochenehret merged commit 0961fb4 into develop Oct 11, 2022
@jochenehret jochenehret deleted the disable_iptables_logger branch October 11, 2022 05:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants