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

Add GetDefaultTarget and SetDefaultTarget APIs #951

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

ArtiomDivak
Copy link
Contributor

@ArtiomDivak ArtiomDivak commented Sep 26, 2024

  • Adds support for GetDefaultTarget and SetDefaultTarget APIs on controller and node
  • Adds get-default and set-default commands to bluechictl
  • Adds integration tests for both APIs

Fixes: #944
Signed-off-by: Artiom Divak [email protected]

@ArtiomDivak ArtiomDivak force-pushed the set-get-default-target branch 4 times, most recently from 1327559 to d1c0dab Compare September 29, 2024 06:18
@coveralls
Copy link

coveralls commented Sep 29, 2024

Coverage Status

coverage: 83.429% (-0.1%) from 83.538%
when pulling d974d1d on ArtiomDivak:set-get-default-target
into bbc9ae4 on eclipse-bluechi:main.

@ArtiomDivak ArtiomDivak force-pushed the set-get-default-target branch 3 times, most recently from d7b39b3 to ef50d6d Compare October 1, 2024 13:44
@ArtiomDivak ArtiomDivak marked this pull request as ready for review October 1, 2024 13:44
Copy link
Member

@mwperina mwperina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generally it looks great, some small nits posted

tests/bluechi_test/bluechictl.py Outdated Show resolved Hide resolved
tests/bluechi_test/bluechictl.py Outdated Show resolved Hide resolved
tests/tests/tier0/bluechi-get-default-target/main.fmf Outdated Show resolved Hide resolved
tests/tests/tier0/bluechi-set-default-target/main.fmf Outdated Show resolved Hide resolved
src/client/method-get-default-target.c Outdated Show resolved Hide resolved
@mwperina mwperina changed the title Get Default Target Add GetDefaultTarget and SetDefaultTarget APIs Oct 2, 2024
@mwperina
Copy link
Member

mwperina commented Oct 2, 2024

Also please don't forget to update bluechictl man page: https://github.com/eclipse-bluechi/bluechi/blob/main/doc/man/bluechictl.1.md

@ArtiomDivak ArtiomDivak force-pushed the set-get-default-target branch 4 times, most recently from 2e008ad to 4b970c2 Compare October 3, 2024 09:02
src/client/method-help.c Outdated Show resolved Hide resolved
@ArtiomDivak ArtiomDivak force-pushed the set-get-default-target branch 2 times, most recently from 68a974d to b7f51c8 Compare October 3, 2024 12:25
src/client/method-default-target.c Outdated Show resolved Hide resolved
doc/man/bluechictl.1.md Outdated Show resolved Hide resolved
src/client/method-default-target.c Outdated Show resolved Hide resolved
tests/bluechi_test/bluechictl.py Outdated Show resolved Hide resolved
@ArtiomDivak ArtiomDivak force-pushed the set-get-default-target branch 5 times, most recently from 863766d to 607f4a5 Compare October 6, 2024 07:20
@ArtiomDivak ArtiomDivak force-pushed the set-get-default-target branch 2 times, most recently from be1c18a to a39d66e Compare October 7, 2024 06:51
tests/bluechi_test/systemctl.py Outdated Show resolved Hide resolved
tests/bluechi_test/systemctl.py Outdated Show resolved Hide resolved
tests/bluechi_test/systemctl.py Outdated Show resolved Hide resolved
tests/bluechi_test/systemctl.py Outdated Show resolved Hide resolved
tests/bluechi_test/systemctl.py Show resolved Hide resolved
src/client/main.c Show resolved Hide resolved
@ArtiomDivak ArtiomDivak force-pushed the set-get-default-target branch 6 times, most recently from d4967b4 to f6bbdb6 Compare October 7, 2024 08:43
This feature will get and set the default target for a specific node.

Solves eclipse-bluechi#944
Signed-off-by: Artiom Divak <[email protected]>
Copy link
Member

@mwperina mwperina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@mkemel mkemel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mwperina mwperina merged commit 8c96f7b into eclipse-bluechi:main Oct 7, 2024
22 checks passed
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.

RFE: Add GetDefaultTarget and SetDefaultTarget API
4 participants