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

[tests-only][full-ci] web e2e with multiple storage-users services #10482

Draft
wants to merge 25 commits into
base: master
Choose a base branch
from

Conversation

saw-jan
Copy link
Member

@saw-jan saw-jan commented Nov 6, 2024

Description

Related Issue

Motivation and Context

How Has This Been Tested?

  • test environment:
  • test case 1:
  • test case 2:
  • ...

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@saw-jan saw-jan added the QA:team label Nov 6, 2024
@saw-jan saw-jan self-assigned this Nov 6, 2024
@saw-jan saw-jan changed the title [test-only][full-ci] web e2e with multiple storage-users services [tests-only][full-ci] web e2e with multiple storage-users services Nov 6, 2024
@owncloud owncloud deleted a comment from update-docs bot Nov 6, 2024
@saw-jan saw-jan force-pushed the tests/loadbalance-services branch 5 times, most recently from 1d01b11 to 622dff2 Compare November 12, 2024 06:30
@micbar
Copy link
Contributor

micbar commented Nov 12, 2024

@saw-jan nice idea!

@dragonchaser @butonic couldn‘t we do that without nginx?

@saw-jan saw-jan force-pushed the tests/loadbalance-services branch 2 times, most recently from 172f45c to 836a8cc Compare November 12, 2024 07:01
@saw-jan
Copy link
Member Author

saw-jan commented Nov 12, 2024

couldn‘t we do that without nginx?

Could be possible. but I don't know if requests will be balanced by running just two services.

@saw-jan
Copy link
Member Author

saw-jan commented Nov 12, 2024

Tried in docker-compose setup. both services are getting the request logs. So it might work without nginx

storageusers-2-1  | {"level":"info","service":"storage-users","pkg":"rgrpc","traceid":"aa8668aa09b3ac33fc846d5e1147b7dc","data-server":"http://host.docker.internal:9159/data","ref":{"resource_id":{"storage_id":"storage-users-1","opaque_id":"1505e2a2-4a9e-4769-8171-722772b86c0f","space_id":"1505e2a2-4a9e-4769-8171-722772b86c0f"},"path":"./New file (1).txt"},"time":"2024-11-12T07:08:37Z","line":"github.com/cs3org/reva/[email protected]/internal/grpc/services/storageprovider/storageprovider.go:296","message":"file download"}
storageusers-1-1  | {"level":"info","service":"storage-users","pkg":"rgrpc","traceid":"a40a35729027edafc3d25a817af3548f","data-server":"http://host.docker.internal:9158/data/simple/a850ea6e-2aed-4283-ac84-58f9c54ede7b","fn":"./New file (1).txt","xs":"map[md5:100 unset:1000]","time":"2024-11-12T07:08:41Z","line":"github.com/cs3org/reva/[email protected]/internal/grpc/services/storageprovider/storageprovider.go:470","message":"file upload"}
storageusers-1-1  | {"level":"info","service":"storage-users","pkg":"rgrpc","traceid":"a40a35729027edafc3d25a817af3548f","data-server":"http://host.docker.internal:9158/data/tus/a850ea6e-2aed-4283-ac84-58f9c54ede7b","fn":"./New file (1).txt","xs":"map[md5:100 unset:1000]","time":"2024-11-12T07:08:41Z","line":"github.com/cs3org/reva/[email protected]/internal/grpc/services/storageprovider/storageprovider.go:470","message":"file upload"}
storageusers-2-1  | {"level":"info","service":"storage-users","pkg":"rgrpc","traceid":"b04816b03e5b8ad839787e5f76d02970","data-server":"http://host.docker.internal:9159/data","ref":{"resource_id":{"storage_id":"storage-users-1","opaque_id":"1505e2a2-4a9e-4769-8171-722772b86c0f","space_id":"1505e2a2-4a9e-4769-8171-722772b86c0f"},"path":"./New file (1).txt"},"time":"2024-11-12T07:08:43Z","line":"github.com/cs3org/reva/[email protected]/internal/grpc/services/storageprovider/storageprovider.go:296","message":"file download"}

@saw-jan saw-jan force-pushed the tests/loadbalance-services branch 4 times, most recently from 34e655f to ef0a1c2 Compare November 13, 2024 04:25
@saw-jan saw-jan force-pushed the tests/loadbalance-services branch 10 times, most recently from 79656b9 to 8e12b78 Compare November 15, 2024 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants