| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-12 | |||
| 21:35:06 | openstackgerrit | Merged openstack/nova master: Revert "Fixes race condition with privsep utime" https://review.openstack.org/641038 | |
| 21:35:14 | mriedem | feel free to proxy my backports as an implicit +2 | |
| 21:35:40 | melwitt | copy that | |
| 21:42:07 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add functional test to delete a server while in VERIFY_RESIZE https://review.openstack.org/615644 | |
| 21:43:10 | mriedem | melwitt: this is another old change that you've reviewed at one point https://review.openstack.org/#/c/541420/ but without it is going to make our eventual data migration for old volume attachments take that much longer | |
| 21:44:42 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP/PoC: Bypass ironicclient for node.get https://review.openstack.org/642899 | |
| 21:45:03 | efried | TheJulia, jroll, dtantsur|afk: This should interest you ^ | |
| 21:45:24 | melwitt | mriedem: oh yeah, I remember that one. it fell off my radar a long time ago. I'll re-queue it | |
| 21:45:53 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add retry_on_deadlock to migration_update DB API https://review.openstack.org/633075 | |
| 21:45:55 | efried | TheJulia, jroll, dtantsur|afk: I decided that other thing - getting ironicclient to accept a pre-made Adapter - was going to be pretty heinous and an unnecessary step on the road to getting ironicclient out of the picture. This is a different approach. | |
| 21:46:25 | TheJulia | efried: ++++++++++++++++++= | |
| 21:47:42 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Create volume attachment during boot from volume in compute https://review.openstack.org/541420 | |
| 21:48:18 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP/PoC: Bypass ironicclient for node.get https://review.openstack.org/642899 | |
| 21:49:34 | mriedem | efried: now that you're an oslo.policy master https://review.openstack.org/#/c/624185/ | |
| 21:49:42 | efried | ayfkm? | |
| 21:51:08 | melwitt | mriedem: speaking of old patches, I have an old patch bug fix that I've been wanting review on, if you could add it to your list. the patch has been verified by an operator https://review.openstack.org/611974 | |
| 21:51:32 | efried | mriedem: I'm not touching it until I see lbragstad's +1 | |
| 21:52:32 | mriedem | melwitt: that looks like something for lyarwood and/or mdbooth to look at first | |
| 21:53:46 | mriedem | hell or even cfriesen since he reported the bug | |
| 21:54:30 | melwitt | ok. I guess I forgot to ask them | |
| 21:56:10 | melwitt | looks like I added them to the review but didn't bug them. I'll do that | |
| 21:56:14 | cfriesen | I can probably confirm that it works, but I don't know the code | |
| 21:56:25 | melwitt | oh, no I didn't. you did. guh | |
| 22:01:20 | lbragstad | mriedem oh - i forgot about that guy | |
| 22:01:30 | lbragstad | looked again, left comments | |
| 22:08:29 | mriedem | oh boy ok that's enough policy for me for one day | |
| 22:23:37 | openstackgerrit | Sam Morrison proposed openstack/nova-specs master: add spec "flavor-classes" https://review.openstack.org/635408 | |
| #openstack-nova - 2019-03-13 | |||
| 00:12:00 | gmann | nova-next job is failing on Bionic migration. i think for TLS console proxy - https://bugs.launchpad.net/nova/+bug/1819794 | |
| 00:12:00 | openstack | Launchpad bug 1819794 in OpenStack Compute (nova) "nova-next job fail on Ubuntu Bionic" [Undecided,New] | |
| 00:13:00 | gmann | Bionic migration will happen tomorrow EOD, either we need to make this job as n-v till fix the issue or fix it before that. | |
| 00:14:54 | gmann | melwitt: efried dansmith bauzas gibi stephenfin ^^ if any one of you know or figure out the issue. | |
| 00:15:05 | melwitt | gmann: thanks. I didn't realize the tls-proxy service was old and to not be used anymore | |
| 00:15:57 | gmann | melwitt: it is enable and working fine for devstack based job but i think nova-next is the only legacy job (running on xenial) which enbale tls-proxy | |
| 00:16:25 | melwitt | we added it as an "easy" way to get tls certs setup in the env, for testing console proxies with tls enabled. BUT, it's been several months that I noticed the job was no longer running console proxies with tls ("cannot find cert") so we're not gaining anything by enabling it for a long time now | |
| 00:17:06 | gmann | yeah it run on xenial. my patch move the job to run on bionic. | |
| 00:19:09 | gmann | rest all legacy job like nova-live-migration, nova-lvm etc run fine but nova-next fail | |
| 00:19:40 | melwitt | ok, I think we need to stop enabling it on nova-next and leave a TODO that we need to find out what's the right way to configure tls certs in test env today | |
| 00:20:21 | gmann | melwitt: ok. let me give try with that if failure is only because of tls cert | |
| 00:21:37 | melwitt | gmann: to be clear, the only reason we were including tls-proxy service was to get the automatic setup tasks that devstack did when it was enabled | |
| 00:22:34 | gmann | melwitt: i see. | |
| 00:22:47 | melwitt | it used to copy the certs to right directories etc. but several months ago, maybe even a year ago, those setup tasks no longer worked and we found our console proxies running without tls (they couldn't find cert that they used to be able to find) | |
| 00:23:05 | melwitt | probably related to the legacy/deprecated nature of tls-proxy service | |
| 00:23:45 | melwitt | so tls-proxy isn't really related directly to our testing in nova-next, so since it's causing a problem, we can take it out. it wasn't helping us anymore anyway | |
| 00:23:55 | gmann | may be, we need to check if it is working in tempest-full. if so then we can wait to move nova-next on devstack based job | |
| 00:24:08 | gmann | sure. | |
| 00:24:48 | melwitt | gmann: nova-next is the only place where we try to test console proxy with tls. because the tls certs etc has to be setup | |
| 00:25:15 | openstackgerrit | Merged openstack/nova master: Allow utime call to fail on qcow2 image base file https://review.openstack.org/641315 | |
| 00:25:23 | openstackgerrit | Merged openstack/nova stable/rocky: Handle missing exception in instance creation code https://review.openstack.org/641401 | |
| 00:25:30 | openstackgerrit | Merged openstack/nova stable/rocky: Fix WeighedHost logging regression https://review.openstack.org/641355 | |
| 00:25:31 | melwitt | we just don't know the right way to do that nowadays. we used to piggyback on tls-proxy to do it | |
| 00:26:04 | gmann | ok | |
| 00:26:57 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: Testing nova legacy jobs on bionic https://review.openstack.org/639017 | |
| 00:27:08 | gmann | melwitt: ^^ if it work then, i will mark nova OK in https://etherpad.openstack.org/p/legacy-job-bionic | |
| 00:27:45 | melwitt | gmann: OK, sounds good. thanks | |
| 00:28:12 | gmann | melwitt: thanks for help and updates on tls thing. | |
| 00:28:44 | melwitt | gmann: np, thanks for the heads up about the failure on bionic | |
| 00:42:02 | openstackgerrit | Merged openstack/nova stable/rocky: ironic: stop hammering ironic API in power sync loop https://review.openstack.org/640771 | |
| 00:42:08 | openstackgerrit | Merged openstack/nova stable/queens: Add description of custom resource classes https://review.openstack.org/619125 | |
| 01:08:29 | mnaser | efried: mind joining #openstack-tc -- just have some questions about powervm state | |
| 01:39:35 | melwitt | mriedem: I just noticed in #openstack-tc they're talking about jobs moving from xenial => bionic and project job failures. I don't understand most of what's in this email by fungi, but I think might know and be able to help: http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003746.html | |
| 01:39:51 | melwitt | *I think you might know | |
| 01:40:04 | fungi | i am happy to explain too | |
| 01:40:22 | fungi | right now the only failure i see for nova is your functional-py35 job | |
| 01:41:21 | fungi | the problem is that when master branch jobs all switch to running on bionic by default, py35 jobs will still need to run on xenial because bionic doesn't provide python3.5 | |
| 01:41:58 | fungi | we solved this for any of the shared/abstract py35-specific jobs by setting them to use an ubuntu-xenial nodeset unconditionally | |
| 01:42:17 | fungi | but nova's functional-py35 job doesn't have any of those as a parent | |
| 01:42:42 | fungi | the options there are to change the parent on the job, or set an explicit nodeset in it to use ubuntu-xenial | |
| 01:43:18 | fungi | either fix should work and is only needed on master since the current stable branches already get their jobs set to use ubuntu-xenial | |
| 01:54:02 | melwitt | fungi: thanks | |
| 01:56:47 | fungi | one possible solution can be found in my edit to nova's .zuul.yaml in https://review.openstack.org/642580 | |
| 02:00:21 | melwitt | thanks | |
| 02:01:12 | fungi | nova gets this somewhat easier than other teams, since proposing changes to different branches of it is how i performed my preliminary testing | |
| 02:18:22 | openstackgerrit | Merged openstack/nova stable/queens: Make host_manager use scatter-gather and ignore down cells https://review.openstack.org/637599 | |
| 02:20:50 | openstackgerrit | melanie witt proposed openstack/nova master: Update nova-tox-functional-py35 parent job https://review.openstack.org/642940 | |
| 03:01:59 | openstackgerrit | Boxiang Zhu proposed openstack/nova master: Trivialfix for help description of images_type https://review.openstack.org/642942 | |
| 04:08:15 | openstackgerrit | Merged openstack/nova stable/queens: Don't emit warning when ironic properties are zero https://review.openstack.org/608611 | |
| 05:14:59 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: Testing nova legacy jobs on bionic https://review.openstack.org/639017 | |
| 05:15:32 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: DNM: Testing nova legacy jobs on bionic https://review.openstack.org/639017 | |
| 06:18:21 | openstackgerrit | Chris Friesen proposed openstack/nova master: Add support for resize and cold migration of emulated TPM files https://review.openstack.org/639934 | |
| 09:07:23 | Kevin_Zheng | ping gmann | |
| 09:15:19 | dtantsur | efried: agreed! | |
| 09:51:59 | openstackgerrit | zhaixiaojun proposed openstack/nova master: Fix bug: non-admin user can not filter instance by instance-uuid https://review.openstack.org/642403 | |
| 09:55:33 | gibi | stephenfin: I'm wondering about the interference between the bandwidth feature and the numa aware vswitches feature | |
| 09:56:35 | gibi | stephenfin: is it a supported case that there are a single physnet with two ovs bridges one bridge connected to a nic on numa0 and the other bridge is connected to a nic on numa1? | |
| 09:57:14 | gibi | stephenfin: if it is supported how nova selects which bridge a port shall be plugged for this physnet? | |
| 10:16:10 | openstackgerrit | Merged openstack/nova stable/pike: Lock detach_volume https://review.openstack.org/637536 | |
| 10:43:31 | frickler | in case others have - like me - difficulties to decide between melwitt and efried for PTL, watch this ML thread before you vote: http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003765.html | |
| 10:45:20 | openstackgerrit | Neil Jerram proposed openstack/nova stable/queens: Allow utime call to fail on qcow2 image base file https://review.openstack.org/642997 | |
| 10:47:03 | openstackgerrit | Neil Jerram proposed openstack/nova stable/rocky: Allow utime call to fail on qcow2 image base file https://review.openstack.org/642998 | |
| 10:49:08 | cdent | frickler: you might have more success if you ask some specific questions. Their nomination statements amount to "some statements explaining why they would be the better PTL" | |
| 10:49:47 | cdent | if you want them to differentiate themselves more, you need to ask them about a specific concern you have or present a scenario that you wonder how they would deal with | |
| 11:26:27 | mriedem | . | |
| 11:33:01 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Allow utime call to fail on qcow2 image base file https://review.openstack.org/643011 | |
| 11:50:50 | openstackgerrit | Adrian Chiris proposed openstack/nova master: Move get_pci_mapping_for_migration to MigrationContext https://review.openstack.org/643023 | |
| 11:50:50 | openstackgerrit | Adrian Chiris proposed openstack/nova master: Allow driver to properly unplug VIFs on destination on confirm resize https://review.openstack.org/643024 | |
| 12:08:40 | openstackgerrit | Merged openstack/nova master: pass endpoint interface to Ironic client https://review.openstack.org/640879 | |
| 12:11:10 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: libvirt: Add workaround to cleanup instance dir when using rbd https://review.openstack.org/628726 | |
| 12:24:44 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Allow utime call to fail on qcow2 image base file https://review.openstack.org/633493 | |
| 12:30:02 | mriedem | tonyb: what do you think about backporting this? https://review.openstack.org/#/c/641351/ | |
| 12:30:07 | mriedem | i asked in the ML as well, but no bites. | |