| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-02-02 | |||
| 17:20:15 | openstackgerrit | melanie witt proposed openstack/nova master: Don't wait for vif plug events during _hard_reboot https://review.openstack.org/540168 | |
| 17:20:51 | melwitt | mriedem bauwser ^ | |
| 17:24:06 | mriedem | heading to lunch, will take a look when i'm back | |
| 17:24:17 | melwitt | cool, thanks | |
| 17:24:18 | mriedem | btw, do we get a network-changed event at least from neutron during reboot after we unplug the vifs? | |
| 17:24:24 | mriedem | i didn't look for that yet in the logs | |
| 17:25:20 | melwitt | we don't. as sean-k-mooney mentioned, the os-vif unplug method for linuxbridge is a no-op, it just does a pass, so that seems consistent with the lack of any event about it | |
| 17:27:28 | melwitt | here's the unplug http://logs.openstack.org/42/525842/11/check/neutron-tempest-linuxbridge/2502b64/logs/screen-n-cpu.txt.gz#_Jan_29_16_01_37_704617 | |
| 17:56:25 | figleaf | giblet: looks like zuul's environment matches mine: http://logs.openstack.org/86/540386/5/check/openstack-tox-py27/2b58498/job-output.txt.gz#_2018-02-02_17_26_13_388933 | |
| 18:28:37 | sean-k-mooney | melwitt: looking at http://logs.openstack.org/42/525842/11/check/neutron-tempest-linuxbridge/2502b64/logs/screen-n-cpu.txt.gz#_Jan_29_16_01_37_811903 we are configuring libvirt to add the interface to the bridge iteself. the linuxbridge os-vif plugin plug method seams to be doing a lot of work that is not strictly require for neutron and presuable is there for nova network | |
| 18:30:24 | sean-k-mooney | for example it also addes the vm port to the bridge if not already done so and also ensure the bridge existis and has an uplink port (phyical or vlan) added and brings up the bridge | |
| 18:31:25 | sean-k-mooney | the only thing its doing that is really required is setting the mtu and making sure ip adresses are not configured on the tap on the host. | |
| 18:32:05 | sean-k-mooney | i would guess we can make that driver alot smaller once nova-networks support can be dropped. | |
| 19:18:07 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: doc: Add user index page https://review.openstack.org/540494 | |
| 19:18:08 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Migrate "launch instance" user guide docs https://review.openstack.org/540495 | |
| 20:01:14 | openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: fix native luks encryption failure to find volume_id https://review.openstack.org/539739 | |
| 20:18:24 | openstackgerrit | Brianna Poulos proposed openstack/nova master: docs: Add booting from an encrypted volume https://review.openstack.org/540506 | |
| 20:30:52 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix test_get_allocation_candidates tests https://review.openstack.org/540513 | |
| 20:30:54 | mriedem_afk | ^ fixes a gate issue that was introduced yesterday | |
| 20:43:25 | openstackgerrit | Matt Riedemann proposed openstack/nova master: docs: Add booting from an encrypted volume https://review.openstack.org/540506 | |
| 20:43:54 | openstackgerrit | Brianna Poulos proposed openstack/nova master: docs: Add booting from an encrypted volume https://review.openstack.org/540506 | |
| 20:43:59 | mriedem | d'oh! | |
| 20:47:10 | bpoulos | mriedem: looks like you just beat me to it :) | |
| 20:48:17 | mriedem | np, thanks for moving that over | |
| 20:48:44 | imacdonn | wow, race condition ;) | |
| 20:49:29 | fried_rice | leakypipes: It happened again: http://logs.openstack.org/60/531260/27/check/nova-tox-functional-py35/cdf1b02/job-output.txt.gz | |
| 20:52:53 | fried_rice | mriedem: Is https://bugs.launchpad.net/nova/+bug/1747063 / https://review.openstack.org/#/c/540513 a duplicate of https://bugs.launchpad.net/nova/+bug/1747001 / https://review.openstack.org/#/c/540420 ? | |
| 20:52:55 | openstack | Launchpad bug 1747063 in OpenStack Compute (nova) "TestProviderOperations.test_get_allocation_candidates randomly fails AssertionError because of hash seed" [High,In progress] - Assigned to Matt Riedemann (mriedem) | |
| 20:52:56 | openstack | Launchpad bug 1747001 in OpenStack Compute (nova) "Use of parse.urlencode with dict in nova/tests/unit/scheduler/client/test_report.py can result in unpredictable query strings and thus unreliable tests" [Low,In progress] - Assigned to Chris Dent (cdent) | |
| 21:07:02 | tssurya | dansmith, mriedem : after the upgrade to Ocata yesterday and we are observing that "memory_mb_used" in "compute_nodes" tables is not correct. However, the resource_tracker log in the hypervisor is correct. Was wondering if there is a known bug or if you have heard anything similar to this issue? | |
| 21:09:49 | mriedem | fried_rice: yes looks like it, i'll close mine | |
| 21:10:03 | fried_rice | rgr | |
| 21:11:33 | mriedem | tssurya: maybe | |
| 21:11:51 | mriedem | tssurya: likely https://review.openstack.org/#/c/520024/ | |
| 21:11:57 | mriedem | https://bugs.launchpad.net/nova/+bug/1729621 | |
| 21:11:58 | openstack | Launchpad bug 1729621 in OpenStack Compute (nova) "Inconsistent value for vcpu_used" [High,In progress] - Assigned to Maciej Jozefczyk (maciej.jozefczyk) | |
| 21:14:41 | bauwser | melwitt: https://review.openstack.org/#/c/540168/ +2d | |
| 21:14:54 | bauwser | call it a week | |
| 21:14:56 | bauwser | \o | |
| 21:27:38 | tssurya | mriedem : thank you, the issue we have looks related to the one you pointed. | |
| 21:32:32 | openstackgerrit | Matthew Edmonds proposed openstack/nova master: improve support matrix notes https://review.openstack.org/540534 | |
| 21:46:58 | yaaaaarwood | mriedem: evening | |
| 21:47:21 | yaaaaarwood | mriedem: back for a few hours, finally worked out why I didn't see failures due to https://review.openstack.org/#/c/539739/ in my LM tests | |
| 21:48:13 | yaaaaarwood | mriedem: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/migration.py#L153-L159 - without encryption_secret_uuid we just skip adding the encryption XML | |
| 21:48:22 | yaaaaarwood | mriedem: that's impossible to see in the logs at present | |
| 21:48:57 | yaaaaarwood | mriedem: mriedem the LM still completes, the instance just has an encrypted volume attached on the dest | |
| 21:49:51 | mriedem | unencrypted you mean? | |
| 21:50:26 | yaaaaarwood | yamahata: very much encrypted, the encrypted XML decrypts, without it the volume is presented as encrypted to the guest | |
| 21:50:36 | yaaaaarwood | encryption XML* | |
| 21:50:46 | mriedem | so is this another bug? | |
| 21:51:17 | yaaaaarwood | mriedem: no, it's not another bug | |
| 21:51:22 | mriedem | or just, the connection_info['volume_id'] was wrong but the LM tests weren't failing on it? | |
| 21:51:33 | mriedem | b/c ^reasons | |
| 21:52:01 | yaaaaarwood | mriedem: correct, without the volume_id set we didn't lookup the local secret and stash the UUID | |
| 21:52:13 | yaaaaarwood | mriedem: that's then used on the source to update the XML | |
| 21:52:35 | mriedem | ok so https://github.com/openstack/nova/blob/master/nova/virt/libvirt/migration.py#L153 evals to False and we just, whatever | |
| 21:52:36 | mriedem | don't care | |
| 21:53:03 | mriedem | how would a user notice this? not be able to use the encrypted volume? | |
| 21:53:35 | yaaaaarwood | yeah, they would see a LUKS block device within the instance | |
| 21:54:26 | mriedem | ok | |
| 21:54:30 | mriedem | well my patch should be all happy now | |
| 21:55:02 | yaaaaarwood | kk, thanks, I'll try to get it tested in the DNM change pulling in the various tempest and devstack changes now | |
| 21:55:08 | yaaaaarwood | for LM testing | |
| 21:59:01 | openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM: Test LM with encrypted volumes https://review.openstack.org/536350 | |
| 22:09:14 | openstackgerrit | Merged openstack/nova-specs master: trivial: Resolve Python 3 issues https://review.openstack.org/539907 | |
| 22:22:53 | openstackgerrit | AJAY KALAMBUR proposed openstack/nova master: Fix incorrect port unbind request in nova on instance delete https://review.openstack.org/387687 | |
| 22:26:05 | openstackgerrit | AJAY KALAMBUR proposed openstack/nova master: Fix incorrect port unbind request in nova on instance delete https://review.openstack.org/387687 | |
| 23:14:03 | openstackgerrit | Merged openstack/nova master: Don't rely on parse.urlencode in url comparisons https://review.openstack.org/540420 | |
| 23:14:12 | openstackgerrit | Merged openstack/nova master: Add the nova-multiattach job https://review.openstack.org/532689 | |
| 23:22:29 | openstackgerrit | Lee Yarwood proposed openstack/nova master: Enable test_volume_backed_live_migration in tempest https://review.openstack.org/528104 | |
| 23:23:43 | openstackgerrit | Merged openstack/nova master: Don't wait for vif plug events during _hard_reboot https://review.openstack.org/540168 | |
| 23:39:46 | openstackgerrit | Eric Fried proposed openstack/nova master: Ensure resource classes correctly https://review.openstack.org/539738 | |
| 23:39:58 | fried_rice | leakypipes: Still needing some guidance on what to mock for that test ^ | |
| 23:40:06 | fried_rice | (regardless of which approach we land on) | |
| 23:42:46 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Don't wait for vif plug events during _hard_reboot https://review.openstack.org/540555 | |
| 23:55:38 | openstackgerrit | Eric Fried proposed openstack/nova master: Ensure resource classes correctly https://review.openstack.org/539738 | |
| #openstack-nova - 2018-02-03 | |||
| 00:02:11 | openstackgerrit | Eric Fried proposed openstack/nova master: Doc: Nix os-traits link from POST resource_classes https://review.openstack.org/540559 | |
| 00:02:17 | fried_rice | leakypipes: Trivial ^ | |
| 00:04:40 | fried_rice | melwitt: ^ | |
| 00:05:07 | fried_rice | (Not that it's urgent; just hate to have an open bug muddying the waters any longer than it needs to) | |
| 00:31:17 | Spazmotic | Morning | |
| 00:35:55 | Spazmotic | If anyone gets a chance to look at this commit and has any questions, i'll be around all night :) https://review.openstack.org/#/c/538415/ | |
| 00:39:14 | Spazmotic | Would love to get some eyes on it before I go to Korea and cannot do anything with it for 2 weeks | |
| 03:39:11 | openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Encode libvirt domain XML in UTF-8 https://review.openstack.org/522161 | |
| 05:41:13 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Enable NUMA Features for Libvirt/QEMU Driver https://review.openstack.org/533077 | |
| 07:04:46 | openstackgerrit | Andreas Jaeger proposed openstack/nova master: Zuul: Remove project name https://review.openstack.org/540326 | |
| 08:46:33 | openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: doc: merge numa.rst to cpu-topologies.rst https://review.openstack.org/540585 | |
| 11:44:43 | openstackgerrit | int32bit proposed openstack/nova master: Refix disk size during live migration with disk over-commit https://review.openstack.org/536351 | |
| 19:00:38 | openstackgerrit | Eric Fried proposed openstack/nova master: Ensure resource classes correctly https://review.openstack.org/539738 | |
| #openstack-nova - 2018-02-04 | |||
| 01:16:51 | Spazmotic | Morning weekend Novaers | |
| 01:16:53 | Spazmotic | Should you exist | |
| 06:33:47 | openstackgerrit | Ameed Ashour proposed openstack/nova master: change instance_system_metadata column type https://review.openstack.org/526900 | |
| 06:34:30 | Spazmotic | I don't understand how this test is successfully completing | |
| 07:02:26 | Spazmotic | O.. i see.. interesting | |
| 07:40:44 | openstackgerrit | Ameed Ashour proposed openstack/nova master: detach instance volumes when VM creation fails https://review.openstack.org/528385 | |
| 09:43:42 | Roamer` | (on some days it's not enough to convince even me) | |
| 09:49:56 | Spazmotic | I'd need to see the test_roamer_exists method to be sure.. | |
| 10:00:04 | Spazmotic | The trouble with tests is the ability to Mock your way to victory.. so really it's unreliable.. i'd need someone else to write that test. When AIs right their own test it's a bad day | |
| 10:09:19 | Roamer` | that's why in some projects at work we don't rely on regression tests only, but we have a test framework that ties into the actual application code, then issues random requests (and records the random seed every now and then) and checks various aspects of the internal state | |
| 10:10:07 | Roamer` | and it's still not enough, because it only checks what it's been written to check - hence one of our lead developers saying "if <the random test framework> does not check it, it has bugs" | |