*** tetsuro has joined #openstack-nova | 00:21 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add docs and releasenotes for BP policy-defaults-refresh https://review.opendev.org/720129 | 00:37 |
---|---|---|
gmann | lbragstad: can you verify the steps for migration to new policy. I cannot find the keystone exact doc please let me know if there is any so that i can link that in my doc - https://review.opendev.org/#/c/720129/4..5/doc/source/configuration/policy-new-defaults.rst@227 | 00:40 |
*** derekh has quit IRC | 00:42 | |
*** maohongbo1 has joined #openstack-nova | 00:48 | |
*** maohongbo has quit IRC | 00:50 | |
*** maohongbo1 is now known as maohongbo | 00:50 | |
openstackgerrit | melanie witt proposed openstack/nova master: Enable test_volume_backed_live_migration in tempest https://review.opendev.org/528104 | 00:51 |
lbragstad | gmann sounds good - i'm about to pick up the review i started earlier for https://review.opendev.org/#/c/720129/4/doc/source/configuration/policy-new-defaults.rst | 00:53 |
lbragstad | i'll take a look at the new patch set | 00:53 |
*** ociuhandu has joined #openstack-nova | 01:01 | |
openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage db purge to nova-audit https://review.opendev.org/708783 | 01:01 |
openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage db archive_deleted_rows to nova-audit https://review.opendev.org/708784 | 01:02 |
openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage cell_v2 discover_hosts to nova-manage https://review.opendev.org/708785 | 01:02 |
openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage cell_v2 map_instances to nova-audit https://review.opendev.org/708786 | 01:02 |
openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage placement sync_aggregates to nova-audit https://review.opendev.org/708787 | 01:02 |
openstackgerrit | melanie witt proposed openstack/nova master: Move nova-manage placement heal_allocations to nova-audit https://review.opendev.org/708788 | 01:02 |
*** maohongbo1 has joined #openstack-nova | 01:04 | |
*** ociuhandu has quit IRC | 01:05 | |
*** maohongbo has quit IRC | 01:06 | |
*** maohongbo1 is now known as maohongbo | 01:06 | |
*** Liang__ has joined #openstack-nova | 01:10 | |
openstackgerrit | Merged openstack/nova-specs master: Cleanup py27 support https://review.opendev.org/718368 | 01:20 |
*** maohongbo1 has joined #openstack-nova | 01:28 | |
*** maohongbo has quit IRC | 01:30 | |
*** maohongbo1 is now known as maohongbo | 01:30 | |
*** maohongbo1 has joined #openstack-nova | 01:36 | |
*** maohongbo has quit IRC | 01:38 | |
*** maohongbo1 is now known as maohongbo | 01:38 | |
*** yaawang has quit IRC | 01:46 | |
*** yaawang has joined #openstack-nova | 01:47 | |
*** yaawang has quit IRC | 01:52 | |
*** yaawang has joined #openstack-nova | 01:53 | |
*** sapd1 has quit IRC | 01:53 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: Accurately clean up ARQs resources during build_instances in conductor https://review.opendev.org/720439 | 02:03 |
*** tetsuro has quit IRC | 02:06 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: Accurately clean up ARQs resources during build_instances in conductor https://review.opendev.org/720439 | 02:07 |
*** maohongbo has quit IRC | 02:08 | |
*** maohongbo has joined #openstack-nova | 02:08 | |
*** tetsuro has joined #openstack-nova | 02:12 | |
openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Re-proposes 'Proposal for a safer remote console with password authentication' https://review.opendev.org/719720 | 02:23 |
*** rambo_li has joined #openstack-nova | 02:24 | |
*** sapd1 has joined #openstack-nova | 02:31 | |
rambo_li | sean-k-mooney: maybe their is an error, for the bug :https://bugs.launchpad.net/nova/+bug/1841932. when we set the metadata "hide_hypervisor_id=true"in aggreate,so we will go to the https://github.com/openstack/nova/blob/46a3bcd80b41e99ec4923c7cf3d0f8dd8505e97c/nova/scheduler/filters/aggregate_instance_extra_specs.py#L69-L71, but this should not break, because the aggregate_val doesn't match the req. | 02:31 |
openstack | Launchpad bug 1841932 in OpenStack Compute (nova) "hide_hypervisor_id extra_specs in nova flavor cannot pass AggregateInstanceExtraSpecsFilter" [Undecided,New] | 02:31 |
*** igordc has joined #openstack-nova | 02:46 | |
rambo_li | maybe we should delete the for loop in https://github.com/openstack/nova/blob/46a3bcd80b41e99ec4923c7cf3d0f8dd8505e97c/nova/scheduler/filters/aggregate_instance_extra_specs.py#L69 | 02:48 |
*** mkrai has joined #openstack-nova | 02:50 | |
openstackgerrit | Merged openstack/os-traits master: Cleanup py27 support https://review.opendev.org/719350 | 02:51 |
*** igordc has quit IRC | 02:52 | |
*** igordc has joined #openstack-nova | 02:52 | |
*** igordc has quit IRC | 02:57 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: Accurately clean up ARQs resources during build_instances in conductor https://review.opendev.org/720439 | 03:06 |
*** rcernin has quit IRC | 03:10 | |
*** psachin has joined #openstack-nova | 03:17 | |
*** rcernin has joined #openstack-nova | 03:24 | |
*** ttsiouts_ has joined #openstack-nova | 03:31 | |
*** ttsiouts has quit IRC | 03:34 | |
*** rambo_li has quit IRC | 03:35 | |
*** yaawang has quit IRC | 03:40 | |
*** yaawang has joined #openstack-nova | 03:41 | |
*** ociuhandu has joined #openstack-nova | 03:44 | |
*** ociuhandu has quit IRC | 03:49 | |
*** tetsuro has quit IRC | 03:51 | |
*** rambo_li has joined #openstack-nova | 03:54 | |
*** rambo_li has quit IRC | 04:14 | |
*** rambo_li has joined #openstack-nova | 04:17 | |
*** evrardjp has quit IRC | 04:37 | |
*** evrardjp has joined #openstack-nova | 04:37 | |
*** kevinz has joined #openstack-nova | 04:42 | |
*** mkrai has quit IRC | 04:53 | |
*** mkrai has joined #openstack-nova | 04:54 | |
*** mkrai has quit IRC | 04:56 | |
*** mkrai has joined #openstack-nova | 04:57 | |
*** mnasiadka has quit IRC | 05:10 | |
*** Anticimex has quit IRC | 05:10 | |
*** tinwood has quit IRC | 05:10 | |
*** elod has quit IRC | 05:10 | |
*** tetsuro has joined #openstack-nova | 05:13 | |
*** mnasiadka has joined #openstack-nova | 05:15 | |
*** Anticimex has joined #openstack-nova | 05:15 | |
*** tinwood has joined #openstack-nova | 05:15 | |
*** elod has joined #openstack-nova | 05:15 | |
*** rambo_li has quit IRC | 05:24 | |
*** ratailor has joined #openstack-nova | 05:24 | |
*** mkrai_ has joined #openstack-nova | 05:33 | |
*** mkrai has quit IRC | 05:33 | |
*** udesale has joined #openstack-nova | 05:35 | |
*** ociuhandu has joined #openstack-nova | 05:37 | |
*** mkrai_ has quit IRC | 05:37 | |
*** rambo_li has joined #openstack-nova | 05:42 | |
*** ociuhandu has quit IRC | 05:47 | |
*** gyee has quit IRC | 05:53 | |
*** mkrai has joined #openstack-nova | 06:02 | |
*** mkrai has quit IRC | 06:10 | |
*** mkrai_ has joined #openstack-nova | 06:10 | |
*** dklyle has quit IRC | 06:26 | |
*** udesale has quit IRC | 06:26 | |
*** udesale has joined #openstack-nova | 06:32 | |
brinzhang | gibi, bauzas: I open a new bug https://bugs.launchpad.net/nova/+bug/1873387, please check while you are wake up. | 06:36 |
openstack | Launchpad bug 1873387 in OpenStack Compute (nova) "Unable to delete instance because cyborg.get_client() failed" [Medium,Confirmed] - Assigned to Brin Zhang (zhangbailin) | 06:36 |
*** xek has joined #openstack-nova | 06:36 | |
*** ociuhandu has joined #openstack-nova | 06:37 | |
*** ociuhandu has quit IRC | 06:43 | |
gibi | brinzhang: thanks for finding that bug | 06:45 |
gibi | brinzhang: I marked it as ussuri-rc-potential as it is a new regression in Ussuri | 06:46 |
brinzhang | gibi: Got it | 06:51 |
brinzhang | thanks | 06:51 |
*** maohongbo1 has joined #openstack-nova | 06:51 | |
*** maohongbo has quit IRC | 06:54 | |
*** maohongbo1 is now known as maohongbo | 06:54 | |
*** spatel has joined #openstack-nova | 06:55 | |
*** links has joined #openstack-nova | 07:00 | |
*** spatel has quit IRC | 07:00 | |
*** nightmare_unreal has joined #openstack-nova | 07:02 | |
*** jhesketh has quit IRC | 07:04 | |
*** ociuhandu has joined #openstack-nova | 07:09 | |
*** slaweq has joined #openstack-nova | 07:11 | |
*** maohongbo1 has joined #openstack-nova | 07:14 | |
*** maohongbo has quit IRC | 07:15 | |
*** maohongbo1 is now known as maohongbo | 07:15 | |
*** tesseract has joined #openstack-nova | 07:17 | |
*** rpittau|afk is now known as rpittau | 07:19 | |
*** maciejjozefczyk has joined #openstack-nova | 07:24 | |
brinzhang | gibi: for bug 1872934 we pushed the fix patch https://review.opendev.org/#/c/720439/, please review if you are free | 07:29 |
openstack | bug 1872934 in OpenStack Compute (nova) "Accurately clean up ARQs resources during build_instances() in conductor" [Low,In progress] https://launchpad.net/bugs/1872934 - Assigned to Brin Zhang (zhangbailin) | 07:29 |
*** tosky has joined #openstack-nova | 07:30 | |
*** ociuhandu has quit IRC | 07:30 | |
*** harsha24 has joined #openstack-nova | 07:36 | |
*** ralonsoh has joined #openstack-nova | 07:38 | |
*** maohongbo has quit IRC | 07:49 | |
*** maohongbo1 has joined #openstack-nova | 07:49 | |
*** maohongbo1 is now known as maohongbo | 07:51 | |
*** ccamacho has joined #openstack-nova | 07:56 | |
*** threestrands has quit IRC | 07:56 | |
*** derekh has joined #openstack-nova | 08:02 | |
*** ociuhandu has joined #openstack-nova | 08:04 | |
*** ttsiouts_ has quit IRC | 08:11 | |
*** dtantsur|afk is now known as dtantsur | 08:13 | |
*** lbragstad_ has joined #openstack-nova | 08:17 | |
*** lbragstad has quit IRC | 08:20 | |
*** ttsiouts has joined #openstack-nova | 08:21 | |
*** tetsuro has quit IRC | 08:22 | |
*** ociuhandu has quit IRC | 08:23 | |
*** ociuhandu has joined #openstack-nova | 08:23 | |
*** maciejjozefczyk has quit IRC | 08:37 | |
*** maciejjozefczyk has joined #openstack-nova | 08:39 | |
*** harsha24 has quit IRC | 08:41 | |
*** sapd1 has quit IRC | 08:41 | |
*** derekh has quit IRC | 08:43 | |
*** ociuhandu has quit IRC | 08:45 | |
*** ttsiouts has quit IRC | 08:47 | |
*** maohongbo1 has joined #openstack-nova | 08:48 | |
*** maohongbo has quit IRC | 08:48 | |
*** maohongbo1 is now known as maohongbo | 08:49 | |
*** tkajinam has quit IRC | 08:51 | |
*** derekh has joined #openstack-nova | 08:56 | |
*** ttsiouts has joined #openstack-nova | 09:00 | |
*** mkrai_ has quit IRC | 09:01 | |
*** maohongbo1 has joined #openstack-nova | 09:11 | |
*** maohongbo has quit IRC | 09:12 | |
*** maohongbo1 is now known as maohongbo | 09:12 | |
*** ttsiouts has quit IRC | 09:16 | |
openstackgerrit | Wenping Song proposed openstack/nova master: handle to get cyborg client exception when delete instance https://review.opendev.org/720670 | 09:17 |
*** ociuhandu has joined #openstack-nova | 09:25 | |
*** maohongbo has quit IRC | 09:32 | |
*** maohongbo has joined #openstack-nova | 09:33 | |
*** mkrai has joined #openstack-nova | 09:33 | |
*** ttsiouts has joined #openstack-nova | 09:39 | |
*** ociuhandu has quit IRC | 09:43 | |
*** ociuhandu has joined #openstack-nova | 09:49 | |
*** Liang__ has quit IRC | 09:52 | |
hrw | lyarwood: replied to your comment ;D | 10:05 |
openstackgerrit | Wenping Song proposed openstack/nova master: Handle to get cyborg client exception when delete instance https://review.opendev.org/720670 | 10:11 |
lyarwood | hrw: /me rolls eyes | 10:15 |
lyarwood | hrw: just a suggestion | 10:15 |
hrw | lyarwood: a bit of pure sarcasm in a morning | 10:19 |
lyarwood | :) I've not had enough coffee for that yet | 10:19 |
hrw | I just finished second | 10:19 |
*** rambo_li has quit IRC | 10:25 | |
lyarwood | just starting mine, tgif++ | 10:26 |
lyarwood | https://review.opendev.org/#/c/669674/ - could use a non-RH core on this given it has an impact across virt drivers. | 10:26 |
*** rpittau is now known as rpittau|bbl | 10:30 | |
*** szaher has joined #openstack-nova | 10:34 | |
*** brinzhang has quit IRC | 10:35 | |
*** ociuhandu has quit IRC | 10:48 | |
*** ociuhandu has joined #openstack-nova | 10:48 | |
*** ttsiouts has quit IRC | 10:49 | |
*** ttsiouts has joined #openstack-nova | 10:50 | |
*** ociuhandu has quit IRC | 10:52 | |
*** spatel has joined #openstack-nova | 10:57 | |
*** spatel has quit IRC | 11:02 | |
*** mkrai has quit IRC | 11:09 | |
*** mkrai has joined #openstack-nova | 11:09 | |
*** bbowen_ has joined #openstack-nova | 11:18 | |
*** bbowen has quit IRC | 11:19 | |
*** ociuhandu has joined #openstack-nova | 11:23 | |
*** bbowen has joined #openstack-nova | 11:37 | |
*** bbowen_ has quit IRC | 11:39 | |
*** belmoreira has joined #openstack-nova | 11:47 | |
*** belmoreira has quit IRC | 11:48 | |
*** belmoreira has joined #openstack-nova | 11:48 | |
*** averi has joined #openstack-nova | 11:52 | |
averi | hi, anyone has an idea on why code changes are not picked up by nova_compute containers (OSP 13)? | 11:52 |
averi | even when spinning up a new image based on the nova-compute one, it doesn't work, code changes are there but they're like ignored all together | 11:53 |
averi | even killing one of the library files results in the binary not to fail like the file was still around | 11:53 |
*** tkajinam has joined #openstack-nova | 11:56 | |
*** udesale_ has joined #openstack-nova | 12:02 | |
*** tkajinam has quit IRC | 12:04 | |
*** udesale has quit IRC | 12:05 | |
*** rcernin has quit IRC | 12:11 | |
*** ttsiouts has quit IRC | 12:12 | |
*** sapd1 has joined #openstack-nova | 12:16 | |
*** rpittau|bbl is now known as rpittau | 12:19 | |
*** mgariepy has joined #openstack-nova | 12:24 | |
*** psachin has quit IRC | 12:26 | |
*** mkrai has quit IRC | 12:28 | |
*** mdbooth has joined #openstack-nova | 12:31 | |
*** ttsiouts has joined #openstack-nova | 12:42 | |
gibi | dansmith, sean-k-mooney, brinzhang: I need a second opinion in https://review.opendev.org/#/c/720670 | 12:47 |
*** ttsiouts has quit IRC | 12:48 | |
*** ociuhandu has quit IRC | 12:48 | |
*** ociuhandu has joined #openstack-nova | 12:49 | |
sean-k-mooney | gibi: well my view is we should have been storing the acell profile in the request spec or in the instnace like we cache the flavor so we dont have this proble but ill go read what the patch acatully does now | 12:51 |
gibi | sean-k-mooney: it is like create an instance with accel, then remove cyborg service from deployment, then try to delete the instance which will fail | 12:52 |
sean-k-mooney | gibi: im kind of felling like we should do what ever we would do if cinder or neutron went away | 12:52 |
sean-k-mooney | i would assume we would fail to delete but i confess i have never tried it | 12:53 |
gibi | I can try that | 12:53 |
sean-k-mooney | cool you should jsut need to delete the entry in the service in keystone to emulate that right | 12:54 |
gibi | I'm not sure that a) this is a valid scenarios that cyborg service is removed from the deployment b) if yes then which is more important in general keeping cyborg - nova in sync or allowing to delete an instance during a temporary failure of cyborg | 12:55 |
sean-k-mooney | gibi: i am leaning towords this si operator error for removing a service before its consumer are removed | 12:56 |
sean-k-mooney | that said i dont think they should have to do db surgury to remove the instance | 12:56 |
*** nweinber has joined #openstack-nova | 12:58 | |
gibi | sean-k-mooney: without neutron endpoint in keystone the instance delete fails with HTTP 500 | 13:01 |
gibi | http://paste.openstack.org/show/792326/ | 13:02 |
sean-k-mooney | ack | 13:02 |
sean-k-mooney | ya that is baskcially what i expected | 13:02 |
sean-k-mooney | and i would expect the same for cyborg | 13:02 |
gibi | so this also points towards that the scenario of removing cyborg is invalid | 13:02 |
sean-k-mooney | yes i was going to comment with a proceedure to do that in the bug | 13:03 |
sean-k-mooney | but just removing it i think is invalid | 13:03 |
sean-k-mooney | *patch | 13:03 |
openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM - Test stable device rescue tests with BFV instances https://review.opendev.org/710050 | 13:05 |
sean-k-mooney | gibi: left a comment with my toughts | 13:06 |
gibi | sean-k-mooney: thanks | 13:06 |
*** lbragstad_ is now known as lbragstad | 13:15 | |
*** ratailor has quit IRC | 13:16 | |
*** ratailor has joined #openstack-nova | 13:17 | |
*** ratailor has quit IRC | 13:20 | |
*** ratailor has joined #openstack-nova | 13:23 | |
*** ttsiouts has joined #openstack-nova | 13:23 | |
*** ratailor has quit IRC | 13:26 | |
*** ratailor has joined #openstack-nova | 13:26 | |
*** ttsiouts has quit IRC | 13:32 | |
*** mkrai has joined #openstack-nova | 13:32 | |
*** ttsiouts has joined #openstack-nova | 13:32 | |
*** ratailor has quit IRC | 13:33 | |
*** ratailor has joined #openstack-nova | 13:35 | |
*** ratailor has quit IRC | 13:38 | |
gibi | sean-k-mooney, dansmith thanks for the reviews on the https://review.opendev.org/#/c/720670 I marked the bug invalid | 13:38 |
dansmith | cool | 13:38 |
*** rambo_li has joined #openstack-nova | 13:48 | |
sean-k-mooney | dansmith: gibi cross cell migrate was complete this cycle right it was not supported in train | 13:48 |
sean-k-mooney | well cross cell resize. same thing | 13:48 |
averi | dansmith, sean-k-mooney: sorry for the direct poke, got any pointer on the question above? | 13:48 |
sean-k-mooney | i did not see the quest can you repeat it. i can check scollback | 13:49 |
averi | ah sure, sorry | 13:49 |
averi | <averi> hi, anyone has an idea on why code changes are not picked up by nova_compute containers (OSP 13)? | 13:49 |
averi | <averi> even when spinning up a new image based on the nova-compute one, it doesn't work, code changes are there but they're like ignored all together | 13:49 |
averi | <averi> even killing one of the library files results in the binary not to fail like the file was still around | 13:49 |
averi | I cannot troubleshooting anything as soon as oslo service comes into play | 13:49 |
*** ttsiouts has quit IRC | 13:49 | |
sean-k-mooney | how are you modifying the code and restarting thet container | 13:50 |
sean-k-mooney | are you using docker directly | 13:50 |
*** lbragstad_ has joined #openstack-nova | 13:51 | |
sean-k-mooney | if you exec into the contienr with doeck an modify the code then do a docker restart for the container it will pick up the changes | 13:51 |
dansmith | sean-k-mooney: I haven't tried it, but yeah it was U not T | 13:51 |
averi | I spin up a new container based on the upstream image, then exec within it, modify the library (interested in pymysql debugging), then shut down the container and start it again | 13:51 |
sean-k-mooney | if you are using paunch it pull a clean copy of the image every time you get it to restat the container so your changes will be lost | 13:52 |
averi | sean-k-mooney, the changes are there in the code, they persist, but they are completely ignored | 13:52 |
sean-k-mooney | dansmith: ok just traiging a donwstream bug | 13:52 |
*** lbragstad has quit IRC | 13:52 | |
dansmith | averi: this is probably not the best channel for that question | 13:52 |
averi | dansmith, sorry, a contact internally mentioned this was a possible good channel for that type of question | 13:52 |
sean-k-mooney | averi: then i would guess its not using the files you are modifying and the kolla or ooo folks would be betere able to help | 13:53 |
averi | what would be the correct channel? | 13:53 |
sean-k-mooney | averi: #openstack-kolla or better #openstack-tripleo | 13:53 |
averi | thanks | 13:53 |
averi | mm.. 4 folks in #openstack-tripleo | 13:54 |
dansmith | averi: are you a redhatter? | 13:54 |
averi | yes | 13:54 |
sean-k-mooney | maybe that is not there normal channel | 13:55 |
*** dklyle has joined #openstack-nova | 13:55 | |
dansmith | averi: this is the upstream nova channel.. people here don't know anything about OSP :) I'll pm | 13:55 |
sean-k-mooney | averi: fyi is just #tripleo | 13:56 |
averi | ack | 13:56 |
gibi | sean-k-mooney: cross cell resize merged in Ussuri not supported in Train | 14:00 |
*** mlavalle has joined #openstack-nova | 14:00 | |
*** lbragstad_ has quit IRC | 14:00 | |
sean-k-mooney | gibi: yep i just have a resize bug specificly in a multi cell enve and i wanted to triple check that cross cell resize is not supported | 14:01 |
bauzas | gibi: sorry, I was not around since this morning | 14:02 |
bauzas | gibi: I haven't yet looked at the bugs | 14:02 |
bauzas | thanks for working on it | 14:03 |
*** lbragstad has joined #openstack-nova | 14:03 | |
rambo_li | sean-k-mooney: maybe their is an error, for the bug :https://bugs.launchpad.net/nova/+bug/1841932. when we set the metadata "hide_hypervisor_id=true"in aggreate,so we will go to the https://github.com/openstack/nova/blob/46a3bcd80b41e99ec4923c7cf3d0f8dd8505e97c/nova/scheduler/filters/aggregate_instance_extra_specs.py#L69-L71, but this should not break, because the aggregate_val doesn't match the req. | 14:03 |
openstack | Launchpad bug 1841932 in OpenStack Compute (nova) "hide_hypervisor_id extra_specs in nova flavor cannot pass AggregateInstanceExtraSpecsFilter" [Undecided,New] | 14:03 |
gibi | bauzas: no worries there was one potential regression around cyborg but we just ruled it out as unsupported scenario | 14:03 |
gibi | bauzas: and honestly I did not triaged any bug, I'm just reviewing https://review.opendev.org/#/c/719163 | 14:03 |
rambo_li | the aggregate_vals is "true" | 14:04 |
gmann | melwitt: stephenfin can i get your review on this which cover remaining policy of server (not reviewed previously ) - https://review.opendev.org/#/c/720104/ and its scope and defaults change patch. | 14:04 |
*** ttsiouts has joined #openstack-nova | 14:06 | |
*** ociuhandu has quit IRC | 14:07 | |
melwitt | gmann: yeah can do. I didn't review it yet because earlier patch in the stack I wanted to wait for johnthetubaguy since he spotted issues before. but you are right, can still look at the others | 14:08 |
gmann | melwitt: yeah, thanks. | 14:08 |
bauzas | gibi: ack, will do as well later | 14:09 |
melwitt | gibi: hi, I have small follow ups here for the host_status:unknown-only policy rule here if you could please look https://review.opendev.org/713295 | 14:09 |
gibi | melwitt: sure. looking | 14:09 |
melwitt | thanks | 14:09 |
*** dtantsur is now known as dtantsur|brb | 14:09 | |
*** mriedem has joined #openstack-nova | 14:10 | |
openstackgerrit | Andreas Jaeger proposed openstack/nova master: Remove Babel requirement https://review.opendev.org/720725 | 14:11 |
*** ociuhandu has joined #openstack-nova | 14:11 | |
rambo_li | sean-k-mooney: if we set the metadata "hide_hypervisor_id = true "in aggregate, so the code https://github.com/openstack/nova/blob/master/nova/scheduler/filters/aggregate_instance_extra_specs.py#L62 , the aggregate_vals in code is "true". so last will also return False.so workaround this is to set hw:hide_hypervisor_id replace hide_hypervisor_id in flavor extra_spec and aggregate metadata. | 14:23 |
*** ttsiouts has quit IRC | 14:31 | |
*** rambo_li has quit IRC | 14:31 | |
*** ttsiouts has joined #openstack-nova | 14:36 | |
gibi | melwitt: +A on the followup | 14:39 |
melwitt | danke | 14:39 |
*** ttsiouts has quit IRC | 14:48 | |
*** ttsiouts has joined #openstack-nova | 14:50 | |
melwitt | gmann, bauzas, gibi, dansmith: does anyone know about babel to help review this? https://review.opendev.org/720725 it seems ok based on the ML post but I don't understand it well enough whether there could be any issue with removing it as a requirement | 14:51 |
melwitt | stephenfin too | 14:51 |
bauzas | weirdo | 14:51 |
gibi | melwitt: I have to pass on that now. I hope somebody else can take it | 14:52 |
bauzas | we don't use babel for translations | 14:52 |
melwitt | it's not urgent, just wanted to mention it while people are around | 14:52 |
bauzas | but I can git blame | 14:52 |
bauzas | melwitt: wow, this was old | 14:53 |
bauzas | melwitt: https://review.opendev.org/#/c/3870/ | 14:53 |
bauzas | but the translation team no longer uses it AFAIK | 14:53 |
bauzas | (for a while) | 14:53 |
bauzas | https://docs.openstack.org/i18n/latest/infra.html | 14:54 |
bauzas | this is now zanata | 14:54 |
bauzas | so I think we can drop it | 14:54 |
bauzas | but I leave the decision to other folks :p | 14:54 |
melwitt | yeah. hm, so maybe we would want to remove all references. ajaeger had said babel is used to build translations so I was not sure what's going on | 14:55 |
stephenfin | melwitt: That _seems_ correct. You need Babel to run the e.g. 'python setup.py extract_messages' distutils command but I think _that_ would require it already be present, like setuptools | 14:55 |
bauzas | I'm just afraid of breaking things so close to the HardStringFreeze so I'd defer this to later | 14:55 |
melwitt | ah, nevermind | 14:55 |
stephenfin | Also, AJaeger suggested it and he knows these things so I'd trust him on that | 14:55 |
stephenfin | *on this | 14:55 |
bauzas | FWIW, that's how translations work atm https://docs.openstack.org/i18n/latest/reviewing-translation-import.html | 14:56 |
melwitt | I thought it'd be wise to see if anyone in the team understands this before merging it | 14:56 |
bauzas | melwitt: like I said, I would maybe defer this to post-RC1 | 14:57 |
melwitt | if literally no one understands it, then fine, but I wanted to check first | 14:57 |
bauzas | melwitt: we should first look at the translation jobs I think | 14:58 |
bauzas | or ask someone in their team | 14:58 |
bauzas | stephenfin: sean-k-mooney: trust me, if I was able to use the existing functional helpers, I'd use them | 15:00 |
bauzas | stephenfin: sean-k-mooney: but all those stuff like https://github.com/openstack/nova/blob/e1359567e4985e9a671359d4c0d53404a8ba64ab/nova/tests/functional/integrated_helpers.py#L1057-L1067 are from a class that's separated from the general base test class | 15:01 |
bauzas | if the whole thing was a mixin, I'd sign off for this | 15:01 |
bauzas | but using those helpers requires me to modify the inheritance of my own classes | 15:01 |
bauzas | stephenfin: sean-k-mooney: https://github.com/openstack/nova/blob/e1359567e4985e9a671359d4c0d53404a8ba64ab/nova/tests/functional/integrated_helpers.py#L498-L500 | 15:01 |
*** dtantsur|brb is now known as dtantsur | 15:03 | |
sean-k-mooney | bauzas: they are a mixin | 15:04 |
stephenfin | melwitt, bauzas: Okay, they're not using the setuptools integration any more so yes, we can remove it https://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/roles/prepare-zanata-client/files/common_translation_update.sh#L371-L379 | 15:04 |
stephenfin | Will ask AJaeger for more information in the commit message though | 15:04 |
bauzas | sean-k-mooney: huh ? https://github.com/openstack/nova/blob/e1359567e4985e9a671359d4c0d53404a8ba64ab/nova/tests/functional/integrated_helpers.py#L500 | 15:05 |
sean-k-mooney | oh they are not in the mixin class you are right | 15:07 |
sean-k-mooney | bauzas: that said you are doing testing with placment so you proably should be inheriting form ProviderUsageBaseTestCase | 15:08 |
bauzas | sean-k-mooney: this was my original plan | 15:08 |
sean-k-mooney | bauzas: why dont you move them up to the mixin class | 15:08 |
sean-k-mooney | e.g. the resize functions | 15:09 |
bauzas | sean-k-mooney: see the FIXME on top | 15:09 |
bauzas | sean-k-mooney: oh, you mean moving my own methods to integrated_helpers ? | 15:09 |
bauzas | I can do this for sure | 15:09 |
sean-k-mooney | i mean move the existing resize one form the ProviderUsageBaseTestCase to the mixin | 15:10 |
bauzas | sean-k-mooney: this would be doable | 15:10 |
sean-k-mooney | then you can use the mixin | 15:10 |
*** ociuhandu has quit IRC | 15:10 | |
sean-k-mooney | and other that want the mixin but not the other things can too | 15:10 |
gibi | bauzas: fyi you got answers in https://bugs.launchpad.net/nova/+bug/1860913 | 15:11 |
openstack | Launchpad bug 1860913 in OpenStack Compute (nova) "Instance uses base image file when it is rebooted after snapshot creation if cinder nfs backend is used" [Undecided,New] | 15:11 |
bauzas | gibi: this looks a bug then | 15:12 |
*** belmoreira has quit IRC | 15:12 | |
bauzas | lyarwood: https://bugs.launchpad.net/nova/+bug/1860913 we got answers on this one, I'm about to confirm it | 15:13 |
openstack | Launchpad bug 1860913 in OpenStack Compute (nova) "Instance uses base image file when it is rebooted after snapshot creation if cinder nfs backend is used" [Undecided,New] | 15:13 |
gibi | bauzas: yeah it is almost even triaged by pointing to the problem of the missing bdm update | 15:13 |
sean-k-mooney | gibi: that sound familar | 15:13 |
bauzas | gibi: agreed, I think we could dup this one | 15:13 |
gibi | but I haven't checked the code | 15:13 |
sean-k-mooney | lyarwood: where you looking into https://bugs.launchpad.net/nova/+bug/1860913 downstream | 15:13 |
*** ociuhandu has joined #openstack-nova | 15:13 | |
gibi | I let you folks to handle it :) | 15:14 |
bauzas | the 'you folks' is important :) | 15:14 |
bauzas | :D | 15:14 |
lyarwood | sean-k-mooney: nope I don't recall seeing this | 15:15 |
sean-k-mooney | gibi: well this is basically an issue with how we handel the snapshot | 15:15 |
sean-k-mooney | https://bugzilla.redhat.com/show_bug.cgi?id=1757691#c1 | 15:15 |
openstack | bugzilla.redhat.com bug 1757691 in openstack-nova "Instance uses base image file when it is rebooted after snapshot creation if cinder nfs backend is used" [Urgent,New] - Assigned to nova-maint | 15:15 |
lyarwood | looks like Matt triaged it a while ago | 15:15 |
sean-k-mooney | we discussed it on the triage call | 15:15 |
sean-k-mooney | whats happening is during the snapshot we cate a path form cinder to use a a temp file or something which updates the xml but if you do a hard reboot we regenerate it and use the one it had beofre | 15:16 |
sean-k-mooney | the real fix is to not allow reboots while we do snapshots | 15:17 |
bauzas | either way, the upstream bug is triaged with no assignee | 15:17 |
bauzas | feel free to pick it, teammates | 15:17 |
lyarwood | I'll take it | 15:18 |
bauzas | lyarwood: thanks | 15:18 |
lyarwood | we should be able to fix this easily by updating the connection_info | 15:18 |
*** mkrai has quit IRC | 15:18 | |
sean-k-mooney | maybe i still think we need to lock the instance when you intialise the snapshot of the volume via cinder somehow | 15:19 |
stephenfin | bauzas: RE: the integrated_helpers duplication, I'm okay with you doing that as a follow-up, so long as it's actually done and you stick a TODO in in your patch saying that it will be done :) | 15:23 |
stephenfin | Will happily review the follow-up too | 15:23 |
sean-k-mooney | ya i guss i would be fine with that as well. | 15:23 |
stephenfin | need the TODOs though | 15:24 |
*** maciejjozefczyk has quit IRC | 15:34 | |
gibi | lyarwood: do you think https://bugs.launchpad.net/nova/+bug/1858402 is a duplicate of https://bugs.launchpad.net/nova/+bug/1868996 ? | 15:37 |
openstack | Launchpad bug 1858402 in OpenStack Compute (nova) "In-use volume does not support the same ceph cluster retype" [Undecided,New] | 15:37 |
openstack | Launchpad bug 1868996 in OpenStack Compute (nova) "Unable to swap between RBD volumes using virDomainBlockRebase" [High,Fix released] - Assigned to Lee Yarwood (lyarwood) | 15:37 |
*** ociuhandu has quit IRC | 15:40 | |
*** sean-k-mooney has quit IRC | 15:41 | |
*** ociuhandu has joined #openstack-nova | 15:42 | |
bauzas | stephenfin: nah, no worries, I'm pretty done with the new revision, since I need to update it for the main revert resize issue that dansmith said, I won't provide a FUP | 15:46 |
*** ociuhandu has quit IRC | 15:47 | |
lyarwood | gibi: hey sorry yes that's a duplicate | 15:49 |
bauzas | gibi: any idea why I got this weird exception : http://paste.openstack.org/show/792339/ | 15:49 |
*** dpawlik has quit IRC | 15:50 | |
gibi | lyarwood: tahnks | 15:51 |
lyarwood | gibi: updated the bug | 15:51 |
gibi | lyarwood: thanks a lot | 15:51 |
lyarwood | np | 15:52 |
gibi | bauzas: looking | 15:52 |
bauzas | thanks, I know it's late | 15:52 |
bauzas | gibi: and that's what I wait for : | 15:53 |
bauzas | fake_notifier.wait_for_versioned_notifications( | 15:53 |
bauzas | 'instance.resize_revert.end') | 15:53 |
bauzas | oh, but I understand the error, I was expecting 'instance.resize_revert' but i never got it | 15:54 |
bauzas | weirdo | 15:54 |
gibi | that is my guess yes | 15:54 |
gibi | you can print all the notification that you received | 15:54 |
bauzas | gibi: the paste says [] | 15:55 |
gibi | fake_notifier.VERSIONED_NOTIFICATIONS | 15:55 |
bauzas | do I need to setup something specific in setUp ? | 15:55 |
gibi | you need to call stub_notifier() | 15:56 |
gibi | from fake_notifier | 15:56 |
bauzas | gibi: I moved https://github.com/openstack/nova/blob/cd16ae2/nova/tests/functional/integrated_helpers.py#L1009-L1019 to https://github.com/openstack/nova/blob/cd16ae2/nova/tests/functional/integrated_helpers.py#L75 | 15:57 |
gibi | fake_notifier.stub_notifier(self) | 15:57 |
gibi | self.addCleanup(fake_notifier.reset) | 15:57 |
bauzas | ah, thanks | 15:57 |
bauzas | TIL | 15:57 |
gibi | cool | 15:58 |
*** dustinc has joined #openstack-nova | 16:00 | |
gibi | OK, I'm leaving for today. See you on Monday | 16:02 |
*** tesseract has quit IRC | 16:03 | |
*** ociuhandu has joined #openstack-nova | 16:05 | |
openstackgerrit | Lee Yarwood proposed openstack/nova master: WIP block_device: Use the attachment update API to refresh connection_info https://review.opendev.org/720769 | 16:07 |
*** sapd1 has quit IRC | 16:07 | |
bauzas | gibi: \o | 16:08 |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: Allocate mdevs when resizing or reverting resize https://review.opendev.org/712741 | 16:09 |
bauzas | stephenfin: ^ updated the last change for the vgpu resize thing | 16:09 |
stephenfin | ack | 16:10 |
*** ttsiouts has quit IRC | 16:12 | |
*** ociuhandu has quit IRC | 16:15 | |
*** ociuhandu has joined #openstack-nova | 16:16 | |
*** dtantsur is now known as dtantsur|afk | 16:16 | |
*** rpittau is now known as rpittau|afk | 16:17 | |
*** sapd1 has joined #openstack-nova | 16:19 | |
*** ociuhandu has quit IRC | 16:21 | |
*** udesale_ has quit IRC | 16:23 | |
*** hamzy__ has joined #openstack-nova | 16:31 | |
* bauzas disappears for the week-end, bye folks | 16:31 | |
*** gyee has joined #openstack-nova | 16:31 | |
*** hamzy_ has quit IRC | 16:31 | |
*** yaawang_ has joined #openstack-nova | 16:33 | |
*** dtruong has quit IRC | 16:33 | |
*** yaawang has quit IRC | 16:34 | |
*** mlavalle has quit IRC | 16:34 | |
*** dtruong has joined #openstack-nova | 16:34 | |
*** evrardjp has quit IRC | 16:37 | |
*** evrardjp has joined #openstack-nova | 16:37 | |
*** kevinz has quit IRC | 16:40 | |
*** ociuhandu has joined #openstack-nova | 16:40 | |
*** mlavalle has joined #openstack-nova | 16:43 | |
*** nightmare_unreal has quit IRC | 16:44 | |
*** ociuhandu has quit IRC | 17:03 | |
*** ttsiouts has joined #openstack-nova | 17:04 | |
*** igordc has joined #openstack-nova | 17:04 | |
*** igordc has quit IRC | 17:09 | |
*** derekh has quit IRC | 17:12 | |
*** links has quit IRC | 17:26 | |
*** ociuhandu has joined #openstack-nova | 17:30 | |
*** ociuhandu has quit IRC | 17:43 | |
*** ralonsoh has quit IRC | 17:53 | |
*** sapd1 has quit IRC | 17:58 | |
*** tbachman_ has joined #openstack-nova | 18:00 | |
*** tbachman has quit IRC | 18:02 | |
*** tbachman_ is now known as tbachman | 18:02 | |
*** tbachman_ has joined #openstack-nova | 18:05 | |
*** tbachman has quit IRC | 18:08 | |
*** tbachman_ is now known as tbachman | 18:08 | |
*** ociuhandu has joined #openstack-nova | 18:09 | |
*** ttsiouts has quit IRC | 18:14 | |
*** ociuhandu has quit IRC | 18:22 | |
*** ociuhandu has joined #openstack-nova | 18:23 | |
artom | *snerk* _get_all_rp_uuids_in_a_tree() | 18:26 |
artom | Why does that make me giggle | 18:26 |
artom | Maybe because I'm imagining an actual tree | 18:26 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add docs and releasenotes for BP policy-defaults-refresh https://review.opendev.org/720129 | 18:26 |
artom | Or that song... | 18:26 |
*** ociuhandu has quit IRC | 18:28 | |
openstackgerrit | melanie witt proposed openstack/nova stable/pike: Fix os-simple-tenant-usage result order https://review.opendev.org/709600 | 18:29 |
openstackgerrit | melanie witt proposed openstack/nova stable/ocata: Fix os-simple-tenant-usage result order https://review.opendev.org/709603 | 18:36 |
openstackgerrit | melanie witt proposed openstack/nova stable/ocata: Fix os-simple-tenant-usage result order https://review.opendev.org/709603 | 18:37 |
*** grandchild has joined #openstack-nova | 19:16 | |
artom | Oh, I forgot func tests are broken on queens | 19:25 |
artom | sqlalchemy.exc.NoSuchTableError: migration_tmp | 19:26 |
*** ttsiouts has joined #openstack-nova | 19:26 | |
artom | https://bugs.launchpad.net/ubuntu/+source/nova/+bug/1807262 ? | 19:27 |
openstack | Launchpad bug 1807262 in nova (Ubuntu) "stein unit tests fail with sqlalchemy.exc.NoSuchTableError: migration_tmp" [High,Fix released] | 19:27 |
*** ociuhandu has joined #openstack-nova | 19:35 | |
*** ociuhandu has quit IRC | 19:42 | |
*** ttsiouts has quit IRC | 19:47 | |
*** ttsiouts has joined #openstack-nova | 19:47 | |
*** dustinc has quit IRC | 20:29 | |
*** ociuhandu has joined #openstack-nova | 20:45 | |
*** ociuhandu has quit IRC | 21:06 | |
*** nweinber has quit IRC | 21:06 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova stable/train: DNM: Add a placement audit command https://review.opendev.org/720838 | 21:07 |
*** ociuhandu has joined #openstack-nova | 21:08 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova stable/stein: DNM: Add a placement audit command https://review.opendev.org/720839 | 21:10 |
*** ociuhandu has quit IRC | 21:13 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova stable/rocky: DNM: Partial cherry-pick of assertRequestMatchesUsage() https://review.opendev.org/720840 | 21:21 |
openstackgerrit | Artom Lifshitz proposed openstack/nova stable/rocky: DNM: Partial cherry-pick of _check_allocation_during_evacuate() https://review.opendev.org/720841 | 21:21 |
openstackgerrit | Artom Lifshitz proposed openstack/nova stable/rocky: DNM: Add a placement audit command https://review.opendev.org/720842 | 21:21 |
artom | I'm going to let CI chew on that as an extra layer of safety before even thinking about the rocky -> queens backport | 21:22 |
*** ccamacho has quit IRC | 21:22 | |
artom | But it's all going downstream-only eventually | 21:22 |
gmann | dansmith: lbragstad melwitt artom updated the review comments. this is ready for another re-review - https://review.opendev.org/#/c/720129/ | 21:29 |
artom | gmann, will take a look after supper, I guess | 21:30 |
artom | gmann, btw, if my initial comments came off as flippant or "agressive" that was not at all the intention | 21:30 |
artom | I guess I was having a bad say, or was annoyed at something else | 21:30 |
gmann | artom: :) those were good feedback. | 21:31 |
artom | Maybe I could get a screen reader so I can listen to that doc while making supper... | 21:31 |
gmann | hehe or invite me for dinner, i can read for you :) | 21:33 |
artom | gmann, hahaha | 21:33 |
*** ttsiouts has quit IRC | 21:56 | |
*** mgariepy has quit IRC | 22:06 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add docs and releasenotes for BP policy-defaults-refresh https://review.opendev.org/720129 | 22:11 |
*** xek has quit IRC | 22:12 | |
*** ociuhandu has joined #openstack-nova | 22:32 | |
*** mriedem has left #openstack-nova | 22:34 | |
*** ociuhandu has quit IRC | 22:38 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova master: Allocate mdevs when resizing or reverting resize https://review.opendev.org/712741 | 22:40 |
*** ttsiouts has joined #openstack-nova | 22:43 | |
*** ttsiouts has quit IRC | 22:52 | |
*** grandchild has quit IRC | 22:55 | |
*** igordc has joined #openstack-nova | 23:08 | |
*** gyee has quit IRC | 23:10 | |
*** tosky has quit IRC | 23:21 | |
openstackgerrit | melanie witt proposed openstack/nova stable/pike: Fix os-simple-tenant-usage result order https://review.opendev.org/709600 | 23:23 |
openstackgerrit | melanie witt proposed openstack/nova stable/pike: Require stestr < 3.0.0 in test-requirements https://review.opendev.org/720849 | 23:23 |
openstackgerrit | melanie witt proposed openstack/nova stable/pike: Require stestr < 3.0.0 in test-requirements https://review.opendev.org/720849 | 23:25 |
openstackgerrit | melanie witt proposed openstack/nova stable/pike: Fix os-simple-tenant-usage result order https://review.opendev.org/709600 | 23:25 |
*** igordc has quit IRC | 23:27 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!