Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-03-02
10:12:29 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Add framework for extra spec validation https://review.opendev.org/704643
10:12:29 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Add microversion 2.82, extra spec validation https://review.opendev.org/708436
10:12:30 openstackgerrit Stephen Finucane proposed openstack/nova master: docs: Add documentation for flavor extra specs https://review.opendev.org/710037
10:26:55 openstackgerrit John Garbutt proposed openstack/nova master: Add test coverage of existing os-agents policies https://review.opendev.org/701644
10:28:35 openstackgerrit Brin Zhang proposed openstack/nova master: Add PATCH volume attachments api to os-volume_attachments https://review.opendev.org/693828
10:35:57 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Remove MIN_LIBVIRT_MULTIATTACH https://review.opendev.org/710238
10:49:31 openstackgerrit John Garbutt proposed openstack/nova master: Fix os-attach-interfaces policy to be admin_or_owner https://review.opendev.org/705135
10:49:45 openstackgerrit John Garbutt proposed openstack/nova master: Remove old policy enforcement in attach_interfaces https://review.opendev.org/705127
11:11:10 openstackgerrit John Garbutt proposed openstack/nova master: Fix os-console-output policy to be admin_or_owner https://review.opendev.org/706725
11:12:36 openstackgerrit John Garbutt proposed openstack/nova master: Add test coverage of existing console_output policies https://review.opendev.org/706724
11:28:55 openstackgerrit Merged openstack/nova master: libvirt: Provide the backing file format when creating qcow2 disks https://review.opendev.org/708745
11:38:34 openstackgerrit John Garbutt proposed openstack/nova master: Add test coverage of existing os-assisted_volume_snapshots policies https://review.opendev.org/701835
11:47:19 openstackgerrit John Garbutt proposed openstack/nova master: Introduce scope_types in os-assisted_volume_snapshots policy https://review.opendev.org/701837
11:48:03 openstackgerrit John Garbutt proposed openstack/nova master: Add new default roles in os-assisted_volume_snapshots policies https://review.opendev.org/701840
11:52:52 openstackgerrit John Garbutt proposed openstack/nova master: Pass the actual target in os-assisted_volume_snapshots policy https://review.opendev.org/701841
11:54:05 openstackgerrit John Garbutt proposed openstack/nova master: Add new default roles in Admin Action API policies https://review.opendev.org/676682
11:54:50 openstackgerrit John Garbutt proposed openstack/nova master: Ensure we pass a target in admin actions https://review.opendev.org/663095
11:59:17 openstackgerrit Merged openstack/nova master: Add test coverage of existing admin_password policies https://review.opendev.org/701629
13:49:17 openstackgerrit Merged openstack/nova master: Correct os-availability-zone policy check_str https://review.opendev.org/706683
13:49:24 openstackgerrit Merged openstack/nova master: Add test coverage of existing availability-zone policies https://review.opendev.org/706682
13:49:31 openstackgerrit Merged openstack/nova master: Introduce scope_types in os-availability-zone https://review.opendev.org/706684
13:55:53 openstackgerrit John Garbutt proposed openstack/nova master: Ensure we pass a target in admin actions https://review.opendev.org/663095
13:56:31 openstackgerrit John Garbutt proposed openstack/nova master: Ensure we pass a target in admin actions https://review.opendev.org/663095
14:03:53 openstackgerrit John Garbutt proposed openstack/nova master: Fix os-create-backup policy to be admin_or_owner https://review.opendev.org/706727
14:27:50 openstackgerrit Merged openstack/nova master: Add new default roles in os-availability-zone policies https://review.opendev.org/706686
14:28:04 openstackgerrit Merged openstack/nova master: Add test coverage of existing evacuate policies https://review.opendev.org/708230
14:46:32 openstackgerrit Lee Yarwood proposed openstack/nova master: images: Move qemu-img info calls into privsep https://review.opendev.org/706897
14:46:32 openstackgerrit Lee Yarwood proposed openstack/nova master: images: Allow the output format of qemu-img info to be controlled https://review.opendev.org/706898
14:46:33 openstackgerrit Lee Yarwood proposed openstack/nova master: virt: Pass request context to extend_volume https://review.opendev.org/706899
14:46:33 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Correctly resize encrypted LUKSv1 volumes https://review.opendev.org/706900
14:46:34 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Always provide the size in bytes when calling virDomainBlockResize https://review.opendev.org/707590
14:46:34 openstackgerrit Lee Yarwood proposed openstack/nova master: images: Remove Libvirt specific configurable use from qemu_img_info https://review.opendev.org/707591
14:46:35 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Remove QEMU_VERSION_REQ_SHARED https://review.opendev.org/710239
14:46:35 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Use oslo.utils >= 4.1.0 to fetch format-specific image data https://review.opendev.org/710785
14:47:34 lyarwood stephenfin: ^ vendor'd the oslo.utils change so we can backport this easier, addressed your feedback and fixed your follow up (you missed the removal of a test).
14:49:32 stephenfin ack
14:50:08 openstackgerrit Lee Yarwood proposed openstack/nova master: DNM - Test TEMPEST_EXTEND_ATTACHED_ENCRYPTED_VOLUME https://review.opendev.org/707593
14:50:19 sean-k-mooney lyarwood: looking at your q35 change i dont think the test failture are vaild. i.e. if they are real failure i think its like an issue with how the test is written rather then a real issue
14:51:00 lyarwood sean-k-mooney: agreed, I had a quick look this morning but haven't had time to follow up as yet
14:51:32 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: libvirt: Provide the backing file format when creating qcow2 disks https://review.opendev.org/710788
14:51:44 sean-k-mooney lyarwood: looking at the failure "tempest.lib.exceptions.SSHExecCommandFailed: Command 'set -eu -o pipefail; PATH=$PATH:/sbin:/usr/sbin; blkid | grep -i config-2', exit status: 1, stderr:"
14:52:04 sean-k-mooney i think its looking for the config drive?
14:52:24 sean-k-mooney and for q35 its attached vai sata instead of ide right
14:52:40 lyarwood it's looking for a tagged disk
14:52:41 sean-k-mooney so it might be reported differently in blkid
14:52:45 sean-k-mooney ah ok
14:52:50 lyarwood yeah I think the string might be slightly different
14:53:04 sean-k-mooney ya same effect i guess
14:53:05 lyarwood actually is the tag even there for SCSI
14:53:18 lyarwood you'd likely need to probe /sys
14:53:31 sean-k-mooney im not sure
14:53:39 openstackgerrit Lee Yarwood proposed openstack/nova stable/stein: libvirt: Provide the backing file format when creating qcow2 disks https://review.opendev.org/710790
15:22:34 dansmith sean-k-mooney: for your multinode cyborg tests, are you actually stacking another compute-and-cyborg-only node? I was thinking maybe I could try to hack that into place by just running another copy of compute with a hostname override in conf
15:23:09 sean-k-mooney yes im sinning up two vms 1 is an all in one node and the other is a pure compute
15:23:37 sean-k-mooney by pure compute i mean it just running the nova/neutron/cinder/cyborg compute agents
15:23:57 sean-k-mooney that is what the ci job is doing too
15:26:08 dansmith yeah, okay, just thinking about maybe some lighter way to do that
15:26:13 sean-k-mooney dansmith: this is more or less the local.confs im using http://paste.openstack.org/show/789773/
15:26:36 sean-k-mooney dansmith: ya i have 32G of ram on my laptop so 2 8G vms is fine
15:26:45 sean-k-mooney the config trick might work
15:26:53 sean-k-mooney but it just felt like more effort
15:26:55 dansmith yeah, ram isn't the problem
15:26:57 dansmith could be
15:29:33 sean-k-mooney do you want me to kick off a ci run with the latest set of patches sundar pushed
15:30:35 sean-k-mooney i basically just need to recheck https://review.opendev.org/#/c/709641/ or update the depends-on url to point to the latest patch in the cain
15:31:11 dansmith sean-k-mooney: yeah, would be good I think
15:31:14 sean-k-mooney dansmith: removing the singelton seemed to resolve all the auth issues
15:31:17 dansmith I'm working through those now
15:31:21 dansmith sean-k-mooney: nice :)
15:32:14 sean-k-mooney ok ill point it to pull in the last funcitonal patch in the cain which is the one that block unsupported opperation and we can get the job to assert they all fail/are rejected as we expect
15:35:32 sean-k-mooney cool thats running we will know in about an hour
15:35:44 dansmith schweet
15:40:09 openstackgerrit Merged openstack/nova master: Introduce scope_types in os-evacuate https://review.opendev.org/708235
16:17:11 openstackgerrit John Garbutt proposed openstack/nova master: Add a tests to check when legacy access is removed https://review.opendev.org/710813
16:25:06 openstackgerrit John Garbutt proposed openstack/nova master: Add a tests to check when legacy access is removed https://review.opendev.org/710813
16:29:50 openstackgerrit Merged openstack/nova master: Add test coverage of existing os-agents policies https://review.opendev.org/701644
16:58:09 sean-k-mooney dansmith: the test run is still executing but i am seeing some failes as expected. however the error code is 500 which seams incorrect
16:58:16 sean-k-mooney Body: b'{"computeFault": {"code": 500, "message": "Unexpected API Error. Please report this at http://bugs.launchpad.net/nova/ and attach the Nova API log if possible.\\n"}}' Body: b'{"computeFault": {"code": 500, "message": "Unexpected API Error. Please report this at http://bugs.launchpad.net/nova/ and attach the Nova API log if possible.\\n
16:58:18 sean-k-mooney \'nova.exception.ForbiddenWithAccelerators\'>"}}'
16:58:33 sean-k-mooney it should be a 4XX error correct
16:58:38 dansmith ah yeah
16:58:58 openstackgerrit Balazs Gibizer proposed openstack/nova master: WIP: Hey let's support routed networks y'all! https://review.opendev.org/656885
17:00:05 sean-k-mooney it looks like it should be a 403 here which i dont think is correct either https://review.opendev.org/#/c/674726/3/nova/exception.py
17:00:21 sean-k-mooney ill wait till the jobs completes and check the api log
17:00:29 sean-k-mooney the 500 might be a tempest error code
17:00:56 sean-k-mooney its hard to tell just form the job console
17:16:48 Sundar sean-k-mooney: The error response comes from https://review.opendev.org/#/c/674726/3/nova/api/openstack/compute/servers.py@758 . I see other similar operations grouped in that segment.
17:17:25 sean-k-mooney the job just finished https://zuul.opendev.org/t/openstack/build/0f144bf83a614d31938d353a1bb05935
17:17:37 sean-k-mooney im going to confirm if the 500 is from tempest or nova
17:17:45 Sundar I have tested it from the 'openstack server ...' CLI, so I don;t think it is tempest
17:18:26 sean-k-mooney well the "computefault" bit makes me think its from tempest
17:19:12 Sundar You would see a 500 error if you do 'openstack server suspend foo'.
17:19:34 sean-k-mooney its not a server error so it should not be a 500
17:19:46 sean-k-mooney its a client error because you requested an unsupported operation
17:20:16 sean-k-mooney so it s houl be either a 400 "bad request" 409 "conflict" or 405 "method not supported"
17:20:31 sean-k-mooney proably a 400 or 409 we dont use 405 as far as i know
17:23:28 sean-k-mooney Sundar: so yes we are raising the exeption but it is not being caught and its being converted into a 500 http://paste.openstack.org/show/790209/
17:25:55 openstackgerrit Merged openstack/nova master: Define Cyborg ARQ binding notification event. https://review.opendev.org/692707
17:26:23 Sundar sean-k-mooney: Hmm, let me see what's happening with https://review.opendev.org/#/c/674726/3/nova/api/openstack/compute/servers.py@758
17:27:12 sean-k-mooney the block operation patch needs to be rewored to only block operation that dont work

Earlier   Later