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

[DPE-2454] Move ci.yaml scripts to tox file #43

Open
wants to merge 52 commits into
base: main
Choose a base branch
from
Open

Conversation

phvalguima
Copy link
Contributor

This PR moves the ci.yaml scripts used to validate the snap to a tox format.
It creates a new folder, tests/, that contains all the tests involved in ci.yaml.

* Validated snap upgrade and added CI coverage
* Updated documentation and checks on ops/start.sh for process-control interface
* Added a post-stop-command to remove the pid file
…users can manage their own plugins; move lib/ to SNAP_DATA so CLASSPATH is automatically set
… check for correct filepath in add-file; README: documented new commands
* Removed plugin/keystore special scripts: they are executed as any other opensearch CLI command;
* Removed pre-/post-refresh as the idea is to keep dependencies between snap version upgrades;
* Moved from SNAP_DATA to /var/snap/opensearch/current: no need to edit config files between upgrades anymore;
* Updated tests
phvalguima and others added 22 commits August 4, 2023 11:34
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