Skip to content

Wpb 23988 fix internet access wiab stag#881

Open
mohitrajain wants to merge 12 commits intofix-wiab-stag-feedbackfrom
wpb-23988-fix-internet-access-wiab-stag
Open

Wpb 23988 fix internet access wiab stag#881
mohitrajain wants to merge 12 commits intofix-wiab-stag-feedbackfrom
wpb-23988-fix-internet-access-wiab-stag

Conversation

@mohitrajain
Copy link
Contributor

@mohitrajain mohitrajain commented Mar 17, 2026

Change type

  • Fix
  • Feature
  • Documentation
  • Security / Upgrade

Basic information

  • THIS CHANGE REQUIRES A DEPLOYMENT PACKAGE RELEASE
  • THIS CHANGE REQUIRES A WIRE-DOCS RELEASE

Testing

  • I ran/applied the changes myself, in a test environment.
  • The CI job attached to this repo will test it for me.

Offline Build CI (label-based)

Add one or more labels to trigger offline builds:

  • build-default - Full production build (ansible, terraform, all packages)
  • build-demo - Demo/WIAB build
  • build-wiab-staging - WIAB-staging build
  • build-min - Minimal build (fastest, essential charts only)
  • build-all - Run all three builds

Note: No builds run by default. Add a label to trigger CI.

Tracking

  • I added a new entry in an appropriate subdirectory of changelog.d
  • I mentioned this PR in Jira, OR I mentioned the Jira ticket in this PR.
  • I mentioned this PR in one of the issues attached to one of our repositories.

Knowledge Transfer

  • An Asciinema session is attached to the Jira ticket.

Motivation

Objective

Reason

Use case

@mohitrajain mohitrajain requested review from a team and julialongtin as code owners March 17, 2026 15:48
@mohitrajain mohitrajain changed the base branch from master to wpb-23988-enable-5.25 March 17, 2026 15:48

- Pod → Node → host public IP → DNAT → Node → Ingress

> **Note**: Using Let's encrypt with `cert-manager` requires internet access eg. `acme-v02.api.letsencrypt.org` to issue TLS certs and if you have chosen to keep the network private i.e. `private_deployment=true` for the VMs when applying nftables rules aka no internet access to VMs, then we need to make a temporary exception for this.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
> **Note**: Using Let's encrypt with `cert-manager` requires internet access eg. `acme-v02.api.letsencrypt.org` to issue TLS certs and if you have chosen to keep the network private i.e. `private_deployment=true` for the VMs when applying nftables rules aka no internet access to VMs, then we need to make a temporary exception for this.
> **Note**: Using Let's encrypt with `cert-manager` requires internet access (to at least `acme-v02.api.letsencrypt.org`) to issue TLS certs. If you have chosen to keep the network private i.e. `private_deployment=true` for the VMs when applying nftables rules aka no internet access to VMs, then we need to make a temporary exception for this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am not sure about the let's encrypt's ACME domains, I think it could be different as well. The above one is found from my limited testing. I will link to the ACME Protocol Updates https://letsencrypt.org/docs/acme-protocol-updates/ page to above

If you observe HTTP-01 challenge timeouts or self-check failures in a NAT/bridge environment, hairpin SNAT and relaxed reverse-path filtering handling may be required. One possible approach is:
> **Note:** All `nft` and `sysctl` commands should run on the adminhost.
Copy link
Member

Choose a reason for hiding this comment

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

this should not be a note, it should be an instruction: run these commands to...

Base automatically changed from wpb-23988-enable-5.25 to fix-wiab-stag-feedback March 24, 2026 13:26
@mohitrajain mohitrajain force-pushed the wpb-23988-fix-internet-access-wiab-stag branch from 5fe8c25 to d6ff314 Compare March 24, 2026 15:41
@sonarqubecloud
Copy link

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