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

Cost opt e2e #3212

Merged
merged 5 commits into from
Dec 26, 2024
Merged

Cost opt e2e #3212

merged 5 commits into from
Dec 26, 2024

Conversation

arbulu89
Copy link
Contributor

@arbulu89 arbulu89 commented Dec 23, 2024

Description

Refactor HANA scale up cost-optimized e2e tests adding a new photofinish scenario instead of updating an already existing one.

Refactor of: #3213

@arbulu89 arbulu89 force-pushed the cost-opt-e2e branch 4 times, most recently from 7dec35f to d739f4b Compare December 24, 2024 08:20
@@ -54,6 +54,7 @@ services:
- host.docker.internal:host-gateway
postgres:
image: postgres:15
command: -c 'max_connections=200'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Adding this has fixed the e2e test that was failing always:
https://github.com/trento-project/web/actions/runs/12478685578/job/34826632861

It creates a max_connections hit error, the host takes a while to roll up, and the events sent to this host are excluded as it is in rolling_up state.

@arbulu89 arbulu89 marked this pull request as ready for review December 24, 2024 08:46
@arbulu89 arbulu89 merged commit f77bc88 into main Dec 26, 2024
31 checks passed
@arbulu89 arbulu89 deleted the cost-opt-e2e branch December 26, 2024 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants