-
Notifications
You must be signed in to change notification settings - Fork 79
[KOGITO-850] Add Cucumber test for Prometheus integration #135
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, one minor suggestion.
BTW, why are we not opening JIRAs for those PRs? Could you please open it so we can track this great work you guys are doing?
test/smoke/openshift.go
Outdated
@@ -133,3 +156,34 @@ func waitAndRetrieveRouteURI(namespace, serviceName string) (string, error) { | |||
GetLogger(namespace).Debugf("Got route %s\n", routeURI) | |||
return routeURI, nil | |||
} | |||
|
|||
// waitForPods waits for pods with specific label to be available and running | |||
func waitForPods(namespace, labelName, labelValue string, numberOfPods, timeoutInMin int) error { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since those functions are related to Pods
resource, why not keep them on kubernetes.go
instead?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good point, can do that in next PR, can move there also route functions
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in this PR.
@ricardozanini Good point, added JIRA for this test. |
Signed-off-by: Karel Suta <ksuta@redhat.com>
@ricardozanini Can you please take a look again? |
https://issues.redhat.com/browse/KOGITO-850
Signed-off-by: Karel Suta ksuta@redhat.com
Many thanks for submiting your Pull Request ❤️!
Please make sure that your PR meets the following requirements:
[KOGITO-XYZ] Subject