| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-07-08 | |||
| 17:16:58 | sean-k-mooney | but i can rebase that on top of yours | |
| 17:18:39 | gmann | sean-k-mooney either is fine or once it pass then we can add in 728037 also. | |
| 17:18:52 | sean-k-mooney | sure both works | |
| 17:19:07 | sean-k-mooney | do you know what the plannned eta is for the switch to focal | |
| 17:19:09 | gmann | which can merge independent of focaltion work | |
| 17:19:24 | gmann | focal-migration | |
| 17:19:32 | sean-k-mooney | yep | |
| 17:19:42 | gmann | sean-k-mooney July end for now but i think it might delay. | |
| 17:20:01 | sean-k-mooney | as i said the dep used to be in the nodepool images and its install by devstack so that is why we are just seing this now | |
| 17:20:40 | gmann | yeah, there are lot of updates in images, so not sure if that is one of that. infra team can confirm | |
| 17:32:40 | openstackgerrit | Ghanshyam Mann proposed openstack/os-vif master: DNM: testing os-vif gate on ubuntu focal https://review.opendev.org/738130 | |
| 17:33:11 | openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: Switch legacy Zuul jobs to native Zuul v3 jobs https://review.opendev.org/725163 | |
| 22:39:33 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: Testing system scope in tempest https://review.opendev.org/740124 | |
| #openstack-nova - 2020-07-09 | |||
| 02:39:34 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029 | |
| 02:50:37 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | |
| 02:50:59 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | |
| 03:35:19 | openstackgerrit | XiaolinZhao proposed openstack/nova master: Add UEFI support for architecture MIPS https://review.opendev.org/740151 | |
| 04:34:06 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Function to further validate and retrieve configs https://review.opendev.org/676029 | |
| 07:05:05 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Functions to merge provider configs to provider tree https://review.opendev.org/676522 | |
| 07:09:43 | gibi | good morning nova | |
| 07:11:09 | openstackgerrit | Tony Su proposed openstack/nova master: Provider Config File: Enable loading and merging of provider configs https://review.opendev.org/693460 | |
| 07:24:31 | tony_su | gibi: sean-k-mooney: I am still taking the provider-config-file job and fully supporting its merging into upstream. | |
| 07:25:58 | gibi | tony_su: cool. So I will move the series to a runway slot and we will try to focus on it in the next two weeks | |
| 07:26:09 | gibi | are you available in the next to week to iterate on the series based on our feedback? | |
| 07:27:39 | openstackgerrit | Elancheran S proposed openstack/nova master: Removes the delta file once image is extracted https://review.opendev.org/734421 | |
| 07:27:50 | tony_su | yes, definitely. | |
| 07:28:31 | tony_su | gibi: yes, definitely. | |
| 07:28:38 | gibi | tony_su: cool | |
| 07:28:39 | gibi | thanks | |
| 07:49:43 | huaqiang | gibi: hi can you review https://review.opendev.org/728480? | |
| 07:55:48 | openstackgerrit | Jie Li proposed openstack/nova-specs master: add volume backed server rebuild spec https://review.opendev.org/739349 | |
| 07:55:55 | openstackgerrit | Huaqiang Wang proposed openstack/nova master: doc: correct the link to user/flavor.rst https://review.opendev.org/740176 | |
| 07:57:08 | gibi | huaqiang: is this in the middle of the series? | |
| 07:57:55 | huaqiang | gibi: do you mean https://review.opendev.org/739349 | |
| 07:58:23 | huaqiang | for https://review.opendev.org/739349, yes it is, it belongs to bp/mixed-instance | |
| 07:58:25 | gibi | I mean https://review.opendev.org/#/c/728480/ | |
| 07:58:58 | gibi | I can review https://review.opendev.org/#/c/728480/ but I reviewd the ancestors of that patch in the series already | |
| 07:59:04 | huaqiang | my mistake | |
| 07:59:24 | huaqiang | https://review.opendev.org/#/c/728480/ belongs to bp/mixed-instances | |
| 07:59:43 | huaqiang | Stephen suggests to have your review, | |
| 08:00:22 | huaqiang | let me check | |
| 08:07:36 | gibi | huaqiang: sure I will try to look deeper into the series. I have feedback in https://review.opendev.org/#/c/714701 | |
| 08:07:47 | gibi | and in https://review.opendev.org/#/c/714703 | |
| 08:08:08 | gibi | and also alex_xu has feedback in that series | |
| 08:10:38 | brinzhang0 | gibi: do you have time to review the nova-cyborg patch? https://review.opendev.org/#/c/716186/13 | |
| 08:11:02 | brinzhang0 | gibi: and the cyborg evacuate support patch https://review.opendev.org/#/c/715326/15 | |
| 08:11:59 | brinzhang0 | the cyborg shelve/unshelve support depends on these patch, and these pataches are already to review, thanks | |
| 08:15:32 | huaqiang | gibi: sorry about that, I missed your comments even I go through some of my abandoned changes. | |
| 08:15:49 | huaqiang | gibi: can you have a look again :) | |
| 08:16:09 | huaqiang | gibi: get your message, thanks | |
| 08:20:56 | openstackgerrit | Wenping Song proposed openstack/nova-specs master: Add no user token when get Cyborg client https://review.opendev.org/740184 | |
| 08:22:42 | openstackgerrit | Wenping Song proposed openstack/nova-specs master: Add no user token when get Cyborg client https://review.opendev.org/740184 | |
| 08:29:32 | gibi | brinzhang0: if the cyborg patches are ready then please add them to the runway queue. At the momement I will focus on the 3 series that is currently in the 3 slots. | |
| 08:29:55 | gibi | huaqiang: ack I will look | |
| 08:30:10 | brinzhang0 | gibi: ok, I will add these to the runway queue | |
| 08:31:25 | gibi | brinzhang0: thanks | |
| 08:31:37 | brinzhang0 | gibi: np | |
| 08:32:01 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: Fix os_CODENAME detection and repo refresh during ceph tests https://review.opendev.org/739608 | |
| 08:32:02 | gibi | brinzhang0: also summer vacation time will soon arrives. I will take 3 weeks of during july - august | |
| 08:32:13 | gibi | s/of/off/ | |
| 08:32:34 | gibi | I guess others from EU will do similar things | |
| 08:32:40 | brinzhang0 | gibi: have a holiday ^ | |
| 08:32:49 | gibi | thanks :) | |
| 08:34:56 | lyarwood | elod: https://review.opendev.org/#/c/739608/ should finally fix stable/rocky | |
| 08:44:04 | stephenfin | gibi, elod, lyarwood: Can raise on the team meeting, but what do we think about adding the Backport-Candidate flag to Gerrit like oslo has done? https://review.opendev.org/#/c/740068/ | |
| 08:46:44 | lyarwood | stephenfin: OOoOOOoO shiny | |
| 08:46:58 | lyarwood | stephenfin: +1 from me if we can query on that flag | |
| 08:47:15 | lyarwood | that we must be able to | |
| 08:47:33 | stephenfin | https://review.opendev.org/#/q/label:Backport-Candidate%253D%252B2 | |
| 08:47:34 | stephenfin | Yup | |
| 08:47:39 | stephenfin | (label:Backport-Candidate=+2) | |
| 08:48:37 | gibi | stephenfin: how does this relate to the way we target bugs to series in launchpad? | |
| 08:48:50 | gibi | will we do duble accounting? | |
| 08:48:53 | stephenfin | gibi: what do you mean? | |
| 08:49:24 | stephenfin | oh, as in setting the affected series for a reported bug in LP? | |
| 08:49:36 | gibi | in launchpad I can go to a bug and target it to multpile series (Ussurit, Train...) this way showing that the bugfix needs to be backported to those releases | |
| 08:50:04 | gibi | this also shows _where_ to backport | |
| 08:50:04 | stephenfin | that would sound like double accounting, yes. I guess we could either check both, or decide on which one we'd prefer | |
| 08:50:50 | stephenfin | iiuc we only do that launchpad tracking for some bugs, so we could use the backport-candidate feature for everything, and then be more specific in launchpad if necessary | |
| 08:53:33 | gibi | correct, there are non-bugs that we backport | |
| 08:53:50 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: Fix os_CODENAME detection and repo refresh during ceph tests https://review.opendev.org/740193 | |
| 08:53:52 | gibi | and launchpad does not help there | |
| 08:54:38 | gibi | do you have a good working example for a patch that is backported but would be strange to track it with an LP bug? | |
| 08:55:37 | stephenfin | I wasn't even being that specific. I meant bug reporters often don't set target versions in Launchpad, and I don't tend to set them when reviewing bugfixes | |
| 08:56:30 | stephenfin | But the ability to say "this looks like a fix we probably want to backport" from Gerrit does sound like something I'd use, if only because it doesn't involve context switching | |
| 08:57:34 | stephenfin | so tl;dr: this feels like something we'd collectively use far more, if that makes sense? | |
| 09:01:10 | gibi | stephenfin: I see, so the benefit of the new solution is that the recording of the information is closer to the place where the information is created. That definitely helps | |
| 09:01:42 | stephenfin | Yeah, that's a better way of saying what I'm trying to say :) | |
| 09:02:01 | xiaolin | lyarwood: Do you have time to review https://review.opendev.org/#/c/740151/ , I don not know how to change the branch and proposed a new one against branch master | |
| 09:02:11 | gibi | we can use the gerrit flag, to force a discusson of the question "how far to be backported?" and then the answer can be recorded in LP by assigning series | |
| 09:02:55 | lyarwood | xiaolin: I'll take a look today yes | |
| 09:03:33 | gibi | I'm not against the new flag, I feel it has some cost of re-learning the gerrit interface due to the extra flag a the reply button, but I see the benefit of the flag | |
| 09:03:44 | xiaolin | lyarwood: thanks | |
| 09:10:11 | stephenfin | gibi: I'm already doing that for oslo so not a massive deal for me, personally :) | |
| 09:10:50 | gibi | yeah, I can adapt too :) | |
| 09:31:07 | elod | stephenfin: that could help, yes, it's easy to see if a patch should be backported, in that way | |
| 09:31:44 | elod | stephenfin: i usually check the bugreport as well if a patch was marked to be backported to certain branches, too | |
| 09:32:21 | elod | stephenfin: unfortunately most of the time that detail is not filled in | |
| 09:34:53 | elod | stephenfin: so maybe it's worth adding that option :) | |
| 09:38:40 | hrw | morning | |