| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-25 | |||
| 17:13:30 | dansmith | right, but it means you don't get tested against ours, and we don't get tested with yours | |
| 17:13:30 | jlvillal | But yeah, not with cinder, glance, nova, etc... | |
| 17:14:28 | jlvillal | True. We did that after we migrated over. | |
| 17:14:37 | mriedem | can someone turn on the sdague signal | |
| 17:14:43 | jlvillal | Not sure if the tradeoffs are worth it or not. | |
| 17:15:43 | jlvillal | Too bad they don't have a 'depend' on queue. Instead of everyone goes in the same bucket queue | |
| 17:17:17 | dansmith | jlvillal: depends-on between queues just makes it one queue no? | |
| 17:18:57 | jlvillal | dansmith, Not a gerrit depends-on for one. | |
| 17:19:29 | jlvillal | dansmith, Now if 'ironic' depends on 'nova'. And 'glance' depends on 'nova' But ironic and glance don't depend on each other. | |
| 17:20:01 | jlvillal | dansmith, So in the gate queue a failed glance wouldn't cause the ironic job to get re-queued. | |
| 17:20:32 | jlvillal | dansmith, They do not have this idea of a 'depend' in zuul queues. There is just a queue. | |
| 17:20:42 | jlvillal | It was a random idea. | |
| 17:21:03 | dansmith | I see what you mean, but I'm not sure the benefit is worth that complexity | |
| 17:21:54 | jlvillal | dansmith, Yeah probably right. I just saw your nova jobs that have been 18 hours + in the gate queue. So that is why I mentioned it. | |
| 17:22:11 | dansmith | yeah, we're definitely feeling that pain, but.. | |
| 17:34:49 | mriedem | gibi: once the two approved versioned notification transformation changes are merged https://review.openstack.org/#/q/topic:bp/versioned-notification-transformation-queens+status:open i'm going to close that bp and open another one for rocky for anything left | |
| 17:38:06 | mriedem | so is bauzas out today? | |
| 17:38:22 | dansmith | apparently | |
| 17:38:38 | dansmith | he's not on our calendar as being out, but I've pinged him a few times yesterday and today already and haven't seen any response | |
| 17:39:03 | dansmith | he was doing some conf thing earlier in the week, but I'm surprised he hasn't popped back up yet | |
| 17:48:15 | mriedem | oh right snowbunnies | |
| 17:55:23 | jackie-truong | dansmith: RE: https://review.openstack.org/#/c/537728/ | |
| 17:55:33 | jackie-truong | dansmith: I wasn't sure if we needed to reassign instance._context | |
| 17:55:44 | jackie-truong | dansmith: The more I look at it, I'm thinking "no" | |
| 17:55:49 | jackie-truong | dansmith: Should I remove? | |
| 17:56:11 | dansmith | jackie-truong: it's done on L342 why would you have to do it again? | |
| 17:56:37 | jackie-truong | dansmith: i was a little confused as to how things were getting passed around | |
| 17:56:57 | dansmith | jackie-truong: you're calling that method :) | |
| 17:57:13 | dansmith | jackie-truong: it's set on instance above, then you pass it to another method, so... it's still set | |
| 17:57:28 | jackie-truong | dansmith: kk i'll remove | |
| 17:59:14 | jackie-truong | outlook pop-ups ;-) | |
| 17:59:49 | dansmith | I figured you had an army of interns, one patch per intern, tasked with rapid-fire refreshing until midnight | |
| 18:00:08 | jackie-truong | nope, that was just me sitting on the couch, eating nachos | |
| 18:00:22 | dansmith | hah nice | |
| 18:05:05 | openstackgerrit | Jackie Truong proposed openstack/nova master: Reduce complexity of _from_db_object https://review.openstack.org/537728 | |
| 18:05:26 | dansmith | jackie-truong: why do you keep pushing these up one by one? do you not have them in a proper stack? | |
| 18:06:57 | jackie-truong | dansmith: i don't think i know what i'm doing when it comes to tying the patches together for gerrit | |
| 18:07:16 | dansmith | jackie-truong: so you're doing what? rebasing the second one on the first, then the third on the second, etc? | |
| 18:07:19 | dansmith | all in separate branches? | |
| 18:07:24 | mriedem | git rebase -i | |
| 18:07:26 | mriedem | is your friend | |
| 18:07:28 | dansmith | yeah | |
| 18:07:34 | mriedem | rebase the entire series at once, | |
| 18:07:40 | mriedem | modify the patches you need | |
| 18:08:16 | mriedem | https://docs.openstack.org/infra/manual/developers.html#rebasing-a-commit | |
| 18:08:22 | jackie-truong | oh. i was definitely doing "rebasing the second one on the first, then the third on the second, etc"... | |
| 18:08:23 | dansmith | then you'll hate us a little less for asking them to be split up :) | |
| 18:08:40 | mriedem | step 2 there can be 'git rebase -i master' | |
| 18:08:48 | mriedem | in which case you get a menu to select the commits you want to edit | |
| 18:09:00 | dansmith | jackie-truong: so do your rebase thing like normal this time, but before you submit, rebase the whole thing on master and then try "git rebase -i master" and see what it gives you | |
| 18:09:26 | izombie | I'm trying to setup nova api manually and paste api seems to evasive. When I'm trying to list servers, I'm getting 404. Any suggestions on what could be the reason? | |
| 18:09:28 | dansmith | i.e. git checkout master; git pull; git checkout last-patch-branch; git rebase -i master | |
| 18:10:25 | jackie-truong | yeah, so i usually rebase on master (git rebase -i master), but that's just for one patch | |
| 18:10:57 | jackie-truong | so you're saying checkout the api patch in this case, then rebase on master | |
| 18:11:45 | dansmith | jackie-truong: well, you have modified the bottom one now, so you need to do your normal rebase one by one thing | |
| 18:11:50 | dansmith | jackie-truong: but then once that is done, | |
| 18:12:03 | dansmith | only use the api patch's branch from now on, and run git rebase -i master from that one | |
| 18:12:20 | jackie-truong | dansmith: got it, thanks! | |
| 18:12:21 | dansmith | jackie-truong: it'll show you all your patches from master on, and let you edit them without changing the stack | |
| 18:12:45 | jackie-truong | so much time has been wasted on individual pushes... | |
| 18:12:47 | dansmith | jackie-truong: then when you git review, it'll show you all the ones pending, and you'll see them all go up in one go here in channel | |
| 18:13:05 | dansmith | jackie-truong: ....yeah :) | |
| 18:14:18 | openstackgerrit | Claudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage https://review.openstack.org/447505 | |
| 18:16:49 | mriedem | git review -y to push the entire stack | |
| 18:16:52 | mriedem | it will prompt you though | |
| 18:16:55 | mriedem | if you don't use -y | |
| 18:17:06 | mriedem | -R if you don't want to rebase unchanged patches at the bottom the stack | |
| 18:17:12 | mriedem | but maybe that's getting advanced | |
| 18:17:14 | dansmith | I never use -y | |
| 18:17:20 | mriedem | git rebase -i HEAD~3 etc | |
| 18:17:31 | mriedem | i use git rebase -R -y quite often | |
| 18:17:31 | dansmith | because if I have messed up the rebase, I might resubmit a year's worth of changes | |
| 18:31:32 | melwitt | mriedem: I'm good with this ironic "get IP address for volume connector" change if you wanted to revisit it https://review.openstack.org/#/c/468353 | |
| 18:39:03 | mriedem | melwitt: ok | |
| 18:52:29 | hrw | hello again | |
| 18:53:00 | hrw | stephenfin: https://paste.fedoraproject.org/paste/2PNsiE0OF0VYnF~sKHdXdw is initial version of 'lets add usb host' patch | |
| 18:53:12 | hrw | stephenfin: I though it will be more complicated | |
| 18:54:53 | mriedem | melwitt: wording question in https://review.openstack.org/#/c/537642/ | |
| 18:56:01 | mriedem | edleafe: i don't think your alternate hosts change is ever going to get out of the check queue... | |
| 18:57:07 | edleafe | mriedem: maybe we need alternate CI | |
| 18:57:10 | edleafe | :) | |
| 19:01:49 | openstackgerrit | melanie witt proposed openstack/nova master: Add release note for QEMU native LUKS decryption https://review.openstack.org/537642 | |
| 19:05:06 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: [dnm] Make sure that we have USB Host controller for tablet https://review.openstack.org/538003 | |
| 19:06:25 | hrw | please take a look as I am not sure is it proper way of doing that | |
| 19:10:23 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Ironic: Get IP address for volume connector https://review.openstack.org/468353 | |
| 19:10:24 | mriedem | melwitt: check my release note update on ^ and see if that's still OK, then i'll +W | |
| 19:13:27 | melwitt | mriedem: looks good to me | |
| 19:14:27 | mriedem | want to put your +2 back on? | |
| 19:14:40 | melwitt | yep done | |
| 19:18:37 | mriedem | efried: so i should defer https://blueprints.launchpad.net/nova/+spec/granular-resource-requests to rocky right? | |
| 19:18:47 | mriedem | i think that was mentioned as a rocky thing at this point in the meeting today | |
| 19:19:26 | efried | mriedem It can't be finished in Q, because dependent on nrp alloc cands work, which probably also won't make Q. But 95% of the code is done and merged (months ago). | |
| 19:19:33 | efried | mriedem So however you want to paperwork that... | |
| 19:19:54 | mriedem | can't functionality use it yet from the API right? | |
| 19:19:57 | efried | correct | |
| 19:20:01 | mriedem | ok, then i'll defer | |
| 19:20:03 | efried | ack | |
| 19:24:53 | mriedem | mgoddard: so on the nova ironic traits stuff, | |
| 19:25:01 | mriedem | it looks like the ironic API changes are done https://review.openstack.org/#/q/topic:bug/1722194+(status:open+OR+status:merged) | |
| 19:25:18 | mriedem | and the ironicclient changes to support it are approved, but not yet released, and today is client release freeze | |