| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-03-13 | |||
| 04:36:27 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 06:37:30 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 06:39:17 | reedip | amotoki : hi . I checked your comment for https://review.openstack.org/383025 . I think router removes the port which it creates itself when an interface is attached to it, not sure about the port which is assigned to it by user. I guess the code should be the same. | |
| 07:13:11 | amotoki | reedip: you can check it by the following neutron commands: "neutron port-create", "neutron router-interface-add router1 port=<id>", and then "neutron router-interface-delete" | |
| 07:13:36 | amotoki | reedip: I think you don't see the port you created after router-interface-delete | |
| 07:14:11 | amotoki | reedip: so I think 'openstack port delete' in addCleanup will fail | |
| 07:15:19 | amotoki | reedip: what happens if 'openstack port delete' cleanup fails. are subsequent cleanup called? | |
| 07:27:33 | reedip | amotoki : if port delete fails and does NOT delete the port, subnet delete and network delete will fail | |
| 07:28:39 | amotoki | reedip: a case I concern is that a port is deleted successfully by 'router-interface-delete' and the subsequent call of 'port delete' fails | |
| 07:28:53 | amotoki | reedip: in this case, subnet delete and network delete should succeed | |
| 07:29:08 | reedip | amotoki : Port deletion cleanup function might fail but network delete and subnet delete should succeed . | |
| 07:29:16 | amotoki | reedip: so I wonder 'network delete' registered by addCleanup will be called or not. | |
| 07:30:13 | amotoki | reedip: if you have 100% confident and other reviewers think the current is okay, I have no more objection. | |
| 07:30:24 | reedip | amotoki : :) | |
| 07:30:44 | amotoki | reedip: as one who knows neutron behavior, it just sounds tricky to me :) | |
| 07:31:20 | reedip | amotoki : Dont you think neutron itself is tricky :) ?? | |
| 07:31:59 | reedip | amotoki : this is one of the few patches left for the complete migration. I just wish this completes soon so that we can proceed to other patches and complete migration :) | |
| 07:32:36 | amotoki | reedip: so the simple way is drop addCleanup(port delete) :-) | |
| 07:33:32 | reedip | amotoki : I checked some comments in other patches. I think we can remove the addCleanup for the port for now ! | |
| 07:33:56 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 08:18:11 | rabel | hi there | |
| 08:18:39 | openstack | Launchpad bug 1612898 in python-openstackclient "Separate options for --nic when creating servers" [Medium,Confirmed] - Assigned to Ukesh (ukeshkumar) | |
| 08:18:39 | rabel | is https://bugs.launchpad.net/python-openstackclient/+bug/1612898 still interesting for you? I already wrote a comment in the bug report, because i'd like to work on this issue | |
| 08:29:42 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 08:31:57 | reedip | rabel : As there has been no updates in the bug for some time, I think you can take the issue | |
| 08:35:17 | openstackgerrit | Rui Chen proposed openstack/osc-lib master: Add formattable column base classes https://review.openstack.org/444760 | |
| 08:36:39 | rabel | reedip, ok thanks. i took it. do you think i should wait for answers to my proposal in the bug report or just write a commit and work with the feedback I recieve in the change? | |
| 08:37:55 | reedip | rabel : both are good options but I like the Pythonic way ( put a patch and wait for feedback ) :) | |
| 08:38:16 | rabel | :) | |
| 08:54:25 | openstackgerrit | venkata anil proposed openstack/python-openstackclient master: Allow admin to override distributed router flag https://review.openstack.org/433442 | |
| 09:13:53 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 10:08:20 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 10:08:52 | openstackgerrit | Reedip proposed openstack/python-openstackclient master: "floating ip set/unset port" for OSC https://review.openstack.org/383025 | |
| 10:56:05 | openstackgerrit | Lv Jiawei proposed openstack/python-openstackclient master: Add status filtering options to port list https://review.openstack.org/444810 | |
| 11:32:00 | openstackgerrit | Lv Jiawei proposed openstack/python-openstackclient master: Add extra filtering options to qos policy list https://review.openstack.org/401131 | |
| 12:55:48 | sdague | cdent: you about this morning? | |
| 12:55:54 | cdent | sdague: yup | |
| 12:57:21 | sdague | so, I've been rewriting the microversion blog post for more permanent archival purposes, including trying to get some diagrams in there (though it's not quite what I was hoping for, given the limits of blockdiag) | |
| 12:57:42 | sdague | I was curious if you could take a quick looking and comment on whether this was helping or not? | |
| 12:57:56 | cdent | yeah, sure | |
| 12:58:09 | rabel | when i run tox in python-openstackclient folder without any arguments, i get an invocation error on ostestr. do i have to do something different or is this a bug? | |
| 12:59:36 | openstackgerrit | Sean Dague proposed openstack/api-wg master: WIP: microversion architecture archival doc https://review.openstack.org/444892 | |
| 12:59:56 | sdague | cdent: it will probably be most helpful to let it render into html to see it | |
| 13:07:37 | sdague | cdent: http://docs-draft.openstack.org/92/444892/1/check/gate-api-wg-docs-ubuntu-xenial/d706e92//doc/build/html/guidelines/microversions-architecture.html | |
| 13:10:02 | edleafe | cdent: totally missed the time for getting the dwindling API-WG numbers discussion on the TC agenda | |
| 13:10:24 | cdent | edleafe: looks like the agenda is already pretty full anyway | |
| 13:10:39 | edleafe | yup | |
| 13:10:55 | edleafe | well, there's no time pressure, other than I said I'd do it | |
| 13:11:06 | edleafe | if there's time at the end of the meeting I'll mention it | |
| 13:11:43 | cdent | edleafe: I'm pretty sure that if you don't get it in this week, HTTP will stop working. | |
| 13:12:05 | edleafe | cdent: that was exactly my fear | |
| 13:20:55 | cdent | sdague: some thoughts: I like the idea of having some visual aids but at least for me these visual aids aren't communicating enough information to me | |
| 13:21:12 | cdent | I feel like I'm only able to intrepret them because I already know what's going on | |
| 13:21:37 | sdague | yeh... I have an idea of the visuals that would make this understandable | |
| 13:21:42 | cdent | I wonder if side by side comparisons of something closer to actual requests might convey more info? | |
| 13:21:54 | sdague | however, they exceed the abilities of any of the markup tools | |
| 13:22:05 | sdague | so it's all going to be hand svg | |
| 13:23:18 | sdague | so, I was trying to throw some things together as stand in to write the text, then go back and redo the diagrams | |
| 13:23:25 | cdent | yeah, makes sense | |
| 13:23:59 | cdent | another thing: people who do math find A' and A'' to be full of sense, but I'm not sure that's true for everyone | |
| 13:24:21 | sdague | yeh, that was directly a limitation of blockdiag | |
| 13:24:38 | sdague | given that the blocks don't resize to text inside | |
| 13:25:12 | sdague | ok, well, I'll go with "this might be useful, harder to evaluate until there are actually better diagrams. These definitely don't cut it" | |
| 13:25:26 | cdent | yeah, that's accurate | |
| 13:25:42 | cdent | +1 on it existing a proper archived doc | |
| 13:25:51 | sdague | yeh, I spent 4 hours in the hotel in boston last week looking into alternatives like graphviz directly, and the uml things | |
| 13:25:58 | sdague | as well as looking at the blockdiag source | |
| 13:26:18 | cdent | you so know how to party! | |
| 13:26:19 | sdague | and non of them can be bent to what I need here | |
| 13:26:23 | sdague | heh | |
| 13:27:05 | cdent | I assume you're committed to an editable-in-tree format? | |
| 13:27:26 | cdent | (otherwise png) | |
| 13:27:35 | briancurtin | yuvalb: are you Yuval Shalev? | |
| 13:29:44 | yuvalb | briancurtin: nope | |
| 13:31:59 | sdague | cdent: yeh, svg ideally, though I was having sphinx choke on svg being used directly, which is something I'll have to figure out | |
| 14:20:39 | openstackgerrit | David Rabel proposed openstack/python-openstackclient master: Add --network and --port to server create https://review.openstack.org/444924 | |
| 14:40:19 | openstackgerrit | Brian Curtin proposed openstack/python-openstacksdk master: Shift some compute attributes within request body https://review.openstack.org/444933 | |
| 14:50:10 | openstackgerrit | Jose Castro Leon proposed openstack/python-openstackclient master: Closes-Bug: 1672396 https://review.openstack.org/444934 | |
| 14:50:11 | openstack | bug 1672396 in python-openstackclient "Missing flavor information in server list" [Undecided,New] https://launchpad.net/bugs/1672396 | |
| 14:52:17 | openstackgerrit | Jose Castro Leon proposed openstack/python-openstackclient master: Adds missing flavor information in the server list long command https://review.openstack.org/444934 | |
| 15:10:16 | openstackgerrit | Brian Curtin proposed openstack/python-openstacksdk master: Shift some compute attributes within request body https://review.openstack.org/444933 | |
| 15:15:51 | openstackgerrit | Brian Curtin proposed openstack/python-openstacksdk master: Add image download example https://review.openstack.org/444940 | |
| 15:34:10 | openstackgerrit | Dean Troyer proposed openstack/cliff master: Fix broken test with cmd2 0.7.0 https://review.openstack.org/444951 | |
| 15:34:30 | openstackgerrit | Jose Castro Leon proposed openstack/python-openstackclient master: Adds missing flavor information in the server list long command https://review.openstack.org/444934 | |
| 15:35:13 | openstackgerrit | Nakul Dahiwade proposed openstack/python-openstackclient master: OSC Network Flavor Profile https://review.openstack.org/396783 | |
| 15:37:26 | reedip | ankur-gupta-f4 : o/ \o | |
| 15:37:39 | ankur-gupta-f4 | reedip: good evening | |
| 15:38:13 | reedip | its night :) Shifted to Philippines for a month :) | |
| 15:38:31 | ankur-gupta-f4 | ooo nice. Im in Colorado for a month. Getting some fresh mountain air | |
| 15:39:20 | openstackgerrit | Brian Curtin proposed openstack/python-openstacksdk master: Shift some compute attributes within request body https://review.openstack.org/444933 | |
| 15:39:54 | reedip | and I am getting fresh sea breeze :) | |
| 15:42:51 | openstackgerrit | Shashank Kumar Shankar proposed openstack/python-openstackclient master: Introduce Neutron DHCP agent commands to OSC https://review.openstack.org/387611 | |
| 15:47:20 | ankur-gupta-f4 | reedip: why Philippines if you don't mind me asking. Business or pleasure? :) | |
| 15:48:17 | reedip | Pleasure doing business .... | |
| 15:48:26 | ankur-gupta-f4 | thats the dream | |
| 15:48:30 | reedip | hehehehe | |
| 15:48:53 | reedip | m trying to achieve it | |
| 16:06:41 | openstackgerrit | Ankur proposed openstack/python-openstacksdk master: Introduce Base for Octavia (load balancing) https://review.openstack.org/428414 | |
| 16:43:37 | openstackgerrit | David Rabel proposed openstack/python-openstackclient master: Add --network and --port to server create https://review.openstack.org/444924 | |
| 16:49:31 | rabel | can someone help me with the tests? when i'm running py35, i always get "The test run didn't actually run any tests". | |