| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-31 | |||
| 18:59:54 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Use 3.27 microversion when creating new style volume attachments https://review.openstack.org/469609 | |
| 19:00:36 | jaypipes | mriedem, dansmith, melwitt: your reviews on https://review.openstack.org/#/c/461574/ and the remainder of the series would be awesome. tia. | |
| 19:01:13 | dansmith | jaypipes: I'll trade you for https://review.openstack.org/#/c/464059 | |
| 19:01:27 | jaypipes | dansmith: accepted. | |
| 19:04:41 | cdent | jaypipes, edleafe: sort of security fix for os-traits https://review.openstack.org/#/c/469625/ | |
| 19:04:47 | ildikov | mriedem: jgriffith: if that's the desired microversion direction we could try to put the attach PoC on top of that chain | |
| 19:04:53 | mriedem | ildikov: that's the idea | |
| 19:05:11 | mriedem | ildikov: https://review.openstack.org/#/c/469618/ is an example | |
| 19:05:19 | mriedem | but would eventually be jgriffith's poc | |
| 19:06:23 | jaypipes | cdent: ty ty ty! | |
| 19:06:49 | ildikov | mriedem: ah ok, missed that one, but that's what I had in mind too | |
| 19:07:35 | mriedem | jaypipes: cdent: i think we can test that with a devstack wip change before it's released | |
| 19:07:35 | ildikov | mriedem: glad we're on the same page :) | |
| 19:07:40 | mriedem | give me a few | |
| 19:07:46 | mriedem | jaypipes: which was your change that was failing on that again? | |
| 19:07:48 | jaypipes | mriedem: cool, that would be awesomesauce. | |
| 19:07:55 | jaypipes | mriedem: sec | |
| 19:08:06 | mriedem | bbq sauce will suffice | |
| 19:08:13 | jaypipes | mriedem: https://review.openstack.org/#/c/462770/ | |
| 19:08:17 | mriedem | ack | |
| 19:08:36 | mriedem | can we rebase that quick? | |
| 19:08:42 | mriedem | i'm going to use it as a depends-on | |
| 19:08:48 | mriedem | by we i mean you | |
| 19:08:57 | openstackgerrit | Chris Dent proposed openstack/nova master: Sync os-traits to Traits database https://review.openstack.org/469578 | |
| 19:09:18 | cdent | mriedem: my version of the same functionality has the same problem but is smaller ^ | |
| 19:10:27 | mriedem | cdent: ok | |
| 19:10:27 | openstackgerrit | Dan Smith proposed openstack/nova master: Target cell on local delete https://review.openstack.org/464059 | |
| 19:10:27 | openstackgerrit | Dan Smith proposed openstack/nova master: Fix live migration devstack hook for multicell environment https://review.openstack.org/458537 | |
| 19:13:46 | edleafe | cdent: reviewed the pkgutil fix. One minor issue and I think it's good | |
| 19:13:54 | cdent | thanks edleafe | |
| 19:15:22 | cdent | dood, can I claim "unrelated change" and "didn't you write that?"? | |
| 19:15:56 | edleafe | cdent: sure, but since you're messing with the code... | |
| 19:16:22 | cdent | perhaps a follow up? | |
| 19:16:28 | edleafe | I was going to fix that, but I noticed it just after it merged | |
| 19:21:16 | jgriffith | ildikov mriedem If you both are good with it and there's no feedback from anybody else I'll just add that to my poc now | |
| 19:22:03 | ildikov | jgriffith: works for me | |
| 19:23:10 | mriedem | jgriffith: let me see if this actually works in my DNM change first | |
| 19:26:07 | jgriffith | yeah, I'm waiting | |
| 19:26:36 | jgriffith | or just rebase it in later... whichever works out | |
| 19:28:23 | edleafe | cdent: for the followup, what about making those vars CONSTANTs? | |
| 19:28:46 | cdent | can do | |
| 19:29:01 | edleafe | I can do it, unless you're already into it | |
| 19:32:13 | openstackgerrit | Merged openstack/nova master: Update policy description for os-volumes https://review.openstack.org/466471 | |
| 19:32:21 | jaypipes | efried: I appreciate the good, verbose code comments in your SSP patches. | |
| 19:32:42 | efried | jaypipes Thanks. | |
| 19:33:31 | cdent | edleafe: already doing it | |
| 19:36:13 | cdent | edleafe: https://review.openstack.org/#/c/469631/ | |
| 19:37:14 | openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/469577 | |
| 19:37:19 | openstackgerrit | melanie witt proposed openstack/nova master: WIP Add project_id and user_id to PUT /allocations https://review.openstack.org/469634 | |
| 19:37:19 | openstackgerrit | melanie witt proposed openstack/nova master: Add database migration and model for consumers https://review.openstack.org/469633 | |
| 19:37:20 | openstackgerrit | melanie witt proposed openstack/nova master: WIP Add GET /usages https://review.openstack.org/469635 | |
| 19:40:37 | mriedem | cdent: jaypipes: testing that os-traits fix here https://review.openstack.org/469636 | |
| 19:42:27 | jaypipes | cool, thx mriedem | |
| 19:43:27 | mriedem | jaypipes: cdent: we should probably get a src tempest dsvm job setup on the os-traits repo | |
| 19:43:39 | jaypipes | mriedem: ++ | |
| 19:43:54 | mriedem | again, by we, i mean the turd in my pocket | |
| 19:44:25 | cdent | working at home going well, I see | |
| 19:44:36 | mriedem | never heard that saying? | |
| 19:45:06 | cdent | yeah, but it was more fun to think you've become so lazy... | |
| 19:45:35 | mriedem | i wouldn't soil these beautiful chinos | |
| 19:50:47 | openstackgerrit | Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322 | |
| 20:05:16 | cfriesen | mriedem: sdague: I think https://review.openstack.org/#/c/434514/ is basically ready, all comments have been addressed. It'd be nice if it didn't sit around for so long that another microversion goes in underneath it. | |
| 20:06:51 | mriedem | sean is camping all week i think | |
| 20:06:56 | cfriesen | ah, okay | |
| 20:07:13 | cfriesen | I just included him since he'd reviewed it previously | |
| 20:07:54 | cfriesen | Also, this has +2 from sfinucan but is waiting on another core: https://review.openstack.org/#/c/461456/ | |
| 20:15:39 | mriedem | jaypipes: were you making an os-vif release for that requirements thread in the ML? | |
| 20:15:57 | jaypipes | mriedem: shit, forgot to do that... | |
| 20:16:08 | jaypipes | mriedem: sorry, will do that now. | |
| 20:16:28 | mriedem | thanks | |
| 20:24:08 | melwitt | jaypipes: does this mean the bottom patch should be fixed, that is, does it have a bug? https://review.openstack.org/#/c/461901/10/nova/objects/resource_provider.py@1028 | |
| 20:24:39 | jaypipes | mriedem: https://review.openstack.org/469643 | |
| 20:25:13 | jaypipes | melwitt: yes... | |
| 20:25:20 | jaypipes | melwitt: I'll need to fix in the bottom. | |
| 20:25:34 | jaypipes | thx dansmith and melwitt for keeping me honest ;) | |
| 20:27:00 | mriedem | jaypipes: how was that version released if not via the openstack/releases repo? | |
| 20:27:43 | jaypipes | mriedem: no idea :( but it clearly wasn't listed in the list of Pike deliverables. | |
| 20:27:54 | mriedem | because it was released in ocata | |
| 20:28:07 | jaypipes | April 29? | |
| 20:28:18 | mriedem | https://github.com/openstack/releases/blob/master/deliverables/ocata/os-vif.yaml#L29 | |
| 20:28:28 | mriedem | https://github.com/openstack/releases/commit/1534b7016c66ba6707b299d58d938c93b6346e65 | |
| 20:28:41 | jaypipes | hmmm. | |
| 20:28:46 | mriedem | we need a 1.4.2 for pike | |
| 20:28:48 | mriedem | to uncap pbr | |
| 20:28:57 | mriedem | that's what the reqs team is asking for anyway | |
| 20:29:21 | jaypipes | mriedem: doh. sorry. ok, coming up. | |
| 20:29:31 | jaypipes | mriedem: sorry, I'm out of my element with the release tooling. | |
| 20:30:18 | mriedem | dansmith: i imagine you're going to run a thrilling cells v2 meeting in 30 minutes? | |
| 20:31:10 | jaypipes | mriedem: so you're saying https://review.openstack.org/#/c/467173/ needs to be merged and then cut a 1.4.2 release of os-vif, yes? | |
| 20:32:46 | mriedem | idk, let me ask | |
| 20:32:59 | jaypipes | mriedem: or are they just asking to cut an os-vif release (i.e. push a 1.4.2 tag on os-vif)? | |
| 20:33:09 | jaypipes | mriedem: on the current sha | |
| 20:33:15 | openstackgerrit | dane-fichter proposed openstack/nova master: Add trusted certificates to InstanceExtras https://review.openstack.org/457711 | |
| 20:33:21 | mriedem | i think ^ but it's unclear to me what pbr cap is being discussed | |
| 20:33:29 | mriedem | since i don't see pbr being capped | |
| 20:33:47 | mriedem | oh it was capped in 1.4.1 https://github.com/openstack/os-vif/blob/1.4.1/requirements.txt#L5 | |
| 20:33:51 | mriedem | something at some point uncapped it | |
| 20:33:54 | mriedem | which is unreleased | |
| 20:34:20 | mriedem | yes so release for current sha | |
| 20:35:30 | mriedem | i don't see where that was uncapped so i'm a bit lost | |