Earlier  
Posted Nick Remark
#openstack-sdks - 2019-01-17
17:29:43 cdent mugsie: oh, yeah, I want there to be a single well known url too
17:29:53 cdent I'm just saying that the coding doesn't require that
17:29:55 mugsie in 2-5 years
17:30:02 elmiko heh, ++
17:30:32 elmiko well see, we just add a new entry in the service catalog for healthcheck uris..... (kidding!)
17:31:34 dtantsur welll
17:31:42 dtantsur it's kind of these additional fields in DNS, right?
17:31:57 elmiko metadata all the things! \o/
17:32:15 dtantsur so we probably could do it.. but then the monitoring tools will have to learn about the catalog
17:35:46 elmiko ok, i'm headed out for lunch. take care all o/
17:36:59 dtantsur yeah, I should go as well
#openstack-sdks - 2019-01-18
07:30:24 openstackgerrit Sundar Nadathur proposed openstack/service-types-authority master: Add Cyborg as accelerator service. https://review.openstack.org/631718
17:09:42 openstackgerrit Sundar Nadathur proposed openstack/service-types-authority master: Add Cyborg as accelerator service. https://review.openstack.org/631718
17:16:22 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Return retries on HTTP CONFLICT to baremetal.attach_vif_to_node https://review.openstack.org/631836
17:16:23 creepy_owlet mordred: bugs, bugs everywhere ^^^
17:17:21 mordred creepy_owlet: oh wow
17:17:38 mordred Shrews: you wanna look at that? ^
17:17:51 creepy_owlet why wouldn't you use the same HTTP code for user's error and for an unforeseeable internal condition? >_<
17:27:04 mordred creepy_owlet: no good reason I can see :)
17:27:09 creepy_owlet heh
17:28:09 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Add baremetal Node fields from versions 1.47 - 1.49 https://review.openstack.org/631841
17:38:11 openstackgerrit Merged openstack/service-types-authority master: Add Cyborg as accelerator service. https://review.openstack.org/631718
17:53:05 openstackgerrit OpenStack Proposal Bot proposed openstack/os-service-types master: Updated from OpenStack Service Type Authority https://review.openstack.org/631850
18:00:54 Shrews mordred: looking
18:02:27 Shrews mordred: umm, since dtantsur|afk specifically had a note to NOT retry on conflict, I'd prefer he look at that
18:03:59 Shrews that change is a direct contradiction to that
18:11:27 mordred Shrews: that is a change from dtantsur|afk
18:12:21 Shrews mordred: HAHAHA
18:13:00 Shrews so creepy_owlet was dtantsur|afk ??
18:13:13 Shrews names are hard
18:14:45 mordred Shrews: yah. it's casual nick friday
21:43:42 openstackgerrit Merged openstack/openstacksdk master: Add baremetal Node fields from versions 1.47 - 1.49 https://review.openstack.org/631841
#openstack-sdks - 2019-01-19
13:17:16 openstackgerrit Merged openstack/openstacksdk master: Return retries on HTTP CONFLICT to baremetal.attach_vif_to_node https://review.openstack.org/631836
#openstack-sdks - 2019-01-21
09:45:07 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: WIP Add DNS support https://review.openstack.org/623009
#openstack-sdks - 2019-01-22
01:57:27 openstackgerrit caoyuan proposed openstack/os-client-config master: Update hacking version to latest https://review.openstack.org/628597
07:31:19 openstackgerrit zhouxinyong proposed openstack/api-sig master: Change openstack-dev to openstack-discuss in setup.cfg https://review.openstack.org/632255
08:26:07 openstackgerrit zhouxinyong proposed openstack/cliff master: Change openstack-dev to openstack-discuss in setup.cfg https://review.openstack.org/632295
09:00:14 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Fixed incorrect exception raising in configdrive generation https://review.openstack.org/632326
09:20:01 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Fixed incorrect exception raising in configdrive generation https://review.openstack.org/632326
11:16:06 openstackgerrit Jens Harbott (frickler) proposed openstack/openstacksdk master: Update cirros version for functional tests https://review.openstack.org/632415
11:16:59 frickler mordred: Shrews: ^^ we need to match devstack again
11:27:01 dtantsur ugh
11:35:27 openstackgerrit zhouxinyong proposed openstack/oaktree master: Change openstack-dev to openstack-discuss in setup.cfg https://review.openstack.org/632424
11:38:38 openstackgerrit zhouxinyong proposed openstack/oaktreemodel master: Change openstack-dev to openstack-discuss in setup.cfg https://review.openstack.org/632426
11:52:56 openstackgerrit zhouxinyong proposed openstack/openstackclient master: Change openstack-dev to openstack-discuss in setup.cfg https://review.openstack.org/632439
12:16:04 openstackgerrit zhouxinyong proposed openstack/osc-lib master: Fix author-email in setup.cfg https://review.openstack.org/632450
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

Earlier   Later