| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-01-22 | |||
| 12:40:10 | mordred | frickler: so I have an idea | |
| 12:40:44 | mordred | frickler: what if, in devstack, we uploaded the cirros image as "cirros" and not "cirros-0.4.0-x86_64-disk" | |
| 12:48:15 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Set DEFAULT_IMAGE_NAME in localrc https://review.openstack.org/632456 | |
| 12:48:34 | mordred | frickler, Shrews, dtantsur: ^^ maybe like that (I +3d the image name patch) | |
| 12:48:41 | dtantsur | mordred: cirros-x86_64-disk and cirros-x86_64-uec (there's at least two image types) | |
| 12:49:01 | frickler | mordred: that doesn't sound too far fetched. might break some other stuff, but I like the idea | |
| 12:49:14 | dtantsur | although for this one job it does not matter | |
| 12:49:21 | dtantsur | (I thought you're going to change devstack itself) | |
| 12:49:35 | frickler | ah, that's what I was assuming, too | |
| 12:50:06 | dtantsur | mordred, frickler, please watch out for the ironic job if it inherits from this job | |
| 12:51:22 | mordred | dtantsur, frickler: yeah - I think updating devstack would be lovely - but figured that might take a little bit of time :) | |
| 12:51:33 | mordred | and good to know re: disk and uec | |
| 12:56:36 | dtantsur | I suspect only ironic cares about uec.. | |
| 12:57:24 | mordred | ZOMG. I have now read the devstack code related to this | |
| 12:59:05 | mordred | dtantsur: -uec seems to only be added to IMAGE_URLS if the virt driver is xenserver | |
| 12:59:33 | dtantsur | mordred: ironic has a separate devstack plugin, I guess we add it there | |
| 12:59:43 | mordred | nod | |
| 12:59:48 | mordred | well - my patch above isn't going to work | |
| 12:59:58 | dtantsur | mordred: oh, fun https://github.com/openstack/ironic/blob/39bf7fa86f3c1d5cd53e27e42d616b40aa739023/devstack/common_settings#L36-L46 | |
| 13:00:19 | mordred | DEFAULT_IMAGE_NAME is not used to set the image name for glance | |
| 13:00:37 | mordred | the image name in glance is derived from the filename of the image from its url | |
| 13:00:51 | mordred | the DEFAULT_IMAGE_NAME just tells other things what the image name is | |
| 13:05:44 | dtantsur | mordred: well, there's always this: https://github.com/openstack/metalsmith/blob/master/playbooks/integration/cirros-image.yaml#L8 :D | |
| 13:07:16 | mordred | dtantsur: yeah :) | |
| 14:12:51 | openstackgerrit | Merged openstack/openstacksdk master: Update cirros version for functional tests https://review.openstack.org/632415 | |
| 16:08:57 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Add a non-voting job with metalsmith https://review.openstack.org/632496 | |
| 16:08:58 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: [DNM] Testing metalsmith CI https://review.openstack.org/632497 | |
| 16:10:43 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Add a non-voting job with metalsmith https://review.openstack.org/632496 | |
| 16:10:43 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: [DNM] Testing metalsmith CI https://review.openstack.org/632497 | |
| 18:02:25 | openstackgerrit | Merged openstack/os-service-types master: Updated from OpenStack Service Type Authority https://review.openstack.org/631850 | |
| 21:32:12 | slaweq | mordred: hi | |
| 21:32:48 | slaweq | mordred: do You remember, some time ago I reported bug that using osprofiler with openstackclient is not possible for neutron | |
| 21:33:08 | slaweq | mordred: it's in fact bug in openstacksdk IMO, I reported it in https://storyboard.openstack.org/#!/story/2004618 - please check my comment there | |
| 21:33:24 | slaweq | mordred: maybe You would have any idea how to fix that :) | |
| 21:35:12 | mordred | slaweq: I shall investigate! | |
| 21:35:19 | slaweq | mordred: thx | |
| 21:36:13 | mordred | slaweq: ok - so - there is a set of patches that will make that TaskManager code go away | |
| 21:36:45 | mordred | slaweq: it'll still probably be a couple of weeks - we need to land a patch in keystoneauth first and kmalloc has been rudely "taking vacation" | |
| 21:36:51 | mordred | slaweq: but - I hope to get that finished up soon | |
| 21:37:00 | mordred | and at that point, I expect everything will just magically work :) | |
| 21:37:11 | slaweq | mordred: that would be great :) | |
| 21:37:17 | slaweq | mordred: I can wait for it | |
| 21:37:26 | mordred | let me get you links to the changes ... | |
| 21:38:10 | mordred | slaweq: https://review.openstack.org/#/c/605043/ is the ksa and https://review.openstack.org/#/c/604926/ is the main sdk patch | |
| 21:39:20 | slaweq | mordred: thx, I will take a look and try it on my local devstack vm when I will have few minutes :) | |
| 21:41:10 | mordred | slaweq: all of that free time we have right? :) | |
| 21:41:44 | slaweq | mordred: yep :) | |
| 21:48:09 | kmalloc | mordred: hah | |
| 21:48:13 | kmalloc | mordred: vacation... :P | |
| 21:48:29 | kmalloc | mordred: do we have a ksa patch or we need to create one? | |
| 21:57:54 | mordred | kmalloc: https://review.openstack.org/#/c/605043/ | |
| 21:58:14 | mordred | kmalloc: I think we're still wanting a functional test something something | |
| 21:59:54 | kmalloc | ah yeah | |
| 21:59:57 | kmalloc | now i remember | |
| 22:00:08 | kmalloc | it's the whole i need to take vacation and check out for a month | |
| 22:00:14 | kmalloc | things are slowly coming back to me | |
| 22:00:28 | kmalloc | it's like i haven't written code or even looked at it for quite a while :P | |
| 22:03:05 | mordred | kmalloc: this is the correct life decision | |
| 22:03:47 | kmalloc | i actually have internet at home now, yay old house that makes running things like ethernet unfun | |
| 22:04:31 | kmalloc | now if only my equalizer didn't pickup NPR (i mean, i don't mind listening to radio, but i don't really want to listen to it when i don't want to listen to it) | |
| 22:36:35 | mordred | kmalloc: bwhahaha | |
| 22:55:18 | kmalloc | mordred: aha found it ... bad XLR cable must have had failing sheilding | |
| 22:55:32 | kmalloc | i mean it's still there if i crank the gain to massive, but way better with a diff. xlr | |
| 23:47:28 | openstackgerrit | Merged openstack/openstacksdk master: Fixed incorrect exception raising in configdrive generation https://review.openstack.org/632326 | |
| #openstack-sdks - 2019-01-23 | |||
| 14:45:44 | openstackgerrit | Johannes Kulik proposed openstack/openstacksdk master: Support dict of links in pagination detection https://review.openstack.org/632736 | |
| 15:28:59 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: use overriden base_path in remaining CRUD operations https://review.openstack.org/632757 | |
| 15:33:40 | mordred | kmalloc: https://review.openstack.org/#/c/632736 is to fix a keystone related thing - could you ack? | |
| 17:07:08 | kmalloc | checking | |
| 17:08:39 | openstackgerrit | Morgan Fainberg proposed openstack/openstacksdk master: Support dict of links in pagination detection https://review.openstack.org/632736 | |
| 17:10:36 | mordred | kmalloc: thanks! | |
| 17:13:31 | kmalloc | ooh, thats kinda icky | |
| 17:13:50 | kmalloc | fixed the pep8 btw. | |
| 17:15:05 | kmalloc | links is so useless sometimes. | |
| 17:15:13 | kmalloc | most of the time* especially in keystone | |
| 19:29:57 | jabadia | Can someone +2 this trivial bug fix ? https://review.openstack.org/#/c/627734/1 | |
| 19:30:04 | jabadia | ( Please :) ) | |
| 19:35:02 | tosky | jabadia: I can't grant the +2 permission, but I think that you should replace the line | |
| 19:35:03 | tosky | Fix for bug: https://bugs.launchpad.net/python-openstackclient/+bug/1674213 | |
| 19:35:06 | openstack | Launchpad bug 1674213 in python-openstackclient "openstack output as json end with shell line" [Wishlist,Triaged] - Assigned to Yossi Ovadia (jabadia) | |
| 19:35:07 | tosky | with a simpler | |
| 19:35:13 | tosky | Closes-Bug: 1674213 | |
| 19:35:28 | tosky | see https://wiki.openstack.org/wiki/GitCommitMessages | |
| 19:35:33 | jabadia | thanks | |
| 19:39:28 | openstackgerrit | Yossi Ovadia proposed openstack/cliff master: Missing carriage return in some cases, using -f json https://review.openstack.org/627734 | |
| 19:42:59 | openstackgerrit | Yossi Ovadia proposed openstack/cliff master: Missing carriage return in some cases, using -f json https://review.openstack.org/627734 | |
| #openstack-sdks - 2019-01-24 | |||
| 16:00:45 | edleafe | The API-SIG Office Hour has begun. Come one! Come all! | |
| 16:06:15 | cdent | oh hai | |
| 16:50:11 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: WIP Add DNS support https://review.openstack.org/623009 | |
| 17:00:41 | edleafe | Well, that was an exciting hour! | |
| 19:14:56 | openstackgerrit | James Denton proposed openstack/openstacksdk master: Adds prefixlen to the request body when creating subnets https://review.openstack.org/550558 | |
| #openstack-sdks - 2019-01-25 | |||
| 07:53:05 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: WIP Add DNS support https://review.openstack.org/623009 | |
| #openstack-sdks - 2019-01-26 | |||
| 07:50:35 | openstackgerrit | Kailun Qin proposed openstack/python-openstackclient master: Add network segment range command object https://review.openstack.org/625544 | |
| 09:23:45 | openstackgerrit | Kailun Qin proposed openstack/python-openstackclient master: Add network segment range command object https://review.openstack.org/625544 | |
| 16:22:34 | openstackgerrit | Kailun Qin proposed openstack/python-openstackclient master: Add network segment range command object https://review.openstack.org/625544 | |
| #openstack-sdks - 2019-01-28 | |||
| 10:18:24 | openstackgerrit | Bernard Cafarelli proposed openstack/openstacksdk master: Fixes for Unicode characters in python 2 requests https://review.openstack.org/633468 | |
| 11:36:34 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Fix raise create_server and attach to a network given a net-name param https://review.openstack.org/630743 | |
| 11:38:20 | mordred | dtantsur, Shrews: morning! if you get a chance, I think https://review.openstack.org/#/c/626380/ is ready (dtantsur +2'd once already, but I fixed a review comment) | |
| 11:38:44 | dtantsur | mordred: will do. can accept a review on https://review.openstack.org/632496 in return :) | |
| 11:38:54 | mordred | already +3'd! | |
| 11:39:09 | dtantsur | ah, did not notice the notification :) thanks | |