*** haomaiwang has joined #openstack-nova | 00:03 | |
*** haomaiw__ has quit IRC | 00:03 | |
*** haomaiwang has quit IRC | 00:13 | |
*** zhenguo has joined #openstack-nova | 00:19 | |
*** otter768 has joined #openstack-nova | 00:19 | |
*** otter768 has quit IRC | 00:24 | |
*** Longgeek has quit IRC | 00:29 | |
*** ferest has quit IRC | 00:33 | |
*** ferest has joined #openstack-nova | 00:34 | |
*** ferest has quit IRC | 00:38 | |
*** bjornar has quit IRC | 00:38 | |
*** bjornar has joined #openstack-nova | 00:42 | |
*** eliqiao has joined #openstack-nova | 00:44 | |
*** eliqiao has left #openstack-nova | 00:46 | |
*** eliqiao has joined #openstack-nova | 00:46 | |
*** yingjun has joined #openstack-nova | 00:47 | |
openstackgerrit | Eric Brown proposed openstack/nova: VMware: typo fix in config option help https://review.openstack.org/191596 | 00:48 |
---|---|---|
*** atuvenie has quit IRC | 00:50 | |
*** tjones2 has joined #openstack-nova | 00:57 | |
*** eliqiao has left #openstack-nova | 00:58 | |
*** tjones1 has quit IRC | 01:03 | |
*** liusheng has quit IRC | 01:09 | |
*** tjones2 has quit IRC | 01:18 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Specify current directory during processutils.execute using new cwd param https://review.openstack.org/191604 | 01:23 |
*** mdrabe has quit IRC | 01:29 | |
*** dboik has quit IRC | 01:33 | |
*** tjones1 has joined #openstack-nova | 01:39 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs: Spec to show server addresses consistently https://review.openstack.org/191605 | 01:39 |
*** tjones1 has quit IRC | 01:40 | |
*** Longgeek has joined #openstack-nova | 01:47 | |
*** changbl has joined #openstack-nova | 02:07 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs: Spec to show server addresses consistently https://review.openstack.org/191605 | 02:13 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Specify current directory during processutils.execute using new cwd param https://review.openstack.org/191604 | 02:15 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs: Spec to show server addresses consistently https://review.openstack.org/191605 | 02:19 |
*** otter768 has joined #openstack-nova | 02:20 | |
*** Longgeek has quit IRC | 02:20 | |
*** dimsum__ has quit IRC | 02:22 | |
*** davideagnello has joined #openstack-nova | 02:23 | |
*** otter768 has quit IRC | 02:24 | |
*** yingjun has quit IRC | 02:25 | |
*** yuntong1 has left #openstack-nova | 02:25 | |
*** yingjun has joined #openstack-nova | 02:28 | |
*** dimsum__ has joined #openstack-nova | 02:36 | |
*** cfriesen has joined #openstack-nova | 02:39 | |
*** bkopilov has quit IRC | 02:40 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova-specs: Spec to show server addresses consistently https://review.openstack.org/191605 | 02:41 |
*** kiran-r has joined #openstack-nova | 02:51 | |
*** dsanders has joined #openstack-nova | 02:53 | |
*** vilobhmm has joined #openstack-nova | 02:55 | |
*** dimsum__ has quit IRC | 02:58 | |
*** oomichi has joined #openstack-nova | 03:02 | |
*** yingjun has quit IRC | 03:09 | |
*** dsanders has quit IRC | 03:11 | |
*** dsanders has joined #openstack-nova | 03:12 | |
*** dboik has joined #openstack-nova | 03:15 | |
*** Longgeek has joined #openstack-nova | 03:18 | |
*** ildikov_ has quit IRC | 03:19 | |
*** tobe has joined #openstack-nova | 03:20 | |
*** dsanders has quit IRC | 03:22 | |
*** cfriesen has quit IRC | 03:30 | |
*** yingjun has joined #openstack-nova | 03:31 | |
*** coolsvap|afk is now known as coolsvap | 03:32 | |
*** logan2 has quit IRC | 03:35 | |
*** hanzhang has joined #openstack-nova | 03:43 | |
*** kiran-r has quit IRC | 03:44 | |
*** kiran-r has joined #openstack-nova | 03:44 | |
*** veena has joined #openstack-nova | 03:45 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/191623 | 03:46 |
*** Longgeek has quit IRC | 03:51 | |
*** salv-orl_ has quit IRC | 03:55 | |
*** vilobhmm has quit IRC | 03:55 | |
*** bkopilov has joined #openstack-nova | 03:55 | |
*** amotoki has joined #openstack-nova | 03:56 | |
*** dimsum__ has joined #openstack-nova | 03:58 | |
*** irenab has quit IRC | 04:01 | |
*** pixelb has quit IRC | 04:01 | |
*** dimsum__ has quit IRC | 04:03 | |
*** amotoki has quit IRC | 04:04 | |
*** shihanzhang has joined #openstack-nova | 04:05 | |
*** bkopilov has quit IRC | 04:06 | |
*** achanda has joined #openstack-nova | 04:07 | |
*** yingjun has quit IRC | 04:07 | |
*** amotoki has joined #openstack-nova | 04:08 | |
*** hanzhang has quit IRC | 04:09 | |
*** achanda has quit IRC | 04:17 | |
*** bkopilov has joined #openstack-nova | 04:18 | |
*** otter768 has joined #openstack-nova | 04:21 | |
*** kiran-r has quit IRC | 04:25 | |
*** otter768 has quit IRC | 04:26 | |
*** amotoki has quit IRC | 04:31 | |
*** kiran-r has joined #openstack-nova | 04:32 | |
*** kiranr has joined #openstack-nova | 04:33 | |
*** unicell has quit IRC | 04:34 | |
*** unicell has joined #openstack-nova | 04:35 | |
*** takedakn has joined #openstack-nova | 04:35 | |
*** kiran-r has quit IRC | 04:37 | |
*** kiranr is now known as kiran-r | 04:37 | |
*** takedakn has quit IRC | 04:39 | |
*** arnaud___ has joined #openstack-nova | 04:54 | |
*** sdake_ has joined #openstack-nova | 04:56 | |
*** sdake has quit IRC | 04:59 | |
*** davideagnello has quit IRC | 05:00 | |
*** pradipta has joined #openstack-nova | 05:00 | |
*** flwang1 has quit IRC | 05:08 | |
*** Longgeek has joined #openstack-nova | 05:09 | |
*** atuvenie has joined #openstack-nova | 05:11 | |
*** nihilifer has joined #openstack-nova | 05:18 | |
*** atuvenie has quit IRC | 05:19 | |
*** nkrinner has joined #openstack-nova | 05:20 | |
openstackgerrit | Josh Durgin proposed openstack/nova-specs: RBD instance snapshots https://review.openstack.org/188244 | 05:27 |
*** edsh has quit IRC | 05:31 | |
*** Longgeek has quit IRC | 05:35 | |
*** kiran-r has quit IRC | 05:39 | |
*** edsh has joined #openstack-nova | 05:40 | |
*** davideagnello has joined #openstack-nova | 05:42 | |
*** agireud has joined #openstack-nova | 05:43 | |
*** davideagnello has quit IRC | 05:45 | |
*** yuntong has joined #openstack-nova | 05:46 | |
*** belmoreira has joined #openstack-nova | 05:46 | |
*** bradjones has quit IRC | 05:46 | |
*** deepthi has joined #openstack-nova | 05:48 | |
*** veena has quit IRC | 05:48 | |
*** bradjones has joined #openstack-nova | 05:49 | |
*** bradjones has quit IRC | 05:49 | |
*** bradjones has joined #openstack-nova | 05:49 | |
*** agireud has quit IRC | 05:50 | |
*** davideagnello has joined #openstack-nova | 05:51 | |
*** garyk has joined #openstack-nova | 05:55 | |
*** yuntong1 has joined #openstack-nova | 05:56 | |
*** yuntong has quit IRC | 05:57 | |
*** tteggel has quit IRC | 05:59 | |
*** tteggel has joined #openstack-nova | 06:01 | |
*** masayukig has quit IRC | 06:02 | |
*** Kennan2 has joined #openstack-nova | 06:03 | |
*** Kennan has quit IRC | 06:04 | |
*** shihanzhang has quit IRC | 06:06 | |
*** shihanzhang has joined #openstack-nova | 06:06 | |
*** masayukig has joined #openstack-nova | 06:07 | |
*** ijw has joined #openstack-nova | 06:08 | |
*** liusheng has joined #openstack-nova | 06:09 | |
*** radek__ has joined #openstack-nova | 06:09 | |
*** eglynn has joined #openstack-nova | 06:11 | |
*** vilobhmm has joined #openstack-nova | 06:12 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex https://review.openstack.org/190034 | 06:13 |
*** Kennan2 is now known as Kennan | 06:14 | |
*** vilobhmm has quit IRC | 06:16 | |
*** vilobhmm has joined #openstack-nova | 06:17 | |
*** Longgeek has joined #openstack-nova | 06:17 | |
*** mpaolino has joined #openstack-nova | 06:19 | |
*** lpetrut has joined #openstack-nova | 06:21 | |
*** otter768 has joined #openstack-nova | 06:22 | |
*** davideagnello has quit IRC | 06:25 | |
*** otter768 has quit IRC | 06:27 | |
*** pbandzi has joined #openstack-nova | 06:34 | |
*** jyuso1 has joined #openstack-nova | 06:35 | |
*** takedakn has joined #openstack-nova | 06:39 | |
openstackgerrit | wangxiyuan proposed openstack/nova: Add specific error information when createBackup is not supported https://review.openstack.org/187445 | 06:41 |
*** ian_brown has quit IRC | 06:43 | |
*** tobe has quit IRC | 06:44 | |
*** arnaud___ has quit IRC | 06:44 | |
*** sahid has joined #openstack-nova | 06:45 | |
*** takedakn has quit IRC | 06:46 | |
*** kiran-r has joined #openstack-nova | 06:54 | |
*** bkopilov has quit IRC | 06:56 | |
*** sdake has joined #openstack-nova | 06:56 | |
*** sdake has quit IRC | 06:57 | |
*** sdake has joined #openstack-nova | 06:57 | |
*** sdake_ has quit IRC | 07:00 | |
*** kaisers has joined #openstack-nova | 07:05 | |
*** tobe has joined #openstack-nova | 07:07 | |
*** liusheng has quit IRC | 07:08 | |
*** tojuvone has joined #openstack-nova | 07:09 | |
*** liusheng has joined #openstack-nova | 07:09 | |
*** romainh has joined #openstack-nova | 07:12 | |
*** dsanders has joined #openstack-nova | 07:18 | |
*** markus_z has joined #openstack-nova | 07:19 | |
*** eglynn has quit IRC | 07:19 | |
*** veena has joined #openstack-nova | 07:21 | |
*** dulek has joined #openstack-nova | 07:21 | |
*** ildikov has joined #openstack-nova | 07:22 | |
*** matrohon has joined #openstack-nova | 07:26 | |
*** abhishekk has joined #openstack-nova | 07:26 | |
*** destiny has joined #openstack-nova | 07:27 | |
shihanzhang | hi all, who can help to review this patch: https://review.openstack.org/#/c/175742/ | 07:33 |
*** subscope has joined #openstack-nova | 07:34 | |
openstackgerrit | Joe Gordon proposed openstack/nova-specs: kilo: flavor-from-sysmeta-to-blob is implemented https://review.openstack.org/191091 | 07:35 |
*** dsanders has quit IRC | 07:35 | |
*** jlanoux has joined #openstack-nova | 07:36 | |
alex_xu | oomichi: do you think this is ok for you https://review.openstack.org/#/c/187112/10 if we think this is general ok, then I will push ironic folks review it also. then hope we can merge it more early :) | 07:36 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Merge availability_zone tests between v2 and v2.1 https://review.openstack.org/171017 | 07:37 |
garyk | shihanzhang: asking for reviews is so 80's :) | 07:38 |
garyk | now we need to beg, borrow or steal… it is like a dessert out here | 07:39 |
garyk | shihanzhang: my only concern with this is that this was added a few cycles ago as the guys from HP hit this issue in their cloud. | 07:40 |
garyk | i think it would be nice if phil day could comment on this patch | 07:40 |
gilliard | garyk: Phil's moved away from Nova a bit, but I'll add some people to the review. | 07:41 |
garyk | gilliard: thanks for the heads up. | 07:41 |
*** ndipanov has joined #openstack-nova | 07:42 | |
*** pilgrimstack has joined #openstack-nova | 07:42 | |
garyk | shihanzhang: i have added in a comment - in general i am in favor of the patch. | 07:46 |
*** salv-orlando has joined #openstack-nova | 07:48 | |
shihanzhang | garyk, I am reading | 07:48 |
*** browne has quit IRC | 07:50 | |
*** lpetrut has quit IRC | 07:54 | |
*** veena has quit IRC | 07:54 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Merge availability_zone tests between v2 and v2.1 https://review.openstack.org/171017 | 07:55 |
shihanzhang | garyk, I think the better solution is that neutron can provide a API like nova 'usage-list', but neutron doesn't has | 07:56 |
garyk | shihanzhang: as far as i know salvatore is working on proviing a more robust quota interface on the neutron side of things. | 07:58 |
garyk | so hopefully that will cover things :) | 07:58 |
oomichi | alex_xu: thanks for updating, the overview seems good for me. especially, line 37-38 is very good for me | 07:59 |
oomichi | alex_xu: I will take more time for reviewing later | 07:59 |
shihanzhang | I has read this spec https://blueprints.launchpad.net/neutron/+spec/better-quotas, it will not cover this | 08:00 |
*** ZZelle has quit IRC | 08:02 | |
*** ZZelle has joined #openstack-nova | 08:02 | |
*** lexloofer is now known as lxsli | 08:06 | |
*** sdake has quit IRC | 08:06 | |
*** bkopilov has joined #openstack-nova | 08:06 | |
*** lucasagomes has joined #openstack-nova | 08:06 | |
openstackgerrit | Madhuri Kumari proposed openstack/nova: Merge SchedulerHints functional tests of v2 and v2.1 https://review.openstack.org/167903 | 08:08 |
sahid | jogo: hello, any chance you take a look at this? https://review.openstack.org/#/c/187089/ | 08:10 |
sahid | sdague: ^ :) | 08:11 |
*** haypo has joined #openstack-nova | 08:11 | |
*** vilobhmm has quit IRC | 08:11 | |
*** haypo has quit IRC | 08:11 | |
*** jcoufal has joined #openstack-nova | 08:12 | |
*** davideagnello has joined #openstack-nova | 08:13 | |
*** jyuso1 has quit IRC | 08:17 | |
*** eglynn has joined #openstack-nova | 08:18 | |
*** davideagnello has quit IRC | 08:18 | |
*** alex_klimov has joined #openstack-nova | 08:20 | |
alex_xu | oomichi: thanks | 08:21 |
*** oro has joined #openstack-nova | 08:21 | |
*** belmoreira has quit IRC | 08:22 | |
*** yassine has joined #openstack-nova | 08:23 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Merge preserve_ephemeral_rebuild tests between v2 & v2.1 https://review.openstack.org/173284 | 08:23 |
*** otter768 has joined #openstack-nova | 08:23 | |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: hypervisor support matrix: add feature "serial console" https://review.openstack.org/180912 | 08:23 |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: hypervisor support matrix: add status "unknown" https://review.openstack.org/191697 | 08:23 |
*** danpb has joined #openstack-nova | 08:23 | |
*** Maike has joined #openstack-nova | 08:26 | |
*** otter768 has quit IRC | 08:27 | |
*** doude has joined #openstack-nova | 08:28 | |
openstackgerrit | Markus Zoeller (markus_z) proposed openstack/nova: hypervisor support matrix: add feature "serial console" https://review.openstack.org/180912 | 08:30 |
johnthetubaguy | shihanzhang: that looks like the patch is missing its blueprint, for the v2.1 work, its worth checking with alex_xu and oomichi how they are tracking those, I know I keep asking the priority groups to list their reviews in here: https://etherpad.openstack.org/p/liberty-nova-priorities-tracking | 08:30 |
lifeless | johnthetubaguy: what TZ are you in? | 08:32 |
johnthetubaguy | lifeless: UK basically | 08:33 |
*** lpetrut has joined #openstack-nova | 08:33 | |
openstackgerrit | Eduard Biceri-Matei proposed openstack/nova: libvirt: New driver for volumes backed by local file. https://review.openstack.org/134134 | 08:33 |
oomichi | alex_xu: a question about your api-wg spec for microversions | 08:34 |
alex_xu | oomichi: yea, sure | 08:35 |
oomichi | alex_xu: thanks. | 08:35 |
lifeless | johnthetubaguy: ah! less surprising then to see you up :) | 08:35 |
oomichi | alex_xu: did you aleady discuss which is min/max microversions header or body? | 08:35 |
alex_xu | oomichi: there is mail thread, let me find link | 08:36 |
johnthetubaguy | lifeless: lol, yeah, I am little later than most UK folks I guess, to give me another hour of overlap with US folks, it seems to happen if I make time for it or not | 08:36 |
oomichi | alex_xu: I found it. | 08:37 |
alex_xu | oomichi: cool...I'm too slow | 08:37 |
oomichi | alex_xu: your mail was the first | 08:37 |
oomichi | alex_xu: [api] [Nova] [Ironic] [Magnum] Microversion guideline in API-WG | 08:37 |
alex_xu | oomichi: as my understand, the agreement is both have header and body | 08:37 |
alex_xu | oomichi: but no agreemennt on the format of body yet | 08:38 |
oomichi | alex_xu: ok, thanks. will investigate the thread more | 08:38 |
alex_xu | oomichi: thanks a lot | 08:39 |
*** takedakn has joined #openstack-nova | 08:42 | |
*** takedakn has quit IRC | 08:43 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Merge config_drive functional tests between v2 and v2.1 https://review.openstack.org/171029 | 08:46 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Merge disk-config func test between v2 and v2.1 https://review.openstack.org/171453 | 08:47 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Merge server rescue tests between v2 and v2.1 https://review.openstack.org/173651 | 08:48 |
*** chaochin has joined #openstack-nova | 08:48 | |
sahid | oomichi: hello we have a fix ready to be merged i guess. can you take a look? https://review.openstack.org/#/c/187089/ | 08:48 |
bauwser | morning | 08:48 |
bauwser | oops | 08:48 |
*** bauwser is now known as bauzas | 08:48 | |
lxsli | morning | 08:48 |
bauzas | just forgot to remove my batsuit | 08:49 |
*** yingjun has joined #openstack-nova | 08:52 | |
*** subscope has quit IRC | 08:54 | |
openstackgerrit | John Garbutt proposed openstack/nova-specs: Add tooz service group driver https://review.openstack.org/138607 | 08:56 |
*** lennyb has quit IRC | 08:58 | |
openstackgerrit | Rodolfo Alonso proposed openstack/nova: Added new scheduler filter: AggregateTypeExtraSpecsAffinityFilter https://review.openstack.org/189279 | 09:03 |
*** lennyb has joined #openstack-nova | 09:03 | |
oomichi | sahid: did it. I feel current code seems wrong. | 09:04 |
oomichi | sahid: and I need to go back home right now. so I will see it again even if I was wrong. sorry about that. | 09:05 |
*** oomichi has quit IRC | 09:05 | |
openstackgerrit | Alexander Schmidt proposed openstack/nova: WIP: serial console: enable live migration https://review.openstack.org/191035 | 09:08 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova: Merge SchedulerHints functional tests of v2 and v2.1 https://review.openstack.org/167903 | 09:19 |
*** rotbeard has joined #openstack-nova | 09:22 | |
*** RuiChen has joined #openstack-nova | 09:22 | |
*** gabriel-bezerra has quit IRC | 09:23 | |
*** ericksonsantos has quit IRC | 09:23 | |
*** afaranha has quit IRC | 09:24 | |
*** tellesnobrega has quit IRC | 09:24 | |
*** samueldmq has quit IRC | 09:24 | |
openstackgerrit | wangxiyuan proposed openstack/nova: Add specific error information when createBackup is not supported https://review.openstack.org/187445 | 09:24 |
openstackgerrit | Rui Chen proposed openstack/nova: Fix resource leaking when consume_from_instance raise exception https://review.openstack.org/182165 | 09:26 |
openstackgerrit | Sridhar Gaddam proposed openstack/nova: Fix noVNC console access for an IPv6 setup https://review.openstack.org/187089 | 09:26 |
*** foexle has joined #openstack-nova | 09:34 | |
*** rushiagr_away is now known as rushiagr | 09:34 | |
*** alex_klimov has quit IRC | 09:34 | |
*** veena has joined #openstack-nova | 09:35 | |
*** haomaiwang has joined #openstack-nova | 09:38 | |
*** afazekas has joined #openstack-nova | 09:39 | |
openstackgerrit | Rui Chen proposed openstack/nova-specs: CreateVM supports subnet specified https://review.openstack.org/169638 | 09:41 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/191623 | 09:43 |
kashyap | Hmm, so Libvirt KVM (s390x) can do evacuation of instances from host, but not Libvirt KVM x86 - http://docs.openstack.org/developer/nova/support-matrix.html | 09:43 |
markus_z | kashyap: There is a ML discussion about that. I think x86 can do that too. | 09:44 |
*** yingjun has quit IRC | 09:44 | |
kashyap | markus_z: If x86 can do that, then the above matrix should be updated. | 09:44 |
markus_z | kashyap: I'm searching for the discussion, one second. | 09:45 |
openstackgerrit | Rui Chen proposed openstack/nova-specs: CreateVM supports subnet specified https://review.openstack.org/169638 | 09:46 |
markus_z | kashyap: ML: http://openstack.markmail.org/thread/nd37hdnop3zelzbk | 09:47 |
*** Murali has joined #openstack-nova | 09:47 | |
markus_z | kashyap: I was a bit unsure after I got no response. | 09:47 |
kashyap | Thanks for the pointer, reading. | 09:47 |
markus_z | kashyap: I'm going to lunch. Back in about 45 minutes if there are questions. | 09:49 |
kashyap | Yep. | 09:50 |
*** salv-orl_ has joined #openstack-nova | 09:51 | |
*** salv-orlando has quit IRC | 09:52 | |
kashyap | markus_z: When you're back - if you think for x86 the said feature should be work (as you say your manual tests are successful), you might want to submit a patch to correct that, with relevant notes. | 09:53 |
kashyap | s/should be work/should work/ | 09:54 |
sahid | danpb: hello, any chance to have your review here I have introduced new object and your comments can be really usefull https://review.openstack.org/#/c/190210/ | 09:54 |
*** oro has quit IRC | 09:55 | |
*** f13o has joined #openstack-nova | 09:56 | |
*** tojuvone has quit IRC | 09:57 | |
bauzas | ndipanov: around for discussing https://review.openstack.org/#/c/179224/6/specs/liberty/approved/check-destination-on-migrations.rst,cm ? | 10:00 |
*** zhenguo has quit IRC | 10:02 | |
*** davideagnello has joined #openstack-nova | 10:02 | |
*** Kennan2 has joined #openstack-nova | 10:03 | |
*** Kennan has quit IRC | 10:05 | |
ndipanov | bauzas, ugh doing several things but yeah shoot and I may just respond with a lag | 10:05 |
bauzas | ndipanov: np, just wanted to discuss on how to check the scheduler when live-migrate or evacuate was having a dest host | 10:06 |
bauzas | about the design | 10:07 |
*** ianbrown has joined #openstack-nova | 10:07 | |
bauzas | ndipanov: so, I was providing a separate method because I didn't wanted to provide a call to consume_from_instance() (and _schedule()) | 10:07 |
ndipanov | imho that's wrong | 10:07 |
ndipanov | as I explained on the review | 10:07 |
*** davideagnello has quit IRC | 10:07 | |
ndipanov | why don't we call consume_from_instance... ? we should | 10:08 |
*** RuiChen has quit IRC | 10:08 | |
ndipanov | if the caller plans to use the host which it does | 10:08 |
bauzas | ndipanov: because it's just for *checking* :) | 10:08 |
bauzas | ndipanov: if I'm providing select_dest() then yeah | 10:08 |
*** RuiChen has joined #openstack-nova | 10:08 | |
*** rajesht has joined #openstack-nova | 10:08 | |
openstackgerrit | Carlos Gonçalves proposed openstack/python-novaclient: Support forcing service down https://review.openstack.org/185849 | 10:08 |
*** dimsum__ has joined #openstack-nova | 10:09 | |
ndipanov | so checking is pointless | 10:09 |
ndipanov | as we discussed previously | 10:09 |
bauzas | ndipanov: meaning that this RPC method should be helping the operatoes | 10:09 |
ndipanov | in the current design | 10:09 |
ndipanov | how? | 10:09 |
bauzas | ndipanov: I'm not that sure that it should be pointless | 10:09 |
*** aix has joined #openstack-nova | 10:10 | |
bauzas | ndipanov: because for example, during the Summit, some people asked to have a way to call the scheduler to know how to have a capacity check | 10:10 |
bauzas | ndipanov: for the moment, the backlog spec is not there, but it should probably be something like 'sched, give me how many hosts I have with that flavor' | 10:10 |
ndipanov | well that is a completely different thing than what you are proposing here and trying to incorporate in a live mig workflow | 10:10 |
*** arnaud___ has joined #openstack-nova | 10:11 | |
bauzas | ndipanov: what I'm trying to explain is that while I'm OK with claiming when we call the scheduler for booting, sometimes we just want to know if something is possible at the moment | 10:11 |
bauzas | ndipanov: given that what's possible now won't be possibly possible in 5 mins | 10:12 |
ndipanov | ok then that is a different problem | 10:12 |
bauzas | later | 10:12 |
ndipanov | that makes more sense - but not in the context of live migration | 10:13 |
bauzas | ndipanov: fair point, like I said in my comment | 10:13 |
bauzas | ndipanov: but do you understand that if we're going to your direction, it would call the scheduler anyway for claiming, even if the operator is providing a destination ? | 10:14 |
openstackgerrit | Nikola Dipanov proposed openstack/nova-specs: Scheduler Introduce lightwieght transactional model for HostState https://review.openstack.org/191226 | 10:14 |
bauzas | ndipanov: again, that's fine by me, that's just a behavioural change | 10:14 |
bauzas | ndipanov: meaning that sometimes, operators just want to migrate even if they know they will violate the rules | 10:15 |
*** arnaud___ has quit IRC | 10:15 | |
bauzas | ndipanov: my point being, I'm not really against your design, just saying operators would just -1 it | 10:16 |
bauzas | ndipanov: or, we should invert the logic, call the scheduler anyway even if providing a dest host, and provide a --force flag if we want to migrate anyway | 10:17 |
*** alex_klimov has joined #openstack-nova | 10:17 | |
bauzas | ndipanov: with all the corrollars given by the 'force' thign | 10:17 |
*** marzif_ has joined #openstack-nova | 10:18 | |
bauzas | ndipanov: that way, it should be clearer : we're calling the scheduler in any cases, unless the operator explicitely wants to break the world | 10:18 |
openstackgerrit | Carlos Gonçalves proposed openstack/python-novaclient: Support forcing service down https://review.openstack.org/185849 | 10:20 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Specify current directory during processutils.execute using new cwd param https://review.openstack.org/191604 | 10:22 |
openstackgerrit | Ankit Agrawal proposed openstack/nova: Test patch race condition in delete (do not merge) https://review.openstack.org/191732 | 10:22 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Specify current directory during processutils.execute using new cwd param https://review.openstack.org/191604 | 10:23 |
*** otter768 has joined #openstack-nova | 10:24 | |
openstackgerrit | sahid proposed openstack/nova: libvirt: enhance method to return pointer_model from image prop https://review.openstack.org/174854 | 10:24 |
openstackgerrit | sahid proposed openstack/nova: libvirt: deprecate use_usb_tablet in favor of pointer_model https://review.openstack.org/176242 | 10:24 |
*** otter768 has quit IRC | 10:29 | |
bauzas | ndipanov: okay I need to go out for lunch, will update the spec with the above details | 10:31 |
bauzas | (ie. call scheduler anyway using select_dest() - and claims - and provide a 'force' option for getting rid of it) | 10:32 |
ndipanov | bauzas, adding a comment | 10:33 |
*** ianbrown has quit IRC | 10:33 | |
ndipanov | I may have missunderstood the point of this spec | 10:33 |
*** Murali_ has joined #openstack-nova | 10:34 | |
*** ankit_ag has joined #openstack-nova | 10:34 | |
*** ypraveen has joined #openstack-nova | 10:35 | |
*** mwagner_afk has quit IRC | 10:35 | |
*** Murali has quit IRC | 10:35 | |
*** Murali_ is now known as Murali | 10:36 | |
*** mnestratov has joined #openstack-nova | 10:36 | |
ndipanov | btw johnthetubaguy https://review.openstack.org/191226 as discussed | 10:37 |
ndipanov | would appreciate your comments | 10:38 |
PaulMurray | ndipanov, can we talk about your comments on https://review.openstack.org/#/c/84048/ | 10:41 |
*** marzif_ has quit IRC | 10:41 | |
*** marzif_ has joined #openstack-nova | 10:41 | |
*** lan has quit IRC | 10:41 | |
ndipanov | PaulMurray, yes :) | 10:42 |
*** lan has joined #openstack-nova | 10:42 | |
ndipanov | but let me read what other people commented since | 10:42 |
PaulMurray | ndipanov, firstly, sorry - we should have discussed this with you before now, I know you have an interest | 10:42 |
ndipanov | PaulMurray, I do but sadly don't have the resources to get involved so I can only comment | 10:43 |
*** veena has quit IRC | 10:43 | |
*** kiranr has joined #openstack-nova | 10:43 | |
*** kiran-r has quit IRC | 10:43 | |
PaulMurray | ndipanov, its ok, we don't need you to do anything, just wanted you to be on board - there is more to it than just this patch | 10:43 |
andrearosa | ndipanov: PaulMurray: thanks for reviewing the spec. Unfortunately in 15 mins I'll be off-line for a couple of hours but I'll read back the conversation when I am back | 10:44 |
PaulMurray | ndipanov, we have cinder people in HP (and some others) that want to make the cinder api do the right thing | 10:45 |
PaulMurray | ndipanov, and we want to change the nova code to fit - but that is not immediate - cinder needs to work out their side | 10:45 |
ndipanov | PaulMurray, that sounds like the right direction - yes | 10:46 |
PaulMurray | ndipanov, so we should end up with an interaction that does the kind of thing you describe and has better failure handling | 10:46 |
PaulMurray | ndipanov, our problem is that right now we get stuck, so we want to add the force detach to get back into a known state | 10:47 |
PaulMurray | we can do that now, but the long term plan is to do something better | 10:48 |
ndipanov | PaulMurray, that is definitely worht mentioning in the spec! | 10:48 |
PaulMurray | ndipanov, sorry, this was discussed in some depth with cinder people in the room at the summit, it certainly should be explained - yes | 10:49 |
PaulMurray | ndipanov, would you be on board with that plan? | 10:49 |
PaulMurray | i.e. if we explain that in the spec would you be for it? | 10:49 |
ndipanov | PaulMurray, in principle :) | 10:51 |
ndipanov | let me read the comments first and I'll ping you with questions | 10:51 |
PaulMurray | ok | 10:52 |
*** kiran-r has joined #openstack-nova | 10:55 | |
*** kiranr has quit IRC | 10:55 | |
*** gszasz has joined #openstack-nova | 10:56 | |
*** atuvenie has joined #openstack-nova | 10:57 | |
*** atuvenie has quit IRC | 10:57 | |
*** yingjun has joined #openstack-nova | 10:57 | |
*** samueldmq has joined #openstack-nova | 10:57 | |
*** tellesnobrega has joined #openstack-nova | 10:57 | |
*** ericksonsantos has joined #openstack-nova | 10:58 | |
*** afaranha has joined #openstack-nova | 10:58 | |
*** jaypipes has joined #openstack-nova | 11:00 | |
*** veena has joined #openstack-nova | 11:02 | |
*** flwang1 has joined #openstack-nova | 11:05 | |
*** Longgeek_ has joined #openstack-nova | 11:06 | |
*** edsh has quit IRC | 11:08 | |
*** Longgeek has quit IRC | 11:09 | |
*** rushiagr is now known as rushiagr_away | 11:15 | |
openstackgerrit | wangxiyuan proposed openstack/nova: Add specific error information when createBackup is not supported https://review.openstack.org/187445 | 11:16 |
*** dguitarbite has joined #openstack-nova | 11:16 | |
*** edsh has joined #openstack-nova | 11:18 | |
*** Murali_ has joined #openstack-nova | 11:19 | |
*** mpaolino has quit IRC | 11:19 | |
*** edsh has quit IRC | 11:20 | |
*** deepthi has quit IRC | 11:20 | |
*** Murali has quit IRC | 11:21 | |
*** Murali_ is now known as Murali | 11:21 | |
*** baoli has joined #openstack-nova | 11:22 | |
openstackgerrit | Jay Pipes proposed openstack/nova: Add object models for comparing resource amounts https://review.openstack.org/128992 | 11:23 |
openstackgerrit | Jay Pipes proposed openstack/nova: Adds resource type classes https://review.openstack.org/128999 | 11:23 |
*** ypraveen has quit IRC | 11:23 | |
*** marzif_ has quit IRC | 11:24 | |
*** tobe has quit IRC | 11:25 | |
*** baoli has quit IRC | 11:26 | |
*** rushiagr_away is now known as rushiagr | 11:27 | |
*** marzif_ has joined #openstack-nova | 11:27 | |
*** ypraveen has joined #openstack-nova | 11:27 | |
*** gabriel-bezerra has joined #openstack-nova | 11:27 | |
jaypipes | lxsli: thx much for your comments on the above two patches! I've pushed fixes and explanations. | 11:28 |
jaypipes | PaulMurray: mornin! | 11:28 |
*** aix has quit IRC | 11:28 | |
jaypipes | PaulMurray: wanted to chat with you about this patch: https://review.openstack.org/#/c/188525/ | 11:28 |
*** yingjun has quit IRC | 11:31 | |
*** veena has quit IRC | 11:32 | |
*** Longgeek_ has quit IRC | 11:34 | |
*** Longgeek has joined #openstack-nova | 11:35 | |
openstackgerrit | Hans Lindgren proposed openstack/nova: Remove and deprecate conductor task_log methods https://review.openstack.org/174247 | 11:36 |
openstackgerrit | Hans Lindgren proposed openstack/nova: Remove unused compute utils methods https://review.openstack.org/174246 | 11:36 |
openstackgerrit | Hans Lindgren proposed openstack/nova: Make instance usage audit use the brand new TaskLog object https://review.openstack.org/174245 | 11:36 |
openstackgerrit | Hans Lindgren proposed openstack/nova: Add a TaskLog object https://review.openstack.org/174244 | 11:36 |
*** aarefiev has quit IRC | 11:38 | |
*** aarefiev has joined #openstack-nova | 11:39 | |
*** lucasagomes is now known as lucas-hungry | 11:39 | |
*** pixelb has joined #openstack-nova | 11:40 | |
*** edsh has joined #openstack-nova | 11:41 | |
*** yingjun has joined #openstack-nova | 11:41 | |
*** edsh has quit IRC | 11:43 | |
PaulMurray | hi jaypipes, I'm here | 11:49 |
jaypipes | heyo :) | 11:50 |
PaulMurray | jaypipes, I wanted to see what you were going to suggest | 11:50 |
jaypipes | PaulMurray: fancy a hangout to get a little higher bandwidth comms? | 11:50 |
*** pradipta has quit IRC | 11:50 | |
PaulMurray | jaypipes, sure - I'll have to go to a room, so put up a link and i'll be there in a minute | 11:51 |
*** davideagnello has joined #openstack-nova | 11:51 | |
jaypipes | lxsli: can you join a hangout with PaulMurray ? | 11:51 |
alex_xu | sdague: would you like take a look this https://review.openstack.org/187112 when you wake up, if we think it is generic ok, I will push ironic to review it also, hope we can merge this early, then can begin to code the missing part in nova. | 11:52 |
openstackgerrit | Sergey Nikitin proposed openstack/nova: Added method exists to the Tag object https://review.openstack.org/178103 | 11:52 |
sdague | alex_xu: yeh, I'll add it to my review queue today | 11:52 |
jaypipes | alex_xu: hey, I've been doing my best to get through your devref patches and provide rviews. sorry for the delay on them, but I should have them all done today. | 11:53 |
*** oro has joined #openstack-nova | 11:53 | |
jaypipes | lxsli, PaulMurray: https://plus.google.com/hangouts/_/gv6qyfayerqixvsqf5bufmux2ea?hl=en | 11:53 |
*** sdake has joined #openstack-nova | 11:54 | |
alex_xu | sdague: thanks | 11:54 |
*** sdake has quit IRC | 11:55 | |
alex_xu | jaypipes: thanks :) and there is start by sdague about bump mircoversion https://review.openstack.org/191188 | 11:55 |
*** sdake has joined #openstack-nova | 11:55 | |
jaypipes | alex_xu: yup, saw that one. | 11:55 |
*** davideagnello has quit IRC | 11:56 | |
alex_xu | sdague: ironic meeting is after 5 hours, if you can give feedback before that, I will try to join ironic meeting :) | 11:56 |
*** dheeraj-gupta-4 has joined #openstack-nova | 11:57 | |
dheeraj-gupta-4 | Hi, can anyone tell me how the object hash values are computed in nova.tests.test_object E.g. - https://github.com/openstack/nova/blob/master/nova/tests/unit/objects/test_objects.py#L1075? | 11:59 |
openstackgerrit | Andrey Kurilin proposed openstack/nova-specs: Nova API Microversions support in NovaClient https://review.openstack.org/188410 | 12:01 |
sdague | alex_xu: I'll see. Do we have someone already working on the relaxation of the schema enforcement? | 12:01 |
alex_xu | sdague: no, not yet, I can help on also | 12:03 |
sdague | ok, that seems like the most important thing to get underway. | 12:03 |
alex_xu | remove extension still need a spec yet | 12:04 |
openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo https://review.openstack.org/184311 | 12:06 |
*** mpaolino has joined #openstack-nova | 12:09 | |
sdague | jaypipes / ndipanov / johnthetubaguy anyone up for +Aing a doc string patch that's 4 months old - https://review.openstack.org/#/c/116623/12 - it's in some level of being picked to death, and we should either approve it, or abandon it. No one is going to own fixing it further I don't think (I tried to just clean up a past complaint to get it landed) | 12:09 |
sdague | alex_xu: sure, but I think the version relaxation is the next most important thing | 12:09 |
alex_xu | sdague: ok | 12:11 |
*** bradjones has quit IRC | 12:13 | |
*** bradjones has joined #openstack-nova | 12:15 | |
*** bradjones has quit IRC | 12:15 | |
*** bradjones has joined #openstack-nova | 12:15 | |
*** aix has joined #openstack-nova | 12:15 | |
jaypipes | sdague: done | 12:16 |
sdague | jaypipes: thanks | 12:20 |
*** diogogmt has quit IRC | 12:20 | |
*** belmoreira has joined #openstack-nova | 12:21 | |
johnthetubaguy | sdague: curious, did we see any increase or decrease in job times with pymysql and nova (just saw your note about the neutron failures)? | 12:24 |
sdague | johnthetubaguy: I didn't notice anything significant | 12:25 |
*** otter768 has joined #openstack-nova | 12:25 | |
*** yingjun has quit IRC | 12:25 | |
*** garyk1 has joined #openstack-nova | 12:25 | |
*** garyk has quit IRC | 12:25 | |
sdague | honestly, I'd be surprised if it exposed there | 12:25 |
johnthetubaguy | sdague: yeah, I guess the DB performance isn't dominant in those tests | 12:26 |
sdague | because our time is so dominated by guest boots | 12:26 |
johnthetubaguy | yeah, makes sense | 12:26 |
sdague | the large ops jobs might show something | 12:26 |
*** VW has joined #openstack-nova | 12:26 | |
sdague | also, turbo hipster might be a good place to also try to test this | 12:26 |
johnthetubaguy | I guess we run so many nova-conductors we avoid a lot of the locking too | 12:26 |
sdague | right, which is it's own weirdness | 12:28 |
*** edmondsw has joined #openstack-nova | 12:29 | |
*** otter768 has quit IRC | 12:30 | |
*** rfolco has joined #openstack-nova | 12:31 | |
*** marzif_ has quit IRC | 12:33 | |
*** ivasev has joined #openstack-nova | 12:33 | |
*** raildo has joined #openstack-nova | 12:34 | |
*** aysyanne has joined #openstack-nova | 12:37 | |
openstackgerrit | Jay Pipes proposed openstack/nova: Add object models for comparing resource amounts https://review.openstack.org/128992 | 12:37 |
jaypipes | lxsli: fixed ^ :) | 12:38 |
*** lucas-hungry is now known as lucasagomes | 12:38 | |
*** dheeraj-gupta-4 has left #openstack-nova | 12:39 | |
*** kylek3h has joined #openstack-nova | 12:39 | |
danpb | johnthetubaguy: was there any progress made at the summit wrt killing of nova-network, or is it still just a future aspiration | 12:40 |
*** mmedvede has quit IRC | 12:41 | |
bauzas | ndipanov: so I'm back and read your comment | 12:42 |
bauzas | ndipanov: I think force_hosts is a separate problem IMHO | 12:43 |
bauzas | ndipanov: my own perspective is that it should die with fire | 12:43 |
bauzas | ndipanov: and not being done by the scheduler | 12:43 |
bauzas | ndipanov: that's why I'm saying : | 12:43 |
bauzas | ndipanov: 1/ make all the migrations calling the scheduler anyway (and will run the filters) | 12:44 |
bauzas | (that still needs to be done once ReqSpec is persisted, for making sure the resource questions are valid) | 12:44 |
bauzas | 2/ provide a way for operators for just breaking the rules, and IMHO, even for NUMA | 12:45 |
jaypipes | danpb: future aspiration. something that (potentially) closes a bit of the gap: https://review.openstack.org/#/c/184857/ | 12:45 |
bauzas | (I would even say, in particular due to NUMA) | 12:45 |
openstackgerrit | Hans Lindgren proposed openstack/nova: Remove/deprecate unused parts of the compute node object https://review.openstack.org/186757 | 12:46 |
openstackgerrit | Hans Lindgren proposed openstack/nova: Make ComputeNode.service_id nullable to match db schema https://review.openstack.org/169284 | 12:46 |
ndipanov | bauzas, well ok | 12:47 |
ndipanov | but then at the very least you need to somehow mark those instances as - BROKE_POLICY | 12:47 |
bauzas | ndipanov: I remember there is a bug related to force_hosts | 12:47 |
*** jcoufal has quit IRC | 12:47 | |
bauzas | ndipanov: instances belonging to the dest host ? | 12:48 |
openstackgerrit | Hans Lindgren proposed openstack/nova: Avoid querying for Service in resource tracker https://review.openstack.org/169288 | 12:48 |
*** VW has quit IRC | 12:48 | |
bauzas | ndipanov: I mean, that's a force flag, so a very explanatory help string should just say 'if you do that, make sure that you know what you do' | 12:48 |
bauzas | ndipanov: because it's a cloud | 12:49 |
bauzas | ndipanov: and CLI users shouldn't expect a distinct behaviour based on a specific enabled filter | 12:49 |
bauzas | ndipanov: so, since it's an operator command, he has to know what to do, because he knows the filter | 12:50 |
bauzas | ndipanov: anyway, your comments are really appreciated, because indeed my original proposal sucked | 12:51 |
*** dimsum__ has quit IRC | 12:52 | |
danpb | jaypipes: ok, so it'll live on another cycle or two then | 12:52 |
jaypipes | danpb: yes, apparently. | 12:52 |
*** f13o has quit IRC | 12:54 | |
*** dimsum__ has joined #openstack-nova | 12:54 | |
*** dimsum__ is now known as dims | 12:55 | |
*** jcoufal has joined #openstack-nova | 12:55 | |
*** exploreshaifali has joined #openstack-nova | 12:55 | |
*** dims has quit IRC | 12:55 | |
*** dims has joined #openstack-nova | 12:56 | |
*** oro has quit IRC | 12:57 | |
*** oro has joined #openstack-nova | 12:58 | |
*** nelsnelson has joined #openstack-nova | 12:58 | |
*** nelsnelson has quit IRC | 12:58 | |
*** cbader has joined #openstack-nova | 13:00 | |
lxsli | jaypipes: hiya, thanks for updates on resource object implementation. Could you take a peek at https://review.openstack.org/#/c/188759/ please? | 13:01 |
lxsli | jaypipes: ideal outcome is it gets squashed in | 13:01 |
*** jcoufal has quit IRC | 13:03 | |
*** dpaterson has joined #openstack-nova | 13:04 | |
openstackgerrit | Radoslav Gerganov proposed openstack/nova: VMware: Remove configuration check https://review.openstack.org/191787 | 13:04 |
*** nihilifer has quit IRC | 13:06 | |
*** abhishekk has quit IRC | 13:07 | |
*** Marga_ has joined #openstack-nova | 13:08 | |
*** mdrabe has joined #openstack-nova | 13:08 | |
ndipanov | bauzas, hey was otp | 13:10 |
ndipanov | so I think that "if operators want to break this they can" is not good enough | 13:11 |
*** Marga_ has quit IRC | 13:11 | |
ndipanov | there needs to be some kind of a record that this is happening | 13:11 |
bauzas | ndipanov: I see | 13:11 |
openstackgerrit | Alexandre Levine proposed openstack/nova-specs: user_data modification https://review.openstack.org/191791 | 13:11 |
bauzas | ndipanov: like FORCED_ACTIVE ? | 13:11 |
ndipanov | otherwise when there's more than 3 instances | 13:11 |
ndipanov | no one can keep track of what was going on | 13:11 |
*** Marga_ has joined #openstack-nova | 13:11 | |
ndipanov | and they shouldn't really | 13:12 |
ndipanov | that's exactly the job nova should do | 13:12 |
bauzas | ndipanov: the question is, given I have 3 instances | 13:12 |
bauzas | ndipanov: instA is on a host to be migrated | 13:12 |
ndipanov | no I mean more than 3 instances that were forced | 13:12 |
bauzas | ndipanov: instB and instC are on host2, but they are also pinned by CPU | 13:12 |
andrearosa | hi | 13:12 |
andrearosa | ]wrong window | 13:12 |
*** VW has joined #openstack-nova | 13:13 | |
ndipanov | doesn't matter which is which or what policy is not being respected | 13:13 |
*** kiran-r has quit IRC | 13:13 | |
bauzas | ndipanov: given the operator would migrate instA without calling the scheduler, it would probably pin the instA vpcu up to a pCPU already taken by either instB or instC, right ? | 13:13 |
ndipanov | just the fact that an operator would need to manually track that their users instances may not be giving them the features they asked for is something Nova should do | 13:13 |
ndipanov | right | 13:13 |
ndipanov | and that might be OK | 13:13 |
ndipanov | for a short period of time or whatever | 13:14 |
*** rushiagr is now known as rushiagr_away | 13:14 | |
ndipanov | but Nova should keep track of that and say - hey operator - this one instance is in breach of scheduling rules | 13:14 |
bauzas | ndipanov: okay, so given instA is very critical and instB and instC are just low-prio, I can understand that ops want to migrate instA quickly | 13:14 |
ndipanov | yes | 13:15 |
ndipanov | that is a valid feature to ask for | 13:15 |
bauzas | ndipanov: and potentially break instB and instC | 13:15 |
*** mmedvede has joined #openstack-nova | 13:15 | |
ndipanov | and I didn't understand that this is what you want to add | 13:15 |
*** jwcroppe has quit IRC | 13:15 | |
ndipanov | so asking where can I do it | 13:15 |
bauzas | ndipanov: so, my question is, you said we should keep track of that | 13:15 |
ndipanov | yeah | 13:15 |
bauzas | ndipanov: any idea about the best way to do such ? | 13:15 |
ndipanov | if you force it - it should be recorded so that you can query forced instances later and reschedule them | 13:15 |
ndipanov | bauzas, that's the spec propser's job :) | 13:16 |
bauzas | ndipanov: instance-actions should perhaps be something to consider nope ? | 13:16 |
bauzas | ndipanov: fair point man :) | 13:16 |
ndipanov | well instance state could work too | 13:16 |
ndipanov | which we would not show to the tenant user | 13:16 |
ndipanov | but show to the admin user | 13:16 |
ndipanov | or something like that | 13:16 |
bauzas | okay, rewriting the spec atm, will consider that in the rewriting - need to think about it further | 13:17 |
ndipanov | sure! | 13:17 |
*** yassine has quit IRC | 13:18 | |
*** PaulMurray has quit IRC | 13:18 | |
*** yassine has joined #openstack-nova | 13:19 | |
ndipanov | jaypipes, you might as well appreciate this. https://review.openstack.org/#/c/191226/ | 13:19 |
jaypipes | lxsli: done. | 13:19 |
*** jaypipes has left #openstack-nova | 13:19 | |
*** jaypipes has joined #openstack-nova | 13:19 | |
lxsli | jaypipes: great thanks, checking now | 13:19 |
*** jcoufal has joined #openstack-nova | 13:20 | |
bauzas | lxsli: jaypipes: thanks for squashing the two specs, would ease the reviews :) | 13:20 |
openstackgerrit | Ankit Agrawal proposed openstack/nova: Test patch race condition in delete (do not merge) https://review.openstack.org/191732 | 13:21 |
*** scottda has joined #openstack-nova | 13:22 | |
*** kylek3h has quit IRC | 13:22 | |
*** jwcroppe has joined #openstack-nova | 13:24 | |
*** garyk1 has quit IRC | 13:26 | |
*** garyk has joined #openstack-nova | 13:26 | |
sahid | jaypipes: hello, any news about the spec "resource templates" ? | 13:27 |
*** dboik has quit IRC | 13:27 | |
*** mriedem has joined #openstack-nova | 13:27 | |
*** VW has quit IRC | 13:28 | |
johnthetubaguy | danpb: in terms of killing nova-network, it feels further away than ever, frankly. There are more items for neutron to work on in terms of gaps | 13:29 |
*** ferest has joined #openstack-nova | 13:29 | |
*** kylek3h has joined #openstack-nova | 13:31 | |
pkholkin | hello guys, could somebody show a doc where is said that VMs in ERROR state consume resources? | 13:34 |
pkholkin | we think that it is a feature but not a bug but we don't find any docs about it | 13:34 |
lxsli | jaypipes: can you explain the difference between "class Cat", "class CatType" and "class CatSpec" to you please? | 13:34 |
*** edmondsw has quit IRC | 13:34 | |
lxsli | jaypipes: to me they are all classes, IE types | 13:35 |
mriedem | pkholkin: i don't think there is any doc on that | 13:35 |
openstackgerrit | Joe Gordon proposed openstack/nova: Remove references to suds https://review.openstack.org/191795 | 13:35 |
mriedem | instances in ERROR state will take up quota until they are deleted | 13:35 |
*** oomichi has joined #openstack-nova | 13:35 | |
pkholkin | mriedem: should it be mentioned somewhere? maybe we should file a bug to docs? | 13:35 |
oomichi | mriedem: hello | 13:36 |
mriedem | pkholkin: you could probably push a change to nova's devref to add something in the state model section | 13:36 |
openstackgerrit | Maxim Nestratov proposed openstack/nova-specs: libvirt: rename 'parallels' virt_type to 'vz' https://review.openstack.org/191797 | 13:36 |
mriedem | pkholkin: would probably go in here somewhere http://docs.openstack.org/developer/nova/devref/vmstates.html | 13:37 |
mriedem | oomichi: hi | 13:37 |
oomichi | mriedem: can you take a look at https://review.openstack.org/#/c/189541/ again ? | 13:37 |
*** PaulMurray has joined #openstack-nova | 13:38 | |
*** edmondsw has joined #openstack-nova | 13:38 | |
*** vladikr has joined #openstack-nova | 13:38 | |
*** VW has joined #openstack-nova | 13:39 | |
openstackgerrit | Alexander Schmidt proposed openstack/nova: serial console: enable live migration https://review.openstack.org/191035 | 13:40 |
mriedem | oomichi: yeah | 13:40 |
*** baoli has joined #openstack-nova | 13:40 | |
*** mwagner_afk has joined #openstack-nova | 13:40 | |
*** davideagnello has joined #openstack-nova | 13:40 | |
oomichi | mriedem: thanks | 13:40 |
pkholkin | mriedem: thanks a lot | 13:41 |
mriedem | oomichi: comments inline | 13:41 |
garyk | danpb: hi, would it be possible that you please look at https://review.openstack.org/188321 | 13:41 |
*** adrienverge has joined #openstack-nova | 13:41 | |
*** tonytan4ever has joined #openstack-nova | 13:41 | |
mriedem | pkholkin: maybe like a 'Notes' section in that page or something | 13:41 |
danpb | garyk: sure | 13:42 |
garyk | thanks | 13:42 |
*** thangp has joined #openstack-nova | 13:43 | |
openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo https://review.openstack.org/184311 | 13:43 |
*** baoli has quit IRC | 13:44 | |
*** davideagnello has quit IRC | 13:45 | |
*** snikitin has joined #openstack-nova | 13:45 | |
*** zhenguo has joined #openstack-nova | 13:45 | |
*** claudiub has joined #openstack-nova | 13:47 | |
*** dboik has joined #openstack-nova | 13:48 | |
*** oro has quit IRC | 13:49 | |
superdan | this has been sitting for a while: https://review.openstack.org/#/c/183331/ | 13:49 |
superdan | and the patch above it, which I want to get in before I start working on further robustification | 13:49 |
*** superdan is now known as dansmith | 13:49 | |
jroll | that's a great word | 13:49 |
*** ankit_ag has quit IRC | 13:50 | |
dansmith | :) | 13:50 |
*** jecarey has joined #openstack-nova | 13:51 | |
openstackgerrit | Maxim Nestratov proposed openstack/nova: libvirt: don't add filesystem disk to parallels containers unconditionally https://review.openstack.org/186858 | 13:54 |
mriedem | danpb: kashyap: http://goo.gl/JMtH5S shows that with that libvirt destroy ebusy failure, in the slim chance that we retry 3 times and it passes, it's only happening on centos7/fc21 jobs with newer libvirt/qemu | 13:55 |
*** doude has quit IRC | 13:57 | |
*** baoli has joined #openstack-nova | 13:57 | |
*** doude has joined #openstack-nova | 13:57 | |
mriedem | 1.2.2 vs 1.2.8 (centos 7.1) or 1.2.9 (fc21) | 13:57 |
kashyap | mriedem: Yeah, was just reading the comment and turned to IRC to check about that | 13:59 |
lxsli | jaypipes: aha, I see Spec is short for Specifier not Specification | 14:00 |
lxsli | jaypipes: still it adds nothing | 14:01 |
kashyap | mriedem: 1.2.2 is from Mar-2014, there have been 16 libvirt releases after that, so your comment " something working better in newer versions of qemu/libvirt here" is most likely it :-) | 14:01 |
*** ociuhandu has joined #openstack-nova | 14:06 | |
*** wverdugo500 has joined #openstack-nova | 14:07 | |
dansmith | (master)% git branch | wc -l | 14:08 |
dansmith | 165 | 14:08 |
* dansmith needs to do some cleanup | 14:08 | |
alaski | 64 for me. I need to do some coding | 14:09 |
andrearosa | lol | 14:09 |
dansmith | heh | 14:09 |
* bauzas proposes git branch -r --merged origin/stable | 14:11 | |
alaski | dansmith: I was going to wait until melwitt was around to share the good news, but you'll appreciate it as well | 14:11 |
claudiub | johnthetubaguy: hi. You had a +2 on this spec previously. A new patchset has been sent to address your previous comments. Can you take a look? :) https://review.openstack.org/#/c/175422/ | 14:11 |
alaski | since her patch merged there have been two cells job failures, both because the code caused mutliple jobs to fail | 14:11 |
dansmith | alaski: woot! | 14:11 |
dansmith | that's fantastic | 14:11 |
alaski | yeah, freaking awesome | 14:12 |
*** eharney has joined #openstack-nova | 14:12 | |
bauzas | can I just say \o/ | 14:12 |
bauzas | ? | 14:12 |
* bauzas is shy | 14:12 | |
alaski | heh | 14:12 |
* lxsli throws hands in the air like he just don't care. | 14:13 | |
mriedem | alaski: are you going to push a change to project-config to make the job voting then? | 14:13 |
mriedem | probably should | 14:13 |
mriedem | note that it also runs on glance and cinder changes i think, unless that's the devstack exercises job... | 14:13 |
alaski | I'll let melwitt have the honor | 14:14 |
alaski | but it would be good to get that up today for review | 14:14 |
sahid | ndipanov: if you want to +W https://review.openstack.org/#/c/187089/ now :) | 14:14 |
edleafe | alaski: melwitt: that's awesome! | 14:15 |
bauzas | alaski: so, I'm sorry because I had zero time to work on the job failure | 14:16 |
bauzas | alaski: but is it still accurate ? | 14:16 |
*** IanGovett has joined #openstack-nova | 14:17 | |
*** wverdugo500 has left #openstack-nova | 14:18 | |
*** arnaud___ has joined #openstack-nova | 14:18 | |
alaski | bauzas: no worries, you contributed quite a bit to get to this point as well | 14:18 |
alaski | bauzas: is what still accurate? | 14:18 |
bauzas | alaski: I had a bug to look at | 14:19 |
bauzas | alaski: http://eavesdrop.openstack.org/meetings/nova_cells/2015/nova_cells.2015-06-03-17.00.log.html https://bugs.launchpad.net/nova/+bug/1448302 | 14:20 |
openstack | Launchpad bug 1448302 in OpenStack Compute (nova) "cells: intermittent KeyError when deleting instance metadata" [Low,Confirmed] | 14:20 |
alaski | bauzas: ahh. according to logstash it's still happening. it isn't a failure though | 14:21 |
bauzas | alaski: fair point, will dig into then | 14:21 |
*** Murali has quit IRC | 14:21 | |
markus_z | kashyap: sorry, I missed your message about the evacuate feature. I'll test it again and change the matrix if the test was successful. | 14:23 |
*** openstackgerrit has quit IRC | 14:24 | |
kashyap | mriedem: No worries; thanks. | 14:24 |
*** bnemec has joined #openstack-nova | 14:24 | |
*** diogogmt has joined #openstack-nova | 14:24 | |
*** openstackgerrit has joined #openstack-nova | 14:24 | |
*** otter768 has joined #openstack-nova | 14:26 | |
kashyap | mriedem: Your unanswered question -- "Maybe we could change the timeouts based on libvirt version?" So you mean, setting VIR_ERR_OPERATION_TIMEOUT for 150 or so depending on version? | 14:26 |
mriedem | kashyap: well, i mean changing the attempt limit in the libvirt driver code based on libvirt version | 14:27 |
*** ijuwang has joined #openstack-nova | 14:27 | |
kashyap | mriedem: Ah, okay - first I guessed the same, wrongly corrected myself. | 14:27 |
*** achanda has joined #openstack-nova | 14:27 | |
kashyap | Since your tests prove that helps, your suggestion sounds good, fwiw. | 14:28 |
openstackgerrit | Nikola Dipanov proposed openstack/nova-specs: Scheduler Introduce lightwieght transactional model for HostState https://review.openstack.org/191226 | 14:29 |
*** otter768 has quit IRC | 14:30 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Remove openstack/common/versionutils module https://review.openstack.org/189118 | 14:32 |
*** unicell1 has joined #openstack-nova | 14:33 | |
*** unicell has quit IRC | 14:33 | |
*** zsais has joined #openstack-nova | 14:33 | |
*** htruta has joined #openstack-nova | 14:34 | |
*** coolsvap is now known as coolsvap|afk | 14:37 | |
*** cfriesen has joined #openstack-nova | 14:38 | |
*** kiran-r has joined #openstack-nova | 14:39 | |
*** electrocucaracha has joined #openstack-nova | 14:39 | |
*** arnaud___ has quit IRC | 14:39 | |
*** agireud has joined #openstack-nova | 14:39 | |
*** sambol has quit IRC | 14:40 | |
*** sambol has joined #openstack-nova | 14:40 | |
*** oomichi has quit IRC | 14:41 | |
*** msambol has joined #openstack-nova | 14:42 | |
*** sambol has quit IRC | 14:42 | |
*** achanda has quit IRC | 14:42 | |
*** agireud has quit IRC | 14:44 | |
*** oomichi has joined #openstack-nova | 14:45 | |
*** marcusvrn has joined #openstack-nova | 14:45 | |
andrearosa | I heard that nova has a plan for using os-brick, is that true? any ref you can share with me, please? | 14:47 |
*** marun has joined #openstack-nova | 14:47 | |
openstackgerrit | Alexis Lee proposed openstack/nova-specs: Amend resource objects proposal https://review.openstack.org/188759 | 14:49 |
mriedem | andrearosa: http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/use-os-brick-library.html | 14:49 |
*** artom has joined #openstack-nova | 14:49 | |
*** msambol has quit IRC | 14:50 | |
*** msambol has joined #openstack-nova | 14:50 | |
johnthetubaguy | alaski: awesome news on the cells testing front :D | 14:51 |
*** browne has joined #openstack-nova | 14:52 | |
openstackgerrit | Sylvain Bauza proposed openstack/nova-specs: Proposes to check destination on migrations https://review.openstack.org/179224 | 14:53 |
*** whenry has joined #openstack-nova | 14:53 | |
*** tsekiyama has joined #openstack-nova | 14:53 | |
*** achanda has joined #openstack-nova | 14:53 | |
bauzas | johnthetubaguy: ndipanov: alaski: ^^ I think you should be interested in https://review.openstack.org/179224 | 14:53 |
*** dulek has quit IRC | 14:53 | |
alaski | bauzas: cool, added to the queue | 14:54 |
*** doron has joined #openstack-nova | 14:54 | |
bauzas | alaski: (mainly because it adds a new field to the Spec) | 14:54 |
*** patrickeast has joined #openstack-nova | 14:54 | |
bauzas | sorry for bugging you folks, but we're close to Lib-1 :) | 14:54 |
*** patrickeast has quit IRC | 14:55 | |
*** msambol has quit IRC | 14:56 | |
*** patrickeast has joined #openstack-nova | 14:56 | |
andrearosa | mriedem: thanks | 14:56 |
*** achanda_ has joined #openstack-nova | 14:58 | |
*** zzzeek has joined #openstack-nova | 14:58 | |
*** doron has quit IRC | 14:59 | |
*** doron_ has joined #openstack-nova | 14:59 | |
*** BobBall has quit IRC | 14:59 | |
*** tcammann has joined #openstack-nova | 14:59 | |
*** sambol has joined #openstack-nova | 14:59 | |
*** irenab has joined #openstack-nova | 15:00 | |
*** achanda has quit IRC | 15:00 | |
*** veena has joined #openstack-nova | 15:01 | |
*** amotoki has joined #openstack-nova | 15:01 | |
*** nkrinner has quit IRC | 15:01 | |
*** adrienverge has quit IRC | 15:03 | |
*** BobBall has joined #openstack-nova | 15:03 | |
*** haomaiw__ has joined #openstack-nova | 15:03 | |
*** BobBall is now known as Guest29736 | 15:04 | |
*** haomaiwang has quit IRC | 15:04 | |
*** mdrabe has quit IRC | 15:04 | |
*** artom has quit IRC | 15:06 | |
*** agireud has joined #openstack-nova | 15:06 | |
*** artom has joined #openstack-nova | 15:07 | |
*** kiranr has joined #openstack-nova | 15:09 | |
*** kiran-r has quit IRC | 15:09 | |
*** haomaiwa_ has joined #openstack-nova | 15:11 | |
*** haomaiw__ has quit IRC | 15:13 | |
*** wwriverrat has quit IRC | 15:13 | |
*** belmoreira has quit IRC | 15:18 | |
*** tjones1 has joined #openstack-nova | 15:21 | |
*** mdrabe has joined #openstack-nova | 15:21 | |
*** oomichi has quit IRC | 15:22 | |
*** vladikr has quit IRC | 15:23 | |
*** sdake_ has joined #openstack-nova | 15:23 | |
*** browne has quit IRC | 15:23 | |
*** electrocucaracha has quit IRC | 15:23 | |
*** oomichi has joined #openstack-nova | 15:25 | |
*** oomichi has quit IRC | 15:26 | |
*** electrocucaracha has joined #openstack-nova | 15:26 | |
*** sdake has quit IRC | 15:27 | |
openstackgerrit | Andrey Pavlov proposed openstack/nova: template for disabe_terminate and shutdown_terminate flags https://review.openstack.org/189112 | 15:28 |
*** davideagnello has joined #openstack-nova | 15:29 | |
*** jcoufal has quit IRC | 15:30 | |
*** rajesht has quit IRC | 15:30 | |
*** foexle has quit IRC | 15:31 | |
*** davideagnello has quit IRC | 15:33 | |
*** dsanders has joined #openstack-nova | 15:33 | |
openstackgerrit | Andrey Pavlov proposed openstack/nova: Repairing disabe_terminate and shutdown_terminate flags https://review.openstack.org/189112 | 15:34 |
openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: virtuozzo instance resize support https://review.openstack.org/182257 | 15:36 |
*** vladikr has joined #openstack-nova | 15:37 | |
openstackgerrit | Andrey Pavlov proposed openstack/nova: Repairing disable_terminate and shutdown_terminate flags https://review.openstack.org/189112 | 15:39 |
*** isq_ has quit IRC | 15:39 | |
*** isq_ has joined #openstack-nova | 15:39 | |
*** wwriverrat has joined #openstack-nova | 15:40 | |
*** doron_ is now known as doron_afk | 15:42 | |
*** IanGovett has quit IRC | 15:42 | |
*** IanGovett has joined #openstack-nova | 15:42 | |
openstackgerrit | Andrey Pavlov proposed openstack/nova: Repairing disable_terminate and shutdown_terminate flags https://review.openstack.org/189112 | 15:44 |
*** kiranr has quit IRC | 15:46 | |
*** IanGovett has quit IRC | 15:46 | |
*** kiranr has joined #openstack-nova | 15:47 | |
*** alex_klimov has quit IRC | 15:47 | |
*** kfox1111 has joined #openstack-nova | 15:50 | |
*** kiranr is now known as kiran-r | 15:50 | |
kfox1111 | Any further thoughts on https://review.openstack.org/#/c/186617 ? | 15:50 |
*** afazekas has quit IRC | 15:50 | |
*** pbandzi has quit IRC | 15:50 | |
kfox1111 | I'd really like to get into implementing it if the general theory is ok. | 15:50 |
*** Sukhdev has joined #openstack-nova | 15:51 | |
*** kiran-r has quit IRC | 15:51 | |
*** kiran-r has joined #openstack-nova | 15:52 | |
*** patrickeast has quit IRC | 15:52 | |
*** lucasagomes is now known as lucas-brb | 15:52 | |
johnthetubaguy | kfox1111: it would be great to get some votes on that spec from some keystone and barbican folks | 15:55 |
kfox1111 | the silo'd nature of openstack's making that somewhat difficult. :/ | 15:56 |
kfox1111 | the barbican guys where the ones to encurage the instance user concept in the first place. | 15:56 |
openstackgerrit | Matthew Booth proposed openstack/nova-specs: Add the oslo_db enginefacade proposal https://review.openstack.org/150434 | 15:56 |
kfox1111 | I'll see if I can get them to do some +1's though. I think they've been reluctant to talk about implementation details for a project not their own. | 15:56 |
*** yassine has quit IRC | 15:57 | |
mnestratov | johnthetubaguy: Hi. I've made a spec for "parallels to vz renaming" blueprint as you suggested, could you please take a look at it? https://review.openstack.org/#/c/191797/ | 15:57 |
*** thedodd has joined #openstack-nova | 15:58 | |
bauzas | PaulMurray: still around ? I have a question about a local failing py27 test that you introduced | 15:59 |
PaulMurray | bauzas, yo | 15:59 |
bauzas | PaulMurray: hey | 16:00 |
bauzas | PaulMurray: so http://paste.openstack.org/show/294340/ | 16:00 |
*** kiran-r has quit IRC | 16:00 | |
bauzas | PaulMurray: but my change is not impacting this test IIUC | 16:00 |
bauzas | PaulMurray: http://paste.openstack.org/show/294341/ | 16:01 |
PaulMurray | interesting | 16:01 |
danpb | dansmith: ping - got a question about versioning of objects when you have subclassing | 16:02 |
danpb | dansmith: eg consider this set of 4 objects http://paste.fedoraproject.org/232215/83391143/ | 16:02 |
bauzas | PaulMurray: so the question is, am I blind ? | 16:02 |
*** kiran-r has joined #openstack-nova | 16:02 | |
*** kiran-r has quit IRC | 16:02 | |
*** kiranr has joined #openstack-nova | 16:02 | |
*** exploreshaifali has quit IRC | 16:02 | |
*** jaypipes has quit IRC | 16:03 | |
dansmith | danpb: I'm jumping on a call, can we chat in +60m? | 16:03 |
bauzas | PaulMurray: (note it comes from a rebase) | 16:03 |
*** jlanoux has quit IRC | 16:03 | |
danpb | dansmith: sure | 16:03 |
PaulMurray | do you mind if I look at the patch? | 16:03 |
dansmith | thanks | 16:03 |
*** kiranr is now known as kiran | 16:03 | |
*** kiran is now known as kiran-r | 16:03 | |
danpb | dansmith: my Q's are inline in that pastebin when you get time | 16:04 |
dansmith | danpb: okay, I'll queue it up | 16:04 |
*** jaypipes has joined #openstack-nova | 16:05 | |
*** kiran-r has quit IRC | 16:05 | |
*** kiran-r has joined #openstack-nova | 16:05 | |
*** kiran-r has quit IRC | 16:05 | |
*** kiran-r has joined #openstack-nova | 16:05 | |
bauzas | PaulMurray: just wondering if that's not due to https://github.com/openstack/nova/commit/07c93c897e60a6eb9487dc770522e1424794ee34 | 16:08 |
bauzas | PaulMurray: checking my local master branch | 16:08 |
lxsli | dansmith: quickly re: your qs about no-more-soft-delete - is it enough to say EG "fixed_ip_disassociate" is used by Nova network, or do you want to know all the calls? | 16:08 |
lxsli | dansmith: the latter will take quite some time | 16:09 |
*** lpetrut has quit IRC | 16:09 | |
mriedem | danpb: when dan and i were talking about this last week for https://docs.google.com/drawings/d/1geSKQXz4SqfXllq1Pk5o2YVCycZVf_i6ThY88r9YF4A/edit?usp=sharing it sounded like any time any of the subclasses change you bump the version on the parent | 16:10 |
bauzas | dansmith: PaulMurray: confirmed, the regression is on master | 16:10 |
mriedem | danpb: so essentially the version changes for all | 16:10 |
bauzas | PaulMurray: dansmith: http://paste.openstack.org/show/294340/ | 16:10 |
PaulMurray | bauzas, I can reproduce it from master | 16:10 |
bauzas | PaulMurray: yup, Houston we have a problem | 16:10 |
mriedem | bauzas: PaulMurray: is it failing in the jenkins jobs? | 16:11 |
*** sdake has joined #openstack-nova | 16:11 | |
bauzas | mriedem: checking | 16:11 |
danpb | mriedem: in my case that means you essentially end up with one single version for a huge set of objects, which felt sub-optimal | 16:11 |
mriedem | danpb: yeah | 16:12 |
*** armax has joined #openstack-nova | 16:12 | |
mriedem | bauzas: not seeing it in jenkins | 16:12 |
bauzas | mriedem: strange http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiT2JqZWN0cyB0aGF0IGRpZmZlciBpbiBhbnkgZmllbGQgc2hvdWxkIG5vdCBiZSBlcXVhbFwiIEFORCB0YWdzOlwiY29uc29sZS5odG1sXCIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6IjE0NDAwIiwiZ3JhcGhtb2RlIjoiY291bnQiLCJ0aW1lIjp7InVzZXJfaW50ZXJ2YWwiOjB9LCJzdGFtcCI6MTQzNDM4NDczNTk0NH0= | 16:12 |
*** tonytan_brb has joined #openstack-nova | 16:12 | |
*** salv-orlando has joined #openstack-nova | 16:12 | |
mriedem | bauzas: PaulMurray: are you guys working on a shared topic branch that has a race? | 16:12 |
mriedem | or some issue | 16:12 |
bauzas | mriedem: so PaulMurray and I can reproduce the bug on our local branches | 16:12 |
bauzas | mriedem: on the local master branch, updated 5 min ago | 16:13 |
*** hogepodge has quit IRC | 16:13 | |
mriedem | let me see if i can recreate | 16:13 |
PaulMurray | mriedem, I checked out master and found the test failed | 16:13 |
PaulMurray | I'm checking what the bug is | 16:14 |
*** sdake_ has quit IRC | 16:14 | |
*** garyk has quit IRC | 16:15 | |
*** kiran-r has quit IRC | 16:15 | |
bauzas | trying to bisect it | 16:15 |
*** kiran-r has joined #openstack-nova | 16:15 | |
*** tonytan_brb has quit IRC | 16:15 | |
*** tonytan4ever has quit IRC | 16:16 | |
*** salv-orl_ has quit IRC | 16:16 | |
openstackgerrit | jichenjc proposed openstack/nova: Uitlize object directly of instance and flavor https://review.openstack.org/185068 | 16:18 |
*** unicell1 has quit IRC | 16:19 | |
PaulMurray | mriedem, bauzas it looks like its in obj_to_primitive() it is always returning false for obj_attr_is_set() | 16:20 |
*** hogepodge has joined #openstack-nova | 16:20 | |
jaypipes | sdague: https://review.openstack.org/#/c/191188/ reviewed. thx! | 16:20 |
PaulMurray | mriedem, bauzas in the test that is failing | 16:21 |
bauzas | b9bae02af2168ad64d3b3d28c97c3853cee73272 | 16:21 |
*** kiran-r has quit IRC | 16:21 | |
bauzas | [d95c05a8fd2f762765408b9cf84da7c8d34a7892] Replace metaclass registry with explicit opt-in registry from oslo | 16:21 |
*** Longgeek has quit IRC | 16:21 | |
bauzas | that one ^ is the faulty one | 16:21 |
*** zz_jgrimm is now known as jgrimm | 16:22 | |
bauzas | PaulMurray: mriedem | 16:23 |
bauzas | ^ | 16:23 |
cfriesen | can we get another approver to take a look at https://review.openstack.org/#/c/190359/ ? This fixes a bug that we saw in the lab. | 16:23 |
mriedem | still recreating tox venv | 16:23 |
cfriesen | Simple fix, quick review | 16:23 |
bauzas | so, sounds like I need to go babysitting :/ | 16:24 |
bauzas | my ears are just suffering | 16:24 |
*** dsanders has quit IRC | 16:25 | |
*** mwagner_afk has quit IRC | 16:25 | |
*** matrohon has quit IRC | 16:26 | |
*** otter768 has joined #openstack-nova | 16:26 | |
sdague | jaypipes: cool, i'll take another run at that shortly | 16:27 |
*** patrickeast has joined #openstack-nova | 16:29 | |
*** ypraveen has quit IRC | 16:30 | |
*** baoli has quit IRC | 16:30 | |
*** vilobhmm has joined #openstack-nova | 16:30 | |
*** doron_afk is now known as doron_ | 16:31 | |
*** baoli has joined #openstack-nova | 16:31 | |
*** otter768 has quit IRC | 16:31 | |
*** kiran-r has joined #openstack-nova | 16:32 | |
*** romainh has quit IRC | 16:33 | |
*** dprince has joined #openstack-nova | 16:34 | |
*** nic has joined #openstack-nova | 16:36 | |
*** Maike has quit IRC | 16:39 | |
*** markus_z has quit IRC | 16:41 | |
jaypipes | PaulMurray: https://review.openstack.org/#/c/160942/ reviewed. | 16:41 |
PaulMurray | thanks jaypipes - I'll go though it | 16:41 |
jaypipes | np | 16:41 |
PaulMurray | s/though/through/ | 16:42 |
*** baoli has quit IRC | 16:42 | |
*** baoli has joined #openstack-nova | 16:45 | |
*** ferest has quit IRC | 16:47 | |
mriedem | bauzas: PaulMurray: i hit the same test failure btw | 16:48 |
*** lucas-brb is now known as lucasagomes | 16:48 | |
bauzas | mriedem: so the question is, why Jenkins doesn't see it ? | 16:48 |
mriedem | bauzas: PaulMurray: did either of you report a bug? | 16:48 |
bauzas | mriedem: not yet, babysitting feeding time | 16:49 |
bauzas | those are like sharks when they're hungry | 16:49 |
mriedem | yeah, np, i'll poke around | 16:49 |
PaulMurray | mriedem, bauzas not yet either - is this not happening in the check jobs? | 16:49 |
mriedem | PaulMurray: nope | 16:49 |
mriedem | reverting d95c05a8fd2f762765408b9cf84da7c8d34a7892 fixed it for me locally | 16:49 |
bauzas | PaulMurray: logstash is silent on it | 16:49 |
bauzas | mriedem: agreed, my bisect is showing it bad | 16:50 |
PaulMurray | mriedem, noticed that the names are not getting mangled - the change should be prefixing them with _obj_ | 16:50 |
PaulMurray | mriedem, before that change it was using _ | 16:50 |
bauzas | mriedem: that said, I'd rather prefer not to revert it but skip the faulty test | 16:50 |
*** sahid has quit IRC | 16:51 | |
*** browne has joined #openstack-nova | 16:51 | |
PaulMurray | mriedem, it must be the metaclass being applied | 16:51 |
bauzas | mriedem: since the related change is far more important than PaulMurray's test, with all my respects to you Paul :) | 16:51 |
PaulMurray | bauzas, its not the test that is failing - the underlying objects code is wrong | 16:51 |
*** unicell has joined #openstack-nova | 16:52 | |
PaulMurray | or rather - the object code is causing the test to fail | 16:52 |
bauzas | okay, that's what happens when your brain is dumped by a crying baby | 16:52 |
*** ZZelle_ has joined #openstack-nova | 16:52 | |
bauzas | dansmith: around ? see ^^^ | 16:52 |
PaulMurray | so the change you pointed out changed the metaclass - I think that is the problem | 16:53 |
bauzas | mriedem: I'm gonna create the bug, did you ? | 16:53 |
mriedem | not yet | 16:53 |
bauzas | ok doing | 16:54 |
PaulMurray | bauzas, the problem is that in that test the obj_to_primitive() is creating empty primitives - so whatever you compare they are equal (so long as same object type) | 16:54 |
PaulMurray | I can add that to the bug if you like | 16:54 |
PaulMurray | bauzas, the reason is the name managling isn't happening correctly | 16:55 |
ndipanov | edleafe, https://review.openstack.org/#/c/191226/ revisit pls | 16:55 |
PaulMurray | bauzas, I'm guessing this is something environmental to do oslo versioned objects so it only happens locally | 16:55 |
PaulMurray | bauzas, I can't tell how the metaclass is picked up (if any is?) | 16:56 |
mriedem | PaulMurray: jenkins is running with o.vo 0.4.0 and so is my local tox | 16:56 |
mriedem | the only differences i saw with my local and jenkins py27 run was versions of pip and tox, and i don't see how that would matter | 16:56 |
*** pixelb has quit IRC | 16:56 | |
*** IanGovett has joined #openstack-nova | 16:57 | |
bauzas | https://bugs.launchpad.net/nova/+bug/1465344 set to High as the CI is not (yet?) impacted | 16:57 |
openstack | Launchpad bug 1465344 in OpenStack Compute (nova) "TestObjEqualPrims.test_object_not_equal is failing" [High,Confirmed] | 16:57 |
*** mpaolino has quit IRC | 16:58 | |
bauzas | mriedem: I upated my venv right before yelling at it here | 16:58 |
*** edsh has joined #openstack-nova | 16:58 | |
*** mpaolino has joined #openstack-nova | 16:58 | |
PaulMurray | bauzas, guess what - I just updated my venv and now it works for me ! | 16:59 |
PaulMurray | mriedem, ^^ | 16:59 |
*** exploreshaifali has joined #openstack-nova | 17:00 | |
*** agireud has quit IRC | 17:00 | |
PaulMurray | mriedem, bauzas ah - interesting - no it didn't | 17:01 |
*** IanGovett has quit IRC | 17:01 | |
*** IanGovett1 has joined #openstack-nova | 17:01 | |
PaulMurray | I was on a branch I just pulled down using git review -d | 17:01 |
*** electrocucaracha has quit IRC | 17:01 | |
*** edsh has quit IRC | 17:03 | |
*** haomaiw__ has joined #openstack-nova | 17:03 | |
PaulMurray | yeah - that needed rebasing - so it does fail | 17:03 |
*** haomaiwa_ has quit IRC | 17:04 | |
*** electrocucaracha has joined #openstack-nova | 17:04 | |
*** sambol has quit IRC | 17:04 | |
*** mwagner_afk has joined #openstack-nova | 17:06 | |
*** davideagnello has joined #openstack-nova | 17:06 | |
bauzas | PaulMurray: as I said, before asking for help here, I made a ./run_tests.sh -u | 17:07 |
bauzas | PaulMurray: not a full venv recreate, but still updating my reqs | 17:07 |
bauzas | PaulMurray: that said, I can verify both o.voo versions between my venv and what Jenkins pulls | 17:07 |
PaulMurray | bauzas, yes - I tried it again to be sure and forgot I had switched branches - duh - its getting to the end of a long day | 17:08 |
bauzas | PaulMurray: so, my test fails with o.voo 0.4.0 | 17:08 |
vilobhmm | jaypipes : need your opinion on https://review.openstack.org/#/c/190322/...whenever you have time | 17:08 |
*** pixelb has joined #openstack-nova | 17:09 | |
bauzas | PaulMurray: and the same goes for Jenkins but passes | 17:10 |
dansmith | bauzas: something is still using _name ? | 17:10 |
bauzas | dansmith: you mean in the TestCase ? | 17:10 |
*** kiran-r has quit IRC | 17:11 | |
*** dane-fichter has joined #openstack-nova | 17:11 | |
dansmith | bauzas: not sure what you're saying | 17:11 |
bauzas | dansmith: heh, not sure you too | 17:11 |
bauzas | :) | 17:11 |
*** ssurana has joined #openstack-nova | 17:12 | |
dansmith | bauzas: are you saying this is happening in CI? https://bugs.launchpad.net/nova/+bug/1465344 | 17:12 |
openstack | Launchpad bug 1465344 in OpenStack Compute (nova) "TestObjEqualPrims.test_object_not_equal is failing" [High,Confirmed] | 17:12 |
bauzas | dansmith: no, that's why it's not a critical one | 17:12 |
bauzas | dansmith: I'm trying to see the diffs between my venv and Jenkins | 17:13 |
vilobhmm | melwitt : i think i have addressed most of your questions here https://review.openstack.org/#/c/190322/7/specs/liberty/approved/servicegroup-api-control-plane.rst …please check | 17:14 |
*** mspreitz has joined #openstack-nova | 17:14 | |
PaulMurray | dansmith, I added a comment to the bug report | 17:14 |
dansmith | danpb: no, I don't think we need to register the abstract base | 17:14 |
vilobhmm | melwitt : all of your questions* | 17:14 |
*** mpaolino has quit IRC | 17:14 | |
PaulMurray | dansmith, seems to me that the object fields are not getting the prefix when set, so they are not found by the getter | 17:15 |
dansmith | PaulMurray: okay, the thing responsible for that has moved | 17:15 |
*** electrocucaracha has quit IRC | 17:16 | |
openstackgerrit | Eugeniya Kudryashova proposed openstack/nova: Raise correct exception https://review.openstack.org/191909 | 17:16 |
dansmith | PaulMurray: I don't think so, let me show you a paste | 17:16 |
*** nihilifer has joined #openstack-nova | 17:17 | |
*** dsanders has joined #openstack-nova | 17:17 | |
dansmith | PaulMurray: http://pastebin.com/MET94mu8 | 17:17 |
bauzas | I can try a full venv recreate to mimic Jenkins | 17:17 |
*** rotbeard has quit IRC | 17:19 | |
*** electrocucaracha has joined #openstack-nova | 17:19 | |
*** pixelb has quit IRC | 17:23 | |
openstackgerrit | Jay Pipes proposed openstack/nova: Add object models for comparing resource amounts https://review.openstack.org/128992 | 17:23 |
jaypipes | lxsli: ^ | 17:24 |
PaulMurray | dansmith, I think I might have something - I use the test object called MyObj | 17:24 |
PaulMurray | dansmith, it looks like that is now registered in _BaseTestCase | 17:25 |
ndipanov | jaypipes, since you're the scheduler leader - would really appreciate thoughts on this https://review.openstack.org/#/c/191226/ (not sure you got the ping this AM) | 17:25 |
*** gszasz has quit IRC | 17:25 | |
*** doron_ is now known as doron_afk | 17:25 | |
jaypipes | ndipanov: yes, I did. sorry, it's in the backlog... | 17:26 |
PaulMurray | dansmith, but the test class I used doesn't extend that - it extends tests.NoDbTestCase | 17:26 |
dansmith | PaulMurray: ah, yep! | 17:26 |
PaulMurray | dansmith, would that make a difference? | 17:26 |
*** rotbeard has joined #openstack-nova | 17:26 | |
dansmith | PaulMurray: yes, so you don't get any of the special goo around property handling if you don't register | 17:26 |
*** dims has quit IRC | 17:27 | |
dansmith | PaulMurray: what you want to do is either inherit that base, or you can now call register_if(False), which won't actually register you, but will do the rest of the object initialization | 17:27 |
bauzas | PaulMurray: got it, that's the reset_changes() | 17:27 |
*** rajesht has joined #openstack-nova | 17:27 | |
dansmith | PaulMurray: I would have fixed it when I hit it during testing that change, but obviously I wasn't seeing it on my side etiher | 17:27 |
*** dims has joined #openstack-nova | 17:28 | |
bauzas | dansmith: PaulMurray: if not calling obj.reset_changes() then the assert is correct | 17:28 |
PaulMurray | dansmith, bauzas shall I knock up a quick patch then | 17:28 |
bauzas | oh nvm | 17:28 |
bauzas | I'm just confounding True and Flase | 17:29 |
dansmith | PaulMurray: cool, thanks.. good detective work | 17:29 |
bauzas | so the test should inherit from _BaseTest? | 17:29 |
dansmith | that's one option | 17:29 |
bauzas | now, the biggest question is : we know that's wrong, but why Jenkins is still happy ? | 17:30 |
dansmith | bauzas: well, if it runs on the same worker, | 17:31 |
*** rajesht has quit IRC | 17:31 | |
dansmith | then MyObj will have been initialized by the run of one of the other bases | 17:31 |
dansmith | if it's on another worker, then not | 17:31 |
mriedem | so run it locally with --concurrency=1 | 17:32 |
mriedem | that doesn't help | 17:33 |
dansmith | right, that makes sure you won't hit the problem | 17:34 |
mriedem | i hit it locally either way | 17:34 |
*** marcusvrn has quit IRC | 17:34 | |
*** ypraveen has joined #openstack-nova | 17:35 | |
bauzas | I can try to run with nosetests and see | 17:35 |
bauzas | or at least not using testr rather | 17:35 |
*** doude has quit IRC | 17:35 | |
dansmith | mriedem: well, if it runs first in the same worker, then still broken | 17:35 |
*** salv-orlando has quit IRC | 17:37 | |
bauzas | dansmith: Jenkins is running testr, right? using multiple workers, right? | 17:38 |
dansmith | bauzas: yeah | 17:38 |
bauzas | dansmith: so we should predictably say that there would be at least a very small but still there amount of failures | 17:38 |
edleafe | ndipanov: updated my comments. Thanks for the explanation | 17:38 |
bauzas | dansmith: what just worries me is that logstash is perfectly silent over the last 6 hours | 17:39 |
openstackgerrit | Paul Murray proposed openstack/nova: Fix TestObjEqualPrims test object registration https://review.openstack.org/191911 | 17:39 |
bauzas | unless my query is wrong... /cc mriedem http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiT2JqZWN0cyB0aGF0IGRpZmZlciBpbiBhbnkgZmllbGQgc2hvdWxkIG5vdCBiZSBlcXVhbFwiIEFORCB0YWdzOlwiY29uc29sZS5odG1sXCIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6IjQzMjAwIiwiZ3JhcGhtb2RlIjoiY291bnQiLCJ0aW1lIjp7InVzZXJfaW50ZXJ2YWwiOjB9LCJzdGFtcCI6MTQzNDM4OTk5MDM1NH0= | 17:40 |
*** arnaud___ has joined #openstack-nova | 17:40 | |
mriedem | bauzas: no i didn't get any hits either | 17:40 |
PaulMurray | dansmith, mriedem ^^^ https://review.openstack.org/#/c/191911/ | 17:40 |
dansmith | bauzas: I don't think it's surprising | 17:40 |
dansmith | bauzas: there are many more tests where it is registered | 17:41 |
bauzas | I see | 17:41 |
*** ijw has quit IRC | 17:41 | |
dansmith | bauzas: so it's very unlikely that in the same repeatable environment it never runs before one that has registered that object | 17:41 |
*** IanGovett1 has quit IRC | 17:41 | |
dansmith | PaulMurray: thanks | 17:41 |
bauzas | anyway, testing PaulMurray's patch | 17:42 |
*** Marga_ has quit IRC | 17:42 | |
mriedem | fixed it for me locally, approved | 17:42 |
bauzas | as well | 17:43 |
bauzas | okay, PaulMurray, thanks | 17:43 |
*** pixelb has joined #openstack-nova | 17:43 | |
bauzas | need to drop off now | 17:43 |
bauzas | seeya | 17:43 |
PaulMurray | bauzas, dansmith mriedem cheers guys - I'm off home | 17:43 |
openstackgerrit | John Garbutt proposed openstack/nova-specs: Parallel Scheduler Proposal https://review.openstack.org/191914 | 17:44 |
*** arnaud___ has quit IRC | 17:45 | |
*** promulo has joined #openstack-nova | 17:46 | |
*** Sukhdev has quit IRC | 17:51 | |
*** pixelb has quit IRC | 17:53 | |
*** zhenguo has quit IRC | 17:53 | |
openstackgerrit | melanie witt proposed openstack/python-novaclient: Add a sample clouds.yaml https://review.openstack.org/191917 | 17:53 |
*** ijw has joined #openstack-nova | 18:01 | |
*** veena has quit IRC | 18:03 | |
*** agireud has joined #openstack-nova | 18:04 | |
openstackgerrit | John Garbutt proposed openstack/nova-specs: Parallel Scheduler Proposal https://review.openstack.org/191914 | 18:07 |
*** rotbeard has quit IRC | 18:07 | |
*** promulo__ has joined #openstack-nova | 18:08 | |
*** promulo has quit IRC | 18:09 | |
*** aix has quit IRC | 18:09 | |
*** ypraveen has quit IRC | 18:09 | |
openstackgerrit | Philip Marc Schwartz proposed openstack/nova: Implement online schema migrations https://review.openstack.org/154521 | 18:09 |
*** dsanders has quit IRC | 18:12 | |
*** sambol has joined #openstack-nova | 18:12 | |
*** dsanders has joined #openstack-nova | 18:13 | |
*** Longgeek has joined #openstack-nova | 18:17 | |
*** tonytan4ever has joined #openstack-nova | 18:18 | |
*** Marga_ has joined #openstack-nova | 18:19 | |
*** Mike_D_laptop has joined #openstack-nova | 18:23 | |
*** pixelb has joined #openstack-nova | 18:26 | |
*** jmccrory has quit IRC | 18:26 | |
*** lucasagomes is now known as lucas-dinner | 18:26 | |
openstackgerrit | John Garbutt proposed openstack/nova-specs: Parallel Scheduler Proposal https://review.openstack.org/191914 | 18:27 |
*** otter768 has joined #openstack-nova | 18:28 | |
*** lpetrut has joined #openstack-nova | 18:28 | |
openstackgerrit | Nicolas Simonds proposed openstack/nova: Use utf8_bin collation on the flavor extra-specs table in MySQL https://review.openstack.org/190300 | 18:29 |
*** sdake_ has joined #openstack-nova | 18:29 | |
*** jmccrory has joined #openstack-nova | 18:31 | |
*** otter768 has quit IRC | 18:32 | |
*** exploreshaifali has quit IRC | 18:33 | |
*** sdake has quit IRC | 18:34 | |
*** Marga_ has quit IRC | 18:35 | |
*** Marga_ has joined #openstack-nova | 18:36 | |
*** electrocucaracha has quit IRC | 18:39 | |
*** achanda_ has quit IRC | 18:40 | |
*** electrocucaracha has joined #openstack-nova | 18:41 | |
*** electrocucaracha has quit IRC | 18:42 | |
*** ildikov has quit IRC | 18:47 | |
*** ypraveen has joined #openstack-nova | 18:47 | |
*** Longgeek has quit IRC | 18:52 | |
openstackgerrit | Philip Marc Schwartz proposed openstack/nova: Added documentation around datanase upgrades https://review.openstack.org/191932 | 18:53 |
*** Marga_ has quit IRC | 18:54 | |
*** Marga_ has joined #openstack-nova | 18:54 | |
*** thangp has quit IRC | 18:56 | |
*** baoli has quit IRC | 18:57 | |
*** Mike_D_laptop has quit IRC | 18:57 | |
*** eglynn has quit IRC | 18:59 | |
*** thangp has joined #openstack-nova | 18:59 | |
*** ociuhandu has quit IRC | 19:00 | |
openstackgerrit | Sean Dague proposed openstack/nova: Document when we should have a microversion https://review.openstack.org/191188 | 19:01 |
openstackgerrit | Andrew Laski proposed openstack/nova: Cells: Limit instances pulled in _heal_instances https://review.openstack.org/176925 | 19:04 |
openstackgerrit | Andrey Pavlov proposed openstack/nova: Repairing disable_terminate and shutdown_terminate flags https://review.openstack.org/189112 | 19:10 |
openstackgerrit | Philip Marc Schwartz proposed openstack/nova: Added documentation around datanase upgrades https://review.openstack.org/191932 | 19:14 |
*** exploreshaifali has joined #openstack-nova | 19:14 | |
mriedem | dansmith: are you going to be able to handle lemon chicken, pasta bar and beef stir fry for the meetup? | 19:15 |
dansmith | mriedem: hah, yeah, thanks | 19:16 |
dansmith | assume that could have been a private query, but not nearly as embarrassing for me, so... nicely played. | 19:16 |
mriedem | correct | 19:16 |
dansmith | heh | 19:17 |
mriedem | i wasn't at the kilo meetup at vmware but assume the table arrangement was like intel with pods | 19:18 |
dansmith | nope | 19:18 |
dansmith | just a big thing down the middle | 19:18 |
dansmith | like two-tables wide, but the length of the room | 19:18 |
dansmith | it worked well | 19:18 |
mriedem | oh, well, i think there is a column in the way of that working | 19:19 |
dansmith | in intel, it was pretty far to shout to someone all the way on the other side of the room | 19:19 |
dansmith | oh wonderful | 19:19 |
dansmith | I had the joy of working at an IBM building not designed by IBM | 19:19 |
dansmith | so we didn't have columns in the middles of rooms and the ceiling were higher than 5'10" | 19:19 |
*** mspreitz has quit IRC | 19:20 | |
mriedem | well, the other adjacent room was already reserved, there wouldn't have been a column that way | 19:20 |
dansmith | and more angles than just 90, 180, and 270 :) | 19:20 |
dansmith | heh | 19:20 |
mriedem | i figured table pods was at least better than classroom style so i'm not staring at the back of someone's head | 19:21 |
dansmith | yeah | 19:22 |
mriedem | we have another bigger room where that could work, but it's not wired the same way, so that would suck | 19:23 |
openstackgerrit | Artom Lifshitz proposed openstack/python-novaclient: Revert "Allow admin user to get all tenant's floating IPs" https://review.openstack.org/191941 | 19:27 |
*** mtanino has joined #openstack-nova | 19:28 | |
*** ociuhandu has joined #openstack-nova | 19:29 | |
*** Marga_ has quit IRC | 19:30 | |
*** Marga_ has joined #openstack-nova | 19:30 | |
*** amotoki has quit IRC | 19:33 | |
*** dane-fichter has quit IRC | 19:34 | |
*** dprince has quit IRC | 19:39 | |
*** ijw_ has joined #openstack-nova | 19:43 | |
*** jgrimm is now known as zz_jgrimm | 19:44 | |
*** afazekas has joined #openstack-nova | 19:45 | |
bauzas | ndipanov: still around ? | 19:45 |
bauzas | ndipanov: uh, late for you | 19:45 |
cfriesen | so currently cold migration/resize uses passwordless ssh between compute nodes to determine whether or not it's on shared storage. As far as I know this is the only user of passwordless ssh (if using libvirt tunneling which is the default). Is there any interest in switching it to use RPC like live migration? | 19:46 |
*** ijw has quit IRC | 19:46 | |
bauzas | ndipanov: just wanted to discuss about your opinion on a vm state for forced hosts | 19:47 |
*** electrocucaracha has joined #openstack-nova | 19:47 | |
bauzas | ndipanov: no worries if you can't, let's ping back on tomorrow | 19:47 |
*** alexpilotti has quit IRC | 19:48 | |
melwitt | cfriesen: I think there is interest. it's one of the feedback from operators that has come up several times https://etherpad.openstack.org/p/YVR-ops-nova and there are deployments where ssh between compute nodes is not allowed for security reasons, and such deployments are unable to use cold migrate because of it | 19:49 |
*** yamahata has joined #openstack-nova | 19:49 | |
*** afazekas has quit IRC | 19:49 | |
*** achanda has joined #openstack-nova | 19:50 | |
*** boris-42 has quit IRC | 19:52 | |
*** sdake has joined #openstack-nova | 19:54 | |
*** flwang1 has quit IRC | 19:56 | |
*** afazekas has joined #openstack-nova | 19:57 | |
*** sdake_ has quit IRC | 19:58 | |
*** electrocucaracha has quit IRC | 20:00 | |
*** sdake_ has joined #openstack-nova | 20:01 | |
*** ijw has joined #openstack-nova | 20:02 | |
*** VW has quit IRC | 20:02 | |
*** afazekas has quit IRC | 20:04 | |
*** ijw__ has joined #openstack-nova | 20:04 | |
*** ijw_ has quit IRC | 20:05 | |
*** sdake has quit IRC | 20:05 | |
*** ildikov has joined #openstack-nova | 20:06 | |
*** ijw__ has quit IRC | 20:06 | |
*** ijw_ has joined #openstack-nova | 20:07 | |
*** VW has joined #openstack-nova | 20:07 | |
*** ijw has quit IRC | 20:07 | |
*** tonytan4ever has quit IRC | 20:08 | |
*** doron_afk has quit IRC | 20:09 | |
dansmith | cfriesen: we use the ssh connection to move the image if not on shared storage, right? | 20:11 |
*** mdrabe has quit IRC | 20:11 | |
openstackgerrit | Ed Leafe proposed openstack/nova: Create RequestSpec object https://review.openstack.org/145528 | 20:12 |
*** mdrabe has joined #openstack-nova | 20:13 | |
*** electrocucaracha has joined #openstack-nova | 20:14 | |
*** zzzeek has quit IRC | 20:16 | |
*** belmoreira has joined #openstack-nova | 20:16 | |
*** tonytan4ever has joined #openstack-nova | 20:19 | |
*** lpetrut has quit IRC | 20:22 | |
*** dboik has quit IRC | 20:23 | |
*** zzzeek has joined #openstack-nova | 20:24 | |
*** ijw_ has quit IRC | 20:24 | |
*** Marga_ has quit IRC | 20:24 | |
*** Marga_ has joined #openstack-nova | 20:24 | |
*** ijw has joined #openstack-nova | 20:24 | |
*** afazekas has joined #openstack-nova | 20:26 | |
openstackgerrit | Joel Coffman proposed openstack/nova: libvirt: Disconnect dm-crypt on instance suspend/stop https://review.openstack.org/141485 | 20:28 |
*** otter768 has joined #openstack-nova | 20:28 | |
*** yamahata has quit IRC | 20:29 | |
*** afazekas has quit IRC | 20:31 | |
*** otter768 has quit IRC | 20:33 | |
*** achanda has quit IRC | 20:34 | |
*** achanda_ has joined #openstack-nova | 20:34 | |
cfriesen | melwitt: good to know about the operators. dansmith: essentially, yes....it'll use rsync or scp to copy the image over. (it uses ssh directly for checking if on shared storage and for creating/deleting the remote directory) | 20:36 |
dansmith | melwitt: right, there's nothing "secure" about what we require in the way of ssh | 20:36 |
dansmith | melwitt: there is the "passwordless ssh" problem, and also the "any compute node talks to any other compute node" problem | 20:37 |
dansmith | cfriesen: right, so removing passwordless ssh just for the shared check doesn't get you anything, right? | 20:37 |
cfriesen | dansmith: yeah, we'd have to figure out how to do the image transport | 20:38 |
*** afazekas has joined #openstack-nova | 20:38 | |
*** nihilifer has quit IRC | 20:38 | |
dansmith | definitely not doing that over rpc, so, yeah, that's an important detail :) | 20:38 |
cfriesen | dansmith: how about rsh? :) | 20:38 |
*** zzzeek has quit IRC | 20:38 | |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Remove openstack/common/versionutils module https://review.openstack.org/189118 | 20:39 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/nova: Remove openstack/common/versionutils module https://review.openstack.org/189118 | 20:40 |
*** vladikr has quit IRC | 20:40 | |
*** ivasev has quit IRC | 20:40 | |
*** matrohon has joined #openstack-nova | 20:43 | |
*** danpb has quit IRC | 20:44 | |
*** afazekas has quit IRC | 20:44 | |
*** sdake has joined #openstack-nova | 20:44 | |
*** baoli has joined #openstack-nova | 20:44 | |
*** achanda_ has quit IRC | 20:46 | |
*** msambol has joined #openstack-nova | 20:47 | |
*** baoli has quit IRC | 20:47 | |
*** agireud has quit IRC | 20:47 | |
*** baoli has joined #openstack-nova | 20:48 | |
*** sdake_ has quit IRC | 20:48 | |
cfriesen | melwitt: dansmith: do we need to be able to operation in an environment where we don't trust the path between the compute nodes? If so then it seems like we'd need some sort of key sharing no matter what solution is used. | 20:48 |
*** Longgeek has joined #openstack-nova | 20:49 | |
dansmith | cfriesen: well, we have that today, in a static form | 20:50 |
dansmith | cfriesen: but yeah, I don't think we can do host-based auth, if that's what you mean.. that's the complaint about the current approach of course | 20:50 |
*** sambol has quit IRC | 20:50 | |
cfriesen | melwitt: how do the operators that don't allow ssh handle live-migration? | 20:51 |
dansmith | cfriesen: you mean migration, right? | 20:51 |
dansmith | either way, the ones that are concerned about that don't allow it | 20:51 |
dansmith | or regular migration | 20:52 |
cfriesen | dansmith: no, live migration. (since it has encryption options too) | 20:52 |
dansmith | live migration can be done all with libvirt TLS auth and encryption, right? | 20:52 |
flwang | jaypipes: ping | 20:52 |
*** msambol has quit IRC | 20:52 | |
cfriesen | dansmith: yeah. I was wondering if that was okay since it wasn't using generic ssh | 20:52 |
dansmith | that's much more secure than passwordless ssh | 20:53 |
dansmith | that helps, | 20:53 |
jaypipes | flwang: just about to start a couple hours of meetings :( | 20:53 |
dansmith | but doesn't address the people that want to not have any communication between compute nodes | 20:53 |
jaypipes | flwang: I did not create the spec for the glance_store and move to v2 glance API work :( | 20:53 |
flwang | :( ok, I will catch up with you later | 20:53 |
flwang | jaypipes: ok, just wanna let you know, i'm rebasing the flaper87's patch | 20:54 |
flwang | and I have fixed most of the failed cases | 20:54 |
flwang | meanwhile, i'm pushing the changes-since work on glance side | 20:54 |
*** Marga_ has quit IRC | 20:55 | |
jaypipes | flwang: awesome. link me the patch and I will try to get to a review. | 20:55 |
flwang | hope we can make jenkins happy by the end of this month | 20:55 |
*** Marga_ has joined #openstack-nova | 20:55 | |
cfriesen | dansmith: If they don't want any communication between compute nodes, what do they expect? Bounce the whole migration off the controller? Also, why do we have a distinction between cold migration and live-migration-without-VIR_MIGRATE_LIVE? | 20:55 |
*** raildo has quit IRC | 20:55 | |
flwang | jaypipes: https://review.openstack.org/144875 | 20:55 |
jaypipes | cheers | 20:55 |
cfriesen | dansmith: or rather virDomainMigrate() without VIR_MIGRATE_LIVE | 20:56 |
dansmith | cfriesen: yeah, that sort of thing has been suggested (using a proxy). However, many of them don't care because it's not a thing they need | 20:56 |
flwang | jaypipes: np | 20:56 |
cfriesen | dansmith: okay, fair enough | 20:56 |
dansmith | cfriesen: well, lots of reasons, but for one: live migration is in-place and regular migration is not | 20:56 |
*** dims_ has joined #openstack-nova | 20:56 | |
dansmith | cfriesen: that was in answer to the libvirt question | 20:58 |
dansmith | cfriesen: the reasons they're not unified is "because they're not", not really for any good reason I think | 20:59 |
dansmith | other than the fact that they do different things | 20:59 |
*** sambol has joined #openstack-nova | 20:59 | |
openstackgerrit | Baodong (Robert) Li proposed openstack/nova: Handle port delete initiated by neutron https://review.openstack.org/179390 | 20:59 |
*** dims has quit IRC | 21:00 | |
*** dims_ has quit IRC | 21:00 | |
openstackgerrit | Matt Riedemann proposed openstack/nova: Add DB2 support https://review.openstack.org/69047 | 21:04 |
*** xyang1 has joined #openstack-nova | 21:05 | |
*** ijw_ has joined #openstack-nova | 21:08 | |
cfriesen | dansmith: functionally speaking they're close, virDomainMigrate() just pauses the instance instead of shutting it down. But the existing cold migration path is probably more generic across hypervisors. | 21:10 |
*** gabriel-bezerra has quit IRC | 21:10 | |
*** ericksonsantos has quit IRC | 21:11 | |
*** htruta has quit IRC | 21:11 | |
*** samueldmq has quit IRC | 21:11 | |
*** tellesnobrega has quit IRC | 21:11 | |
*** afaranha has quit IRC | 21:11 | |
*** ijw has quit IRC | 21:11 | |
dansmith | cfriesen: the current one is almost hypervisor agnostic, yeah | 21:14 |
dansmith | cfriesen: virdomainmigrate can be made to leave the instance unchanged on the source machine? | 21:15 |
dansmith | cfriesen: and make it runnable on the destination at the same time/ | 21:15 |
*** baoli has quit IRC | 21:17 | |
cfriesen | dansmith: yeah...VIR_MIGRATE_UNDEFINE_SOURCE controls whether you delete the source or not | 21:18 |
dansmith | okay | 21:18 |
* melwitt reads scrollback | 21:18 | |
dansmith | I think we undefine the domain during a normal migration, right? | 21:19 |
dansmith | I guess as long as no disks are removed, we could still allow undefine_source and use the live mechanism | 21:19 |
dansmith | anyway, it's been proposed before, and would probably be a welcome change for some, but it'd require refactoring the existing stuff to allow for that possibility | 21:19 |
cfriesen | dansmith: yes, default live migration settings are VIR_MIGRATE_UNDEFINE_SOURCE, VIR_MIGRATE_PEER2PEER, VIR_MIGRATE_LIVE, VIR_MIGRATE_TUNNELLED | 21:20 |
dansmith | no, I meant with regular migration | 21:20 |
*** penick has joined #openstack-nova | 21:21 | |
cfriesen | dansmith: I can't remember when we undefine...thought maybe we didn't undefine until migration was confirmed | 21:22 |
*** Longgeek has quit IRC | 21:22 | |
*** artom has quit IRC | 21:22 | |
*** alex_klimov has joined #openstack-nova | 21:22 | |
*** thangp has quit IRC | 21:22 | |
*** artom has joined #openstack-nova | 21:22 | |
dansmith | cfriesen: yep, we do | 21:22 |
melwitt | cfriesen: to answer your earlier question, operators that don't allow compute nodes to talk to each other don't do migrations at all, live or otherwise | 21:23 |
melwitt | cfriesen: and to do it at all would require tunneling through a proxy, I think, like dansmith mentioned | 21:24 |
cfriesen | melwitt: so they're okay with the computes talking to the controller, but not to each other? | 21:25 |
dansmith | yeah | 21:25 |
dansmith | cross-compute conversations could be leaking data from coke to pepsi | 21:25 |
dansmith | if you can live migrate into another host with no restriction, then once you break out of one hypervisor, you can go anywhere you want | 21:26 |
dansmith | a proxy would only proxy the migration if it's been set up from the outside | 21:26 |
cfriesen | dansmith: makes sense | 21:26 |
dansmith | there is a giant lie in all of this, | 21:26 |
cfriesen | for the paranoid | 21:26 |
dansmith | which is related to the fact that messages on the bus cannot be strongly authenticated at the moment | 21:27 |
dansmith | but it's a massively smaller surface than allowing any compute to jump to any other | 21:27 |
dansmith | cfriesen: s/paranoid/prepared/ | 21:27 |
dansmith | cfriesen: see venom | 21:27 |
cfriesen | dansmith: oh, agreed. paranoia often eventually turns out to be justified | 21:29 |
*** cbader has quit IRC | 21:29 | |
*** exploreshaifali has quit IRC | 21:30 | |
melwitt | I think about it as instances that are open to the internet, running applications that take requests from the public. if there's some exploit someone can do from outside, want to limit how far that can propagate once they're inside | 21:30 |
penick | that’s another concern we have. If a person owns a VM and its hypervisor, they could send a bogus migration call up to the queue, and use that migration to hitch a ride to other uncompromised hypervisors, and own them too. | 21:31 |
penick | yep | 21:31 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Add to_dict() to ImageMeta and ImageMetaProps objects https://review.openstack.org/186474 | 21:31 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Add RequestSpec methods for primitiving into dicts https://review.openstack.org/187197 | 21:31 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Use RequestSpec object in HostManager https://review.openstack.org/191251 | 21:31 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Use the same pci_requests field for all filters and HostManager https://review.openstack.org/191250 | 21:31 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Remove unnecessary method in FilterScheduler https://review.openstack.org/188485 | 21:31 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Consider that all scheduler calls are IO Ops https://review.openstack.org/188486 | 21:31 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Create RequestSpec object https://review.openstack.org/145528 | 21:31 |
openstackgerrit | Sylvain Bauza proposed openstack/nova: Use RequestSpec object in the FilterScheduler https://review.openstack.org/188487 | 21:31 |
dansmith | penick: yep, well-understood as the need for trusted messaging | 21:31 |
*** zzzeek has joined #openstack-nova | 21:32 | |
penick | I suppose the API server could construct the proxy prior to dumping the migration notification on the queue. I’m guessing that’s not ok, since it would step outside of the normal openstack design pattern? | 21:33 |
*** baoli has joined #openstack-nova | 21:35 | |
dansmith | it doesn't fix anything really | 21:35 |
dansmith | without trusted messaging, we don't really know who is running what and from where (for certain) | 21:35 |
*** dims has joined #openstack-nova | 21:35 | |
dansmith | if we had it, we'd say that only conductor can create migration objects and without it, nothing will start the proxy | 21:36 |
dansmith | but we can't enforce the first part right now | 21:36 |
*** promulo__ has quit IRC | 21:37 | |
penick | makes sense | 21:38 |
*** mdrabe has quit IRC | 21:38 | |
*** afaranha has joined #openstack-nova | 21:39 | |
*** tellesnobrega has joined #openstack-nova | 21:39 | |
*** htruta has joined #openstack-nova | 21:39 | |
*** samueldmq has joined #openstack-nova | 21:39 | |
*** ericksonsantos has joined #openstack-nova | 21:39 | |
*** gabriel-bezerra has joined #openstack-nova | 21:40 | |
*** belmoreira has quit IRC | 21:42 | |
*** ianbrown has joined #openstack-nova | 21:48 | |
*** mdrabe has joined #openstack-nova | 21:52 | |
*** edmondsw has quit IRC | 21:56 | |
*** marun has quit IRC | 21:58 | |
*** oro has joined #openstack-nova | 21:59 | |
*** eharney has quit IRC | 22:01 | |
*** burt has quit IRC | 22:02 | |
*** boris-42 has joined #openstack-nova | 22:02 | |
*** matrohon has quit IRC | 22:02 | |
*** yamahata has joined #openstack-nova | 22:03 | |
*** claudiub has quit IRC | 22:04 | |
*** oro has quit IRC | 22:08 | |
*** mriedem is now known as mriedem_away | 22:09 | |
*** aysyanne has quit IRC | 22:12 | |
*** exploreshaifali has joined #openstack-nova | 22:13 | |
*** flwang has quit IRC | 22:16 | |
*** Marga_ has quit IRC | 22:18 | |
*** Marga_ has joined #openstack-nova | 22:19 | |
*** lucas-dinner has quit IRC | 22:20 | |
openstackgerrit | Ed Leafe proposed openstack/nova-specs: Improve reporting when Scheduler returns NoValidHost https://review.openstack.org/187739 | 22:20 |
*** zsais has quit IRC | 22:22 | |
*** alex_klimov has quit IRC | 22:24 | |
*** mingy has joined #openstack-nova | 22:28 | |
*** flwang has joined #openstack-nova | 22:29 | |
*** otter768 has joined #openstack-nova | 22:29 | |
*** david-lyle has quit IRC | 22:31 | |
*** yamahata has quit IRC | 22:33 | |
*** otter768 has quit IRC | 22:34 | |
*** dims has quit IRC | 22:34 | |
*** exploreshaifali has quit IRC | 22:35 | |
*** ypraveen has quit IRC | 22:36 | |
*** jecarey has quit IRC | 22:37 | |
*** thedodd has quit IRC | 22:38 | |
openstackgerrit | Dan Smith proposed openstack/nova: Fix up some bits of resource_tracker to use instance flavors https://review.openstack.org/191994 | 22:40 |
openstackgerrit | Kevin Fox proposed openstack/nova-specs: Instance Users for Cloud Interaction https://review.openstack.org/186617 | 22:40 |
*** kfox1111 has quit IRC | 22:40 | |
*** claudiub has joined #openstack-nova | 22:51 | |
*** hogepodge has quit IRC | 22:52 | |
*** ZZelle_ has quit IRC | 22:53 | |
*** diogogmt has quit IRC | 22:53 | |
*** zzzeek has quit IRC | 22:56 | |
openstackgerrit | Kevin Fox proposed openstack/nova-specs: Instance Users for Cloud Interaction https://review.openstack.org/186617 | 23:00 |
*** alexpilotti has joined #openstack-nova | 23:01 | |
*** electrocucaracha has quit IRC | 23:01 | |
*** dims has joined #openstack-nova | 23:03 | |
*** baoli has quit IRC | 23:03 | |
*** alexpilotti has quit IRC | 23:03 | |
*** dims has quit IRC | 23:03 | |
*** dims has joined #openstack-nova | 23:04 | |
*** penick has quit IRC | 23:04 | |
*** Marga_ has quit IRC | 23:05 | |
*** Marga_ has joined #openstack-nova | 23:06 | |
*** VW has quit IRC | 23:06 | |
*** tonytan4ever has quit IRC | 23:07 | |
*** zzzeek has joined #openstack-nova | 23:08 | |
*** dboik has joined #openstack-nova | 23:08 | |
*** david-lyle has joined #openstack-nova | 23:11 | |
*** ypraveen has joined #openstack-nova | 23:13 | |
*** penick has joined #openstack-nova | 23:17 | |
*** Longgeek has joined #openstack-nova | 23:19 | |
*** hogepodge has joined #openstack-nova | 23:19 | |
*** ypraveen has quit IRC | 23:20 | |
*** EmilienM|afk is now known as EmilienM | 23:24 | |
*** zzzeek has quit IRC | 23:25 | |
*** baoli has joined #openstack-nova | 23:28 | |
*** ijw_ has quit IRC | 23:28 | |
*** ijw has joined #openstack-nova | 23:29 | |
*** diogogmt has joined #openstack-nova | 23:30 | |
openstackgerrit | Tomoki Sekiyama proposed openstack/nova: libvirt: Enhance iSCSI volume multipath support https://review.openstack.org/134683 | 23:32 |
*** penick has quit IRC | 23:34 | |
*** tjones1 has quit IRC | 23:34 | |
*** Longgeek has quit IRC | 23:35 | |
*** artom has quit IRC | 23:39 | |
*** penick has joined #openstack-nova | 23:42 | |
*** ijw has quit IRC | 23:45 | |
*** electrocucaracha has joined #openstack-nova | 23:48 | |
*** Marga_ has quit IRC | 23:49 | |
*** Marga_ has joined #openstack-nova | 23:49 | |
*** baoli has quit IRC | 23:50 | |
*** dboik_ has joined #openstack-nova | 23:51 | |
*** otter768 has joined #openstack-nova | 23:53 | |
*** dboik has quit IRC | 23:54 | |
*** ssurana has quit IRC | 23:55 | |
*** penick has quit IRC | 23:56 | |
*** asettle has joined #openstack-nova | 23:57 | |
*** electrocucaracha has quit IRC | 23:57 | |
*** electrocucaracha has joined #openstack-nova | 23:57 | |
*** Sukhdev has joined #openstack-nova | 23:58 | |
*** zzzeek has joined #openstack-nova | 23:59 | |
*** mdrabe has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!