Skip to content

Add http-proxy integration test #191

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Oct 24, 2025
Merged

Add http-proxy integration test #191

merged 6 commits into from
Oct 24, 2025

Conversation

@javierdelapuente
Copy link
Collaborator

@javierdelapuente javierdelapuente commented Oct 23, 2025 •
edited
Loading

Applicable spec:

Overview

Add an integration test for the http-proxy endpoint that proves that it works with the http-proxy-configurator.

Fix minor issue, as we need to listen to the changed relation to get the env vars updated after integrating.

Rationale

Juju Events Changes

Module Changes

Library Changes

Checklist

@javierdelapuente javierdelapuente changed the title Add proxy test Add http-proxy integration test Oct 24, 2025
@javierdelapuente javierdelapuente marked this pull request as ready for review October 24, 2025 07:15
@javierdelapuente javierdelapuente requested a review from a team as a code owner October 24, 2025 07:15
alithethird
alithethird approved these changes Oct 24, 2025
View reviewed changes
Copy link
Contributor

@alithethird alithethird left a comment

Choose a reason for hiding this comment

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

1 nitpick, 🚀

swetha1654
swetha1654 reviewed Oct 24, 2025
View reviewed changes
@github-actions
Copy link
Contributor

Unused entries in .trivyignore

Image: ghcr.io/canonical/go-app:af0c50b1ea4871865f397a60d62e95bd70c09275-_0.1_amd64

The following CVEs are in .trivyignore but not ignored by Trivy anymore:

@github-actions
Copy link
Contributor

Unused entries in .trivyignore

Image: ghcr.io/canonical/flask-minimal-app:bced48fd5dff7c209bccfc6aab35d0b86f07f3b5-_0.1_amd64

The following CVEs are in .trivyignore but not ignored by Trivy anymore:

@github-actions
Copy link
Contributor

Unused entries in .trivyignore

Image: ghcr.io/canonical/test-db-flask:c6ae7ab2cf348e0414f1f503ec48212790df48cb-_0.1_amd64

The following CVEs are in .trivyignore but not ignored by Trivy anymore:

@github-actions
Copy link
Contributor

Unused entries in .trivyignore

Image: ghcr.io/canonical/django-app:a2922565b9f4d1fa8d1e17564ac6ac46472fc57a-_0.1_amd64

The following CVEs are in .trivyignore but not ignored by Trivy anymore:

@github-actions
Copy link
Contributor

Unused entries in .trivyignore

Image: ghcr.io/canonical/test-async-flask:bc8532950626589247a45878bcc3401933459c8f-_0.1_amd64

The following CVEs are in .trivyignore but not ignored by Trivy anymore:

@github-actions
Copy link
Contributor

Unused entries in .trivyignore

Image: ghcr.io/canonical/test-flask:ac29e1faf47d5a93c86a5a026bb1906edc2b2fff-_0.1_amd64

The following CVEs are in .trivyignore but not ignored by Trivy anymore:

@github-actions
Copy link
Contributor

Unused entries in .trivyignore

Image: ghcr.io/canonical/django-async-app:2ef79746bfea7ece6eda3c2230473e40031534ca-_0.1_amd64

The following CVEs are in .trivyignore but not ignored by Trivy anymore:

@github-actions
Copy link
Contributor

Unused entries in .trivyignore

Image: ghcr.io/canonical/fastapi-app:386a452f4a5681a2e763675a7ddeca4217a3721c-_0.1_amd64

The following CVEs are in .trivyignore but not ignored by Trivy anymore:

@github-actions
Copy link
Contributor

Unused entries in .trivyignore

Image: ghcr.io/canonical/paas-spring-boot-app:c0b1a2b123e28186347616ed0a6ab9d3b870361c-_0.1_amd64

The following CVEs are in .trivyignore but not ignored by Trivy anymore:

@github-actions
Copy link
Contributor

Unused entries in .trivyignore

Image: ghcr.io/canonical/expressjs-app:41351c04cfda137c5432794b0c80131ab8b20daa-_0.1_amd64

The following CVEs are in .trivyignore but not ignored by Trivy anymore:

@javierdelapuente javierdelapuente merged commit b76b1a0 into main Oct 24, 2025
73 checks passed
@javierdelapuente javierdelapuente deleted the add-integ-test branch October 24, 2025 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@seb4stien seb4stien seb4stien approved these changes

@alithethird alithethird alithethird approved these changes

@nrobinaubertin nrobinaubertin Awaiting requested review from nrobinaubertin nrobinaubertin is a code owner automatically assigned from canonical/platform-engineering

@amandahla amandahla Awaiting requested review from amandahla amandahla is a code owner automatically assigned from canonical/platform-engineering

@swetha1654 swetha1654 Awaiting requested review from swetha1654

Assignees

No one assigned

Labels

trivial

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@javierdelapuente @seb4stien @alithethird @swetha1654