Skip to content

HDDS-9341. om-roles.robot is not part of any test runs#5352

Merged
adoroszlai merged 1 commit intoapache:masterfrom
LZD-PratyushBhatt:HDDS-9341
Sep 23, 2023
Merged

HDDS-9341. om-roles.robot is not part of any test runs#5352
adoroszlai merged 1 commit intoapache:masterfrom
LZD-PratyushBhatt:HDDS-9341

Conversation

@LZD-PratyushBhatt
Copy link
Contributor

What changes were proposed in this pull request?

The om-roles.robot test added in HDDS-9305 is not executed in any of the acceptance test. Aim is to add it in ozone-ha and ozonesecure-ha environments.

What is the link to the Apache JIRA

HDDS-9341

How was this patch tested?

Acceptance test CI, passing in local fork.
ha-secure: https://github.com/LZD-PratyushBhatt/ozone/actions/runs/6277586606/job/17050342303

Om-Roles :: Smoke test for listing om roles.                                  
==============================================================================
List om roles with OM service ID passed                               | PASS |
------------------------------------------------------------------------------
List om roles without OM service ID passed                            | PASS |
------------------------------------------------------------------------------
List om roles as JSON with OM service ID passed                       | PASS |
------------------------------------------------------------------------------
List om roles as JSON without OM service ID passed                    | PASS |
------------------------------------------------------------------------------
Om-Roles :: Smoke test for listing om roles.                          | PASS |
4 tests, 4 passed, 0 failed

ha-unsecure: https://github.com/LZD-PratyushBhatt/ozone/actions/runs/6277586606/job/17050342650

Om-Roles :: Smoke test for listing om roles.                                  
==============================================================================
List om roles with OM service ID passed                               | PASS |
------------------------------------------------------------------------------
List om roles without OM service ID passed                            | PASS |
------------------------------------------------------------------------------
List om roles as JSON with OM service ID passed                       | PASS |
------------------------------------------------------------------------------
List om roles as JSON without OM service ID passed                    | PASS |
------------------------------------------------------------------------------
Om-Roles :: Smoke test for listing om roles.                          | PASS |
4 tests, 4 passed, 0 failed
==============================================================================
Output:  /tmp/smoketest/ozone-ha/result/robot-7.xml

@LZD-PratyushBhatt
Copy link
Contributor Author

Hi @adoroszlai , can you please review this? Thanks!

Copy link
Contributor

@adoroszlai adoroszlai left a comment

Choose a reason for hiding this comment

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

Thanks @LZD-PratyushBhatt for the patch.

@adoroszlai adoroszlai merged commit 263cd46 into apache:master Sep 23, 2023
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