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

[full-ci][tests-only]Add support to stop and start service in ociswrapper #10776

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

[full-ci][tests-only]Add support to stop and start service in ociswrapper #10776

wants to merge 5 commits into from
+264 −102

Conversation

amrita-shrestha
Copy link
Contributor

@amrita-shrestha amrita-shrestha commented Dec 17, 2024 •
edited
Loading

Description

This adds new feature on ocis wrapper to stop/ start sevices.

Test cases

  • start ocis
  • restart ocis with different env
  • stop ocis
  • rollback ocis (check env is set to default or not)
  • start service with different env
  • stop service

Related Issue

Motivation and Context

How Has This Been Tested?

  • locally

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Technical debt
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation ticket raised:

@amrita-shrestha amrita-shrestha self-assigned this Dec 17, 2024
@amrita-shrestha amrita-shrestha force-pushed the add-support-to-stop-and-start-service-in-ociswrapper branch 3 times, most recently from 2125c45 to b3c2e20 Compare December 18, 2024 11:37
@amrita-shrestha
Copy link
Contributor Author

amrita-shrestha commented Dec 30, 2024 •
edited
Loading

waiting for #10808 to get merged

@amrita-shrestha amrita-shrestha force-pushed the add-support-to-stop-and-start-service-in-ociswrapper branch 5 times, most recently from cc95565 to 5f509bd Compare January 7, 2025 16:37
@amrita-shrestha amrita-shrestha mentioned this pull request Jan 8, 2025
Open
2 tasks
@amrita-shrestha amrita-shrestha force-pushed the add-support-to-stop-and-start-service-in-ociswrapper branch 3 times, most recently from 43f62bd to 27074ac Compare January 9, 2025 06:37
@amrita-shrestha amrita-shrestha marked this pull request as ready for review January 9, 2025 10:12
saw-jan
saw-jan requested changes Jan 10, 2025
View reviewed changes
tests/ociswrapper/wrapper/wrapper.go Outdated Show resolved Hide resolved
tests/ociswrapper/wrapper/handlers/handler.go Outdated Show resolved Hide resolved
tests/ociswrapper/wrapper/handlers/handler.go Outdated Show resolved Hide resolved
@amrita-shrestha amrita-shrestha force-pushed the add-support-to-stop-and-start-service-in-ociswrapper branch from df7c67f to 296bdb6 Compare January 13, 2025 08:20
PrajwolAmatya
PrajwolAmatya reviewed Jan 13, 2025
View reviewed changes
tests/acceptance/bootstrap/OcisConfigContext.php Outdated Show resolved Hide resolved
tests/acceptance/bootstrap/OcisConfigContext.php Outdated Show resolved Hide resolved
tests/ociswrapper/ocis/ocis.go Outdated Show resolved Hide resolved
@amrita-shrestha amrita-shrestha force-pushed the add-support-to-stop-and-start-service-in-ociswrapper branch from 296bdb6 to e78d3cc Compare January 13, 2025 11:42
@amrita-shrestha amrita-shrestha force-pushed the add-support-to-stop-and-start-service-in-ociswrapper branch from e78d3cc to d7778d4 Compare January 13, 2025 11:47
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@S-Panta S-Panta Awaiting requested review from S-Panta

@saw-jan saw-jan Awaiting requested review from saw-jan

@nirajacharya2 nirajacharya2 Awaiting requested review from nirajacharya2

@PrajwolAmatya PrajwolAmatya Awaiting requested review from PrajwolAmatya

@ishabaral ishabaral Awaiting requested review from ishabaral

@Salipa-Gurung Salipa-Gurung Awaiting requested review from Salipa-Gurung

@nabim777 nabim777 Awaiting requested review from nabim777

@anon-pradip anon-pradip Awaiting requested review from anon-pradip

@prashant-gurung899 prashant-gurung899 Awaiting requested review from prashant-gurung899

Requested changes must be addressed to merge this pull request.

Assignees

@amrita-shrestha amrita-shrestha

Labels
QA:team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@amrita-shrestha @saw-jan @PrajwolAmatya @nirajacharya2