| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-04-24 | |||
| 21:13:28 | sean-k-mooney | the nova issue we can prably debug more here | |
| 21:13:34 | sean-k-mooney | is it repoducable? | |
| 21:13:48 | JamesBenson | most likely. | |
| 21:14:09 | sean-k-mooney | only with xen or do you think i would hit it with libvir kvm too | |
| 21:14:27 | sean-k-mooney | this does not look realted to xen to be honest | |
| 21:14:36 | sean-k-mooney | well the db issues dont | |
| 21:15:19 | sean-k-mooney | there are some other issue with the xenapi too at the end | |
| 21:15:25 | mriedem | efried: yeah i've had the patch up since earlier today | |
| 21:16:38 | JamesBenson | I'm not sure where it exactly crosses over, but I know definitely those two packages have issues, kolla-ansible and os-xenapi. Nova will need to dig deeper into to see where it breaks. | |
| 21:16:49 | JamesBenson | But I've never dug into nova this deep. | |
| 21:18:11 | sean-k-mooney | well there seams to be 2 basic erros in the nova log | |
| 21:18:15 | sean-k-mooney | DBNotAllowed: nova-compute | |
| 21:18:20 | sean-k-mooney | and then the xen api one | |
| 21:18:59 | JamesBenson | agreed | |
| 21:19:24 | JamesBenson | Is it possible that kolla mucked up the install of that package? | |
| 21:19:52 | JamesBenson | But I thought that was pulled from the nova repo. | |
| 21:19:53 | JamesBenson | ? | |
| 21:20:47 | sean-k-mooney | well its posibel that the xen dirver need other config option to be set that kolla is nto setting in the nova.conf | |
| 21:22:28 | sean-k-mooney | the default for the databse are set here https://github.com/openstack/kolla-ansible/blob/2dd69e9140b1ce1bd248c5c09217fb3a6502a9fc/ansible/roles/nova/defaults/main.yml#L161-L167 and used to generate teh config nova.conf here https://github.com/openstack/kolla-ansible/blob/2dd69e9140b1ce1bd248c5c09217fb3a6502a9fc/ansible/roles/nova/templates/nova.conf.j2#L156-L164 | |
| 21:23:01 | sean-k-mooney | that code should be virt dirver independent | |
| 21:23:25 | mriedem | melwitt: heh yeah that PS4 on the transactional create patch is about a billion times cleaner | |
| 21:25:58 | mriedem | efried: https://review.opendev.org/#/c/652868/ | |
| 21:30:21 | openstackgerrit | Dustin Cowles proposed openstack/nova master: WIP/PoC: Use SDK instead of ironicclient for node.get https://review.opendev.org/642899 | |
| 21:32:23 | efried | mriedem: Right, just wanted to make sure that was the final | |
| 21:32:39 | efried | mriedem: and presumably the one on top fails tests until the bottom one merges and the tag is created. | |
| 21:36:17 | jaypipes | dansmith, mriedem: is it possible to update metadata on a server that failed to boot properly? (i.e. the instance went to cell0) | |
| 21:36:35 | dansmith | jaypipes: metadata? like user metadata? | |
| 21:37:10 | mriedem | efried: i'm trying to figure out why the one on top is failing | |
| 21:37:12 | jaypipes | dansmith: no, sorry, metadata was a bad term. any attribute of the instance... say, it's name. | |
| 21:37:27 | efried | mriedem: It's because the tags aren't created yet | |
| 21:37:44 | efried | mriedem: I guess the job isn't smart enough to infer them from the releases in the predecessor patch. | |
| 21:37:57 | dansmith | jaypipes: shouldn't really be anything cell0 specific I don't think.. if you can do it in error, then you can do it | |
| 21:38:06 | dansmith | jaypipes: anything that requires talking to the host the instance is on won't work, of course | |
| 21:39:01 | mriedem | jaypipes: like dansmith says it depends on the API and what it restricts, | |
| 21:39:09 | mriedem | but i don't see any vm_state restrictions on the server update API | |
| 21:39:29 | dansmith | and what I meant was, there could be vm_state restrictions, but none that are specific to cell0 that I know of, | |
| 21:39:33 | dansmith | except something that ends up with an rpc call | |
| 21:39:56 | jaypipes | dansmith, mriedem: this is some custom Yahoo! API plugin ... it grabs the Instance object, updates some custom attribute on it and then calls instance.save(). Unfortunately, that isn't working in a cellsv2 environment... | |
| 21:39:59 | mriedem | interestingly there are restrictions on updating things like metadata and tags | |
| 21:40:17 | mriedem | jaypipes: likely b/c it's not targeted to the cell0 db | |
| 21:40:24 | dansmith | jaypipes: should be fine, just .. yeah, make sure it's targeted, | |
| 21:40:31 | mriedem | it's not cells-aware | |
| 21:40:35 | jaypipes | yeahg | |
| 21:40:44 | jaypipes | juno code. | |
| 21:40:51 | jaypipes | being converted to ocata. ugh. | |
| 21:40:54 | dansmith | juno we don't give a shit, right? | |
| 21:40:54 | mriedem | if it were smart and used the common.get_instance method, the Instance._context would already be targeted | |
| 21:41:08 | jaypipes | ocata. | |
| 21:41:23 | jaypipes | none of this is "smart" :) | |
| 21:41:30 | dansmith | JU .. NO | |
| 21:41:35 | mriedem | can i count verizon media group as a fallback employer to fix these old forks in case i get canned? | |
| 21:41:52 | dansmith | can anyone count on verizon media group as an employer? | |
| 21:41:56 | dansmith | bah dum dum | |
| 21:47:48 | mriedem | ugh https://github.com/openstack/nova/blob/a991980863f056323c1ee9fd6a46dbc4cb899eca/nova/availability_zones.py#L125 | |
| 21:48:02 | mriedem | https://github.com/openstack/nova/blob/a991980863f056323c1ee9fd6a46dbc4cb899eca/nova/compute/api.py#L4972 | |
| 21:48:08 | mriedem | badum https://review.opendev.org/#/c/649197/ | |
| 22:03:20 | efried | gmann: Thursday 1200-1230 okay for API inconsistency cleanup? | |
| 22:03:52 | gmann | efried: yeah, prefect. | |
| 22:04:09 | efried | gmann: d'oh, I meant Friday 1200-1230 | |
| 22:05:05 | gmann | efried: ok, friday also fine at 12. | |
| 22:05:10 | efried | gmann: thanks | |
| 22:09:17 | efried | dustinc: I've got the sdk topic scheduled for Thursday 1415-1430. mordred plans to be there for it too. | |
| 22:09:48 | jaypipes | dansmith, mriedem: cells(v2) is a requirement in ocata, no? meaning, there's no "turning off" or not using cellsv2, yeah? | |
| 22:11:33 | jaypipes | dansmith, mriedem: that's what the nova-manage map_instances call enables I thought? | |
| 22:12:36 | mriedem | cells v2 is required since ocata yeah | |
| 22:12:40 | openstackgerrit | Matt Riedemann proposed openstack/nova master: AZ list performance optimization: avoid double service list DB fetch https://review.opendev.org/636947 | |
| 22:13:13 | mriedem | efried: i think i got the test leak plugged ^ and actually found another slight perf improvement in the process | |
| 22:13:24 | efried | nice | |
| 22:13:26 | mriedem | turns out globally mocking the DB API is not a great idea! | |
| 22:13:33 | efried | who woulda thunk | |
| 22:14:51 | melwitt | mriedem: yeah, jaypipes saved the day :D | |
| 22:15:22 | mriedem | so the spandex and cape aren't unusual | |
| 22:15:45 | jaypipes | nobody needs to see me in spandex. eww. | |
| 22:15:58 | melwitt | not even the pugs | |
| 22:17:34 | jaypipes | well, sure, Winnie in a cape would be cool. | |
| 22:17:54 | melwitt | pugs in spandex, smooth out the rolls | |
| 22:18:08 | jaypipes | she's already got the Rosie the Pugeter iron-on decal on her Brownies jersey. | |
| 22:18:49 | melwitt | hells yeah | |
| 22:21:30 | melwitt | hm, somehow the unit tests get caught up in the database poison fixture when I back out the change | |
| 22:21:46 | jaypipes | that girl is poison. | |
| 22:22:06 | melwitt | that's a blast from the past | |
| 22:22:13 | jaypipes | that's right... whipped out some Bell Biv there. | |
| 22:22:28 | jaypipes | jeebus I'm old. | |
| 22:23:10 | melwitt | oh, it's just because of the database context manager decorator on the helper method, that triggers the POISON | |
| 22:23:30 | melwitt | arthritis-- | |
| 22:29:28 | efried | aspiers: are you around through Saturday at the PTG? | |
| 22:29:33 | aspiers | yup | |
| 22:32:37 | efried | aspiers: Friday 1615-beer good for you? | |
| 22:33:12 | aspiers | $day $time beer is good for me | |
| 22:33:44 | aspiers | that particular tuple is no exception :) | |
| 22:33:59 | efried | sorry, what I mean is, Friday 1615 until it's time to go get beer. | |
| 22:34:04 | efried | I'm not saying you get beer at 1615 | |
| 22:34:08 | aspiers | awww | |
| 22:34:15 | efried | you can bring your own, I won't stop you. | |
| 22:34:16 | aspiers | raise up my hopes then cruelly dash them, why don't you | |
| 22:34:29 | aspiers | but sure :) | |
| 22:34:34 | efried | I'm cruelly dashing that way | |
| 22:34:35 | aspiers | I guess 1615 is kinda early to start ;-) | |
| 22:34:54 | efried | it'll be 2315 in your time zone | |
| 22:35:00 | aspiers | true | |