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

[stable23] Run tests with primary object storage in CI + large upload fixes #31453

Merged
merged 6 commits into from
Apr 14, 2022

Conversation

icewind1991
Copy link
Member

Backport of test changes from #31266

@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Mar 4, 2022
@icewind1991 icewind1991 added this to the Nextcloud 23.0.4 milestone Mar 4, 2022
@PVince81 PVince81 changed the title Run tests with primary object storage in CI [stable23] Run tests with primary object storage in CI Mar 4, 2022
@@ -167,7 +167,10 @@ public function getAvailability() {
'last_checked' => $row['last_checked']
];
} else {
return null;
return [
Copy link
Member

Choose a reason for hiding this comment

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

also backporting behavior changes?

Copy link
Member Author

Choose a reason for hiding this comment

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

As far as I can tell the previous behavior has never hit, as none of the code paths handle the null case.

@PVince81 PVince81 changed the title [stable23] Run tests with primary object storage in CI [stable23] Run tests with primary object storage in CI + large upload fixes Mar 8, 2022
Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

env:
MINIO_ACCESS_KEY: minio
MINIO_SECRET_KEY: minio123
image: bitnami/minio:2021.10.6
Copy link
Contributor

Choose a reason for hiding this comment

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

could this run into limits due to the new docker hub policy?

@blizzz
Copy link
Member

blizzz commented Mar 10, 2022

/rebase

Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
…torage

Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
…g part files

Signed-off-by: Robin Appelman <robin@icewind.nl>
@blizzz blizzz merged commit ea3a9ba into stable23 Apr 14, 2022
@blizzz blizzz deleted the s3-primary-ci-23 branch April 14, 2022 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants