Skip to content

Alert-system: Map to internal cluster eoapi url - #2773

Merged
susilnem merged 3 commits into
developfrom
feat/url-remap
Aug 10, 2026
Merged

Alert-system: Map to internal cluster eoapi url#2773
susilnem merged 3 commits into
developfrom
feat/url-remap

Conversation

@sandeshit

@sandeshit sandeshit commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Changes

  • Map external urls for related events to internal cluster eoapi url.
  • Avoid raise on empty data.

Things to remember

We need to set two env for this change.
for these values, take reference from montandon configurations

EOAPI_STAC_INTERNAL_URL=
EOAPI_STAC_EXTERNAL_URL=

Checklist

  • Updated/ran unit tests
  • Updated CHANGELOG.md

NOTE

  • This is a breaking change in staging environment.

@sandeshit
sandeshit marked this pull request as draft July 1, 2026 06:07
@sandeshit
sandeshit force-pushed the feat/url-remap branch 2 times, most recently from e7bfb81 to 925387c Compare July 31, 2026 06:19

@susilnem susilnem left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@sandeshit Can you verify this changes?

Comment thread alert_system/helpers.py Outdated
@sandeshit
sandeshit requested a review from susilnem August 5, 2026 08:08
- Remap urls to internal cluster urls.
- Remove explicit failures on empty event data.
@susilnem
susilnem marked this pull request as ready for review August 10, 2026 05:53
@susilnem
susilnem merged commit 6b8e3e4 into develop Aug 10, 2026
3 checks passed
@susilnem
susilnem deleted the feat/url-remap branch August 10, 2026 05:53
@susilnem

Copy link
Copy Markdown
Member

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