| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-08-19 | |||
| 11:11:15 | openstackgerrit | Andreas Jaeger proposed openstack/nova stable/rocky: Remove experimental job on openSUSE 42.3 https://review.opendev.org/677176 | |
| 11:12:52 | openstackgerrit | Andreas Jaeger proposed openstack/nova stable/queens: Remove experimental job on openSUSE 42.3 https://review.opendev.org/677177 | |
| 11:14:04 | openstackgerrit | Andreas Jaeger proposed openstack/nova stable/pike: Remove experimental job on openSUSE 42.3 https://review.opendev.org/677178 | |
| 11:15:20 | alex_xu | stephenfin: I replied your email, but not sure I understand the problem fully | |
| 11:16:10 | openstackgerrit | Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851 | |
| 11:26:18 | openstackgerrit | Adam Spiers proposed openstack/nova master: Make it easier to run a selection of tests relevant to ongoing work https://review.opendev.org/677180 | |
| 11:26:23 | aspiers | stephenfin, alex_xu: happy Monday ;-) ^^^^ | |
| 11:29:57 | openstackgerrit | Adam Spiers proposed openstack/nova master: Make it easier to run a selection of tests relevant to ongoing work https://review.opendev.org/677180 | |
| 11:50:08 | openstackgerrit | Adam Spiers proposed openstack/nova master: Switch some GitHub URLs to point to opendev.org https://review.opendev.org/677185 | |
| 12:21:33 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix wrong assertions in unit tests https://review.opendev.org/677102 | |
| 12:23:33 | yonglihe | stephenfin: Hi, good morning. | |
| 12:27:46 | openstackgerrit | Takashi NATSUME proposed openstack/nova master: Tests: autospecs all the mock.patch usages https://review.opendev.org/470775 | |
| 12:37:46 | sean-k-mooney | yonglihe: i have not seen stephenfin online yet today | |
| 12:38:17 | sean-k-mooney | he proably is around but im guessing he is still catching up on email/gerrit after the weekend | |
| 12:42:04 | stephenfin | no, I'm here | |
| 12:42:28 | stephenfin | (sean-k-mooney: See discussion with bhagyashris from earlier :)) | |
| 12:42:57 | sean-k-mooney | stephenfin: ah ok sorry | |
| 12:43:07 | sean-k-mooney | yep i missed that | |
| 12:44:10 | stephenfin | yonglihe: Those patches are on my list, but there's a big queue of stuff ahead of them. I'll get to it eventually :) | |
| 12:47:12 | openstackgerrit | Brin Zhang proposed openstack/python-novaclient master: Microversion 2.77: Support Specifying AZ to unshelve https://review.opendev.org/665136 | |
| 12:49:49 | sean-k-mooney | stephenfin: did you reach any decision with the path forward for PCPU upgrades | |
| 13:01:05 | stephenfin | sean-k-mooney: I'm still on the fence, tbh | |
| 13:01:37 | sean-k-mooney | i think the global + 3 cycle process will be the safest | |
| 13:01:38 | stephenfin | I'm leaning towards the global config option still, but we're going to need some way to ensure people don't toggle it on the compute node first | |
| 13:01:55 | sean-k-mooney | ya | |
| 13:03:18 | sean-k-mooney | i think our best way to prevent that is to work with the intaller project to support enabling it | |
| 13:03:27 | stephenfin | I think so too | |
| 13:03:36 | stephenfin | But I've no idea how to fix things if people do it wrong | |
| 13:03:49 | stephenfin | We're kind of handing people a loaded gun here | |
| 13:03:53 | sean-k-mooney | e.g how to rever a reshape | |
| 13:04:19 | sean-k-mooney | ya it is. its not the only one we have given people in the past | |
| 13:04:27 | stephenfin | Pretty much | |
| 13:04:31 | stephenfin | Or how to keep reshaping | |
| 13:05:18 | sean-k-mooney | have we talked to the OSA,kolla or triplo folk about it yet? | |
| 13:06:07 | sean-k-mooney | i know we plan to go over the triplo stuff internally with our deployment folks but did any of the install project comment on your ml post | |
| 13:06:41 | stephenfin | efried: I've started discussing it with TripleO guys, of course | |
| 13:06:54 | stephenfin | But not OSA or Kolla. I don't think Kolla supported pinned CPUs as a rule atm | |
| 13:07:21 | stephenfin | I recall being in a call where there was talk of it at a previous summit but I don't know how much further it went | |
| 13:09:46 | stephenfin | efried: I've noticed that there are many more gaps in test coverage with that privsep series. Are you okay with a giant follow-up to close said gaps? | |
| 13:09:57 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add extra spec parameter and image property for memory encryption https://review.opendev.org/664420 | |
| 13:10:14 | stephenfin | Cos I'll have to recuse myself from reviews if I go add said missing tests to each patch | |
| 13:10:38 | stephenfin | unless you're happy to single +2 the tests, that is | |
| 13:10:43 | efried | stephenfin: I have no doubt there are gaps in test coverage overall, but what bugs me is the coverage being reduced by each patch. Is that what you're talking about? | |
| 13:10:44 | aspiers | stephenfin: responses to your feedback there ^^^ | |
| 13:10:56 | stephenfin | efried: Yeah, that specifically | |
| 13:11:11 | stephenfin | i.e. missing tests in nova/tests/unit/privsep | |
| 13:11:13 | stephenfin | aspiers: sweet | |
| 13:11:17 | efried | In that case, yes, a fup is good for me, but I'll want to wait to merge a given patch until its coverage is at least proposed. | |
| 13:11:50 | aspiers | stephenfin: but you might find more enjoyment in https://review.opendev.org/#/c/677180/ | |
| 13:12:00 | efried | and yes, good plan wrt reviews, since it's tough to convince other cores to care about that series, we should take advantage of the two of us being willing to dig into it :P | |
| 13:12:01 | aspiers | efried: you too :) | |
| 13:12:41 | aspiers | my dream is never having to *manually* run tests locally again | |
| 13:12:52 | aspiers | just edit files and watch things turn red or green | |
| 13:13:07 | aspiers | it's entirely doable | |
| 13:13:14 | aspiers | I've done it with many other projects | |
| 13:13:17 | stephenfin | efried: Yeah, of course. Maybe just keep an eye out for gaps as you're reviewing so I've something to hit "Done" on | |
| 13:14:25 | efried | wfm | |
| 13:15:59 | efried | aspiers, stephenfin: Love this idea, add it to the list of workflow improvements it's hard to get people to pay attention to: https://review.opendev.org/#/c/673356/ | |
| 13:16:29 | aspiers | efried: personally I *never* tire of workflow improvements | |
| 13:16:39 | aspiers | you can always count me in for reviewing those | |
| 13:18:04 | efried | aspiers: what's mybranch^! ? | |
| 13:18:21 | aspiers | the commit at the tip of mybranch | |
| 13:18:31 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Exclude build request marker from server listing https://review.opendev.org/677201 | |
| 13:19:02 | aspiers | efried: https://git-scm.com/docs/git-rev-parse#_other_rev_parent_shorthand_notations | |
| 13:19:21 | stephenfin | Well it's not a workflow improvement, but this would save us some time https://review.opendev.org/#/c/470775/ | |
| 13:19:52 | stephenfin | (make mock fail if you call a mocked function incorrectly or a function that doesn't exist) | |
| 13:20:35 | stephenfin | and it's been around in one form or another for...2 years, 2 months and 17 days :'D | |
| 13:21:05 | efried | yeah, I thought claudiub had been working on this for ages and was never able to get it quiiite right. | |
| 13:21:09 | efried | is it right now? | |
| 13:21:19 | efried | I guess you think so according to your +2 :) | |
| 13:21:48 | stephenfin | It all looks sane and I'm relying on CI to catch anything we might have missed | |
| 13:21:55 | stephenfin | so...yes. | |
| 13:22:47 | efried | aspiers: So foo^! is like foo~1..foo ? | |
| 13:24:59 | efried | I admit, I learned like two of the weird shorthands and then cordoned off the rest of my brain for other things. | |
| 13:25:13 | efried | 80/20 rule | |
| 13:27:01 | sean-k-mooney | woudl foo~1..foo not be git show | |
| 13:27:16 | sean-k-mooney | e.g. compare ing n-1 to n | |
| 13:27:41 | sean-k-mooney | which is the diff of n or what git show displays | |
| 13:27:48 | efried | if you're using it for a diff, yes | |
| 13:27:55 | sean-k-mooney | ok | |
| 13:28:01 | sean-k-mooney | just makeing sure | |
| 13:28:55 | sean-k-mooney | the ~1 thing always confused me | |
| 13:29:15 | sean-k-mooney | i only ever use it if im doing "git reset --hard HEAD~1" | |
| 13:29:37 | sean-k-mooney | to remove the top commit form my branch when i commit to master by mistake | |
| 13:33:34 | efried | when I commit to master by mistake, I do something like: | |
| 13:33:34 | efried | git branch -m oops | |
| 13:33:34 | efried | git checkout -b master origin/master | |
| 13:35:54 | stephenfin | Somewhat related https://github.blog/2019-08-16-highlights-from-git-2-23/ | |
| 13:36:40 | stephenfin | (git 2.23 adds experimental 'git restore' and 'git switch' commands that each do a bit of what 'git checkout' is doing currently) | |
| 13:47:03 | mriedem | gibi: answered your question about multi-cell migration reschedule https://review.opendev.org/#/c/635668/41 | |
| 13:47:23 | yonglihe | stephenfin: Thanks, its stressful. | |
| 13:47:23 | mriedem | gibi: tl;dr yes it's a todo, i just didn't do it yet since i wanted to get the main flow written, tested and reviewed first | |
| 13:52:45 | gregwork | what kind of access does a tenant need to create private flavors within their tenant | |
| 13:54:50 | gregwork | getting ERROR: You are not authorized to use resource_types: OS::Nova::Flavor when deploying a stack in this tenant as _member_... I also get this error as ResellerAdmin and Creator | |
| 13:55:30 | gregwork | the "admin" role appears to transform the tenant into a cloud operator able to see and modify all other tenants so this is not useful | |
| 14:03:31 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Address nits from privsep series https://review.opendev.org/677210 | |
| 14:03:35 | stephenfin | efried: ^ | |
| 14:05:17 | efried | ack | |
| 14:17:02 | mriedem | gibi: now that there has been more review on the cross-cell-resize series, i'm thinking about moving https://review.opendev.org/#/c/642590/ and https://review.opendev.org/#/c/642183/ and https://review.opendev.org/#/c/637605/ to the front of the series to get those merged out of the way (next time i do a rebase) - any problems with that? | |
| 14:20:10 | mriedem | dansmith: this is a pretty simple +2d bottom of the cross-cell series which is the prep_resize equivalent and would help move a few things https://review.opendev.org/#/c/633293/ - it's basically just a call to the dest compute to do a resize_claim for pci/numa and send back the migration context | |