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

Remove benchmark-operator's role dependencies #413

Merged
merged 2 commits into from
Jun 7, 2022
Merged

Remove benchmark-operator's role dependencies #413

merged 2 commits into from
Jun 7, 2022

Conversation

rsevilla87
Copy link
Member

Signed-off-by: Raul Sevilla rsevilla@redhat.com

Description

Depends-on: cloud-bulldozer/benchmark-operator#763

Fixes

@rsevilla87
Remove benchmark-operator's role dependencies
8eac7be 
Signed-off-by: Raul Sevilla <rsevilla@redhat.com>
dry923
dry923 approved these changes Jun 1, 2022
View reviewed changes
Copy link
Member

@dry923 dry923 left a comment

Choose a reason for hiding this comment

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

LGTM assuming the clusterAdmin role works as expected

@rsevilla87 rsevilla87 added the ok to test Kick off our CI framework label Jun 7, 2022
@rsevilla87
Standarice operator deployment
26dc933 
Signed-off-by: Raul Sevilla <rsevilla@redhat.com>
@rsevilla87
Copy link
Member Author

/rerun all

@cloud-bulldozer cloud-bulldozer deleted a comment from comet-perf-ci Jun 7, 2022
@cloud-bulldozer cloud-bulldozer deleted a comment from comet-perf-ci Jun 7, 2022
@comet-perf-ci
Copy link
Collaborator

Results for e2e-benchmarking CI Tests

Workload Test Result Runtime
router-perf-v2 ./ingress-performance.sh PASS 00:06:03
etcd-perf ./run_etcd_tests_fromgit.sh FAIL 00:10:02

@comet-perf-ci
Copy link
Collaborator

Results for e2e-benchmarking CI Tests

Workload Test Result Runtime

@rsevilla87
Copy link
Member Author

/rerun all

@comet-perf-ci
Copy link
Collaborator

Results for e2e-benchmarking CI Tests

Workload Test Result Runtime
router-perf-v2 ./ingress-performance.sh PASS 00:06:02
etcd-perf ./run_etcd_tests_fromgit.sh PASS 00:02:20
scale-perf ./run_scale_fromgit.sh PASS 00:15:11
network-perf ./run.sh PASS 00:06:47
kube-burner WORKLOAD=cluster-density ./run.sh PASS 00:05:01
kube-burner WORKLOAD=node-density ./run.sh PASS 00:05:09
kube-burner WORKLOAD=node-density-heavy ./run.sh PASS 00:06:10
kube-burner WORKLOAD=pod-density-heavy ./run.sh PASS 00:06:00
kube-burner WORKLOAD=max-services ./run.sh PASS 00:04:55
kube-burner WORKLOAD=max-namespaces ./run.sh PASS 00:05:35
upgrade-perf ./run_upgrade_fromgit.sh PASS 00:01:00

@rsevilla87 rsevilla87 merged commit 7574219 into cloud-bulldozer:master Jun 7, 2022
@rsevilla87 rsevilla87 deleted the role_dependencies branch June 7, 2022 10:09
vishnuchalla pushed a commit that referenced this pull request Sep 6, 2023
@rsevilla87
Remove benchmark-operator's role dependencies (#413)
7147869 
* Remove benchmark-operator's role dependencies

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>

* Standarice operator deployment

Signed-off-by: Raul Sevilla <rsevilla@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dry923 dry923 dry923 approved these changes

@mkarg75 mkarg75 Awaiting requested review from mkarg75

@mohit-sheth mohit-sheth Awaiting requested review from mohit-sheth

Assignees
No one assigned
Labels
ok to test Kick off our CI framework
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rsevilla87 @comet-perf-ci @dry923