Skip to content

MCK: Release 1.0.1 #114

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

Merged
merged 5 commits into from
May 14, 2025
Merged

MCK: Release 1.0.1 #114

merged 5 commits into from
May 14, 2025

Conversation

MaciejKaras
Copy link
Collaborator

@MaciejKaras MaciejKaras commented May 14, 2025

Summary

Merge release-1.0 branch back to master

Proof of Work

Passing CI checks is enough.

Checklist

  • Have you linked a jira ticket and/or is the ticket in the title?
  • Have you checked whether your jira ticket required DOCSP changes?
  • Have you checked for release_note changes?

Reminder (Please remove this when merging)

  • Please try to Approve or Reject Changes the PR, keep PRs in review as short as possible
  • Our Short Guide for PRs: Link
  • Remember the following Communication Standards - use comment prefixes for clarity:
    • blocking: Must be addressed before approval.
    • follow-up: Can be addressed in a later PR or ticket.
    • q: Clarifying question.
    • nit: Non-blocking suggestions.
    • note: Side-note, non-actionable. Example: Praise
    • --> no prefix is considered a question

@MaciejKaras MaciejKaras marked this pull request as ready for review May 14, 2025 09:37
@MaciejKaras MaciejKaras requested a review from a team as a code owner May 14, 2025 09:37
@MaciejKaras MaciejKaras mentioned this pull request May 14, 2025
3 tasks
Copy link
Member

@SimonBaeumer SimonBaeumer left a comment

Choose a reason for hiding this comment

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

We need to cherry-pick the changes in 1.0.1 and open PRs to master. Our changes will be squashed in this case.

@MaciejKaras MaciejKaras force-pushed the mkaras/merge-release-1.0 branch 2 times, most recently from f8379a2 to 64cdb6a Compare May 14, 2025 10:12
SimonBaeumer
SimonBaeumer approved these changes May 14, 2025
@MaciejKaras
Copy link
Collaborator Author

Talked with @SimonBaeumer about merging the PR via squash commit vs moving the commits from release-1.0 branch to preserve history. We have agreed to use Rebase and merge strategy for this PR.

MaciejKaras and others added 5 commits May 14, 2025 14:33
# Summary

Add missing `e2e_smoke_release` and `e2e_static_smoke_release` context.

Previous error:
```(!!) Cannot switch context: File scripts/dev/contexts/e2e_static_smoke_release does not exist.```

## Proof of Work

https://spruce.mongodb.com/version/68224ca9feee0e0007c41c3c/tasks?sorts=STATUS%3AASC%3BBASE_STATUS%3ADESC

## Checklist
- [ ] Have you linked a jira ticket and/or is the ticket in the title?
- [ ] Have you checked whether your jira ticket required DOCSP changes?
- [ ] Have you checked for release_note changes?

## Reminder (Please remove this when merging)
- Please try to Approve or Reject Changes the PR, keep PRs in review as short as possible
- Our Short Guide for PRs: [Link](https://docs.google.com/document/d/1T93KUtdvONq43vfTfUt8l92uo4e4SEEvFbIEKOxGr44/edit?tab=t.0)
- Remember the following Communication Standards - use comment prefixes for clarity:
  * **blocking**: Must be addressed before approval.
  * **follow-up**: Can be addressed in a later PR or ticket.
  * **q**: Clarifying question.
  * **nit**: Non-blocking suggestions.
  * **note**: Side-note, non-actionable. Example: Praise 
  * --> no prefix is considered a question

---------

Co-authored-by: Nam Nguyen <[email protected]>
# Summary

Sync dockerfiles after release of `1.0.1` version

## Proof of Work

Not needed.

## Checklist
- [ ] Have you linked a jira ticket and/or is the ticket in the title?
- [ ] Have you checked whether your jira ticket required DOCSP changes?
- [ ] Have you checked for release_note changes?

## Reminder (Please remove this when merging)
- Please try to Approve or Reject Changes the PR, keep PRs in review as
short as possible
- Our Short Guide for PRs:
[Link](https://docs.google.com/document/d/1T93KUtdvONq43vfTfUt8l92uo4e4SEEvFbIEKOxGr44/edit?tab=t.0)
- Remember the following Communication Standards - use comment prefixes
for clarity:
  * **blocking**: Must be addressed before approval.
  * **follow-up**: Can be addressed in a later PR or ticket.
  * **q**: Clarifying question.
  * **nit**: Non-blocking suggestions.
  * **note**: Side-note, non-actionable. Example: Praise 
  * --> no prefix is considered a question
# Summary

Missing release notes + goreleaser fix/todo

## Proof of Work

No changes in the code. `goreleaser` was successfully tested during
release

## Checklist
- [ ] Have you linked a jira ticket and/or is the ticket in the title?
- [ ] Have you checked whether your jira ticket required DOCSP changes?
- [ ] Have you checked for release_note changes?

## Reminder (Please remove this when merging)
- Please try to Approve or Reject Changes the PR, keep PRs in review as
short as possible
- Our Short Guide for PRs:
[Link](https://docs.google.com/document/d/1T93KUtdvONq43vfTfUt8l92uo4e4SEEvFbIEKOxGr44/edit?tab=t.0)
- Remember the following Communication Standards - use comment prefixes
for clarity:
  * **blocking**: Must be addressed before approval.
  * **follow-up**: Can be addressed in a later PR or ticket.
  * **q**: Clarifying question.
  * **nit**: Non-blocking suggestions.
  * **note**: Side-note, non-actionable. Example: Praise
  * --> no prefix is considered a question

---------

Co-authored-by: Simon Bäumer <[email protected]>
@MaciejKaras MaciejKaras force-pushed the mkaras/merge-release-1.0 branch from c8bb636 to f01d2d1 Compare May 14, 2025 12:34
@MaciejKaras MaciejKaras enabled auto-merge (rebase) May 14, 2025 12:35
@MaciejKaras MaciejKaras requested a review from nammn May 14, 2025 12:35
@MaciejKaras MaciejKaras merged commit ba0ddb1 into master May 14, 2025
4 of 5 checks passed
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.

4 participants