| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-01-19 | |||
| 16:13:00 | dtroyer | I need to digest that new thing, this is the first I've even heard of it, and we have a nice history of pissing off Cider folk by changing their names for things, which sometimes are unfortunate choices in the bigger scheme of things | |
| 16:14:15 | waj334 | dtroyer: As long as I can say it's been implemented, I'm cool with however it gets done | |
| 16:18:28 | openstackgerrit | Merged openstack/os-client-config: Fix typo for baremetal_service_type https://review.openstack.org/422381 | |
| 16:49:29 | openstackgerrit | Justin A Wilson proposed openstack/python-openstackclient: Refactor volume service tests https://review.openstack.org/422757 | |
| 16:52:06 | edleafe | elmiko: If you could review https://etherpad.openstack.org/p/api-wg-newsletter and make any changes you think are needed, that would be awesome | |
| 16:52:51 | openstackgerrit | Merged openstack/api-wg: Removes unnecessary utf-8 encoding https://review.openstack.org/414782 | |
| 17:04:19 | elmiko | edleafe: ack, looking now | |
| 17:05:01 | elmiko | edleafe: lgtm +1 | |
| 17:05:09 | elmiko | thanks =) | |
| 18:02:44 | stevemar | waj334: dtroyer pay dem bills | |
| 19:24:40 | openstackgerrit | Merged openstack/python-openstacksdk: Network L3 Router Commands https://review.openstack.org/385728 | |
| 19:58:18 | openstackgerrit | Nikita Gerasimov proposed openstack/python-openstackclient: Switch server create to block_device_mapping_v2 https://review.openstack.org/407111 | |
| 20:24:26 | ankur-gupta-f1 | did I miss the IRC meeting again :| | |
| 20:56:59 | openstackgerrit | Justin A Wilson proposed openstack/python-openstackclient: Refactor volume service v2 tests https://review.openstack.org/422757 | |
| 20:58:22 | stevemar | oh no, i think i did too | |
| 20:58:42 | stevemar | dtroyer: looks like we all missed the meetign | |
| 20:58:44 | stevemar | :) | |
| 22:14:04 | stevemar | dtroyer: heads down work is fun | |
| #openstack-sdks - 2017-01-20 | |||
| 00:38:07 | openstackgerrit | Corentin Ardeois proposed openstack/js-openstack-lib: Update supported keystone version https://review.openstack.org/422936 | |
| 01:36:08 | openstackgerrit | Merged openstack/js-openstack-lib: Update supported keystone version https://review.openstack.org/422936 | |
| 04:28:05 | huanxuan | dtroyer, stevemar: Sorry for missing the meeting, a bit busy at the end of the school term :( | |
| 04:28:33 | huanxuan | dtroyer, stevemar: I have some questions about the freeze... | |
| 04:28:54 | huanxuan | I don't quite sure about that, what cann't we do during the freeze? we cann't merge new features, or bug fixed, right? | |
| 04:48:51 | stevemar | huanxuan: o/ | |
| 04:49:14 | huanxuan | stevemar: :) | |
| 04:49:29 | stevemar | huanxuan: you are in school! let me know when you graduate and are looking for a job :P | |
| 04:50:32 | stevemar | huanxuan: so, for feature freeze... we try to minimize the major changes doing into OSC this week and next | |
| 04:50:42 | huanxuan | stevemar: Yes... One year left, Will graduate in next year | |
| 04:51:03 | stevemar | huanxuan: if there is a major bug that needs fixing, we should fix it. But we shouldn't be introducing anything big, like a new feature | |
| 04:51:14 | stevemar | huanxuan: so a new command probably shouldn't merge | |
| 04:51:39 | stevemar | huanxuan: i am very impressed that you are doing this while in school, thats amazing | |
| 04:52:30 | huanxuan | stevemar: hehe! half time for study, half time for job :) | |
| 04:52:42 | stevemar | huanxuan: still very impressive! | |
| 04:53:13 | stevemar | huanxuan: the reason we have the freeze is because once we do our next release, it'll be the final release for the Ocata cycle | |
| 04:54:00 | stevemar | and we don't want to merge something that can be 1) incomplete or 2) possibly introduce bugs | |
| 04:54:43 | stevemar | huanxuan: other projects like tripleo and rally use OSC, so we can't have a broken CLI at the end of the release :) | |
| 04:54:57 | huanxuan | stevemar: OK! Get it, Thank you for your explanation | |
| 04:55:27 | huanxuan | stevemar: I need to learn more from you, dtroyer and others | |
| 04:56:23 | stevemar | huanxuan: ask away :) | |
| 04:56:50 | huanxuan | stevemar: Thanks :) | |
| 05:03:58 | reedip | stevemar : Do you wish to be back in school ??? :P | |
| 05:04:11 | stevemar | reedip: tempting offer :P | |
| 05:04:35 | reedip | o really ?? I think you will go back to school soon, for someone else though :) | |
| 05:07:24 | stevemar | reedip: few more years til that happens :D | |
| 05:07:33 | reedip | stevemar :D | |
| 05:18:50 | openstackgerrit | Reedip proposed openstack/python-openstacksdk: Fix Setting Quotas in Neutron https://review.openstack.org/419911 | |
| 05:22:46 | reedip | stevemar : ^^ ?? | |
| 05:24:15 | openstackgerrit | Reedip proposed openstack/python-openstacksdk: Add project ID in QuotaDefault requests https://review.openstack.org/420301 | |
| 05:26:12 | stevemar | lgtm, but i don't know the sdk very well :( | |
| 05:43:54 | reedip | stevemar : hmm, okay | |
| 05:43:56 | reedip | no issues | |
| 05:44:10 | reedip | would wait for braincurtin's viewpoint | |
| 05:46:57 | openstackgerrit | Reedip proposed openstack/python-openstacksdk: Fix Setting Quotas in Neutron https://review.openstack.org/419911 | |
| 06:28:48 | openstackgerrit | jiahui.qiang proposed openstack/python-openstackclient: Functional test for volume qos https://review.openstack.org/421066 | |
| 06:44:46 | openstackgerrit | Rui Chen proposed openstack/python-openstackclient: Replace "Display Name" by "Name" in volume list https://review.openstack.org/423081 | |
| 07:48:11 | reedip | RuiChen : Is there tabination support for OpenstackClients ?? | |
| 07:48:17 | reedip | Just a query | |
| 07:58:30 | RuiChen | reedip: you means tagging support ? | |
| 08:00:03 | reedip | No , If I press a tab, then would the command be auto-complete ? | |
| 08:08:59 | RuiChen | reedip: openstack complete? | |
| 08:09:25 | reedip | ? | |
| 08:10:39 | reedip | oh ok let me try | |
| 08:11:15 | RuiChen | I saw the command "openstack complete" in help message, but i'm not sure which you want | |
| 08:13:15 | reedip | Okay, there is a bug in it :) | |
| 08:13:16 | reedip | openstack complete | |
| 08:13:17 | reedip | __init__() got an unexpected keyword argument 'project_name' | |
| 08:14:25 | openstackgerrit | Rui Chen proposed openstack/python-openstackclient: Replace "Display Name" by "Name" in volume list https://review.openstack.org/423081 | |
| 08:15:33 | RuiChen | looks like that help message error https://review.openstack.org/#/c/418221/ | |
| 08:36:59 | RuiChen | reedip: openstack complete issue you faced, it have the same root reason with help message error, please apply the patch https://review.openstack.org/#/c/418221/ | |
| 08:41:40 | RuiChen | and i think "complete" command is bash auto complete http://docs.openstack.org/developer/cliff/complete.html , that you need? | |
| 08:50:12 | reedip | RuiChen : yup | |
| 08:50:19 | openstackgerrit | Reedip proposed openstack/python-openstackclient: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/415369 | |
| 08:53:20 | openstackgerrit | Reedip proposed openstack/python-openstackclient: Add extra dhcp option to 'port create/set/unset' https://review.openstack.org/356263 | |
| 08:58:35 | openstackgerrit | Reedip proposed openstack/python-openstackclient: Overwrite/Clear Flavor property https://review.openstack.org/415369 | |
| 09:00:14 | openstackgerrit | Reedip proposed openstack/python-openstackclient: Overwrite/Clear Flavor property https://review.openstack.org/415369 | |
| 09:16:11 | RuiChen | openstackclient.tests.functional.image.v1.test_image.ImageTests.test_image_set functional test failed on all patches. | |
| 09:48:15 | openstackgerrit | Reedip proposed openstack/python-openstacksdk: Add project ID in QuotaDefault requests https://review.openstack.org/420301 | |
| 09:54:29 | openstackgerrit | Rui Chen proposed openstack/python-openstackclient: Functional tests debug support https://review.openstack.org/423154 | |
| 10:20:19 | openstackgerrit | Sanu M proposed openstack/openstackclient: Removes unused utf-8 declaration https://review.openstack.org/423177 | |
| 11:22:54 | openstackgerrit | Merged openstack/cliff: Add newline if the output formatter is json https://review.openstack.org/422488 | |
| 12:32:15 | stevemar | RuiChen: is the OSC gate broken? | |
| 12:37:11 | stevemar | sigmavirus: o/ | |
| 12:37:21 | stevemar | did glance change something with the is_public flag for v1 images only? | |
| 12:37:42 | stevemar | we're getting a failure in our functional tests: http://logs.openstack.org/54/423154/1/check/gate-osc-dsvm-functional-ubuntu-xenial/f23fbfa/testr_results.html.gz | |
| 12:46:39 | stevemar | put something out on the mailing list dtroyer rosmaita ^ | |
| 13:11:20 | sigmavirus | stevemar: \o | |
| 13:11:38 | sigmavirus | stevemar: shouldn't but if there's webob v1 in that gate then that could be part of the problem | |
| 13:11:47 | sigmavirus | err webob 1.7 | |
| 13:12:24 | sigmavirus | although the community images work should have merged overnight | |
| 13:12:29 | sigmavirus | so it could be a problem with that | |
| 13:27:57 | dtroyer | yay, let's change defaults! | |
| 13:28:14 | dtroyer | then get ready for a slew of patches with "that's the default now we can remove this we don't need to set it" | |
| 13:28:20 | dtroyer | </ugh> | |
| 13:31:23 | sigmavirus | dtroyer: I'm fairly certain the v1 impact was an unintended consequence | |
| 13:32:44 | dtroyer | ((FWIW, the snark above isn't directed at Glance, it's directed at the seemingly unusual number of patches I've seen recently that want to remove settings because they are defaults, and we know defaults do not change)) | |
| 13:33:15 | dtroyer | this just demonstrates why removing those is sometimes a bad idea | |
| 13:36:28 | dtroyer | <back-to-constructive-mode> | |
| 13:42:13 | dtroyer | looking over the community images review, is it possible that https://github.com/openstack/glance/commit/265659e8c34865331568b069fdb27ea272df4eaa#diff-162f814daca94e2b95fa809d7a054eae needs to be specific to v1 and v2 now? | |
| 13:45:53 | dtroyer | working backwards, more wild speculation: https://github.com/openstack/glance/commit/265659e8c34865331568b069fdb27ea272df4eaa#diff-a4e34865d1737c192a19a108ee8f1359 | |
| 13:46:33 | dtroyer | there are other places the new is_public kwarg is set to True in v1… that may be why the unit tests didn't see this | |