| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-11-19 | |||
| 22:06:26 | dustinc | dansmith: Well the issue is that we need to decide what to do in the case the node is defined more than once in the configuration files. I proposed that we use the first entry that identifies the node and ignore the rest, but efried proposes that we use the $COMPUTE_NODE as a catch all and the specific uuid/name identification should take precedence if both exit (this makes sense the more I think about it) | |
| 22:06:37 | dustinc | the remaining issue we _might_ run into is what happens if the node is identified by name and uuid? | |
| 22:13:51 | efried | dustinc: let me go check whether the ironic node RP name and uuid are in fact the same in placement... | |
| 22:15:35 | efried | dustinc: o look, a comment right at the top of ironic driver's update_provider_tree: | |
| 22:15:35 | efried | # nodename is the ironic node's UUID. | |
| 22:16:26 | efried | dansmith: lacking a better idea, do you think "create a glance image for the virtual device file" is an acceptable (if unlovely) path forward? | |
| 22:17:09 | dustinc | ok so we can identify name/uuid identification conflicts before init with ironic...but what about nova? how can we be sure a name and uuid don't identify the same node? what should we do in the case that happens? | |
| 22:17:44 | efried | dustinc: that we should be able to determine the *first* time the RT hits this code, i.e. on startup. | |
| 22:18:31 | efried | for non-ironic, that is. Because we're looking at the compute node RP already. | |
| 22:19:41 | efried | To cover ironic, we just put an extra check that cfg1['name'] != cfg2['uuid'] for all (cfg1, cfg2). It's a tad specific, but it would be insanity for node[name] on one node to be the same as node[uuid] on another, so I don't feel awful about doing that. | |
| 22:21:21 | efried | btw, to anticipate (ironic) scaling issues, probably a good idea to index the configs by identifier. Which, since we're forbidding name/uuid conflicts, actually helps make ^ seem saner :P | |
| 22:23:56 | efried | so, the load-up logic becomes simply: | |
| 22:23:56 | efried | if identifier in cfgs: | |
| 22:23:56 | efried | puke | |
| 22:23:56 | efried | cfgs[identifier] = cfg | |
| 22:23:56 | efried | And then when we're processing a node: | |
| 22:23:56 | efried | if node.name in cfgs: | |
| 22:23:57 | efried | cfg = cfgs[node.name] | |
| 22:23:57 | efried | elif node.uuid in cfgs: | |
| 22:23:58 | efried | cfg = cfgs[node.uuid] | |
| 22:23:58 | efried | elif '$COMPUTE_NODE' in cfgs: | |
| 22:23:59 | efried | cfg = cfgs['$COMPUTE_NODE'] | |
| 22:24:00 | efried | else: | |
| 22:24:55 | efried | or more terser: | |
| 22:24:56 | efried | cfg = cfgs.get(node.uuid, cfgs.get(node.name, cfgs.get('$COMPUTE_NODE'))) | |
| 22:24:56 | efried | if cfg: ... | |
| 22:44:47 | efried | dansmith: procedurally, to get things moving, how would you feel about "resize/migration/shelve/others will be addressed in a subsequent spec"? | |
| 23:10:07 | efried | mriedem: Is grenade *supposed* to use py2 on the "old" side? | |
| 23:19:53 | melwitt | mriedem: new tempest test shenanigans you might have an opinion about https://review.opendev.org/693096 tl;dr is downstream we want to be able to test with the config_drive: True request param for POST /servers and one way to add it would result in dupe test coverage in the gate | |
| 23:22:48 | gmann | efried: for ussuri jobs no. py2->py3 does not work and required more fixes on pckg installation etc. it was decided to do py3->py3 in grenade ussuri jobs. | |
| 23:24:41 | melwitt | gmann: hey, could you hit up the unified limits spec again? john updated to address your latest comments | |
| 23:25:33 | efried | gmann: Gotcha. How do I make the "old" side run py3 then? | |
| 23:26:06 | gmann | melwitt: done. | |
| 23:26:13 | melwitt | thanks gmann | |
| 23:26:40 | efried | gmann: note that https://review.opendev.org/#/c/695052/ still failed with the old side running py2, so | |
| 23:26:40 | efried | export DEVSTACK_GATE_USE_PYTHON3=True | |
| 23:26:40 | efried | apparently isn't correct/sufficient. | |
| 23:27:07 | gmann | efried: is it, this should make use_python3=True on old node | |
| 23:27:40 | gmann | i did in all other patches also - https://review.opendev.org/#/q/topic:drop-py27-support-devstack-default-py3+(status:open+OR+status:merged) | |
| 23:27:49 | gmann | and grenade-py3 job is doing that only. | |
| 23:33:56 | gmann | for example, watcher patch is working fine - https://review.opendev.org/#/c/695097/1 | |
| 23:36:03 | gmann | from log it seems like DEVSTACK_GATE_USE_PYTHON3 is not set on d-g side | |
| 23:50:24 | efried | gmann: sorry, I didn't mean to bail on you there, I was trying to get to zuul.o.o/jobs and it crashed my machine. This has been happening for a while, I should learn. | |
| 23:50:47 | efried | Anyway, now I *do* need to bail. I'll try to catch up with you tomorrow about the grenade business. | |
| 23:51:14 | gmann | efried: sure. | |
| #openstack-nova - 2019-11-20 | |||
| 00:38:12 | openstackgerrit | sean mooney proposed openstack/nova master: add [libvirt]/max_queues config option https://review.opendev.org/695118 | |
| 01:35:18 | mriedem | melwitt: you're going to have to ping me earlier tomorrow about that tempest patch, it's too late for me to dig into that right now | |
| 01:35:24 | mriedem | but i'll star it | |
| 01:36:03 | melwitt | mriedem: ok, np. it's not urgent but something I thought you might have thoughts on. thanks | |
| 01:40:46 | mriedem | also, | |
| 01:41:00 | mriedem | your downstream ci people know they can just write tempest plugins for the tests they want to use right? | |
| 01:41:04 | mriedem | it doesn't need to be in the main repo | |
| 01:41:32 | mriedem | so i don't think it's a great pattern to set with like, "we'd like to test x which is 99% already covered elsewhere" | |
| 01:43:19 | mriedem | anyway, i've got to drop but i left that comment on there | |
| 01:43:19 | melwitt | I didn't know how tempest plugins work | |
| 01:43:26 | mriedem | cinder has one, check it out | |
| 01:43:36 | mriedem | like devstack plugins | |
| 01:43:41 | mriedem | https://github.com/openstack/cinder-tempest-plugin | |
| 01:43:59 | mriedem | anyway, o/ | |
| 01:44:01 | melwitt | ok | |
| 02:22:22 | mnaser | sean-k-mooney: something in your domain -- i was wondering if libvirt by default makes sure that vcpus for domains live inside the same numa node | |
| 02:22:50 | mnaser | i'm playing with new epyc rome hardware and seeing poor memory access times with sysbench, im wondering if the reasoning behind it is that cpu time is flopping back and forth between numa nodes | |
| 02:38:56 | alex_xu | melwitt: hellow, are you still here | |
| 02:39:12 | alex_xu | s/hellow/hello/ | |
| 02:55:30 | HagunKim | Hi, How can I see Nova version like 19.0.3, 20.0.1 on my nova-compute host? | |
| 03:58:29 | openstackgerrit | Merged openstack/nova master: Remove functional test specific nova code https://review.opendev.org/683609 | |
| 04:58:14 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Add composable flavor properties https://review.opendev.org/663563 | |
| 06:13:13 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717 | |
| 07:57:51 | gibi | HagunKim: $ nova-compute --version | |
| 07:57:51 | gibi | 20.1.0 | |
| 07:58:48 | HagunKim | gibi: Thanks! | |
| 08:28:17 | bauzas | good morning Nova | |
| 08:31:17 | gibi | bauzas: good morning | |
| 08:31:46 | bauzas | gibi: just doing a few stuff now, but if you're OK, I'd like to discuss on the audit command in 30 mins | |
| 08:31:49 | bauzas | fine with you ? | |
| 08:31:58 | gibi | bauzas: sure, ping me | |
| 08:32:35 | bauzas | tl;dr: when calling --delete, you haven't use the --verbose also, so I don't see if we tried to delete the allocation | |
| 08:32:38 | bauzas | gibi: ^ | |
| 08:33:05 | gibi | bauzas: I can quickly try --verbose + --delete | |
| 08:33:20 | bauzas | gibi: would be nice, thanks | |
| 08:33:45 | gibi | bauzas: http://paste.openstack.org/show/786387/ | |
| 08:34:12 | gibi | the output of the --verbose --delete seems like to be the same as the output of only --verbose | |
| 08:34:50 | bauzas | gibi: weirod | |
| 08:34:54 | gibi | yepp | |
| 08:35:12 | gibi | yesterday I had to leave but I can try to attach a debugger now | |
| 08:35:20 | bauzas | looks like the conditional doesn't work | |
| 08:35:38 | bauzas | I mean in https://review.opendev.org/#/c/670112/10/nova/cmd/manage.py@2806 | |
| 08:37:55 | gibi | bauzas: debugger helped it is the args definition that is buggy https://review.opendev.org/#/c/670112/10/nova/cmd/manage.py@2887 | |
| 08:38:33 | bauzas | argh | |
| 08:38:38 | gibi | I guess you are missing some test coverage for --delete :) | |
| 08:38:42 | bauzas | but I'm testing it | |
| 08:39:12 | bauzas | gibi: https://review.opendev.org/#/c/670112/10/nova/tests/functional/test_nova_manage.py@1417 | |
| 08:39:14 | gibi | bauzas: you are calling audit() directly from the test so skipping the arg parser | |
| 08:39:14 | bauzas | OH ! | |
| 08:39:19 | bauzas | yeah this | |
| 08:39:28 | bauzas | how could I test it then ? | |
| 08:41:12 | gibi | bauzas: good question. I just quickly checked and the heal_allocation tests do not cover args parsing either | |
| 08:41:18 | bauzas | yeah | |
| 08:44:07 | gibi | I would not worry about it then in the functional env | |
| 08:44:18 | gibi | it seems we don't have the facility there | |