Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-02-24
19:14:20 openstackgerrit Balazs Gibizer proposed openstack/nova master: handle uninited fields in notification payload https://review.openstack.org/415857
19:17:03 openstackgerrit Stephen Finucane proposed openstack/nova master: setup.cfg: Add note to enable doc warnings https://review.openstack.org/437926
19:17:03 openstackgerrit Stephen Finucane proposed openstack/nova master: doc: Don't put comments inside toctree https://review.openstack.org/437924
19:21:01 sfinucan johnthetubaguy, mriedem: typos fixed on the PCI weigher spec :) https://review.openstack.org/364468
19:27:32 jaypipes gibi: reviewed... perhaps we need to chat about it.
19:28:30 jaypipes gibi: I was under the impression that all instance notifications >= 1.3 object version would have the flavor.projects attribute set to null, not []...
19:28:58 jaypipes gibi: to indicate that the attribute is not set (versus that there are 0 projects the flavor is "shared with")
19:30:15 mriedem sfinucan: questions inline
19:30:29 sfinucan mriedem: (y)
19:36:33 dansmith jbernard: are you still around?
19:45:50 jaypipes gibi: melwitt's also added a good question in that patch
19:50:50 jbernard dansmith: i am
20:03:44 openstackgerrit Sean Dague proposed openstack/nova master: Generate log error message if keystone looks funny https://review.openstack.org/438049
20:07:51 gibi jaypipes, melwitt: thanks for the reviews. I'm working on to answer / fix your comments
20:42:22 melwitt sean-k-mooney: is this it? https://review.openstack.org/#/c/183876/
20:44:06 openstackgerrit Chris Friesen proposed openstack/nova master: show flavor info in server details https://review.openstack.org/434514
20:56:53 openstackgerrit Balazs Gibizer proposed openstack/nova master: handle uninited fields in notification payload https://review.openstack.org/415857
21:00:47 openstackgerrit Balazs Gibizer proposed openstack/nova master: handle uninited fields in notification payload https://review.openstack.org/415857
21:04:55 openstackgerrit Balazs Gibizer proposed openstack/nova master: handle uninited fields in notification payload https://review.openstack.org/415857
21:08:59 efried mriedem: Per PTG feedback, I'm working on splitting up the first PowerVM driver change set. Do y'all have a preference for whether the topmost one is still 391288 (thus preserving its history and 36 patch sets) or a fresh new one?
21:09:24 efried Just a matter of git paperwork, so no biggie either way to me.
21:09:27 mriedem efried: topmost as in the one that should be passing CI?
21:09:30 efried right
21:09:34 mriedem that works
21:09:37 efried which?
21:09:56 mriedem whatever you love
21:10:05 mriedem as long as it's in the series, either first or the one that's passing
21:10:08 efried :) roger wilco. Thanks.
21:10:23 mriedem just don't drop it b/c then we lose the history
21:10:33 efried Ah, okay, that answers my question.
21:11:04 efried (Oh, I see, you're saying I can slot it anywhere in the series, but it should stay in the series.)
21:30:08 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix typo in config drive support matrix docs https://review.openstack.org/438084
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: Implement new attach/detach Cinder flow https://review.openstack.org/330285
00:00:31 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Remove check_attach https://review.openstack.org/335358
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?

Earlier   Later