| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-11-18 | |||
| 22:05:32 | sean-k-mooney | it would be good to read that and provide feedback | |
| 22:06:54 | sean-k-mooney | changing how this works is problematic as alot of the code and external tooling assumes flavor and image are the way resouce requests and qualitive aspects of an instance are defiened | |
| 22:07:52 | sean-k-mooney | that does not mean we cant discuss changing it its just hard not to break things in this if we do | |
| 23:39:19 | openstackgerrit | Merged openstack/nova master: Move rng device checks to the appropriate method https://review.opendev.org/693210 | |
| #openstack-nova - 2019-11-19 | |||
| 02:11:42 | openstackgerrit | Huachang Wang proposed openstack/nova-specs master: Use PCPU and VCPU in one instance https://review.opendev.org/668656 | |
| 02:15:51 | ianw | with the drop python2 change @ https://review.opendev.org/#/c/687954/ ... it get that you got to switch at some point. but devstack doesn't default to python3, so this basically breaks the default case | |
| 02:16:07 | ianw | i feel like it has been done in the wrong order | |
| 02:19:23 | ianw | for example, https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/install-devstack doesn't work any more | |
| 02:28:43 | openstackgerrit | Ian Wienand proposed openstack/nova master: Revert "Stop testing Python 2" https://review.opendev.org/694891 | |
| 02:38:31 | brinzhang_ | ianw: I met the same issue with you, an run many times, the devstack cannot up easily ... | |
| 02:52:09 | sean-k-mooney | there are patches to change the default in devstack but we have to wait untill after m1 to change it | |
| 02:52:26 | sean-k-mooney | devstack also has had support for python3 for years | |
| 02:52:55 | sean-k-mooney | its trivial to set USE_PYTHON3=true | |
| 02:53:41 | ianw | personally i don't see that as a good argument for breaking the defaults | |
| 02:54:45 | sean-k-mooney | we have untill december to drop python 2 testing for non lib projects | |
| 02:55:41 | sean-k-mooney | the only functional change in that patch is the lack of installing contextlib2 | |
| 02:56:09 | sean-k-mooney | i guess the requirement for py36+ as well | |
| 02:56:30 | ianw | yes, and that's the sticking point for mine | |
| 02:57:03 | sean-k-mooney | leaving it work until december 12th? | |
| 02:57:49 | sean-k-mooney | the Ussuri release is not expected to have python 2 support in general | |
| 02:57:59 | brinzhang_ | Other project can do this until december 12th? it's not only exist in nova | |
| 02:58:21 | ianw | brinzhang_: well, yes, but nova is probably the biggest target :) | |
| 02:58:43 | ianw | sean-k-mooney: my suggestion would be to have devstacking using USE_PYTHON3 by default, then switch it | |
| 02:58:47 | sean-k-mooney | nova is proably the most visable | |
| 02:59:26 | ianw | switching to USE_PYTHON3 by default and waiting, say, a week, should probably give any issues time to shake out | |
| 02:59:42 | sean-k-mooney | you mean merge https://review.opendev.org/#/c/649097/ | |
| 03:00:25 | ianw | sean-k-mooney: yes, well work out the issues related to that and get it in | |
| 03:00:39 | ianw | as you say, many (most) people won't even notice the default changing, probably | |
| 03:00:44 | brinzhang_ | I agree | |
| 03:01:56 | sean-k-mooney | i know most deveolpoer i have talked to moved there default a release or two ago | |
| 03:02:27 | sean-k-mooney | not all but alot of devstack user have been using python3 since rocky or stien | |
| 03:03:21 | brinzhang_ | I think there are problem need to consider, about the base environment. In older centos/ubuntu it's dafault python is py27 | |
| 03:03:51 | ianw | sean-k-mooney: i think there's probably a long tail of gate jobs that people haven't converted | |
| 03:04:11 | sean-k-mooney | yep proably | |
| 03:04:21 | sean-k-mooney | we will find out shortly if they start exploding. | |
| 03:05:04 | sean-k-mooney | i think there was a seperate patch up to swap the defualt in devstack gate and anoutther in zuul devstack base job but i dont know what the state of those are | |
| 03:05:26 | brinzhang_ | at least, because of this change I have not build the devstack env now, it's terrible .. | |
| 03:05:52 | sean-k-mooney | brinzhang_: unstack and stack with python3 enabled | |
| 03:06:52 | sean-k-mooney | we proably could have watied to merge it for another week or too | |
| 03:07:17 | sean-k-mooney | but we do need to complete this this cycle | |
| 03:07:48 | sean-k-mooney | and as far as i am aware this is inline with the tc plan for the dropping of python 2 support | |
| 03:08:58 | ianw | not going to argue about the overall dropping of it at all. just it seems a gentler approach to have devstack defaulting to python3, then switch projects to stop supporting python2 | |
| 03:09:25 | ianw | i mean, realistically, with nova refusing to install on py2 it's not even a configuration option in devstack after that point | |
| 03:10:41 | sean-k-mooney | that is fair. althoguh all project were ment to have gate jobs running on python 3 last cycle so all project shoudl be able to at least run on python 3 | |
| 03:10:49 | sean-k-mooney | also not all porject or jobs deploy nova | |
| 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 | |