Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-02-24
21:32:05 diana_clarke LOL, recognize this code anyone? https://twitter.com/redhatway/status/834855421500354561 I wonder who picked this snippet.
21:32:45 diana_clarke someone should port that water bottle to mock
21:33:27 cdent diana_clarke++
21:41:48 mriedem ha
21:41:53 mriedem that's not py35 compliant
22:23:33 NostawRm Any reason there's no option for backups for differential backups for nova/ceph like there is with cinder/ceph other than no one has done it?
23:33:45 openstackgerrit Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/delete #1: no-ops https://review.openstack.org/438119
23:40:20 openstackgerrit Eric Fried proposed openstack/nova master: PowerVM Driver: spawn/delete #1: no-ops https://review.openstack.org/438119
23:54:47 efried mikal ^^ 362LOC! And here's a thing: it _might_ actually pass jenkins/CI.
#openstack-nova - 2017-02-25
01:52:56 openstackgerrit John Garbutt proposed openstack/nova-specs master: Add spec to use Cinder's new attachment APIs https://review.openstack.org/373203
02:06:47 openstackgerrit Sean Dague proposed openstack/nova master: Generate log error message if keystone looks funny https://review.openstack.org/438049
02:45:31 openstackgerrit Sean Dague proposed openstack/nova master: Correct some spelling errors https://review.openstack.org/430646
02:51:21 openstackgerrit John Garbutt proposed openstack/nova-specs master: Add locked-by-service spec https://review.openstack.org/438134
03:08:14 openstackgerrit John Garbutt proposed openstack/nova-specs master: Add policy-docs spec https://review.openstack.org/433010
03:23:25 openstackgerrit John Garbutt proposed openstack/nova-specs master: Add policy-remove-scope-checks spec https://review.openstack.org/433037
10:56:43 openstackgerrit Sean Dague proposed openstack/nova master: Generate log error message if keystone looks funny https://review.openstack.org/438049
12:48:21 openstackgerrit Sean Dague proposed openstack/nova master: Generate log error message if keystone looks funny https://review.openstack.org/438049
12:58:30 openstackgerrit Sean Dague proposed openstack/nova master: api/metadata/vendordata_dynamic: don't import Requests for its constants https://review.openstack.org/434339
15:40:30 openstackgerrit John Garbutt proposed openstack/nova-specs master: Add additional-default-policy-roles spec https://review.openstack.org/427872
18:04:56 mriedem miss_hypothermia: i saw your remove check_attach patch failed some bfv tests in the grenade job, looks like some bdm is invalid error
18:09:52 miss_hypothermia mriedem: I saw that too
18:10:16 miss_hypothermia mriedem: I couldn't figure it out yet what exactly the problem is
18:10:47 miss_hypothermia mriedem: does that test run in other jobs as well BTW?
18:10:55 mriedem miss_hypothermia: i know what it is
18:11:03 mriedem http://logs.openstack.org/58/335358/41/check/gate-grenade-dsvm-neutron-multinode-ubuntu-xenial/d3194cc/logs/subnode-2/old/screen-n-cpu.txt.gz?level=TRACE#_2017-02-24_23_47_29_665
18:11:13 mriedem the pike api is reserving the bfv volume and putting it into 'attaching' status
18:11:21 mriedem the instance is running on the ocata n-cpu node
18:11:34 mriedem where check_attach is still a thing down in nova/virt/block_device.py
18:11:47 mriedem and check_attach fails because the volume is in 'attaching' status from the reserve call in the pike api
18:11:52 mriedem and then, kablammo!
18:12:01 mriedem so you're going to have to add that service version thing back in
18:12:50 miss_hypothermia Ohhh ok, forgot about the mixed env from this perspective
18:13:22 mriedem yeah
18:13:23 mriedem me too
18:13:33 mriedem i'm happy the test caught this though
18:14:00 miss_hypothermia But we're still fine from unreserve perspective so I will add that part back and we should be good to go
18:14:17 mriedem agree
18:14:19 miss_hypothermia mriedem: +1, me too!
18:14:36 mriedem btw, you're still hypothermic?
18:14:45 mriedem it's 13F where i'm going
18:16:22 miss_hypothermia Still sitting in an air conditioned place, but 13F sounds waaaay worse for sure! :/ :S
21:02:27 openstackgerrit Sean Dague proposed openstack/nova master: Generate log error message if keystone looks funny https://review.openstack.org/438049
21:06:26 imad_ Hi all, I am getting the following error in libvirt "unable to execute QEMU command 'query-cpu-definitions': The command query-cpu-definitions has not been found"
21:06:38 imad_ Does anyone have any idea?
21:34:30 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach/detach Cinder flow https://review.openstack.org/330285
21:34:31 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Remove check_attach https://review.openstack.org/335358
21:37:59 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach/detach Cinder flow https://review.openstack.org/330285
21:38:00 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Remove check_attach https://review.openstack.org/335358
21:40:24 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach/detach Cinder flow https://review.openstack.org/330285
21:40:25 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Remove check_attach https://review.openstack.org/335358
23:06:00 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach/detach Cinder flow https://review.openstack.org/330285
23:06:01 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Remove check_attach https://review.openstack.org/335358
#openstack-nova - 2017-02-26
00:00:31 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Remove check_attach https://review.openstack.org/335358
00:00:31 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach/detach Cinder flow https://review.openstack.org/330285
04:52:24 openstackgerrit yuhui proposed openstack/nova master: Add Apache License Content in index.rst https://review.openstack.org/436397
04:56:41 openstackgerrit yuhui proposed openstack/nova master: Add Apache License Content in index.rst https://review.openstack.org/436397
06:10:51 openstackgerrit Danfeng proposed openstack/nova master: Fix typos detected by toolkit misspellings. https://review.openstack.org/438232
12:14:23 openstackgerrit Sean Dague proposed openstack/nova master: api/metadata/vendordata_dynamic: don't import Requests for its constants https://review.openstack.org/434339
15:36:01 miss_hypothermia mriedem: I fixed the issues and added a small test too for the service version check
15:36:33 miss_hypothermia mriedem: smcginnis had a question about when to clean up the code, I hope that's the last issue we need to figure out there and otherwise the fix is good to go :)
15:39:46 mriedem ok i'll take a look
15:43:46 mriedem miss_hypothermia: you're right, it's pike-eol
15:44:53 miss_hypothermia mriedem: ok, cool, tnx
15:56:57 openstackgerrit Kevin Benton proposed openstack/os-vif master: [wip] https://review.openstack.org/438272
16:13:12 mriedem miss_hypothermia: found some test issues
16:15:50 miss_hypothermia mriedem: I'll check the patch if you added comments about it there
16:16:27 mriedem yup still going
16:20:49 miss_hypothermia mriedem: did you find anything that's complicated enough to talk about it here?
16:21:08 mriedem it's not really complicated, just an oversight, but actually something i brought up long long ago i think
16:21:20 mriedem cinder.API.check_attach would raise InvalidVolume,
16:21:41 mriedem we're now calling cinder.API.reserve_volume, which raises InvalidInput if cinder gives a 400
16:22:03 mriedem the tests were mocked to raise InvalidVolume from reserve_volume, but that's not actually what would happen in reality
16:22:23 mriedem so _validate_bdm needs to handle both InvalidVolume and InvalidInput
16:23:38 miss_hypothermia mriedem: oh, ok, so one extra error handling and fix mock in the test
16:23:55 miss_hypothermia mriedem: I will fix it in the afternoon, when I get to the airport
16:24:13 miss_hypothermia thanks for pointing it out
16:25:58 smcginnis mriedem, miss_hypothermia: Commented on there, but if we only care about Ocata talking to newer versions, I'm having a brain block as to why we need to wait for pike-eol.
16:27:15 mriedem smcginnis: you know master is pike now right?
16:27:23 smcginnis mriedem: Yes.
16:27:25 mriedem smcginnis: this is a change to the nova api in pike
16:27:37 mriedem so ocata computes won't have the change introduced here
16:27:45 smcginnis mriedem: And we need to support pre-pike that does not have this change.
16:27:46 mriedem but pike computes will
16:28:03 smcginnis mriedem: So once pre-pike is gone, it's all on the new api.
16:28:24 mriedem which would be ocata-eol, i get what you're saying now
16:28:39 smcginnis mriedem: OK, so I'm not completely insane?
16:28:44 mriedem no
16:28:47 mriedem just partialy
16:28:50 mriedem *partially
16:28:53 smcginnis I can live with that.
16:29:40 mriedem i acked in the review
16:29:43 mriedem miss_hypothermia: comments are in
16:32:49 mriedem i'm going to go numb my brain in front of the tv for awhile now
16:33:48 smcginnis I'm going to go numb my brain and work on my taxes.
17:02:37 miss_hypothermia mriedem: smcginnis: Tnx
17:03:20 miss_hypothermia mriedem: smcginnis: I'll numb my brain at airports and airplanes :)
17:17:22 smcginnis miss_hypothermia: Safe travels!
17:28:00 miss_hypothermia smcginnis: thanks!
18:45:11 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach/detach Cinder flow https://review.openstack.org/330285
18:45:12 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Remove check_attach https://review.openstack.org/335358

Earlier   Later