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.

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

[full-ci][tests-only] Bump ocis master commit id to latest #4875

Merged
merged 1 commit into from
Oct 17, 2024
Merged

[full-ci][tests-only] Bump ocis master commit id to latest #4875

merged 1 commit into from
Oct 17, 2024

Conversation

nirajacharya2
Copy link
Contributor

@nirajacharya2 nirajacharya2 commented Oct 2, 2024 •
edited
Loading

part of: owncloud/QA#863

@nirajacharya2 nirajacharya2 mentioned this pull request Oct 2, 2024
Merged
9 tasks
@nirajacharya2 nirajacharya2 changed the title removing SPACE_ID_FROM_OCIS from test code [full-ci][tests-only] Bump ocis master commit id to latest Oct 3, 2024
@nirajacharya2 nirajacharya2 mentioned this pull request Oct 3, 2024
Closed
17 tasks
@nirajacharya2 nirajacharya2 force-pushed the remove-spaceidfromocis-var branch 18 times, most recently from 463303a to b1dcd72 Compare October 8, 2024 03:15
@phil-davis phil-davis self-requested a review October 8, 2024 03:27
@phil-davis phil-davis self-assigned this Oct 8, 2024
@nirajacharya2 nirajacharya2 force-pushed the remove-spaceidfromocis-var branch 5 times, most recently from d59755e to 090f6ac Compare October 8, 2024 04:23
@cs3org cs3org deleted a comment from update-docs bot Oct 8, 2024
@nirajacharya2 nirajacharya2 force-pushed the remove-spaceidfromocis-var branch 6 times, most recently from 199b4bb to 4fe3eda Compare October 8, 2024 09:58
@saw-jan
Copy link
Contributor

saw-jan commented Oct 8, 2024

This is recently added test scenario. maybe #4872 doesn't fix the code for posixfs?
We might need to skip the test for reva

Failed step: Then the HTTP status code should be "409"
             HTTP status code 500 is not the expected value 409


coreApiWebdavUploadTUS/uploadFile.feature:234
coreApiWebdavUploadTUS/uploadFile.feature:235
coreApiWebdavUploadTUS/uploadFile.feature:236

CC @butonic

@butonic
Copy link
Contributor

butonic commented Oct 17, 2024

cc @aduffeck because he is currently looking into tus/uploads in posixfs

@aduffeck
Copy link
Contributor

yes, the posixfs failures have been fixed with #4877, which also included a ocis bump.

@saw-jan
Copy link
Contributor

saw-jan commented Oct 17, 2024

yes, the posixfs failures have been fixed with #4877, which also included a ocis bump.

THanks. I will bump the latest commit in this PR

@nirajacharya2 nirajacharya2 marked this pull request as ready for review October 17, 2024 09:22
@phil-davis phil-davis merged commit 464930c into cs3org:edge Oct 17, 2024
9 checks passed
@phil-davis phil-davis deleted the remove-spaceidfromocis-var branch October 17, 2024 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@phil-davis phil-davis phil-davis approved these changes

@labkode labkode Awaiting requested review from labkode labkode is a code owner

@glpatcern glpatcern Awaiting requested review from glpatcern glpatcern is a code owner

Assignees

@phil-davis phil-davis

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@nirajacharya2 @saw-jan @butonic @aduffeck @phil-davis