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

Parallel deployment tests are failing with the latest oCIS docker image #2950

Closed
2 tasks
saw-jan opened this issue Jan 12, 2022 · 5 comments
Closed
2 tasks
Labels

Comments

@saw-jan
Copy link
Member

saw-jan commented Jan 12, 2022

https://drone.owncloud.com/owncloud/ocis/8664/56/14
oCIS log:

{"level":"error","service":"storage","pkg":"rgrpc","traceid":"00000000000000000000000000000000","error":"error: already exists: decomposedfs: spaces: space already exists","status":{"code":7,"message":"already exists","trace":"00000000000000000000000000000000"},"request":{"opaque":{"map":{"spaceid":{"decoder":"plain","value":"OTVjYjg3MjQtMDNiMi0xMWViLWEwYTYtYzMzZWY4ZWY1M2Fk"}}},"owner":{"id":{"opaque_id":"95cb8724-03b2-11eb-a0a6-c33ef8ef53ad"}},"type":"metadata","name":"Metadata"},"time":"2022-01-12T01:00:34Z","message":"failed to create storage space"}
{"level":"error","service":"storage","pkg":"rgrpc","traceid":"00000000000000000000000000000000","error":"error: not found: 95cb8724-03b2-11eb-a0a6-c33ef8ef53ad/index.cs3","status":{"code":6,"message":"path not found when statting","trace":"00000000000000000000000000000000"},"reference":{"resource_id":{"storage_id":"95cb8724-03b2-11eb-a0a6-c33ef8ef53ad","opaque_id":"95cb8724-03b2-11eb-a0a6-c33ef8ef53ad"},"path":"./index.cs3"},"time":"2022-01-12T01:00:34Z","message":"failed to stat"}
{"level":"error","service":"storage","pkg":"rgrpc","traceid":"00000000000000000000000000000000","error":"error: not found: 890df00a-5035-4204-8132-b140954f788c/non_unique.github.com.owncloud.ocis.accounts.pkg.proto.v0.Account.Id","status":{"code":6,"message":"path not found when statting","trace":"00000000000000000000000000000000"},"reference":{"resource_id":{"storage_id":"95cb8724-03b2-11eb-a0a6-c33ef8ef53ad","opaque_id":"95cb8724-03b2-11eb-a0a6-c33ef8ef53ad"},"path":"./index.cs3/non_unique.github.com.owncloud.ocis.accounts.pkg.proto.v0.Account.Id"},"time":"2022-01-12T01:00:34Z","message":"failed to stat"}

After this PR #2739 was merged. oCIS docker image got updated with the changes and now the parallel deployment tests are failing with the latest oCIS image.

The reason that the PR passed the CI is that in parallel deployment the oCIS server is built from the docker image, not from the current branch. This means the oCIS server will not get the changes from the current branch.

So two things to do:

  • Find out why the parallel deployment tests failed with the latest changes
  • Build oCIS server from the current branch (PR branch) for parallel deployment tests so that the changes are tested early.
@saw-jan saw-jan self-assigned this Jan 12, 2022
@saw-jan saw-jan changed the title Parallel deployment tests failing with the latest oCIS docker image Parallel deployment tests are failing with the latest oCIS docker image Jan 12, 2022
@micbar
Copy link
Contributor

micbar commented Jan 12, 2022

@butonic @kobergj Seems that we missed something. Can you dig into that?

@saw-jan
Copy link
Member Author

saw-jan commented Jan 13, 2022

Maybe fixed by PR #2952

@saw-jan saw-jan removed their assignment Jan 13, 2022
@phil-davis
Copy link
Contributor

Note: PR #2955 temporarily disabled the tests in CI, so that full-ci can "pass" for now.

Whichever PR actually fixes the problem, that will need to enable the tests in CI again.

@wkloucek
Copy link
Contributor

Tracking ticket for fix in REVA edge: #2972

@wkloucek
Copy link
Contributor

wkloucek commented Feb 1, 2022

Since #3052 the parallel deployment uses the same oCIS binary as all other tests and tests are again passing

@wkloucek wkloucek closed this as completed Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants