Skip to content

Fix/network policies #69

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Fix/network policies #69

wants to merge 7 commits into from

Conversation

puffitos
Copy link
Collaborator

Motivation

  • Outdated go version and components
  • The port for the webhook was missing in the network policy, which was creating timeouts in clusters with hardened network security standards

Changes

Added the port, bumped the dependencies

Tests done

  • Unit & E2E tests pass
  • Tested 4 use cases manually in a CaaS RKE2 cluster
    • public MTR image with valid signature ✅
    • private MTR image with valid signature ✅
    • private gitlab internal registry with valid signature ✅
    • private gitlab registry with valid signature ✅

The new crypto changes require at least 1024 bits for an RSA key. Good.

Signed-off-by: Bruno Bressi <[email protected]>
Signed-off-by: Bruno Bressi <[email protected]>
Signed-off-by: Bruno Bressi <[email protected]>
@puffitos puffitos added dependencies Pull requests that update a dependency file go Pull requests that update go code labels Mar 21, 2025
@puffitos puffitos self-assigned this Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file go Pull requests that update go code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant