| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-11-19 | |||
| 03:10:57 | sean-k-mooney | alot do but not all | |
| 03:12:47 | sean-k-mooney | projects like keysone, swift and glance wont need nova in ther gate jobs in many cases. although i suspect in many cases they get it by default | |
| 03:13:19 | sean-k-mooney | anyway night o/ we proably coudl revert it temporailly but i would personally prefer to fix the broken jobs | |
| 03:20:02 | brinzhang_ | sean-k-mooney: can you provide some guide how to "unstack and stack with python3 enabled"? | |
| 03:27:12 | brinzhang_ | sean-k-mooney: I have encountered this problem several times, excute "python -m pip install --upgrade pip" and also raised the same issue. http://paste.openstack.org/show/786318/ | |
| 03:44:37 | ianw | brinzhang_: you want to set USE_PYTHON3 in your devstack localrc | |
| 04:05:08 | openstackgerrit | Dong Ma proposed openstack/nova master: ZVM: Implement update_provider_tree https://review.opendev.org/694897 | |
| 04:41:32 | openstackgerrit | Merged openstack/nova stable/pike: Functional reproduce for bug 1852207 https://review.opendev.org/693817 | |
| 04:41:32 | openstack | bug 1852207 in OpenStack Compute (nova) pike "reschedule ignores requested availability zone" [Medium,In progress] https://launchpad.net/bugs/1852207 - Assigned to Balazs Gibizer (balazs-gibizer) | |
| 05:37:07 | brinzhang | ianw: thanks, I will try :) | |
| 06:05:52 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: ksa auth conf and client for Cyborg access https://review.opendev.org/631242 | |
| 06:05:52 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add Cyborg device profile groups to request spec. https://review.opendev.org/631243 | |
| 06:05:53 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Define Cyborg ARQ binding notification event. https://review.opendev.org/692707 | |
| 06:05:53 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | |
| 06:05:54 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Get resolved Cyborg ARQs and add PCI BDFs to VM's domain XML. https://review.opendev.org/631245 | |
| 06:05:54 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735 | |
| 06:05:55 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999 | |
| 06:05:55 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Refactor to extract Placement helper functions for functional tests. https://review.opendev.org/694906 | |
| 06:43:11 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova stable/train: Imported Translations from Zanata https://review.opendev.org/694911 | |
| 06:51:45 | openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/694717 | |
| 08:23:04 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Create and bind Cyborg ARQs. https://review.opendev.org/631244 | |
| 08:23:05 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Get resolved Cyborg ARQs and add PCI BDFs to VM's domain XML. https://review.opendev.org/631245 | |
| 08:23:05 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Delete ARQs for an instance when the instance is deleted. https://review.opendev.org/673735 | |
| 08:23:06 | openstackgerrit | Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999 | |
| 08:35:48 | openstackgerrit | Merged openstack/nova stable/pike: doc: fix and clarify --block-device usage in user docs https://review.opendev.org/694450 | |
| 08:42:00 | bauzas | good morning Nova | |
| 08:44:49 | gibi | bauzas: good morning | |
| 08:51:56 | brinzhang_ | Hi gibi, good morning | |
| 08:52:16 | gibi | brinzhang_: o/ | |
| 08:53:43 | brinzhang_ | gibi: Can you review the flavor explosion patach, and sean-k-mooney leave some comments, and we have the different opinion in the model design | |
| 08:53:58 | gibi | brinzhang_: it is on my list to review | |
| 08:54:28 | brinzhang_ | gibi: yeah, thanks. | |
| 08:57:26 | bauzas | brinzhang_: can you ping me again the spec URL ? | |
| 08:57:54 | bauzas | I'll start reviewing some changes today hopefully, I'm late with this :( | |
| 08:59:04 | bauzas | gibi: about the placement audit command, it'd be awesome if you could test again with the latest revision | |
| 08:59:04 | bauzas | https://review.opendev.org/#/c/670112/ | |
| 08:59:14 | bauzas | I fixed both issues | |
| 08:59:24 | gibi | bauzas: on my list too :) | |
| 08:59:35 | bauzas | gibi: FWIW, I'm looking at how to fix the placementfixture issue | |
| 08:59:37 | bauzas | gibi: <3 | |
| 09:00:27 | brinzhang_ | bauzas: https://review.opendev.org/#/c/663563/ | |
| 09:01:06 | bauzas | brinzhang_: thanks | |
| 11:16:10 | openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Make '_wait_for_state_change' behave consistently https://review.opendev.org/689180 | |
| 11:16:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Unify '_wait_until_deleted' implementations https://review.opendev.org/689181 | |
| 11:16:11 | openstackgerrit | Stephen Finucane proposed openstack/nova master: functional: Make 'ServerTestBase' subclass 'InstanceHelperMixin' https://review.opendev.org/689182 | |
| 11:22:42 | gibi | sean-k-mooney, bauzas, brinzhang_, dansmith, efried_pto: I left comments in the flavor explosion spec https://review.opendev.org/#/c/663563/ . I tried to summarize Dan's and Eric's problems behind the -2s and my answers to those the problems. Let's see if this helps or not. | |
| 11:27:18 | gibi | stable-cores: it seems that nova-live-migration job is sad on stable/pike I repeatedly get error like https://zuul.opendev.org/t/openstack/build/0a199eeccc334b98a2eaf67998eef8b5/log/job-output.txt#5821 | |
| 11:29:31 | lyarwood | W: The repository 'http://mirror.dfw.rax.opendev.org/ubuntu-cloud-archive xenial-updates/ocata Release' is not signed. | |
| 11:29:47 | lyarwood | eh, why are we using that on stable/pike? | |
| 11:30:17 | gibi | lyarwood: that is a good question | |
| 11:31:40 | lyarwood | I forget where that's all defined tbh | |
| 11:35:57 | gibi | it seems that job uses the legacy-ubuntu-xenial-2-node nodeset | |
| 11:36:18 | gibi | that explains xenial-updates to me | |
| 11:37:41 | gibi | https://github.com/openstack/nova/blob/9f2d653b4050dbff859c656144c1564ea7c6fa60/.zuul.yaml#L66 | |
| 11:37:47 | gibi | https://github.com/openstack/nova/blob/9f2d653b4050dbff859c656144c1564ea7c6fa60/.zuul.yaml#L27 | |
| 13:00:35 | openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: Boot from volume instance rescue https://review.opendev.org/694063 | |
| 13:32:30 | lyarwood | mdbooth: https://review.opendev.org/#/c/693537/ - apologies, thought I had added you to this a while ago, would you mind taking a look later today? Not a complete fix but it should avoid issues in some situations. | |
| 13:39:42 | corvus | efried, stephenfin: can you take a look at https://review.opendev.org/694891 asap please? | |
| 13:40:15 | efried | corvus: saw that go by. So you're saying "devstack isn't ready for you yet"? | |
| 13:41:26 | corvus | efried: yeah. if someone downloads devstack and runs stack.sh, it'll break. | |
| 13:41:27 | sean-k-mooney | there is a patch up to change the defualt value of devstack | |
| 13:41:42 | sean-k-mooney | corvus: sure so why dont we just merge the other patch | |
| 13:42:13 | corvus | sean-k-mooney: https://review.opendev.org/649097 has set it to WIP | |
| 13:42:34 | sean-k-mooney | yes because it was depending on a change that has now merged | |
| 13:42:58 | kashyap | Hey, folks. Now is the time to speak up, if you have comments on this: http://lists.openstack.org/pipermail/openstack-discuss/2019-November/010907.html ("On next minimum libvirt / QEMU versions for "V" release") | |
| 13:43:06 | stephenfin | corvus: Yeah, I was waiting on mriedem to appear and decide what to do | |
| 13:43:12 | stephenfin | as the owner of https://review.opendev.org/#/c/649097/1 | |
| 13:43:23 | stephenfin | mriedem: for context: |
|
| 13:43:26 | corvus | ftr, you don't need to set wip to avoid something with a depends-on being merged. zuul does that for you. | |
| 13:44:00 | sean-k-mooney | i know i think he might have been looking at the greneade failure too | |
| 13:44:55 | mriedem | how are changes in the nova jobs breaking everything else that's using devstack? | |
| 13:44:55 | sean-k-mooney | he set it to wip because he tought it might need to be rebased on https://review.opendev.org/#/c/688799/ | |
| 13:45:20 | corvus | either way (whether we merge the revert or the default change), we should get USE_PYTHON3 out of the .zuul.yaml in nova -- that's the red flag here. basically, if something is required for devstack to work, it should be in the devstack repo instead of nova; we should only set individual job config tweaks in nova's config. | |
| 13:45:20 | sean-k-mooney | mriedem: the chagne prevents nova being installed in python 2 | |
| 13:45:56 | corvus | yeah, the change simultaneously stopped all py2 testing on nova, and broke installing nova on py2. | |
| 13:45:59 | efried | corvus, stephenfin, sean-k-mooney: I +2ed; now just convince mriedem... | |
| 13:46:10 | sean-k-mooney | corvus: USE_PYTHON3=true is not the problem | |
| 13:46:12 | mriedem | these aren't the only jobs setting the USE_PYTHON3=True flag outside of devstack | |
| 13:46:17 | mriedem | neutron has at least one as well | |
| 13:47:00 | mriedem | sean-k-mooney: why can't it install on py2? because of dependencies that were removed? | |
| 13:47:10 | sean-k-mooney | the reson other jobs broke is python-requires = >=3.6 | |
| 13:47:23 | sean-k-mooney | in setup.cfg | |
| 13:47:24 | mriedem | i'd like to know if we (nova) can switch our jobs to use py3 meanwhile leaving in whatever is needed to install nova on py2 until other jobs move over | |
| 13:47:39 | mriedem | so let's revert that | |
| 13:47:55 | sean-k-mooney | if we just remove that it will allow nova to be installed on python 2 | |
| 13:47:56 | efried | But doesn't "stop testing py2" leave us exposed there? | |
| 13:48:00 | mriedem | with a big ass TODO about dropping that when devstack has USE_PYTHON3=True by default | |
| 13:48:36 | mriedem | either we move, or we don't, i don't really care about dropping py2 support, but that's what the TC has been hammering on for yeras | |
| 13:48:38 | mriedem | *years | |
| 13:48:42 | mriedem | so then we do and things break, and omfg | |
| 13:48:45 | slaweq | hi | |
| 13:48:47 | sean-k-mooney | https://review.opendev.org/#/c/694891/1/setup.cfg@9 | |
| 13:48:57 | mriedem | so let's just do the minimum possible to get things working which is revert the setup.cfg change yeah | |
| 13:48:57 | mriedem | ? | |
| 13:48:59 | sean-k-mooney | nova can move | |
| 13:49:05 | slaweq | I just found out that patch https://review.opendev.org/#/c/687954/ which You are now talking about broke also neutron-grenade job | |
| 13:49:17 | sean-k-mooney | if we remove that 1 line other can install it on python2 | |
| 13:49:18 | mriedem | slaweq: yeah we're just talking about what to revert | |
| 13:49:24 | slaweq | this job is py2 job and is running in many projects | |
| 13:50:13 | slaweq | mriedem: yes, I see. I just wanted to mention about this job which I just noticed that is broken | |