dynamic workloads should have selective resource creation depending on the workload enabled

Bug #1955722 reported by jaison
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
openstack-browbeat
New
Undecided
Unassigned

Bug Description

For example when only is enabled, the dynamic workload also creates extra routers and network.
The routers are created due to the following common code for all workloads.
https://github.com/cloud-bulldozer/browbeat/blob/master/rally/rally-plugins/dynamic-workloads/dynamic_workload.py#L86-L87

There are networks created due to rally_context too. These are the subnets with 172.31.xx.xx
(overcloud) [stack@undercloud ~]$ openstack subnet list
+--------------------------------------+---------------------------+--------------------------------------+----------------+
| ID | Name | Network | Subnet |
+--------------------------------------+---------------------------+--------------------------------------+----------------+
| 030846b3-6145-47ad-b7f6-50193d4a9dc9 | c_rally_7f8df1fd_vMRy10Hu | 8bc6dfa7-38f0-47f6-ad63-ff10c8c78971 | 172.31.24.0/23 |
| 0b48fd34-20ea-4eaa-b9c3-10e49dff2ab8 | s_rally_7f8df1fd_Gd9mKkGh | 8ded706a-533a-4e7c-bab8-313c7f1bd025 | 10.2.17.0/24 |
| 0ca93c4b-ee49-48c5-ab60-63f5704f2d0e | c_rally_7f8df1fd_08K697ik | 3e810d04-9c58-4486-961d-c67a51ad0041 | 172.31.28.0/23 |
| 0de9ea4a-3c59-4cba-b447-598d53de6356 | c_rally_7f8df1fd_P7VCsC5r | 0817536f-64c5-492e-8edc-67c28ad275f6 | 172.31.0.0/23 |
| 0fda87e6-cc23-4ed8-85b2-f0f1dea509bb | c_rally_7f8df1fd_53RT9V1i | 83a3f6f8-964e-4297-b0f2-f28e70e6cbdc | 172.31.20.0/23 |

We need to have another review of the code to understand how we can selectively create resources so that they dont get created for workloads for which its not required.

Revision history for this message
Sanjay Chari (sanjaychari) wrote :

I created a patch to fix the router issue.
https://review.opendev.org/c/x/browbeat/+/823124

Revision history for this message
Sanjay Chari (sanjaychari) wrote :

Changes have been to the same patch to fix the external networks issue as well.

Revision history for this message
Sanjay Chari (sanjaychari) wrote :

The patch that fixes this issue(mentioned in the first comment) has been merged.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.