Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-01-17
17:18:44 edleafe My feelings on API changes is to not release it until there is very little liklihood of it needing to be changed in the foreseeable future. IOW, it should not be an iterative process
17:19:15 elmiko ++ agreed
17:19:38 elmiko mind you, i am not proposing the release but the spec/guideline that proposes the usage
17:24:22 dtantsur yeah, a spec would be nice (I thought we had one?)
17:27:51 elmiko i thought we did too, but maybe mugsie did such a good job presenting it that i just didn't notice XD
17:28:01 cdent clearly the healthcheck can be written, tested, and nearly packaged without ever needing to choose its official name, yes?
17:28:11 elmiko oh definitely
17:28:19 elmiko /s
17:28:23 cdent in fact, ideally the middleware would be written with the name being a parameter
17:28:26 cdent (with a default)
17:28:41 mugsie cdent: I did think about that
17:28:56 mugsie But I like the idea that we have a single well known url
17:29:29 elmiko i do too mugsie
17:29:41 mugsie just make it part of the refstack tests with the good URL, and that should fix it :P
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: [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
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 :)

Earlier   Later