Skip to content
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

deploy-cloud, base config: add explicit deployment config to build stage-stable and stage-beta both from master #2522

Merged
merged 1 commit into from
Sep 1, 2022

Conversation

himdel
Copy link
Collaborator

@himdel himdel commented Sep 1, 2022

npm run deploy reads the current branch name (in cloud mode) and decides to build a beta version when the branch is master
but since #2308, we're deploying to stage-stable as well, leading to issues on stage-stable:

Loading chunk 849 failed. (error: https://console.stage.redhat.com/beta/apps/automation-hub/js/849.1661953046888.a80f84b3e77abc4bde6d.js)

=> adding a HUB_CLOUD_BETA env var to override the autodetection,
and building with HUB_CLOUD_BETA=true for stage-beta, HUB_CLOUD_BETA=false for stage-stable,
and no override for prod builds, as before.

(and moving the npm run deploy call to custom_release.sh so we can run it twice from master)

(The deployment config gets used here: https://github.com/RedHatInsights/frontend-components/blob/9766d2c07b0b62a4410956be85974c06264fcc7c/packages/config/index.js#L37 )

…age-stable and stage-beta both from master

npm run deploy reads the current branch name (in cloud mode) and decides to build a beta version when the branch is master
but since ansible#2308, we're deploying to stage-stable as well
leading to issues on stage-stable:

    Loading chunk 849 failed. (error: https://console.stage.redhat.com/beta/apps/automation-hub/js/849.1661953046888.a80f84b3e77abc4bde6d.js)

adding a HUB_CLOUD_BETA env var to override the autodetection,
and building with HUB_CLOUD_BETA=true for stage-beta, HUB_CLOUD_BETA=false for stage-stable,
and no override for prod builds, as before.

No-Issue
@himdel
Copy link
Collaborator Author

himdel commented Sep 1, 2022

Cc @awcrosby @Hyperkid123

This should fix stage-stable deploys.. I hope :)

@himdel himdel merged commit 2fe04ea into ansible:master Sep 1, 2022
@himdel himdel deleted the stage-stable branch September 1, 2022 14:45
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