Skip to content

Fix redux state for s3 based connections#3068

Merged
korvin89 merged 5 commits intomainfrom
CHARTS-12044-fix-s3-based-conn-state
Oct 28, 2025
Merged

Fix redux state for s3 based connections#3068
korvin89 merged 5 commits intomainfrom
CHARTS-12044-fix-s3-based-conn-state

Conversation

@korvin89
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Oct 27, 2025

📦 Statoscope quick diff with main-branch:

⏱ Build time: 2.2 sec (4.07%)

⚖️ Initial size: -0.08 kb (0%)

🕵️ Validation errors: 0

Full Statoscope report could be found here ↗️

@github-actions
Copy link

github-actions bot commented Oct 27, 2025

E2E Tests No Auth

Playwright test results

passed  187 passed
flaky  1 flaky
skipped  2 skipped

Details

stats  190 tests across 105 suites
duration  2 minutes, 38 seconds
commit  c5a8de0

Flaky tests

opensource › opensource-suites/wizard/visualizations/pivot-table/header.test.ts › Wizard › Pivot table › Grouping rows with markup values @screenshot

Skipped tests

opensource › opensource-suites/dash/revisions/revisionsList.test.ts › Dashboards - Versioning › Dashboard with a long list of revisions, checking the upload and the updated list of revisions
opensource › opensource-suites/dash/selectors/autoupdateRelatedSelectors.test.ts › Dashboards - Autoupdate options of group selectors › Manual selectors with aliases don't affect each other before applying when auto-update is disabled


🧐 Links with traces:

👉 Link with detailed report: report

🐳 Link with docker compose logs: logs

Last updated: 2025-10-27 21:19:54 MSK

@github-actions
Copy link

github-actions bot commented Oct 27, 2025

E2E Tests

Playwright test results

passed  183 passed
flaky  5 flaky
skipped  2 skipped

Details

stats  190 tests across 105 suites
duration  3 minutes, 3 seconds
commit  c5a8de0

Flaky tests

opensource › opensource-suites/connections/postgres.test.ts › Postgres connector interactions › Connector should be created
opensource › opensource-suites/ql/connections/selectSQLConnection.test.ts › SQL connection selection › The connection is made automatically if the SQL chart was created from the connection page
opensource › opensource-suites/wizard/visualizations/flat-table/coloring.test.ts › Wizard › Flat table › Markup with null values is colored without errors
opensource › opensource-suites/wizard/visualizations/flat-table/totals.test.ts › Wizard › Flat table › Totals with no-data
opensource › opensource-suites/wizard/visualizations/pivot-table/coloring.test.ts › Wizard › Pivot table › Custom palette bar @screenshot

Skipped tests

opensource › opensource-suites/dash/revisions/revisionsList.test.ts › Dashboards - Versioning › Dashboard with a long list of revisions, checking the upload and the updated list of revisions
opensource › opensource-suites/dash/selectors/autoupdateRelatedSelectors.test.ts › Dashboards - Autoupdate options of group selectors › Manual selectors with aliases don't affect each other before applying when auto-update is disabled


🧐 Links with traces:

👉 Link with detailed report: report

🐳 Link with docker compose logs: logs

Last updated: 2025-10-27 21:20:45 MSK

@github-actions
Copy link

github-actions bot commented Oct 27, 2025

E2E Tests Farm

Playwright test results

passed  417 passed
flaky  8 flaky
skipped  1 skipped

Details

stats  426 tests across 171 suites
duration  3 minutes, 59 seconds
commit  c5a8de0

Flaky tests

basic › suites/charts/export/forbiddenExport.test.ts › Chart export availability › Wizard with forbidden export › Export menu item is disabled if export is forbidden on connection
basic › suites/charts/export/forbiddenExport.test.ts › Chart export availability › Preview with forbidden export › Ql preview - Export menu item is disabled if export is forbidden on connection
basic › suites/charts/export/forbiddenExport.test.ts › Chart export availability › Ql - Export menu item is disabled if export is forbidden on connection
basic › suites/dataset/parameters/parameterDialogValidation.test.ts › Datasets - Parameter dialog validation › Required fields validation
basic › suites/dataset/parameters/parameterDialogValidation.test.ts › Datasets - Parameter dialog validation › Parameter name validation
basic › suites/dataset/parameters/parameterDialogValidation.test.ts › Datasets - Parameter dialog validation › Parameter default_value validation [number]
basic › suites/wizard/colors/coloredByMeasureNames.test.ts › Wizard - section "Colors" › Multidatasets. Fields with the same name should be automatically colored in different colors
basic › suites/wizard/metric/metricCustomization.test.ts › Wizard - metric chart. Settings › Size change

Skipped tests

basic › suites/wizard/combined-chart/layers-and-multidatasets.test.ts › Multi-faceted charts with layers › The data on the X-axis must be formed correctly from two datasets


👉 Link with detailed report: report

Last updated: 2025-10-27 21:24:13 MSK

dvortsovs
dvortsovs previously approved these changes Oct 27, 2025
dvortsovs
dvortsovs previously approved these changes Oct 27, 2025
@korvin89 korvin89 merged commit 4319fe1 into main Oct 28, 2025
13 checks passed
@korvin89 korvin89 deleted the CHARTS-12044-fix-s3-based-conn-state branch October 28, 2025 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants