Skip to content

Commit

Permalink
Bugfix: work around bug in openstacksdk > 3.1.0 (#694)
Browse files Browse the repository at this point in the history
Signed-off-by: Matthias Büchse <[email protected]>
  • Loading branch information
mbuechse authored Aug 14, 2024
1 parent ba93934 commit 923815c
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 3 deletions.
7 changes: 6 additions & 1 deletion Tests/iaas/entropy/entropy-check.py
Original file line number Diff line number Diff line change
Expand Up @@ -334,7 +334,12 @@ def create_vm(env, all_flavors, image, server_name=SERVER_NAME):
boot_from_volume=True, terminate_volume=True, volume_size=volume_size,
)
logger.debug(f"Server '{server_name}' ('{server.id}') has been created")
return server
# next, do an explicit get_server because, beginning with version 3.2.0, the openstacksdk no longer
# sets the interface attributes such as `public_v4`
# I (mbuechse) consider this a bug in openstacksdk; it was introduced with
# https://opendev.org/openstack/openstacksdk/commit/a8adbadf0c4cdf1539019177fb1be08e04d98e82
# I also consider openstacksdk architecture with the Mixins etc. smelly to say the least
return env.conn.get_server(server.id)


def delete_vm(conn, server_name=SERVER_NAME):
Expand Down
2 changes: 1 addition & 1 deletion Tests/requirements.in
Original file line number Diff line number Diff line change
Expand Up @@ -4,5 +4,5 @@ fabric
kubernetes_asyncio
python-dateutil
PyYAML
openstacksdk < 3.2.0
openstacksdk
requests
2 changes: 1 addition & 1 deletion Tests/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ multidict==6.0.5
# yarl
netifaces==0.11.0
# via openstacksdk
openstacksdk==3.1.0
openstacksdk==3.3.0
# via -r requirements.in
os-service-types==1.7.0
# via
Expand Down

0 comments on commit 923815c

Please sign in to comment.