Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-03-03
15:57:04 gibi I can try to get a look through tomorrow morning
15:59:12 dansmith gibi: cool.. I think test coverage is still a problem, even in some of the cases you've already pointed out, but I've tried to continue to hammer on that
16:03:04 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Add support for stable device rescue https://review.opendev.org/700812
16:03:04 openstackgerrit Lee Yarwood proposed openstack/nova master: virt: Provide block_device_info during rescue https://review.opendev.org/700811
16:03:05 openstackgerrit Lee Yarwood proposed openstack/nova master: api: Introduce microverion 2.83 allowing boot from volume rescue https://review.opendev.org/701430
16:03:05 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Report COMPUTE_RESCUE_BFV and check during rescue https://review.opendev.org/701429
16:03:06 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Support boot from volume stable device instance rescue https://review.opendev.org/701431
16:03:06 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Extract _get_bdm_image_metadata into nova.utils https://review.opendev.org/705212
16:04:50 gibi dansmith: ack
16:05:41 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM - Test stable device rescue tests with BFV instances https://review.opendev.org/710050
16:18:36 mriedem lyarwood: beware this https://review.opendev.org/#/c/710861/
16:20:16 lyarwood mriedem: oh fun
16:20:43 lyarwood mriedem: thanks for the heads up
16:22:45 openstack Launchpad bug 1788014 in OpenStack Compute (nova) rocky "when live migration fails due to a internal error rollback is not handeled correctly." [Medium,Triaged]
16:22:45 mriedem found the related bug https://bugs.launchpad.net/nova/+bug/1788014 so i -1'ed that backport
16:22:54 mriedem looks like the fix never made it to rocky though so it's still an issue there
16:24:43 lyarwood \o/ thanks
17:36:59 openstackgerrit Balazs Gibizer proposed openstack/nova master: WIP: Hey let's support routed networks y'all! https://review.opendev.org/656885
17:37:00 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support routed net during cold migration https://review.opendev.org/711071
19:35:31 openstackgerrit Michael Bayer proposed openstack/nova master: Remove oslo_db.sqlalchemy.compat reference https://review.opendev.org/711093
19:47:32 openstackgerrit Merged openstack/nova master: Pass the actual target in os-assisted_volume_snapshots policy https://review.opendev.org/701841
20:33:38 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Remove VIR_DOMAIN_BLOCK_REBASE_RELATIVE flag check https://review.opendev.org/702021
21:06:36 openstackgerrit Lee Yarwood proposed openstack/nova stable/pike: Remove exp legacy-tempest-dsvm-full-devstack-plugin-nfs https://review.opendev.org/702061
21:45:44 openstackgerrit Merged openstack/nova master: Remove 'nova.image.api' module https://review.opendev.org/702451
21:52:26 openstackgerrit Rafael Weingärtner proposed openstack/nova master: Enhance os-simple-tenant-usage VM hours accounting https://review.opendev.org/711113
21:53:56 openstackgerrit Merged openstack/nova master: Fix two test cases that use side effects in comprehensions https://review.opendev.org/711039
22:00:18 openstackgerrit Rafael Weingärtner proposed openstack/nova master: Enhance os-simple-tenant-usage VM hours accounting https://review.opendev.org/711113
22:06:45 openstackgerrit Rafael Weingärtner proposed openstack/nova master: Enhance os-simple-tenant-usage VM hours accounting https://review.opendev.org/711113
#openstack-nova - 2020-03-04
02:23:00 openstackgerrit Brin Zhang proposed openstack/nova master: Add PATCH volume attachments api to os-volume_attachments https://review.opendev.org/693828
02:23:01 openstackgerrit Brin Zhang proposed openstack/nova master: Add functional tests for PATCH volume attachments API https://review.opendev.org/710965
04:04:49 openstackgerrit Merged openstack/nova master: Unplug VIFs as part of cleanup of networks https://review.opendev.org/663382
04:34:37 prometheanfire new oslo.db breaks nova https://review.opendev.org/711031
06:34:44 openstackgerrit Michael Bayer proposed openstack/nova master: Remove oslo_db.sqlalchemy.compat reference https://review.opendev.org/711093
09:43:58 openstackgerrit John Garbutt proposed openstack/nova master: Ensure we pass a target in admin actions https://review.opendev.org/663095
09:46:41 openstackgerrit John Garbutt proposed openstack/nova master: Pass the actual target in os-availability-zone policy https://review.opendev.org/706691
09:47:07 openstackgerrit John Garbutt proposed openstack/nova master: Add test coverage of existing os-console-auth-tokens policies https://review.opendev.org/706687
09:47:45 hrw morning
09:48:04 openstackgerrit John Garbutt proposed openstack/nova master: Introduce scope_types in os-agents policy https://review.opendev.org/701645
09:48:09 hrw just wanted to inform that we released CirrOS 0.5.0 - https://github.com/cirros-dev/cirros/releases/tag/0.5.0
09:48:13 openstackgerrit John Garbutt proposed openstack/nova master: Add new default roles in os-agents policies https://review.opendev.org/701648
09:48:22 openstackgerrit John Garbutt proposed openstack/nova master: Pass the actual target in os-agents policy https://review.opendev.org/701649
09:48:58 openstackgerrit John Garbutt proposed openstack/nova master: Add test coverage of existing attach_interfaces policies https://review.opendev.org/705126
09:49:40 openstackgerrit John Garbutt proposed openstack/nova master: Remove old policy enforcement in attach_interfaces https://review.opendev.org/705127
09:50:07 openstackgerrit John Garbutt proposed openstack/nova master: Introduce scope_types in os-attach-interfaces https://review.opendev.org/705799
09:54:44 lyarwood kashyap / stephenfin ; https://review.opendev.org/#/c/702021/ - more trivial clean up if you have time today for a review btw
09:58:05 openstackgerrit John Garbutt proposed openstack/nova master: Fix os-os-deferred-delete policy to be admin_or_owner https://review.opendev.org/707457
09:58:14 openstackgerrit John Garbutt proposed openstack/nova master: Add test coverage of existing deferred_delete policies https://review.opendev.org/707455
09:59:42 kashyap lyarwood: Yep, negative-code++
10:18:20 openstackgerrit Brin Zhang proposed openstack/nova master: Add new policy to PATCH update volume API https://review.opendev.org/711194
10:18:20 openstackgerrit Brin Zhang proposed openstack/nova master: Add PATCH volume attachments api to os-volume_attachments https://review.opendev.org/693828
10:19:57 openstackgerrit Brin Zhang proposed openstack/nova master: Add functional tests for PATCH volume attachments API https://review.opendev.org/710965
10:25:04 openstackgerrit Lee Yarwood proposed openstack/nova master: compute: Extract _get_bdm_image_metadata into nova.utils https://review.opendev.org/705212
10:25:04 openstackgerrit Lee Yarwood proposed openstack/nova master: api: Introduce microverion 2.83 allowing boot from volume rescue https://review.opendev.org/701430
10:25:05 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM - Test stable device rescue tests with BFV instances https://review.opendev.org/710050
10:25:05 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Support boot from volume stable device instance rescue https://review.opendev.org/701431
11:46:24 lyarwood https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_b78/706898/6/check/nova-tox-functional-py36/b781ccf/testr_results.html - this is new, TypeError: 'TestOpenStackClient' object is not subscriptable, has anyone seen this?
12:05:54 openstackgerrit Stephen Finucane proposed openstack/nova stable/train: Functional test for UnexpectedDeletingTaskStateError https://review.opendev.org/711210
12:06:00 openstackgerrit Lee Yarwood proposed openstack/nova master: functional: Do not provide self.api to _wait_for_server_parameter within test_bug_1831771 https://review.opendev.org/711211
12:06:06 lyarwood stephenfin: ^ I have no idea how why we are only seeing this failure now, any ideas?
12:07:49 stephenfin Not a clue. Very odd
12:07:57 stephenfin I wonder if that's not being triggered all the time or something
12:12:02 lyarwood it actually passes locally which is super odd
12:19:49 lyarwood right, the race isn't actually being triggered all of the time
12:21:14 lyarwood is there a way to force the functional tests to log in DEBUG (with OS_DEBUG=1) even when they pass?
12:24:26 lyarwood stephenfin: https://github.com/openstack/nova/blob/5aa8df8f5cd4013b47a381ea929edd7ac425810e/nova/tests/functional/regressions/test_bug_1831771.py#L50-L67 - that isn't guaranteed to race everytime we run the test
13:03:40 stephenfin lyarwood: okay, so dodgy test. I'll look into making it a more deterministic before we backport it
13:09:24 lyarwood stephenfin: I've got a change locally to do that, I'll push after lunch
13:15:01 stephenfin ack
13:38:13 openstackgerrit Elod Illes proposed openstack/nova stable/rocky: libvirt: remove conditional on VIR_DOMAIN_EVENT_SUSPENDED_POSTCOPY https://review.opendev.org/711231
13:39:38 openstackgerrit Elod Illes proposed openstack/nova stable/rocky: libvirt: check job status for VIR_DOMAIN_EVENT_SUSPENDED_MIGRATED event https://review.opendev.org/711233
13:40:03 openstackgerrit Lee Yarwood proposed openstack/nova master: functional: Avoid race and fix use of self.api within test_bug_1831771 https://review.opendev.org/711211
13:40:08 lyarwood stephenfin: ^ there you go
13:44:08 openstackgerrit Lucian Petrut proposed openstack/nova master: hyper-v: update support matrix https://review.opendev.org/711234
13:45:40 openstackgerrit Lucian Petrut proposed openstack/nova master: hyper-v: update support matrix https://review.opendev.org/711234
14:07:27 rishabhhpe Hello All, When i am creating instance on a VM which is hosted on nodepool VM .. its not able to ping the DHCP IP and even gateway .. is there any way i can troubleshoot it .. i tried with kernel bridge but it is not working .. is there workarund for this ? as my tempest is failing due to this
14:09:25 openstackgerrit Brin Zhang proposed openstack/nova master: Add new policy to PATCH update volume API https://review.opendev.org/711194
14:10:36 openstackgerrit Brin Zhang proposed openstack/nova master: Add functional tests for PATCH volume attachments API https://review.opendev.org/710965
14:25:59 gibi dansmith: I reviewd the first 3 cyborg integration patch. I still have concerns
14:26:13 dansmith gibi: ack was just catching up
14:26:56 gibi and I have to leave for today
14:27:19 gibi will be off tomorrow but I will check back on Firday
14:28:21 gibi I still left some comments and questions
14:28:27 lyarwood stephenfin: https://review.opendev.org/#/c/666915/ - did you see my review here btw, I think it's just a docs nit really but I might be missing something.
14:36:58 redrobot Hi Nova friends! I'm still looking for one more review on https://review.opendev.org/#/c/680562/
14:38:49 lyarwood efried: https://review.opendev.org/#/c/669121/ - blast from the past but this should be good to land now if you have time to review again.
14:40:52 rishabhhpe Hello All, can anyone please help me on above problem i raised
14:41:01 lyarwood redrobot: The people in this channel don't directly control that project, #puppet-openstack is where you will find them. I've +1'd anyway.
14:41:51 redrobot got it, thanks lyarwood!
14:58:50 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM - Test TEMPEST_EXTEND_ATTACHED_ENCRYPTED_VOLUME https://review.opendev.org/707593
15:25:01 lyarwood https://review.opendev.org/#/c/711211/ - any non-RH cores able to review this fix, appears we are randomly hitting it in the gate at the moment.
15:26:58 stephenfin lyarwood: yup, just hadn't gotten to it now. Will do so before EOD
15:27:03 stephenfin *to it before now
15:27:26 lyarwood stephenfin: ack np
15:27:48 openstackgerrit Stephen Finucane proposed openstack/nova stable/train: Unplug VIFs as part of cleanup of networks https://review.opendev.org/711251
15:27:48 openstackgerrit Stephen Finucane proposed openstack/nova stable/train: Functional test for UnexpectedDeletingTaskStateError https://review.opendev.org/711210
15:27:49 stephenfin meanwhile, care to review these incoming patches? Have the affected functional and unit tests passing now
15:27:58 stephenfin *those (bad timing)
15:28:10 lyarwood yup of course
15:35:56 efried lyarwood: If you want to tweak the commit message, I'll fast approve. If you want to twiddle some of those other things at the same time, ditto (none of them impact logic).
15:37:02 efried lyarwood: gate fix +A

Earlier   Later