Skip to content

[patch] DRO public route app added to gitops has broken GovCloud#425

Merged
tomklapiscak merged 7 commits intomainfrom
fix-dro
Mar 27, 2026
Merged

[patch] DRO public route app added to gitops has broken GovCloud#425
tomklapiscak merged 7 commits intomainfrom
fix-dro

Conversation

@mnivedithaa
Copy link
Copy Markdown
Contributor

Issue: MASCORE-13058

Description

Exposing DNS over public has some optional variables to be set. But the current implementation requires it to be mandatorily available. Added conditions to make them optional
Found this issue breaking both Govcloud and FVT

Testing

Tested in fvtsaas which doesn't have dro_public configured
Screenshot 2026-03-27 at 7 08 27 PM

Copy link
Copy Markdown
Contributor

@tomklapiscak tomklapiscak left a comment

Choose a reason for hiding this comment

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

Verified that these changes resolve the issue in GovCloud

@tomklapiscak tomklapiscak merged commit da402c2 into main Mar 27, 2026
2 checks passed
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