Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-02-02
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"
10:11:29 ameeda Hello, how can I delete network from horizon, when I try to delete it, it gives my error "can't delete network..."
10:29:42 Spazmotic Roamer` Unit tests always kind of bother me.. but it's also because i'm so new at them they're still half mysterious
17:04:27 openstackgerrit Matt Riedemann proposed openstack/nova master: Update cells v2 layout doc caveats for Queens https://review.openstack.org/540667
18:23:50 openstackgerrit Merged openstack/nova stable/pike: Don't wait for vif plug events during _hard_reboot https://review.openstack.org/540555
22:02:46 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Add missing encryption_secret_uuid tests https://review.openstack.org/540679
22:04:02 lyarwood mriedem: ^ was the missing test that I had in mind for https://review.openstack.org/#/c/539739/ , feel free to merge it into your change if you agree.
22:15:08 openstackgerrit Lee Yarwood proposed openstack/nova stable/ocata: Don't wait for vif plug events during _hard_reboot https://review.openstack.org/540680
23:36:19 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608
23:36:55 openstackgerrit Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483
23:37:21 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.61 - List/Show all server migration types https://review.openstack.org/430839
23:38:41 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove unnecessary arguments in notification methods https://review.openstack.org/535599
23:39:22 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add functional tests for traits API https://review.openstack.org/524094
23:39:28 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc https://review.openstack.org/528201
23:39:49 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520
23:40:30 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameter order in rebuild https://review.openstack.org/529718
23:40:55 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823
23:41:10 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (1) https://review.openstack.org/531258
23:41:29 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (2) https://review.openstack.org/531266
23:41:46 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (3) https://review.openstack.org/531280
23:48:53 Spazmotic Ugh pre flight sleeplessness
23:49:01 Spazmotic Morning I guess
23:52:28 Spazmotic Wake me up KPOP.. You sweet necter. I guess I should comment my commits that I will be gone for 2 weeks

Earlier   Later