Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-05
12:04:52 openstackgerrit Balazs Gibizer proposed openstack/nova-specs master: Server move operations with ports having resource request https://review.opendev.org/652608
12:18:39 openstackgerrit Balazs Gibizer proposed openstack/nova master: nova-manage: heal port allocations https://review.opendev.org/637955
14:34:13 openstackgerrit Merged openstack/nova master: Make nova-tox-functional-py36 reusable https://review.opendev.org/657074
17:41:50 openstackgerrit Merged openstack/nova stable/rocky: libvirt: Stop ignoring unknown libvirtError exceptions during volume attach https://review.opendev.org/657051
18:04:16 openstackgerrit Merged openstack/nova stable/rocky: libvirt: Always disconnect volumes after libvirtError exceptions https://review.opendev.org/657110
22:51:25 cfriesen it looks like the PTG etherpad crashed again, did anyone back it up?
#openstack-nova - 2019-05-06
06:53:21 openstackgerrit Boxiang Zhu proposed openstack/nova master: Fix failure to boot instances with qcow2 format images https://review.opendev.org/640271
07:28:55 openstackgerrit Boxiang Zhu proposed openstack/nova master: Remove unsafe convert of lvm imagebackend https://review.opendev.org/642667
08:27:25 openstackgerrit zhufl proposed openstack/nova master: [Trivial fix]Remove unnecessary slash https://review.opendev.org/657342
10:44:00 amansi26 In order to remove this particular warning http://paste.openstack.org/show/750709/ . Do I need to add cafile option at both the places (glance and ssl)? or how is it?
10:59:00 brinzh fatal: unable to access 'https://opendev.org/openstack/nova/': Failed to connect to 2604:e100:3:0:f816:3eff:fe6b:ad62: Network is unreachable
10:59:00 brinzh git review
11:00:24 brinzh How to resolve this error? From the gerrit address changes review.opendev.org, I cannot clone/review and so on...
11:22:55 gibi brinzh: ssh://gibi@review.openstack.org:29418/openstack/nova works for me
11:24:23 brinzh I remember the port of 29418 is forbidden in China
11:24:48 brinzh gibi: Where are you?
11:26:25 brinzh gibi: I will test the ssh to clone, maybe it works ok now :)
11:32:23 gibi brinzh: EU. Are you from China? If yes then I guess my info does not relevant for you
11:34:18 brinzh gibi: Yeah, I am come from China, but port of 29418 is ok, I can clone the code now ^_^, but the 'git review' need to be test later.
11:35:19 brinzh gibi: Ti's process is 86%, thanks for your remind.
11:46:41 brinzh Unfortunately, interrupted...
11:49:29 openstackgerrit Surya Seetharaman proposed openstack/nova master: Microversion 2.73: Support adding the reason behind a server lock https://review.opendev.org/648662
11:53:03 tssurya gibi, alex_xu, gmann: I have added you all to review https://review.opendev.org/#/c/648662/ for the notification/API parts respectively
13:22:31 openstackgerrit Merged openstack/nova master: [Trivial fix]Remove unnecessary slash https://review.opendev.org/657342
13:42:31 efried o/
14:29:51 mriedem hi
14:59:00 gibi tssurya: ack
14:59:57 gibi tssurya: the size of that patch is a bit intimidating
15:10:08 mriedem the locked reason one?
15:10:14 mriedem anything involving API samples is going to be big
15:14:51 gibi mriedem: yes, that one. But I started reading it...
15:26:42 openstackgerrit Merged openstack/nova stable/queens: libvirt: Stop ignoring unknown libvirtError exceptions during volume attach https://review.opendev.org/657050
15:32:42 openstackgerrit Merged openstack/nova stable/queens: libvirt: Always disconnect volumes after libvirtError exceptions https://review.opendev.org/657111
16:06:36 amansi26 fungi:In order to remove this particular warning http://paste.openstack.org/show/750709/ . Do I need to add cafile option at both the groups (glance and ssl)? or how is it?
16:24:01 mriedem amansi26: it appears that way yes https://docs.openstack.org/nova/latest/configuration/config.html#glance.cafile
16:32:29 mriedem fyi i saw a grenade job fail because it took 202 seconds to spawn an instance http://logs.openstack.org/61/621061/5/check/grenade-py3/6aabc8e/logs/screen-n-cpu.txt.gz#_May_05_16_41_08_772233 and the tempest timeout is 196 seconds
16:32:32 mriedem maybe just a slow node
16:33:31 cdent oh noes, i can't find it
16:36:38 efried cfriesen: on TPM, did we decide to do implicit version?
16:40:48 efried => required=$version absent, hw:sev_model=$model present => nova adds required=$version, where $version = 2_0 if $model is CRB else 1_2
16:40:48 efried => required=$version present, hw:sev_model absent => TIM at $version
16:40:48 efried => required=$version present, hw:sev_model=$model present => $model at $version
16:40:48 efried so
16:42:37 efried dah, tpm_model, not sev_model :)
16:46:06 cdent yeah, I'm going to need you to come in on saturday
16:46:22 efried that's tps_report
16:46:25 cfriesen efried: yeah. I started looking at the implementation yesterday and to avoid code duplication I'd have to move a bunch of code around into helper functions
16:46:42 efried really?
16:46:44 efried okay.
16:46:44 mriedem i know this is a bit fugly, but can we at least get another core on the (non-fugly and quite beautiful) recreate test for this stein regression? https://review.opendev.org/#/q/topic:bug/1825034+(status:open+OR+status:merged)
16:47:01 efried mriedem: looking.
16:48:03 mriedem thanks. we at least have 1-2 regression fixes merged back to stable/stein so it'd be good to do a release soonish
16:55:42 cfriesen efried: both the version and the model can be specified in both the image and the flavor, so I've got helper functions down in the libvirt driver to retrieve them. We'd need to move those up to where they're accessible from the sched request filter, which in turn means mucking around with test cases, etc.
16:56:43 efried cfriesen: Oh, okay, well, I would have likely -1'd anyway requesting that the validation be done in the request filter :)
16:58:10 efried No reason to wait until we get all the way to the compute to check for conflicting values, if we can fail at the scheduler
16:58:35 cfriesen efried: yesterday I was looking at whether we could do validation in _validate_flavor_image*.
16:59:01 efried yeah, could. But that doesn't mean you shouldn't *also* do it in the request filter.
16:59:31 efried oh, maybe it does
16:59:54 cfriesen validate all the things everywhere. :)
17:00:08 cfriesen might be overkill
17:00:44 efried yeah, that does validation, which is fine, but you would have to duplicate some of the logic to do the implicit required trait and/or model business.
17:00:59 efried probably not a huge amount of overlap, actually.
17:01:18 efried Anyway, the right place for all of that logic is *not* the virt driver.
17:02:33 cfriesen yeah, it's high-level enough that it would apply to any implementation in other virt drivers
17:02:34 efried um, the logic to do implicit model lives in the virt driver though.
17:03:51 cfriesen I'll rework the change. Hopefully soonish.
17:04:10 efried no hurry, I just wanted to record the design we decided on before I forgot it :)
17:23:53 efried ...I'll fast approve
17:23:53 efried # Since the above (archive stuff) removed the fake instance, do the migration again to recreate it so we can exercise the code path.
17:23:53 efried $archive stuff
17:23:53 efried # The following (archive stuff) is used to prove that the migration created a "fake instance". It is not necessary to trigger the bug.
17:23:53 efried mriedem: https://review.opendev.org/#/c/653098/ <== If you want to add a comment, like:
17:25:13 openstackgerrit Dan Smith proposed openstack/nova master: Add a workaround config toggle to refuse ceph image upload https://review.opendev.org/657078
17:53:01 openstackgerrit Matt Riedemann proposed openstack/nova master: Add regression test for bug 1825034 https://review.opendev.org/653098
17:53:02 openstack bug 1825034 in OpenStack Compute (nova) "listing deleted servers from the API fails after running fill_virtual_interface_list online data migration" [High,In progress] https://launchpad.net/bugs/1825034 - Assigned to Matt Riedemann (mriedem)
17:53:53 mriedem efried: done
17:53:55 efried mriedem: +A
17:53:58 mriedem thanks
17:54:03 efried mriedem: Not sure if the fix needs a rebase now?
17:54:13 openstackgerrit Matt Riedemann proposed openstack/nova master: Exclude fake marker instance when listing servers https://review.opendev.org/653158
17:54:33 mriedem ^
17:54:47 efried neat
17:54:49 efried +A
18:01:36 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: Add regression test for bug 1825034 https://review.opendev.org/657420
18:01:38 openstackgerrit Matt Riedemann proposed openstack/nova stable/stein: Exclude fake marker instance when listing servers https://review.opendev.org/657421
18:01:38 openstack bug 1825034 in OpenStack Compute (nova) "listing deleted servers from the API fails after running fill_virtual_interface_list online data migration" [High,In progress] https://launchpad.net/bugs/1825034 - Assigned to Matt Riedemann (mriedem)
18:14:11 rm_work mriedem: correction -- apparently we use the log driver and ship notifications to splunk via logs, so no RMQ involved >_>
18:15:58 mriedem rm_work: ah ok, i don't hear of many people doing that
18:17:15 rm_work yeah, apparently some of our team didn't realize either... I had no idea what we were doing on the nova side of things at all, but finding out more about our deployments daily by asking around :D
18:45:07 openstackgerrit Merged openstack/nova master: Add regression test for bug 1825034 https://review.opendev.org/653098
18:45:08 openstack bug 1825034 in OpenStack Compute (nova) stein "listing deleted servers from the API fails after running fill_virtual_interface_list online data migration" [High,In progress] https://launchpad.net/bugs/1825034 - Assigned to Matt Riedemann (mriedem)
18:59:58 openstackgerrit Jay Pipes proposed openstack/nova master: add InstanceList.get_all_uuids_by_hosts() method https://review.opendev.org/623557
18:59:59 openstackgerrit Jay Pipes proposed openstack/nova master: single pass instance info fetch in host manager https://review.opendev.org/623558
20:38:42 redkrieg does anyone know how I determine which image property the nova scheduler ImagePropertiesFilter is failing a compute host for?
20:47:35 efried redkrieg: Debug logging switched on for n-sch?
20:48:17 efried or
20:48:17 efried "advertised by the compute node",
20:48:17 efried "but no corresponding supported_instances are "
20:48:17 efried LOG.debug("Instance contains properties %(image_props)s, "
20:48:17 efried you'd be looking for one of these:

Earlier   Later