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

chore: Release/1.41 #3079

Merged
merged 7 commits into from
May 14, 2024
Merged

chore: Release/1.41 #3079

merged 7 commits into from
May 14, 2024

Conversation

markphelps
Copy link
Collaborator

This PR simply shows the diff after cherry picking the following commits for a bugfix release:

These commits are already on main

markphelps and others added 6 commits May 1, 2024 09:55
…3078)

* test(integration): add failing case for rules with no distributions

Signed-off-by: George MacRorie <[email protected]>

* test(storage/fs): add failing case around no rule distributions

Signed-off-by: George MacRorie <[email protected]>

* fix(storage/fs): return empty slice instead of rule not found

Signed-off-by: George MacRorie <[email protected]>

* fix(testing/integration): update assertion to account for new flag cases

Signed-off-by: George MacRorie <[email protected]>

* fix(testing/integration): update assertion to account for new flag cases

Signed-off-by: George MacRorie <[email protected]>

---------

Signed-off-by: George MacRorie <[email protected]>
* fix(storage): make OCI reauthentication with AWS ECR

Signed-off-by: Roman Dmytrenko <[email protected]>

* rework

Signed-off-by: Roman Dmytrenko <[email protected]>

* address PR feedback and added tests

Signed-off-by: Roman Dmytrenko <[email protected]>

* update cache

Signed-off-by: Roman Dmytrenko <[email protected]>

* rework

Signed-off-by: Roman Dmytrenko <[email protected]>

* cleanup

Signed-off-by: Roman Dmytrenko <[email protected]>

* add more tests and cleanup

Signed-off-by: Roman Dmytrenko <[email protected]>

---------

Signed-off-by: Roman Dmytrenko <[email protected]>
@markphelps markphelps requested a review from a team as a code owner May 13, 2024 21:17
@markphelps markphelps requested review from erka and GeorgeMac and removed request for a team May 13, 2024 21:18
@thepabloaguilar
Copy link
Contributor

@markphelps just to let you know, I've deployed using the main branch version! Just waiting the token to expire which will happen in ~12h and see if we'll have any problem with that

@markphelps
Copy link
Collaborator Author

Awesome I will wait to do the release until I hear back @thepabloaguilar

Copy link
Collaborator

@erka erka left a comment

Choose a reason for hiding this comment

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

looks great!

@thepabloaguilar
Copy link
Contributor

It's working @markphelps

@markphelps markphelps merged commit 6e7ed7b into main May 14, 2024
6 of 8 checks passed
@markphelps markphelps deleted the release/1.41 branch May 14, 2024 13:22
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