Skip to content

feat: conditional stac-browser deployment - #634

Open
jjfrench wants to merge 2 commits into
developfrom
issue-589/stac-browser-deployment-verison-change
Open

jjfrench wants to merge 2 commits into
developfrom
issue-589/stac-browser-deployment-verison-change

Conversation

@jjfrench

@jjfrench jjfrench commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Issue

What?

  • Separates stac-browser deployment from veda-backend stack. Checks deployed stack tag against configured tag and deploys if values differ.

*Will need to modify veda-deploy to pull output from stac-browser-outputs.json

Why?

  • Details in linked issue

Testing?

  • make diff STACKS="--all"

    • shows stac-browser in its own stack
    • shows stac-browser removed from veda-backend stack
    • shows same outputs
      • Outputs
        [-] Output stacbrowserbucketname
        [-] Output stacbrowserbucketname088B771F
        [+] Output stacbrowserbucketname
        [+] Output stac-browser/bucket-name
        [+] Output StacBrowserTag StacBrowserTag
  • https://github.com/NASA-IMPACT/veda-backend/actions/runs/31568417133/job/94025888621?pr=634

  • Since this would be the first new deployment of the stac-browser stack, there's nothing for describe-stack to pull. Rerunning the deployment after first deployment should confirm "Tag unchanged — skipping deploy."

@jjfrench jjfrench self-assigned this Aug 12, 2026
@jjfrench
jjfrench requested a review from a team August 12, 2026 06:23
@jjfrench
jjfrench marked this pull request as ready for review August 12, 2026 06:23
Comment thread config.py
)

stac_browser_tag: str = Field(
"v3.2.0",

@jjfrench jjfrench Aug 12, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Once, this PR is addressed - update this to v5.0.0 to address #522
*mostly because there are three cases - stack first deploy, stack updated, stack stays the same

@jjfrench
jjfrench force-pushed the issue-589/stac-browser-deployment-verison-change branch 3 times, most recently from 6cd1101 to 0a13d5d Compare August 19, 2026 16:01
@jjfrench jjfrench linked an issue Aug 19, 2026 that may be closed by this pull request
1 task
@jjfrench
jjfrench force-pushed the issue-589/stac-browser-deployment-verison-change branch from 0a13d5d to 5bd3cd0 Compare September 17, 2026 18:40

This branch was successfully deployed

1 active deployment
dev a6161669 Deployed Sep 17, 2026 by jjfrench via predeploy #1307
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.

Stac-browser should only be built on version change (security improvement)

1 participant