| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-06-06 | |||
| 18:48:50 | efried | oh, never mind | |
| 18:49:03 | efried | was gonna say, nova will be well-behaved and only use Connection for things it's supposed to. | |
| 18:49:10 | efried | But that doesn't help the case where conf is misconfigured | |
| 18:49:25 | efried | though that's a pretty cornery corner. | |
| 18:50:05 | efried | Because we still have to pass in the session, which only exists if we had ksa session opts, which we pull in at the same time as adapter opts. So the only busted path is interface+valid_interfaces I think. | |
| 18:54:43 | mordred | nod | |
| 19:05:46 | openstackgerrit | Merged openstack/python-openstackclient master: Deprecate openstack server migrate --host option https://review.opendev.org/659382 | |
| 20:50:20 | openstackgerrit | Merged openstack/openstacksdk master: Handle oslo.config exceptions in from_conf https://review.opendev.org/663439 | |
| 20:50:22 | openstackgerrit | Merged openstack/openstacksdk master: Add release note for new disable service functionality https://review.opendev.org/663690 | |
| 20:50:23 | openstackgerrit | Merged openstack/openstacksdk master: Minor fixups from from_conf changes https://review.opendev.org/663685 | |
| 20:57:34 | openstackgerrit | Eric Fried proposed openstack/openstacksdk master: Pin to latest os-service-types https://review.opendev.org/663733 | |
| 20:57:43 | efried | mordred: resolved merge conflict ^ | |
| 20:58:22 | mordred | efried: lookie there though - the other patches merged! | |
| 20:58:27 | efried | yup! | |
| 20:58:38 | efried | could get a new release any time now | |
| 20:58:58 | efried | though I'm still waiting for test results on the nova side - we might want to include fixes if those fail | |
| 22:28:07 | mordred | efried: ++ | |
| 23:19:13 | openstackgerrit | Merged openstack/python-openstackclient master: Batch up minor cleanups for release https://review.opendev.org/659982 | |
| #openstack-sdks - 2019-06-07 | |||
| 00:02:52 | openstackgerrit | Merged openstack/openstacksdk master: Pin to latest os-service-types https://review.opendev.org/663733 | |
| 03:49:29 | shubham_potale | Hi all | |
| 03:49:32 | shubham_potale | I have one question regarding the url formation for the APIs like tacker, neutron apis for example openstack --debug network list the url for the above command is http://10.232.48.200:9696/v2.0/networks in above url where that v2.0 is appended is not clear to me. If we see openstack endpoint list we get network base url as http://10.232.48.200:9696/ for this part (/v2.0/networks) i found that /networks is app | |
| 03:50:41 | shubham_potale | if anyone knows about it, please let me know | |
| 04:40:28 | efried | mordred: Nova side looks good (except I needed to bump the ost l-c there too, which seems weird) so for my money we can cut a release any time. LMK if you want me to do the paperwork. | |
| 06:44:50 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add access alias (aka) for the resource attributes https://review.opendev.org/663846 | |
| 06:48:42 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Use Resource layer for next compute methods https://review.opendev.org/663064 | |
| 06:50:16 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Use resource layer for compute flavors https://review.opendev.org/650903 | |
| 09:27:53 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: raise more specific ResourceFailure in wait_for_* methods https://review.opendev.org/663867 | |
| 11:03:41 | dtantsur | folks, a trivial fix ^^^ when you have some time | |
| 11:10:07 | mordred | gtema: ^^ have a sec to look at that patch from dtantsur ? also - nice aka patch | |
| 11:10:41 | gtema | yeah - it found actual problem in the image checksum validation - hehe | |
| 11:13:17 | mordred | \o/ | |
| 11:37:00 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add access alias (aka) for the resource attributes https://review.opendev.org/663846 | |
| 11:55:32 | mordred | gtema: if that passes tests, that looks pretty nice! | |
| 11:55:53 | gtema | thanks | |
| 11:56:13 | gtema | some time later probably rename "alias" to "alternate_attribute" | |
| 11:57:44 | gtema | it actually proved, that in the case of image the checksum validation would have never worked really | |
| 11:58:07 | gtema | since after fetch the checksum was in reality empty | |
| 11:59:44 | gtema | what's happening - already second change fails due to timeout in nodepool job | |
| 12:18:19 | mordred | uhoh | |
| 12:35:05 | openstackgerrit | Yves-Gwenael Bourhis proposed openstack/shade master: URL encode swift objects endpoints https://review.opendev.org/663922 | |
| 12:39:34 | openstackgerrit | Yves-Gwenael Bourhis proposed openstack/openstacksdk master: URL encode swift objects endpoints https://review.opendev.org/663925 | |
| 12:56:11 | openstackgerrit | Merged openstack/openstacksdk master: baremetal: raise more specific ResourceFailure in wait_for_* methods https://review.opendev.org/663867 | |
| 13:14:30 | efried | mordred: We green for a release of sdk? | |
| 13:16:14 | mordred | efried: yup! | |
| 13:16:21 | efried | mordred: I'm proposing, stand by. | |
| 13:17:22 | mordred | remote: https://review.opendev.org/663939 Release 0.31.0 of openstacksdk | |
| 13:17:34 | mordred | efried: oh - crappit - didn't see that last one from you | |
| 13:17:38 | efried | mordred: https://review.opendev.org/663940 | |
| 13:17:59 | efried | dangit, I woulda beat you if I hadn't sat there staring at the commit message. | |
| 13:18:08 | mordred | efried: I'll ack yours and abandon mine | |
| 13:18:13 | efried | either way | |
| 13:18:13 | mordred | your commit message is better | |
| 13:18:37 | efried | :) | |
| 13:25:47 | openstackgerrit | Merged openstack/openstacksdk master: Use Resource layer for compute AZ and Aggregates https://review.opendev.org/662738 | |
| 14:53:19 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add access alias (aka) for the resource attributes https://review.opendev.org/663846 | |
| 15:09:37 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Rework statistics reporting https://review.opendev.org/659841 | |
| 17:27:43 | openstackgerrit | Merged openstack/keystoneauth master: add a handler for unknown HTTP errors https://review.opendev.org/663132 | |
| 18:13:11 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Rework statistics reporting https://review.opendev.org/659841 | |
| 18:21:38 | gtema | mordred: still here? | |
| 20:09:59 | efried | mordred: FYI: https://review.opendev.org/656023 (green CI) is using 0.30.0 | |
| 20:18:00 | mordred | efried: \o/ | |
| #openstack-sdks - 2019-06-08 | |||
| 06:31:14 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add access alias (aka) for the resource attributes https://review.opendev.org/663846 | |
| 08:46:33 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Add access alias (aka) for the resource attributes https://review.opendev.org/663846 | |
| 08:48:16 | mordred | gtema: just a bad test? | |
| 08:48:24 | gtema | yes | |
| 08:48:52 | mordred | gtema: it's a truly lovely bug you found :) | |
| 08:49:09 | gtema | I replaced test_create_image_put_v2_wrong_checksum_delete with the test_create_image_put_v2_checksums I added in PS4 | |
| 08:49:10 | gtema | sure | |
| 08:49:19 | gtema | I was like crazy debugging nodepool logs | |
| 08:49:32 | gtema | was not able to sleep pulling out my hair | |
| 08:49:38 | gtema | :D | |
| 08:50:14 | gtema | and that's all just from one line changed in resource.py | |
| 08:50:37 | gtema | namely line :1105 | |
| 08:51:30 | gtema | with the flavors patch - nodepool seems not to like it either | |
| 08:51:42 | gtema | but I had no time to look in detail | |
| 08:52:01 | mordred | gtema: nodepool is here to keep you from sleeping ;) | |
| 08:52:10 | gtema | hehe | |
| 08:53:08 | gtema | and working on Saturday is also fun | |
| 08:53:21 | gtema | unless wife starts screaming ;-) | |
| 08:53:27 | mordred | so is waking up at 0300 for no good reason :) | |
| 08:53:43 | gtema | sure | |
| 10:22:44 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Use resource layer for compute flavors https://review.opendev.org/650903 | |
| #openstack-sdks - 2019-06-10 | |||
| 05:38:23 | shubham_potale | Hi All, | |
| 05:38:38 | shubham_potale | I have one question regarding the url formation of the APIs like tacker, neutron | |
| 05:40:02 | shubham_potale | for ex. for the command openstack --debug vnf list, url is GET http://10.232.48.200:9890/v1.0/vnfs in this url for this part (v1.0/vnfs) /vnfs is appened in tackerclient but where v1.0 is appended is not clear to me. if we see openstack endponit list base url for tacker is http://10.232.48.200:9890/. if tou know about it let me know | |
| 15:29:34 | stephenfin | dtroyer: Any reason we use 'openstack server resize --revert' and not 'openstack server resize revert', seeing as "resize revert" feels like an action? | |
| 16:09:52 | efried | mordred: https://review.opendev.org/#/c/664381/ bumps u-c for openstacksdk to 0.31.0, for your acking pleasure | |
| 16:14:54 | efried | oh, core in requirements are we? Even better. | |
| 16:26:57 | openstackgerrit | Stephen Finucane proposed openstack/python-openstackclient master: Add 'openstack server migrate [--resize|--confirm]' flags https://review.opendev.org/664393 | |
| 17:41:30 | mnaser | ok, i'm not the only one ever to deal with this but | |
| 17:41:41 | mnaser | is there any reason why we couldnt add a cascade delete option to openstackclient | |
| 17:55:55 | dtroyer | stephenfin: I didn't do that because I didn't think about it. set/unset is a natural opposite pair, revert by itself might be too generic but we certainly could have a two word verb just as much as we have multi-word resources | |
| 17:56:28 | dtroyer | modulo if I was the one who originally mapped that command, I honestly don't remember, so if its wrong I prolly did :) | |
| 17:57:15 | dtroyer | mnaser: is that different from the purge command? I assume so but am not sure exactly how… | |
| 17:57:55 | mnaser | dtroyer: i think purge is meant to kill an entire project, where as this.. the idea is that i can give it a network and it can list all subnets, then list all ports a ttached tos ubnets, and delete from ports up to make the network deletable | |
| 17:58:00 | mnaser | unless purge exists and does just that.. | |
| 17:58:48 | dtroyer | I have not used nor worked on purge, but yeah I do think it is for a whole project. so do we need to come up with a way to specify subsets of project-owned resources? | |
| 17:59:19 | mnaser | dtroyer: yeah well the idea is i'm adding a --cascade command and i guess it'll naively gather the resources we know about at least | |
| 18:00:41 | dtroyer | you are tying that to a network? would it make sense to also have subsets of that? I don't have enough Neutron in my head for it to be obvious where else to use that idea | |
| 18:01:22 | mnaser | dtroyer: correct, you would ask for a cascading network delete, and it would go, list all subnets, then list all ports attached to each subnet, delete all ports, and then now we'll be able to delete all subnets, then we'll be able to delete the network | |