Skip to content

[DPE-6730] Make CI cloud independent#608

Open
phvalguima wants to merge 18 commits into2/edgefrom
DPE-6730-make-CI-cloud-independent
Open

[DPE-6730] Make CI cloud independent#608
phvalguima wants to merge 18 commits into2/edgefrom
DPE-6730-make-CI-cloud-independent

Conversation

@phvalguima
Copy link

@phvalguima phvalguima commented Apr 7, 2025

Currently, our integration tests are tuned to LXD clouds that we use in our runners.

This PR adds:

  • filter_cloud decorator: specifies tests to be executed only in certain clouds
  • TEST_CONSTRAINTS: add the option to pass constraints settings to our deployments, which changes per cloud; as an environment variable
  • SUBSTRATE: option to select which cloud we want to deploy and tests will be filtered accordingly. If not set, lxd is used.

Pedro Guimaraes added 18 commits March 6, 2025 16:57
[DPE-6771] Add sniff_timeout
a5fd59b 
Currently, we are using default sniff_timeout, which is 100ms.

That is not enough for the opensearchpy to a remote cluster.

This PR adds sniff_timeout and extends it to 60s.

Closes #577
Roll back the changes in continuous write
c75ea51
Add first batches to make the CI indepedent from clouds
6f0f8af
Fix lint
b9d88c5
Move to SUBSTRATE and cloud filter
c3c15bc
Rollback to build_charm
cedd262
Move away from build_charm and use the new TEST_CONSTRAINTS whenever …
036ec60 
…needed
Move the test away from build_charm
67ae9a7
Update helpers.py
1441812
Update helpers.py
69d362c
Merge branch 'DPE-6771-add-sniff-timeout' into DPE-6730-make-CI-cloud…
8c25671 
…-independent
Move scaling and upgrade tests to new CI model
fdc38fe
Moving the reminder of the CI
8a8188b
Move the charm as a generic fixture
90a8c71
Make the upgrade check for an existing opensearch
bf9e1d3
Add first trial to skip microceph testing
f8448a2
Revert "Add first trial to skip microceph testing"
b473fef 
This reverts commit f8448a2.
Merge remote-tracking branch 'origin' into DPE-6730-make-CI-cloud-ind…
d5c1118 
…ependent
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Mehdi-Bendriss Mehdi-Bendriss Awaiting requested review from Mehdi-Bendriss

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@phvalguima