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

Docker test for file-based service bindings #1384

Merged
merged 3 commits into from
Dec 18, 2024
Merged

Conversation

jochenehret
Copy link
Contributor

What is this change about?

Acceptance test for file-based service bindings for a Docker app. This PR also includes a GitHub action to build and push a Docker image with the "catnip" test app. The image is uploaded to the github.com package repository:
https://github.com/cloudfoundry/cf-acceptance-tests/pkgs/container/catnip-app

Please provide contextual information.

RFC0030

What version of cf-deployment have you run this cf-acceptance-test change against?

v45.0.0 with patched capi-release and diego-release

Please check all that apply for this PR:

  • introduces a new test --- Are you sure everyone should be running this test?
  • changes an existing test
  • requires an update to a CATs integration-config (must set new flag if you want to run the test)

Did you update the README as appropriate for this change?

  • YES
  • N/A

If you are introducing a new acceptance test, what is your rationale for including it CATs rather than your own acceptance test suite?

File-based service bindings will be generally available with an upcoming cf-deployment release.

How many more (or fewer) seconds of runtime will this change introduce to CATs?

90 seconds

What is the level of urgency for publishing this change?

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

* add Dockerfile to "catnip" test app
* add GH action that builds and upload the image to https://github.com/cloudfoundry/cf-acceptance-tests/pkgs/container/catnip-app
* add new test to create and push the "catnip" Docker app and validate file-based service binding
philippthun
philippthun previously approved these changes Dec 17, 2024
README.md Outdated Show resolved Hide resolved
Co-authored-by: Philipp Thun <[email protected]>
philippthun
philippthun previously approved these changes Dec 17, 2024
@jochenehret jochenehret merged commit 1ed09b9 into develop Dec 18, 2024
5 checks passed
@jochenehret jochenehret deleted the fbsb-docker-app branch December 18, 2024 09:50
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.

3 participants