| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-07-18 | |||
| 11:40:20 | sean-k-mooney | i think you have had enough heartache form it in the last week or two | |
| 11:40:38 | artom | Ugh | |
| 11:41:05 | kashyap | He was teasing you :D | |
| 11:41:32 | sean-k-mooney | more his bad luck | |
| 11:41:55 | artom | I know, the "ugh" was about the reverts, not the tease | |
| 11:48:46 | kashyap | Nod, figured as much. | |
| 12:26:02 | openstackgerrit | Artom Lifshitz proposed openstack/nova master: migrations: libvirt: remove get_machine_ips() https://review.opendev.org/671471 | |
| 12:26:06 | artom | sean-k-mooney, ^^ | |
| 12:26:23 | artom | stephenfin, ^^ you might like the code removal aspect too :) | |
| 12:26:29 | artom | Not sure if I need a bug for that... | |
| 12:27:31 | sean-k-mooney | ya we proably should have a bug and mention the downstream one | |
| 12:27:53 | sean-k-mooney | :) +0,-55 | |
| 12:31:34 | sean-k-mooney | artom: actully we want to backport it so it need a bug | |
| 12:34:06 | stephenfin | artom: Makes sense. Do you want to backport it? | |
| 12:34:29 | sean-k-mooney | stephenfin: we need this on newton downstream | |
| 12:34:34 | sean-k-mooney | so yes | |
| 12:34:35 | stephenfin | I assume so, given your comment in the bug. If so, what sean-k-mooney said | |
| 12:34:36 | artom | stephenfin, it's for an internal escalation, so yeah | |
| 12:34:36 | stephenfin | ack | |
| 12:35:01 | artom | Busy with household'y stuff, will file bug | |
| 12:35:05 | artom | In a few | |
| 12:35:12 | stephenfin | That's going to be an ugly backport. You've been warned | |
| 12:35:29 | artom | I... I can't tell if you're being serious | |
| 12:35:34 | stephenfin | (look at the "Conflicts With" for an idea of the amount of _unmerged_ patches this touches) | |
| 12:35:45 | stephenfin | Oh, I'm always very serious | |
| 12:35:46 | sean-k-mooney | really? why | |
| 12:35:53 | stephenfin | Serious Steve, I've been called | |
| 12:36:03 | sean-k-mooney | i think that is likely jsut change to the module import in most cases | |
| 12:36:08 | artom | I would think most of those are just whitespace-type things | |
| 12:36:28 | artom | Ie, the excision itself is clean, but git is having trouble because what's around it changed | |
| 12:36:32 | stephenfin | Quite possibly. I didn't check. It doesn't matter much either. If it's got to be done, it's got to be done. | |
| 12:36:56 | sean-k-mooney | stephenfin: also if artom going to be the one doing it ... :) | |
| 12:37:10 | artom | Sheesh guys, team spirit and all that | |
| 12:37:34 | sean-k-mooney | we will review it for you down stream too :P | |
| 12:38:01 | stephenfin | Go team? | |
| 12:38:10 | artom | And I'm sure it'll be done most professionally | |
| 12:38:22 | stephenfin | And that's all the team spirit I have for one day. | |
| 12:40:02 | sean-k-mooney | stephenfin: maybe an infution of teen sprit will help https://www.youtube.com/watch?v=hTWKbfoikeg | |
| 12:45:38 | artom | sean-k-mooney, *fist bump* | |
| 12:45:47 | artom | Noice. | |
| 12:58:11 | kashyap | alex_xu: When you're awake: Mind giving an opinion on the Intel CMT thing? -- https://review.opendev.org/#/c/669129/ | |
| 13:19:29 | alex_xu | kashyap: thanks, I will check that | |
| 13:23:28 | kashyap | Thanks! | |
| 13:35:37 | alex_xu | kashyap: I will send an email double check with team, the last time(after the dublin ptg) I check with team, it said ok to remove it. so just wait few more sec, I will get that back to you. | |
| 13:36:04 | kashyap | alex_xu: Sure, no rush. By "remove it", are you suggesting to remove entire support for it, yeah? | |
| 13:36:44 | alex_xu | kashyap: the entire perf feature, right? I didn't heard any requirement from intel for it now. | |
| 13:36:52 | kashyap | Right, indeed | |
| 13:37:03 | kashyap | As we know, the kernel itself has completely nuked it | |
| 13:42:17 | openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP nova-next: Deploy noVNC from source https://review.opendev.org/671490 | |
| 13:42:38 | lyarwood | melwitt: ^ as discussed, hopefully that's enough to make this work. | |
| 14:04:13 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Allow migrating server with port resource request https://review.opendev.org/671497 | |
| 14:13:37 | stephenfin | dansmith: Off the top of your head, any idea what would cause this? http://paste.openstack.org/show/754546/ | |
| 14:14:09 | dansmith | yes? | |
| 14:14:21 | dansmith | you named the things different in the model and the migration | |
| 14:14:29 | dansmith | pcpus vs pcpus_used apparently | |
| 14:14:43 | efried | kashyap: I use git-restack regularly to manage series | |
| 14:14:48 | dansmith | stephenfin: ^ | |
| 14:15:17 | stephenfin | dansmith: I don't see how though http://paste.openstack.org/show/754547/ | |
| 14:16:49 | dansmith | hmm, so it must be claiming that both of those are missing from one or the other | |
| 14:17:23 | dansmith | oh | |
| 14:17:32 | dansmith | stephenfin: aren't you creat5ing them as nullable=True, | |
| 14:17:39 | dansmith | but they're =false in the model | |
| 14:17:47 | stephenfin | god damn it, yup | |
| 14:18:04 | dansmith | that output from the test looks different than I remember for some reason, but .. yeah | |
| 14:18:20 | stephenfin | Thanks. I've been looking at that for at least 30 minutes 😅 | |
| 14:18:34 | dansmith | np, blinders.. it's a thing | |
| 14:21:54 | kashyap | efried: Good to know. Thanks for the view | |
| 14:22:46 | efried | kashyap: in fact, if you're trying to convince people that gerrit is awesomer than other things, how do other things manage series? Cause gerrit does it awesomely. | |
| 14:24:47 | efried | kashyap: I wrote a tutorial, which might be handy: https://docs.openstack.org/contributors/de/code-and-documentation/patch-series-tutorial.html | |
| 14:25:04 | efried | https://docs.openstack.org/contributors/code-and-documentation/patch-series-tutorial.html | |
| 14:30:00 | kashyap | Excellent; I saw that before! | |
| 14:30:27 | kashyap | efried: I don't think this group might consider Gerrit. I wanted to present where Gerrit shines | |
| 14:30:45 | kashyap | s/!// | |
| 14:31:22 | efried | cool | |
| 14:33:42 | kashyap | efried: So 'git-restack' is not exactly "abandonware" is it? | |
| 14:33:55 | efried | what does that mean? | |
| 14:34:13 | kashyap | Sorry, meanining, nobody has time to tend to it | |
| 14:34:36 | kashyap | [Last commit was from 4 years ago.] | |
| 14:34:37 | efried | oh, it's maintained, insofar as that's necessary. I think someone like fungi or dhellmann owns it. | |
| 14:34:49 | kashyap | It might well be that the project is "done", though. Which is perfectly acceptable | |
| 14:34:56 | kashyap | Right, noted. | |
| 14:34:59 | efried | yeah, it doesn't need much. It's just a thin wrapper around `git rebase -i commit-before-wherever-my-current-series-meets-up-with-master` | |
| 14:35:08 | efried | just saves you figuring out that last piece. | |
| 14:36:00 | fungi | efried: it's a side-project of corvus | |
| 14:36:06 | fungi | (like gertty) | |
| 14:36:12 | efried | ah, k | |
| 14:36:29 | sean-k-mooney | efried: i think its a little smarter tehn that and looks for the parent commit where my current series meets up with its parent branch. but i might be mistaken | |
| 14:36:37 | fungi | ahh, nope, it's official opendev. i forgot | |
| 14:36:43 | fungi | he just initially developed it | |
| 14:37:08 | sean-k-mooney | e.g. i think if you were on stable/stien origically it look for where your feature branch meets stable/stine instead of master | |
| 14:37:23 | efried | kashyap: If you wanted to break that 4-year streak, we did recently discuss adding a feature where you could say `git restack --continue` and it determines whether you need a `commit --amend` or just a `rebase --continue`, which is one place I've gotten bitten in the restack workflow from time to time. | |
| 14:37:29 | fungi | https://review.opendev.org/#/admin/groups/git-restack-core | |
| 14:37:34 | fungi | infra-core approves changes for it | |
| 14:37:50 | kashyap | efried: Hehe, noted. | |
| 14:38:03 | efried | sean-k-mooney: yes, "meets up with its parent branch" is more correct. My hyphenated thingy wasn't long enough already. | |
| 14:38:17 | sean-k-mooney | :) | |
| 14:38:39 | fungi | no pending changes for review on opendev/git-restack that i can see | |
| 14:38:44 | fungi | was someone having problems using it? | |
| 14:39:21 | sean-k-mooney | the tool extention i personly want to write at some point is "git review --quad-diff <gerrit id>" or "git-review -Q" if that is free | |
| 14:40:03 | sean-k-mooney | it would print the diff of the current reve to its base difed againt the previs revies to that revions base | |