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.

Sign up for GitHub

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

Jump to bottom

Various CI and CLI improvements #1554

Merged
merged 5 commits into from
Dec 2, 2023
Merged

Various CI and CLI improvements #1554

merged 5 commits into from
Dec 2, 2023

Conversation

tiagolobocastro
Copy link
Contributor

@tiagolobocastro tiagolobocastro commented Nov 30, 2023 •
edited
Loading 

test(python/bdd): cleanup when terminated

Sometimes when testing we want to ^C and run something else, but this leaves
cruft behind.
In such case, we should cleanup to ensure environment is clean.

Signed-off-by: Tiago Castro <[email protected]>

ci(python): add junit reports

Signed-off-by: Tiago Castro <[email protected]>

chore: update dependencies repo

Fixes missing test output on CI failures.

Signed-off-by: Tiago Castro <[email protected]>

test(python): enhance python bdd testing script

Allows specification of a test list
Allows specification of a specific scenario
Adds cleanup args:
--clean-all: cleans all docker-compose clusters
--clean-all-exit: clean all and exit
Adds cleanup on a per-test basis

Signed-off-by: Tiago Castro <[email protected]>

fix(io-engine/cli): fix controller stats

Human output parameters were setup incorrectly..

Signed-off-by: Tiago Castro <[email protected]>

@tiagolobocastro
fix(io-engine/cli): fix controller stats
f2e2035 
Human output parameters were setup incorrectly..

Signed-off-by: Tiago Castro <[email protected]>
@auto-assign auto-assign bot requested a review from dsavitskiy November 30, 2023 19:03
@tiagolobocastro
Copy link
Contributor Author

bors try

bors-openebs-mayastor bot pushed a commit that referenced this pull request Nov 30, 2023
Try #1554:
a27c8ae
@bors-openebs-mayastor
Copy link

try

Build failed:

@tiagolobocastro
Copy link
Contributor Author

bors try

bors-openebs-mayastor bot pushed a commit that referenced this pull request Nov 30, 2023
Try #1554:
f350670
@bors-openebs-mayastor
Copy link

try

Build succeeded:

@tiagolobocastro
test(python): enhance python bdd testing script
3dd497d 
Allows specification of a test list
Allows specification of a specific scenario
Adds cleanup args:
--clean-all: cleans all docker-compose clusters
--clean-all-exit: clean all and exit
Adds cleanup on a per-test basis

Signed-off-by: Tiago Castro <[email protected]>
@tiagolobocastro
chore: update dependencies repo
55d928f 
Fixes missing test output on CI failures.

Signed-off-by: Tiago Castro <[email protected]>
@tiagolobocastro
Copy link
Contributor Author

bors try

bors-openebs-mayastor bot pushed a commit that referenced this pull request Dec 1, 2023
Try #1554:
a6a8a35
@tiagolobocastro
Copy link
Contributor Author

bors cancel

@tiagolobocastro
Copy link
Contributor Author

bors try

@bors-openebs-mayastor
Copy link

try

Already running a review

@tiagolobocastro
Copy link
Contributor Author

bors try-

@tiagolobocastro
Copy link
Contributor Author

bors cancel

@tiagolobocastro
Copy link
Contributor Author

bors ping

@bors-openebs-mayastor
Copy link

pong

@tiagolobocastro
Copy link
Contributor Author

bors try

bors-openebs-mayastor bot pushed a commit that referenced this pull request Dec 1, 2023
Try #1554:
7b0d776
@bors-openebs-mayastor
Copy link

try

Timed out.

@tiagolobocastro tiagolobocastro force-pushed the fix-cli branch 10 times, most recently from 5e92390 to 5f065f0 Compare December 1, 2023 23:55
@tiagolobocastro
ci(python): add junit reports
1b95058 
Signed-off-by: Tiago Castro <[email protected]>
@tiagolobocastro
test(python/bdd): cleanup when terminated
2732e46 
Sometimes when testing we want to ^C and run something else, but this leaves
cruft behind.
In such case, we should cleanup to ensure environment is clean.

Signed-off-by: Tiago Castro <[email protected]>
@tiagolobocastro
Copy link
Contributor Author

bors merge

@bors-openebs-mayastor
Copy link

Build succeeded:

@bors-openebs-mayastor bors-openebs-mayastor bot merged commit 6c12c66 into develop Dec 2, 2023
3 checks passed
@bors-openebs-mayastor bors-openebs-mayastor bot deleted the fix-cli branch December 2, 2023 01:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abhilashshetty04 abhilashshetty04 abhilashshetty04 approved these changes

@Abhinandan-Purkait Abhinandan-Purkait Abhinandan-Purkait approved these changes

@hrudaya21 hrudaya21 hrudaya21 approved these changes

@dsharma-dc dsharma-dc Awaiting requested review from dsharma-dc

@dsavitskiy dsavitskiy Awaiting requested review from dsavitskiy

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tiagolobocastro @hrudaya21 @Abhinandan-Purkait @abhilashshetty04