Skip to content

Conversation

nbttmbrg
Copy link

Fixing #38

Using passed_sid to filter the list returned by sap_system_facts and select only the dict with SID=passed_sid

@nbttmbrg nbttmbrg force-pushed the bugfix/passed_sid_is_ignored branch from 7d2abf7 to aa389af Compare July 21, 2025 10:59
@rhmk
Copy link
Member

rhmk commented Jul 22, 2025

Some merge conflicts may occur here. Sorry

@nbttmbrg
Copy link
Author

Some merge conflicts may occur here. Sorry

I will need to rebase and test, but it should be just a matter of shuffling the tasks arround that new list and change one variable name.

Thank you

nbttmbrg added 2 commits July 23, 2025 10:38
Signed-off-by: Nicolas Bettembourg <nbettembourg@oxya.com>
Signed-off-by: Nicolas Bettembourg <nbettembourg@oxya.com>
@nbttmbrg nbttmbrg force-pushed the bugfix/passed_sid_is_ignored branch from aa389af to 25ec333 Compare July 23, 2025 08:39
@sean-freeman sean-freeman changed the title Bugfix/passed sid is ignored sap_control: Bugfix/passed sid is ignored Aug 4, 2025
Copy link

@marcelmamula marcelmamula left a comment

Choose a reason for hiding this comment

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

LGTM in current state of role.
This will be completely replaced when we get to reworking role to raise up to standard of other code in our project.

@crweller
Copy link
Contributor

When I get a hold on thus bugfix, I will double check for potential conflicts or enhancements.

@marcelmamula marcelmamula merged commit b177316 into sap-linuxlab:dev Aug 21, 2025
1 of 2 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.

4 participants