Skip to content

Release Fix: Updated download center s3 bucket name #1018

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Mar 4, 2022
Merged

Release Fix: Updated download center s3 bucket name #1018

merged 3 commits into from
Mar 4, 2022

Conversation

@andreaangiolillo
Copy link
Collaborator

@andreaangiolillo andreaangiolillo commented Mar 4, 2022 •
edited
Loading

Proposed changes

Checklist

  • I have signed the MongoDB CLA
  • I have added tests that prove my fix is effective or that my feature works
  • I have added any necessary documentation in document requirements section listed in CONTRIBUTING.md (if appropriate)
  • I have addressed the @mongodb/docs-cloud-team comments (if appropriate)
  • I have updated e2e/E2E-TESTS.md (if an e2e test has been added)
  • I have run make fmt and formatted my code

Further comments

Atas msi release task failed because we were adding the msi file to the wrong folder in the download center s3 bucket (error).

This PR adds the atlascli msi file in the same mongocli folder in the download center s3 bucket

@andreaangiolillo andreaangiolillo requested review from a team and gssbzn March 4, 2022 16:34
gssbzn
gssbzn previously approved these changes Mar 4, 2022
View reviewed changes
@andreaangiolillo andreaangiolillo requested a review from gssbzn March 4, 2022 16:40
@andreaangiolillo andreaangiolillo changed the title Release Fix: Don't push atlascli.msi in the download center s3 bucket Release Fix: Updated download center s3 bucket name Mar 4, 2022
@andreaangiolillo andreaangiolillo merged commit cdde2a1 into master Mar 4, 2022
@andreaangiolillo andreaangiolillo deleted the fix_atlas_release_1 branch March 4, 2022 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@blva blva blva approved these changes

@gssbzn gssbzn Awaiting requested review from gssbzn

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@andreaangiolillo @gssbzn @blva