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

[QA][CI] Have tests running with various deployment setup in CI #7614

Closed
3 tasks done
saw-jan opened this issue Oct 30, 2023 · 7 comments · Fixed by #10482
Closed
3 tasks done

[QA][CI] Have tests running with various deployment setup in CI #7614

saw-jan opened this issue Oct 30, 2023 · 7 comments · Fixed by #10482
Assignees

Comments

@saw-jan
Copy link
Member

saw-jan commented Oct 30, 2023

Currently, we run tests against the ocis server started from a single binary. We want to have a CI setup where we can start multiple instances of a service (as a load balancing) so that we can test against various deployment strategies.

Tasks

Taken from:

@saw-jan saw-jan mentioned this issue Oct 30, 2023
6 tasks
@SagarGi SagarGi self-assigned this Oct 30, 2023
@SwikritiT SwikritiT assigned SagarGi and unassigned PrajwolAmatya Dec 11, 2023
@SagarGi
Copy link
Member

SagarGi commented Dec 27, 2023

@saw-jan This issue needs to be discussed whether to invest time for doing and running such things in drone as it seems to be whole lot of task.

@saw-jan
Copy link
Member Author

saw-jan commented Nov 19, 2024

Setup with e2e PR: #10482

@saw-jan
Copy link
Member Author

saw-jan commented Nov 19, 2024

  • unskip smoke e2e test suite (see comment)

#10482 (comment)

It's only the problem with drone setup. I can run the upload test and it passes on local docker compose setup (multi-service)

@saw-jan
Copy link
Member Author

saw-jan commented Nov 19, 2024

I can reproduce the large upload failure with a docker-compose setup.
When running the upload e2e test from the host machine, the test works fine. But if I run the same test from a docker container (this is what drone does) then the large upload test fails.

@saw-jan
Copy link
Member Author

saw-jan commented Nov 19, 2024

Now I can reproduce it even when running from host machine.

If I use this compose, the large upload test passes. Note OCIS_URL

ocis:
    image: owncloud/ocis-rolling:master
    ports:
      - 9200:9200
    environment:
      OCIS_URL: https://localhost:9200
      ...

Run test with:

BASE_URL_OCIS=localhost:9200 \
HEADLESS=true \
pnpm test:e2e:cucumber tests/e2e/cucumber/features/smoke/uploadResumable.feature

But if I do the following then I cannot upload file larger than 9MB

ocis:
    image: owncloud/ocis-rolling:master
    ports:
      - 9200:9200
    environment:
      OCIS_URL: https://host.docker.internal:9200
      ...
    extra_hosts:
      - host.docker.internal:host-gateway

Run test with:

BASE_URL_OCIS=host.docker.internal:9200 \
HEADLESS=true \
pnpm test:e2e:cucumber tests/e2e/cucumber/features/smoke/uploadResumable.feature

@saw-jan
Copy link
Member Author

saw-jan commented Nov 19, 2024

I see these logs:

storageusers-2-1  | {"level":"info","service":"storage-users","pkg":"rhttp","traceid":"47503678543d4903986757df053c350c","time":"2024-11-19T10:25:42Z","line":"github.com/cs3org/reva/[email protected]/internal/http/interceptors/auth/auth.go:195","message":"skipping auth check for: /data/tus/2da25800-d76b-4257-82e4-18b92c086bb9"}
storageusers-2-1  | {"level":"warn","service":"storage-users","pkg":"rhttp","traceid":"47503678543d4903986757df053c350c","time":"2024-11-19T10:25:42Z","line":"github.com/cs3org/reva/[email protected]/internal/http/interceptors/auth/auth.go:248","message":"core access token not set"}

@saw-jan
Copy link
Member Author

saw-jan commented Nov 19, 2024

And using this cors config made the test pass ( 😵‍💫 ):

OCIS_CORS_ALLOW_ORIGINS: https://host.docker.internal:9200,https://host.docker.internal:9201

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants