*** _ducttape_ has quit IRC | 00:00 | |
*** thorst has joined #openstack-nova | 00:02 | |
*** rfolco has joined #openstack-nova | 00:03 | |
*** rfolco has quit IRC | 00:03 | |
*** ducttape_ has joined #openstack-nova | 00:04 | |
*** thorst has quit IRC | 00:07 | |
*** zul_ has joined #openstack-nova | 00:07 | |
*** dimtruck is now known as zz_dimtruck | 00:08 | |
*** artom has quit IRC | 00:08 | |
*** artom has joined #openstack-nova | 00:09 | |
*** artom has quit IRC | 00:10 | |
*** artom has joined #openstack-nova | 00:10 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add empty flavor object info in server api-ref https://review.openstack.org/452078 | 00:11 |
---|---|---|
gmann | oomichi: fixed ^^ | 00:11 |
EmilienM | mriedem: ok so Sem-Ver: api-break is good IIUC | 00:13 |
*** zz_dimtruck is now known as dimtruck | 00:14 | |
oomichi | gmann: thanks, +2 | 00:15 |
gmann | oomichi: thanks | 00:15 |
*** dikonoor has joined #openstack-nova | 00:15 | |
*** xyang1 has quit IRC | 00:19 | |
mriedem | EmilienM: yeah i'm fine with that | 00:20 |
mriedem | if my understanding was correct (my comment in the patch) | 00:20 |
EmilienM | mriedem: cool, thanks! | 00:21 |
EmilienM | I need to figure if I can send the patch to all projects | 00:21 |
EmilienM | I already did nova & ironic so far | 00:21 |
*** yingjun has joined #openstack-nova | 00:21 | |
*** ducttape_ has quit IRC | 00:22 | |
*** yingjun has quit IRC | 00:22 | |
*** owalsh_ has joined #openstack-nova | 00:28 | |
*** owalsh has quit IRC | 00:30 | |
*** yingjun has joined #openstack-nova | 00:34 | |
*** thorst has joined #openstack-nova | 00:34 | |
*** ducttape_ has joined #openstack-nova | 00:34 | |
*** lucasxu has joined #openstack-nova | 00:36 | |
*** thorst has quit IRC | 00:38 | |
*** lucasxu has quit IRC | 00:40 | |
*** Sihan has joined #openstack-nova | 00:40 | |
Sihan | Hi, I want to continue working on the https://review.openstack.org/#/c/348317/, could someone help to restore the patch? | 00:41 |
*** dtp has quit IRC | 00:42 | |
*** hongbin has quit IRC | 00:43 | |
*** wangqwsh has joined #openstack-nova | 00:44 | |
*** zhurong has joined #openstack-nova | 00:46 | |
*** zul_ has quit IRC | 00:46 | |
*** tovin07_ has joined #openstack-nova | 00:49 | |
*** liusheng has quit IRC | 00:53 | |
*** hoonetorg has quit IRC | 01:02 | |
*** nic has quit IRC | 01:02 | |
openstackgerrit | Li Yingjun proposed openstack/nova-specs master: Spec for service create and destroy notification https://review.openstack.org/444731 | 01:06 |
*** vladikr has joined #openstack-nova | 01:06 | |
*** Sihan has quit IRC | 01:06 | |
openstackgerrit | Li Yingjun proposed openstack/nova-specs master: Spec for service create and destroy notification https://review.openstack.org/444731 | 01:07 |
*** liusheng has joined #openstack-nova | 01:08 | |
*** CristinaPauna has quit IRC | 01:08 | |
*** iceyao has joined #openstack-nova | 01:10 | |
*** crushil has quit IRC | 01:10 | |
*** kevinz has joined #openstack-nova | 01:11 | |
*** NikhilS has joined #openstack-nova | 01:12 | |
*** takashin has joined #openstack-nova | 01:14 | |
*** dimtruck is now known as zz_dimtruck | 01:19 | |
*** dikonoor has quit IRC | 01:21 | |
*** READ10 has joined #openstack-nova | 01:22 | |
*** suresh12 has quit IRC | 01:22 | |
*** xinliang has joined #openstack-nova | 01:23 | |
*** xinliang has quit IRC | 01:23 | |
*** xinliang has joined #openstack-nova | 01:23 | |
*** Apoorva_ has joined #openstack-nova | 01:23 | |
*** kaisers has joined #openstack-nova | 01:26 | |
*** Apoorva has quit IRC | 01:27 | |
*** Apoorva_ has quit IRC | 01:27 | |
openstackgerrit | Li Yingjun proposed openstack/nova-specs master: Re-propose compute-node-notification for Pike https://review.openstack.org/345714 | 01:29 |
*** kaisers_ has quit IRC | 01:29 | |
*** amotoki has joined #openstack-nova | 01:31 | |
openstackgerrit | Li Yingjun proposed openstack/nova-specs master: Spec for service create and destroy notification https://review.openstack.org/444731 | 01:33 |
*** Daisy has joined #openstack-nova | 01:35 | |
*** ducttape_ has quit IRC | 01:37 | |
*** lyan has joined #openstack-nova | 01:38 | |
*** thorst has joined #openstack-nova | 01:38 | |
*** tjones has quit IRC | 01:40 | |
*** edmondsw has joined #openstack-nova | 01:41 | |
*** NikhilS has quit IRC | 01:42 | |
*** NikhilS has joined #openstack-nova | 01:42 | |
*** cNilesh has joined #openstack-nova | 01:42 | |
*** thorst has quit IRC | 01:43 | |
*** baoli has joined #openstack-nova | 01:44 | |
*** baoli has quit IRC | 01:45 | |
*** edmondsw has quit IRC | 01:46 | |
*** baoli has joined #openstack-nova | 01:46 | |
*** ducttape_ has joined #openstack-nova | 01:46 | |
*** zz_dimtruck is now known as dimtruck | 01:51 | |
*** zhurong has quit IRC | 01:52 | |
*** jaypipes has quit IRC | 01:52 | |
*** yulijie has quit IRC | 01:56 | |
*** suresh12 has joined #openstack-nova | 01:57 | |
*** masber has quit IRC | 01:57 | |
*** zhurong has joined #openstack-nova | 01:57 | |
*** suresh12 has quit IRC | 02:01 | |
*** gcb has joined #openstack-nova | 02:01 | |
*** ducttape_ has quit IRC | 02:07 | |
*** jamielennox is now known as jamielennox|away | 02:07 | |
*** mtanino has joined #openstack-nova | 02:07 | |
*** masayukig has joined #openstack-nova | 02:08 | |
*** tanee_away is now known as tanee | 02:11 | |
*** Sukhdev has quit IRC | 02:14 | |
*** ducttape_ has joined #openstack-nova | 02:14 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: make it clear that os-cells is for cells v1 https://review.openstack.org/453433 | 02:15 |
*** masayukig has quit IRC | 02:18 | |
*** sree has joined #openstack-nova | 02:18 | |
*** sree has quit IRC | 02:19 | |
*** yonglihe has joined #openstack-nova | 02:19 | |
*** masayukig has joined #openstack-nova | 02:19 | |
*** jamielennox|away is now known as jamielennox | 02:21 | |
*** gongysh has joined #openstack-nova | 02:27 | |
*** adisky_ has quit IRC | 02:29 | |
*** zhurong has quit IRC | 02:38 | |
*** thorst has joined #openstack-nova | 02:40 | |
gmann | mriedem: do you know why i cannot mark this as "Won't Fix" ? - https://bugs.launchpad.net/tempest/+bug/1504641 | 02:43 |
openstack | Launchpad bug 1504641 in OpenStack Compute (nova) "Listing volumes respects osapi_max_limit but does not provide a link to the next element" [Wishlist,New] | 02:43 |
openstackgerrit | Huan Xie proposed openstack/nova master: WIP: Fix race condition when neutron is using minimized poll https://review.openstack.org/444230 | 02:44 |
*** gouthamr has quit IRC | 02:49 | |
*** CristinaPauna has joined #openstack-nova | 02:50 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add empty flavor object info in server api-ref https://review.openstack.org/452078 | 02:52 |
*** Shunli has joined #openstack-nova | 02:56 | |
*** thorst has quit IRC | 02:58 | |
openstackgerrit | Huan Xie proposed openstack/nova master: XenAPI: Create linux bridge in dest host during live migration https://review.openstack.org/451657 | 03:02 |
*** efried has quit IRC | 03:04 | |
*** lyan has quit IRC | 03:04 | |
openstackgerrit | Huan Xie proposed openstack/nova master: XenAPI: Don't use nicira-iface-id for XenServer VIF https://review.openstack.org/451641 | 03:07 |
*** hongbin has joined #openstack-nova | 03:10 | |
*** Kevin_Zheng has quit IRC | 03:13 | |
*** armax has quit IRC | 03:13 | |
*** zhurong has joined #openstack-nova | 03:13 | |
*** efried has joined #openstack-nova | 03:14 | |
*** links has joined #openstack-nova | 03:16 | |
*** nicolasbock has quit IRC | 03:20 | |
*** sridharg has joined #openstack-nova | 03:26 | |
*** vladikr has quit IRC | 03:27 | |
*** bmace has quit IRC | 03:27 | |
*** edmondsw has joined #openstack-nova | 03:30 | |
*** Daisy has quit IRC | 03:31 | |
*** edmondsw has quit IRC | 03:34 | |
*** baoli has quit IRC | 03:35 | |
*** baoli has joined #openstack-nova | 03:35 | |
*** Sukhdev has joined #openstack-nova | 03:38 | |
*** ducttape_ has quit IRC | 03:45 | |
*** ljjjustin has joined #openstack-nova | 03:46 | |
*** gongysh has quit IRC | 03:46 | |
*** ducttape_ has joined #openstack-nova | 03:47 | |
*** trinaths has joined #openstack-nova | 03:53 | |
*** masber has joined #openstack-nova | 03:56 | |
*** phuongnh has joined #openstack-nova | 03:57 | |
*** baoli has quit IRC | 03:58 | |
*** iceyao has quit IRC | 04:04 | |
*** mtanino has quit IRC | 04:05 | |
*** dave-mccowan has quit IRC | 04:05 | |
*** rcernin has joined #openstack-nova | 04:06 | |
*** hongbin has quit IRC | 04:08 | |
*** vks1 has joined #openstack-nova | 04:09 | |
*** amotoki has quit IRC | 04:12 | |
*** sree has joined #openstack-nova | 04:12 | |
*** zhurong has quit IRC | 04:12 | |
*** ducttape_ has quit IRC | 04:12 | |
*** rcernin has quit IRC | 04:13 | |
*** NostawRm has quit IRC | 04:13 | |
*** dharinic has quit IRC | 04:14 | |
*** sree has quit IRC | 04:14 | |
*** sree has joined #openstack-nova | 04:14 | |
*** NostawRm has joined #openstack-nova | 04:15 | |
*** Apoorva has joined #openstack-nova | 04:16 | |
openstackgerrit | Zhang Hua proposed openstack/nova master: Try our best to bypass the dirty BDM to make nova more robust https://review.openstack.org/453451 | 04:17 |
*** marst has joined #openstack-nova | 04:18 | |
*** iceyao has joined #openstack-nova | 04:24 | |
*** dikonoor has joined #openstack-nova | 04:27 | |
*** iceyao has quit IRC | 04:28 | |
*** amotoki has joined #openstack-nova | 04:30 | |
*** dillaman has quit IRC | 04:33 | |
*** zhurong has joined #openstack-nova | 04:35 | |
*** dharinic has joined #openstack-nova | 04:38 | |
*** adisky_ has joined #openstack-nova | 04:39 | |
*** psachin has joined #openstack-nova | 04:39 | |
*** dharinic has quit IRC | 04:41 | |
*** dharinic has joined #openstack-nova | 04:42 | |
openstackgerrit | LIU Yulong proposed openstack/nova-specs master: Reset the instance keypair while rebuilding (spec) https://review.openstack.org/375221 | 04:42 |
*** MarkMielke has joined #openstack-nova | 04:43 | |
MarkMielke | Any Block Device Mapping experts online? | 04:43 |
*** Apoorva has quit IRC | 04:46 | |
*** Apoorva has joined #openstack-nova | 04:47 | |
MarkMielke | I'm having a bit of trouble understanding how to get the root_gb for bdm with destination_type = 'local', boot_index = 0. It doesn't seem to be recorded any more. volume_size is None. | 04:48 |
*** amotoki has quit IRC | 04:48 | |
*** dharinic has quit IRC | 04:49 | |
MarkMielke | I thought no worries - the flavor should work, right? Well, we have instances with local root, even though the flavor specifies root_gb = 0. Probably, it was changed at some point. | 04:49 |
*** Apoorva has quit IRC | 04:51 | |
*** iceyao has joined #openstack-nova | 04:51 | |
*** Shunli has quit IRC | 04:52 | |
*** amotoki has joined #openstack-nova | 04:53 | |
*** thorst has joined #openstack-nova | 04:55 | |
*** thorst has quit IRC | 05:00 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove the usage of extension black/white list opt in user data https://review.openstack.org/450834 | 05:06 |
*** nkrinner_afk is now known as nkrinner | 05:07 | |
*** udesale has joined #openstack-nova | 05:08 | |
*** nmathew has joined #openstack-nova | 05:09 | |
*** jaosorior_away is now known as jaosorior | 05:11 | |
*** ducttape_ has joined #openstack-nova | 05:13 | |
*** kaisers has quit IRC | 05:15 | |
*** ducttape_ has quit IRC | 05:18 | |
*** amotoki has quit IRC | 05:19 | |
*** suresh12 has joined #openstack-nova | 05:25 | |
*** tonyb_ is now known as tonyb | 05:25 | |
*** ratailor has joined #openstack-nova | 05:25 | |
*** gongysh has joined #openstack-nova | 05:27 | |
*** kaisers has joined #openstack-nova | 05:28 | |
*** Shunli has joined #openstack-nova | 05:29 | |
*** Jack_Iv has joined #openstack-nova | 05:29 | |
*** prateek has joined #openstack-nova | 05:32 | |
*** amotoki has joined #openstack-nova | 05:32 | |
*** hoonetorg has joined #openstack-nova | 05:33 | |
*** bkopilov has quit IRC | 05:35 | |
*** bkopilov has joined #openstack-nova | 05:35 | |
*** sridharg has quit IRC | 05:40 | |
*** kaisers has quit IRC | 05:42 | |
*** kaisers has joined #openstack-nova | 05:46 | |
*** Shunli has quit IRC | 05:53 | |
*** Shunli has joined #openstack-nova | 05:54 | |
*** thorst has joined #openstack-nova | 05:56 | |
*** mdnadeem has joined #openstack-nova | 05:58 | |
*** Sukhdev has quit IRC | 05:59 | |
*** thorst has quit IRC | 06:01 | |
*** Jack_Iv_ has joined #openstack-nova | 06:02 | |
*** diga has joined #openstack-nova | 06:03 | |
*** Jack_Iv_ has quit IRC | 06:05 | |
*** Jack_Iv has quit IRC | 06:05 | |
*** yingjun has quit IRC | 06:06 | |
*** Jack_Iv has joined #openstack-nova | 06:07 | |
*** palimarium has joined #openstack-nova | 06:07 | |
*** pcaruana has joined #openstack-nova | 06:08 | |
*** andreas_s has joined #openstack-nova | 06:11 | |
*** Oku_OS-away is now known as Oku_OS | 06:13 | |
*** kaisers has quit IRC | 06:14 | |
*** Jack_Iv_ has joined #openstack-nova | 06:16 | |
openstackgerrit | Jianghua Wang proposed openstack/nova-specs master: Support VGPU https://review.openstack.org/450122 | 06:17 |
*** Jack_Iv has quit IRC | 06:19 | |
*** mnestratov has joined #openstack-nova | 06:20 | |
*** Jack_Iv has joined #openstack-nova | 06:20 | |
*** Jack_Iv_ has quit IRC | 06:21 | |
openstackgerrit | Huan Xie proposed openstack/nova master: WIP: Fix race condition when neutron is using minimized poll https://review.openstack.org/444230 | 06:24 |
*** zyun has joined #openstack-nova | 06:24 | |
*** zyun has quit IRC | 06:25 | |
*** zyun has joined #openstack-nova | 06:26 | |
openstackgerrit | Shunli Zhou proposed openstack/nova master: Remove mox from cells/test_cells_rpc_driver.py https://review.openstack.org/447412 | 06:28 |
*** hoonetorg has quit IRC | 06:29 | |
*** rcernin has joined #openstack-nova | 06:31 | |
*** dimtruck is now known as zz_dimtruck | 06:32 | |
*** MarginHu has joined #openstack-nova | 06:35 | |
*** abalutoiu has quit IRC | 06:35 | |
MarginHu | hi, nova can't get right hypervisor info which hypervisor is a ironic node. | 06:39 |
*** voelzmo has joined #openstack-nova | 06:40 | |
MarginHu | how to clean the cache info ? | 06:42 |
*** hoonetorg has joined #openstack-nova | 06:42 | |
*** ducttape_ has joined #openstack-nova | 06:44 | |
*** voelzmo has quit IRC | 06:45 | |
*** voelzmo has joined #openstack-nova | 06:45 | |
*** ducttape_ has quit IRC | 06:48 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237 | 06:49 |
*** tesseract has joined #openstack-nova | 06:52 | |
*** voelzmo has quit IRC | 06:55 | |
*** voelzmo has joined #openstack-nova | 06:56 | |
*** thorst has joined #openstack-nova | 06:57 | |
*** ltomasbo|away is now known as ltomasbo | 06:58 | |
*** arne_r has joined #openstack-nova | 07:01 | |
*** raorn_ has quit IRC | 07:01 | |
*** thorst has quit IRC | 07:01 | |
*** zsli_ has joined #openstack-nova | 07:02 | |
*** suresh12 has quit IRC | 07:03 | |
*** zsli_ has quit IRC | 07:03 | |
*** zsli_ has joined #openstack-nova | 07:03 | |
*** suresh12 has joined #openstack-nova | 07:03 | |
*** zsli_ has quit IRC | 07:04 | |
*** zsli_ has joined #openstack-nova | 07:04 | |
*** zsli_ has quit IRC | 07:05 | |
openstackgerrit | Huan Xie proposed openstack/nova master: WIP: use os-xenapi V2 https://review.openstack.org/453493 | 07:06 |
*** suresh12 has quit IRC | 07:08 | |
*** kaisers has joined #openstack-nova | 07:09 | |
*** Drankis has joined #openstack-nova | 07:10 | |
*** ralonsoh has joined #openstack-nova | 07:12 | |
openstackgerrit | Béla Vancsics proposed openstack/nova master: Using max api version in notification sample test https://review.openstack.org/443686 | 07:13 |
openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225 | 07:13 |
*** sridharg has joined #openstack-nova | 07:15 | |
*** voelzmo has quit IRC | 07:16 | |
*** karimb has joined #openstack-nova | 07:16 | |
*** avolkov has joined #openstack-nova | 07:17 | |
*** raorn has joined #openstack-nova | 07:19 | |
*** mnestratov has quit IRC | 07:22 | |
*** satyar has joined #openstack-nova | 07:23 | |
*** zz_dimtruck is now known as dimtruck | 07:24 | |
*** voelzmo has joined #openstack-nova | 07:26 | |
*** Shunli has quit IRC | 07:28 | |
*** Shunli has joined #openstack-nova | 07:28 | |
*** Shunli has quit IRC | 07:28 | |
*** Shunli has joined #openstack-nova | 07:28 | |
*** rabel has joined #openstack-nova | 07:32 | |
rabel | hi together! | 07:32 |
rabel | i'm still trying to understand default quotas and quota groups. someone in here who knows a little bit about that? | 07:32 |
*** dimtruck is now known as zz_dimtruck | 07:33 | |
*** ekuris has quit IRC | 07:34 | |
*** voelzmo has quit IRC | 07:35 | |
*** jpena|away is now known as jpena | 07:37 | |
*** ekuris has joined #openstack-nova | 07:39 | |
*** amoralej|off is now known as amoralej | 07:43 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove the usage of extension black/white list opt in user data https://review.openstack.org/450834 | 07:44 |
*** ekuris has quit IRC | 07:45 | |
*** ekuris has joined #openstack-nova | 07:46 | |
*** marst has quit IRC | 07:47 | |
*** marst has joined #openstack-nova | 07:48 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | 07:49 |
*** ekuris has quit IRC | 07:51 | |
*** ekuris has joined #openstack-nova | 07:51 | |
*** yingjun has joined #openstack-nova | 07:51 | |
*** d9k has quit IRC | 07:56 | |
*** thorst has joined #openstack-nova | 07:57 | |
openstackgerrit | Silvan Kaiser proposed openstack/nova master: Removes getfattr from Quobyte Nova driver https://review.openstack.org/428646 | 07:58 |
*** abalutoiu has joined #openstack-nova | 07:58 | |
openstackgerrit | zhangyujun proposed openstack/nova master: Query parameter validation for hosts API https://review.openstack.org/418880 | 07:59 |
*** zzzeek has quit IRC | 08:00 | |
*** markus_z has joined #openstack-nova | 08:00 | |
*** zzzeek has joined #openstack-nova | 08:00 | |
*** trinaths has left #openstack-nova | 08:00 | |
*** gcb has quit IRC | 08:07 | |
*** efoley has joined #openstack-nova | 08:11 | |
*** iceyao has quit IRC | 08:12 | |
*** iceyao has joined #openstack-nova | 08:13 | |
*** efoley_ has joined #openstack-nova | 08:14 | |
*** ducttape_ has joined #openstack-nova | 08:14 | |
openstackgerrit | Lei Zhang proposed openstack/nova master: placement: Enable attach traits to ResourceProvider https://review.openstack.org/376201 | 08:15 |
openstackgerrit | Lei Zhang proposed openstack/nova master: Add sync traits command for placement https://review.openstack.org/450125 | 08:15 |
*** thorst has quit IRC | 08:17 | |
*** efoley has quit IRC | 08:18 | |
*** gcb has joined #openstack-nova | 08:19 | |
*** ducttape_ has quit IRC | 08:19 | |
openstackgerrit | Lei Zhang proposed openstack/nova master: Add sync traits command for placement https://review.openstack.org/450125 | 08:20 |
*** sapcc-bot has joined #openstack-nova | 08:21 | |
*** david_1 has joined #openstack-nova | 08:21 | |
*** carthaca_ has joined #openstack-nova | 08:21 | |
*** dgonzalez_ has joined #openstack-nova | 08:21 | |
*** tpatzig_ has joined #openstack-nova | 08:21 | |
*** mkoderer_ has joined #openstack-nova | 08:21 | |
*** seife_ has joined #openstack-nova | 08:21 | |
*** databus23_ has joined #openstack-nova | 08:21 | |
*** voelzmo has joined #openstack-nova | 08:21 | |
*** dgonzalez_ has quit IRC | 08:21 | |
*** david_1 has quit IRC | 08:21 | |
*** carthaca_ has quit IRC | 08:21 | |
*** tpatzig_ has quit IRC | 08:21 | |
*** databus23_ has quit IRC | 08:21 | |
*** sapcc-bot has quit IRC | 08:21 | |
*** mkoderer_ has quit IRC | 08:21 | |
*** seife_ has quit IRC | 08:21 | |
*** david_1 has joined #openstack-nova | 08:22 | |
*** sapcc-bot has joined #openstack-nova | 08:22 | |
*** carthaca_ has joined #openstack-nova | 08:22 | |
*** tpatzig_ has joined #openstack-nova | 08:22 | |
*** databus23_ has joined #openstack-nova | 08:22 | |
*** seife_ has joined #openstack-nova | 08:22 | |
*** mkoderer_ has joined #openstack-nova | 08:22 | |
*** dgonzalez_ has joined #openstack-nova | 08:22 | |
*** sapcc-bot has quit IRC | 08:22 | |
*** carthaca_ has quit IRC | 08:22 | |
*** dgonzalez_ has quit IRC | 08:22 | |
*** mkoderer_ has quit IRC | 08:22 | |
*** david_1 has quit IRC | 08:22 | |
*** tpatzig_ has quit IRC | 08:22 | |
*** databus23_ has quit IRC | 08:22 | |
*** seife_ has quit IRC | 08:22 | |
*** david_2 has joined #openstack-nova | 08:23 | |
*** seife_ has joined #openstack-nova | 08:23 | |
*** carthaca_ has joined #openstack-nova | 08:23 | |
*** dgonzalez_ has joined #openstack-nova | 08:23 | |
*** databus23_ has joined #openstack-nova | 08:23 | |
*** tpatzig_ has joined #openstack-nova | 08:23 | |
*** mkoderer_ has joined #openstack-nova | 08:23 | |
*** sapcc-bot has joined #openstack-nova | 08:23 | |
*** databus23_ has quit IRC | 08:23 | |
*** mkoderer_ has quit IRC | 08:23 | |
*** carthaca_ has quit IRC | 08:23 | |
*** seife_ has quit IRC | 08:23 | |
*** david_2 has quit IRC | 08:23 | |
*** sapcc-bot has quit IRC | 08:23 | |
*** dgonzalez_ has quit IRC | 08:23 | |
*** tpatzig_ has quit IRC | 08:23 | |
*** david_1 has joined #openstack-nova | 08:23 | |
*** sapcc-bot has joined #openstack-nova | 08:23 | |
*** tpatzig_ has joined #openstack-nova | 08:23 | |
*** dgonzalez_ has joined #openstack-nova | 08:23 | |
*** carthaca_ has joined #openstack-nova | 08:23 | |
*** mkoderer_ has joined #openstack-nova | 08:23 | |
*** seife_ has joined #openstack-nova | 08:23 | |
*** databus23_ has joined #openstack-nova | 08:23 | |
*** zz_dimtruck is now known as dimtruck | 08:24 | |
*** gszasz has joined #openstack-nova | 08:24 | |
*** databus23_ has quit IRC | 08:25 | |
*** seife_ has quit IRC | 08:25 | |
*** mkoderer_ has quit IRC | 08:25 | |
*** tpatzig_ has quit IRC | 08:25 | |
*** dgonzalez_ has quit IRC | 08:25 | |
*** carthaca_ has quit IRC | 08:25 | |
*** david_1 has quit IRC | 08:25 | |
*** voelzmo has quit IRC | 08:26 | |
*** priteau has joined #openstack-nova | 08:28 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | 08:30 |
*** udesale has quit IRC | 08:31 | |
*** karimb has quit IRC | 08:31 | |
*** jaosorior is now known as jaosorior_lunch | 08:31 | |
*** sapcc-bot has quit IRC | 08:32 | |
*** sapcc-bot has joined #openstack-nova | 08:32 | |
*** tpatzig_ has joined #openstack-nova | 08:32 | |
*** dgonzalez_ has joined #openstack-nova | 08:32 | |
*** mkoderer_ has joined #openstack-nova | 08:32 | |
*** carthaca_ has joined #openstack-nova | 08:32 | |
*** seife_ has joined #openstack-nova | 08:32 | |
*** databus23_ has joined #openstack-nova | 08:32 | |
*** david_1 has joined #openstack-nova | 08:32 | |
*** dimtruck is now known as zz_dimtruck | 08:34 | |
*** databus23_ has quit IRC | 08:34 | |
*** seife_ has quit IRC | 08:34 | |
*** mkoderer_ has quit IRC | 08:34 | |
*** carthaca_ has quit IRC | 08:34 | |
*** dgonzalez_ has quit IRC | 08:34 | |
*** tpatzig_ has quit IRC | 08:34 | |
*** david_1 has quit IRC | 08:34 | |
*** karimb has joined #openstack-nova | 08:35 | |
*** lucas-afk is now known as lucasagomes | 08:36 | |
*** d9k has joined #openstack-nova | 08:37 | |
*** zz_dimtruck is now known as dimtruck | 08:38 | |
*** karimb has quit IRC | 08:39 | |
*** derekh has joined #openstack-nova | 08:43 | |
*** diga has quit IRC | 08:43 | |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Permit Live Migration of Rescued Instances https://review.openstack.org/308198 | 08:47 |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: API changes for live migration of rescued instance https://review.openstack.org/328280 | 08:47 |
*** dimtruck is now known as zz_dimtruck | 08:47 | |
*** zenoway has joined #openstack-nova | 08:49 | |
openstackgerrit | Silvan Kaiser proposed openstack/nova master: Switch Nova Quobyte volume driver to mount via systemd-run https://review.openstack.org/432344 | 08:56 |
*** rabel has left #openstack-nova | 09:00 | |
*** karimb has joined #openstack-nova | 09:02 | |
*** cdent has joined #openstack-nova | 09:05 | |
*** sapcc-bot has quit IRC | 09:05 | |
*** mkoderer_ has joined #openstack-nova | 09:05 | |
*** databus23_ has joined #openstack-nova | 09:05 | |
*** tpatzig_ has joined #openstack-nova | 09:05 | |
*** seife_ has joined #openstack-nova | 09:05 | |
*** sapcc-bot has joined #openstack-nova | 09:05 | |
*** carthaca_ has joined #openstack-nova | 09:05 | |
*** dgonzalez_ has joined #openstack-nova | 09:05 | |
*** david_1 has joined #openstack-nova | 09:05 | |
*** databus23_ has quit IRC | 09:06 | |
*** seife_ has quit IRC | 09:06 | |
*** dgonzalez_ has quit IRC | 09:06 | |
*** carthaca_ has quit IRC | 09:06 | |
*** tpatzig_ has quit IRC | 09:06 | |
*** mkoderer_ has quit IRC | 09:06 | |
*** sapcc-bot has quit IRC | 09:06 | |
*** david_1 has quit IRC | 09:06 | |
*** psachin has quit IRC | 09:07 | |
*** sapcc-bot has joined #openstack-nova | 09:07 | |
*** lpetrut has joined #openstack-nova | 09:07 | |
*** lpetrut has quit IRC | 09:08 | |
*** diga has joined #openstack-nova | 09:09 | |
*** sambetts|afk is now known as sambetts | 09:09 | |
openstackgerrit | Silvan Kaiser proposed openstack/nova master: Removes potentially bad exit value from accepted list in Quobyte volume driver https://review.openstack.org/453537 | 09:10 |
*** psachin has joined #openstack-nova | 09:11 | |
*** thorst has joined #openstack-nova | 09:14 | |
*** thorst has quit IRC | 09:18 | |
*** aarefiev_afk is now known as aarefiev | 09:19 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | 09:21 |
*** kaisers has quit IRC | 09:24 | |
*** iceyao has quit IRC | 09:28 | |
*** zsli_ has joined #openstack-nova | 09:29 | |
*** mlakat has joined #openstack-nova | 09:31 | |
*** Shunli has quit IRC | 09:32 | |
*** xinliang has quit IRC | 09:33 | |
*** xinliang has joined #openstack-nova | 09:34 | |
*** xinliang has joined #openstack-nova | 09:34 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | 09:34 |
openstackgerrit | Bob Ball proposed openstack/nova master: Use physical utilisation for cached images https://review.openstack.org/353862 | 09:35 |
*** zsli_ has quit IRC | 09:36 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases https://review.openstack.org/394738 | 09:36 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases https://review.openstack.org/395436 | 09:36 |
*** MarginHu has quit IRC | 09:37 | |
*** MarginHu has joined #openstack-nova | 09:37 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444 | 09:37 |
*** MarginHu has quit IRC | 09:38 | |
*** MarginHu has joined #openstack-nova | 09:38 | |
*** MarginHu has quit IRC | 09:39 | |
*** wangqwsh has quit IRC | 09:41 | |
*** ijw_ has quit IRC | 09:42 | |
*** iceyao has joined #openstack-nova | 09:43 | |
*** zyun has quit IRC | 09:44 | |
*** ducttape_ has joined #openstack-nova | 09:45 | |
*** ducttape_ has quit IRC | 09:49 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | 09:55 |
*** nkrinner has quit IRC | 09:56 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove the usage of extension black/white list opt in user data https://review.openstack.org/450834 | 09:58 |
*** mvk has quit IRC | 10:00 | |
*** NikhilS has quit IRC | 10:01 | |
markus_z | oomichi: sdague: Please have a look at https://review.openstack.org/#/c/453220/ | 10:02 |
*** tovin07_ has quit IRC | 10:04 | |
*** nicolasbock has joined #openstack-nova | 10:07 | |
*** voelzmo has joined #openstack-nova | 10:10 | |
*** mlakat has quit IRC | 10:10 | |
*** ljjjustin has quit IRC | 10:12 | |
*** satyar has quit IRC | 10:13 | |
openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add additional-default-policy-roles spec https://review.openstack.org/427872 | 10:14 |
*** voelzmo has quit IRC | 10:14 | |
*** blaisebool has quit IRC | 10:16 | |
*** mlakat has joined #openstack-nova | 10:16 | |
*** kevinz has quit IRC | 10:20 | |
*** kaisers has joined #openstack-nova | 10:20 | |
openstackgerrit | jichenjc proposed openstack/nova master: Add allocation doc for placement https://review.openstack.org/450550 | 10:20 |
*** rfolco has joined #openstack-nova | 10:21 | |
*** pcaruana|afk| has joined #openstack-nova | 10:21 | |
openstackgerrit | jichenjc proposed openstack/nova master: Add allocation doc for placement https://review.openstack.org/450550 | 10:23 |
*** pcaruana|afk| has quit IRC | 10:23 | |
*** pcaruana has quit IRC | 10:24 | |
openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.resize.error notifications https://review.openstack.org/417882 | 10:24 |
openstackgerrit | jichenjc proposed openstack/nova master: [placement] Add allocation doc for placement https://review.openstack.org/450550 | 10:24 |
*** pcaruana has joined #openstack-nova | 10:25 | |
*** amotoki has quit IRC | 10:26 | |
*** owalsh_ is now known as owalsh | 10:29 | |
*** zhurong has quit IRC | 10:30 | |
*** jaosorior_lunch is now known as jaosorior | 10:31 | |
openstackgerrit | jichenjc proposed openstack/nova master: add usage doc for resource provider https://review.openstack.org/450105 | 10:31 |
*** mvk has joined #openstack-nova | 10:32 | |
*** yingjun has quit IRC | 10:34 | |
*** gongysh has quit IRC | 10:35 | |
*** ijw has joined #openstack-nova | 10:38 | |
*** voelzmo has joined #openstack-nova | 10:38 | |
openstackgerrit | jichenjc proposed openstack/nova master: fixed typos and reword stable api doc https://review.openstack.org/442310 | 10:39 |
*** zz_dimtruck is now known as dimtruck | 10:39 | |
*** garyk has quit IRC | 10:39 | |
*** CristinaPauna has quit IRC | 10:41 | |
*** gongysh has joined #openstack-nova | 10:41 | |
*** haplo37 has quit IRC | 10:43 | |
*** ijw has quit IRC | 10:49 | |
*** dimtruck is now known as zz_dimtruck | 10:49 | |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove '/servers' extensions info from controllers https://review.openstack.org/453571 | 10:49 |
*** tbachman has quit IRC | 10:50 | |
*** rmart04 has joined #openstack-nova | 10:50 | |
*** CristinaPauna has joined #openstack-nova | 10:51 | |
openstackgerrit | jichenjc proposed openstack/nova master: Add resource classes description in api-ref https://review.openstack.org/451661 | 10:51 |
*** haplo37 has joined #openstack-nova | 10:52 | |
*** kaisers has quit IRC | 10:53 | |
*** READ10 has quit IRC | 10:53 | |
*** sapcc-bot has quit IRC | 10:53 | |
*** sapcc-bot has joined #openstack-nova | 10:54 | |
*** david_1 has joined #openstack-nova | 10:55 | |
*** dgonzalez_ has joined #openstack-nova | 10:55 | |
*** mkoderer_ has joined #openstack-nova | 10:55 | |
*** databus23_ has joined #openstack-nova | 10:55 | |
*** carthaca_ has joined #openstack-nova | 10:55 | |
*** gcb has quit IRC | 10:57 | |
*** databus23_ has quit IRC | 10:57 | |
*** mkoderer_ has quit IRC | 10:57 | |
*** dgonzalez_ has quit IRC | 10:57 | |
*** david_1 has quit IRC | 10:57 | |
*** phuongnh has quit IRC | 10:59 | |
*** sree has quit IRC | 11:02 | |
*** sree_ has joined #openstack-nova | 11:04 | |
*** sree_ is now known as Guest55499 | 11:05 | |
*** dave-mccowan has joined #openstack-nova | 11:06 | |
*** Guest55499 has quit IRC | 11:08 | |
*** udesale has joined #openstack-nova | 11:12 | |
*** CristinaPauna has quit IRC | 11:14 | |
*** CristinaPauna has joined #openstack-nova | 11:14 | |
*** thorst has joined #openstack-nova | 11:15 | |
*** Jack_Iv has quit IRC | 11:16 | |
*** udesale has quit IRC | 11:17 | |
johnthetubaguy | bauzas: cdent: are you folks OK with this one by any chance? https://review.openstack.org/#/c/453030 | 11:19 |
*** Guest63262 has quit IRC | 11:19 | |
* cdent looks | 11:19 | |
*** yingjun has joined #openstack-nova | 11:19 | |
cdent | johnthetubaguy: i'm not up to date with the changes since my comments since ps1, but in principle, yes | 11:19 |
*** thorst has quit IRC | 11:19 | |
*** udesale has joined #openstack-nova | 11:20 | |
*** tommylikehu_ has joined #openstack-nova | 11:20 | |
*** Jack_Iv has joined #openstack-nova | 11:22 | |
*** cristicalin has joined #openstack-nova | 11:22 | |
johnthetubaguy | cdent: OK, cool, I just didn't feel I should +W without more placement experts feeling happy with it | 11:23 |
cdent | I'm doing a re-review now just to be sure | 11:23 |
*** tommylikehu_ has quit IRC | 11:24 | |
*** takashin has left #openstack-nova | 11:25 | |
*** liverpooler has quit IRC | 11:26 | |
*** dixiaoli has joined #openstack-nova | 11:27 | |
*** dixiaoli has quit IRC | 11:27 | |
*** dixiaoli has joined #openstack-nova | 11:27 | |
*** dixiaoli has quit IRC | 11:28 | |
cdent | johnthetubaguy: had some minor suggestions, but I think they could be followups | 11:28 |
*** Guest63262 has joined #openstack-nova | 11:29 | |
*** diga_ has joined #openstack-nova | 11:31 | |
*** diga has quit IRC | 11:31 | |
*** gongysh has quit IRC | 11:37 | |
openstackgerrit | Emilien Macchi proposed openstack/nova master: https://review.openstack.org/453338 | 11:37 |
*** amotoki has joined #openstack-nova | 11:38 | |
openstackgerrit | Emilien Macchi proposed openstack/nova master: Add sem-ver flag so pbr generates correct version https://review.openstack.org/453338 | 11:39 |
*** adisky_ has quit IRC | 11:39 | |
*** cNilesh has quit IRC | 11:39 | |
*** adisky_ has joined #openstack-nova | 11:40 | |
*** zz_dimtruck is now known as dimtruck | 11:40 | |
*** sapcc-bot1 has joined #openstack-nova | 11:41 | |
*** gongysh has joined #openstack-nova | 11:41 | |
*** thorst has joined #openstack-nova | 11:41 | |
*** carthaca_ has quit IRC | 11:41 | |
*** sapcc-bot has quit IRC | 11:41 | |
*** carthaca_ has joined #openstack-nova | 11:41 | |
*** dgonzalez_ has joined #openstack-nova | 11:41 | |
*** seife_ has joined #openstack-nova | 11:41 | |
*** databus23_ has joined #openstack-nova | 11:41 | |
*** mkoderer_ has joined #openstack-nova | 11:41 | |
*** david_1 has joined #openstack-nova | 11:41 | |
*** david_2 has joined #openstack-nova | 11:42 | |
*** databus23_1 has joined #openstack-nova | 11:42 | |
*** seife_1 has joined #openstack-nova | 11:42 | |
*** mkoderer_1 has joined #openstack-nova | 11:42 | |
*** dgonzalez_1 has joined #openstack-nova | 11:42 | |
*** carthaca_1 has joined #openstack-nova | 11:42 | |
*** sapcc-bot has joined #openstack-nova | 11:42 | |
openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-exists notification https://review.openstack.org/403660 | 11:42 |
*** karimb has quit IRC | 11:43 | |
*** carthaca_ has quit IRC | 11:43 | |
*** dgonzalez_ has quit IRC | 11:43 | |
*** seife_ has quit IRC | 11:43 | |
*** david_1 has quit IRC | 11:43 | |
*** databus23_ has quit IRC | 11:43 | |
*** mkoderer_ has quit IRC | 11:43 | |
*** carthaca_1 has quit IRC | 11:44 | |
*** seife_1 has quit IRC | 11:44 | |
*** dgonzalez_1 has quit IRC | 11:44 | |
*** mkoderer_1 has quit IRC | 11:44 | |
*** databus23_1 has quit IRC | 11:44 | |
*** david_2 has quit IRC | 11:44 | |
*** smatzek has joined #openstack-nova | 11:44 | |
*** tommylikehu_ has joined #openstack-nova | 11:46 | |
*** smatzek has quit IRC | 11:46 | |
*** smatzek has joined #openstack-nova | 11:46 | |
*** kaisers has joined #openstack-nova | 11:46 | |
*** gongysh has quit IRC | 11:47 | |
*** tommylikehu_ has quit IRC | 11:47 | |
*** tommylikehu_ has joined #openstack-nova | 11:47 | |
*** timello has joined #openstack-nova | 11:48 | |
*** vladikr has joined #openstack-nova | 11:49 | |
*** dimtruck is now known as zz_dimtruck | 11:50 | |
*** tommylikehu_ has quit IRC | 11:50 | |
*** tommylikehu_ has joined #openstack-nova | 11:50 | |
*** cristicalin has quit IRC | 11:52 | |
*** yasemin_ has joined #openstack-nova | 11:54 | |
yasemin_ | hi, i installed stable/ocata , multinode devstack and when i add compute node, it gives an error like Failed to perform requested operation on instance "demo", the instance has an error status: Please try again later [Error: Host 'compute-1' is not mapped to any cell]. | 11:56 |
yasemin_ | can you help me ? | 11:56 |
*** abalutoiu has quit IRC | 11:56 | |
*** gongysh has joined #openstack-nova | 12:00 | |
artom | yasemin_, I think you need to run nova-manage cell_v2 discover_hosts | 12:01 |
*** tbachman has joined #openstack-nova | 12:02 | |
yasemin_ | artom, how can i run it ? | 12:03 |
*** cdent has quit IRC | 12:03 | |
artom | yasemin_, from the command line on the controller node | 12:03 |
*** gongysh has quit IRC | 12:05 | |
yasemin_ | atom, how is the command ? or can you suggest any doc ? | 12:10 |
*** vks1 has quit IRC | 12:10 | |
*** edmondsw has joined #openstack-nova | 12:10 | |
*** kaisers has quit IRC | 12:15 | |
artom | yasemin_, the command is 'nova-manage cell_v2 discover_hosts' :) | 12:16 |
frickler | yasemin_: https://docs.openstack.org/developer/nova/cells.html#setup-of-cells-v2 | 12:16 |
artom | frickler, I think his issue is that he added a compute, and the compute isn't mapped to a cell | 12:17 |
*** Jack_Iv has quit IRC | 12:17 | |
*** mnestratov has joined #openstack-nova | 12:18 | |
yasemin_ | artom, :) sorry | 12:18 |
yasemin_ | /usr/local/lib/python2.7/dist-packages/pymysql/cursors.py:166: Warning: (3090, u"Changing sql mode 'NO_AUTO_CREATE_USER' is deprecated. It will be removed in a future release.") | 12:18 |
yasemin_ | result = self._query(quer | 12:18 |
frickler | artom: yes, and the step you mention is correct, I was just adding the doc component for it | 12:19 |
artom | I'm pretty sure there are plans to run discover_hosts automatically, either in a periodic task, or upon manual compute node addition, but those haven't materialized yet, so it has to be run manually | 12:19 |
*** abalutoiu has joined #openstack-nova | 12:19 | |
*** liverpooler has joined #openstack-nova | 12:19 | |
frickler | artom: one can enable it with this I think: https://review.openstack.org/#/c/453004/1/doc/install-guide/source/nova-compute-install.rst | 12:20 |
yasemin_ | i think , it should be automatically :) | 12:20 |
openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use plain routes list for '/servers' endpoint instead of stevedore https://review.openstack.org/445864 | 12:24 |
*** ducttape_ has joined #openstack-nova | 12:26 | |
*** ZZelle has joined #openstack-nova | 12:27 | |
*** ZZelle is now known as Guest85496 | 12:27 | |
*** Guest85496 is now known as ZZelle_ | 12:27 | |
*** Jack_Iv has joined #openstack-nova | 12:28 | |
*** gouthamr has joined #openstack-nova | 12:29 | |
*** jaypipes has joined #openstack-nova | 12:29 | |
*** yingjun has quit IRC | 12:31 | |
*** Jack_Iv has quit IRC | 12:32 | |
markus_z | oomichi: Please have a look at https://review.openstack.org/#/c/453220/ | 12:35 |
*** Swami has joined #openstack-nova | 12:35 | |
*** karimb has joined #openstack-nova | 12:36 | |
*** kfarr has joined #openstack-nova | 12:39 | |
efried | sdague Remember that thing the other day where threading was killinme and you pointed me to recent eventlet changes? | 12:40 |
efried | sdague https://review.openstack.org/#/c/453301/ is part of the solution. You're a requirements bigwig, right? | 12:40 |
efried | Would you mind having a look? | 12:40 |
*** zz_dimtruck is now known as dimtruck | 12:41 | |
*** amotoki has quit IRC | 12:41 | |
*** sapcc-bot1 has quit IRC | 12:41 | |
*** sapcc-bot has quit IRC | 12:41 | |
*** sapcc-bot has joined #openstack-nova | 12:41 | |
*** jpena is now known as jpena|lunch | 12:41 | |
*** david_1 has joined #openstack-nova | 12:41 | |
*** carthaca_ has joined #openstack-nova | 12:41 | |
*** dgonzalez_ has joined #openstack-nova | 12:41 | |
*** mkoderer_ has joined #openstack-nova | 12:42 | |
*** mkoderer_ has quit IRC | 12:43 | |
*** dgonzalez_ has quit IRC | 12:43 | |
*** carthaca_ has quit IRC | 12:43 | |
*** david_1 has quit IRC | 12:43 | |
*** ducttape_ has quit IRC | 12:45 | |
*** ducttape_ has joined #openstack-nova | 12:45 | |
jaypipes | jianghuaw_: morning (or evening) :) | 12:46 |
*** garyk has joined #openstack-nova | 12:46 | |
jaypipes | jianghuaw_: so I've been wondering about something in the VGPU spec... | 12:46 |
sdague | efried: +2 | 12:46 |
efried | sdague Thank you sir. | 12:46 |
jaypipes | are you and Bob Ball available for a Google hangout this morning to discuss a couple things? | 12:46 |
*** dillaman has joined #openstack-nova | 12:47 | |
*** tongli has joined #openstack-nova | 12:49 | |
*** Drankis has quit IRC | 12:49 | |
*** dimtruck is now known as zz_dimtruck | 12:50 | |
gmann | Nova API meeting in 10 min on #openstack-meeting-4 | 12:51 |
-openstackstatus- NOTICE: The Gerrit service on http://review.openstack.org is being restarted to address hung remote replication tasks, and should return to an operable state momentarily | 12:51 | |
*** suresh12 has joined #openstack-nova | 12:51 | |
*** psachin has quit IRC | 12:52 | |
*** gouthamr has quit IRC | 12:53 | |
*** Jack_Iv has joined #openstack-nova | 12:53 | |
*** palimarium has quit IRC | 12:53 | |
markus_z | alex_xu: Please have a look at https://review.openstack.org/#/c/453220/ | 12:54 |
*** markvoelker has joined #openstack-nova | 12:55 | |
*** lyan has joined #openstack-nova | 12:55 | |
*** amotoki has joined #openstack-nova | 12:56 | |
*** yingjun has joined #openstack-nova | 12:56 | |
jaypipes | mdbooth: "searchlight should load it from nova" <-- hmm, pretty sure Searchlight isn't going to want to do that :) | 12:58 |
jaypipes | mdbooth: though I agree with your general statement. | 12:58 |
jaypipes | about long-term maintenance of the code. | 12:58 |
*** gouthamr has joined #openstack-nova | 12:58 | |
mdbooth | jaypipes: I briefly poked around in searchlight's nova plugin, and it contains a bunch of assumptions about state transitions | 12:59 |
mdbooth | Things which we don't even do a good job of following just within compute manager | 12:59 |
jaypipes | mdbooth: like I said, I agree with your general statement :) | 12:59 |
jaypipes | mdbooth: I'm just saying I don't believe searchlight will want to make a dependency on any nova Python package. | 12:59 |
*** mtanino has joined #openstack-nova | 12:59 | |
mdbooth | Duplicating in a proverbial locked cabinet behind a door marked 'beware of the leopard' seems unwise | 12:59 |
*** eharney has joined #openstack-nova | 13:00 | |
jaypipes | mdbooth: unless we created a different wheel entirely for this, housed in the nova source tree, but packaged separately. | 13:00 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications https://review.openstack.org/453077 | 13:00 |
mdbooth | jaypipes: Right, I wasn't really suggesting that. More that if they defined a plugin api, they could just load a bunch of plugins | 13:00 |
*** Jack_Iv has quit IRC | 13:00 | |
jaypipes | ya | 13:00 |
mdbooth | And the nova plugin would be part of nova | 13:00 |
jaypipes | totally makes sense. | 13:00 |
mriedem | searchlight wouldn't import nova, it would load a stevedore extension right? | 13:01 |
mriedem | all runtime shenanigans | 13:01 |
mdbooth | mriedem: Yeah, totally at runtime | 13:01 |
jaypipes | mriedem: yeah, but how would the Nova plugin code "get" to searchlight? | 13:01 |
jaypipes | mriedem: you need to ship the code from something... | 13:01 |
jaypipes | mriedem: that's all I'm referring to here. the mechanism for delivering that code into a place searchlight knows about... | 13:02 |
jaypipes | alex_xu: around? I'd love to make progress on https://review.openstack.org/#/c/376200/ today if you can. | 13:02 |
*** felipemonteiro has joined #openstack-nova | 13:03 | |
*** iceyao has quit IRC | 13:03 | |
bauzas | johnthetubaguy: reviewing it now | 13:04 |
*** ratailor has quit IRC | 13:04 | |
*** psachin has joined #openstack-nova | 13:05 | |
*** links has quit IRC | 13:08 | |
*** dane-fichter has joined #openstack-nova | 13:09 | |
yasemin_ | artom, which one is true for compute nodes, placement-api or placement-client? | 13:09 |
*** catintheroof has joined #openstack-nova | 13:10 | |
*** catintheroof has quit IRC | 13:10 | |
dane-fichter | jaypipes: have you looked at the newest draft of the cert validation spec? | 13:11 |
*** kaisers has joined #openstack-nova | 13:11 | |
*** catintheroof has joined #openstack-nova | 13:11 | |
jaypipes | dane-fichter: no sir, not yet. | 13:11 |
dane-fichter | jaypipes: ok. I think peter posted it yesterday but it was after y'all had already kind of established your queues for spec review day | 13:11 |
*** zz_dimtruck is now known as dimtruck | 13:12 | |
*** ducttape_ has quit IRC | 13:12 | |
jaypipes | dane-fichter: will take another look today. | 13:12 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add snapshot id to the snapshot notifications https://review.openstack.org/453077 | 13:12 |
dane-fichter | jaypipes: thanks jay! | 13:13 |
yasemin_ | frickler, artom, which one is true for compute nodes, placement-api or placement-client? | 13:13 |
*** esberglu has joined #openstack-nova | 13:15 | |
*** Jack_Iv has joined #openstack-nova | 13:16 | |
*** amoralej is now known as amoralej|lunch | 13:17 | |
*** mdrabe has joined #openstack-nova | 13:17 | |
*** liusheng has quit IRC | 13:17 | |
*** liusheng has joined #openstack-nova | 13:18 | |
*** xyang1 has joined #openstack-nova | 13:20 | |
*** Jack_Iv has quit IRC | 13:21 | |
*** mdnadeem_ has joined #openstack-nova | 13:23 | |
*** prateek_ has joined #openstack-nova | 13:23 | |
*** abalutoiu has quit IRC | 13:23 | |
*** peter-hamilton has joined #openstack-nova | 13:23 | |
*** psachin_ has joined #openstack-nova | 13:23 | |
*** derekh has quit IRC | 13:24 | |
*** prateek__ has joined #openstack-nova | 13:25 | |
*** prateek has quit IRC | 13:26 | |
*** psachin has quit IRC | 13:26 | |
*** mdnadeem has quit IRC | 13:26 | |
*** prateek_ has quit IRC | 13:28 | |
*** cleong has joined #openstack-nova | 13:28 | |
*** mkoderer has joined #openstack-nova | 13:28 | |
*** psachin_ has quit IRC | 13:28 | |
*** udesale has quit IRC | 13:28 | |
*** mdnadeem_ has quit IRC | 13:28 | |
mriedem | claudiub: for volume attach, does hyperv honor the user-supplied device name (if one is provided)? | 13:28 |
*** stvnoyes has joined #openstack-nova | 13:29 | |
*** zenoway has quit IRC | 13:31 | |
*** zenoway has joined #openstack-nova | 13:31 | |
*** prateek_ has joined #openstack-nova | 13:32 | |
*** psachin_ has joined #openstack-nova | 13:32 | |
*** mdnadeem_ has joined #openstack-nova | 13:33 | |
*** Jack_Iv has joined #openstack-nova | 13:33 | |
*** rmart04 has quit IRC | 13:33 | |
*** mnestratov has quit IRC | 13:35 | |
*** rmart04 has joined #openstack-nova | 13:35 | |
*** prateek has joined #openstack-nova | 13:35 | |
*** prateek__ has quit IRC | 13:35 | |
*** ducttape_ has joined #openstack-nova | 13:36 | |
*** karimb has quit IRC | 13:36 | |
artom | mriedem, wait, in a Windows guest? | 13:36 |
artom | As in, 'D:' ? | 13:36 |
*** garyk has quit IRC | 13:37 | |
*** mdnadeem_ has quit IRC | 13:37 | |
mriedem | hyperv can run linux guests right? | 13:37 |
*** psachin_ has quit IRC | 13:37 | |
*** prateek_ has quit IRC | 13:37 | |
mriedem | but that's a good question - what if it's a windows guest | 13:37 |
artom | yasemin_, I'm not sure I understand your question, how can placement-api be true on a compute node? | 13:37 |
mriedem | yasemin_: for devstack? placement-client | 13:38 |
mriedem | as a subnode | 13:38 |
mriedem | in devstack | 13:38 |
yasemin_ | artom, yes for devstack compute node enable services | 13:38 |
artom | mriedem, huh, TIL hyperv isn't limited to Windows guests | 13:38 |
mriedem | yasemin_: https://github.com/openstack-infra/devstack-gate/blob/master/features.yaml#L269 | 13:38 |
mriedem | yasemin_: that's when d-g is setting up a subnode | 13:39 |
artom | yasemin_, compute node only needs placement-client, methinks | 13:39 |
yasemin_ | okey | 13:40 |
*** zenoway has quit IRC | 13:40 | |
EmilienM | hello here, can you review this patch please? https://review.openstack.org/#/c/453338 - it would help TripleO CI to test upgrades from Ocata to Pike. Thanks a lot! | 13:40 |
jaypipes | melwitt: pls see my comments on https://review.openstack.org/#/c/453030/... | 13:42 |
*** lpetrut has joined #openstack-nova | 13:42 | |
jaypipes | melwitt: it's already been approved, but I have some requests for you in there. | 13:42 |
jaypipes | melwitt: we should really be using GET /usages, not GET /allocations for the quota-counting call. | 13:43 |
*** crushil has joined #openstack-nova | 13:43 | |
*** rmart04 has quit IRC | 13:44 | |
*** jpena|lunch is now known as jpena | 13:44 | |
*** jamesdenton has joined #openstack-nova | 13:45 | |
*** hshiina has joined #openstack-nova | 13:45 | |
claudiub | mriedem: hi. nope, we can't honor that. | 13:46 |
*** prateek_ has joined #openstack-nova | 13:46 | |
claudiub | mriedem: and ofc, hyper-v can run linux guests. :) | 13:46 |
*** smatzek has quit IRC | 13:46 | |
mriedem | claudiub: thanks | 13:47 |
mriedem | i'm not sure about vmware | 13:47 |
mriedem | i think xen might be the only one | 13:47 |
*** david_1 has joined #openstack-nova | 13:47 | |
*** seife_ has joined #openstack-nova | 13:47 | |
*** dgonzalez_ has joined #openstack-nova | 13:47 | |
*** mkoderer_ has joined #openstack-nova | 13:47 | |
*** carthaca_ has joined #openstack-nova | 13:47 | |
*** tpatzig_ has joined #openstack-nova | 13:47 | |
*** prateek__ has joined #openstack-nova | 13:48 | |
*** dimtruck is now known as zz_dimtruck | 13:48 | |
*** baoli has joined #openstack-nova | 13:48 | |
*** gcb has joined #openstack-nova | 13:48 | |
bauzas | jaypipes: heh, split-brain is not exactly happening in systems with leader elections | 13:49 |
*** carthaca_ has quit IRC | 13:49 | |
*** dgonzalez_ has quit IRC | 13:49 | |
*** mkoderer_ has quit IRC | 13:49 | |
*** seife_ has quit IRC | 13:49 | |
*** tpatzig_ has quit IRC | 13:49 | |
*** david_1 has quit IRC | 13:49 | |
bauzas | it can happen there, but it's just a case | 13:49 |
*** psachin_ has joined #openstack-nova | 13:49 | |
bauzas | my own definition of split-brain is when you have 2 datasets being inconsistent | 13:49 |
*** adisky_ has quit IRC | 13:49 | |
bauzas | which is larger than yours :) | 13:49 |
*** prateek has quit IRC | 13:49 | |
*** ekuris has quit IRC | 13:50 | |
*** zenoway has joined #openstack-nova | 13:50 | |
*** diga_ has quit IRC | 13:51 | |
*** prateek_ has quit IRC | 13:51 | |
*** prateek__ has quit IRC | 13:51 | |
*** zenoway has quit IRC | 13:55 | |
*** rmart04 has joined #openstack-nova | 13:55 | |
melwitt | jaypipes: okay, thanks. I can propose an amendment to the spec | 13:56 |
*** rmart04 has quit IRC | 13:57 | |
*** amoralej|lunch is now known as amoralej | 13:57 | |
*** garyk has joined #openstack-nova | 13:57 | |
*** lpetrut has quit IRC | 13:58 | |
*** lpetrut has joined #openstack-nova | 13:58 | |
openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add remove-nova-cert spec https://review.openstack.org/448882 | 13:58 |
*** tbachman has quit IRC | 14:00 | |
*** ducttape_ has quit IRC | 14:01 | |
*** rmart04 has joined #openstack-nova | 14:01 | |
gmann | sdague: on extension removal, as i updated the base patch in that series, i ll rebase others if needed that might need re approval | 14:02 |
gmann | johnthetubaguy: can you check this base patch https://review.openstack.org/#/c/450834/ | 14:03 |
*** sree has joined #openstack-nova | 14:03 | |
*** iceyao has joined #openstack-nova | 14:03 | |
*** zenoway has joined #openstack-nova | 14:05 | |
*** zul has quit IRC | 14:05 | |
dane-fichter | mriedem: when you get a chance, could you take a look at the newest cursive patch? it passes the experimental check. | 14:05 |
dane-fichter | mriedem: https://review.openstack.org/#/c/351232/ | 14:06 |
*** rmart04 has quit IRC | 14:06 | |
*** hongbin has joined #openstack-nova | 14:06 | |
mriedem | yeah | 14:07 |
*** jistr is now known as jistr|mtg | 14:07 | |
*** sree has quit IRC | 14:07 | |
*** iceyao has quit IRC | 14:08 | |
*** zenoway has quit IRC | 14:09 | |
*** smatzek has joined #openstack-nova | 14:10 | |
*** udesale has joined #openstack-nova | 14:12 | |
*** zenoway has joined #openstack-nova | 14:12 | |
*** arne_r has quit IRC | 14:15 | |
mriedem | jaypipes: on https://review.openstack.org/#/c/453030/ to be clear, are you saying both GET /allocations *and* GET /usages, or just the latter as a new handler? | 14:15 |
*** awaugama has joined #openstack-nova | 14:16 | |
melwitt | is the placement-api-ref on docs.o.o anywhere? I can't find it | 14:17 |
jaypipes | mriedem: implement both in the REST API but have the quota-handling call only the GET /usages | 14:17 |
*** zz_dimtruck is now known as dimtruck | 14:17 | |
*** mdavidson has quit IRC | 14:18 | |
mriedem | jaypipes: why both | 14:18 |
mriedem | ? | 14:18 |
mriedem | why do we need two handlers for the same thing? | 14:18 |
jaypipes | mriedem: well, one is the detail view, the other is the aggregate view. | 14:18 |
mriedem | i think we need a respin of the spec then with the rest api impact section filled out with those details | 14:19 |
mriedem | if we're adding a new handler | 14:19 |
melwitt | mriedem: I'm working on an amendment for the spec to address everyone's comments | 14:19 |
sdague | melwitt: I'm not sure that publish loop ever fully got setup | 14:20 |
*** abalutoiu has joined #openstack-nova | 14:20 | |
melwitt | sdague: ack, thanks | 14:20 |
bauzas | melwitt: it's in devref | 14:21 |
bauzas | melwitt: https://docs.openstack.org/developer/nova/placement.html | 14:21 |
melwitt | bauzas: I found this but it says it's forthcoming https://docs.openstack.org/developer/nova/placement.html#api-reference | 14:21 |
bauzas | oh ok, my bad then | 14:21 |
*** zul has joined #openstack-nova | 14:22 | |
*** david_1 has joined #openstack-nova | 14:23 | |
*** carthaca_ has joined #openstack-nova | 14:23 | |
*** tpatzig_ has joined #openstack-nova | 14:23 | |
*** dgonzalez_ has joined #openstack-nova | 14:23 | |
*** databus23_ has joined #openstack-nova | 14:23 | |
*** mkoderer_ has joined #openstack-nova | 14:23 | |
*** satyar has joined #openstack-nova | 14:24 | |
*** mkoderer_ has quit IRC | 14:25 | |
*** databus23_ has quit IRC | 14:25 | |
*** dgonzalez_ has quit IRC | 14:25 | |
*** tpatzig_ has quit IRC | 14:25 | |
*** carthaca_ has quit IRC | 14:25 | |
*** david_1 has quit IRC | 14:25 | |
*** dgonzalez_ has joined #openstack-nova | 14:26 | |
*** tommylikehu_ has quit IRC | 14:26 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: explain payload inheritance in notification devref https://review.openstack.org/453667 | 14:27 |
*** ericyoung has joined #openstack-nova | 14:29 | |
*** abalutoiu has quit IRC | 14:29 | |
*** mnestratov has joined #openstack-nova | 14:29 | |
*** sridharg has quit IRC | 14:30 | |
*** nmathew has quit IRC | 14:30 | |
*** eharney has quit IRC | 14:30 | |
*** cdent has joined #openstack-nova | 14:32 | |
*** derekh has joined #openstack-nova | 14:32 | |
*** trinaths has joined #openstack-nova | 14:33 | |
*** zenoway has quit IRC | 14:34 | |
*** zenoway has joined #openstack-nova | 14:35 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged nic attachment https://review.openstack.org/415622 | 14:36 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete nic metadata when detaching interface https://review.openstack.org/435387 | 14:36 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged volume attachment https://review.openstack.org/418100 | 14:36 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Delete disk metadata when detaching volume https://review.openstack.org/435082 | 14:36 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Plumbing for tagged volume attachment https://review.openstack.org/391941 | 14:36 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: API support for tagged device attachment https://review.openstack.org/376021 | 14:37 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Libvirt support for tagged nic attachment https://review.openstack.org/415520 | 14:37 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add method to invalidate metadata cache https://review.openstack.org/453673 | 14:37 |
*** iceyao has joined #openstack-nova | 14:37 | |
*** eharney has joined #openstack-nova | 14:40 | |
*** armax has joined #openstack-nova | 14:41 | |
*** lpetrut has quit IRC | 14:41 | |
*** trinaths has left #openstack-nova | 14:44 | |
*** zenoway has quit IRC | 14:44 | |
*** dikonoor has quit IRC | 14:45 | |
*** garyk has quit IRC | 14:46 | |
*** zul_ has joined #openstack-nova | 14:47 | |
*** marst has quit IRC | 14:48 | |
*** vks1 has joined #openstack-nova | 14:48 | |
*** crushil has quit IRC | 14:50 | |
*** crushil has joined #openstack-nova | 14:51 | |
*** efried has quit IRC | 14:52 | |
openstackgerrit | John Garbutt proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | 14:52 |
*** zenoway has joined #openstack-nova | 14:54 | |
*** vks1 has quit IRC | 14:55 | |
*** karimb has joined #openstack-nova | 14:56 | |
*** timello has quit IRC | 14:56 | |
*** marst has joined #openstack-nova | 14:57 | |
*** zenoway has quit IRC | 14:58 | |
*** zul_ has quit IRC | 14:59 | |
*** kaisers has quit IRC | 14:59 | |
*** kaisers has joined #openstack-nova | 14:59 | |
*** zenoway has joined #openstack-nova | 15:00 | |
jaypipes | cdent: it's not about "doing a bit of math on the client side". rather, it's just making the REST API more useful, IMHO. | 15:00 |
*** voelzmo has quit IRC | 15:01 | |
*** udesale has quit IRC | 15:01 | |
cdent | I get where you're coming from, but you solution adds an endpoint for a specific result that can be serviced by something more generic that fits in with the existing grammar | 15:02 |
cdent | And I'm trying to toe the line on "only add stuff if we have to" | 15:02 |
cdent | and in this case there are plenty of ways to "not have to" | 15:02 |
cdent | If we decide to go with /usages, that's fine it's not a big deal | 15:03 |
edleafe | And is "usage" a resource? (in the REST sense of the word) | 15:03 |
cdent | but since we can do it another way, that's fine too | 15:03 |
jaypipes | edleafe: we already have /r-p/{uuid}/usages | 15:03 |
jaypipes | cdent: in either, case I'm fine doing it only in /allocations to start off and adding the /usages at a later time. | 15:05 |
jaypipes | mriedem: ^ | 15:05 |
cdent | jaypipes: that seems like a reasonable path: if it turns out it is just too damn slow or code-messy to do the math client side, then adding later is great | 15:06 |
*** dharinic has joined #openstack-nova | 15:06 | |
jaypipes | yep, ok with me. | 15:06 |
jaypipes | for some reason I thought we had a top-level /usages endpoint already :( | 15:06 |
*** armax has quit IRC | 15:07 | |
cdent | jaypipes: yeah, I had to check to confirm it. I keep going back to the DECLARATIONS var in handler.py over and over. Makes me very glad it is there. | 15:07 |
edleafe | So would usages be a summary of allocations by resource class? | 15:07 |
jaypipes | correct. | 15:07 |
jaypipes | as in "a usage report". | 15:08 |
edleafe | ok, that's what I thought | 15:08 |
*** garyk has joined #openstack-nova | 15:08 | |
*** vks1 has joined #openstack-nova | 15:08 | |
*** ducttape_ has joined #openstack-nova | 15:08 | |
mriedem | ala https://developer.openstack.org/api-ref/compute/?expanded=show-rate-and-absolute-limits-detail#show-rate-and-absolute-limits | 15:09 |
mriedem | "maxTotalCores": 20, | 15:09 |
mriedem | "totalCoresUsed": 0, | 15:09 |
*** garyk has quit IRC | 15:09 | |
mriedem | yay quotas in placement | 15:09 |
melwitt | each compute node is a RP, right? so with the /allocations query, there would be maybe hundreds of entries to add together for every quota check | 15:10 |
*** garyk has joined #openstack-nova | 15:10 | |
* cdent has a sudden thought | 15:11 | |
cdent | one sec but it may be that we have to do it jay's way | 15:11 |
* cdent looks at code | 15:11 | |
cdent | nope, never mind | 15:12 |
jaypipes | melwitt: correct. | 15:12 |
cdent | I was worried that if a user had multiple instances on the same rp the output would be weird | 15:12 |
cdent | but yeah, could be hundreds | 15:12 |
jaypipes | melwitt: it's not that it cannot be done, just that it could be done more simply by the placement API. but I'm cool doing that in a followup patch. | 15:12 |
melwitt | yeah, just thinking about it more | 15:13 |
*** mdrabe has quit IRC | 15:13 | |
mriedem | how would it be hundreds when the default max limit on instances is 20? | 15:14 |
*** psachin_ has quit IRC | 15:14 | |
melwitt | hundreds of compute nodes | 15:14 |
melwitt | is common | 15:14 |
mriedem | yeah but aren't you asking for allocations for a given project? | 15:14 |
melwitt | and allocations are per resource provider | 15:14 |
cdent | mriedem: you make an excellent point | 15:14 |
mriedem | but my project by default can only allocate up to 20 instances right? | 15:15 |
mriedem | b/c allocations.consumer_id == instance.uuid | 15:15 |
melwitt | oh, hm | 15:15 |
jaypipes | there will still be 3x that number of records in the normal case (a record per instance for VCPU, MEMORY_MB, and DISK_GB of course) | 15:15 |
jaypipes | but anyway, like I said, it's nothing more than an ease-of-use thing (doing the top-level /usages endpoint) for me. | 15:16 |
mriedem | my point is i don't like 2 ways to skin the cat | 15:16 |
jaypipes | mriedem: one is a summary view, the other is a details view, that's all this is. | 15:17 |
mriedem | if /usages is the thing that's better and keeps the math on the server side, then that seems fine | 15:17 |
*** tongli has quit IRC | 15:17 | |
mriedem | i don't want to provide 2 apis when one is needed | 15:17 |
jaypipes | mriedem: we already have the /allocations endpoint, though. which is what cdent is saying. | 15:17 |
mriedem | so let's pick the one correct api | 15:17 |
melwitt | yeah, I was kinda thinking why not just do /usages only then | 15:17 |
mriedem | has anyone looked at the turd sandwich that is the compute api lately? :) | 15:17 |
mriedem | let's not do that again | 15:18 |
jaypipes | mriedem: Matt, nobody here is recommending making placement API into a turd sandwich. | 15:18 |
mriedem | jaypipes: so rather than change /allocations to take query params, we add /usages if that's better | 15:18 |
cdent | we do not have just /allocations yet, we have /allocations/{consumer_uuid}, burt limiting the numbers of paths seems nice | 15:18 |
jaypipes | mriedem: both the detail view (/allocations) and the summary view (/usages) are useful. | 15:18 |
jaypipes | cdent: good point. | 15:18 |
mriedem | jaypipes: they are not useful if we aren't using them both | 15:19 |
mriedem | one will be technical debt | 15:19 |
mriedem | so what is the one true best api that we can provide for melwitt's quota counting client side code? | 15:19 |
mriedem | and let's do that | 15:19 |
jaypipes | mriedem: my guess is that as soon as we add a summary view, someone will say "but what allocations are underneath that summary view?" | 15:19 |
jaypipes | mriedem: but meh. | 15:19 |
*** mdrabe has joined #openstack-nova | 15:20 | |
cdent | jaypipes: exactly my point | 15:20 |
mriedem | if someone has a valid use case later, then we add a new thing | 15:20 |
melwitt | I am still not clear on what the difference between a usage object and a allocation object is either | 15:20 |
cdent | people will want the details, and we have code that creates most of those details already | 15:20 |
jaypipes | mriedem: for the quota-counting use case, I believe GET /usages?project_id=X&user_id=Y is the best fit. | 15:20 |
mriedem | melwitt: idk either | 15:20 |
mriedem | because it wasn't modeled in the spec as it came up later | 15:20 |
mriedem | jaypipes: did you think /usages already existed when you commented on that in the spec? | 15:21 |
jaypipes | melwitt: a usage object is an aggregate view (total number of VCPU used, instead of X number of allocation records each with a VCPU record) | 15:21 |
mriedem | oh i see /resource_providers/{uuid}/usages | 15:21 |
melwitt | I'm not seeing any example of one in a response in the gabbits | 15:21 |
mriedem | melwitt: it's in the allocations gabbits | 15:22 |
mriedem | for /resource_providers/{uuid}/usages | 15:22 |
melwitt | ah, thanks | 15:22 |
mriedem | and usage.yaml | 15:23 |
melwitt | in usage.yaml all the responses are empty, I think. I was just looking to get an idea what the existing usages response looks like | 15:24 |
melwitt | but it's been explained, aggregate view, and that would be ideal for the quota checks | 15:25 |
*** zenoway has quit IRC | 15:26 | |
mriedem | {'resource_provider_generation': 1, 'usages': {'VCPU': 2, 'MEMORY_MB': 2048, 'DISK_GB': 80}} | 15:26 |
mriedem | something like that i think | 15:26 |
melwitt | gotcha, thanks | 15:27 |
cdent | mriedem, melwitt line 353 in this https://review.openstack.org/#/c/370204/2/nova/tests/functional/api/openstack/placement/gabbits/fullrep.yaml | 15:27 |
cdent | and 441 | 15:27 |
melwitt | perfect, thanks | 15:28 |
*** gszasz has quit IRC | 15:30 | |
*** zenoway has joined #openstack-nova | 15:31 | |
*** iceyao has quit IRC | 15:31 | |
*** mtanino has quit IRC | 15:31 | |
*** annegentle has joined #openstack-nova | 15:33 | |
*** ducttape_ has quit IRC | 15:34 | |
*** markus_z has quit IRC | 15:35 | |
*** kaisers has quit IRC | 15:36 | |
*** zenoway has quit IRC | 15:36 | |
dansmith | successful devstack multi-cell run: https://review.openstack.org/#/c/436094 | 15:37 |
*** Sukhdev has joined #openstack-nova | 15:39 | |
*** jistr|mtg is now known as jistr | 15:40 | |
*** mlavalle has joined #openstack-nova | 15:40 | |
*** yingjun has quit IRC | 15:41 | |
*** aarefiev is now known as aarefiev_afk | 15:46 | |
*** Hosam has joined #openstack-nova | 15:51 | |
*** chyka has joined #openstack-nova | 15:53 | |
openstackgerrit | Huan Xie proposed openstack/nova master: WIP: use os-xenapi V2 https://review.openstack.org/453493 | 15:58 |
*** andreas_s has quit IRC | 15:58 | |
*** ducttape_ has joined #openstack-nova | 15:59 | |
*** lucasagomes is now known as lucas-afk | 16:00 | |
*** Apoorva has joined #openstack-nova | 16:00 | |
*** nic has joined #openstack-nova | 16:02 | |
sfinucan | mriedem: Quick question on https://review.openstack.org/#/c/433603/ | 16:03 |
sfinucan | I've the Sphinx integration for cliff done, but it'll be a while before it merges and a suitable version makes it's way into g-r | 16:04 |
sfinucan | I also need to drop a load of commands from nova-manage that are no longer necessary, but I figure we need a cycle of deprecation before doing so | 16:04 |
sfinucan | How do I track that from a spec/bp perspective? specless bp for this cycle (to deprecate commands) and a specful one for Queens to do the rewrite? | 16:05 |
*** tommylikehu_ has joined #openstack-nova | 16:05 | |
*** kaisers has joined #openstack-nova | 16:06 | |
mriedem | sfinucan: we don't need a bp to deprecate commands | 16:06 |
sfinucan | No? Sweet. I'll re-target that for Queens so | 16:07 |
*** efried has joined #openstack-nova | 16:07 | |
openstackgerrit | Chris Dent proposed openstack/nova-specs master: A spec for idempotent custom resource classe creation https://review.openstack.org/453732 | 16:09 |
cdent | mriedem, jaypipes, edleafe ^ | 16:09 |
cdent | feh, typo right at the start mehness | 16:09 |
jaypipes | cdent: cool, thanks! | 16:09 |
openstackgerrit | Chris Dent proposed openstack/nova-specs master: A spec for idempotent custom resource class creation https://review.openstack.org/453732 | 16:10 |
edleafe | cdent: added | 16:10 |
*** gszasz has joined #openstack-nova | 16:14 | |
*** ducttape_ has quit IRC | 16:15 | |
*** tommylikehu_ has quit IRC | 16:15 | |
*** ijw has joined #openstack-nova | 16:15 | |
*** ijw_ has joined #openstack-nova | 16:16 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_detach notification https://review.openstack.org/408676 | 16:16 |
*** suresh12 has quit IRC | 16:17 | |
*** armax has joined #openstack-nova | 16:17 | |
*** ijw has quit IRC | 16:19 | |
*** chyka has quit IRC | 16:20 | |
openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Add 'move-nova-cmds-to-cliff' spec https://review.openstack.org/433603 | 16:21 |
openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Create specs directory for Queens https://review.openstack.org/453737 | 16:21 |
mriedem | dane-fichter: i'm completely confused by how we don't end up with an empty tuple for registered types here https://review.openstack.org/#/c/351232/16/nova/objects/fields.py | 16:21 |
*** chyka has joined #openstack-nova | 16:22 | |
*** nmathew has joined #openstack-nova | 16:23 | |
*** hshiina has quit IRC | 16:23 | |
*** nmathew has quit IRC | 16:23 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | 16:23 |
openstackgerrit | Sujitha proposed openstack/nova master: Add sync_instances action in os-cells API https://review.openstack.org/453366 | 16:23 |
*** esberglu has quit IRC | 16:24 | |
*** Jack_Iv has quit IRC | 16:24 | |
*** esberglu has joined #openstack-nova | 16:24 | |
*** gcb has quit IRC | 16:25 | |
efried | mriedem The register method can stuff things into the class member REGISTERED_TYPES. | 16:26 |
*** ijw_ has quit IRC | 16:26 | |
efried | mriedem https://pastebin.com/aECAcEFK | 16:26 |
*** tbachman has joined #openstack-nova | 16:27 | |
*** armax has quit IRC | 16:27 | |
mriedem | is register a special method in python? | 16:28 |
efried | Not that I know of. Does it need to be? | 16:28 |
*** marst has quit IRC | 16:28 | |
efried | It's defined on the SignatureKeyType class. | 16:29 |
*** esberglu has quit IRC | 16:29 | |
*** amotoki has quit IRC | 16:29 | |
mriedem | yeah, but without calling SignatureKeyType.register, which the nova code doesn't do, how does REGISTERED_TYPES get populated? | 16:29 |
*** mnestratov has quit IRC | 16:30 | |
efried | mriedem Well, okay, I wasn't assuming we knew for sure that register never got called anywhere... | 16:30 |
*** armax has joined #openstack-nova | 16:31 | |
*** mtanino has joined #openstack-nova | 16:31 | |
*** dtp has joined #openstack-nova | 16:32 | |
openstackgerrit | Chris Dent proposed openstack/nova-specs master: A spec for idempotent custom resource class creation https://review.openstack.org/453732 | 16:32 |
*** baoli has quit IRC | 16:33 | |
*** dtp has quit IRC | 16:33 | |
mriedem | it seems to work, i just don't know how | 16:34 |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_detach notification https://review.openstack.org/408676 | 16:34 |
mriedem | efried: dane-fichter: ah import time fun https://github.com/openstack/cursive/blob/0.1.2/cursive/signature_utils.py#L172-L180 | 16:35 |
*** abalutoiu has joined #openstack-nova | 16:35 | |
efried | I was gonna say, "must get called from within cursive somewhere". But I would have been grasping at straws ;-) | 16:35 |
*** kaisers has quit IRC | 16:36 | |
*** kaisers has joined #openstack-nova | 16:36 | |
*** kaisers has quit IRC | 16:36 | |
*** kaisers has joined #openstack-nova | 16:37 | |
*** dtp has joined #openstack-nova | 16:37 | |
*** iceyao has joined #openstack-nova | 16:39 | |
*** dimtruck is now known as zz_dimtruck | 16:40 | |
*** zz_dimtruck is now known as dimtruck | 16:41 | |
*** derekh has quit IRC | 16:41 | |
*** kaisers has quit IRC | 16:41 | |
openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in security_group_default_rules.py https://review.openstack.org/453748 | 16:42 |
*** esberglu has joined #openstack-nova | 16:43 | |
*** tesseract has quit IRC | 16:43 | |
*** baoli has joined #openstack-nova | 16:43 | |
*** iceyao has quit IRC | 16:43 | |
*** armax has quit IRC | 16:43 | |
*** vks1 has quit IRC | 16:45 | |
*** marst has joined #openstack-nova | 16:47 | |
johnthetubaguy | aunnam sneti this is the keystone spec I was raising about the additional roles spec: https://review.openstack.org/453739 | 16:49 |
*** hieulq_ has joined #openstack-nova | 16:49 | |
aunnam | johnthetubaguy thanks, will review it | 16:50 |
sneti | johnthetubaguy, will check that.. | 16:50 |
*** Swami has quit IRC | 16:54 | |
openstackgerrit | Sujitha proposed openstack/nova master: Add sync_instances action in os-cells API https://review.openstack.org/453366 | 16:56 |
*** Sukhdev has quit IRC | 16:58 | |
*** _ducttape_ has joined #openstack-nova | 16:59 | |
*** Sukhdev has joined #openstack-nova | 16:59 | |
*** chyka has quit IRC | 17:00 | |
*** chyka has joined #openstack-nova | 17:01 | |
*** crushil has quit IRC | 17:01 | |
*** gus has quit IRC | 17:04 | |
*** jamielennox has quit IRC | 17:04 | |
*** baoli_ has joined #openstack-nova | 17:05 | |
*** chyka has quit IRC | 17:05 | |
*** gus has joined #openstack-nova | 17:05 | |
openstackgerrit | Bob Ball proposed openstack/nova master: Use physical utilisation for cached images https://review.openstack.org/353862 | 17:06 |
*** baoli_ has quit IRC | 17:07 | |
*** baoli has quit IRC | 17:07 | |
*** efoley_ has quit IRC | 17:07 | |
mriedem | melwitt: so just for closure, were you going to amend the placement usages spec to use a new /usages handler rather than /allocations? | 17:07 |
*** jamielennox has joined #openstack-nova | 17:07 | |
melwitt | mriedem: yes. will do that today | 17:07 |
*** chyka has joined #openstack-nova | 17:07 | |
mriedem | ok cool | 17:07 |
mriedem | i got confused by all of the options | 17:08 |
*** chyka has quit IRC | 17:08 | |
mriedem | plus, someone jangled some keys in front of me | 17:08 |
*** jaosorior is now known as jaosorior_away | 17:08 | |
*** chyka has joined #openstack-nova | 17:08 | |
mriedem | sdague: well with https://review.openstack.org/#/c/447728/ merged i think we're ready for novaclient 8.0.0 which drops all of the proxy crap | 17:09 |
melwitt | heh, yeah. I've had a lot of meetings this morning so I didn't get to it yet | 17:09 |
*** baoli has joined #openstack-nova | 17:13 | |
*** suresh12 has joined #openstack-nova | 17:17 | |
*** _ducttape_ has quit IRC | 17:18 | |
*** mvk has quit IRC | 17:22 | |
*** suresh12 has quit IRC | 17:27 | |
*** timello has joined #openstack-nova | 17:28 | |
*** hieulq__ has joined #openstack-nova | 17:30 | |
*** Hosam has quit IRC | 17:31 | |
dane-fichter | mriedem: sorry I had to step out for a meeting | 17:32 |
openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Update reno for removed network CLIs https://review.openstack.org/453800 | 17:32 |
mriedem | melwitt: dansmith: want to get ^ quick and then i can releaes? | 17:32 |
mriedem | *release | 17:32 |
mriedem | dane-fichter: np, +2 | 17:32 |
dane-fichter | mriedem: great, thanks. | 17:32 |
*** mnestratov has joined #openstack-nova | 17:33 | |
*** sambetts is now known as sambetts|afk | 17:33 | |
*** hieulq_ has quit IRC | 17:34 | |
*** Swami has joined #openstack-nova | 17:35 | |
melwitt | mriedem: +W | 17:36 |
*** Jack_Iv has joined #openstack-nova | 17:38 | |
*** armax has joined #openstack-nova | 17:39 | |
*** jpena is now known as jpena|off | 17:43 | |
*** crushil has joined #openstack-nova | 17:46 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Deprecate '--version' parameters https://review.openstack.org/453808 | 17:50 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-manage: Rename 'api_db', 'cellsv2' commands https://review.openstack.org/453809 | 17:50 |
sfinucan | ^ artom Per your comments in the 'move-nova-cmds-to-cliff' spec yesterday | 17:50 |
sfinucan | the first one is particularly disgusting :) | 17:51 |
*** suresh12 has joined #openstack-nova | 17:52 | |
*** crushil has quit IRC | 17:53 | |
*** crushil has joined #openstack-nova | 17:53 | |
*** suresh12 has quit IRC | 17:56 | |
*** hieulq__ has quit IRC | 17:56 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: make it clear that os-cells is for cells v1 https://review.openstack.org/453433 | 17:57 |
mriedem | this is a pretty simple fix for a long-standing bug https://review.openstack.org/#/c/452577/ | 17:58 |
mriedem | which i'm going to backport once it's merged | 17:58 |
dansmith | ooh | 17:59 |
*** suresh12 has joined #openstack-nova | 17:59 | |
*** mvk has joined #openstack-nova | 18:00 | |
mriedem | ooh is right | 18:00 |
*** Apoorva_ has joined #openstack-nova | 18:01 | |
*** Apoorva__ has joined #openstack-nova | 18:03 | |
*** Apoorva_ has quit IRC | 18:03 | |
*** sree has joined #openstack-nova | 18:04 | |
*** Apoorva has quit IRC | 18:04 | |
*** krtaylor has quit IRC | 18:07 | |
*** ralonsoh has quit IRC | 18:08 | |
*** sree has quit IRC | 18:08 | |
*** ducttape_ has joined #openstack-nova | 18:09 | |
*** armax has quit IRC | 18:09 | |
*** garyk has quit IRC | 18:09 | |
*** ltomasbo is now known as ltomasbo|away | 18:09 | |
*** yonglihe has quit IRC | 18:09 | |
*** READ10 has joined #openstack-nova | 18:13 | |
*** Jack_Iv_ has joined #openstack-nova | 18:16 | |
*** jaypipes has quit IRC | 18:16 | |
mriedem | ericyoung: what's your launchpad id? | 18:16 |
*** Jack_Iv has quit IRC | 18:17 | |
*** lucasxu has joined #openstack-nova | 18:18 | |
*** armax has joined #openstack-nova | 18:18 | |
mriedem | dansmith: melwitt: we also have this regression from ocata to get backported https://review.openstack.org/#/c/452359/ - i've been waiting on that one before fixing https://review.openstack.org/#/c/449334/ since they are going to conflict | 18:21 |
dansmith | guh | 18:22 |
dansmith | I cannot wait for counting quotas | 18:22 |
artom | sfinucan, so you're going to add deprecation warnings in Pike, and remove the old commands in Queens entirely? | 18:22 |
artom | I think there needs to be a period of overlap when both coexist | 18:22 |
*** ducttape_ has quit IRC | 18:22 | |
artom | For instance, does tripleo call them? | 18:23 |
artom | I think we've broken tripleo enough as is ;) | 18:23 |
artom | Let's give them a chance to adjust because we yank commands from under them | 18:23 |
artom | s/because/before | 18:23 |
mriedem | i hope tripleo isn't using deprecated nova clis | 18:24 |
*** Sukhdev has quit IRC | 18:25 | |
artom | mriedem, they're not deprecated yet :) I'm trying to think of ways to make the deprecation as painless as possible | 18:25 |
mriedem | artom: i mean novaclient | 18:25 |
mriedem | not nova-manage | 18:25 |
mriedem | https://review.openstack.org/#/c/453819/ | 18:25 |
mriedem | https://governance.openstack.org/tc/reference/tags/assert_follows-standard-deprecation.html | 18:26 |
*** vladikr has quit IRC | 18:26 | |
dansmith | our delete path(s) are just hilarious | 18:27 |
mriedem | dansmith: i know, | 18:27 |
dansmith | maybe we should get rid of delete? | 18:27 |
mriedem | dansmith: that's why i'm holding https://review.openstack.org/#/c/449334/ to the end, | 18:27 |
mriedem | because i think the fix for that bug is to re-use the _local_delete method in the compute api | 18:28 |
mriedem | but i think there will be dragons | 18:28 |
artom | soft-delete in the client is the way to go ;) | 18:28 |
mriedem | _local_delete is what cleans up ports and volumes | 18:28 |
mriedem | and was missing from the duplicate 'local delete' that was added for the cells v2 case | 18:28 |
mriedem | once these ocata regression bugs are fixed, i think we spend some time just removing the transition code in the delete path for cells v2 | 18:29 |
mriedem | but need the fixes and tests in place before we unwind that onion | 18:29 |
*** ijw has joined #openstack-nova | 18:29 | |
mriedem | artom: per the deprecation policy, we can deprecate in pike and remove in queens as long as it's been deprecated for at least 3 months | 18:30 |
mriedem | so you can't deprecate something at the very end of pike and drop it immediately in queens if it's under 3 months | 18:30 |
mriedem | the 3 months thing is for CD | 18:30 |
artom | mriedem, I can't argue with the policy | 18:30 |
artom | I just think it's safer to have a period when both coexist - so that whatever depends on them can change to the new way of doing things while the old way still works | 18:31 |
mriedem | that's the whole point of the 3 months | 18:31 |
mriedem | and full stable branch | 18:31 |
artom | Right, assuming the new way is ready and available during those 3 months ;) | 18:32 |
artom | Good point for stable though | 18:32 |
*** harlowja has quit IRC | 18:33 | |
sdague | mriedem: you have 7.0.0 instead of 8.0.0 in https://review.openstack.org/#/c/453819/1 | 18:33 |
sdague | which is why it's failing tets | 18:33 |
sdague | tests | 18:33 |
mriedem | derp | 18:33 |
mriedem | copy/paste | 18:33 |
mriedem | it's dependent on a novaclient reno change for now anyway | 18:34 |
*** ijw has quit IRC | 18:34 | |
*** kaisers has joined #openstack-nova | 18:36 | |
mriedem | dansmith: don't forget the regression test patch below it | 18:36 |
dansmith | yeah dude, it takes a while :) | 18:37 |
mriedem | sorry | 18:37 |
*** READ10 has quit IRC | 18:39 | |
ericyoung | mriedem: eric-aceshome | 18:40 |
mriedem | ok | 18:40 |
*** amoralej is now known as amoralej|off | 18:41 | |
melwitt | mriedem: is this part not a separate fix on its own? https://review.openstack.org/#/c/452351/4/nova/compute/api.py | 18:41 |
dansmith | it is and he says that in the commit message, | 18:41 |
mriedem | i could split that out if needed | 18:41 |
dansmith | but there's not much to indicate what the thinking is there | 18:41 |
melwitt | like, continuing on to the other delete code seems like a bug in itself | 18:42 |
melwitt | okay | 18:42 |
mriedem | if you undo that and run the functional test, you'll see | 18:42 |
dansmith | I think I have it worked out in my head.. we would just go on to try to delete bdms and maybe fail or something? | 18:42 |
mriedem | yes, the problem is it continues to the rest of the other delete code | 18:42 |
mriedem | yes | 18:42 |
mriedem | i can split that out if it helps | 18:42 |
dansmith | so the user would see it as a 500 or something on the racing second delete attempt? | 18:43 |
dansmith | mriedem: since these are for backports, I'm fine with either, but might be good to just add more words about what that change is | 18:43 |
mriedem | i think it results in a 404 | 18:43 |
melwitt | I was just thinking out loud | 18:43 |
mriedem | looking up the bdms | 18:43 |
mriedem | i can test here locally quick to be sure | 18:43 |
dansmith | mriedem: "500 or something" includes "404" :P | 18:43 |
mriedem | "sh is f'ed" | 18:44 |
mriedem | includes both | 18:44 |
dansmith | heh | 18:44 |
melwitt | I understand what the change is for, I was just asking if that needs to be documented as a separate bug I guess | 18:45 |
mriedem | i just lazily lumped it in in the commit message | 18:45 |
mriedem | "This also fixes another bug where we would not short-circuit | 18:45 |
mriedem | and return from _delete if we hit InstanceNotFound." | 18:45 |
dansmith | right | 18:46 |
dansmith | that's the thing I think is a little, uh, light on the detail | 18:46 |
mriedem | heh, well, spent a half day writing the recreate functional regression test, | 18:46 |
*** suresh12 has quit IRC | 18:46 | |
mriedem | i was too tired to care about that detail :P | 18:46 |
mriedem | psh | 18:46 |
melwitt | yeah, I see it now. was just a surprise to see a major "let's stop continuing on to the other delete code we shouldn't be running" lumped in with "add regression test" :P | 18:47 |
dansmith | I haven't even gotten to that part yet :P | 18:47 |
mriedem | let me see what it does when i remove the return and rerun the func test | 18:47 |
mriedem | oh, well, | 18:47 |
mriedem | so the func test fails with a 500 because consoleauth isn't running, | 18:47 |
mriedem | because it tries going further down that path | 18:47 |
melwitt | it's fine, I was just confused | 18:48 |
*** suresh12 has joined #openstack-nova | 18:48 | |
mriedem | it's this | 18:49 |
mriedem | self.consoleauth_rpcapi.delete_tokens_for_instance( | 18:49 |
mriedem | context, instance.uuid) | 18:49 |
mriedem | let me start consoleauth and see | 18:49 |
mriedem | ok, so if i start consoleauth, then the functional test is ok if i remove the return statement to short-circuit | 18:50 |
mriedem | so it's up to you guys i guess | 18:50 |
mriedem | i think if we hit InstanceNotFound, it makes sense to short-circuit and return, but i can also remove it | 18:51 |
*** hoonetorg has quit IRC | 18:51 | |
*** krtaylor has joined #openstack-nova | 18:52 | |
melwitt | mriedem: I think the return definitely needs to be there. I was just thinking about if we needed that broken out and documented as a separate bug fix. but it looks like that bug is sort of harmless, like extra stuff that's effectively a no-op | 18:52 |
melwitt | and since that's the case, I think you can just leave it the way it is | 18:52 |
*** iceyao has joined #openstack-nova | 18:52 | |
dansmith | I think the commit message should get more words | 18:52 |
dansmith | but I'm okay with them being together if you want 'em that way | 18:53 |
dansmith | the other thing I guess, | 18:53 |
dansmith | is this covers up the fact that you're not unit testing this change | 18:53 |
dansmith | and if you split them, it'd be more obvious | 18:53 |
dansmith | maybe that's a reason to not split, I dunno | 18:53 |
dansmith | but more words at a minimum I think | 18:53 |
mriedem | the return? | 18:53 |
mriedem | that's https://review.openstack.org/#/c/452351/4/nova/tests/unit/compute/test_compute_api.py | 18:53 |
dansmith | because in six months we'll not remember this | 18:53 |
mriedem | "# This just lets us exit the test early." | 18:53 |
dansmith | oh sorry, I thought the functional test was the last file | 18:54 |
dansmith | I seee now | 18:54 |
mriedem | i had to put that in that compute api unit test earlier because of the lack of the short circuit | 18:54 |
mriedem | what i could do is rather than remove that, assert bdms isn't called | 18:54 |
*** crushil has quit IRC | 18:54 | |
mriedem | to enforce the return is there | 18:54 |
mriedem | you like that better? | 18:54 |
dansmith | I just wasn't seeing that you had any unit test change to account for it | 18:54 |
dansmith | so I don't care | 18:54 |
dansmith | well | 18:55 |
*** crushil has joined #openstack-nova | 18:55 | |
dansmith | yeah, I guess asserting it wasn't called is the best actually | 18:55 |
mriedem | hows about, | 18:55 |
mriedem | i split it out, give it it's own special 3 paragraph commit message, and change the removal in the unit test to assert_not_called() | 18:55 |
mriedem | i think we'll all be happy then | 18:55 |
dansmith | heh, yeah | 18:55 |
mriedem | easy peasy | 18:55 |
melwitt | but ... I'm already happy ... just look at me --> :) | 18:57 |
mriedem | o/. | 18:57 |
mriedem | o/~ | 18:57 |
*** iceyao has quit IRC | 18:57 | |
mriedem | btw, laura has some new amazing deodorant she wants everyone to try, now that i'm thinking about o/~ | 18:57 |
mriedem | she's constantly shoving her bare pits in my face | 18:57 |
mriedem | telling me to sniff | 18:58 |
melwitt | lol | 18:58 |
*** Sukhdev has joined #openstack-nova | 18:58 | |
*** sree has joined #openstack-nova | 19:01 | |
*** vladikr has joined #openstack-nova | 19:01 | |
*** Apoorva__ has quit IRC | 19:02 | |
*** Apoorva has joined #openstack-nova | 19:03 | |
*** sree has quit IRC | 19:05 | |
*** lyan has quit IRC | 19:07 | |
*** armax has quit IRC | 19:08 | |
*** abalutoiu has quit IRC | 19:12 | |
*** ducttape_ has joined #openstack-nova | 19:12 | |
*** Jack_Iv_ has quit IRC | 19:17 | |
mriedem | melwitt: dansmith: here it comes | 19:19 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Commit usage decrement after destroying instance https://review.openstack.org/452359 | 19:19 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add regression test for quota decrement bug 1678326 https://review.openstack.org/452351 | 19:19 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Short-circuit local delete path for cells v2 and InstanceNotFound https://review.openstack.org/453840 | 19:19 |
openstack | bug 1678326 in OpenStack Compute (nova) ocata "Quota is decremented during instance delete in cell0 even if the instance destroy fails" [High,In progress] https://launchpad.net/bugs/1678326 - Assigned to Matt Riedemann (mriedem) | 19:19 |
*** bswartz has quit IRC | 19:22 | |
*** rfolco has quit IRC | 19:24 | |
*** voelzmo has joined #openstack-nova | 19:25 | |
*** gszasz has quit IRC | 19:28 | |
*** bmace has joined #openstack-nova | 19:28 | |
*** mdrabe_ has joined #openstack-nova | 19:30 | |
*** mdrabe has quit IRC | 19:31 | |
*** ijw has joined #openstack-nova | 19:31 | |
*** Jack_Iv has joined #openstack-nova | 19:32 | |
*** Jack_Iv has quit IRC | 19:34 | |
*** Jack_Iv has joined #openstack-nova | 19:35 | |
*** blaisebool has joined #openstack-nova | 19:35 | |
*** ijw has quit IRC | 19:36 | |
*** Jack_Iv has quit IRC | 19:36 | |
*** Jack_Iv has joined #openstack-nova | 19:36 | |
*** Jack_Iv has quit IRC | 19:39 | |
*** smatzek has quit IRC | 19:40 | |
mriedem | dansmith: melwitt: thanks for the reviews | 19:42 |
melwitt | mriedem: thanks for fixing local delete. again | 19:42 |
*** ducttape_ has quit IRC | 19:43 | |
mriedem | well, still got this other one to fix now | 19:43 |
mriedem | will start working on that | 19:43 |
mriedem | i suppose we have to get the bdms from the cell db that the instance is in | 19:43 |
mriedem | the callback method might be a pita | 19:45 |
*** Jack_Iv has joined #openstack-nova | 19:45 | |
*** karimb has quit IRC | 19:47 | |
*** oanson has quit IRC | 19:47 | |
*** karimb has joined #openstack-nova | 19:48 | |
mriedem | eww this gets into the whole soft delete mess | 19:52 |
mriedem | but i think we already figured out that you can't soft delete an instance in cell0 | 19:52 |
mriedem | remember getting caught up in this on the notification patch | 19:52 |
dansmith | correct | 19:52 |
dansmith | because there's no way to undelete it | 19:53 |
*** voelzmo has quit IRC | 19:53 | |
*** irenab has quit IRC | 19:53 | |
mriedem | in the 'normal' flow we set the task_state to 'deleting' before cleaning up bdms and all that | 19:53 |
*** liverpooler has quit IRC | 19:54 | |
mriedem | w/o that we just go from ERROR/None to DELETED/None to destroyed | 19:54 |
*** awaugama has quit IRC | 19:55 | |
*** oanson has joined #openstack-nova | 19:55 | |
*** karimb has quit IRC | 19:56 | |
*** irenab has joined #openstack-nova | 19:56 | |
mriedem | god also https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1913-L1930 | 19:57 |
*** harlowja has joined #openstack-nova | 19:58 | |
mriedem | https://review.openstack.org/#/c/61462/ | 19:58 |
mriedem | so because someone thought we should deleted the shelved snapshot image when the instance is deleted, we did | 19:59 |
*** pcaruana has quit IRC | 20:01 | |
*** timello has quit IRC | 20:01 | |
*** marst_ has joined #openstack-nova | 20:03 | |
*** marst has quit IRC | 20:03 | |
mriedem | this might be easier than i think for shelved offloaded. | 20:05 |
*** kfarr has quit IRC | 20:06 | |
*** smatzek has joined #openstack-nova | 20:07 | |
mriedem | because if the instance is shelved offloaded, it's in a cell | 20:10 |
*** annegentle has quit IRC | 20:10 | |
mriedem | so the if not instance.host check at the very beginning is just not considering that case | 20:10 |
mriedem | and assumes the instance never got to a cell | 20:10 |
dansmith | if it's offloaded it's in a cell yeah | 20:12 |
mriedem | this is literally the fix: | 20:12 |
mriedem | if not (instance.host or | 20:12 |
mriedem | instance.vm_state == vm_states.SHELVED_OFFLOADED): | 20:12 |
*** jamielennox is now known as jamielennox|away | 20:12 | |
melwitt | yeah, later code does "if not instance.host and not shelved_offloaded:" | 20:13 |
openstackgerrit | Eric Fried proposed openstack/nova master: PowerVM Driver: SSP emphemeral disk support https://review.openstack.org/443189 | 20:13 |
melwitt | and vm_states.ERROR should be in there too, which is what the patch I've had up for ages does | 20:13 |
mriedem | so i was trying to think of what we miss now in the first local delete block for cells v2 where instance.host is None | 20:14 |
mriedem | and it's not shelved offloade | 20:14 |
mriedem | i think that's just the cell0 case | 20:14 |
*** lyan has joined #openstack-nova | 20:14 | |
mriedem | where i don't think we need to worry about cleaning up bdms, because conductor doesn't create the bdms in cell0 | 20:15 |
mriedem | _bury_in_cell0 creates the instance in cell0, then deletes the build request | 20:15 |
*** liusheng has quit IRC | 20:16 | |
*** liusheng has joined #openstack-nova | 20:17 | |
*** dane-fichter has quit IRC | 20:18 | |
*** crushil has quit IRC | 20:20 | |
*** satyar has quit IRC | 20:20 | |
*** armax has joined #openstack-nova | 20:20 | |
*** Sukhdev has quit IRC | 20:21 | |
mriedem | crap we might have this same issue in newton https://github.com/openstack/nova/blob/stable/newton/nova/compute/api.py#L1682 | 20:23 |
*** Jack_Iv has quit IRC | 20:29 | |
openstackgerrit | Scott Moser proposed openstack/nova master: Limit exposure of network device types to the guest. https://review.openstack.org/400883 | 20:31 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Regression test for local delete with an attached volume https://review.openstack.org/449334 | 20:31 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Set size/status during image create with FakeImageService https://review.openstack.org/449702 | 20:31 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Perform old-style local delete for shelved offloaded instances https://review.openstack.org/453859 | 20:31 |
mriedem | thar she blar | 20:31 |
dtp | mriedem how do you guys do rollbacks w/o downgrade on the migrations? | 20:32 |
mriedem | dtp: we dont | 20:33 |
mriedem | roll forward to the maxxx | 20:33 |
mriedem | i usually do the dew while rolling forward | 20:33 |
dtp | nova-manage db sync supports downgrade | 20:34 |
mriedem | it might say it does | 20:34 |
mriedem | which would be a bug | 20:34 |
dtp | i think it actually does....or else my upgrade is already idempotent | 20:34 |
dtp | because 358 > 359 > 358 > 359 works | 20:35 |
mriedem | well it works for your migration b/c you implemented downgrade | 20:35 |
mriedem | as i said in the review, i'm surprised we didn't have a test that would fail saying we don't support downgrade | 20:36 |
dtp | ok ok | 20:36 |
*** mdrabe__ has joined #openstack-nova | 20:36 | |
dtp | thanks for the review, btw | 20:36 |
mriedem | np | 20:37 |
*** mdrabe_ has quit IRC | 20:37 | |
mriedem | dtp: so i guess we weren't idempotent when adding the compute_nodes.uuid column either https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/migrate_repo/versions/314_add_resource_provider_tables.py#L78 | 20:37 |
mriedem | but we should be, e.g. https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/migrate_repo/versions/358_bdm_attachment_id.py#L27 | 20:38 |
dtp | makes sense | 20:39 |
mriedem | dtp: btw, when you said nova-manage db sync supports downgrade, you mean by passing in the --version right? | 20:40 |
*** crushil has joined #openstack-nova | 20:40 | |
dtp | yep | 20:40 |
mriedem | yeah i think if you tried going further, it'd explode | 20:40 |
*** cdent has quit IRC | 20:41 | |
*** annegentle has joined #openstack-nova | 20:41 | |
dtp | surely | 20:42 |
mriedem | huh clearly test_no_migrations_have_downgrade failed it's job | 20:43 |
edleafe | OK, what am I doing wrong? I'm making a commit to the os-traits library. I have git-review installed, and a .gitreview file in the directory, just as in my nova project. But when I commit, I'm not getting a Change-Id added to the commit message. | 20:43 |
*** ducttape_ has joined #openstack-nova | 20:43 | |
mriedem | edleafe: git review -s | 20:43 |
mriedem | in the repo | 20:44 |
mriedem | you probably don't have the gerrit remote | 20:44 |
edleafe | mriedem: that worked, but I did have the remote | 20:44 |
mriedem | maybe git review -s installs a hook or something, idk | 20:44 |
edleafe | dunno, but thanks for setting me straight | 20:45 |
*** rcernin has quit IRC | 20:45 | |
melwitt | edleafe: you have to do git review -s anytime you clone a new repo, fwiw | 20:46 |
*** mdrabe__ is now known as mdrabe | 20:47 | |
edleafe | melwitt: yeah, I thought I had, but I guess not... | 20:47 |
edleafe | thanks | 20:48 |
*** ducttape_ has quit IRC | 20:48 | |
*** thorst has quit IRC | 20:50 | |
*** cdent has joined #openstack-nova | 20:50 | |
*** thorst has joined #openstack-nova | 20:51 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/os-vif master: Updated from global requirements https://review.openstack.org/451049 | 20:51 |
*** hoonetorg has joined #openstack-nova | 20:53 | |
*** thorst has quit IRC | 20:55 | |
*** kaisers has quit IRC | 20:56 | |
cdent | edleafe: that os-traits stuff you did is very satisfying | 20:57 |
*** cleong has quit IRC | 21:00 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix test_no_migrations_have_downgrade https://review.openstack.org/453872 | 21:00 |
*** mariusv has quit IRC | 21:01 | |
*** mriedem has left #openstack-nova | 21:01 | |
*** mriedem has joined #openstack-nova | 21:01 | |
*** mariusv has joined #openstack-nova | 21:03 | |
*** Apoorva_ has joined #openstack-nova | 21:05 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/453896 | 21:07 |
*** Apoorva has quit IRC | 21:08 | |
*** karimb has joined #openstack-nova | 21:08 | |
edleafe | cdent: a lot easier on the eyes, no? | 21:08 |
cdent | quite | 21:09 |
*** dimtruck is now known as zz_dimtruck | 21:09 | |
*** priteau has quit IRC | 21:10 | |
*** smatzek has quit IRC | 21:13 | |
*** sree has joined #openstack-nova | 21:17 | |
*** eliqiao has quit IRC | 21:18 | |
*** thorst has joined #openstack-nova | 21:18 | |
*** eliqiao has joined #openstack-nova | 21:19 | |
openstackgerrit | Dan Peschman proposed openstack/nova master: Add uuid to Service model https://review.openstack.org/453025 | 21:20 |
mriedem | oh dansmith that reminds me, did the functional tests for https://review.openstack.org/#/c/438025/ work? | 21:21 |
*** sree has quit IRC | 21:21 | |
mriedem | oh https://review.openstack.org/#/c/452006/2 | 21:21 |
*** gouthamr has quit IRC | 21:21 | |
*** avolkov has quit IRC | 21:21 | |
dansmith | yeah I think I got it working | 21:21 |
* dansmith runs | 21:21 | |
mriedem | you should take that off the end of the series | 21:22 |
mriedem | since it's testing something that's already merged | 21:22 |
*** thorst has quit IRC | 21:22 | |
mriedem | we can just work that in parallel | 21:22 |
*** rfolco has joined #openstack-nova | 21:29 | |
*** eharney has quit IRC | 21:31 | |
*** zz_dimtruck is now known as dimtruck | 21:37 | |
*** trinaths1 has joined #openstack-nova | 21:38 | |
*** trinaths2 has joined #openstack-nova | 21:38 | |
*** cdent has quit IRC | 21:39 | |
*** trinaths1 has quit IRC | 21:42 | |
*** jamesdenton has quit IRC | 21:45 | |
*** lyan has quit IRC | 21:48 | |
*** artom has quit IRC | 21:49 | |
*** thorst has joined #openstack-nova | 21:53 | |
*** catintheroof has quit IRC | 21:54 | |
*** Sukhdev has joined #openstack-nova | 21:56 | |
dansmith | mriedem: sure | 21:56 |
*** Swami has quit IRC | 21:56 | |
mriedem | dansmith: there are some unnecessary changes in https://review.openstack.org/#/c/439891/ i think, so i'll let you check the comments, but otherwise i'm +2 | 21:58 |
dansmith | mriedem: yeah they're all rebase noise, I'll fix | 22:02 |
mriedem | cool | 22:02 |
*** Apoorva_ has quit IRC | 22:04 | |
*** Apoorva has joined #openstack-nova | 22:04 | |
*** harlowja has quit IRC | 22:05 | |
mriedem | where does the compute call the scheduler on init if not _update_scheduler_instance_info? | 22:06 |
dansmith | there, | 22:06 |
*** efried has quit IRC | 22:07 | |
dansmith | which is a synchronous call all the way into scheduler, | 22:07 |
dansmith | which will look up hostmapping | 22:07 |
dansmith | and fail | 22:07 |
dansmith | if we haven't created it yet | 22:07 |
mriedem | that's an rcp cast | 22:07 |
mriedem | *rpc | 22:07 |
dansmith | not if we have CastAsCall | 22:07 |
mriedem | oh | 22:08 |
mriedem | heh | 22:08 |
dansmith | which is the devil | 22:08 |
*** xyang1 has quit IRC | 22:09 | |
*** mlavalle has quit IRC | 22:09 | |
mriedem | alright that's fun | 22:10 |
mriedem | left a comment to remind myself | 22:10 |
mriedem | also, i see that none of that compute -> scheduler stuff is necessary if you're not using the filter scheduler, but meh | 22:11 |
*** thorst has quit IRC | 22:12 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/453896 | 22:12 |
*** jamielennox|away is now known as jamielennox | 22:13 | |
*** ducttape_ has joined #openstack-nova | 22:14 | |
*** abalutoiu has joined #openstack-nova | 22:14 | |
*** lucasxu has quit IRC | 22:15 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Mention track_instance_changes in scheduler.driver conf help https://review.openstack.org/453916 | 22:16 |
*** gongysh has joined #openstack-nova | 22:19 | |
*** ducttape_ has quit IRC | 22:21 | |
*** edmondsw has quit IRC | 22:22 | |
*** mdrabe has quit IRC | 22:23 | |
*** edmondsw has joined #openstack-nova | 22:24 | |
*** edmondsw has quit IRC | 22:29 | |
*** Jack_Iv has joined #openstack-nova | 22:29 | |
*** baoli has quit IRC | 22:31 | |
*** Jack_Iv has quit IRC | 22:33 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Add functional test for two-cell scheduler behaviors https://review.openstack.org/452006 | 22:35 |
*** kaisers has joined #openstack-nova | 22:36 | |
*** annegentle has quit IRC | 22:36 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Teach HostAPI about cells https://review.openstack.org/442162 | 22:38 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | 22:38 |
openstackgerrit | Dan Smith proposed openstack/nova master: Sort CellMappingList.get_all() for safety https://review.openstack.org/443174 | 22:38 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add get_by_instance_uuids() to InstanceMappingList https://review.openstack.org/443292 | 22:38 |
openstackgerrit | Dan Smith proposed openstack/nova master: Clean up ClientRouter debt https://review.openstack.org/444487 | 22:38 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make server_groups determine deleted-ness from InstanceMappingList https://review.openstack.org/443293 | 22:38 |
openstackgerrit | Dan Smith proposed openstack/nova master: Remove Mitaka-era service version check https://review.openstack.org/442861 | 22:38 |
openstackgerrit | Dan Smith proposed openstack/nova master: Add workaround to disable group policy check upcall https://review.openstack.org/442736 | 22:38 |
*** Guest63262 has quit IRC | 22:41 | |
*** kaisers has quit IRC | 22:41 | |
*** krtaylor has quit IRC | 22:45 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't send instance updates from compute if not using filter scheduler https://review.openstack.org/453916 | 22:47 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't send instance updates from compute if not using filter scheduler https://review.openstack.org/453916 | 22:49 |
*** Guest63262 has joined #openstack-nova | 22:50 | |
*** felipemonteiro has quit IRC | 22:50 | |
*** marst_ has quit IRC | 22:52 | |
*** marst_ has joined #openstack-nova | 22:52 | |
*** marst_ has quit IRC | 22:56 | |
*** dillaman has quit IRC | 22:58 | |
*** thorst has joined #openstack-nova | 22:59 | |
*** ijw has joined #openstack-nova | 23:01 | |
*** ijw has quit IRC | 23:02 | |
*** ijw has joined #openstack-nova | 23:02 | |
*** sree has joined #openstack-nova | 23:03 | |
*** thorst has quit IRC | 23:04 | |
openstackgerrit | Octave Orgeron proposed openstack/nova master: Enables MySQL Cluster Support for Nova https://review.openstack.org/446643 | 23:05 |
*** Sukhdev has quit IRC | 23:06 | |
*** jamesdenton has joined #openstack-nova | 23:08 | |
*** sree has quit IRC | 23:08 | |
*** jamesdenton has quit IRC | 23:09 | |
*** dimtruck is now known as zz_dimtruck | 23:10 | |
*** priteau has joined #openstack-nova | 23:10 | |
*** dillaman has joined #openstack-nova | 23:11 | |
*** jamesdenton has joined #openstack-nova | 23:11 | |
*** priteau has quit IRC | 23:15 | |
*** gongysh has quit IRC | 23:20 | |
*** nic has quit IRC | 23:20 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup wording on compute service version checks in API https://review.openstack.org/453923 | 23:22 |
*** harlowja has joined #openstack-nova | 23:25 | |
*** ducttape_ has joined #openstack-nova | 23:28 | |
*** chyka has quit IRC | 23:30 | |
*** chyka has joined #openstack-nova | 23:30 | |
*** baoli has joined #openstack-nova | 23:32 | |
*** baoli has quit IRC | 23:33 | |
*** baoli has joined #openstack-nova | 23:34 | |
*** hshiina has joined #openstack-nova | 23:34 | |
*** thorst has joined #openstack-nova | 23:34 | |
*** chyka has quit IRC | 23:35 | |
*** Sukhdev has joined #openstack-nova | 23:35 | |
*** jamesdenton has quit IRC | 23:37 | |
*** bswartz has joined #openstack-nova | 23:39 | |
*** ducttape_ has quit IRC | 23:39 | |
*** ducttape_ has joined #openstack-nova | 23:40 | |
*** jamielennox is now known as jamielennox|away | 23:42 | |
openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-exists notification https://review.openstack.org/403660 | 23:43 |
*** zz_dimtruck is now known as dimtruck | 23:44 | |
*** jamielennox|away is now known as jamielennox | 23:45 | |
*** thorst has quit IRC | 23:49 | |
*** artom has joined #openstack-nova | 23:51 | |
*** krtaylor has joined #openstack-nova | 23:51 | |
*** ducttape_ has quit IRC | 23:59 | |
*** yingjun has joined #openstack-nova | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!