Skip to content

Improve the logic for checking the network bridge configuration#894

Merged
rdmark merged 2 commits intodevelopfrom
rdmark-bridge-check-logic
Oct 5, 2022
Merged

Improve the logic for checking the network bridge configuration#894
rdmark merged 2 commits intodevelopfrom
rdmark-bridge-check-logic

Conversation

@rdmark
Copy link
Copy Markdown
Member

@rdmark rdmark commented Oct 5, 2022

  • is_bridge_configured now returns the standard pattern of dict with status and msg. A successful result now returns True instead of False.

@rdmark rdmark merged commit 52259c3 into develop Oct 5, 2022
@rdmark rdmark deleted the rdmark-bridge-check-logic branch October 5, 2022 21:14
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Oct 5, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

uweseimet added a commit that referenced this pull request Oct 6, 2022
commit a304382
Author: Uwe Seimet <48174652+uweseimet@users.noreply.github.com>
Date:   Thu Oct 6 16:15:19 2022 +0200

    Moved rascsi/rasctl specific classes to sub-folders, cleaned up code, fixed SonarCloud issues (#889)

    * Moved rasctl/rascsi core code to folders

    * Improved granularity in order to add more unit tests

    * Pointer handling update

    * Updated ID and controller handling

    * Updated memory management

    * Added unit tests

    * Fixed SonarCloud issues

commit 52259c3
Author: Daniel Markstedt <markstedt@gmail.com>
Date:   Wed Oct 5 14:14:48 2022 -0700

    Improve the logic for checking the network bridge configuration (#894)

    * Improve the logic for checking the network bridge configuration
rdmark added a commit that referenced this pull request Oct 7, 2022
* Improve the logic for checking the network bridge configuration
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