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

Add SEP TC for Monitoring #640

Merged
merged 15 commits into from
Dec 6, 2022
Merged

Add SEP TC for Monitoring #640

merged 15 commits into from
Dec 6, 2022

Conversation

mwaykole
Copy link
Member

@mwaykole mwaykole commented Dec 3, 2022

Signed-off-by: mwaykole mwaykole@redhat.com

github-advanced-security[bot]
github-advanced-security bot found potential problems Dec 3, 2022
View reviewed changes
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

Robocop found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2022 •
edited
Loading

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
275 0 0 275 100

@bdattoma bdattoma added needs testing Needs to be tested in Jenkins new test New test(s) added (PR will be listed in release-notes) ⚠️ high priority review I need this to be reviewed ASAP labels Dec 5, 2022
@mwaykole
Add SEP TC for Monitoring
a7fc850 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@mwaykole
Add SEP TC for Monitoring
077f6c9 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@mwaykole
Add SEP TC for Monitoring
ebd966f 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@mwaykole
Add SEP TC for Monitoring
2b4ef14 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@mwaykole
Add SEP TC for Monitoring
53c8e30 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@mwaykole mwaykole requested a review from bdattoma December 6, 2022 09:59
lugi0
lugi0 previously approved these changes Dec 6, 2022
View reviewed changes
@mwaykole
Add SEP TC for Monitoring
2cd9a53 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@mwaykole
Copy link
Member Author

mwaykole commented Dec 6, 2022 

==============================================================================
Run STARBURST Query And Check Values Are Not Empty :: Verifies the... | PASS |
------------------------------------------------------------------------------
Tests.Ai Apps.Starburst.Managed Starburst Monitoring :: Test suite... | PASS |
1 test, 1 passed, 0 failed
==============================================================================
Tests.Ai Apps.Starburst                                               | PASS |
1 test, 1 passed, 0 failed
==============================================================================
Tests.Ai Apps                                                         | PASS |
1 test, 1 passed, 0 failed
==============================================================================
Tests                                                                 | PASS |
1 test, 1 passed, 0 failed

@mwaykole
Add SEP TC for Monitoring
e6ee668 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@mwaykole
Add SEP TC for Monitoring
586b6a3 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@mwaykole
Add SEP TC for Monitoring
2c3777a 
Signed-off-by: mwaykole <mwaykole@redhat.com>
lugi0
lugi0 previously approved these changes Dec 6, 2022
View reviewed changes
@mwaykole
Add SEP TC for Monitoring
dcce65f 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@mwaykole
Add SEP TC for Monitoring
5d14f0b 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@mwaykole
Add SEP TC for Monitoring
e34c3dd 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@mwaykole
Add SEP TC for Monitoring
81711e2 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@mwaykole
Add SEP TC for Monitoring
9f3d167 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@mwaykole
Add SEP TC for Monitoring
38aa3b3 
Signed-off-by: mwaykole <mwaykole@redhat.com>
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 6, 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 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@bdattoma bdattoma requested a review from lugi0 December 6, 2022 15:35
@mwaykole mwaykole merged commit c477270 into red-hat-data-services:master Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lugi0 lugi0 lugi0 approved these changes

@bdattoma bdattoma bdattoma approved these changes

@pablofelix pablofelix Awaiting requested review from pablofelix

@jgarciao jgarciao Awaiting requested review from jgarciao

@tarukumar tarukumar Awaiting requested review from tarukumar

Assignees

@mwaykole mwaykole

Labels
needs testing Needs to be tested in Jenkins new test New test(s) added (PR will be listed in release-notes) ⚠️ high priority review I need this to be reviewed ASAP
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mwaykole @lugi0 @bdattoma