| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-05-31 | |||
| 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 | |
| 20:37:48 | dansmith | mriedem: I imagine so | |
| 20:38:12 | mriedem | jaypipes: it's going to have to be 1.5.0 | |
| 20:38:16 | mriedem | because 1.4.x is the ocata series | |
| 20:38:33 | mriedem | yeah https://github.com/openstack/os-vif/blob/stable/ocata/requirements.txt#L5 | |
| 20:38:39 | mriedem | it's ocata that has pbr capped | |
| 20:39:06 | jaypipes | mriedem: https://review.openstack.org/469649 | |
| 20:40:01 | mriedem | thanks - found the weirdo ocata change https://review.openstack.org/#/c/431683/ | |
| 20:43:01 | cdent | mriedem: because clearly you don't have enough to think about, your devstack test of os-traits didn't seem to work: http://logs.openstack.org/36/469636/1/check/gate-tempest-dsvm-py35-ubuntu-xenial/94685c6/logs/devstacklog.txt.gz#_2017-05-31_19_56_23_031 | |
| 20:44:27 | mriedem | cdent: i know, just pushed https://review.openstack.org/469636 | |
| 20:44:50 | cdent | gold star | |
| 20:44:59 | mriedem | root beer scratch n snifff | |
| 20:45:06 | mriedem | the extra f is for fun | |
| 20:45:51 | cdent | I was hoping your previous attempt was going to work because it was so...tidy | |
| 20:48:19 | openstackgerrit | Merged openstack/nova master: Remove mox from nova/tests/unit/api/openstack/compute/test_virtual_interfaces.py https://review.openstack.org/448377 | |
| 21:11:19 | mriedem | jgriffith: ildikov: it works http://logs.openstack.org/18/469618/2/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/39938f9/logs/screen-n-api.txt.gz?level=TRACE#_May_31_19_39_07_441549 | |
| 21:14:16 | ildikov | mriedem: yep, sweet, I was looking into the logs too | |
| 21:14:22 | ildikov | mriedem: tnx! | |