Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-11-06
18:10:40 stephenfin bar()
18:10:40 stephenfin foo = 'test'
18:10:50 stephenfin ^
18:12:18 lyarwood right so the error is misleading, it's not the read in the conditional that's at fault, it's the import trying to overwrite it?
18:12:44 stephenfin sort of
18:12:54 stephenfin by trying to write to it, the variable becomes locally scoped
18:13:02 lyarwood yup got it
18:13:38 stephenfin flake8 can't catch these kind of issues, unfortunately
18:14:54 sean-k-mooney lyarwood: before it was locally importing every time and using the local import if it was not visable
18:15:08 sean-k-mooney you can read gloabls without the keyword
18:15:24 sean-k-mooney you just can write to them it creates a local alias of it
18:16:52 lyarwood sean-k-mooney: yeah got it
18:17:27 lyarwood stephenfin: still failing btw
18:17:42 stephenfin darn, same error?
18:17:56 lyarwood stephenfin: different, this looks more like a test issue
18:18:00 lyarwood stephenfin: https://7d2bb7e91c376d06349f-0e5efcd7961a07f1b6cc368c11ff4914.ssl.cf1.rackcdn.com/761762/5/check/openstack-tox-cover/b207b75/testr_results.html
18:19:18 lyarwood stephenfin: I need to drop now, feel free to leave that and I can address it on Monday
18:19:34 stephenfin Yeah, it's not the easy fix we though so let's leave it
18:19:45 stephenfin lyarwood: Have a good one o/
18:20:42 lyarwood and you \o
18:23:30 sean-k-mooney damit... i dont have precommit on my laptop and didnt run pep8
18:28:59 mnaser i've been confused about this for aw hile
18:29:20 mnaser say i have a queens install and enabled_vgpu_types was changed -- do i have to go muck about the placement service?
18:29:47 mnaser i cant find where this is 'defined' and being 'published' to placement
18:32:12 mnaser inventories only shows 3 resorce classes which seem to be disk/vcpu/mem
18:39:53 gmann stephenfin: ack, lance and I are going to have call on Tuesday for testing strategy for these changes, at least we need unit test like we did in nova.
18:40:24 gmann stephenfin: these are in my list for next week.
18:40:59 gmann these are meeting details - http://lists.openstack.org/pipermail/openstack-discuss/2020-November/018512.html
19:40:53 mnaser `virsh nodedev-dumpxml` shows availableInstances = 0
19:40:56 mnaser so that's .. progress
20:15:56 mnaser ok turns out there was devices that were not removed
20:47:57 openstackgerrit Merged openstack/nova master: zuul: Replace nova-live-migration with zuulv3 jobs https://review.opendev.org/752557
#openstack-nova - 2020-11-07
03:25:11 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.moves https://review.opendev.org/727224
03:25:29 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.iteritems/itervalues/iterkeys https://review.opendev.org/727757
03:25:44 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.byte2int/int2byte https://review.opendev.org/727777
03:26:01 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/728094
03:26:24 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/728109
03:27:01 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117
08:27:52 openstackgerrit Merged openstack/nova master: Allow excluding image type capabilities https://review.opendev.org/756534
09:23:01 frickler nova api experts, I could need some help regarding how flavors are attached to servers, see https://review.opendev.org/761772
09:58:26 gibi frickler: tried to answer in the review ^^
10:22:58 frickler gibi: thanks, I think that explains it, at least for me. I'm just wondering whether we should make the difference more obvious to the user, too, by either changing the column name to "original flavor" or something similar. just to make clear that doing a "show flavor" on the shown flavor name may have nothing in common with the flavor actually applied to the server
11:11:27 openstackgerrit Vlad Gusev proposed openstack/nova stable/victoria: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/761809
11:13:08 openstackgerrit Vlad Gusev proposed openstack/nova stable/ussuri: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/761810
11:14:09 openstackgerrit Vlad Gusev proposed openstack/nova stable/train: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/761811
11:15:18 openstackgerrit Vlad Gusev proposed openstack/nova stable/stein: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/761812
11:16:03 openstackgerrit Vlad Gusev proposed openstack/nova stable/rocky: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/761813
11:17:29 openstackgerrit Vlad Gusev proposed openstack/nova stable/queens: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/761814
13:39:59 gibi frickler: I'm hesitant about 'original flavor', as what else we would show there if not the name of the flavor we booted with. So it might not explain for the <= 2.46 case that the flavor shown there might be not the falvor we booted the server with but a flavor that has the same id than the flavor that was used for the boot
13:40:39 gibi so I don't think original flavor would actually help removing the confusion
14:55:59 openstackgerrit Merged openstack/nova master: virt: Remove 'set_bootable' API https://review.opendev.org/749314
15:52:14 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test compute RPC signature compatibility https://review.opendev.org/761816
15:55:00 gibi dansmith bauzas stephenfin: ^^ I made a generic functional test that can catch RCP signature incompatibility, like the one we dealth with this week
15:55:48 gibi It is not pretty but I think the benefit outweights the ugliness of the necessary hacks
16:05:55 openstackgerrit Merged openstack/nova stable/victoria: Modify glance's copy_image permission for nova-ceph-multistore https://review.opendev.org/761755
20:02:03 openstackgerrit Ghanshyam Mann proposed openstack/nova master: DNM: Testing system scope in tempest https://review.opendev.org/740124
20:02:55 openstackgerrit Ghanshyam Mann proposed openstack/nova master: DNM: Testing system scope in tempest https://review.opendev.org/740124
23:46:31 openstackgerrit Vlad Gusev proposed openstack/nova stable/victoria: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/761809
#openstack-nova - 2020-11-08
01:21:59 openstackgerrit Hemanth Nakkina proposed openstack/nova stable/rocky: Update pci stat pools based on PCI device changes https://review.opendev.org/761824
01:29:56 openstackgerrit Hemanth Nakkina proposed openstack/nova stable/queens: Update pci stat pools based on PCI device changes https://review.opendev.org/761825
04:30:16 openstackgerrit Merged openstack/nova master: Prevent starting services with older than N-1 computes https://review.opendev.org/738482
09:28:43 openstackgerrit Balazs Gibizer proposed openstack/nova master: Test compute RPC signature compatibility https://review.opendev.org/761816
09:45:14 openstackgerrit Merged openstack/nova master: virt: Remove various aggregate APIs https://review.opendev.org/749317
09:53:01 openstackgerrit Merged openstack/nova master: trivial: Remove irrelevant comment https://review.opendev.org/759112
12:27:48 openstackgerrit Merged openstack/nova master: Add upgrade check about old computes https://review.opendev.org/760520
18:43:27 openstackgerrit Ghanshyam Mann proposed openstack/nova master: DNM: Testing system scope in tempest https://review.opendev.org/740124
#openstack-nova - 2020-11-09
02:05:38 deke997 anyone here know the latest status of spice integration?
05:48:48 xinranwang gibi: Hi gibi, thanks for your comments for smartnic spec. For bandiwidth qos, can I assume that the co-existance is not support at the first step.
05:51:31 xinranwang maybe we can add this restirction into document for the first phase
05:52:18 xinranwang Is it acceptable?
07:50:13 gibi xinranwang: yes, such limitation is acceptable for me
07:59:48 xinranwang gibi: great, thanks. I will update the spec soon.:)
08:00:46 gibi I guess you mean coexistens for the same physical device.
08:54:30 openstackgerrit Lee Yarwood proposed openstack/nova master: zuul: Reintroduce nova-dsvm-multinode-base https://review.opendev.org/761870
09:05:25 bauzas good morning Nova
09:12:02 gibi bauzas: o/
10:22:54 openstackgerrit Lee Yarwood proposed openstack/nova master: rbd: Only log import failures when the RbdDriver is used https://review.opendev.org/761762
10:26:26 lyarwood stephenfin: ^ hopefully fixed btw
10:29:53 lyarwood gibi / stephenfin / bauzas ; https://review.opendev.org/761870 could use reviews today if you have time, for whatever reason zuul didn't raise an error with the original change, it just removes the job silently. I'll file a bug later for this.
10:30:05 stephenfin already done :)
10:30:15 gibi I will look shortly
10:30:34 lyarwood thanks
10:40:58 openstackgerrit Lee Yarwood proposed openstack/nova master: Add os-volume_attachments reference docs https://review.opendev.org/760971
10:41:20 lyarwood stephenfin: ^ some basic ref docs for volume attachment btw if you're interested, came up as part of a spec.
10:41:59 lyarwood plan on adding lots of additional detail later in the cycle but the locking part is a good enough start at the moment
10:42:25 stephenfin sure thing, I can take a look
10:42:28 lyarwood thanks
12:45:52 openstackgerrit Merged openstack/nova master: zuul: Reintroduce nova-dsvm-multinode-base https://review.opendev.org/761870
13:53:00 openstackgerrit Balazs Gibizer proposed openstack/nova stable/victoria: Warn when starting services with older than N-1 computes https://review.opendev.org/761923
13:54:26 openstackgerrit Balazs Gibizer proposed openstack/nova stable/victoria: Add upgrade check about old computes https://review.opendev.org/761924
14:18:03 stephenfin lyarwood: Thanks for the review on the 'openstack server evacuate' patch btw. I sent it on its way. Long overdue.
14:18:38 stephenfin I need to go through the API with a fine comb but I _think_ we have the vast majority of the API covered at this point
14:20:45 gibi stephenfin: what is the equvivalent for nova instance-action-list ? I allways fall back to the nova client to do that
14:20:59 lyarwood gibi: openstack server event list
14:21:01 lyarwood I think
14:21:05 gibi /o\
14:21:10 stephenfin yeah, that
14:21:23 gibi indeed
14:21:25 gibi thanks
14:21:31 gibi I need to build muscle memory for that

Earlier   Later