openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.resize_confirm notification https://review.openstack.org/390654 | 00:01 |
---|---|---|
*** marst_ has quit IRC | 00:02 | |
*** tbachman has quit IRC | 00:05 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | 00:05 |
*** salv-orlando has quit IRC | 00:09 | |
openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies to availability_zone https://review.openstack.org/440836 | 00:11 |
*** karimb has joined #openstack-nova | 00:14 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237 | 00:15 |
openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/407237 | 00:15 |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova master: Allow live migrate paused instance when post copy is enabled https://review.openstack.org/444517 | 00:16 |
*** mnestratov has quit IRC | 00:18 | |
*** armax has quit IRC | 00:20 | |
*** marst has joined #openstack-nova | 00:20 | |
*** dtp has quit IRC | 00:21 | |
*** karimb has quit IRC | 00:22 | |
*** penick has quit IRC | 00:24 | |
openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to volumes_attachments.py https://review.openstack.org/445252 | 00:26 |
openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.resize_revert notification https://review.openstack.org/396811 | 00:29 |
*** browne has quit IRC | 00:37 | |
*** liangy has joined #openstack-nova | 00:38 | |
*** gcb has quit IRC | 00:42 | |
*** Shunli has joined #openstack-nova | 00:45 | |
*** rdo_ has quit IRC | 00:49 | |
*** liangy has quit IRC | 00:49 | |
*** tovin07_ has joined #openstack-nova | 00:51 | |
*** liangy has joined #openstack-nova | 00:53 | |
*** esberglu has joined #openstack-nova | 00:54 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/443357 | 00:54 |
*** edmondsw has joined #openstack-nova | 00:56 | |
*** phuongnh has joined #openstack-nova | 00:56 | |
*** sdague has quit IRC | 00:57 | |
*** rdo has joined #openstack-nova | 00:57 | |
*** jamielennox is now known as jamielennox|away | 00:58 | |
*** esberglu has quit IRC | 00:58 | |
*** tblakes has quit IRC | 01:00 | |
openstackgerrit | STEW TY proposed openstack/nova master: Transform instance.rescue notifications https://review.openstack.org/385644 | 01:01 |
*** ssurana has joined #openstack-nova | 01:01 | |
*** iceyao has joined #openstack-nova | 01:01 | |
*** ssurana has quit IRC | 01:05 | |
*** yingjun has joined #openstack-nova | 01:09 | |
*** liangy has quit IRC | 01:09 | |
*** kevinz has joined #openstack-nova | 01:11 | |
*** kevinz has quit IRC | 01:11 | |
*** jamielennox|away is now known as jamielennox | 01:12 | |
*** kevinz has joined #openstack-nova | 01:17 | |
*** tuanluong has joined #openstack-nova | 01:17 | |
*** browne has joined #openstack-nova | 01:18 | |
*** browne has quit IRC | 01:18 | |
*** Apoorva has quit IRC | 01:19 | |
*** Shunli has quit IRC | 01:19 | |
*** tblakes has joined #openstack-nova | 01:19 | |
*** raunak has quit IRC | 01:19 | |
*** tbachman has joined #openstack-nova | 01:20 | |
*** Shunli has joined #openstack-nova | 01:20 | |
*** Sukhdev has quit IRC | 01:26 | |
*** nic has quit IRC | 01:28 | |
*** zhurong has joined #openstack-nova | 01:32 | |
*** yingjun has quit IRC | 01:33 | |
*** namnh has joined #openstack-nova | 01:37 | |
*** tovin07 has joined #openstack-nova | 01:42 | |
*** yamamoto has joined #openstack-nova | 01:43 | |
*** takashin_ has joined #openstack-nova | 01:50 | |
*** takashin_ has quit IRC | 01:50 | |
*** takashin has quit IRC | 01:52 | |
*** zz_dimtruck is now known as dimtruck | 01:57 | |
*** hongbin has joined #openstack-nova | 02:00 | |
*** yingjun has joined #openstack-nova | 02:00 | |
*** takashin has joined #openstack-nova | 02:04 | |
*** wangqun has joined #openstack-nova | 02:09 | |
*** esberglu has joined #openstack-nova | 02:09 | |
*** unicell has quit IRC | 02:11 | |
*** tanee_away is now known as tanee | 02:13 | |
*** esberglu has quit IRC | 02:14 | |
*** raunak has joined #openstack-nova | 02:16 | |
*** fragatina has quit IRC | 02:19 | |
*** fragatina has joined #openstack-nova | 02:20 | |
*** fragatina has quit IRC | 02:21 | |
*** fragatina has joined #openstack-nova | 02:22 | |
*** raunak has quit IRC | 02:22 | |
*** fragatina has quit IRC | 02:26 | |
*** fragatina has joined #openstack-nova | 02:29 | |
*** raunak has joined #openstack-nova | 02:30 | |
*** zzzeek has quit IRC | 02:30 | |
*** zzzeek has joined #openstack-nova | 02:31 | |
*** fragatina has quit IRC | 02:33 | |
*** gouthamr has quit IRC | 02:33 | |
*** guchihiro has joined #openstack-nova | 02:37 | |
*** gongysh has joined #openstack-nova | 02:40 | |
*** yamahata has quit IRC | 02:42 | |
*** baoli has joined #openstack-nova | 02:43 | |
*** hongbin has quit IRC | 02:46 | |
*** hongbin has joined #openstack-nova | 02:48 | |
*** liverpooler has quit IRC | 02:48 | |
*** masber has quit IRC | 02:51 | |
*** masber has joined #openstack-nova | 02:51 | |
*** sudipto has joined #openstack-nova | 02:54 | |
*** sudipto has quit IRC | 02:56 | |
*** raunak has quit IRC | 02:58 | |
*** dnalezyty has quit IRC | 02:59 | |
*** namnh_ has joined #openstack-nova | 02:59 | |
*** raunak has joined #openstack-nova | 03:00 | |
*** namnh has quit IRC | 03:03 | |
*** crushil has joined #openstack-nova | 03:06 | |
*** namnh_ has quit IRC | 03:07 | |
*** crushil has quit IRC | 03:08 | |
*** crushil has joined #openstack-nova | 03:08 | |
*** kaisers_ has joined #openstack-nova | 03:09 | |
*** salv-orlando has joined #openstack-nova | 03:10 | |
*** priteau has joined #openstack-nova | 03:11 | |
*** guchihiro has quit IRC | 03:12 | |
*** nicolasbock has quit IRC | 03:13 | |
*** salv-orlando has quit IRC | 03:15 | |
*** priteau has quit IRC | 03:15 | |
*** takashin has quit IRC | 03:16 | |
*** takashin has joined #openstack-nova | 03:16 | |
*** youzp has joined #openstack-nova | 03:18 | |
*** esberglu has joined #openstack-nova | 03:20 | |
*** tblakes has quit IRC | 03:25 | |
*** esberglu has quit IRC | 03:25 | |
*** unicell has joined #openstack-nova | 03:27 | |
*** mdnadeem has joined #openstack-nova | 03:32 | |
*** tbachman has quit IRC | 03:39 | |
*** namnh has joined #openstack-nova | 03:40 | |
*** tbachman has joined #openstack-nova | 03:53 | |
*** nmathew has joined #openstack-nova | 03:53 | |
*** Dinesh_Bhor has joined #openstack-nova | 03:55 | |
*** zsli_ has joined #openstack-nova | 03:56 | |
*** Sukhdev has joined #openstack-nova | 03:57 | |
*** edmondsw has quit IRC | 03:58 | |
*** fragatina has joined #openstack-nova | 03:58 | |
*** Shunli has quit IRC | 03:59 | |
*** hongbin has quit IRC | 03:59 | |
*** sree has joined #openstack-nova | 04:01 | |
*** sree has quit IRC | 04:01 | |
*** sree has joined #openstack-nova | 04:02 | |
*** fragatina has quit IRC | 04:03 | |
*** baoli has quit IRC | 04:04 | |
*** fragatina has joined #openstack-nova | 04:08 | |
*** tbachman has quit IRC | 04:08 | |
*** fragatina has quit IRC | 04:08 | |
*** fragatina has joined #openstack-nova | 04:09 | |
*** iceyao has quit IRC | 04:12 | |
*** kfarr has quit IRC | 04:13 | |
*** youzp has quit IRC | 04:13 | |
*** zsli__ has joined #openstack-nova | 04:15 | |
*** iceyao has joined #openstack-nova | 04:15 | |
*** zsli__ has quit IRC | 04:18 | |
*** zsli_ has quit IRC | 04:18 | |
*** zsli__ has joined #openstack-nova | 04:18 | |
*** iceyao has quit IRC | 04:20 | |
*** psachin has joined #openstack-nova | 04:25 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 04:26 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 04:27 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types https://review.openstack.org/430608 | 04:27 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 04:28 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 04:28 |
*** Sukhdev has quit IRC | 04:28 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Refactor a test method including 3 test cases https://review.openstack.org/394738 | 04:30 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Refactor a test method including 7 test cases https://review.openstack.org/395436 | 04:30 |
*** iceyao has joined #openstack-nova | 04:31 | |
*** kaisers_ has quit IRC | 04:32 | |
*** crushil has quit IRC | 04:34 | |
*** zsli_ has joined #openstack-nova | 04:34 | |
*** links has joined #openstack-nova | 04:35 | |
*** guchihiro has joined #openstack-nova | 04:36 | |
*** zsli__ has quit IRC | 04:36 | |
*** youzp has joined #openstack-nova | 04:37 | |
*** gongysh has quit IRC | 04:40 | |
*** edmondsw has joined #openstack-nova | 04:53 | |
*** gongysh has joined #openstack-nova | 04:53 | |
*** youzp has quit IRC | 04:55 | |
*** zsli_ has quit IRC | 04:55 | |
*** gongysh has quit IRC | 04:56 | |
*** edmondsw has quit IRC | 04:57 | |
*** diga has joined #openstack-nova | 04:58 | |
*** udesale has joined #openstack-nova | 04:59 | |
*** zsli_ has joined #openstack-nova | 05:00 | |
*** zsli_ has quit IRC | 05:00 | |
*** raghav has quit IRC | 05:01 | |
*** kaisers_ has joined #openstack-nova | 05:02 | |
*** ratailor has joined #openstack-nova | 05:04 | |
*** ayogi has joined #openstack-nova | 05:07 | |
*** esberglu has joined #openstack-nova | 05:08 | |
*** salv-orlando has joined #openstack-nova | 05:11 | |
*** youzp has joined #openstack-nova | 05:11 | |
*** dimtruck is now known as zz_dimtruck | 05:12 | |
*** esberglu has quit IRC | 05:13 | |
*** salv-orlando has quit IRC | 05:15 | |
*** gcb has joined #openstack-nova | 05:17 | |
*** trinaths has joined #openstack-nova | 05:21 | |
*** diga has quit IRC | 05:24 | |
*** youzp has quit IRC | 05:25 | |
*** tojuvone has quit IRC | 05:26 | |
*** tojuvone has joined #openstack-nova | 05:27 | |
*** NikhilS has joined #openstack-nova | 05:30 | |
*** prateek has joined #openstack-nova | 05:33 | |
*** fragatina has quit IRC | 05:33 | |
*** diga has joined #openstack-nova | 05:36 | |
*** sridharg has joined #openstack-nova | 05:46 | |
*** adisky_ has joined #openstack-nova | 05:48 | |
*** fragatina has joined #openstack-nova | 05:49 | |
*** gongysh has joined #openstack-nova | 05:51 | |
*** claudiub has joined #openstack-nova | 05:51 | |
*** dikonoor has joined #openstack-nova | 05:57 | |
*** kaisers_ has quit IRC | 06:06 | |
*** nkrinner_afk is now known as nkrinner | 06:06 | |
*** salv-orlando has joined #openstack-nova | 06:12 | |
*** bkopilov has joined #openstack-nova | 06:14 | |
*** yamahata has joined #openstack-nova | 06:15 | |
*** prateek has quit IRC | 06:15 | |
*** prateek has joined #openstack-nova | 06:16 | |
*** salv-orlando has quit IRC | 06:17 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Repropose tag-instance-when-boot https://review.openstack.org/415315 | 06:23 |
*** satyar has joined #openstack-nova | 06:25 | |
*** salv-orlando has joined #openstack-nova | 06:26 | |
*** david-lyle has quit IRC | 06:27 | |
*** belmoreira has joined #openstack-nova | 06:29 | |
*** Shunli has joined #openstack-nova | 06:34 | |
*** faizy has joined #openstack-nova | 06:35 | |
*** moshele has joined #openstack-nova | 06:37 | |
*** vks1 has joined #openstack-nova | 06:37 | |
gmann | sfinucan: thansk, actually my concern is we are separating options in this release only, means old one is still available for neutron also - https://review.openstack.org/#/c/431038/4 | 06:39 |
*** edmondsw has joined #openstack-nova | 06:41 | |
*** unicell has quit IRC | 06:42 | |
*** kaisers_ has joined #openstack-nova | 06:44 | |
*** gyee has joined #openstack-nova | 06:45 | |
*** edmondsw has quit IRC | 06:46 | |
*** steveq has quit IRC | 06:46 | |
*** gyee has quit IRC | 06:46 | |
*** andreas_s has joined #openstack-nova | 06:47 | |
vks1 | hi i built my environment using devstack. i am trying with ocata . my instances are always stucked intp "Scheduling" state | 06:48 |
*** zsli_ has joined #openstack-nova | 06:49 | |
*** gyee has joined #openstack-nova | 06:49 | |
*** kaisers_ has quit IRC | 06:49 | |
*** lpetrut has joined #openstack-nova | 06:49 | |
*** gyee has quit IRC | 06:50 | |
*** mnestratov has joined #openstack-nova | 06:50 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Send out notifications when instance tags chaned https://review.openstack.org/429569 | 06:51 |
*** kaisers_ has joined #openstack-nova | 06:51 | |
*** Shunli has quit IRC | 06:51 | |
*** arne_r has joined #openstack-nova | 06:51 | |
*** fragatina has quit IRC | 06:52 | |
*** bmace has quit IRC | 06:54 | |
*** bmace has joined #openstack-nova | 06:55 | |
*** kaisers_ has quit IRC | 06:56 | |
*** esberglu has joined #openstack-nova | 06:57 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Don't delete neutron port when attach failed https://review.openstack.org/403568 | 06:57 |
*** zsli_ has quit IRC | 06:59 | |
*** zsli_ has joined #openstack-nova | 06:59 | |
openstackgerrit | Zhenyu Zheng proposed openstack/nova-specs master: Repropose tag-instance-when-boot https://review.openstack.org/415315 | 07:00 |
*** zsli__ has joined #openstack-nova | 07:01 | |
*** esberglu has quit IRC | 07:01 | |
*** moshele has quit IRC | 07:02 | |
*** zsli_ has quit IRC | 07:04 | |
*** nkrinner has quit IRC | 07:09 | |
*** nkrinner has joined #openstack-nova | 07:14 | |
*** mriedem has quit IRC | 07:17 | |
*** tesseract has joined #openstack-nova | 07:20 | |
*** salv-orl_ has joined #openstack-nova | 07:23 | |
*** youzp has joined #openstack-nova | 07:23 | |
*** youzp has quit IRC | 07:23 | |
*** moshele has joined #openstack-nova | 07:26 | |
*** salv-orlando has quit IRC | 07:26 | |
*** youzp has joined #openstack-nova | 07:28 | |
crusio | i'm using ova image to create VM on vmware, i got the error in nova compute log: "VimFaultException: A specified parameter was not correct: fileType" , can somebody help me? | 07:30 |
crusio | it is doing "copy_virtual_disk" in the process of creating virtual machine | 07:32 |
*** zsli__ has quit IRC | 07:33 | |
*** slaweq__ has quit IRC | 07:33 | |
*** slaweq has joined #openstack-nova | 07:34 | |
*** youzp has quit IRC | 07:36 | |
openstackgerrit | Dinesh Bhor proposed openstack/python-novaclient master: Remove duplicate methods https://review.openstack.org/445338 | 07:37 |
*** markus_z has joined #openstack-nova | 07:45 | |
*** takashin has left #openstack-nova | 07:48 | |
*** guchihiro has quit IRC | 07:50 | |
*** pcaruana has joined #openstack-nova | 07:50 | |
*** priteau has joined #openstack-nova | 07:51 | |
*** kaisers_ has joined #openstack-nova | 07:52 | |
*** lpetrut has quit IRC | 07:55 | |
*** CristinaPauna has joined #openstack-nova | 07:55 | |
*** claudiub has quit IRC | 07:55 | |
*** diga has quit IRC | 07:56 | |
bauzas | p!spy | 07:58 |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-nova | 08:00 | |
*** alexpilo_ has joined #openstack-nova | 08:03 | |
*** alexpilotti has quit IRC | 08:03 | |
*** raunak has quit IRC | 08:04 | |
*** diga has joined #openstack-nova | 08:05 | |
*** salv-orl_ has quit IRC | 08:10 | |
openstackgerrit | jichenjc proposed openstack/nova master: fixed typos and reword stable api doc https://review.openstack.org/442310 | 08:11 |
*** esberglu has joined #openstack-nova | 08:12 | |
*** ltomasbo|away is now known as ltomasbo | 08:14 | |
*** esberglu has quit IRC | 08:16 | |
*** kaisers_ has quit IRC | 08:24 | |
*** jaosorior has joined #openstack-nova | 08:24 | |
*** udesale has quit IRC | 08:24 | |
*** diga has quit IRC | 08:28 | |
*** edmondsw has joined #openstack-nova | 08:29 | |
*** aarefiev_afk is now known as aarefiev | 08:32 | |
*** edmondsw has quit IRC | 08:34 | |
*** sandanar has joined #openstack-nova | 08:34 | |
*** diga has joined #openstack-nova | 08:36 | |
*** karimb has joined #openstack-nova | 08:36 | |
openstackgerrit | Maho Koshiya proposed openstack/nova master: Add interfaces functional negative tests https://review.openstack.org/442892 | 08:36 |
*** udesale has joined #openstack-nova | 08:37 | |
*** amoralej|off is now known as amoralej | 08:39 | |
openstackgerrit | Alex Xu proposed openstack/nova master: Let us see how terrible to remove stevedore https://review.openstack.org/445361 | 08:42 |
*** jpena|off is now known as jpena | 08:43 | |
*** gongysh has quit IRC | 08:45 | |
*** ralonsoh has joined #openstack-nova | 08:46 | |
*** zioproto has joined #openstack-nova | 08:49 | |
*** NikhilS has quit IRC | 08:52 | |
*** NikhilS has joined #openstack-nova | 08:52 | |
*** salv-orlando has joined #openstack-nova | 08:54 | |
*** jpena is now known as jpena|off | 08:56 | |
*** NikhilS has quit IRC | 08:57 | |
*** udesale has quit IRC | 09:00 | |
*** udesale has joined #openstack-nova | 09:01 | |
*** iceyao has quit IRC | 09:02 | |
*** jpena|off is now known as jpena | 09:03 | |
robcresswell | Quick question; someone's added a patch to Horizon that hides the Soft Reboot button if the Instance state is anything other than Active; is this correct? Had a look at the API docs but they only seem to show how to form the request, not any of the conditions around its usage. | 09:04 |
jaosorior | gibi: hey, could you revisit this https://review.openstack.org/#/c/444947/ ? Had a typo on a test :/ | 09:04 |
*** rmart04 has joined #openstack-nova | 09:07 | |
*** Shunli has joined #openstack-nova | 09:08 | |
*** Shunli has quit IRC | 09:09 | |
*** Shunli has joined #openstack-nova | 09:09 | |
*** efoley__ has joined #openstack-nova | 09:10 | |
*** gongysh has joined #openstack-nova | 09:12 | |
*** efoley_ has joined #openstack-nova | 09:13 | |
*** lucas-afk is now known as lucasagomes | 09:15 | |
*** iceyao has joined #openstack-nova | 09:15 | |
*** romainh has joined #openstack-nova | 09:16 | |
*** NikhilS has joined #openstack-nova | 09:16 | |
*** efoley__ has quit IRC | 09:17 | |
*** kaisers_ has joined #openstack-nova | 09:19 | |
gibi | jaosorior: hi! looking... | 09:21 |
*** haplo37 has quit IRC | 09:21 | |
gibi | jaosorior: +1 from me :) | 09:22 |
jaosorior | gibi: thanks | 09:23 |
*** haplo37 has joined #openstack-nova | 09:30 | |
*** gongysh has quit IRC | 09:33 | |
*** abalutoiu_ has quit IRC | 09:34 | |
*** yingjun has quit IRC | 09:35 | |
*** alexpilotti has joined #openstack-nova | 09:36 | |
*** Shunli has quit IRC | 09:37 | |
*** alexpilo_ has quit IRC | 09:37 | |
*** esberglu has joined #openstack-nova | 09:41 | |
*** derekh has joined #openstack-nova | 09:42 | |
*** esberglu has quit IRC | 09:45 | |
*** wangqun has quit IRC | 09:49 | |
*** satyar has quit IRC | 09:54 | |
*** zhurong has quit IRC | 09:54 | |
*** kaisers_ has quit IRC | 09:59 | |
*** avolkov has joined #openstack-nova | 10:00 | |
*** tovin07_ has quit IRC | 10:03 | |
*** nicolasbock has joined #openstack-nova | 10:04 | |
*** lennyb has quit IRC | 10:06 | |
asettle | Morning nova! | 10:09 |
*** namnh has quit IRC | 10:09 | |
asettle | johnthetubaguy or bauzas - do you konw anyone that could help out with Brian's request here? | 10:09 |
asettle | https://review.openstack.org/#/c/438328/ | 10:09 |
asettle | Apparently the instructions are looking good, but just need someone to help out with the Ubuntu steps to make sure they're all okay. | 10:10 |
asettle | Maybe nicolasbock later? :) | 10:10 |
johnthetubaguy | asettle: sorry, I don't really know of anyone that uses the distro packages | 10:11 |
*** gcb has quit IRC | 10:11 | |
asettle | No problem johnthetubaguy :) | 10:11 |
bauzas | me too | 10:11 |
asettle | Kevin_Zheng: if you're able to help, lemme know! | 10:11 |
asettle | bauzas: as in, you don't know anyone either? | 10:11 |
*** lennyb has joined #openstack-nova | 10:11 | |
bauzas | I'm a Hatter, so .deb packages are not known by me :p | 10:11 |
asettle | bauzas: if you're able to test the instructions using RHEL or CentOS that would be even *more* awesome? :D | 10:12 |
*** edmondsw has joined #openstack-nova | 10:17 | |
*** kevinz has quit IRC | 10:17 | |
*** salv-orlando has quit IRC | 10:18 | |
vks1 | hi, i need help. instances on my openstack ocata setup are getting stucked in Scheduling state | 10:18 |
*** diga has quit IRC | 10:21 | |
*** edmondsw has quit IRC | 10:22 | |
*** youzp has joined #openstack-nova | 10:23 | |
*** ducnc has quit IRC | 10:23 | |
*** kaisers_ has joined #openstack-nova | 10:24 | |
*** ducnc has joined #openstack-nova | 10:24 | |
*** NikhilS has quit IRC | 10:24 | |
*** NikhilS has joined #openstack-nova | 10:25 | |
*** phuongnh has quit IRC | 10:26 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Move 'floating_ips' opts into 'network' https://review.openstack.org/431039 | 10:28 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate 'default_floating_pool' https://review.openstack.org/431038 | 10:28 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Add neutron.floating_pool https://review.openstack.org/431037 | 10:28 |
sfinucan | gmann: ^ | 10:28 |
sfinucan | think that's what you're suggesting? | 10:28 |
*** youzp has quit IRC | 10:28 | |
sfinucan | jaypipes: You have a good Gerrit dash that includes os-vif? | 10:29 |
*** CristinaPauna has quit IRC | 10:29 | |
* sfinucan is missing os-vif reviews as he's no easy way to find them | 10:30 | |
*** CristinaPauna has joined #openstack-nova | 10:30 | |
*** Raildo has joined #openstack-nova | 10:31 | |
openstackgerrit | Roman Podoliaka proposed openstack/nova master: nova-status: require placement >= 1.4 https://review.openstack.org/441881 | 10:41 |
*** nshetty has joined #openstack-nova | 10:42 | |
openstackgerrit | Roman Podoliaka proposed openstack/nova master: Make eventlet hub use a monotonic clock https://review.openstack.org/434327 | 10:42 |
openstackgerrit | Roman Podoliaka proposed openstack/nova master: block_device: fix propagation of instance AZ to Cinder https://review.openstack.org/366724 | 10:42 |
*** trinaths has left #openstack-nova | 10:42 | |
*** abalutoiu has joined #openstack-nova | 10:44 | |
*** salv-orlando has joined #openstack-nova | 10:45 | |
*** salv-orl_ has joined #openstack-nova | 10:46 | |
*** NightKhaos has quit IRC | 10:46 | |
*** lpetrut has joined #openstack-nova | 10:46 | |
*** nicolasbock has quit IRC | 10:46 | |
*** sdague has joined #openstack-nova | 10:47 | |
*** nicolasbock has joined #openstack-nova | 10:48 | |
*** salv-orlando has quit IRC | 10:49 | |
*** NightKhaos has joined #openstack-nova | 10:51 | |
*** salv-orl_ has quit IRC | 10:51 | |
*** youzp has joined #openstack-nova | 10:53 | |
*** kaisers_ has quit IRC | 10:56 | |
*** youzp has quit IRC | 10:58 | |
*** nshetty has quit IRC | 11:00 | |
*** nshetty has joined #openstack-nova | 11:02 | |
*** Guest91536 has quit IRC | 11:03 | |
*** nshetty has quit IRC | 11:03 | |
*** yassine has joined #openstack-nova | 11:03 | |
*** yassine is now known as Guest47 | 11:04 | |
*** sree has quit IRC | 11:08 | |
*** smatzek has joined #openstack-nova | 11:08 | |
*** sree has joined #openstack-nova | 11:08 | |
*** cdent has joined #openstack-nova | 11:09 | |
lyarwood | mdbooth / johnthetubaguy ; https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/cinder-new-attach-apis - if you have any bandwidth today/tomorrow the DB & object changes here could really use some attention before the next cinder/nova meeting on Thursday | 11:11 |
*** Raildo is now known as raildo | 11:13 | |
*** abalutoiu has quit IRC | 11:15 | |
*** raildo has left #openstack-nova | 11:18 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Improve documentation for defer_iptables_apply https://review.openstack.org/445436 | 11:18 |
*** bvanhav has joined #openstack-nova | 11:18 | |
*** vks1 has quit IRC | 11:19 | |
*** moshele has quit IRC | 11:20 | |
openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Remove templates from approved/implemented specs https://review.openstack.org/405550 | 11:21 |
sfinucan | johnthetubaguy: did you mean remove specs/backlog-template.rst or update it? ^ | 11:22 |
alex_xu | gmann: just done the review for https://review.openstack.org/#/c/421760/, sorry for the late | 11:23 |
*** udesale__ has joined #openstack-nova | 11:24 | |
*** udesale has quit IRC | 11:24 | |
frickler | sdague: cdent hinted that you might have more information (as in some documentation) regarding the replacement of the %(tenant_id)s in the compute endpoints | 11:26 |
*** sree has quit IRC | 11:27 | |
*** sree has joined #openstack-nova | 11:27 | |
*** youzp has joined #openstack-nova | 11:28 | |
*** sree has quit IRC | 11:28 | |
*** sree has joined #openstack-nova | 11:29 | |
*** esberglu has joined #openstack-nova | 11:29 | |
*** tbachman has joined #openstack-nova | 11:31 | |
*** sree has quit IRC | 11:33 | |
*** esberglu has quit IRC | 11:34 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Fix backwards compatibility for InstanceNUMACell https://review.openstack.org/396184 | 11:35 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: Combine obj_make_compatible tests https://review.openstack.org/426261 | 11:35 |
*** diga has joined #openstack-nova | 11:35 | |
*** links has quit IRC | 11:36 | |
nicolasbock | asettle, I am going through the instructions again for SLES 12 (https://review.openstack.org/#/c/438328) | 11:36 |
asettle | nicolasbock: great :D | 11:36 |
asettle | thanks | 11:36 |
nicolasbock | asettle, no problem | 11:36 |
asettle | Let me know if you run into anything/updates :) | 11:37 |
asettle | Would appreciate it | 11:37 |
nicolasbock | definitely | 11:37 |
*** NikhilS has quit IRC | 11:37 | |
*** namnh has joined #openstack-nova | 11:38 | |
openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Remove templates from approved/implemented specs https://review.openstack.org/405550 | 11:38 |
*** faizy has quit IRC | 11:40 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Using max api version in notificationt sample test https://review.openstack.org/443686 | 11:41 |
*** NikhilS has joined #openstack-nova | 11:42 | |
*** namnh has quit IRC | 11:43 | |
*** namnh has joined #openstack-nova | 11:43 | |
*** Matias_ has quit IRC | 11:44 | |
*** Matias_ has joined #openstack-nova | 11:46 | |
*** nmathew has quit IRC | 11:46 | |
*** links has joined #openstack-nova | 11:48 | |
*** liverpooler has joined #openstack-nova | 11:50 | |
jaypipes | sfinucan: unfortunately, no. :( | 11:52 |
*** namnh has quit IRC | 11:52 | |
sfinucan | darn | 11:52 |
*** kaisers_ has joined #openstack-nova | 11:52 | |
jaypipes | sfinucan: I used to use gertty but since reformatting my workstation haven't installed it because it takes so long to sync and I forgot to backup my gertty.db | 11:53 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Short circuit notifiations when not enabled https://review.openstack.org/428260 | 11:53 |
sfinucan | jaypipes: Oh, yes. I forgot you use gertty | 11:54 |
*** gszasz has joined #openstack-nova | 11:54 | |
* sfinucan has it on his todo list, but inertia is a wonderful thing | 11:55 | |
cdent | sfinucan, jaypipes : I had to give up on gertty because of sync problems | 11:56 |
jaypipes | cdent: ya | 11:56 |
cdent | It's too easy to think you're up to date only to find out you're not | 11:56 |
jaypipes | yep | 11:56 |
cdent | after making a mess | 11:57 |
jaypipes | though it's easy enough to do that with the web UI as well ;) | 11:57 |
sdague | frickler: it's just not needed at all any more when you create endpoint definitions | 11:57 |
sdague | frickler: what are you looking for in particular? | 11:58 |
*** dave-mccowan has joined #openstack-nova | 12:00 | |
*** ratailor has quit IRC | 12:02 | |
frickler | frickler: 1. how the endpoint should look like instead. 2. how to upgrade old installations 3. some doc to link to for the change in our deployment scripts/guides | 12:03 |
*** kevinz has joined #openstack-nova | 12:03 | |
frickler | sdague: ^^ | 12:04 |
*** liverpooler has quit IRC | 12:04 | |
*** liverpooler has joined #openstack-nova | 12:04 | |
*** tuanluong has quit IRC | 12:05 | |
frickler | sdague: like in step 4 here http://docs-draft.openstack.org/28/438328/10/check/gate-openstack-manuals-tox-doc-publish-checkbuild/5844224//publish-docs/draft/install-guide-ubuntu/nova-controller-install.html#prerequisites | 12:05 |
*** kevinz has quit IRC | 12:07 | |
*** aysyd has joined #openstack-nova | 12:07 | |
*** gabor_antal has joined #openstack-nova | 12:07 | |
sdague | frickler: ah, where is the source doc for that? I didn't realize that never was updated | 12:07 |
sdague | basically you just don't include %(tenant_id)s | 12:08 |
frickler | sdague: that is a change that updates the doc, 438328 | 12:08 |
sdague | frickler: ok, https://review.openstack.org/445462 is the doc update | 12:17 |
*** hshiina has joined #openstack-nova | 12:18 | |
frickler | sdague: great, thx | 12:19 |
frickler | sdague: should one also update this for existing installations? if yes, during upgrade to which release? | 12:20 |
sdague | frickler: well, it works either way, but I think this starts working in Mitaka | 12:22 |
*** ociuhandu has quit IRC | 12:22 | |
*** kevinz has joined #openstack-nova | 12:22 | |
*** edmondsw has joined #openstack-nova | 12:22 | |
*** vks1 has joined #openstack-nova | 12:23 | |
sdague | https://github.com/openstack/nova/blob/409f04b2185cc2da9bf4452b20b3d3d9b03beab8/nova/api/openstack/compute/rest_api_version_history.rst#L167-L169 | 12:24 |
sdague | yeh, that's in the Mitaka range | 12:24 |
sdague | Mitaka was 2.25 | 12:24 |
frickler | hmm, but "works either way" would make me rather not change an existing setup | 12:25 |
*** faizy has joined #openstack-nova | 12:25 | |
*** edmondsw_ has joined #openstack-nova | 12:26 | |
*** abalutoiu has joined #openstack-nova | 12:26 | |
openstackgerrit | Tadas ustinavičius proposed openstack/nova master: Implements: blueprint custom-emulator-path https://review.openstack.org/445466 | 12:26 |
*** edmondsw has quit IRC | 12:27 | |
*** vks1 has quit IRC | 12:27 | |
*** vks1 has joined #openstack-nova | 12:28 | |
*** rfolco has joined #openstack-nova | 12:29 | |
sdague | frickler: that's fine, we were doing a soft transition. | 12:30 |
sdague | we should document this is the prefered mechanism going forward, and honestly probably backport that installguide change | 12:30 |
*** kaisers_ has quit IRC | 12:32 | |
*** faizy has quit IRC | 12:34 | |
*** gongysh has joined #openstack-nova | 12:37 | |
*** efried has joined #openstack-nova | 12:37 | |
*** mdrabe has joined #openstack-nova | 12:38 | |
vdrok | jaypipes: cdent good morning. when you're around, wanted to discuss https://review.openstack.org/443628. these are the integration tests for resource classes in ironic. for now I just copy-pasted some stuff in the services/placement there to make the calls to placement api. and from what I heard from jaypipes there is no plan to have such client in tempest? | 12:42 |
vdrok | should we just put some stripped out version that will be doing get requests to some subset of endpoints in our tempest plugin? | 12:42 |
cdent | vdrok: looking | 12:42 |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_attach notification https://review.openstack.org/401992 | 12:47 |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_detach notification https://review.openstack.org/408676 | 12:50 |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.volume_detach notification https://review.openstack.org/408676 | 12:51 |
*** gouthamr has joined #openstack-nova | 12:52 | |
*** links has quit IRC | 12:52 | |
*** baoli has joined #openstack-nova | 12:52 | |
*** links has joined #openstack-nova | 12:53 | |
*** amoralej is now known as amoralej|lunch | 12:53 | |
cdent | vdrok: there are no immediate plans for creating a tempest client, since as you've discovered it is relatively easy to create one that "just does requests". | 12:53 |
vdrok | cdent: yup, for me personally, there is no problem with that. just thought that at least neutron may want to reuse it | 12:54 |
*** amotoki has joined #openstack-nova | 12:54 | |
*** tblakes has joined #openstack-nova | 12:54 | |
*** lucasagomes is now known as lucas-hungry | 12:55 | |
*** baoli has quit IRC | 12:55 | |
vdrok | jroll: so, are you ok with keeping the minimal client in our tree? | 12:56 |
cdent | vdrok: they've got one that they started too (internal to neutron, not part of tempest). I suspect at some point we'll want to extract something to remove duplication. | 12:56 |
*** baoli has joined #openstack-nova | 12:56 | |
cdent | vdrok: there's also a branch that was started to create a proper python-placementclient: | 12:56 |
cdent | https://github.com/miguellavalle/python-placementclient/tree/adding-grp-support | 12:56 |
cdent | but it ended up feeling like overkill | 12:56 |
cdent | at some point it will be necessary though as there are plans for managing things like shared disk from the command line | 12:57 |
*** abalutoiu has quit IRC | 12:57 | |
vdrok | mhm, I see, thanks cdent ! | 12:58 |
*** catintheroof has joined #openstack-nova | 12:58 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Transform instance.reboot notifications https://review.openstack.org/382959 | 12:59 |
*** catintheroof has quit IRC | 12:59 | |
*** catintheroof has joined #openstack-nova | 12:59 | |
*** baoli has quit IRC | 13:00 | |
*** baoli has joined #openstack-nova | 13:02 | |
*** vladikr has joined #openstack-nova | 13:02 | |
*** esberglu has joined #openstack-nova | 13:03 | |
*** hoangcx has joined #openstack-nova | 13:03 | |
*** gongysh has quit IRC | 13:03 | |
*** ociuhandu has joined #openstack-nova | 13:06 | |
*** tbachman has quit IRC | 13:07 | |
jroll | vdrok: if we agree those tests are useful, yes, I'm fine with that client in our tree for now | 13:10 |
*** bvanhav_ has joined #openstack-nova | 13:11 | |
*** raghav has joined #openstack-nova | 13:11 | |
*** markvoelker has quit IRC | 13:11 | |
raghav | while doing devstack of ocata release | 13:11 |
raghav | i am getting below error from nova | 13:11 |
vdrok | jroll: do you think it would be enough just having the functional ones in nova? | 13:11 |
*** bvanhav has quit IRC | 13:11 | |
raghav | NO hosts found to map to cell | 13:12 |
raghav | i have not enabled n-cell then why | 13:12 |
raghav | cell related error is coming | 13:12 |
jroll | vdrok: yeah, probably, but more tests can't hurt | 13:12 |
vdrok | jroll: I think for now, yes, they are not super-useful, as we're basically testing that ironic is returning what we expect in the virt driver. but, not mocking out ironic is a good thing :) | 13:14 |
jroll | vdrok: yeah, I think we'll expand, and later if ironic pushes data directly to placement they'll be very useful | 13:15 |
jroll | vdrok: I'm fine with landing them | 13:15 |
vdrok | ok cool :) I'll polish it a bit more, remove some unnecessary stuff today and they'll be ready for review | 13:16 |
*** ayogi has quit IRC | 13:16 | |
*** markvoelker has joined #openstack-nova | 13:16 | |
cdent | vdrok: if you could add me to the review when it is ready, I'd appreciate it | 13:17 |
vdrok | cdent: sure, will do :) | 13:17 |
*** claudiub has joined #openstack-nova | 13:18 | |
jroll | vdrok: awesome, thanks! | 13:19 |
*** krtaylor has joined #openstack-nova | 13:19 | |
*** crushil has joined #openstack-nova | 13:21 | |
*** josecastroleon has joined #openstack-nova | 13:21 | |
*** abalutoiu has joined #openstack-nova | 13:21 | |
*** faizy has joined #openstack-nova | 13:21 | |
*** kevinz has quit IRC | 13:21 | |
raghav | hi all i am getting below error..while stacking | 13:22 |
raghav | NO hosts found to map to cell | 13:22 |
raghav | ocata release | 13:22 |
raghav | i have not enabled n-cell servcie then also i am getting this error | 13:23 |
raghav | could anyone please help me | 13:23 |
sdague | raghav: this is a completely fresh install? | 13:24 |
*** prateek has quit IRC | 13:25 | |
johnthetubaguy | raghav: the install guide is in the process of being updated for ocata, so if you followed that there are some missing steps right now | 13:26 |
*** liangy has joined #openstack-nova | 13:27 | |
johnthetubaguy | sfinucan: sorry, I mean update the link | 13:27 |
raghav | i have done clean.sh on my previous devstack installation and then did stack | 13:27 |
johnthetubaguy | for context I was thinking about https://review.openstack.org/#/c/438328/ | 13:27 |
johnthetubaguy | oh | 13:27 |
raghav | john: i am using devstack | 13:27 |
*** kaisers_ has joined #openstack-nova | 13:27 | |
sdague | raghav: honestly, given some of the other issues, please start with a completely fresh OS install | 13:28 |
sdague | once changes have been made to config, clean.sh may not delete some of those things | 13:28 |
sfinucan | johnthetubaguy: OK, think I've done that now | 13:28 |
*** felipemonteiro has joined #openstack-nova | 13:29 | |
johnthetubaguy | hmm gerrit appears to hate me | 13:30 |
raghav | sdague: can there be any other soln than re installation | 13:30 |
*** artom has quit IRC | 13:32 | |
sdague | raghav: given the set of issues you were showing yesterday, I think that debugging the current state of your machine isn't going to be a productive use of anyone's time, including yours | 13:32 |
sdague | I would start fresh | 13:32 |
*** kaisers_ has quit IRC | 13:33 | |
raghav | sdague: ok thank you | 13:33 |
*** smatzek has quit IRC | 13:34 | |
*** mriedem has joined #openstack-nova | 13:35 | |
*** sree has joined #openstack-nova | 13:36 | |
-openstackstatus- NOTICE: Gerrit is going to be restarted due to performance problems | 13:40 | |
*** ChanServ changes topic to "Gerrit is going to be restarted due to performance problems" | 13:40 | |
*** catinthe_ has joined #openstack-nova | 13:41 | |
*** catintheroof has quit IRC | 13:41 | |
*** tbachman has joined #openstack-nova | 13:42 | |
*** jpena is now known as jpena|lunch | 13:43 | |
raghav | sdague: Hi in fresh installation aslo i got same error | 13:43 |
sdague | raghav: please pastebin all the logs and your local.conf, we know that it's working in the general case, it's tested all the time | 13:44 |
*** kevinz has joined #openstack-nova | 13:44 | |
*** catintheroof has joined #openstack-nova | 13:45 | |
*** catinthe_ has quit IRC | 13:45 | |
-openstackstatus- NOTICE: Gerrit has been successfully restarted | 13:46 | |
*** ChanServ changes topic to "Gerrit has been successfully restarted" | 13:46 | |
*** eharney has joined #openstack-nova | 13:48 | |
raghav | sdague: http://paste.openstack.org/show/602688/ please find local.conf | 13:49 |
tdurakov | Hi folks, caught cold, so couldn't run live migration meeting this week, sorry for inconvenience | 13:49 |
cdent | tdurakov: we share an internet virus | 13:50 |
tdurakov | Get well cdent | 13:50 |
cdent | you too | 13:51 |
sdague | raghav: why did you set *_BRANCH ? | 13:52 |
*** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Ocata_Release_Schedule" | 13:52 | |
-openstackstatus- NOTICE: Gerrit has been successfully restarted | 13:52 | |
mriedem | lyarwood: this is kind of a monster backport but is something we need to get released sooner than later https://review.openstack.org/#/c/445236/ | 13:52 |
*** lucas-hungry is now known as lucasagomes | 13:53 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform instance.reboot.error notification https://review.openstack.org/411791 | 13:53 |
*** burt has joined #openstack-nova | 13:55 | |
*** diga has quit IRC | 13:55 | |
openstackgerrit | Danfly proposed openstack/nova master: Completely remove mox from unit/network/test_linux_net.py https://review.openstack.org/445504 | 13:55 |
lyarwood | mriedem: ack, I'll queue that up now, can you unblock https://review.openstack.org/#/c/431540/ when you have time, should be good to go now. | 13:56 |
mriedem | lyarwood: yeah will look now | 13:57 |
lyarwood | thanks | 13:57 |
*** mlavalle has joined #openstack-nova | 13:57 | |
*** links has quit IRC | 13:57 | |
*** smatzek has joined #openstack-nova | 13:59 | |
mriedem | lyarwood: +2 | 14:00 |
mriedem | tonyb: mtreinish: can one of you hit this? https://review.openstack.org/#/c/431540/ | 14:01 |
*** mdnadeem has quit IRC | 14:01 | |
mtreinish | mriedem: looking | 14:01 |
openstackgerrit | Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | 14:02 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163 | 14:02 |
mriedem | bauzas: cdent: jaypipes: so do we want to start looking at creating a placement client? i know mlavalle had already started an osc plugin. | 14:02 |
cdent | mriedem: just had a chat earlier in the day about that with vdrok | 14:02 |
dansmith | I thought the point of the simple design was to make it so we don't need a client | 14:02 |
mriedem | https://github.com/miguellavalle/python-placementclient/tree/adding-grp-support | 14:03 |
cdent | and we said pretty much what dansmith just said | 14:03 |
mtreinish | mriedem: +A | 14:03 |
mriedem | the original spec called out osc CLIs for things like associating resource providers via aggregates, | 14:03 |
*** awaugama has joined #openstack-nova | 14:03 | |
mriedem | or simply listing resource providers as a sanity check | 14:03 |
mriedem | we expect everyone to just be using curl? | 14:04 |
mriedem | bra | 14:04 |
mriedem | *brb | 14:04 |
cdent | yes, the spec did say that, but I'm not sure we've actually hit those use cases yet, so might be just as easy to wait? | 14:04 |
* cdent doesn't no | 14:04 | |
cdent | sick bra, have a great time | 14:04 |
bauzas | mriedem: yeah, I began to work on providing a separate module for that, but I'd prefer to have a separate client | 14:05 |
dansmith | mriedem: a client for use in cron may make sense, but not a separate client for server code, IMHO | 14:06 |
dansmith | mriedem: like a simple thing you can run from the cli to report a particular thing, that's not overly complex, but I would expect cinder to just use requests | 14:06 |
bauzas | dansmith: well, a python client could be cool for OSC | 14:06 |
bauzas | and for other projects | 14:06 |
johnthetubaguy | dansmith: melwitt: macsz: seeing bug patterns emerge around this kind of thing: https://bugs.launchpad.net/nova/+bug/1672624 | 14:07 |
openstack | Launchpad bug 1672624 in OpenStack Compute (nova) "Ceph volumes attached to local deleted instance could not be correctly handled" [Undecided,New] | 14:07 |
dansmith | bauzas: for human usage, yes, which I think exists today | 14:07 |
openstackgerrit | Danfly proposed openstack/nova master: Completely remove mox from unit/network/test_linux_net.py https://review.openstack.org/445504 | 14:07 |
*** amoralej|lunch is now known as amoralej | 14:08 | |
sdague | bauzas: so, honestly, I really think that clients are an anti-pattern | 14:08 |
bauzas | sdague: not sure I understand your concerns | 14:09 |
bauzas | IMHO, we could provide a python placementclient package so OSC could use it for providing CLI | 14:09 |
sdague | here is the issue with that | 14:10 |
sdague | and why I think it's an anti pattern | 14:10 |
bauzas | exactly like the same difference between nova CLI and novaclient python module | 14:10 |
sdague | if you say that the client code is the REST api, you have documentation and examples for that, which are part of our normal process | 14:10 |
*** ayogi has joined #openstack-nova | 14:11 | |
sdague | if you say that the client is this other library, then you need to duplicate the entire set of documentation specific to the class structure for that. And no longer is your rest api your documentation. | 14:11 |
sdague | which needs to include all the examples, etc | 14:11 |
*** lucasxu has joined #openstack-nova | 14:11 | |
bauzas | well, other projects do that, nope ? | 14:12 |
bauzas | we have REST API documentation, but we also have CLI and python bindings docs | 14:12 |
sdague | they don't, which is why OSC has been gutting a bunch of that | 14:12 |
bauzas | and when I was an operator, it was not a problem for me | 14:12 |
bauzas | for example, when I was an operator by Diablo, I looked at the novaclient python bindings docs | 14:13 |
bauzas | well, Essex sorry by that time | 14:13 |
bauzas | and I wrote my scripts using those | 14:13 |
bauzas | for bash things I used CLI docs | 14:13 |
bauzas | and when I wanted to play with the REST API, well, then... actually I never did this :) | 14:14 |
cdent | sdague: I have never agreed with you more than I agree with this conclusion " And no longer is your rest api your documentation." | 14:14 |
bauzas | because getting a token is something a bit confusing honestly | 14:14 |
bauzas | now, I understand how to get a token | 14:14 |
sdague | bauzas: it is, and that's why we hide that behind keystoneauth1 | 14:14 |
bauzas | but as an user, I'd prefer to just use a python client for that | 14:15 |
sdague | but after that point, GET some json is pretty straight forward | 14:15 |
bauzas | I agree | 14:15 |
bauzas | if the REST API is well documented :) | 14:15 |
*** ekuris has quit IRC | 14:15 | |
bauzas | which was not the case previously | 14:15 |
sdague | bauzas: right, and it would be much better to make that true | 14:15 |
sdague | then write tons of other equally poor bits of documentation | 14:16 |
bauzas | sdague: maybe because I'm more a developer than an operator, I looked at how novaclient was working | 14:16 |
sdague | I am excited that you found it good enough for your use case | 14:16 |
*** vladikr has quit IRC | 14:16 | |
bauzas | sdague: but it was easier for me to look at the client code to understand the logic than trying to see how the REST API was working | 14:16 |
sdague | but the experience I've hit with other people is basically why they all use shade now instead of any of the clients | 14:17 |
sdague | bauzas: the REST api docs were pretty terrible at that time | 14:17 |
bauzas | that said, it's now 4 years later, and we're in a far better situation for the REST API thanks to you and other API people :) | 14:17 |
bauzas | yeah zactly | 14:17 |
sdague | bauzas: so, please don't use the lessons from 5 years ago about the right patterns for today | 14:17 |
bauzas | fair enough | 14:17 |
bauzas | but the scheduler.report client module is doing more than just GET and POST, right? | 14:18 |
*** zz_dimtruck is now known as dimtruck | 14:18 | |
bauzas | should we leave other service projects writing their own methods like we do in that current scheduler module ? | 14:19 |
macsz | johnthetubaguy: thanks, will take a look as soon as I get a break from the training | 14:19 |
sdague | bauzas: the things that seem to be common is pretty much just the safe_connect wrapper there, what bits of complex logic would you expect would be needed in neutron? | 14:21 |
dansmith | no, I think a lot of what is there is pretty nova-specific | 14:22 |
sdague | bauzas: I'd much rather start under the assumption that a library is not needed, walk through doing an implementation in another service, figure out if there are common things that would be handy to share (like the safe connect bit) and maybe put that into a utility mixin or back in keystoneauth | 14:22 |
dansmith | there may be a few re-usable things, like "ensure_resource_provider()" but that's really just a get-and-post-on-404 | 14:22 |
dansmith | which is not worth a separate client | 14:22 |
bauzas | sdague: this for example https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L484 | 14:22 |
bauzas | neutron would provide their inventories, right? | 14:23 |
bauzas | or cinder | 14:23 |
cdent | (safe connect in keystoneauth makes good sense) | 14:24 |
sdague | bauzas: maybe, how about implement that first in one of those services and see | 14:24 |
*** iceyao_ has joined #openstack-nova | 14:24 | |
openstackgerrit | Danil Akhmetov proposed openstack/nova master: Use proper user and tenant in the owner section of libvirt.xml. https://review.openstack.org/399679 | 14:24 |
bauzas | sdague: yeah we could possibly wait for other service projects calling the placement API and see if we need a common client package, maybe | 14:25 |
dansmith | bauzas: it's long, but really not much meat in there.. retry on 409 | 14:25 |
cdent | bauzas: neutron built in their own custom client for inventories: https://review.openstack.org/#/c/358658/ | 14:25 |
*** iceyao has quit IRC | 14:25 | |
bauzas | cdent: but they don't have our story yet | 14:26 |
bauzas | sorry | 14:26 |
bauzas | s/story/background | 14:26 |
*** hongbin has joined #openstack-nova | 14:26 | |
cdent | meaning what? | 14:26 |
bauzas | so I'm afraid of them missing some corner case | 14:26 |
bauzas | that we could have fixed in the scheduler.report module | 14:26 |
bauzas | like https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L596 | 14:26 |
bauzas | anyway, I'm not *that* concerned | 14:27 |
bauzas | if there is no traction around a specific common client, fine by me | 14:27 |
sdague | bauzas: honestly, I feel like this is part of the anti pattern. Deciding that projects that consume other project apis don't need to know how to use those APIs, and we'll hide everything in a library. If the API is hard to use correctly, that's data, and we should take it back to the API, not put a bunch of work around code around it | 14:28 |
bauzas | let's wait other projects using the placement API, and we'll see if we need a common one | 14:28 |
sdague | I mean, we were seriously talking 2 cycles ago about the neutron team building a nova consumption interface on top of their neutron client, because it was too hard to use. | 14:28 |
bauzas | sdague: that's a reasonable concern | 14:28 |
sdague | maybe it was more than 2 cycles ago | 14:29 |
*** kaisers_ has joined #openstack-nova | 14:29 | |
bauzas | :) | 14:29 |
*** NikhilS has quit IRC | 14:29 | |
bauzas | yeah, that's a trade-off, I agree | 14:29 |
*** sree has quit IRC | 14:29 | |
bauzas | if we can keep things simple, I'm all good | 14:30 |
bauzas | again, I'm seeing myself in a minority on that, so fair enough | 14:30 |
*** sree has joined #openstack-nova | 14:30 | |
mriedem | i'm back, caught up | 14:30 |
*** satyar has joined #openstack-nova | 14:30 | |
jaypipes | mriedem: I'm strongly opposed to a separate client for placement. Having an openstackclient module for resource providers and inventories/allocations/aggregate associations is A-OK, but not a separate python client. | 14:30 |
mriedem | so no to the client for APIs, and no to CLI for osc? | 14:30 |
bauzas | it's a valid concern, and if people don't really want to have that common client, plus the concerns you mention, then I'm okay | 14:30 |
mriedem | i'm good with not having a python API client | 14:31 |
bauzas | jaypipes: OSC being only a CLI wrapper ? | 14:31 |
mriedem | i was wondering more about CLI | 14:31 |
mriedem | osc is a cli wrapper, not an api library | 14:31 |
mriedem | it uses the other client API libraries and the sdk | 14:31 |
bauzas | yeah, just wanted to reassure that | 14:31 |
bauzas | that we won't push smart code in OSC | 14:32 |
*** eharney_ has joined #openstack-nova | 14:32 | |
bauzas | again, if people feel confident with that, let's stick with dansmith's plan given in the ML | 14:32 |
bauzas | I don't want to divert too much on that | 14:32 |
bauzas | and we could raise a priority on writing a python client module if we need to | 14:33 |
*** eharney has quit IRC | 14:33 | |
*** sree has quit IRC | 14:35 | |
jaypipes | mriedem: OSC doesn't *have* to import a placementclient lib, though. for example, we could just have make_client() return something similar to the keystone session object in the scheduler report client. | 14:35 |
jaypipes | mriedem: in other words, here's the compute example make_client() https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/client.py#L38 | 14:35 |
*** eharney_ is now known as eharney | 14:35 | |
jaypipes | mriedem: we could just as easily change that code to just make a "client" for placement be the requests session and keystoneauth session | 14:35 |
*** dave-mccowan has quit IRC | 14:36 | |
bauzas | jaypipes: given how we heavily mention OSC in our docs, and given we would need that for shared storage stories, is there someone actively working on that? | 14:38 |
bauzas | for Pike, I mean | 14:38 |
jroll | jaypipes: your ironic get_inventory() thing lgtm, nice work | 14:38 |
jaypipes | bauzas: pretty sure we can find folks, yes. | 14:39 |
bauzas | cool :) | 14:40 |
mriedem | jaypipes: i know, i was actually thinking we'd have the osc plugin in nova, | 14:40 |
openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | 14:40 |
mriedem | and that gets split out with the server api stuff when it all happens | 14:40 |
*** edmondsw has joined #openstack-nova | 14:40 | |
mriedem | i thought projects were supposed to host osc plugins now, rather than try and put them all in osc itself and make that core team own them, | 14:40 |
mriedem | that's what i remember from a session in austin anyway | 14:41 |
jaypipes | mriedem: that's fine by me. | 14:41 |
mriedem | sdague: dansmith: are you against an osc plugin living in nova purely for CLI? | 14:41 |
mriedem | or meh? | 14:41 |
dansmith | against existence, or against living in nova? | 14:41 |
jaypipes | mriedem: probably a question for dtroyer too? | 14:42 |
mriedem | the former | 14:42 |
dansmith | no | 14:42 |
*** youzp has quit IRC | 14:42 | |
openstackgerrit | Béla Vancsics proposed openstack/nova master: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225 | 14:42 |
dansmith | there is that thing already yes? | 14:42 |
mriedem | dansmith: i don't see one | 14:42 |
mriedem | mlavalle's is a full blown client, like novaclient | 14:42 |
dansmith | oh, I thought it was a plugin | 14:43 |
mriedem | https://github.com/miguellavalle/python-placementclient/tree/adding-grp-support | 14:43 |
sdague | mriedem: in the nova tree? | 14:44 |
mriedem | sdague: yes, where placement currently lives | 14:44 |
sdague | I'm definitely -1 on in the nova tree, those things shouldn't be coupled that way | 14:44 |
mriedem | so openstack/placement-osc-plugin? | 14:44 |
sdague | mriedem: sure | 14:44 |
*** dikonoor has quit IRC | 14:45 | |
sdague | it can be under nova control, but you don't want to pip install all of nova | 14:45 |
sdague | to get 200 lines of osc plugin | 14:45 |
mriedem | oh right i didn't think about that | 14:45 |
bauzas | good point | 14:45 |
sean-k-mooney | mriedem: would it not make sense to have the placement osc plugin live in the placement repo when that is created | 14:45 |
mriedem | sean-k-mooney: but that's not pike, | 14:45 |
mriedem | or maybe even queens | 14:45 |
sdague | sean-k-mooney: no, it would not | 14:45 |
sdague | osc plugins should not be in the trees of larger projects | 14:45 |
bauzas | yeah | 14:45 |
jroll | mriedem: how we do osc stuff is the osc plugin goes in the python-ironicclient tree | 14:45 |
sdague | because osc plugins should be super minimal | 14:46 |
jroll | seems to work well, and you'll need the ironicclient dep anyway | 14:46 |
mriedem | jroll: we wouldn't have a dep on a placementclient though | 14:46 |
sean-k-mooney | sdague: oh ok. i guess you can use depends-on to track patches to both but i just assumend it would be eaisier to keep them in sync if the clients and apis were in the same repo | 14:46 |
*** smatzek_ has joined #openstack-nova | 14:46 | |
openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance-exists notification https://review.openstack.org/403660 | 14:46 |
sdague | sean-k-mooney: the whole point of osc is to optimize for consumption | 14:47 |
sean-k-mooney | sdague: ah so you want them to be seperate to minimies the code footprint | 14:47 |
jroll | mriedem: ah, yeah I guess just it's own repo makes sense to me then | 14:47 |
sdague | and, honestly, I'm not really sure why we would do things that way vs. just put it into osc directly | 14:47 |
mriedem | sdague: i'm talking with dtroyer in -dev | 14:47 |
mriedem | if you want to ask about it living in osc directly | 14:48 |
mriedem | because i thought they didn't want more stuff living in osc directly, | 14:48 |
openstackgerrit | Alex Szarka proposed openstack/nova master: Transform instance.resize.error notifications https://review.openstack.org/417882 | 14:48 |
mriedem | just like tempest wants to move things to plugins | 14:48 |
*** markus_z has quit IRC | 14:48 | |
*** smatzek has quit IRC | 14:49 | |
dansmith | I guess there's no support for generating multiple packages from one source, like deb and rpm can do? | 14:51 |
*** dave-mccowan has joined #openstack-nova | 14:51 | |
jroll | dansmith: pbr doesn't do that, no | 14:51 |
*** jpena|lunch is now known as jpena | 14:51 | |
*** edmondsw has quit IRC | 14:53 | |
mriedem | melwitt: did you happen to get a functional test started for https://review.openstack.org/#/c/444106/ ? if not, i could try starting something there | 14:53 |
dansmith | jroll: yeah, that's a bummer | 14:54 |
jroll | indeed | 14:54 |
*** marst has quit IRC | 14:55 | |
*** artom has joined #openstack-nova | 14:55 | |
*** tbachman has quit IRC | 14:55 | |
*** faizy_ has joined #openstack-nova | 14:58 | |
*** tbachman has joined #openstack-nova | 14:58 | |
*** kevinz has quit IRC | 14:58 | |
*** kaisers_ has quit IRC | 14:59 | |
*** prateek has joined #openstack-nova | 15:00 | |
*** faizy has quit IRC | 15:00 | |
*** ayogi has quit IRC | 15:01 | |
*** yingjun has joined #openstack-nova | 15:02 | |
*** yingjun has quit IRC | 15:03 | |
*** lucasxu has quit IRC | 15:04 | |
*** armax has joined #openstack-nova | 15:04 | |
*** brad[] has quit IRC | 15:07 | |
mriedem | cdent: bauzas: dansmith: sdague: jaypipes: ok here is a simple blueprint to start tracking the osc plugin stuff https://blueprints.launchpad.net/nova/+spec/placement-osc-plugin | 15:10 |
*** marst has joined #openstack-nova | 15:10 | |
mriedem | with enough detail to hopefully get someone started, maybe someone from osic - they are an infinite resource right? | 15:10 |
cdent | ✔ | 15:11 |
*** adrian_otto has joined #openstack-nova | 15:15 | |
*** iceyao has joined #openstack-nova | 15:15 | |
*** arne_r has quit IRC | 15:16 | |
*** hshiina has quit IRC | 15:17 | |
*** bvanhav_ has quit IRC | 15:17 | |
*** lucasxu has joined #openstack-nova | 15:17 | |
johnthetubaguy | mriedem: I wish we had that ;) | 15:18 |
johnthetubaguy | raj_singh: I guess everyone is busy right now right? | 15:19 |
*** iceyao_ has quit IRC | 15:19 | |
*** brad[] has joined #openstack-nova | 15:19 | |
*** andreas_s has quit IRC | 15:20 | |
johnthetubaguy | mriedem: but +1 documenting it so someone can take that on | 15:20 |
sdague | dansmith / jroll it's not just a pbr limitation, there are other substantial issues there with the python packaging toolchain | 15:20 |
*** hoangcx has quit IRC | 15:21 | |
raj_singh | johnthetubaguy: yea running out of bandwidth for this cycle | 15:21 |
jroll | sdague: that's the understatement of the day :) | 15:22 |
jroll | I know pbr has hard blockers for it, that's all I wanted to point out | 15:22 |
mriedem | Kevin_Zheng: i know it's late for you, just fyi that https://review.openstack.org/#/c/429569/ could use some small improvements and then i'm +2 | 15:22 |
*** xyang1 has joined #openstack-nova | 15:23 | |
*** udesale__ has quit IRC | 15:24 | |
*** psachin has quit IRC | 15:25 | |
openstackgerrit | Dan Smith proposed openstack/nova master: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | 15:25 |
dansmith | bauzas: can you look at this ^ ? | 15:25 |
*** raj_singh_ has quit IRC | 15:26 | |
dansmith | bauzas: I think that's what you're going for.. I still need to do some tests and fix the commit message and stuff, but I think that will efficiently make the caching scheduler behave the same as the filter scheduler | 15:26 |
bauzas | dansmith: sure, will do | 15:26 |
dansmith | i.e. filter based on cell per request, from the global collected list | 15:26 |
bauzas | dansmith: I need to go to the garage soon, but I'll look at it quickly | 15:26 |
dansmith | just a brief look to see if it's what you meant would be good | 15:27 |
dansmith | then I'll release you to the garage | 15:27 |
*** icey has joined #openstack-nova | 15:28 | |
bauzas | dansmith: so https://review.openstack.org/#/c/438025/23/nova/scheduler/caching_scheduler.py | 15:28 |
bauzas | dansmith: lgtm, but I was also thinking about _get_by_hosts() returning a dict instead of a list | 15:29 |
*** seba has joined #openstack-nova | 15:29 | |
bauzas | oops _get_up_hosts | 15:29 |
*** ociuhandu has quit IRC | 15:29 | |
bauzas | dansmith: so we could just lookup by the cell name | 15:29 |
*** mtanino has quit IRC | 15:29 | |
hamzy | hey mriedem I think that I am running into https://bugs.launchpad.net/nova/+bug/1670627 is there a way to clean up the instances from the CLI? I guess that I need to do some DB cleanup? | 15:30 |
openstack | Launchpad bug 1670627 in OpenStack Compute (nova) ocata "quota is always in-use after delete the ERROR instances " [Critical,In progress] - Assigned to Matt Riedemann (mriedem) | 15:30 |
dansmith | bauzas: okay, I need to see how much change that is to the other stuff, but probably okay | 15:30 |
bauzas | dansmith: it's just a nit tho | 15:31 |
bauzas | dansmith: but I'd prefer both of them rather than a filter | 15:31 |
mriedem | hamzy: the instances are deleted right? | 15:31 |
mriedem | hamzy: the problem is the quota usage isn't decremented | 15:32 |
*** penick has joined #openstack-nova | 15:32 | |
hamzy | mriedem, yes "nova list --all-tenants" shows nothing | 15:32 |
hamzy | My compute overview page shows "RAM Used 48GB of 50GB" and 6 used instances... | 15:32 |
* bauzas disappears | 15:32 | |
dansmith | bauzas: yeah, I don't know why I didn't think to do this when johnthetubaguy and I were talking about it.. probably just misunderstanding the cachingscheduler at the time | 15:32 |
mriedem | hamzy: yeah, and the "nova_cell0.usages" table should have that information in it, can you confirm | 15:33 |
mriedem | ? | 15:33 |
mriedem | oh wait, sorry, | 15:33 |
mriedem | not nova_cell0, | 15:33 |
mriedem | just nova.usages | 15:33 |
bauzas | dansmith: the problem with a filter is that operators would need to modify their config for an upgrade | 15:33 |
hamzy | mriedem, I have an AIO installation with a bunch of containers... do I need to attach to one of them> | 15:33 |
bauzas | dansmith: or they wouldn't check the cells | 15:33 |
bauzas | if they forget to modify their conf opt | 15:34 |
dansmith | bauzas: they wouldn't until they add another cell | 15:34 |
mriedem | hamzy: let me check something, we might have a CLI to run | 15:34 |
dansmith | bauzas: but I know, this is better, I'm not arguing | 15:34 |
bauzas | dansmith: sure, but they could also opt-out verifying cells, which I also dislike :) | 15:34 |
bauzas | anyway | 15:34 |
mriedem | cdent: replied to the ML thread on the placement client topic for completeness | 15:34 |
bauzas | mriedem: thanks for that, appreciatedf | 15:34 |
mriedem | and so i don't forget what we agreed on in 4 months time | 15:34 |
cdent | mriedem: yeah, saw that, thanks | 15:34 |
dansmith | bauzas: I'm not arguing for a filter | 15:34 |
bauzas | yeah no worries :) | 15:35 |
mriedem | hamzy: https://docs.openstack.org/developer/nova/man/nova-manage.html#nova-project | 15:35 |
* bauzas really needs to go find his car :) | 15:35 | |
mriedem | hamzy: i'm wondering if "nova-manage project quota_usage_refresh <project_id> [--user <user_id>] [--key <key>]" will help | 15:35 |
*** bvanhav_ has joined #openstack-nova | 15:35 | |
mriedem | hamzy: can you try that for the project in question | 15:35 |
hamzy | yeah let me figure out the project_id | 15:36 |
*** jvgrant_ has quit IRC | 15:37 | |
hamzy | mriedem, is there an openstack CLI equivalent? | 15:38 |
mriedem | hamzy: nope | 15:38 |
mriedem | it's direct to the nova db | 15:38 |
mriedem | and something in osc cli would require a rest api for this in the compute endpoint, which we don't have | 15:38 |
hamzy | mriedem, trying to figure out how to install it in my AIO environment | 15:41 |
mriedem | hamzy: nova-manage is available on the controller | 15:41 |
*** ociuhandu has joined #openstack-nova | 15:41 | |
mriedem | it's a CLI | 15:41 |
mriedem | so "which nova-manage" | 15:41 |
mriedem | AIO == all in one? | 15:42 |
hamzy | yes | 15:42 |
hamzy | installed with openstack-ansible | 15:42 |
*** jvgrant_ has joined #openstack-nova | 15:42 | |
hamzy | mriedem, http://paste.openstack.org/show/602707/ that CLI is not in the utility-container | 15:44 |
mriedem | hamzy: i've never used openstack-ansible, so not sure how things are deployed, | 15:44 |
mriedem | hamzy: how about aio1_nova_conductor_container-41603c3d ? | 15:45 |
johnthetubaguy | not sure which container they run the nova-manage stuff in, worth asking andymccr in #openstack-ansible | 15:45 |
hamzy | nova command is there but not nova-manage | 15:45 |
*** ociuhandu has quit IRC | 15:45 | |
hamzy | same with aio1_nova_console_container-fb9219e4... sigh | 15:46 |
andymccr | hamzy: it will be in the nova containers, but inside the venv for nova | 15:46 |
hamzy | ah | 15:46 |
hamzy | \o/ | 15:47 |
*** MasterofJOKers has joined #openstack-nova | 15:48 | |
mriedem | now to see if that command fixes the usage | 15:48 |
raj_singh | johnthetubaguy: regarding https://review.openstack.org/#/c/438467/, I am wondering if new "force_complete" option could have choices of "abort, pause, post-copy", then we can deprecate live_migration_permit_post_copy? | 15:51 |
*** lpetrut has quit IRC | 15:51 | |
johnthetubaguy | raj_singh: but what about the force complete API that still uses that config? | 15:51 |
hamzy | mriedem, I don't think it did, but the overview page no longer shows the pie charts anymore | 15:51 |
* raj_singh thinking | 15:52 | |
johnthetubaguy | raj_singh: I think folks want to opt into post copy, most want it turned off, so a single config to do that makes sense I think | 15:52 |
raj_singh | johnthetubaguy: ok, let me rework the spec | 15:53 |
*** unicell has joined #openstack-nova | 15:53 | |
mriedem | hamzy: ok, we can check via nova cli, sec | 15:53 |
hamzy | mriedem, FYI I ran quota_usage_refresh on all four projects that I had | 15:54 |
mriedem | hamzy: https://docs.openstack.org/cli-reference/nova.html#nova-quota-show | 15:54 |
johnthetubaguy | raj_singh: we should add that context in the spec | 15:54 |
mriedem | nova quota-show --tenant <tenant id> | 15:54 |
mriedem | plus maybe --detail | 15:54 |
mriedem | nova quota-show --tenant <tenant id> --detail | 15:55 |
mriedem | see if the instances are back to 10 for limit and 0 for usage | 15:55 |
raj_singh | johnthetubaguy: yes, will update it soon. | 15:55 |
*** kaisers_ has joined #openstack-nova | 15:55 | |
*** markvoelker has quit IRC | 15:56 | |
*** iceyao has quit IRC | 15:56 | |
hamzy | mriedem, http://paste.openstack.org/show/602711/ | 15:57 |
johnthetubaguy | raj_singh: got a few nits on the follow on spec too | 15:58 |
*** markvoelker has joined #openstack-nova | 15:58 | |
mriedem | | instances | {u'limit': 10, u'reserved': 0, u'in_use': 6} | | 15:58 |
mriedem | yeah usage is still wrong | 15:58 |
raj_singh | johnthetubaguy: thanks for the review | 15:58 |
*** tbachman_ has joined #openstack-nova | 15:59 | |
mriedem | hamzy: is the [database]/connection option in the nova.conf that the aio1-nova-conductor-container-41603c3d container is using pointing at the 'nova' database? | 16:00 |
*** tbachman has quit IRC | 16:00 | |
*** penick has quit IRC | 16:00 | |
*** tbachman has joined #openstack-nova | 16:01 | |
openstackgerrit | sahid proposed openstack/nova master: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | 16:01 |
openstackgerrit | sahid proposed openstack/nova master: numa: fit instance NUMA node with cpus reserved onto host NUMA node https://review.openstack.org/400875 | 16:01 |
openstackgerrit | sahid proposed openstack/nova master: virt: update overhead to take into account vCPUs https://review.openstack.org/385024 | 16:01 |
hamzy | mriedem, there is a line "connection = mysql+pymysql://nova:random_password@172.29.236.100/nova?charset=utf8" | 16:01 |
openstackgerrit | sahid proposed openstack/nova master: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | 16:01 |
openstackgerrit | sahid proposed openstack/nova master: numa: take into account cpus reserved https://review.openstack.org/400876 | 16:01 |
openstackgerrit | sahid proposed openstack/nova master: numa: update numa usage incording the cpu reserved https://review.openstack.org/400877 | 16:01 |
mriedem | hamzy: yeah that's the nova db | 16:02 |
mriedem | /nova is the nova db | 16:02 |
*** sridharg has quit IRC | 16:03 | |
*** salv-orlando has joined #openstack-nova | 16:03 | |
mriedem | so i'm thinking the refresh CLI doesn't fix things because the refresh operation thinks that usage is correct, because of the bug, we never decremented usage | 16:04 |
*** tbachman_ has quit IRC | 16:04 | |
hamzy | so no --force option? :) | 16:04 |
mriedem | nova quota-update will just update the limits, not the usages, | 16:05 |
*** salv-orlando has quit IRC | 16:05 | |
*** felipemonteiro has quit IRC | 16:05 | |
*** salv-orlando has joined #openstack-nova | 16:05 | |
*** crushil has quit IRC | 16:05 | |
mriedem | i'm not sure if "nova quota-delete" will do what you want, i suspect that will reset quota limits to the default, but not touch usages, | 16:06 |
mriedem | i can look quick | 16:06 |
hamzy | thanks | 16:06 |
mriedem | actually it looks like it will, | 16:07 |
mriedem | it should delete the quota limit, usage and any pending reservations | 16:07 |
*** felipemonteiro has joined #openstack-nova | 16:07 | |
mriedem | it sucks, but if you can reset the quota for that project, then that's what you want "nova quota-delete --tenant <tenant id>" | 16:08 |
hamzy | mriedem, and then how do you recreate the quota? I don't see a quota-create | 16:08 |
*** nic has joined #openstack-nova | 16:08 | |
mriedem | there are global defaults for quota in nova.conf | 16:08 |
mriedem | now might be a good time for me to cleanup https://review.openstack.org/#/c/417563/ | 16:09 |
mriedem | but ^ is an attempt and documenting some of how quotas work in nova | 16:09 |
openstackgerrit | Tadas Ustinavičius proposed openstack/nova master: Fixing flake8 errors https://review.openstack.org/445558 | 16:11 |
melwitt | mriedem: no, didn't do anything with a functional test yet. so feel free :) | 16:12 |
mriedem | melwitt: i think i'm going to just get ocata 13.0.1 out the door for this quotas issue, and add a release note with a known issue for the retry bug, | 16:12 |
*** nic1 has joined #openstack-nova | 16:13 | |
mriedem | and then once that's fixed (with a functional recreate test) we can do a 13.0.2 shortly after | 16:13 |
mriedem | which reminds me to beg for stable reviews on https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/ocata | 16:13 |
hamzy | mriedem, http://paste.openstack.org/show/602712/ and thanks... "openstack server create" was able to boot an instance | 16:13 |
melwitt | okay | 16:13 |
mriedem | once those are in we can do a 13.0.1 release | 16:13 |
*** salv-orlando has quit IRC | 16:13 | |
mriedem | hamzy: nice, cool. | 16:13 |
mriedem | hamzy: i'll make a note of the workaround instructions in the release note for ocata 13.0.1 | 16:13 |
*** Oku_OS is now known as Oku_OS-away | 16:14 | |
mriedem | *er 15.0.1 | 16:14 |
*** aarefiev is now known as aarefiev_afk | 16:14 | |
hamzy | yeah that would be great | 16:14 |
*** nic2 has joined #openstack-nova | 16:14 | |
*** nic has quit IRC | 16:15 | |
*** nic2 has quit IRC | 16:15 | |
*** raunak has joined #openstack-nova | 16:15 | |
*** sandanar has quit IRC | 16:16 | |
*** dtp has joined #openstack-nova | 16:16 | |
mriedem | https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/ocata | 16:16 |
mriedem | oops | 16:16 |
mriedem | sdague: bauzas: claudiub: ^ can you help out here? | 16:16 |
*** nic1 has quit IRC | 16:17 | |
*** romainh1 has joined #openstack-nova | 16:19 | |
openstackgerrit | Gábor Antal proposed openstack/nova-specs master: Show persisted scheduler hints in server details https://review.openstack.org/440580 | 16:19 |
*** nic has joined #openstack-nova | 16:21 | |
*** romainh has quit IRC | 16:21 | |
*** annegentle has joined #openstack-nova | 16:22 | |
melwitt | Kevin_Zheng: I was just starting to look at this bug you reported https://bugs.launchpad.net/nova/+bug/1672624 do you know if it's a regression? | 16:24 |
openstack | Launchpad bug 1672624 in OpenStack Compute (nova) "Ceph volumes attached to local deleted instance could not be correctly handled" [Undecided,New] | 16:24 |
*** ayogi has joined #openstack-nova | 16:25 | |
sdague | mriedem: after I get off a call | 16:26 |
*** annegentle has quit IRC | 16:26 | |
*** annegentle has joined #openstack-nova | 16:27 | |
*** alexpilo_ has joined #openstack-nova | 16:27 | |
*** alexpilotti has quit IRC | 16:28 | |
*** nkrinner is now known as nkrinner_afk | 16:30 | |
*** crushil has joined #openstack-nova | 16:31 | |
*** yamahata has quit IRC | 16:32 | |
*** annegentle has quit IRC | 16:32 | |
*** vks1 has quit IRC | 16:32 | |
*** Apoorva has joined #openstack-nova | 16:33 | |
*** Apoorva has quit IRC | 16:33 | |
*** penick has joined #openstack-nova | 16:34 | |
*** prateek has quit IRC | 16:35 | |
*** vladikr has joined #openstack-nova | 16:36 | |
*** Dinesh_Bhor has quit IRC | 16:37 | |
*** lucasxu has quit IRC | 16:38 | |
*** prateek has joined #openstack-nova | 16:39 | |
*** belmoreira has quit IRC | 16:39 | |
*** claudiub has quit IRC | 16:40 | |
*** zioproto has quit IRC | 16:41 | |
*** david-lyle has joined #openstack-nova | 16:42 | |
*** 21WAAS6DN has joined #openstack-nova | 16:42 | |
jaypipes | rpodolyaka: we'd welcome you help on the osc plugin for placement. :) | 16:45 |
*** ltomasbo is now known as ltomasbo|away | 16:45 | |
openstackgerrit | Matthew Booth proposed openstack/nova master: Add InstanceMappingList.get_by_instance_uuids https://review.openstack.org/445139 | 16:45 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Make server_external_events use an optimal number of db queries https://review.openstack.org/445142 | 16:45 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Add RequestContext.for_cell() https://review.openstack.org/445140 | 16:45 |
openstackgerrit | Matthew Booth proposed openstack/nova master: Add InstanceList.get_by_instance_mapping_list() https://review.openstack.org/445141 | 16:45 |
*** Dinesh_Bhor has joined #openstack-nova | 16:48 | |
*** unicell has quit IRC | 16:49 | |
rpodolyaka | jaypipes: sure, np! :) | 16:49 |
mriedem | rpodolyaka: thanks, you're the owner on the bp now | 16:50 |
*** adrian_otto has quit IRC | 16:52 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor InstancePayload creation https://review.openstack.org/418488 | 16:52 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove **kwargs passing in payload __init__ https://review.openstack.org/418489 | 16:52 |
rpodolyaka | mriedem: thanks! | 16:52 |
*** udesale has joined #openstack-nova | 16:53 | |
*** udesale has quit IRC | 16:53 | |
mriedem | lyarwood: i just posted a release note to ocata for 15.0.1 also https://review.openstack.org/#/c/445575/ | 16:54 |
*** timello has quit IRC | 16:54 | |
*** fragatina has joined #openstack-nova | 16:56 | |
*** fragatina has quit IRC | 16:57 | |
*** ssurana has joined #openstack-nova | 16:57 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform aggregate.add_host notification https://review.openstack.org/396210 | 16:57 |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform aggregate.remove_host notification https://review.openstack.org/396211 | 16:57 |
*** fragatina has joined #openstack-nova | 16:57 | |
openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add locked-by-service spec https://review.openstack.org/438134 | 16:58 |
*** timello has joined #openstack-nova | 17:02 | |
johnthetubaguy | siva_krishnan: thinking about this spec: https://review.openstack.org/#/c/347161 are we not now back to basically what the original spec said now? | 17:03 |
*** fragatina has quit IRC | 17:03 | |
mdbooth | johnthetubaguy: What's a service locked instance? | 17:04 |
* mdbooth thinks service == nova compute on host X | 17:04 | |
johnthetubaguy | mdbooth: its writing up the old story we had around trove "locking" and instance from a user deleting it | 17:04 |
siva_krishnan | johnthetubaguy: yeah only with one change. instead of generating unrescue.xml from database we are copying it to destination from source | 17:05 |
johnthetubaguy | siva_krishnan: ah, OK, could we add a not in the git commit saying thats the only change? | 17:05 |
*** ltomasbo|away is now known as ltomasbo | 17:05 | |
johnthetubaguy | s/not/note/ | 17:05 |
openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in aggregates.py https://review.openstack.org/445230 | 17:06 |
johnthetubaguy | mdbooth: yeah, I hadn't noticed that overloaded term, damm | 17:06 |
mdbooth | What's a service, then? | 17:06 |
johnthetubaguy | mdbooth: in that context, is the service token in the context | 17:06 |
johnthetubaguy | mdbooth: so trove has a service token, it should be in the spec, although something tells me its probably not clear in there | 17:07 |
siva_krishnan | johnthetubaguy: sure. will update the commit message | 17:07 |
mdbooth | I've read the description and use cases sections a couple of times and hadn't managed to infer it from context. | 17:08 |
mdbooth | That could just be me, though. | 17:08 |
openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add locked-by-service spec https://review.openstack.org/438134 | 17:09 |
johnthetubaguy | mdbooth: not sure, the folks reading that so far have a lot of shared context around the problem | 17:10 |
*** eharney has quit IRC | 17:12 | |
openstackgerrit | Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677 | 17:12 |
*** david-lyle_ has joined #openstack-nova | 17:13 | |
*** david-lyle has quit IRC | 17:13 | |
*** yamahata has joined #openstack-nova | 17:13 | |
*** rmart04 has quit IRC | 17:17 | |
*** slaweq_ has quit IRC | 17:17 | |
johnthetubaguy | siva_krishnan: thanks | 17:18 |
mdbooth | johnthetubaguy: Incidentally, you mentioned (last week, I think) you'd have another look at https://review.openstack.org/#/c/383859/ | 17:19 |
*** ltomasbo is now known as ltomasbo|away | 17:19 | |
johnthetubaguy | mdbooth: damm, yes, still not got out of the specs really | 17:20 |
mdbooth | I'd really like to get that landed, or at least know what needs doing to it. | 17:20 |
mdbooth | It's a bit in limbo right now. | 17:21 |
openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova-specs master: Live Migration of Rescued Instances https://review.openstack.org/347161 | 17:21 |
jaypipes | mriedem, dansmith: your reviews on https://review.openstack.org/#/c/441544/ would be great. thanks in advance! | 17:22 |
johnthetubaguy | siva_krishnan: acutally that spec is really different from the previously approved one: http://specs.openstack.org/openstack/nova-specs/specs/newton/approved/live-migrate-rescued-instances.html | 17:22 |
mriedem | jaypipes: ack | 17:22 |
johnthetubaguy | mdbooth: understood, I hope to get there really soon | 17:22 |
sdague | mriedem: you are going to have to explain the flavor param on https://review.openstack.org/#/c/445236/1to me | 17:22 |
jaypipes | dansmith: what's your most pressing patch for me to review? | 17:22 |
sdague | because I'm not seeing any logic that makes that a different value in the new code than the old code | 17:23 |
dansmith | sdague: it's because of lazy loading | 17:23 |
dansmith | sdague: needing to go to the right cell when that happens | 17:23 |
*** fragatina has joined #openstack-nova | 17:23 | |
dansmith | jaypipes: I'm currently moving code around so.. nothing right now :( | 17:24 |
mriedem | sdague: must did | 17:24 |
mriedem | *just did | 17:24 |
mdbooth | johnthetubaguy: Thanks! | 17:24 |
dansmith | jaypipes: you can look at this if you want: https://review.openstack.org/#/c/427903 | 17:24 |
mriedem | sdague: it's all about cell context | 17:24 |
jaypipes | dansmith: cool, will do. | 17:24 |
dansmith | but it's not super pressing | 17:24 |
dansmith | jaypipes: not sure I have enough context in those commit messages for you to know why that's important actually.. unless you were paying attention at the PTG :) | 17:25 |
siva_krishnan | johnthetubaguy: we have almost changed all the textual content of previously approved spec but functionality wise this was the only change we agreed on. | 17:25 |
mriedem | dansmith: nor the bp description :) | 17:25 |
dansmith | I know | 17:26 |
jaypipes | dansmith: I'll ask if I don't understand :) | 17:26 |
* dansmith can't type any faster | 17:26 | |
mriedem | i'll add notes to the bp | 17:26 |
*** eharney has joined #openstack-nova | 17:26 | |
*** lucasagomes is now known as lucas-afk | 17:26 | |
*** unicell has joined #openstack-nova | 17:27 | |
*** bkopilov has quit IRC | 17:27 | |
mriedem | dansmith: there https://blueprints.launchpad.net/nova/+spec/discover-hosts-faster | 17:29 |
dansmith | mriedem: <3 | 17:30 |
dansmith | jaypipes: ^ | 17:30 |
sdague | mriedem / dansmith ok, so you end up having to change context to look up the flavor, because it's only in the the cell db ? | 17:30 |
mriedem | sdague: the instance is in the cell0 db | 17:30 |
sdague | but then you are making all the quota changes at the api db? | 17:30 |
mriedem | the context, by default, looks in the nova db | 17:30 |
jaypipes | bueno. | 17:30 |
mriedem | no, the quota changes are in the nova db | 17:31 |
mriedem | quotas aren't in the api db yet, which is why we have this bug | 17:31 |
dansmith | well, | 17:31 |
dansmith | not really | 17:31 |
dansmith | but if he buys that, then run with it | 17:31 |
dansmith | oops, was that public? | 17:31 |
*** gabor_antal_ has joined #openstack-nova | 17:31 | |
mriedem | ha | 17:31 |
*** gabor_antal has quit IRC | 17:31 | |
mriedem | you typed too fast | 17:31 |
dansmith | sdague: api hedged and put the quota reservation in the non-cell0 cell the instance was destined for, | 17:32 |
dansmith | but then we put it in cell0 when we couldn't schedule it | 17:32 |
*** kaisers_ has quit IRC | 17:32 | |
sdague | so, I guess I'm not sure I understand why the internals interface doesn't make instance.flavor just go lazy load from the right place | 17:32 |
dansmith | sdague: once our quotas are "fixed for cells" then this won't be a problem | 17:32 |
sdague | or is that just because this was a late breaking bug, and this was the smallest backportable thing? | 17:32 |
dansmith | sdague: because the instance is now in cell0, | 17:33 |
dansmith | sdague: but the reservations were created in cellN in a previous request | 17:33 |
mriedem | "why the internals interface doesn't make instance.flavor just go lazy load from the right place" the instance object would have to lookup it's own instance mapping to find the cell it's in, and then do a lookup using that context i believe, | 17:34 |
dansmith | no, it's worse | 17:34 |
mriedem | which could probably work, but wasn't something i thought of doing, at least in the mainline path | 17:34 |
dansmith | because the reservations are in the cell it's not in, right? | 17:34 |
mriedem | correct, | 17:35 |
sdague | I don't know, I just see you did a lookup call to figure out what cell the instance is in right above it | 17:35 |
mriedem | but i think sdague is asking why i have to be explicit about looking up the instance.flavor in the proper cell, rather than instance.flavor lazy-load doing that for us implicitly | 17:35 |
sdague | so from a naive pov, it seems like that could be stashed in instance itself | 17:35 |
mriedem | we could handle InstanceNotFound in the obj_load_attr in the Instance object and if the context isn't targeted for a cell (no db_connection attr) we could do an instance mapping lookup on ourselves | 17:36 |
dansmith | sdague: because that's not how it works right now | 17:36 |
dansmith | :) | 17:36 |
mriedem | but that's not going to happen in the backport | 17:36 |
mriedem | dansmith: do you think doing a notfound / lookup / re-query in the instance.obj_load_attr is worth looking into so we can avoid explicit context switching like this? | 17:37 |
dansmith | no | 17:37 |
dansmith | the instance object should never know about instancemapping | 17:37 |
mriedem | too magical and hard to debug if it's wrong? | 17:37 |
mriedem | ok | 17:37 |
dansmith | they're in different databases, and that thing will fail if we hit that notfound if we're on a compute for example | 17:37 |
*** bkopilov has joined #openstack-nova | 17:38 | |
mriedem | yeah | 17:38 |
sdague | ok, well this feels kind of fragile, but if that's what we got, so be it | 17:39 |
mriedem | i don't disagree that this is gorpy, the instance delete path has always been bad and this isn't helping | 17:40 |
dansmith | the instance delete path is like some of the worst code we have | 17:40 |
dansmith | right behind BDMs probably :) | 17:40 |
mriedem | what about quotas? | 17:40 |
mriedem | this is quotas + delete | 17:40 |
mriedem | worst ever | 17:40 |
melwitt | local delete | 17:40 |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/nova master: Make versioned notifications topics configurable https://review.openstack.org/444947 | 17:40 |
mriedem | i do think we have some stuff we can cleanup in the delete flow that was meant for the transition to cells v2 | 17:40 |
mriedem | i started trying that cleanup here https://review.openstack.org/#/c/443405/ | 17:41 |
mriedem | but apparently part of it is still needed | 17:41 |
melwitt | that reminds me, I still have a patch up for one of the local delete problems https://review.openstack.org/340614 | 17:41 |
mriedem | melwitt: the 10th revert of the 10th revert? | 17:41 |
melwitt | yeah | 17:42 |
mriedem | dansmith: if you can hit the 2 remaining ocata patches https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/ocata then i can prep the release | 17:43 |
mriedem | i have a date with a 4th notary | 17:43 |
mriedem | jaypipes: that ironic inventory patch is on my list for the afternoon once i get rejected by this other notary | 17:44 |
*** ssurana has quit IRC | 17:44 | |
johnthetubaguy | bauzas: I think there are quite a few unresolved comments from previous patch sets on this one: https://review.openstack.org/#/c/440580 | 17:45 |
*** jaosorior has quit IRC | 17:45 | |
johnthetubaguy | bauzas: am I missing context on another discussion that resolved them? | 17:45 |
*** raunak has quit IRC | 17:45 | |
*** slaweq_ has joined #openstack-nova | 17:48 | |
*** tesseract has quit IRC | 17:49 | |
*** lucasxu has joined #openstack-nova | 17:52 | |
jaypipes | mriedem: cool, thx | 17:53 |
*** browne has joined #openstack-nova | 17:55 | |
*** derekh has quit IRC | 17:55 | |
melwitt | dansmith: on the host_manager thing, I don't think you need to change anything. I'm just missing how the double iterating would have to happen if it were changed | 17:56 |
dansmith | I just explained | 17:56 |
melwitt | from the context in that file, I just wasn't seeing it | 17:56 |
dansmith | and I've already changed it | 17:56 |
dansmith | there's not a double iteration if I put the conditional in the caller, I just thought the point was to avoid the conditional, | 17:57 |
dansmith | and if not, then I don't get the point of moving | 17:57 |
dansmith | but it's done anyway | 17:57 |
melwitt | hm, okay. I thought the point was to avoid the skip all but one in the specific cell case. but I'm probably missing something else | 17:58 |
johnthetubaguy | sfinucan: did you have some pbr fix for this: https://review.openstack.org/#/c/407364 | 17:59 |
*** raunak has joined #openstack-nova | 17:59 | |
*** jpena is now known as jpena|away | 18:00 | |
melwitt | mriedem: it looks like https://bugs.launchpad.net/nova/+bug/1672624 is another regression. I'm working on a local repro | 18:00 |
openstack | Launchpad bug 1672624 in OpenStack Compute (nova) "Ceph volumes attached to local deleted instance could not be correctly handled" [Undecided,New] | 18:00 |
*** faizy_ has quit IRC | 18:00 | |
*** raunak has quit IRC | 18:01 | |
*** ralonsoh has quit IRC | 18:02 | |
dansmith | melwitt: should we halt this and put it in the issues like the other? https://review.openstack.org/#/c/445575/ | 18:03 |
*** ociuhandu has joined #openstack-nova | 18:03 | |
*** pcaruana has quit IRC | 18:03 | |
*** mgagne_ is now known as mgagne | 18:05 | |
melwitt | dansmith: like the other "fix coming soon" one? argh, maybe. I'm not 100% of when this regressed and I haven't repro-d it yet. from the code, it doesn't look like it should be happening | 18:05 |
dansmith | yeah, that | 18:05 |
melwitt | I guess I hesitate about mentioning it in there when I haven't repro'd it yet | 18:06 |
melwitt | I just tried it on an older devstack ceph I have and the local delete worked fine. working on a try with master | 18:07 |
*** rmart04 has joined #openstack-nova | 18:10 | |
*** Sukhdev has joined #openstack-nova | 18:11 | |
*** gabor_antal_ is now known as gabor_antal | 18:13 | |
*** raunak has joined #openstack-nova | 18:15 | |
*** annegentle has joined #openstack-nova | 18:17 | |
*** bvanhav__ has joined #openstack-nova | 18:19 | |
*** abalutoiu has quit IRC | 18:19 | |
*** annegentle has quit IRC | 18:20 | |
*** bvanhav_ has quit IRC | 18:20 | |
johnthetubaguy | cdent: jaypipes: I guess we don't have an api-ref site for placement yet? | 18:20 |
*** annegentle has joined #openstack-nova | 18:20 | |
jaypipes | johnthetubaguy: not yet, no. one is needed for sure. | 18:20 |
*** raunak has quit IRC | 18:22 | |
johnthetubaguy | mriedem: would be nice to get this policy spec moving, folks seem generally happy with the approach now: https://review.openstack.org/#/c/433010/ | 18:24 |
*** rmart04 has quit IRC | 18:24 | |
dansmith | jaypipes: if I make that server_default=0 thing can I assume that just reading existing records will give me zero? | 18:24 |
dansmith | jaypipes: and, does that require writing the table again, or is it smart? | 18:24 |
*** ayogi has quit IRC | 18:25 | |
*** annegentle has quit IRC | 18:25 | |
jaypipes | dansmith: adding server_default="0" will allow you to make the field non-nullable from the start. | 18:25 |
dansmith | jaypipes: I assume that means "yes" to the first part | 18:25 |
jaypipes | dansmith: yes | 18:26 |
jaypipes | :) | 18:26 |
jaypipes | dansmith: and no on the second (no rewriting table) | 18:26 |
dansmith | okay | 18:26 |
*** efoley__ has joined #openstack-nova | 18:26 | |
openstackgerrit | Pushkar Umaranikar proposed openstack/nova-specs master: Stop using Neutron admin token https://review.openstack.org/439890 | 18:27 |
*** lpetrut has joined #openstack-nova | 18:29 | |
*** efoley_ has quit IRC | 18:30 | |
*** claudiub has joined #openstack-nova | 18:32 | |
*** spotz is now known as spotz_zzz | 18:32 | |
*** raunak has joined #openstack-nova | 18:33 | |
*** spotz_zzz is now known as spotz | 18:33 | |
openstackgerrit | Anusha Unnam proposed openstack/nova master: Add keypairs field to InstancePayload https://review.openstack.org/419730 | 18:36 |
*** mkoderer has quit IRC | 18:37 | |
*** aysyd has quit IRC | 18:41 | |
*** timello has quit IRC | 18:41 | |
*** kaisers_ has joined #openstack-nova | 18:43 | |
*** fragatina has quit IRC | 18:44 | |
*** fragatina has joined #openstack-nova | 18:44 | |
openstackgerrit | Maxim Nestratov proposed openstack/nova-specs master: libvirt: reintroduce runtime image type detection https://review.openstack.org/363547 | 18:45 |
*** timello has joined #openstack-nova | 18:45 | |
dansmith | jaypipes: so.. that doesn't seem to work.. when I'm reading the result of the sql create, I don't have a mapped field, and if I try to read it, sqla lazy-loads (and fails in this case) | 18:48 |
openstackgerrit | dane-fichter proposed openstack/nova master: Use cursive for signature verification https://review.openstack.org/351232 | 18:50 |
dansmith | jaypipes: also, the oslo.db model sync checker does't seem to support sever_default | 18:50 |
dansmith | jaypipes: http://pastebin.com/ckkJXUMi | 18:51 |
*** 21WAAS6DN has quit IRC | 18:51 | |
jaypipes | dansmith: guh. ok... lemme +2 that patch then. | 18:52 |
dansmith | jaypipes: I'm kina disappointed, because server_default is what I want | 18:53 |
jaypipes | ya | 18:53 |
jaypipes | dansmith: vote changed :) | 18:53 |
dansmith | that probably needs fixing in oslo.db | 18:54 |
*** amotoki has quit IRC | 18:56 | |
*** penick has quit IRC | 19:04 | |
dansmith | gibi: around by chance? | 19:04 |
openstackgerrit | sean mooney proposed openstack/nova master: remove flake8-import-order for test requirements https://review.openstack.org/445622 | 19:04 |
openstackgerrit | Dan Smith proposed openstack/nova master: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | 19:06 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make os-availability-zones know about cells https://review.openstack.org/442163 | 19:06 |
openstackgerrit | Dan Smith proposed openstack/nova master: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | 19:08 |
*** gszasz has quit IRC | 19:08 | |
*** ociuhandu has quit IRC | 19:09 | |
cdent | johnthetubaguy: I've started but not made much progress: https://review.openstack.org/#/c/409340/ (and one above it). I keep asking for volunteers from the reading audience of the placement update emails, but I think labor is oversubscribed | 19:09 |
*** penick has joined #openstack-nova | 19:09 | |
melwitt | dansmith: just looked at the update. what I was missing was that you needed the entire cell objects in _get_computes_for_cells, and not just the uuids | 19:13 |
dansmith | okay | 19:13 |
bauzas | johnthetubaguy: good points about https://review.openstack.org/#/c/440580/5 I missed them | 19:13 |
melwitt | if you only needed the uuids, then it would have saved something but since you need the whole objects, it doesn't | 19:14 |
*** kaisers_ has quit IRC | 19:15 | |
openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Add live-migration-force-after-timeout spec https://review.openstack.org/438467 | 19:19 |
*** artom has quit IRC | 19:22 | |
*** satyar has quit IRC | 19:22 | |
*** dtp is now known as dtp-afk | 19:24 | |
*** aysyd has joined #openstack-nova | 19:24 | |
*** dtp-afk has quit IRC | 19:24 | |
*** dave-mccowan has quit IRC | 19:26 | |
*** penick has quit IRC | 19:33 | |
*** satyar has joined #openstack-nova | 19:34 | |
*** ociuhandu has joined #openstack-nova | 19:35 | |
*** penick has joined #openstack-nova | 19:36 | |
*** bvanhav__ has quit IRC | 19:40 | |
*** amoralej is now known as amoralej|off | 19:43 | |
*** rmart04 has joined #openstack-nova | 19:48 | |
openstackgerrit | Anusha Unnam proposed openstack/nova master: Add description to policies in volumes.py https://review.openstack.org/445637 | 19:55 |
jroll | mriedem: all the tooz traffic on the ML reminds me to bug you about this again, you had a question there, I answered it, what say you? https://review.openstack.org/#/c/399022/ | 19:55 |
*** penick has quit IRC | 19:58 | |
*** ociuhandu has quit IRC | 20:03 | |
*** lpetrut has quit IRC | 20:05 | |
*** armax has quit IRC | 20:07 | |
*** satyar has quit IRC | 20:07 | |
efried | mriedem Re: "...how the libvirt driver handles connection failures, it will auto-disable the compute service record in the database, and when it reconnects it will auto-enable the compute service record in the DB." - can you give me a pointer to some code to look at (and shamelessly mimic)? | 20:11 |
*** kaisers_ has joined #openstack-nova | 20:11 | |
*** liangy has quit IRC | 20:12 | |
*** david-lyle_ is now known as david-lyle | 20:12 | |
*** r-daneel has joined #openstack-nova | 20:18 | |
openstackgerrit | Juan Antonio Osorio Robles proposed openstack/nova master: Make versioned notifications topics configurable https://review.openstack.org/444947 | 20:18 |
*** rmart04 has quit IRC | 20:25 | |
*** artom has joined #openstack-nova | 20:25 | |
*** ociuhandu has joined #openstack-nova | 20:27 | |
*** smatzek_ has quit IRC | 20:28 | |
openstackgerrit | Sarafraj Singh proposed openstack/nova-specs master: Spec to add per instance timeout for LM https://review.openstack.org/442809 | 20:30 |
*** Matias_ has quit IRC | 20:32 | |
openstackgerrit | Anusha Unnam proposed openstack/nova master: POC: add descriptions to server tags https://review.openstack.org/434842 | 20:32 |
*** romainh1 has quit IRC | 20:34 | |
*** romainh has joined #openstack-nova | 20:34 | |
*** crushil has quit IRC | 20:36 | |
*** dimtruck is now known as zz_dimtruck | 20:38 | |
*** prateek has quit IRC | 20:38 | |
*** liangy has joined #openstack-nova | 20:41 | |
*** dtp-afk has joined #openstack-nova | 20:42 | |
*** dtp-afk is now known as dtp | 20:42 | |
*** penick has joined #openstack-nova | 20:42 | |
*** jaypipes has quit IRC | 20:43 | |
*** crushil has joined #openstack-nova | 20:45 | |
*** armax has joined #openstack-nova | 20:48 | |
mriedem | dansmith: i failed to mention the van in the yard covered in snow | 20:50 |
dansmith | mriedem: not that scary | 20:51 |
mriedem | dansmith: the government center can't do the thing i need, but gene fuckin' turner gets it done | 20:51 |
dansmith | did it have wheels? | 20:51 |
dansmith | hah | 20:51 |
mriedem | gray-faced old ass pug met me at the door | 20:51 |
mriedem | lots of cat food cans and cat hair but no cat in sight | 20:51 |
mriedem | the van had wheels | 20:51 |
mriedem | and rust | 20:51 |
mriedem | gene mothertruckin' turner | 20:51 |
dansmith | nice | 20:51 |
openstackgerrit | Anusha Unnam proposed openstack/nova master: Add descripiton to policies in virtual_interfaces.py https://review.openstack.org/445652 | 20:52 |
mriedem | it was like i saw into jay's future | 20:52 |
dansmith | haha | 20:52 |
mriedem | smelled like the mrs was in the kitchen cooking up that cat food | 20:52 |
dansmith | jay's present is jay's future | 20:52 |
*** antwash has joined #openstack-nova | 20:55 | |
mriedem | edleafe: this is the callback method in the libvirt driver https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L458 | 20:56 |
mriedem | called from the event listener in nova/virt/libvirt/host.py | 20:56 |
mriedem | then the magic happens here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L3396 | 20:57 |
antwash | Hey nova team, I have a review for you that's related to the move of policy in code and add more descriptive policies for deployers. https://review.openstack.org/#/c/439070/ | 20:57 |
antwash | ^ feedback is always welcomed :) | 20:57 |
edleafe | mriedem: thx - will dig into that in a bit | 20:58 |
antwash | ^ johnthetubaguy -- I think this patch is cleaned up nice now lol | 20:58 |
*** adisky_ has quit IRC | 20:59 | |
openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in aggregates.py https://review.openstack.org/445230 | 21:02 |
*** zz_dimtruck is now known as dimtruck | 21:03 | |
*** smatzek has joined #openstack-nova | 21:04 | |
*** ociuhandu has quit IRC | 21:04 | |
*** lucasxu has quit IRC | 21:04 | |
*** lucasxu has joined #openstack-nova | 21:05 | |
openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in admin_password.py https://review.openstack.org/443957 | 21:05 |
openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in admin_password.py https://review.openstack.org/443957 | 21:05 |
*** Apoorva has joined #openstack-nova | 21:06 | |
mriedem | jroll: i have spoken | 21:07 |
jroll | joy! | 21:07 |
*** jpena|away is now known as jpena|off | 21:07 | |
jroll | mriedem: nice, thank you much | 21:08 |
mriedem | i meh'ed all over that thing | 21:08 |
jroll | same | 21:11 |
*** crushil has quit IRC | 21:15 | |
*** crushil has joined #openstack-nova | 21:15 | |
*** rmart04 has joined #openstack-nova | 21:18 | |
*** liangy has quit IRC | 21:19 | |
*** rmart04 has quit IRC | 21:20 | |
*** abalutoiu has joined #openstack-nova | 21:20 | |
*** aysyd has quit IRC | 21:25 | |
*** gouthamr has quit IRC | 21:25 | |
*** crushil has quit IRC | 21:31 | |
*** felipemonteiro has quit IRC | 21:32 | |
*** slaweq_ has quit IRC | 21:33 | |
*** slaweq_ has joined #openstack-nova | 21:33 | |
*** Jeffrey4l has quit IRC | 21:35 | |
*** liangy has joined #openstack-nova | 21:35 | |
*** Jeffrey4l has joined #openstack-nova | 21:35 | |
*** dimtruck is now known as zz_dimtruck | 21:36 | |
*** Sukhdev has quit IRC | 21:37 | |
*** zz_dimtruck is now known as dimtruck | 21:37 | |
*** eharney has quit IRC | 21:38 | |
*** smatzek has quit IRC | 21:39 | |
*** tblakes has quit IRC | 21:44 | |
*** slaweq_ has quit IRC | 21:46 | |
*** catintheroof has quit IRC | 21:46 | |
*** slaweq_ has joined #openstack-nova | 21:47 | |
efried | edleafe I suspect mriedem's pointers to _set_host_enabled were for me ;-) | 21:48 |
efried | If not, I'm interested in talking to you... | 21:48 |
efried | mriedem Hypothetically, if I wanted to use that thing, could I move it out of libvirt and into someplace common? (Rather than duping it) | 21:49 |
efried | (Looks like vmware has a [simpler] version of same) | 21:49 |
*** Sukhdev has joined #openstack-nova | 21:51 | |
mriedem | ha yes they were | 21:52 |
mriedem | sorry ed | 21:52 |
*** edmondsw_ has quit IRC | 21:52 | |
mriedem | efried: so vcenter has a way for active session on startup with the backing vcenter? | 21:53 |
mriedem | *wait | 21:53 |
*** ZZelle has joined #openstack-nova | 21:53 | |
efried | mriedem Well, they have a _set_host_enabled that does the same thing as libvirt's. Haven't yet drilled down into how it's used. | 21:53 |
mriedem | efried: i think we need something like that, otherwise the powervm driver init_host is blocking the nova-compute process from being 'started' for potentially what 10 minutes? | 21:53 |
*** rfolco has quit IRC | 21:53 | |
efried | mriedem Well, yeah, but... | 21:53 |
*** xyang1 has quit IRC | 21:54 | |
mriedem | it does seem sort of weird to start the nova-compute service but automatically disable it, | 21:54 |
mriedem | as noted in the review, i'm not sure if the vcenter driver does anything similar | 21:54 |
mriedem | or how other people feel about this | 21:54 |
efried | mriedem That service *should* start in under a minute, unless the admin misconfigured the nvl (compute host VM) | 21:54 |
efried | And I might have other ways of figuring out if that service is trying to start, or dead in the water. | 21:54 |
ZZelle | Hi everyone | 21:55 |
efried | (ugly-ass native dbus calls equivalent to systemctl status polling) | 21:55 |
*** edmondsw has joined #openstack-nova | 21:55 | |
ZZelle | Who should i ask to review VMware related changes? | 21:55 |
ZZelle | (in nova of course :)) | 21:55 |
efried | mriedem So... if I could get that polling down to, say, two minutes, with a more reliable dead-in-the-water detection, could I get away with avoiding this subthread compute service disable/enable thing? | 21:56 |
mriedem | ZZelle: generally you should start with the vmware subteam, which i think right now is browne | 21:56 |
ZZelle | mriedem, only one person in VMware subteam? | 21:56 |
efried | mriedem ...which, as you say, seems a bit weird - and fairly involved to validate | 21:56 |
browne | ZZelle: sure, just put me on the patch to review | 21:57 |
ZZelle | browne, ok :) | 21:57 |
*** cdent has quit IRC | 21:57 | |
edleafe | efried: ah, ok. Thought it was related to another libvirt thing mriedem and I were discussing | 21:57 |
efried | ZZelle https://etherpad.openstack.org/p/pike-nova-priorities-tracking has a list of subteam members, FYI. | 21:57 |
mriedem | browne: since you're here, do you know if the vcenter driver has to wait at all to make a connection to the backing vcenter on startup? | 21:58 |
efried | ZZelle Not sure if your patch would be appropriate for inclusion in that subteam review list. | 21:58 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: add some documentation around quotas https://review.openstack.org/417563 | 21:58 |
browne | mriedem: i believe there was a bug related to that. where startup of nova-compute was slow because it was connecting to vcenter on startup to retrieve all objects or some effect like that | 21:59 |
browne | let me do a search for it | 21:59 |
efried | mriedem https://github.com/openstack/nova/blob/edf51119fa59ff8a3337abb9107a06fa33d3c68f/nova/virt/vmwareapi/host.py#L49-L118 | 21:59 |
*** edmondsw has quit IRC | 21:59 | |
efried | and browne ^^ | 21:59 |
ZZelle | efried, i should add my changes in "Ready for subteam review:" part? | 22:00 |
efried | mriedem browne Looks like on both init and get_host_stats, it checks and updates the service state. | 22:00 |
efried | ZZelle Not sure if your change set needs to be special to be included there - that's a question for browne et al. | 22:00 |
mriedem | browne: i remember that too i think, but i might also be thinking of a cache issue with the _sync_instance_power_state periodic task in the compute | 22:00 |
ZZelle | efried, ok | 22:00 |
browne | efried: yeah, and i believe someone created a patch to lazy init the state on startup, but wasn't approved | 22:01 |
efried | browne The above code looks like it kinda does that. | 22:01 |
efried | lazy and (kinda) polling | 22:01 |
browne | yeah, so maybe it did get in | 22:01 |
efried | I like how it cheats by using get_host_stats to kick off the recheck. | 22:01 |
efried | I can git blame that sucker to determine the provenance... | 22:02 |
ZZelle | efried, or it's a way to ensure VMware connection during nova-compute-vmware startup? | 22:02 |
*** ociuhandu has joined #openstack-nova | 22:03 | |
efried | ZZelle Yeah, that's the idea. But it doesn't block; it just marks the compute service as down whenever it detects the external connection is dead - and then re-tests it whenever get_host_stats runs. | 22:03 |
efried | mriedem and I are trying to figure out if a similar approach is apppropriate for the PowerVM driver. | 22:03 |
efried | (or if that's more trouble than it's worth) | 22:04 |
ZZelle | efried, iirc (at least in kilo) the nova-compute crashs if it fails to connect to VMware during service startup | 22:04 |
efried | ZZelle And that's what the PowerVM external driver does (after trying to connect for 10 minutes) | 22:05 |
*** efoley__ has quit IRC | 22:06 | |
efried | ZZelle mriedem browne Looks like it happened here https://review.openstack.org/#/c/290315/ as a result of https://launchpad.net/bugs/1316928 | 22:06 |
openstack | Launchpad bug 1316928 in OpenStack Compute (nova) "VMware driver does not change compute node state to unavailable after disconnecting from vCenter" [Medium,Fix released] - Assigned to Giridhar Jayavelu (gjayavelu) | 22:06 |
*** kaisers_ has quit IRC | 22:07 | |
browne | efried: i was thinking of a different bug, but this one looks familiar too | 22:08 |
efried | So mriedem, I'm thinking this is a pretty involved bit of stuff to try to drop into the initial patch. Whaddaya say I knock that timeout down to like two minutes, and develop the state changer thingy in a subsequent change set? | 22:11 |
*** claudiub has quit IRC | 22:11 | |
*** penick has quit IRC | 22:14 | |
*** esberglu has quit IRC | 22:15 | |
*** romainh1 has joined #openstack-nova | 22:16 | |
mriedem | efried: sure, and leave a TODO in the initial patch or whatever | 22:17 |
efried | mriedem Cool beans, thanks. | 22:17 |
*** romainh has quit IRC | 22:17 | |
mriedem | bmace: let's chat here, | 22:17 |
mriedem | bmace: there is a weekly nova/cinder meeting http://eavesdrop.openstack.org/#Cinder,_Nova | 22:17 |
mriedem | bmace: for anyone interested in the multiattach stuff, that's a good meeting to be in for latest status and what needs attention or help | 22:17 |
*** edmondsw has joined #openstack-nova | 22:18 | |
bmace | mriedem: thanks much | 22:18 |
*** gouthamr has joined #openstack-nova | 22:18 | |
mriedem | bmace: we're tracking review priorities per subteam in here https://etherpad.openstack.org/p/pike-nova-priorities-tracking | 22:18 |
mriedem | cinder stuff is at L75 | 22:19 |
mriedem | i'm trying to keep as much of it contained to per-week increments as possible, | 22:19 |
mriedem | so right now the priorities i believe are getting a cinderclient release done so nova can determine the min/max versions from the cinder api, | 22:19 |
mriedem | getting some data model changes into the nova block_device_mappings table (lyarwood's patches), | 22:20 |
mriedem | and reviewing johnthetubaguy's spec and making sure it looks ok | 22:20 |
*** erhudy has quit IRC | 22:20 | |
*** priteau has quit IRC | 22:20 | |
mriedem | so eyes on the spec would be good - the more people looking the better. i don't know how familiar your people are with the bdm attach/detach flows in nova, or the compute manager and block_device code | 22:20 |
mriedem | it gets a bit hairy | 22:20 |
*** edmondsw has quit IRC | 22:22 | |
bmace | mriedem: i have heard some stories from the folks looking at it about the hair ;) | 22:22 |
smcginnis | mriedem: cinderclient is released. I think we need to raise the minimum version though. | 22:25 |
smcginnis | mriedem: But new deployments are likely getting the latest and goodest. | 22:26 |
*** rmart04 has joined #openstack-nova | 22:27 | |
openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - move urlmap.py https://review.openstack.org/444524 | 22:27 |
openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - Combine the two auth.py files https://review.openstack.org/444521 | 22:27 |
openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - Move __init__.py https://review.openstack.org/444522 | 22:27 |
openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - Move wsgi.py https://review.openstack.org/444523 | 22:27 |
openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - Move common.py https://review.openstack.org/445668 | 22:27 |
openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - move api_version_request https://review.openstack.org/445669 | 22:27 |
openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - move compute directory https://review.openstack.org/445670 | 22:27 |
openstackgerrit | Ed Leafe proposed openstack/nova master: WIP - Move the test fakes https://review.openstack.org/445671 | 22:27 |
*** esberglu has joined #openstack-nova | 22:32 | |
mriedem | smcginnis: ok yeah we need the g-r bump then for ildikov's patch to check the max available version in nova | 22:32 |
*** rmart04 has quit IRC | 22:32 | |
*** esberglu has quit IRC | 22:33 | |
*** slaweq__ has joined #openstack-nova | 22:39 | |
*** slaweq_ has quit IRC | 22:40 | |
*** ijw has joined #openstack-nova | 22:45 | |
openstackgerrit | Anusha Unnam proposed openstack/nova master: POC: add descriptions to server tags https://review.openstack.org/434842 | 22:45 |
EmilienM | mriedem: FYI I abandon https://review.openstack.org/#/c/428424/ - you win | 22:46 |
*** rfolco has joined #openstack-nova | 22:54 | |
openstackgerrit | Samantha Blanco proposed openstack/nova master: [WIP] Make flavor-rxtx policy more granular https://review.openstack.org/444036 | 22:57 |
*** slaweq__ has quit IRC | 23:00 | |
*** slaweq__ has joined #openstack-nova | 23:03 | |
*** kaisers_ has joined #openstack-nova | 23:03 | |
*** romainh1 has quit IRC | 23:04 | |
*** kaisers_ has quit IRC | 23:09 | |
*** Swami has quit IRC | 23:11 | |
*** baoli has quit IRC | 23:26 | |
*** rmart04 has joined #openstack-nova | 23:28 | |
*** sdague has quit IRC | 23:29 | |
*** karimb has quit IRC | 23:30 | |
*** dimtruck is now known as zz_dimtruck | 23:31 | |
*** rmart04 has quit IRC | 23:33 | |
*** liangy has quit IRC | 23:38 | |
*** mlavalle has quit IRC | 23:43 | |
*** unicell has quit IRC | 23:44 | |
*** hongbin has quit IRC | 23:45 | |
*** unicell has joined #openstack-nova | 23:47 | |
*** marst has quit IRC | 23:50 | |
*** dtp has quit IRC | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!