Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-03-17
14:02:05 dansmith does anyone know if that third party CI status page is still up anywhere? I don't seem to be able to find it
14:02:15 dansmith looks like zvm CI has been off the rails for a while as well
14:02:21 dansmith failing changes and the logs are 404
14:35:02 openstackgerrit Brin Zhang proposed openstack/nova master: Add PATCH volume attachments api to os-volume_attachments https://review.opendev.org/693828
14:35:02 openstackgerrit Brin Zhang proposed openstack/nova master: Add new policy to PATCH update volume API https://review.opendev.org/711194
14:35:03 openstackgerrit Brin Zhang proposed openstack/nova master: Add functional tests for PATCH volume attachments API https://review.opendev.org/710965
14:35:51 bauzas dansmith: you meant tritri something ?
14:36:04 dansmith bauzas: yeah
14:36:42 brinzhang_ lyarwood: https://review.opendev.org/#/c/710965/8/nova/tests/functional/api_sample_tests/test_volumes.py I think this is not necessary, and I cannot get the response too.
14:38:06 brinzhang_ lyarwood: In https://review.opendev.org/#/c/693828/19/nova/tests/unit/api/openstack/compute/test_volumes.py@1179 I update the unit test case, and assert the delete_on_termination after bdm.save(), I think it's enough, what do you think?
14:38:47 bauzas dansmith: AFAIR it moved recently, trying to find the link
14:40:16 bauzas it was something like [[local|localrc]]
14:40:17 bauzas ADMIN_PASSWORD=secret
14:40:17 bauzas DATABASE_PASSWORD=$ADMIN_PASSWORD
14:40:17 bauzas RABBIT_PASSWORD=$ADMIN_PASSWORD
14:40:17 bauzas SERVICE_PASSWORD=$ADMIN_PASSWORD
14:40:19 bauzas oh shit
14:40:21 bauzas openstack-ci.tintri.com
14:40:35 bauzas dansmith: but it's no longer responding
14:41:08 dansmith ack, yeah
14:45:43 lyarwood bauzas: just updated with a comment, we should be able to get the updated attachment from the fixture
14:45:53 lyarwood ah sorry
14:45:58 lyarwood brinzhang_: ^ tab--
14:47:00 brinzhang_ lyarwood: I will check again
14:47:06 lyarwood brinzhang_: actually ignore that this isn't a fixture, it's n-api.
14:47:14 lyarwood brinzhang_: we should be able to test this directly
14:49:50 brinzhang_ lyarwood: yeas, I think I know what should I do, will update ^^
14:50:01 lyarwood brinzhang_: excellent thanks
14:50:22 brinzhang_ lyarwood: np, thanks too.
14:56:22 openstackgerrit Merged openstack/nova master: Clarify fitting hugepages log message https://review.opendev.org/702981
15:04:21 openstackgerrit Merged openstack/python-novaclient master: Microversion 2.82 - nova cyborg interaction https://review.opendev.org/712700
15:05:03 openstackgerrit Brin Zhang proposed openstack/nova master: Add functional tests for PATCH volume attachments API https://review.opendev.org/710965
15:08:00 brinzhang_ lyarwood: done. Can you check is that rirht?
15:11:44 lyarwood brinzhang_: yup LGTM now, tested locally and it's passing.
15:12:38 brinzhang_ lyarwood: thanks ^^
15:15:01 brinzhang_ lyarwood: do you know Andrey Kurilin's irc name ?
15:15:49 brinzhang_ lyarwood: I donot think we should add @api_versions.wraps("2.83") to https://review.opendev.org/#/c/712651/4/novaclient/v2/volumes.py@127
15:16:51 brinzhang_ lyarwood: the patch_server_volume() function is only one, is it need to add the @api_versions.wraps("2.83") to limit?
15:17:18 lyarwood brinzhang_: I don't sorry
15:18:10 lyarwood brinzhang_: yeah I thought you had done it correctly
15:18:31 lyarwood brinzhang_: however create_server_volume appears to do it
15:19:11 bauzas lyarwood: brinzhang_: https://launchpad.net/~andreykurilin
15:19:39 brinzhang_ lyarwood: create_server_volume appears is there are some microversion added in it, so that need to add the api_version_wraps to limit it's microversion
15:20:26 lyarwood ah I see
15:21:03 brinzhang_ lyarwood: And the patch volume api just the only one to do, if there is another microversion need to add in the future, I think that add the wrap is ok
15:22:02 brinzhang_ so I will keep it not limit by api_version_wraps, unless andreykurilin insists.
15:22:15 brinzhang_ bauzas: thanks
15:28:07 brinzhang_ andreykurilin: are you around?
15:39:02 openstackgerrit Brin Zhang proposed openstack/python-novaclient master: Microversion 2.83: Add volume-patch CLI https://review.opendev.org/712651
15:43:50 mriedem oh wow nova's first PATCH API https://review.opendev.org/#/c/693828/
16:09:46 dansmith hmm, we had a discussion recently about whether or not to add a PATCH API and I thought the agreement was to not
16:11:41 lyarwood I wanted to move swap_volume out from under the UPDATE API but IIRC it was eventually agreed in the spec to go with PUT
16:11:58 lyarwood https://review.opendev.org/#/c/580336/ yeah
16:12:49 openstackgerrit Brin Zhang proposed openstack/python-novaclient master: Microversion 2.83: Add volume-attachment-update CLI https://review.opendev.org/712651
16:35:59 andreykurilin brinzhang_, lyarwood: hi! wrapping with `@api_versions.wraps(2.83)` decorator is preventive protection to avoid redundant calls to Nova API side in case when novaclient is initialized with <2.83 version
17:23:48 openstackgerrit Merged openstack/nova master: images: Move qemu-img info calls into privsep https://review.opendev.org/706897
17:43:08 openstackgerrit John Garbutt proposed openstack/nova master: Add legacy limits and usage to unified limits https://review.opendev.org/713498
17:43:08 openstackgerrit John Garbutt proposed openstack/nova master: WIP: update quota apis with keystone limits and usage https://review.opendev.org/713499
18:21:00 openstackgerrit Merged openstack/nova master: libvirt: Report storage bus traits https://review.opendev.org/666914
20:17:22 openstackgerrit Merged openstack/nova stable/train: Fix intermittently failing regression case https://review.opendev.org/713384
20:35:09 artom Do people still look at http://status.openstack.org/reviews/#nova ?
20:35:32 artom The first, like, 20 or so, are so old and are either in merge conflict of with legit -1 reviews, and can be force-abandonned
20:35:46 artom (IMO, at any rate)
#openstack-nova - 2020-03-18
00:20:01 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing flavor_access policies https://review.opendev.org/713556
00:35:34 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing flavor_access policies https://review.opendev.org/713556
00:46:16 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-flavor-access https://review.opendev.org/713559
00:48:09 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing flavor_access policies https://review.opendev.org/713556
00:48:21 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-flavor-access https://review.opendev.org/713559
01:05:24 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing flavor_access policies https://review.opendev.org/713556
01:05:56 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-flavor-access https://review.opendev.org/713559
01:06:15 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-flavor-access https://review.opendev.org/713559
01:28:58 openstackgerrit Merged openstack/nova master: FUP: Remove noqa and tone down an exception https://review.opendev.org/698054
01:45:53 openstackgerrit Kevin Zhao proposed openstack/nova master: fix scsi disk unit number of the attaching volume when cdrom bus is scsi https://review.opendev.org/712607
02:43:40 openstackgerrit Brin Zhang proposed openstack/python-novaclient master: Microversion 2.83: Add volume-attachment-update CLI https://review.opendev.org/712651
03:04:20 openstackgerrit melanie witt proposed openstack/nova master: DNM: try to get some debug info for bug 1844929 https://review.opendev.org/701478
03:04:20 openstack bug 1844929 in OpenStack Compute (nova) "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,Confirmed] https://launchpad.net/bugs/1844929
03:18:04 openstackgerrit Yingji Sun proposed openstack/nova master: set different VirtualDevice.key https://review.opendev.org/713565
03:53:58 openstackgerrit Yingji Sun proposed openstack/nova master: Set different VirtualDevice.key https://review.opendev.org/713565
05:47:38 openstackgerrit Luyao Zhong proposed openstack/nova master: support live migration with vpmems https://review.opendev.org/687856
05:47:38 openstackgerrit Luyao Zhong proposed openstack/nova master: Track orphan instances and error migrations in resource tracker https://review.opendev.org/678451
06:19:17 openstackgerrit mashuting proposed openstack/nova master: Revert "Deprecate the vmwareapi driver" https://review.opendev.org/713401
06:50:57 openstackgerrit Kevin Zhao proposed openstack/nova master: fix scsi disk unit number of the attaching volume when cdrom bus is scsi https://review.opendev.org/712607
07:39:46 gibi artom: ack, I will do a sweep
07:55:58 gibi bauzas: do you still working on https://review.opendev.org/#/c/472685 ?
08:08:26 gibi stephenfin: is this still relevant? https://review.opendev.org/#/c/498820
08:08:56 gibi stephenfin: and this https://review.opendev.org/#/c/498819 ?
09:03:37 bauzas gibi: morning
09:03:43 gibi bauzas: hi!
09:03:45 bauzas gibi: not really, I can abandon it
09:03:58 bauzas gibi: how are you ? are you locked down too ?
09:04:00 gibi bauzas: please abandon it. thanks
09:04:33 gibi bauzas: general lockdown yes. groceries are open between 6:00 - 15:00 everything else is closed
09:05:00 bauzas gibi: and you can walk outside home ?
09:05:31 gibi bauzas: yeah walks and forests are open
09:05:40 gibi only the gatherings are forbidden
09:06:05 gibi bauzas: do you only allowed to walk outside if you have a dog?
09:06:11 bauzas okay, b/c for us, we have to stay home unless 5 reasons
09:06:55 bauzas food, work (but only if very needed), urgent needing, and dogwalks yes :p
09:07:23 bauzas you can't for example run outside
09:07:34 bauzas only around your house (like 100m)

Earlier   Later