| gthiemon1e | skraynev_: oh, i don't remember filing this bug :D IMHO the API should return an clear explanation ("..already allocated.." instead of just "conflict") | 07:00 |
|---|---|---|
| gthiemon1e | skraynev_: I guess the first raised exception is not propagated correctly (msg lost at one point) | 07:01 |
| skraynev_ | gthiemon1e: I recently faced with this bug and a bit confused. From one side: it could be handled with correct error message (instead of 500), but also it could be validated on creation and it fails before request to neutron (but in this case it's necessary to take all ports from subnet and check that ip is not busy) | 07:03 |
| priteau | Hello Octavia team. May I ask for a review of this simple doc fix? https://review.opendev.org/c/openstack/octavia/+/937766 | 10:19 |
| gthiemon1e | priteau: done, thanks! | 10:43 |
| priteau | gthiemon1e: I agree with your statement about confusion of role and and `role:foo` syntax, should I switch e.g. role:reader to reader as well? | 10:45 |
| gthiemon1e | priteau: no we can leave it for now, johnsom has a huge patch that changes the default roles in Octavia (https://review.opendev.org/c/openstack/octavia/+/929580) we may have to revisit the doc | 10:49 |
| priteau | OK | 10:51 |
| priteau | Thanks for the review | 10:51 |
| *** tkajinam is now known as Guest9449 | 13:10 | |
| -opendevstatus- NOTICE: nominations for the OpenStack PTL and TC positions are closing soon, for details see https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/7DKEV7IEHOTHED7RVEFG7WIDVUC4MY3Z/ | 15:59 | |
| opendevreview | Merged openstack/octavia-tempest-plugin master: Use the correct parameters for role assignements list https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/941928 | 17:58 |
| opendevreview | Merged openstack/octavia master: Make keystone default roles the default RBAC https://review.opendev.org/c/openstack/octavia/+/929580 | 17:58 |
| opendevreview | Merged openstack/octavia-tempest-plugin master: Remove system scope token usage https://review.opendev.org/c/openstack/octavia-tempest-plugin/+/926867 | 21:08 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!