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

backport to rel/7.0: missing mpu bucket prefix [S3C-632] #815

Merged
merged 2 commits into from
Jul 12, 2017

Conversation

electrachong
Copy link
Contributor

(cherry picked from commit 2fb0b68)

I think we should backport this fix to rel/7.0 since the release was affected by this bug as well.

@electrachong electrachong changed the title backport to 7/0: missing mpu bucket prefix backport to rel/7.0: missing mpu bucket prefix Jul 10, 2017
@electrachong
Copy link
Contributor Author

@ironman-machine

@ironman-machine
Copy link
Contributor

Hello @electrachong

Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "rel/7.0", "SCALITY_S3_BRANCH": "backport/fix-mpu-prefix"}

Please follow http://ci.ironmann.io/gh/scality/Integration/13967 for CI status.

@electrachong
Copy link
Contributor Author

end-to-end passed (I think regular ci tests are failing because they are not run with rel/7.0 arsenal branch)

@rahulreddy
Copy link
Collaborator

Wasn't the bug introduced when the routes were extracted in Arsenal which was after the 7.0 release? If yes, why do we need this to be backported?

@electrachong
Copy link
Contributor Author

I think it was merged before the 7.0 release / branch was created:
https://github.com/scality/S3/commits/rel/7.0

@electrachong electrachong force-pushed the backport/fix-mpu-prefix branch from 0113ede to dda6c4e Compare July 11, 2017 18:46
@electrachong
Copy link
Contributor Author

electrachong commented Jul 11, 2017

@ironman-machine after locking dependency packages

@ironman-machine
Copy link
Contributor

Hello @electrachong

Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "DEFAULT_BRANCH": "rel/7.0", "SCALITY_S3_BRANCH": "backport/fix-mpu-prefix"}

Please follow http://ci.ironmann.io/gh/scality/Integration/13980 for CI status.

@scality scality deleted a comment from ironman-machine Jul 11, 2017
@electrachong
Copy link
Contributor Author

end to end passed, altho regular ci is still failing (I guess this time because of cache..)

@electrachong electrachong changed the title backport to rel/7.0: missing mpu bucket prefix backport to rel/7.0: missing mpu bucket prefix [S3C-632] Jul 11, 2017
@LaurenSpiegel LaurenSpiegel merged commit 7c05150 into rel/7.0 Jul 12, 2017
@LaurenSpiegel LaurenSpiegel deleted the backport/fix-mpu-prefix branch July 12, 2017 00:05
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.

5 participants