| *** undefined is now known as Guest5598 | 03:35 | |
| opendevreview | Yusuke Niimi proposed openstack/keystoneauth master: Add doc of OAuth2.0 Client Credentials Grant Flow https://review.opendev.org/c/openstack/keystoneauth/+/838104 | 05:20 |
|---|---|---|
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Remove deprecated stuff from baremetal_node https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849668 | 06:47 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Add a wait argument to baremetal_node https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849653 | 06:47 |
| opendevreview | Merged openstack/ansible-collections-openstack master: Add a wait argument to baremetal_node https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849653 | 09:08 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored port and port_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/850627 | 10:22 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored server and server_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849423 | 10:24 |
| opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves stack module from cloud to proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/848266 | 11:28 |
| *** Guest5598 is now known as rcastillo | 13:05 | |
| opendevreview | Jakob Meng proposed openstack/openstacksdk master: Allow to pass description parameter to cloud.create_server() https://review.opendev.org/c/openstack/openstacksdk/+/850671 | 15:49 |
| jm1 | gtema: hello :) is there a reason why cloud.create_server does not accept a description parameter? https://review.opendev.org/c/openstack/openstacksdk/+/850671 | 15:50 |
| gtema | historical - not implemented in cloud layer | 15:50 |
| jm1 | gtema: so we can add it? | 15:52 |
| gtema | sure | 15:52 |
| jm1 | gtema: cool, then we can continue to use create_server in openstack.cloud.server. would require a lot of new code in that module without cloud.create_server() | 15:53 |
| gtema | for server sadly you NEED to continue using cloud layer, since there are too many things happening under the hood | 15:53 |
| gtema | but glad you reached it now ;-) | 15:54 |
| jm1 | gtema: adding more tests for server revealed more issues 🙈 like the one above where i would like to pass description but create_server does not accept that.. | 15:57 |
| gtema | yeah, and we also have open issue about volume | 15:57 |
| gtema | (in the server create) | 15:57 |
| jm1 | gtema: refering to any bug report? maybe this one? https://storyboard.openstack.org/#!/story/2010103 | 16:02 |
| gtema | yes, this one | 16:03 |
| jm1 | gtema: yeah, working on this one | 16:14 |
| gtema | awesome | 16:14 |
| jm1 | gtema: servers will always be assigned a security group by nova even if no security group was requested? | 16:22 |
| gtema | hmm, not sure | 16:23 |
| gtema | from docs: | 16:24 |
| gtema | One or more security groups. Specify the name of the security group in the name attribute. If you omit this attribute, the API creates the server in the default security group. Requested security groups are not applied to pre-existing ports. | 16:24 |
| gtema | whether this "default" SG is configurable I do not know | 16:24 |
| jm1 | gtema: omg, i should have rtfm 🙈 thanks! | 17:03 |
| opendevreview | Merged openstack/openstacksdk master: Allow to pass description parameter to cloud.create_server() https://review.opendev.org/c/openstack/openstacksdk/+/850671 | 18:58 |
| jm1 | gtema: still working? | 19:10 |
| jm1 | gtema: compute.add_security_group_to_server() claims it takes an id as security_group. https://opendev.org/openstack/openstacksdk/src/commit/1ce15c9a8758b4d978eb5239bae100ddc13c8875/openstack/compute/v2/_proxy.py#L1028 | 19:11 |
| jm1 | gtema: doing so results in BadRequestException: 400: Client Error for url: http://***/compute/v2.1/servers/***/action, Security group name cannot be empty | 19:12 |
| jm1 | gtema: passing a SecurityGroup object instead works as expected | 19:12 |
| jm1 | gtema: another issue is, after calling compute.fetch_server_security_groups one cannot call compute.update_server with this server object because raises "BadRequestException: 400: Client Error for url:... Invalid input for field/attribute server. Value: ... Additional properties are not allowed ('security_groups' was unexpected)" | 19:18 |
| opendevreview | Violet Kurtz proposed openstack/python-openstackclient master: Migrate server_groups to the new API https://review.opendev.org/c/openstack/python-openstackclient/+/837615 | 19:48 |
| opendevreview | Violet Kurtz proposed openstack/python-openstackclient master: Migrate server_groups to the new API https://review.opendev.org/c/openstack/python-openstackclient/+/837615 | 19:54 |
| opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update router for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/846446 | 20:46 |
| opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update router for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/846446 | 21:18 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!