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

CB-27475 Cleaning up express onboarding outputs #52

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

daszabo
Copy link
Contributor

@daszabo daszabo commented Oct 24, 2024

  • Cleaned up script outputs
  • Made changes so that the scripts should be idempotent now
  • Updated the S3 uploader GitHub workflow, to also upload the destroy scripts

@jimright jimright self-assigned this Oct 24, 2024
@jimright jimright self-requested a review October 24, 2024 16:04
@jimright jimright added the enhancement New feature or request label Oct 24, 2024
Copy link
Contributor

@jimright jimright left a comment

Choose a reason for hiding this comment

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

@daszabo,

See comments for some suggestions and issues encountered during testing this. Hopefully these should be relatively quick fixes.

@@ -59,4 +59,5 @@ jobs:
for csp in aws azure gcp
do
aws s3 cp ${csp}/quickstart.sh s3://${{ vars.S3_BUCKET }}/${csp}/${{ env.S3_FOLDER }}/
aws s3 cp ${csp}/destroy.sh s3://${{ vars.S3_BUCKET }}/${csp}/${{ env.S3_FOLDER }}/
Copy link
Contributor

Choose a reason for hiding this comment

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

Ooops, I have this in main but forgot to add here. We should rebase the eob-devel branch onto main at some point to get everything aligned.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, I didn't notice. I agree we should rebase this branch

aws/quickstart.sh Outdated Show resolved Hide resolved
azure/quickstart.sh Outdated Show resolved Hide resolved
azure/quickstart.sh Outdated Show resolved Hide resolved
azure/quickstart.sh Outdated Show resolved Hide resolved
gcp/quickstart.sh Outdated Show resolved Hide resolved
gcp/quickstart.sh Outdated Show resolved Hide resolved
gcp/quickstart.sh Outdated Show resolved Hide resolved
Copy link
Contributor

@jimright jimright left a comment

Choose a reason for hiding this comment

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

Looks good. Approved.

@jimright jimright merged commit 8e76482 into cloudera-labs:eob-devel Oct 30, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants