| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-19 | |||
| 18:09:05 | bbansal | as it tried to copy this and there were no such file present | |
| 18:12:44 | sdague | bbansal: ah, gotcha | |
| 18:22:31 | openstackgerrit | Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459 | |
| 18:29:07 | openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Microversion 2.52 - Support tag when boot https://review.openstack.org/484617 | |
| 18:29:50 | cdent | jaypipes: i gotta dash, but ought to be later if you wanna chat about that weirdness gibi uncovered | |
| 18:30:37 | mriedem | sdague: https://review.openstack.org/#/c/484617/ should be good to go | |
| 18:35:24 | sean-k-mooney | jaypipes: stephenfin mriedem: o/ https://review.openstack.org/#/c/485304 is the review for the new os-vif release. we still need to merge https://review.openstack.org/#/c/485125 but it has been rebased on master so the commit id will not change. | |
| 18:35:40 | sdague | mriedem: looking | |
| 18:36:08 | jaypipes | sean-k-mooney: I'm going to make those little nit corrections I just dinged jangutter for since I think jangutter is afk | |
| 18:36:09 | sean-k-mooney | jaypipes: did you forget to save the suggestions on https://review.openstack.org/#/c/485125/ | |
| 18:36:15 | sdague | mriedem: that's a fun new 3.5 minute test in tempest I noticed | |
| 18:36:18 | mriedem | sean-k-mooney: merge commit? | |
| 18:36:18 | jaypipes | sean-k-mooney: no... | |
| 18:36:27 | mriedem | sdague: what's fun | |
| 18:36:27 | mriedem | ? | |
| 18:36:32 | jaypipes | sean-k-mooney: just got delayed :) | |
| 18:36:45 | sdague | mriedem: build 3 volumes and 2 networks and plug plug plug | |
| 18:36:50 | jangutter | jaypipes, sean-k-mooney: jangutter is not afk, but operating at about 20% caffeine. | |
| 18:36:50 | sdague | for 3+ minutes | |
| 18:36:57 | mriedem | sdague: did i add it? | |
| 18:37:08 | sean-k-mooney | jaypipes: i dont see them in gerrit i assume you were talking to him on irc | |
| 18:37:20 | sdague | mriedem: I don't know, I was just trying to figure out why we were close to 2 hours on runs, and that one jumped out as long and new | |
| 18:37:25 | sdague | I didn't look at the authorship | |
| 18:37:36 | mriedem | ok, your context switch from server tags to that threw me off | |
| 18:37:47 | mriedem | i killed a spider on my porch | |
| 18:37:51 | mriedem | and it's babies | |
| 18:38:00 | jaypipes | sean-k-mooney: guh, was on a prior revision. :( | |
| 18:38:02 | jaypipes | sean-k-mooney: sec | |
| 18:38:08 | sean-k-mooney | mriedem: i had asked jangutter to rebase on master so the that commit id would not change and i would not have to update the sha in the commit to the release repo. | |
| 18:38:12 | sean-k-mooney | jaypipes: no worries | |
| 18:39:15 | mriedem | sean-k-mooney: if it's got new features and stuff it should be a minor version bump | |
| 18:39:51 | sean-k-mooney | mriedem: i could bump to 1.7 but the bump to 1.6 was mainly for the hardware offload feature | |
| 18:40:12 | sean-k-mooney | 1.6.1 was just removing some vendor specific assumtions but more or less the same | |
| 18:40:31 | mriedem | sean-k-mooney: there is a new feature in there, and there is a new min bump on some dependencies | |
| 18:40:34 | mriedem | so it's 1.7 | |
| 18:40:35 | mriedem | or bust | |
| 18:40:44 | mriedem | +oslo.config!=4.3.0,!=4.4.0,>=4.0.0 # Apache-2.0 | |
| 18:41:05 | sean-k-mooney | mriedem: ill have to respin the patch any when jaypipes or jangutter respins so cool ill make it 1.7.0 | |
| 18:41:29 | jaypipes | sean-k-mooney: one sec. pushing now. | |
| 18:41:40 | openstackgerrit | Jay Pipes proposed openstack/os-vif master: Improve OVS Representor VF Lookup https://review.openstack.org/485125 | |
| 18:41:45 | jaypipes | sean-k-mooney: ^ | |
| 18:41:59 | sean-k-mooney | jaypipes: cool thanks jay | |
| 18:42:53 | jangutter | sean-k-mooney, jaypipes: thanks! | |
| 18:44:05 | jangutter | jaypipes: amazingly, moving those regex comment lines up didn't break anything else. I regard regexes as "write-only-code", so I tend to leave my future self notes as to what they should do. | |
| 18:44:15 | sean-k-mooney | mriedem: https://review.openstack.org/#/c/485304 should now be up to date | |
| 18:45:07 | sean-k-mooney | jangutter: yes regexes to tend to be like perl, write once debug never | |
| 18:49:14 | sean-k-mooney | moshele seams to be offline and the new melonox ci dose not seam to be commenting back on this patchset. | |
| 18:50:08 | sean-k-mooney | do we want to leave moshele have the final approval/wait for the ci to comment on https://review.openstack.org/#/c/485125 | |
| 18:50:50 | mriedem | melwitt: some nits in https://review.openstack.org/#/c/446243/ | |
| 18:51:26 | melwitt | mriedem: cool, thanks | |
| 18:52:41 | jaypipes | sean-k-mooney: approved it. not a high risk thing. | |
| 18:53:24 | jangutter | sean-k-mooney: if it fails on the Mellanox CI, then there's a missing test case... | |
| 18:55:04 | sean-k-mooney | jaypipes: cool well hopefully we can do the release and bump the upper constriants and global requirement tomorrow once https://review.openstack.org/#/c/485304 is merged. | |
| 18:55:51 | sean-k-mooney | im going to log off for the night so ill chat to ye tomorrow | |
| 18:58:58 | jangutter | ditto, thanks very much! | |
| 18:59:11 | mriedem | i'll take care of that today | |
| 18:59:59 | sean-k-mooney | mriedem: the requirement or was that in responce to something else | |
| 19:01:14 | mriedem | to your stuff | |
| 19:01:21 | mriedem | i will make all your dreams come true | |
| 19:01:50 | mriedem | https://www.youtube.com/watch?v=Ps7tVvQHLyo | |
| 19:03:20 | mriedem | TheJulia: so for the bfv change, is there a passing ci for it in https://review.openstack.org/#/c/472740/ ? | |
| 19:03:51 | mriedem | oh i see, | |
| 19:04:03 | mriedem | answer is 'no' because the project-config changes for the experimental queue job aren't landed | |
| 19:04:18 | jaypipes | cfriesen: yet another reason the resize workflow (and its manual confirm/revert step) is antithetical to cloudy automation :( | |
| 19:05:17 | mriedem | cfriesen: you guys have some people doing perf testing against master? | |
| 19:05:27 | mriedem | like at least rally testing? | |
| 19:06:58 | mriedem | jlvillal: left a suggestion in https://review.openstack.org/#/c/470307/ | |
| 19:07:10 | mriedem | jlvillal: there are ways to test things using devstack-gate before they land as a job in project-config | |
| 19:07:57 | jlvillal | mriedem, Thanks. Yeah I know about the hack devstack-gate thing :) | |
| 19:08:21 | jlvillal | mriedem, We actually have a different patch which I think is ready to merge | |
| 19:08:25 | TheJulia | mriedem: your own investigation beat me to answering | |
| 19:08:27 | mriedem | btw, what's the difference between https://review.openstack.org/#/c/480332/ and https://review.openstack.org/#/c/470307/ ? | |
| 19:08:37 | mriedem | jlvillal: that's what i was wondering | |
| 19:08:42 | jlvillal | mriedem, https://review.openstack.org/#/c/480332/ | |
| 19:08:42 | mriedem | can the merge conflict one be dropped | |
| 19:08:55 | jlvillal | mriedem, I think so. I'll leave a comment | |
| 19:09:14 | mriedem | cool, because i'd like to see this series actually pass ci somewhere before we approve it in nova | |
| 19:09:50 | TheJulia | mriedem: derekh was working on it and then had to disappear for a week or two so hshiina picked it up and made a new patch | |
| 19:09:59 | mriedem | ok | |
| 19:10:11 | TheJulia | 470307 likely just needs to be abandoned at this point | |
| 19:10:19 | TheJulia | since it is just creating confusion | |
| 19:10:52 | mriedem | sdague: can you abandon? https://review.openstack.org/#/c/470307/ | |
| 19:11:39 | sdague | mriedem: done | |
| 19:12:07 | TheJulia | thanks sdague | |
| 19:14:39 | openstackgerrit | melanie witt proposed openstack/nova master: Deprecate useless quota_usage_refresh from nova-manage https://review.openstack.org/446243 | |
| 19:14:43 | openstackgerrit | melanie witt proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | |
| 19:14:47 | openstackgerrit | melanie witt proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | |
| 19:25:18 | mriedem | melwitt: doc build failed on the man page https://review.openstack.org/#/c/446243/43/doc/source/cli/nova-manage.rst | |
| 19:25:21 | mriedem | you need a blank line in there | |
| 19:25:25 | mriedem | otherwise +2 | |
| 19:25:37 | melwitt | oh, dangit | |
| 19:27:16 | melwitt | forgot about the tox docs target | |
| 19:27:55 | openstackgerrit | Merged openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559 | |
| 19:28:07 | openstackgerrit | Merged openstack/python-novaclient master: Add resize down test which also verifies quota changes https://review.openstack.org/484151 | |
| 19:28:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: claim resources in placement API during schedule() https://review.openstack.org/483566 | |
| 19:28:13 | openstackgerrit | Jay Pipes proposed openstack/nova master: add dict of allocation requests to select_dests() https://review.openstack.org/483565 | |
| 19:28:14 | openstackgerrit | Jay Pipes proposed openstack/nova master: Handle None returned from get_allocation_candidates due to connect failure https://review.openstack.org/484988 | |
| 19:28:32 | jlvillal | mriedem, As an FYI, I have asked for reviews on https://review.openstack.org/#/c/480332/ , but I know infra is pretty busy | |
| 19:28:49 | mriedem | ok | |
| 19:29:08 | jaypipes | mriedem, dansmith, bauzas, edleafe: rebased the placement-claims stuff on top of mriedem's Handle None returned from get_allocation... patch and addressed all of your review comments. | |