| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-06-08 | |||
| 19:58:33 | sean-k-mooney | policies-irrelevant-files | |
| 19:58:37 | sean-k-mooney | on line 390 | |
| 19:59:14 | sean-k-mooney | commented | |
| 19:59:21 | dansmith | oh that wasn't in yours I guess so I just block-removed | |
| 19:59:59 | sean-k-mooney | ya just comment out the jobs in gate and you shoudl be fine | |
| 20:00:35 | sean-k-mooney | it would be nice if there was a cleaner way to do this but it is what it is | |
| 20:00:54 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184 | |
| 20:01:05 | dansmith | thanks | |
| 20:01:38 | sean-k-mooney | no worries that has bit me in the past too | |
| 20:02:45 | sean-k-mooney | looking at https://zuul.openstack.org/status its queue so i think it passed syntax checking | |
| 20:03:01 | sean-k-mooney | zuul is really quick to comment abck if there is a sysntax error | |
| 20:03:26 | sean-k-mooney | although it was a bit delayed that time | |
| 21:19:39 | sean-k-mooney | dansmith: looks like your job jsut started running https://zuul.openstack.org/stream/98f29ccd122c4ff080aca6cd71be7d52?logfile=console.log so it will be a while but i proably will work at this point assuming the glance config options were correct | |
| 21:22:39 | dansmith | sweet | |
| #openstack-nova - 2020-06-09 | |||
| 00:34:18 | openstackgerrit | melanie witt proposed openstack/nova master: Raise InstanceMappingNotFound if StaleDataError is encountered https://review.opendev.org/734400 | |
| 01:59:25 | alex_xu | gibi: do we have priorities for this release? | |
| 06:10:34 | openstackgerrit | Elancheran S proposed openstack/nova master: Removes the delta file once image is extracted https://review.opendev.org/734421 | |
| 06:13:23 | openstackgerrit | Elancheran S proposed openstack/nova master: Removes the delta file once image is extracted https://review.opendev.org/734421 | |
| 08:49:23 | openstackgerrit | Hervé Beraud proposed openstack/nova master: Use unittest.mock instead of mock https://review.opendev.org/734447 | |
| 08:49:23 | openstackgerrit | Hervé Beraud proposed openstack/nova master: Use unittest.mock instead of mock https://review.opendev.org/734447 | |
| 09:36:47 | gibi | alex_xu: we did not talk about priorities. If you feel it is needed then start a thread on the ML about it and let see what list we can put together | |
| 09:37:10 | gibi | alex_xu: I myself would prioritize the whathever thing will end up in a runway slot | |
| 09:39:07 | brinzhang0 | Luckily we have runway slot to track^ | |
| 09:39:44 | gibi | brinzhang0: yeah, I polished the etherpad for victoria yesterday | |
| 10:51:29 | sean-k-mooney | gibi: i think it would be nice to prioities some of the feature that were close but did not quite make it and see if we can get them merged early (before m1) so that they dont drag on | |
| 10:51:40 | sean-k-mooney | gibi: im thinking of the provider.yaml file | |
| 10:52:39 | sean-k-mooney | im sure there are other features that were similarly close. | |
| 10:55:03 | lvdombrkr | hello guys, any updates on this? https://review.opendev.org/#/c/733982/ | |
| 10:56:13 | sean-k-mooney | lvdombrkr: it looks like the review feed back has not been adressed | |
| 10:56:50 | sean-k-mooney | so untill that is done its not going to make any more progress | |
| 10:57:37 | lvdombrkr | sean-k-mooney: thanks for answer. who can/should proceed this? | |
| 10:57:40 | sean-k-mooney | it was also only upladed 2 days ago so its a relitivly new patch an di suspect frickler has not had time to look at the comments form yesterday yet | |
| 10:58:17 | lvdombrkr | sean-k-mooney: ok, i see, thank you for detailed answer | |
| 10:58:20 | sean-k-mooney | lvdombrkr: well frickler is the authour but if your are interested in it reach out to them and ask if you can help | |
| 11:00:32 | gibi | sean-k-mooney: I encurage the authors of such patch to put them in the runway queue. If there are priority feature and there are patches in the runway then we could easily get confliting priorities. Still as I said to alex_xu let's start an ML thread about it if you feel that project priorities helps | |
| 11:02:28 | sean-k-mooney | gibi: yep i know that is definetly the way to go, im not sure if project prioritys are needed but they can sometimes be helpful. unified limits would be high on my personal list of project prioites but i have not really looked at the rest of what is proposed to intuit what else shoudl be on that list | |
| 11:04:17 | gibi | I did not have a list either | |
| 11:10:28 | frickler | lvdombrkr: I don't have much time currently to work on the inject fix, if you or someone else wants to take over, feel free to do so. seems this is getting ever more complicated from the simple py2/3 adaption I assumed initially :-/ | |
| 11:12:25 | sean-k-mooney | frickler: lvdombrkr i think the main feedback is keep the data as text strings and only convert to bytes if and when needed to do so. althoug i just read the top level coments and didnt actully review the patch | |
| 11:36:20 | lvdombrkr | frickler sean-k-mooney : ok i see guys, unfrortinetly i will be not able contribute by myself because im not too confident on this.. | |
| 14:41:09 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184 | |
| 15:40:46 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/stein: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732696 | |
| 15:50:47 | stephenfin | melwitt: Could I get your eyes on https://review.opendev.org/#/q/topic:bug/1550919+status:open today, please? | |
| 16:04:44 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/rocky: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732708 | |
| 16:17:00 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: Add functional test for bug 1550919 https://review.opendev.org/732715 | |
| 16:17:00 | openstack | bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,Fix released] https://launchpad.net/bugs/1550919 - Assigned to Lee Yarwood (lyarwood) | |
| 16:17:00 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: libvirt: Fix misleading debug msg "Instance is running" https://review.opendev.org/732716 | |
| 16:17:00 | openstackgerrit | Stephen Finucane proposed openstack/nova stable/queens: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732717 | |
| 16:37:52 | openstackgerrit | Elancheran S proposed openstack/nova master: Removes the delta file once image is extracted https://review.opendev.org/734421 | |
| 17:48:52 | openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Don't allow "reserving" file-backed memory https://review.opendev.org/734658 | |
| 17:49:00 | stephenfin | sean-k-mooney: ^ | |
| 17:54:27 | sean-k-mooney | stephenfin: oh you have a fix altreay i asumed you were just going to file the bug | |
| 17:55:35 | sean-k-mooney | stephenfin: the code does not do what the title says | |
| 17:56:00 | sean-k-mooney | we still allow you to reserve the memory | |
| 18:02:09 | sean-k-mooney | dansmith: https://review.opendev.org/#/c/734184/ seames to work | |
| 18:02:16 | sean-k-mooney | dansmith: did you enable any test that use it | |
| 18:07:24 | sean-k-mooney | there is a bunch of stuff like this https://zuul.opendev.org/t/openstack/build/fe828e4e683841f5a079bb9342c48493/log/controller/logs/screen-n-cpu.txt#34772 | |
| 18:07:37 | sean-k-mooney | asking the Image API where its store is | |
| 18:07:54 | sean-k-mooney | then creating snapshot(45941487007e4ecb876ca326b1245c50) on rbd image(c6f91c2a-085f-469e-a5f7-96652d96e9da_disk) | |
| 18:10:04 | sean-k-mooney | i gues this rbd image import is what your looking to see https://zuul.opendev.org/t/openstack/build/fe828e4e683841f5a079bb9342c48493/log/controller/logs/screen-n-cpu.txt#12725 | |
| 18:11:16 | sean-k-mooney | in all the other cases it seams to just create a snapshot of the already imported image | |
| 18:12:14 | sean-k-mooney | dansmith: so that shows the its working properly right ^ | |
| 18:13:04 | sean-k-mooney | we are now creatinging thin image snapshots on the RBD pool instead of copying the image over and over again | |
| 18:32:25 | melwitt | stephenfin: will do | |
| 18:33:32 | melwitt | elod: would you mind taking a look at these too please https://review.opendev.org/#/q/topic:bug/1550919+status:open | |
| 19:09:50 | dansmith | sean-k-mooney: I just got back. I think I'll need to further tweak it to make sure it uploads to one store and does the copy to the other, unless the default store thing magically made that happen | |
| 19:09:52 | dansmith | will check in a biot | |
| 19:35:57 | elod | melwitt: started to look at them but was a bit distracted, will do tomorrow (my) morning | |
| 19:36:25 | melwitt | thank you :) | |
| 19:37:04 | elod | no problem :] | |
| 19:46:53 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184 | |
| 22:28:51 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184 | |
| 22:29:52 | openstackgerrit | Dan Smith proposed openstack/nova master: DNM: Try to make a glance multistore job https://review.opendev.org/734184 | |
| #openstack-nova - 2020-06-10 | |||
| 00:05:34 | openstackgerrit | melanie witt proposed openstack/nova stable/ussuri: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/731248 | |
| 00:44:09 | openstackgerrit | melanie witt proposed openstack/nova stable/train: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/731273 | |
| 02:24:53 | openstackgerrit | melanie witt proposed openstack/nova stable/stein: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732696 | |
| 02:40:36 | openstackgerrit | sunhao proposed openstack/nova master: Add checks for volume status when rebuilding https://review.opendev.org/734734 | |
| 02:49:28 | openstackgerrit | melanie witt proposed openstack/nova stable/rocky: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732708 | |
| 03:07:50 | openstackgerrit | melanie witt proposed openstack/nova stable/queens: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/732717 | |
| 06:41:48 | tony_su | dansmith: A reviewer of ProviderConfigFile patch wanted to understand why additionProperties is set to true by default in the first place. https://review.opendev.org/#/c/673341/45/nova/compute/provider_config_schemas/provider_config_schema_v1.yaml might require your attention. Thanks. | |
| 07:09:45 | bauzas | good morning Nova | |
| 07:20:37 | gibi | bauzas: good morning | |
| 07:21:12 | bauzas | gibi: morning | |
| 07:21:29 | bauzas | fwiw, just created a blueprint for tracking reshape tooling work for Ussuri https://blueprints.launchpad.net/nova/+spec/offline-reshape-tool | |
| 07:22:46 | gibi | bauzas: cool. Do you agree that we had enough discussion about FFU on the PTG so that I can approve that bp without further discussion on the weekly meeting? | |
| 07:23:28 | openstackgerrit | sunhao proposed openstack/nova master: Add checks for volume status when rebuilding https://review.opendev.org/734734 | |
| 07:31:04 | bauzas | gibi: well, I'm torn about it | |
| 07:31:28 | bauzas | gibi: we discussed at the PTG about how to call placement without having the service running | |
| 07:31:37 | bauzas | but i'm not sure we had a consensus | |
| 07:32:22 | bauzas | we have a way to do it that was accepted for the reshape spec in https://specs.openstack.org/openstack/nova-specs/specs/stein/approved/reshape-provider-tree.html#direct-interface-to-placement | |
| 07:35:28 | gibi | bauzas: OK, then let's double check this with the team on the weekly meeting. | |
| 07:35:36 | bauzas | thanks | |
| 07:35:49 | gibi | I will add it to the agenda | |
| 07:36:04 | bauzas | gibi: looking at the spec, looks like we agreed on using PlacementDirect for this | |
| 07:36:28 | bauzas | so if folks agree in the meeting, we can have a specless bp | |
| 07:36:36 | gibi | bauzas: yes, but on the PTG I was tried to mention that fact and get a pushback because that would require DB access from the compute nodes | |
| 07:36:58 | bauzas | gibi: PlacementDirect supports oslo.config | |
| 07:37:28 | bauzas | so you would run the nova-manage command that would call PlacementDirect with DB options | |
| 07:37:29 | gibi | true, but our deployment model does not require db access for compute hosts so far | |
| 07:37:53 | gibi | so the db server might not even accessible from the compute hosts | |