| *** takashin has joined #openstack-nova | 00:03 | |
| *** jistr has quit IRC | 00:03 | |
| *** jistr has joined #openstack-nova | 00:07 | |
| *** dave-mcc_ has joined #openstack-nova | 00:08 | |
| *** dave-mccowan has quit IRC | 00:10 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters and response in os-quota-sets.inc https://review.openstack.org/416461 | 00:12 |
|---|---|---|
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix path parameters in os-hypervisors.inc https://review.openstack.org/416911 | 00:12 |
| *** dave-mcc_ has quit IRC | 00:13 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882 | 00:13 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix a parameter in os-availability-zone.inc https://review.openstack.org/417233 | 00:14 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases https://review.openstack.org/394738 | 00:15 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases https://review.openstack.org/395436 | 00:15 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters in os-hosts.inc https://review.openstack.org/416444 | 00:15 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876 | 00:15 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: Amend cold-migration-with-target-ocata spec https://review.openstack.org/397193 | 00:17 |
| *** hfu has joined #openstack-nova | 00:24 | |
| *** salv-orl_ has quit IRC | 00:27 | |
| *** dave-mccowan has joined #openstack-nova | 00:40 | |
| *** thorst has joined #openstack-nova | 00:41 | |
| *** tuanluong has joined #openstack-nova | 00:42 | |
| *** mingyu has joined #openstack-nova | 00:45 | |
| *** thorst has quit IRC | 00:45 | |
| *** thorst has joined #openstack-nova | 00:46 | |
| *** thorst has quit IRC | 00:46 | |
| *** mingyu has quit IRC | 00:48 | |
| *** hoangcx has joined #openstack-nova | 00:50 | |
| *** dimtruck is now known as zz_dimtruck | 00:50 | |
| *** zz_dimtruck is now known as dimtruck | 00:50 | |
| *** dave-mccowan has quit IRC | 00:51 | |
| *** hfu has quit IRC | 00:56 | |
| *** Apoorva has joined #openstack-nova | 00:57 | |
| *** dimtruck is now known as zz_dimtruck | 01:00 | |
| *** phuongnh has joined #openstack-nova | 01:03 | |
| *** claudiub|2 has quit IRC | 01:05 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add some cellsv2 setup docs https://review.openstack.org/420198 | 01:10 |
| *** liusheng has quit IRC | 01:10 | |
| *** gouthamr has joined #openstack-nova | 01:14 | |
| *** Apoorva has quit IRC | 01:17 | |
| *** liusheng has joined #openstack-nova | 01:18 | |
| *** tovin07 has joined #openstack-nova | 01:27 | |
| *** salv-orlando has joined #openstack-nova | 01:28 | |
| *** kevinz has joined #openstack-nova | 01:30 | |
| *** sacharya has joined #openstack-nova | 01:31 | |
| *** hfu has joined #openstack-nova | 01:31 | |
| *** salv-orlando has quit IRC | 01:35 | |
| *** voelzmo has joined #openstack-nova | 01:35 | |
| *** voelzmo has quit IRC | 01:40 | |
| *** gouthamr has quit IRC | 01:41 | |
| *** amotoki has joined #openstack-nova | 01:42 | |
| *** ijw has joined #openstack-nova | 01:43 | |
| *** ijw has quit IRC | 01:47 | |
| *** gcb has joined #openstack-nova | 01:54 | |
| *** liusheng has quit IRC | 01:54 | |
| openstackgerrit | Alex Szarka proposed openstack/nova: Transform instance-live_migration_abort notification https://review.openstack.org/420453 | 01:55 |
| *** yuntongjin has joined #openstack-nova | 01:56 | |
| *** yonglihe has joined #openstack-nova | 01:57 | |
| *** mrda is now known as mrda-lca | 02:00 | |
| *** mrda-lca is now known as mrda | 02:00 | |
| *** armax has joined #openstack-nova | 02:01 | |
| *** armax has quit IRC | 02:01 | |
| *** tiendc has joined #openstack-nova | 02:06 | |
| *** mingyu has joined #openstack-nova | 02:11 | |
| *** nkrinner_afk has quit IRC | 02:12 | |
| *** nkrinner_afk has joined #openstack-nova | 02:12 | |
| *** liusheng has joined #openstack-nova | 02:12 | |
| *** mingyu_ has joined #openstack-nova | 02:14 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 02:14 |
| *** mingyu has quit IRC | 02:15 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964 | 02:15 |
| *** hongbin has joined #openstack-nova | 02:17 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042 | 02:17 |
| *** qianli has joined #openstack-nova | 02:17 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926 | 02:17 |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient: Microversion 2.42 - Enable cold migration with target host https://review.openstack.org/406707 | 02:18 |
| *** qianli has quit IRC | 02:18 | |
| *** stevemar has quit IRC | 02:21 | |
| *** zz_dimtruck is now known as dimtruck | 02:21 | |
| *** stevemar has joined #openstack-nova | 02:21 | |
| *** baoli has joined #openstack-nova | 02:24 | |
| *** links has joined #openstack-nova | 02:25 | |
| *** hfu has quit IRC | 02:28 | |
| *** salv-orlando has joined #openstack-nova | 02:31 | |
| *** mingyu_ has quit IRC | 02:34 | |
| *** salv-orlando has quit IRC | 02:36 | |
| *** yuntongjin has quit IRC | 02:36 | |
| *** hfu has joined #openstack-nova | 02:36 | |
| *** links has quit IRC | 02:38 | |
| *** hfu has quit IRC | 02:38 | |
| *** mingyu has joined #openstack-nova | 02:39 | |
| *** yuntongjin has joined #openstack-nova | 02:45 | |
| *** links has joined #openstack-nova | 02:46 | |
| *** thorst has joined #openstack-nova | 02:47 | |
| *** yuntongjin has quit IRC | 02:49 | |
| *** thorst has quit IRC | 02:52 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: adds REST API for nested providers https://review.openstack.org/384807 | 02:53 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: allow filter providers in tree https://review.openstack.org/377215 | 02:53 |
| openstackgerrit | Jay Pipes proposed openstack/nova: pci: add uuid field to PciDevice object https://review.openstack.org/414768 | 02:53 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: integrate ProviderTree to report client https://review.openstack.org/415921 | 02:53 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: SRIOV PF devices as child providers https://review.openstack.org/414769 | 02:53 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: adds ProviderTree for nested resources https://review.openstack.org/415920 | 02:53 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: update client to set parent provider https://review.openstack.org/385693 | 02:53 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: add nested resource providers https://review.openstack.org/377138 | 02:53 |
| *** dave-mccowan has joined #openstack-nova | 03:02 | |
| *** moshele has joined #openstack-nova | 03:03 | |
| *** dave-mccowan has quit IRC | 03:06 | |
| *** mingyu has quit IRC | 03:06 | |
| *** tlian has quit IRC | 03:13 | |
| *** thorst has joined #openstack-nova | 03:14 | |
| *** thorst has quit IRC | 03:14 | |
| *** mriedem has quit IRC | 03:15 | |
| *** yuntongjin has joined #openstack-nova | 03:16 | |
| *** Jeffrey4l_ has quit IRC | 03:21 | |
| *** Jeffrey4l has joined #openstack-nova | 03:21 | |
| *** yuntongjin has quit IRC | 03:31 | |
| *** pbandark has joined #openstack-nova | 03:32 | |
| *** dave-mccowan has joined #openstack-nova | 03:32 | |
| *** salv-orlando has joined #openstack-nova | 03:32 | |
| *** voelzmo has joined #openstack-nova | 03:36 | |
| *** salv-orlando has quit IRC | 03:36 | |
| *** voelzmo has quit IRC | 03:41 | |
| *** dave-mccowan has quit IRC | 03:45 | |
| openstackgerrit | Tristan Cacqueray proposed openstack/nova: Mask the token used to allow access to consoles https://review.openstack.org/220622 | 03:47 |
| *** sudipto_ has joined #openstack-nova | 03:50 | |
| *** sudipto has joined #openstack-nova | 03:50 | |
| *** ijw has joined #openstack-nova | 03:53 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/os-vif: H803 hacking have been deprecated https://review.openstack.org/420489 | 03:56 |
| *** psachin has joined #openstack-nova | 04:01 | |
| *** bswartz has quit IRC | 04:03 | |
| *** mdnadeem has joined #openstack-nova | 04:05 | |
| *** hongbin has quit IRC | 04:07 | |
| *** hongbin has joined #openstack-nova | 04:07 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Strict pattern match query parameters https://review.openstack.org/420494 | 04:10 |
| *** moshele has quit IRC | 04:10 | |
| *** hongbin has quit IRC | 04:13 | |
| *** yuntongjin has joined #openstack-nova | 04:15 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/os-vif: H803 hacking have been deprecated https://review.openstack.org/420489 | 04:17 |
| *** ijw has quit IRC | 04:22 | |
| *** gouthamr has joined #openstack-nova | 04:24 | |
| *** diga has joined #openstack-nova | 04:30 | |
| *** bswartz has joined #openstack-nova | 04:31 | |
| *** bswartz has quit IRC | 04:32 | |
| *** salv-orlando has joined #openstack-nova | 04:33 | |
| *** udesale has joined #openstack-nova | 04:33 | |
| *** hfu has joined #openstack-nova | 04:35 | |
| *** bswartz has joined #openstack-nova | 04:36 | |
| *** baoli has quit IRC | 04:36 | |
| *** salv-orlando has quit IRC | 04:38 | |
| *** bswartz has quit IRC | 04:44 | |
| *** dimtruck is now known as zz_dimtruck | 04:49 | |
| *** ayogi has joined #openstack-nova | 04:50 | |
| *** moshele has joined #openstack-nova | 04:53 | |
| *** ratailor has joined #openstack-nova | 04:53 | |
| *** bswartz has joined #openstack-nova | 04:54 | |
| *** Sukhdev has joined #openstack-nova | 04:59 | |
| *** sudipto_ has quit IRC | 05:01 | |
| *** sudipto has quit IRC | 05:01 | |
| *** amotoki_ has joined #openstack-nova | 05:09 | |
| *** unicell has joined #openstack-nova | 05:11 | |
| *** amotoki has quit IRC | 05:12 | |
| *** thorst has joined #openstack-nova | 05:15 | |
| *** thorst has quit IRC | 05:20 | |
| *** tiendc_ has joined #openstack-nova | 05:27 | |
| *** prateek has joined #openstack-nova | 05:28 | |
| *** tiendc has quit IRC | 05:28 | |
| *** Apoorva has joined #openstack-nova | 05:29 | |
| *** ecelik has joined #openstack-nova | 05:30 | |
| *** ecelik has left #openstack-nova | 05:30 | |
| *** sgordon has quit IRC | 05:31 | |
| *** salv-orlando has joined #openstack-nova | 05:34 | |
| *** sudipto has joined #openstack-nova | 05:36 | |
| *** sudipto_ has joined #openstack-nova | 05:36 | |
| *** Apoorva has quit IRC | 05:37 | |
| *** sacharya has quit IRC | 05:38 | |
| *** salv-orlando has quit IRC | 05:38 | |
| *** sacharya has joined #openstack-nova | 05:39 | |
| *** sree has joined #openstack-nova | 05:39 | |
| *** sgordon has joined #openstack-nova | 05:39 | |
| openstackgerrit | Tristan Cacqueray proposed openstack/nova: Mask the token used to allow access to consoles https://review.openstack.org/220622 | 05:45 |
| *** fandi has joined #openstack-nova | 05:49 | |
| *** Sukhdev has quit IRC | 05:50 | |
| openstackgerrit | jichenjc proposed openstack/nova: Fix broken link of Doc https://review.openstack.org/420504 | 05:51 |
| *** huanxie has joined #openstack-nova | 05:54 | |
| *** sridharg has joined #openstack-nova | 05:55 | |
| *** ljxiash has joined #openstack-nova | 06:02 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:05 | |
| *** bkopilov_ has joined #openstack-nova | 06:14 | |
| *** claudiub has joined #openstack-nova | 06:17 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | 06:18 |
| *** unicell has quit IRC | 06:22 | |
| *** satyar has joined #openstack-nova | 06:23 | |
| satyar | Hi jeffrey4l | 06:25 |
| Jeffrey4l | hi satyar. ( nova channel ;) ) | 06:26 |
| satyar | opps lets go to our challel :P | 06:26 |
| satyar | channel | 06:26 |
| *** voelzmo has joined #openstack-nova | 06:27 | |
| openstackgerrit | Yoshihiko Atsumi proposed openstack/nova: Add resize server functional negative tests https://review.openstack.org/420514 | 06:28 |
| *** moshele has quit IRC | 06:30 | |
| *** ekuris has joined #openstack-nova | 06:30 | |
| openstackgerrit | jichenjc proposed openstack/nova: Fix broken link of Doc https://review.openstack.org/420504 | 06:32 |
| *** voelzmo has quit IRC | 06:32 | |
| *** ijw has joined #openstack-nova | 06:34 | |
| *** salv-orlando has joined #openstack-nova | 06:35 | |
| *** ijw has quit IRC | 06:39 | |
| *** salv-orlando has quit IRC | 06:39 | |
| *** lpetrut has joined #openstack-nova | 06:42 | |
| *** karthiks has joined #openstack-nova | 06:47 | |
| *** tyrola has quit IRC | 06:47 | |
| *** BobBall has quit IRC | 06:48 | |
| *** tyrola has joined #openstack-nova | 06:49 | |
| *** greghaynes has quit IRC | 06:52 | |
| *** mnaser has quit IRC | 06:53 | |
| *** afazekas has quit IRC | 06:53 | |
| *** mordred has quit IRC | 06:53 | |
| *** afazekas has joined #openstack-nova | 06:57 | |
| *** greghaynes has joined #openstack-nova | 06:58 | |
| *** mordred has joined #openstack-nova | 07:02 | |
| *** afazekas has quit IRC | 07:02 | |
| *** greghaynes has quit IRC | 07:04 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Enable virt.test_virt_drivers.AbstractDriverTestCase on Python 3 https://review.openstack.org/414419 | 07:12 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Enable more virt.xenapi tests on Python 3 https://review.openstack.org/414421 | 07:12 |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Enable virt.vmwareapi test cases on Python https://review.openstack.org/414420 | 07:12 |
| *** Oku_OS-away is now known as Oku_OS | 07:12 | |
| *** afazekas has joined #openstack-nova | 07:13 | |
| *** mnaser has joined #openstack-nova | 07:14 | |
| *** moshele has joined #openstack-nova | 07:14 | |
| *** greghaynes has joined #openstack-nova | 07:15 | |
| *** thorst has joined #openstack-nova | 07:16 | |
| openstackgerrit | huangtianhua proposed openstack/python-novaclient: Make _console() public https://review.openstack.org/414364 | 07:17 |
| *** voelzmo has joined #openstack-nova | 07:21 | |
| *** thorst has quit IRC | 07:21 | |
| *** tesseract has joined #openstack-nova | 07:21 | |
| *** hogepodge_ has joined #openstack-nova | 07:24 | |
| *** nkrinner_afk is now known as nkrinner | 07:24 | |
| *** mnestratov|2 has joined #openstack-nova | 07:26 | |
| *** mdnadeem has quit IRC | 07:27 | |
| *** xinliang has joined #openstack-nova | 07:31 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | 07:31 |
| *** gouthamr has quit IRC | 07:32 | |
| *** markus_z has joined #openstack-nova | 07:32 | |
| *** pcaruana has joined #openstack-nova | 07:34 | |
| *** fandi has quit IRC | 07:35 | |
| *** salv-orlando has joined #openstack-nova | 07:36 | |
| *** yuntongjin has quit IRC | 07:36 | |
| *** hogepodge_ has quit IRC | 07:37 | |
| *** marst has quit IRC | 07:40 | |
| *** marst has joined #openstack-nova | 07:40 | |
| *** salv-orlando has quit IRC | 07:40 | |
| *** andreas_s has joined #openstack-nova | 07:42 | |
| *** mdnadeem has joined #openstack-nova | 07:42 | |
| *** ralonsoh has joined #openstack-nova | 07:43 | |
| *** gcb has quit IRC | 07:44 | |
| *** gcb has joined #openstack-nova | 07:46 | |
| *** abalutoiu has quit IRC | 07:46 | |
| *** migi has joined #openstack-nova | 07:46 | |
| *** roeyc has joined #openstack-nova | 07:51 | |
| *** amotoki has joined #openstack-nova | 08:07 | |
| openstackgerrit | Roman Dobosz proposed openstack/nova-specs: Provide detailed error information for placement API https://review.openstack.org/418393 | 08:10 |
| *** amotoki_ has quit IRC | 08:11 | |
| *** mjura has joined #openstack-nova | 08:12 | |
| snikitin | alex_xu: Hi, Alex! I updated diagnostics patch to fix problem with upgrade. https://review.openstack.org/#/c/355540/ Can you look, please? | 08:16 |
| *** jpena|off is now known as jpena | 08:21 | |
| *** ijw has joined #openstack-nova | 08:22 | |
| *** roeyc has quit IRC | 08:26 | |
| *** ijw has quit IRC | 08:27 | |
| *** trananhkma has joined #openstack-nova | 08:30 | |
| trananhkma | hi nova core reviewers | 08:31 |
| trananhkma | I have 2 patches which have +2 from Stephen Finucane in last week | 08:32 |
| *** slaweq__ has joined #openstack-nova | 08:32 | |
| trananhkma | would you mind helping me review these patches: https://review.openstack.org/#/c/322003/ https://review.openstack.org/#/c/321326/ | 08:32 |
| *** obre has joined #openstack-nova | 08:33 | |
| *** Matias has quit IRC | 08:33 | |
| *** jpena is now known as jpena|off | 08:34 | |
| *** adisky_ has joined #openstack-nova | 08:35 | |
| *** salv-orlando has joined #openstack-nova | 08:37 | |
| *** amoralej|off is now known as amoralej | 08:37 | |
| *** Matias has joined #openstack-nova | 08:40 | |
| *** salv-orlando has quit IRC | 08:41 | |
| *** jpena|off is now known as jpena | 08:42 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova: Add nova-manage cell_v2 create_cell command https://review.openstack.org/332713 | 08:43 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova: handle uninited fields in notification payload https://review.openstack.org/415857 | 08:47 |
| *** mingyu has joined #openstack-nova | 08:48 | |
| openstackgerrit | wanghao proposed openstack/nova: Set min_disk in the image meta based on the root volume's size https://review.openstack.org/407397 | 08:52 |
| openstackgerrit | Andrey Volkov proposed openstack/nova: Add nova-manage cell_v2 create_cell command https://review.openstack.org/332713 | 08:55 |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Stop using mox in virt/xenapi/image/test_glance.py https://review.openstack.org/327457 | 08:57 |
| *** zzzeek has quit IRC | 09:00 | |
| *** zzzeek has joined #openstack-nova | 09:00 | |
| *** houming has joined #openstack-nova | 09:09 | |
| openstackgerrit | Yoshihiko Atsumi proposed openstack/nova: Add resize server functional negative tests https://review.openstack.org/420514 | 09:11 |
| bauzas | p!spy | 09:12 |
| *** ayogi has quit IRC | 09:13 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330 | 09:13 |
| *** thorst has joined #openstack-nova | 09:17 | |
| *** thorst has quit IRC | 09:22 | |
| *** roeyc has joined #openstack-nova | 09:29 | |
| *** yassine has joined #openstack-nova | 09:32 | |
| *** yassine is now known as Guest7011 | 09:32 | |
| *** rha has joined #openstack-nova | 09:33 | |
| *** rha has quit IRC | 09:33 | |
| *** rha has joined #openstack-nova | 09:33 | |
| *** salv-orlando has joined #openstack-nova | 09:37 | |
| *** derekh has joined #openstack-nova | 09:40 | |
| *** sgordon has quit IRC | 09:41 | |
| *** salv-orlando has quit IRC | 09:42 | |
| openstackgerrit | Dao Cong Tien proposed openstack/nova: Stop using mox in compute/test_hypervisors.py https://review.openstack.org/326814 | 09:43 |
| *** mvk has quit IRC | 09:47 | |
| *** lucas-afk is now known as lucasagomes | 09:47 | |
| *** GnomeKing has joined #openstack-nova | 09:48 | |
| *** GnomeKing is now known as BobBall | 09:49 | |
| *** sgordon has joined #openstack-nova | 09:49 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova: WIP: Hyper-V PCI Passthrough https://review.openstack.org/420614 | 09:54 |
| *** sree has quit IRC | 09:55 | |
| *** sree has joined #openstack-nova | 09:56 | |
| *** roeyc has quit IRC | 09:59 | |
| *** sree has quit IRC | 10:00 | |
| *** roeyc has joined #openstack-nova | 10:01 | |
| *** amotoki has quit IRC | 10:01 | |
| *** mnestratov|2 has quit IRC | 10:02 | |
| *** hachi has joined #openstack-nova | 10:04 | |
| *** owalsh has joined #openstack-nova | 10:04 | |
| *** kevinz has quit IRC | 10:06 | |
| openstackgerrit | Béla Vancsics proposed openstack/nova: Reduced the complexity of the _init_instance method (3) https://review.openstack.org/420619 | 10:06 |
| *** tuanluong has quit IRC | 10:07 | |
| *** yonglihe has quit IRC | 10:07 | |
| *** amotoki has joined #openstack-nova | 10:08 | |
| *** hfu has quit IRC | 10:08 | |
| *** phuongnh has quit IRC | 10:10 | |
| *** sree has joined #openstack-nova | 10:10 | |
| *** roeyc has quit IRC | 10:14 | |
| *** hoangcx has quit IRC | 10:14 | |
| *** sree has quit IRC | 10:14 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: workaround findmnt behaviour change https://review.openstack.org/420146 | 10:18 |
| *** mvk has joined #openstack-nova | 10:20 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: workaround findmnt behaviour change https://review.openstack.org/420146 | 10:22 |
| lyarwood | mdbooth: ^ that's the hackaround for the findmnt change we spoke about on Friday btw | 10:23 |
| mdbooth | lyarwood: Did you report a bug against util-linux, btw? | 10:23 |
| *** amotoki has quit IRC | 10:24 | |
| lyarwood | mdbooth: someone already had - https://bugzilla.redhat.com/show_bug.cgi?id=1405238 | 10:24 |
| openstack | bugzilla.redhat.com bug 1405238 in util-linux "findmnt --target behaviour changed in 7.3, shows all mount-points in chroot" [Unspecified,Post] - Assigned to kzak | 10:24 |
| mdbooth | lyarwood: Reading. Might add REGRESSION to the bz title if it's not getting attention. | 10:25 |
| lyarwood | mdbooth: seems to be POST'd for 7.4, I'll ask for 7.3.z now | 10:25 |
| *** udesale has quit IRC | 10:28 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix keypair API parameters https://review.openstack.org/420626 | 10:28 |
| mdbooth | lyarwood: Brilliant, thanks. | 10:31 |
| * mdbooth reads the workaround. | 10:31 | |
| openstackgerrit | Spencer Yu proposed openstack/nova: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/420629 | 10:34 |
| *** satyar has quit IRC | 10:36 | |
| *** Guest7011 has quit IRC | 10:37 | |
| *** salv-orlando has joined #openstack-nova | 10:38 | |
| takashin | bauzas: alex_xu: Are you around? Could you review https://review.openstack.org/#/c/408955/ again and check my comments? | 10:39 |
| openstackgerrit | Béla Vancsics proposed openstack/nova: Reduced the complexity of the _init_instance method (4) https://review.openstack.org/420632 | 10:39 |
| *** alineb has quit IRC | 10:40 | |
| *** salv-orlando has quit IRC | 10:43 | |
| *** voelzmo has quit IRC | 10:43 | |
| mdbooth | lyarwood: Reviewed. Would you like me to add the requested test case and resubmit? It'll only take a couple of minutes. | 10:44 |
| mdbooth | Also the os.stat() question. | 10:45 |
| lyarwood | mdbooth: yup sure | 10:45 |
| mdbooth | Ok, will do. | 10:45 |
| *** Guest7011 has joined #openstack-nova | 10:45 | |
| *** mingyu has quit IRC | 10:47 | |
| openstackgerrit | zhangyujun proposed openstack/nova: Query parameter validation for hosts API https://review.openstack.org/418880 | 10:48 |
| lyarwood | mdbooth: thanks btw, I've answered the os.stat question in the review, https://github.com/openstack/nova/blob/ad1c7ac2b102112280a25927d731edb168f80998/nova/virt/libvirt/volume/quobyte.py#L120 is the reason. | 10:50 |
| *** rmart04 has joined #openstack-nova | 10:50 | |
| openstackgerrit | ChangBo Guo(gcb) proposed openstack/nova: Set access_policy for messaging's dispatcher https://review.openstack.org/408494 | 10:51 |
| lyarwood | hmmmmm this might not work actually | 10:51 |
| lyarwood | mdbooth: what do you make of L115 from the link above? | 10:53 |
| *** moshele has quit IRC | 10:53 | |
| lyarwood | mdbooth: I don't think that was even valid with findmnt | 10:53 |
| mdbooth | lyarwood: I was just reading it more carefully myself. | 10:53 |
| mdbooth | Surely it previously returned false? | 10:53 |
| mdbooth | I don't think it's mocked in the test. | 10:53 |
| *** moshele has joined #openstack-nova | 10:54 | |
| *** cdent has joined #openstack-nova | 10:55 | |
| mdbooth | lyarwood: That depends how quobyte volumes are known to mount | 10:56 |
| lyarwood | mdbooth: I think mounted was False previously in this test so os.stat was nnenver called | 10:56 |
| mdbooth | lyarwood: Right. I suspect it was false because it wasn't mocked, and therefore was never going to work in a test environment | 10:56 |
| mdbooth | I think this test needs it to be false, though. | 10:57 |
| mdbooth | That test is bizarre | 10:57 |
| mdbooth | Still trying to wrap my head round what it's doing | 10:58 |
| mdbooth | It's raising an exception if validate_volume() is called with any arguments? | 10:59 |
| mdbooth | Why? | 10:59 |
| lyarwood | validate_volume is calling command = ['getfattr', "-n", "quobyte.info", mnt_base] so I guess it's just mocking a failure of that | 11:00 |
| lyarwood | so it doesn't really matter what we do in connect_volume as long as the volume is mounted in the end | 11:00 |
| lyarwood | urgh | 11:00 |
| mdbooth | It's a terrible test | 11:00 |
| mdbooth | I wouldn't rathole on it. I'd change the is_mounted mock to return False, because that's the previous behaviour, and remove the os.stat mock. | 11:00 |
| mdbooth | Fixing it isn't what you're doing here. | 11:01 |
| lyarwood | yeah true | 11:01 |
| mdbooth | Would you like me to make that change, too? | 11:01 |
| lyarwood | if you have everything open already | 11:01 |
| mdbooth | Yeah. | 11:01 |
| lyarwood | cool thanks | 11:01 |
| *** lpetrut has quit IRC | 11:03 | |
| *** rfolco has joined #openstack-nova | 11:05 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs: Fix formatting errors and warnings https://review.openstack.org/407364 | 11:06 |
| *** ljxiash has quit IRC | 11:09 | |
| *** sahid has joined #openstack-nova | 11:09 | |
| *** ljxiash has joined #openstack-nova | 11:10 | |
| *** JoseMello has joined #openstack-nova | 11:13 | |
| *** ljxiash has quit IRC | 11:14 | |
| *** erlon_pto has joined #openstack-nova | 11:15 | |
| *** sudipto has quit IRC | 11:17 | |
| *** sudipto_ has quit IRC | 11:17 | |
| mdbooth | lyarwood: Trying trying to work out what that quobyte test did before in case we're missing something. | 11:18 |
| *** thorst has joined #openstack-nova | 11:18 | |
| mdbooth | I wonder if it wasn't doing what the author thought it was doing. | 11:18 |
| openstackgerrit | Chris Dent proposed openstack/nova: [placement] add api-ref for GET /resource_providers https://review.openstack.org/418503 | 11:19 |
| lyarwood | mdbooth: meh, it might be an older test that tested something useful in the past that's just been left in tbh | 11:19 |
| lyarwood | mdbooth: asserting that we raise the exception that's thrown by the final call in the method is just odd | 11:20 |
| *** tiendc_ has quit IRC | 11:20 | |
| lyarwood | mdbooth: what else could we possibly do? | 11:20 |
| mdbooth | Yeah, I wonder if the exception was raise by something else | 11:20 |
| mdbooth | Checking now, shouldn't take too long | 11:20 |
| lyarwood | kk | 11:20 |
| lyarwood | thanks again | 11:20 |
| *** thorst has quit IRC | 11:23 | |
| mdbooth | lyarwood: The previous code returned True, and didn't handle the os.stat() error in the test | 11:25 |
| mdbooth | It returned true because of fake_execute | 11:25 |
| lyarwood | ahhhh | 11:26 |
| *** lpetrut has joined #openstack-nova | 11:26 | |
| mdbooth | The os.stat() error is handled by the surrounding code | 11:26 |
| mdbooth | Yup, the os.stat() mock can just be removed | 11:27 |
| mdbooth | It remains a terrible test | 11:27 |
| mdbooth | lyarwood: You never did add anything to openstack-dev-hacks, btw | 11:30 |
| *** lpetrut has quit IRC | 11:30 | |
| *** ekuris has quit IRC | 11:31 | |
| lyarwood | mdbooth: yeah sorry keep meaning to | 11:31 |
| lyarwood | mdbooth: I'll try to get around to it this morning | 11:31 |
| mdbooth | Don't both spending time cleaning it up, btw, just chuck it in there | 11:31 |
| mdbooth | We can cleanup later if required | 11:31 |
| lyarwood | yeah I wanted to remove the grep foo | grep bar crap etc ;) | 11:32 |
| mdbooth | Meh, just drop it in there ;) | 11:32 |
| openstackgerrit | Matthew Booth proposed openstack/nova: libvirt: workaround findmnt behaviour change https://review.openstack.org/420146 | 11:32 |
| *** sree has joined #openstack-nova | 11:34 | |
| *** amotoki has joined #openstack-nova | 11:37 | |
| *** salv-orlando has joined #openstack-nova | 11:39 | |
| *** alineb- has joined #openstack-nova | 11:41 | |
| *** voelzmo has joined #openstack-nova | 11:41 | |
| *** alineb- is now known as alineb | 11:42 | |
| *** salv-orlando has quit IRC | 11:43 | |
| *** ekuris has joined #openstack-nova | 11:45 | |
| lyarwood | mdbooth: pushed, also dumped my current crop of git aliases in there, not sure it's that useful so feel free to rm | 11:46 |
| *** ratailor has quit IRC | 11:46 | |
| markus_z | Would someone please review a tempest test for l-m: https://review.openstack.org/#/c/346815/22 ? | 11:47 |
| *** roeyc has joined #openstack-nova | 11:49 | |
| markus_z | tonyb: Would you have a look at the newton backports for bug 1455252 please? https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/newton+topic:%22bug+1455252%22 | 11:51 |
| openstack | bug 1455252 in OpenStack Compute (nova) "enabling serial console breaks live migration" [High,Fix released] https://launchpad.net/bugs/1455252 - Assigned to Markus Zoeller (markus_z) (mzoeller) | 11:51 |
| lyarwood | markus_z: I can take a look this afternoon | 11:52 |
| markus_z | lyarwood: Thanks, that would be awesome. | 11:52 |
| *** voelzmo has quit IRC | 11:55 | |
| *** voelzmo has joined #openstack-nova | 11:56 | |
| mdbooth | lyarwood: Thanks! | 11:57 |
| *** bkopilov_ has quit IRC | 11:57 | |
| *** bkopilov_ has joined #openstack-nova | 11:58 | |
| sahid | johnthetubaguy: hello | 11:59 |
| sahid | i remember that you were kind of interested about the feature emulator threads | 11:59 |
| *** lpetrut has joined #openstack-nova | 11:59 | |
| sahid | and unfortunatly even if the code has been pushed 2 months ago I did not had a lot of reviews | 11:59 |
| sahid | would you be interested on helping jay pipes to review it ? | 12:00 |
| *** jbernard has quit IRC | 12:05 | |
| *** ljxiash has joined #openstack-nova | 12:05 | |
| *** rtheis has joined #openstack-nova | 12:07 | |
| *** jbernard has joined #openstack-nova | 12:11 | |
| *** dikonoor has joined #openstack-nova | 12:13 | |
| openstackgerrit | Chris Dent proposed openstack/nova: Add rudimentary CORS support to placement API https://review.openstack.org/392891 | 12:13 |
| *** ociuhandu has quit IRC | 12:15 | |
| *** smatzek has joined #openstack-nova | 12:17 | |
| *** ociuhandu has joined #openstack-nova | 12:19 | |
| *** bvanhav_ has joined #openstack-nova | 12:19 | |
| *** kevinz has joined #openstack-nova | 12:20 | |
| *** kevinz has quit IRC | 12:21 | |
| *** ekuris has quit IRC | 12:21 | |
| *** sdague has joined #openstack-nova | 12:22 | |
| *** krtaylor has quit IRC | 12:29 | |
| *** sudipto has joined #openstack-nova | 12:31 | |
| *** sudipto_ has joined #openstack-nova | 12:31 | |
| *** dikonoor has quit IRC | 12:34 | |
| *** ekuris has joined #openstack-nova | 12:34 | |
| *** dikonoor has joined #openstack-nova | 12:34 | |
| *** sahid has quit IRC | 12:34 | |
| *** nicolasbock has joined #openstack-nova | 12:34 | |
| *** sudipto_ has quit IRC | 12:37 | |
| *** sudipto has quit IRC | 12:37 | |
| *** khushbu_ has joined #openstack-nova | 12:37 | |
| *** sudipto has joined #openstack-nova | 12:40 | |
| *** salv-orlando has joined #openstack-nova | 12:40 | |
| *** sudipto_ has joined #openstack-nova | 12:41 | |
| *** sahid has joined #openstack-nova | 12:41 | |
| *** takashin has left #openstack-nova | 12:42 | |
| *** jpena is now known as jpena|lunch | 12:44 | |
| *** salv-orlando has quit IRC | 12:44 | |
| *** thorst has joined #openstack-nova | 12:45 | |
| *** gszasz has joined #openstack-nova | 12:45 | |
| *** lucasagomes is now known as lucas-hungry | 12:45 | |
| *** salv-orlando has joined #openstack-nova | 12:46 | |
| *** sahid has quit IRC | 12:48 | |
| *** sahid has joined #openstack-nova | 12:49 | |
| *** houming has quit IRC | 12:49 | |
| *** sahid has quit IRC | 12:50 | |
| *** sahid has joined #openstack-nova | 12:51 | |
| lyarwood | sfinucan: would you mind taking a look at https://review.openstack.org/#/c/420146/ if you have a chance this afternoon | 12:57 |
| *** sree has quit IRC | 12:57 | |
| *** sree has joined #openstack-nova | 12:58 | |
| *** gcb has quit IRC | 13:01 | |
| *** sree has quit IRC | 13:03 | |
| *** moshele has quit IRC | 13:03 | |
| *** moshele has joined #openstack-nova | 13:04 | |
| *** tblakes has joined #openstack-nova | 13:05 | |
| *** beagles_afk is now known as beagles | 13:08 | |
| *** weshay_mod is now known as weshay | 13:10 | |
| *** Cristina_ is now known as CristinaPauna | 13:12 | |
| *** Guest7011 has quit IRC | 13:13 | |
| *** khushbu_ has quit IRC | 13:16 | |
| *** jaypipes has joined #openstack-nova | 13:19 | |
| *** Guest7011 has joined #openstack-nova | 13:20 | |
| *** amoralej is now known as amoralej|lunch | 13:21 | |
| *** coreywright has quit IRC | 13:21 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: ephemeral disk support for virtuozzo containers https://review.openstack.org/312488 | 13:21 |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: ephemeral disk support for virtuozzo containers https://review.openstack.org/312488 | 13:25 |
| dims | sdague : bauzas : Got some time for a review that enable *all* functional tests in py35 (except 1) https://review.openstack.org/#/c/419968/ ? please let me know | 13:26 |
| *** coreywright has joined #openstack-nova | 13:26 | |
| dims | cdent : the lone hold out is the snowman/gabbi test (unicode_get_a_raw_snowman_unicode.test_request) fails as the snowman char is not ascii, so never reaches the server side | 13:27 |
| *** liverpooler has joined #openstack-nova | 13:28 | |
| dikonoor | alex_xu: sdague: Hi..I have some queries around quotas..who would be the right person to contact ? | 13:29 |
| *** mnestratov has joined #openstack-nova | 13:29 | |
| *** krtaylor has joined #openstack-nova | 13:30 | |
| *** sahid_ has joined #openstack-nova | 13:30 | |
| cdent | dims: odd. i wonder which part of the stack is messing with that, because python3 tests in gabbi itself are pretty okay with that sort of thing, can you link me to a failing trace? | 13:30 |
| *** mdrabe has joined #openstack-nova | 13:31 | |
| *** mdrabe has quit IRC | 13:31 | |
| dims | one sec, digging up | 13:31 |
| *** sahid has quit IRC | 13:32 | |
| dims | cdent : http://logs.openstack.org/68/419968/9/check/gate-nova-tox-db-functional-py35-ubuntu-xenial/dd20ee5/testr_results.html.gz | 13:32 |
| cdent | thanks | 13:32 |
| sdague | dims: lgtm, maybe once this is all done some of those conditional utf8 encode functions could get an oslo safe_encode wrapper or the like | 13:33 |
| sdague | just to get the if PY2 flags out of the code | 13:33 |
| *** erlon_pto is now known as erlon | 13:33 | |
| cdent | dims: that's bizarre, I need to compare with a similar test in gabbi itself | 13:33 |
| dims | cdent : ++ | 13:34 |
| *** mingyu has joined #openstack-nova | 13:34 | |
| *** psachin has quit IRC | 13:34 | |
| *** moshele has quit IRC | 13:39 | |
| cdent | dims: actually that test is bogus on python3 and the error is essentially saying "yo, you've made a bad url" | 13:40 |
| cdent | gabbi is not responsible for url encoding and while python2 is okay with that badly formed url, python3 is not | 13:40 |
| *** claudiub|2 has joined #openstack-nova | 13:40 | |
| cdent | dims: so I would say: okay to remove that one test as it is bogus | 13:41 |
| *** vladikr has joined #openstack-nova | 13:42 | |
| *** claudiub has quit IRC | 13:43 | |
| *** xyang1 has joined #openstack-nova | 13:44 | |
| *** baoli has joined #openstack-nova | 13:45 | |
| *** salv-orlando has quit IRC | 13:46 | |
| *** mdnadeem has quit IRC | 13:46 | |
| *** mdrabe has joined #openstack-nova | 13:47 | |
| dims | cdent : ++ | 13:47 |
| *** moshele has joined #openstack-nova | 13:48 | |
| *** mingyu has quit IRC | 13:51 | |
| *** sacharya has quit IRC | 13:52 | |
| openstackgerrit | sahid proposed openstack/nova: numa: add numa constraints for emulator threads policy https://review.openstack.org/385023 | 13:55 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | 13:55 |
| openstackgerrit | sahid proposed openstack/nova: objects: add new field cpus_reserved in NUMACell https://review.openstack.org/400874 | 13:55 |
| openstackgerrit | sahid proposed openstack/nova: numa: fit instance topology with cpus reserved onto cores https://review.openstack.org/400875 | 13:55 |
| openstackgerrit | sahid proposed openstack/nova: virt: update overhead to take into account vCPUs https://review.openstack.org/385024 | 13:55 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | 13:55 |
| openstackgerrit | sahid proposed openstack/nova: numa: take into account cpus reserved https://review.openstack.org/400876 | 13:55 |
| openstackgerrit | sahid proposed openstack/nova: numa: update numa usage incording the cpu reserved https://review.openstack.org/400877 | 13:55 |
| *** lucas-hungry is now known as lucasagomes | 13:55 | |
| lyarwood | sdague: Morning, https://review.openstack.org/#/c/374373/ is +W but I'm still working on the Nova change it depends on, rechecking this isn't going to cause any issues correct? | 13:55 |
| *** tlian has joined #openstack-nova | 13:55 | |
| openstackgerrit | Vladyslav Drok proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756 | 13:57 |
| *** bvanhav__ has joined #openstack-nova | 13:58 | |
| *** bvanhav__ is now known as bvanhav | 13:58 | |
| *** bvanhav_ has quit IRC | 13:59 | |
| *** baoli has quit IRC | 13:59 | |
| *** jpena|lunch is now known as jpena | 13:59 | |
| sahid_ | jaypipes: hello, btw i have addressed your comments here https://review.openstack.org/#/c/400875/18, not sure if you noticed that | 13:59 |
| diga | jaypipes: Hi | 13:59 |
| diga | Hi | 13:59 |
| *** diga has quit IRC | 13:59 | |
| jaypipes | sahid_: no, I hadn't noticed (was in CA all last week and spent yesterday getting my own patch series rebased) :) | 14:00 |
| *** jwcroppe_ has quit IRC | 14:00 | |
| *** diga has joined #openstack-nova | 14:00 | |
| jaypipes | sahid_: will review yours later today after scheduler meeting. | 14:00 |
| jaypipes | diga: heya | 14:00 |
| diga | jaypipes: Hi | 14:00 |
| sahid_ | jaypipes: i see no problem, thanks a lot | 14:01 |
| diga | jaypipes: ohh, meeting is started I guess | 14:01 |
| sdague | lyarwood: correct | 14:04 |
| *** baoli has joined #openstack-nova | 14:09 | |
| lyarwood | sdague: thanks, just wanted to check | 14:10 |
| *** prateek has quit IRC | 14:11 | |
| *** jwcroppe has joined #openstack-nova | 14:13 | |
| *** paw has quit IRC | 14:13 | |
| *** dave-mccowan has joined #openstack-nova | 14:14 | |
| *** esberglu has joined #openstack-nova | 14:14 | |
| *** jheroux has joined #openstack-nova | 14:17 | |
| *** pradk has joined #openstack-nova | 14:18 | |
| *** mkoderer has joined #openstack-nova | 14:20 | |
| hachi | Hello | 14:26 |
| hachi | Can you please confirm whether the following nova blueprint works with Nova API 2.1: | 14:26 |
| hachi | https://blueprints.launchpad.net/nova/+spec/user-id-based-policy-enforcement | 14:26 |
| hachi | ? | 14:27 |
| *** hongbin has joined #openstack-nova | 14:27 | |
| *** Guest7011 has quit IRC | 14:28 | |
| dims | jaypipes : can you please peek at this? https://review.openstack.org/#/c/419968/ (enables all the functional tests for py3, has +2 from sdague) | 14:29 |
| *** links has quit IRC | 14:30 | |
| dims | jaypipes : welcome back :) | 14:30 |
| jaypipes | dims: thx :) | 14:30 |
| jaypipes | dims: reviewing shortly. just wrapping up scheduler meeting. | 14:31 |
| dims | ack thanks | 14:31 |
| *** owalsh has quit IRC | 14:32 | |
| *** mingyu has joined #openstack-nova | 14:36 | |
| *** mriedem has joined #openstack-nova | 14:38 | |
| mriedem | o/ | 14:39 |
| *** ekuris has quit IRC | 14:40 | |
| *** Guest7011 has joined #openstack-nova | 14:42 | |
| *** salv-orlando has joined #openstack-nova | 14:47 | |
| *** baoli has quit IRC | 14:47 | |
| *** pbandark has quit IRC | 14:47 | |
| *** baoli has joined #openstack-nova | 14:47 | |
| *** gcb has joined #openstack-nova | 14:50 | |
| *** salv-orlando has quit IRC | 14:51 | |
| *** pbandark has joined #openstack-nova | 14:51 | |
| *** mingyu has quit IRC | 14:51 | |
| *** dikonoor has quit IRC | 14:51 | |
| *** owalsh has joined #openstack-nova | 14:52 | |
| *** mingyu has joined #openstack-nova | 14:53 | |
| *** pbandark has quit IRC | 14:54 | |
| *** ekuris has joined #openstack-nova | 14:55 | |
| *** paw has joined #openstack-nova | 14:57 | |
| *** beagles is now known as beagles_brb | 14:57 | |
| *** mtanino has joined #openstack-nova | 15:01 | |
| jaypipes | dims: done | 15:05 |
| dims | thanks jaypipes ! | 15:05 |
| jaypipes | any time! :) | 15:05 |
| *** hongbin has quit IRC | 15:08 | |
| *** roeyc has quit IRC | 15:11 | |
| jaypipes | cdent: k, I have read your email about can_host. | 15:11 |
| *** markus_z has quit IRC | 15:11 | |
| jaypipes | cdent: you want me to respond or should we discuss in hangout? | 15:11 |
| jaypipes | cdent: or both? :) | 15:11 |
| cdent | both would be awesome | 15:11 |
| jaypipes | k. | 15:11 |
| cdent | because in the hangout it will just be whoever is there | 15:11 |
| cdent | although, if your answer is sufficiently informative in writing I'm not sure we need the hangout. that was mriedem's idea | 15:12 |
| bauzas | cdent: jaypipes: just a question, should we need to wait for edleafe since he's on PTO ? | 15:12 |
| mriedem | we're not going to wait no | 15:12 |
| mriedem | jay and i are both out next week | 15:13 |
| cdent | mriedem: can you (re-)share the hangout link | 15:14 |
| bauzas | okay | 15:14 |
| *** zz_dimtruck is now known as dimtruck | 15:14 | |
| bauzas | I'm fine with that, just asking | 15:14 |
| mriedem | alaski: happy birthday :) | 15:14 |
| mriedem | MLK steals your thunder | 15:15 |
| mriedem | cdent: https://hangouts.google.com/hangouts/_/calendar/bWF0dHJpZWQyMkBnbWFpbC5jb20.uve66g1e93k1ooj42jr29g1bug?authuser=0 | 15:15 |
| cdent | thanks | 15:15 |
| jaypipes | bauzas: no rest for the weary I'm afraid. | 15:16 |
| *** edmondsw has joined #openstack-nova | 15:20 | |
| *** jaosorior has joined #openstack-nova | 15:25 | |
| *** satyar has joined #openstack-nova | 15:25 | |
| *** beekneemech is now known as bnemec | 15:26 | |
| *** edmondsw has quit IRC | 15:30 | |
| *** cdent_ has joined #openstack-nova | 15:31 | |
| *** cdent has quit IRC | 15:31 | |
| *** marst has quit IRC | 15:31 | |
| *** cdent_ is now known as cdent | 15:31 | |
| *** yellow has joined #openstack-nova | 15:32 | |
| openstackgerrit | sahid proposed openstack/nova: libvirt: make coherent logs when reboot success https://review.openstack.org/416994 | 15:32 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: rework _destroy method https://review.openstack.org/416995 | 15:32 |
| openstackgerrit | sahid proposed openstack/nova: libvirt: rework hard reboot https://review.openstack.org/376350 | 15:32 |
| *** ijw has joined #openstack-nova | 15:34 | |
| *** nkrinner is now known as nkrinner_afk | 15:35 | |
| yellow | ping #openstack-nova: Looking for the best way to clean a nova db. I see alot of error messages where the domain UUID is not found .. | 15:35 |
| ildikov | mriedem: johnthetubaguy: hi. just a quick check, will you be able to make it to the Cinder-Nova meeting today? | 15:35 |
| mriedem | probably | 15:36 |
| yellow | On mitaka I see the addition of nova-manage db command but its not useful for my situation since the UUID is not null | 15:36 |
| ildikov | mriedem: cool, tnx | 15:36 |
| *** cdent has quit IRC | 15:36 | |
| *** cdent_ has joined #openstack-nova | 15:37 | |
| *** amoralej|lunch is now known as amoralej | 15:37 | |
| *** ijw has quit IRC | 15:39 | |
| *** beagles_brb is now known as beagles | 15:41 | |
| *** cdent_ has quit IRC | 15:42 | |
| *** marst has joined #openstack-nova | 15:43 | |
| *** ekuris has quit IRC | 15:45 | |
| *** gouthamr has joined #openstack-nova | 15:48 | |
| *** salv-orlando has joined #openstack-nova | 15:48 | |
| *** hongbin has joined #openstack-nova | 15:49 | |
| *** dave-mccowan has quit IRC | 15:50 | |
| *** cdent has joined #openstack-nova | 15:50 | |
| *** mvk has quit IRC | 15:50 | |
| *** rtheis has quit IRC | 15:51 | |
| *** ducttape_ has joined #openstack-nova | 15:52 | |
| *** lpetrut has quit IRC | 15:52 | |
| *** salv-orlando has quit IRC | 15:52 | |
| mriedem | jaypipes: this is what i was talking about https://github.com/openstack/nova/blob/7e0edc67108b3b9d23549a6d3c0d3106dfc2acec/nova/objects/resource_provider.py#L563 - but as noted on the call, it's not in allowed_filters (or the schema now) | 15:52 |
| jaypipes | mriedem: ah. well, yeah, it could never have been added via the API or exposed to it. | 15:53 |
| mriedem | jaypipes: you said you need to rebase this? https://review.openstack.org/#/c/407309/ | 15:54 |
| jaypipes | mriedem: yeah. | 15:54 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: live-mig: Add claims and proper resource tracking https://review.openstack.org/244489 | 15:55 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: libvirt: pass libvirt reference methods to migration https://review.openstack.org/328997 | 15:55 |
| openstackgerrit | Pawel Koniszewski proposed openstack/nova: libvirt: live-migrate updates NUMA and cpus in the XML https://review.openstack.org/286744 | 15:55 |
| *** ducttape_ has quit IRC | 15:55 | |
| *** ratailor has joined #openstack-nova | 15:56 | |
| *** abalutoiu has joined #openstack-nova | 15:56 | |
| *** amotoki has quit IRC | 15:57 | |
| *** eharney has joined #openstack-nova | 15:58 | |
| bauzas | jaypipes: refrain your pencil, I'm litterally rebasing now | 15:58 |
| bauzas | jaypipes: that will still make Jenkins unhappy, but at least you'd get a fresh copy | 15:59 |
| cdent | I'm going to emboss "refrain your pencil" somewhere. Has a very nice ring to it. | 15:59 |
| *** cdent has quit IRC | 16:00 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add ComputeNode.get_by_uuid method https://review.openstack.org/418134 | 16:04 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | 16:04 |
| bauzas | jaypipes: mriedem: there it is | 16:04 |
| *** eharney has quit IRC | 16:05 | |
| *** dave-mccowan has joined #openstack-nova | 16:05 | |
| *** Oku_OS is now known as Oku_OS-away | 16:06 | |
| mriedem | bauzas: ok issue in the bottom change | 16:06 |
| bauzas | snap snap | 16:06 |
| bauzas | mriedem: roger, this. | 16:07 |
| bauzas | mriedem: on it | 16:07 |
| *** dave-mcc_ has joined #openstack-nova | 16:08 | |
| *** cdent has joined #openstack-nova | 16:08 | |
| mriedem | do we have a migration that added a uuid to every existing compute_nodes record? | 16:08 |
| *** moshele has quit IRC | 16:08 | |
| mriedem | cdent: in honor of you calling out my haggard looks i've brewed another cup of coffee | 16:08 |
| cdent | I am your enabler? | 16:09 |
| mriedem | yes | 16:09 |
| cdent | Is that a paid position? | 16:09 |
| mriedem | now when i get the caffeine shakes i know who to blame | 16:10 |
| openstackgerrit | Andrey Volkov proposed openstack/nova: Add nova-manage cell_v2 create_cell command https://review.openstack.org/332713 | 16:10 |
| *** dave-mccowan has quit IRC | 16:10 | |
| openstackgerrit | Vasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413 | 16:11 |
| openstackgerrit | Vasyl Saienko proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756 | 16:11 |
| mriedem | ah migration 330 | 16:11 |
| mriedem | cool | 16:11 |
| * cdent gets air | 16:12 | |
| bauzas | mriedem: oh oops, sorry missed your question | 16:12 |
| bauzas | mriedem: yeah, it was done AFAIK | 16:12 |
| *** burgerk has joined #openstack-nova | 16:12 | |
| *** rama_y has joined #openstack-nova | 16:13 | |
| openstackgerrit | Andrey Volkov proposed openstack/nova: Add nova-manage cell_v2 create_cell command https://review.openstack.org/332713 | 16:13 |
| *** voelzmo has quit IRC | 16:13 | |
| melwitt | mriedem: I had thought the map_cell0 name/connection was intentionally based on the api db because they're expected to be co-located together | 16:15 |
| *** voelzmo has joined #openstack-nova | 16:15 | |
| *** jose-phillips has joined #openstack-nova | 16:16 | |
| *** jose-phillips has quit IRC | 16:16 | |
| *** mingyu has quit IRC | 16:17 | |
| *** eharney has joined #openstack-nova | 16:18 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: Add ComputeNode.get_by_uuid method https://review.openstack.org/418134 | 16:19 |
| openstackgerrit | Sylvain Bauza proposed openstack/nova: WIP: Scheduler calling the Placement API https://review.openstack.org/417961 | 16:19 |
| gibi | mriedem: hi! I managed finish the notification lazy load field bugfix. I didn't manage to split it up nicely. If you have time could you please check it? https://review.openstack.org/#/c/415857/ | 16:20 |
| *** bvanhav has quit IRC | 16:22 | |
| *** mingyu has joined #openstack-nova | 16:23 | |
| *** bvanhav has joined #openstack-nova | 16:23 | |
| openstackgerrit | Mikhail Feoktistov proposed openstack/nova: libvirt: ephemeral disk support for virtuozzo containers https://review.openstack.org/312488 | 16:23 |
| *** kfarr has joined #openstack-nova | 16:23 | |
| *** Apoorva has joined #openstack-nova | 16:24 | |
| *** armax has joined #openstack-nova | 16:26 | |
| *** mingyu has quit IRC | 16:27 | |
| *** mingyu has joined #openstack-nova | 16:27 | |
| openstackgerrit | Vasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413 | 16:28 |
| openstackgerrit | Vasyl Saienko proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756 | 16:28 |
| *** ratailor has quit IRC | 16:29 | |
| mriedem | melwitt: i'm not sure, we'd probably have to ask alaski if he remembers a specific reason for calling it nova_api_cell0 even though it's using the main db schema | 16:29 |
| *** salv-orlando has joined #openstack-nova | 16:30 | |
| melwitt | that was my understanding anyway | 16:31 |
| openstackgerrit | Vasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413 | 16:31 |
| openstackgerrit | Vasyl Saienko proposed openstack/nova: Generate necessary network metadata for ironic port groups https://review.openstack.org/388756 | 16:31 |
| *** kiran-r has joined #openstack-nova | 16:32 | |
| *** slaweq_ has joined #openstack-nova | 16:33 | |
| *** ducttape_ has joined #openstack-nova | 16:34 | |
| mriedem | melwitt: still, co-locating the nova_api db and cell0 doesn't mean we should name it nova_api_cell0 right? unless it just meant something like, cell0 is only used at the api/controller level | 16:36 |
| mriedem | idk | 16:36 |
| mriedem | it's probably confusing either way | 16:36 |
| *** slaweq_ has quit IRC | 16:37 | |
| mriedem | we should probably figure out what we want though before all of dansmith's changes for it go in | 16:37 |
| *** slaweq_ has joined #openstack-nova | 16:37 | |
| *** mlavalle has joined #openstack-nova | 16:38 | |
| melwitt | mriedem: yeah, I could see the name being confusing. I'm trying to think if there's any caveat to making it default to the main db connection. I thought the idea was it's an api cell construct, so default to the connection of the other api cell db | 16:39 |
| mriedem | melwitt: you want to comment on https://review.openstack.org/#/c/420439/ ? | 16:41 |
| mriedem | it's held up by dependencies right now | 16:41 |
| melwitt | okay | 16:41 |
| mriedem | it would be nice if we didn't have to make the change because it's also in devstack and grenade, and potentially backports | 16:41 |
| *** gcb has quit IRC | 16:42 | |
| mriedem | but this comment in the code was wrong: "The cell0 database will use the same database scheme and netloc as the API database" | 16:42 |
| mriedem | which leads me to believe it was just a mistake from the beginning | 16:42 |
| *** sahid has joined #openstack-nova | 16:44 | |
| *** sahid has quit IRC | 16:44 | |
| *** Apoorva has quit IRC | 16:44 | |
| *** sahid has joined #openstack-nova | 16:44 | |
| *** sahid_ has quit IRC | 16:45 | |
| *** andreas_s has quit IRC | 16:45 | |
| *** kfarr has quit IRC | 16:49 | |
| *** sridharg has quit IRC | 16:50 | |
| melwitt | mriedem: I was thinking about that and I think it meant scheme as in, url format. not database schema | 16:51 |
| *** jose-phillips has joined #openstack-nova | 16:52 | |
| melwitt | I left a comment on the review, fwiw | 16:52 |
| mriedem | oh maybe | 16:52 |
| *** voelzmo has quit IRC | 16:52 | |
| mdbooth | http://eventlet.net/doc/modules/greenthread.html#eventlet.greenthread.spawn | 16:53 |
| mdbooth | Is there any way to deterministically get a GreenThread object before it starts running? | 16:53 |
| *** jose-phi_ has joined #openstack-nova | 16:53 | |
| mdbooth | I want to do: | 16:54 |
| mdbooth | x = GreenThread() | 16:54 |
| mdbooth | ...register thread somewhere... | 16:54 |
| mdbooth | x.run() | 16:54 |
| *** jose-phillips has quit IRC | 16:54 | |
| mriedem | mnaser: i'm assumed at your '99% of this volume multiattach stuff is done on the cinder side' comment in the ops list :) | 16:55 |
| mnaser | mriedem hi | 16:55 |
| mnaser | yeah, i think cinder side has multiattachments already setup | 16:55 |
| mnaser | APIs have all that stuff already wired up | 16:55 |
| *** MarcF has joined #openstack-nova | 16:56 | |
| mnaser | i think the part that never merged was the nova side that made the appropriate api calls and interacted with it | 16:56 |
| *** MarcF has quit IRC | 16:56 | |
| mriedem | mnaser: yes i'm fully aware of the history | 16:56 |
| mriedem | but it's not that simple | 16:56 |
| mnaser | mriedem: of course it's not :) it's probably why it never got finished up, i'm very understanding of that | 16:56 |
| *** weshay is now known as weshay_lunch | 16:56 | |
| mnaser | sorry if that came off as "nova team, finish your job" .. i understand there's complexities involved but i meant it more of "half of the battle is done" | 16:57 |
| *** amotoki has joined #openstack-nova | 16:57 | |
| mriedem | yeah i know, i've already replied to that thread with details | 16:57 |
| mnaser | mriedem: awesome, thank you so much for following up and being always on top of things :) | 16:58 |
| *** ducttape_ has quit IRC | 16:58 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.reboot notifications https://review.openstack.org/382959 | 17:00 |
| *** rmart04_ has joined #openstack-nova | 17:00 | |
| *** hshiina has joined #openstack-nova | 17:00 | |
| *** markmcclain has quit IRC | 17:02 | |
| *** rmart04_ has quit IRC | 17:02 | |
| *** rmart04 has quit IRC | 17:02 | |
| *** kiran-r has quit IRC | 17:04 | |
| *** armax has quit IRC | 17:05 | |
| *** dave-mcc_ has quit IRC | 17:06 | |
| *** dave-johnston has joined #openstack-nova | 17:09 | |
| *** sahid has quit IRC | 17:10 | |
| *** sahid has joined #openstack-nova | 17:12 | |
| *** mingyu has quit IRC | 17:12 | |
| mriedem | sdague: some fun 'what do you want in pike' big items in the operators list, which are cross-project fodder for the TC probably at the ptg/boston summit | 17:14 |
| mriedem | avolkov: i'm going to rebase my list and delete cells patches on top of your create_cell patch | 17:16 |
| mriedem | just to avoid all of the eventual conflicts | 17:16 |
| sdague | mriedem: ok, I'll take a look. I've been a little heads down today recoding the devstack ini / local.conf manipulation in python as the complexity of merging local.conf files in grenade exceeds the bash code we have | 17:16 |
| sdague | and that's the hold up for d-g support of local.conf | 17:16 |
| mriedem | yeah i saw that in qa | 17:16 |
| dansmith | mriedem: so, it would be super awesome if we could merge the bottom three patches in my set today before I go, or at least get some real review on them | 17:16 |
| dansmith | that would be here and down: https://review.openstack.org/#/c/410473/10 | 17:16 |
| mriedem | yeah it's just the next 2 then | 17:16 |
| mriedem | dansmith: did you see the convo i was having with melwitt about about the cell0 naming thing? | 17:17 |
| *** Apoorva has joined #openstack-nova | 17:17 | |
| dansmith | mriedem: no, was on calls sorry | 17:17 |
| mriedem | she also commented on the change, sec | 17:17 |
| mriedem | https://review.openstack.org/#/c/420439/5/nova/cmd/manage.py | 17:17 |
| mriedem | if she's right then we should sort that out before the grenade/devstack/series merges | 17:18 |
| mriedem | plus possible newton backports | 17:18 |
| *** markmcclain has joined #openstack-nova | 17:19 | |
| dansmith | if she's right about wht? | 17:19 |
| *** mingyu has joined #openstack-nova | 17:19 | |
| dansmith | I was just replying | 17:20 |
| mriedem | well about the connection url and naming | 17:20 |
| openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.reboot notifications https://review.openstack.org/382959 | 17:20 |
| dansmith | I'm sure the intent was because the cell0 db may often be co-located, but it's far too confusing, IMHO | 17:20 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add InstanceMappingList.get_by_cell_id https://review.openstack.org/420450 | 17:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add nova-manage cell_v2 create_cell command https://review.openstack.org/332713 | 17:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add nova-manage cell_v2 delete_cell command https://review.openstack.org/420451 | 17:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Create HostMappingList object https://review.openstack.org/420449 | 17:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add nova-manage cell_v2 list_cells command https://review.openstack.org/420440 | 17:21 |
| melwitt | what's confusing about it? that is, did something specific happen to incite this change? | 17:22 |
| mriedem | while reviewing the docs patch, | 17:23 |
| mriedem | i pointed out that if --database_connection is not provided to map_cell0 the connection url comes from the CONF.api_database.connection, not CONF.database.connection as dan had originally written in the docs patch | 17:23 |
| *** burgerk has quit IRC | 17:23 | |
| dansmith | it's confusing because we're taking a url and database name that points to an api schema thing, mangling it, and putting main schema in it, and calling it something with "api" in the name | 17:23 |
| mriedem | which took us down this path | 17:23 |
| *** acabot has quit IRC | 17:24 | |
| *** jose-phi_ has quit IRC | 17:24 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/420844 | 17:24 |
| *** sahid has quit IRC | 17:24 | |
| melwitt | okay. yeah, I can agree the name could be confusing, but I thought the connection, which is like host/user/password makes sense to be the same as the api db as a default. no? | 17:25 |
| *** mdnadeem has joined #openstack-nova | 17:25 | |
| dansmith | I don't see why it makes sense for those to be the same? | 17:25 |
| melwitt | that's what we wanted to encourage, that these are the two api dbs | 17:25 |
| melwitt | and it makes sense to put them on the same hosts | 17:26 |
| mriedem | maybe we should have just made the connection arg required | 17:26 |
| mriedem | so we didn't have to guess | 17:26 |
| dansmith | cell0 will grow indefinitely and need to be purged, api db will not.. that means different planning for those types of things | 17:26 |
| dansmith | mriedem: that's fine too | 17:26 |
| melwitt | dansmith: hm, okay. I hadn't thought of that | 17:27 |
| *** sudipto has quit IRC | 17:27 | |
| *** sudipto_ has quit IRC | 17:27 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements https://review.openstack.org/420861 | 17:28 |
| *** tesseract has quit IRC | 17:29 | |
| *** hshiina has quit IRC | 17:30 | |
| *** jose-phillips has joined #openstack-nova | 17:30 | |
| *** johnsom has quit IRC | 17:30 | |
| *** johnsom has joined #openstack-nova | 17:30 | |
| *** moshele has joined #openstack-nova | 17:35 | |
| *** jose-phillips has quit IRC | 17:35 | |
| *** jose-phillips has joined #openstack-nova | 17:37 | |
| *** mingyu has quit IRC | 17:37 | |
| *** diga has quit IRC | 17:39 | |
| *** jose-phi_ has joined #openstack-nova | 17:40 | |
| *** armax has joined #openstack-nova | 17:42 | |
| *** jose-phillips has quit IRC | 17:43 | |
| *** lucasagomes is now known as lucas-afk | 17:43 | |
| openstackgerrit | Dan Peschman proposed openstack/nova: nova-manage cell_v2 map_cell0 exit 0 https://review.openstack.org/420132 | 17:44 |
| openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.reboot notifications https://review.openstack.org/382959 | 17:45 |
| *** armax has quit IRC | 17:47 | |
| mriedem | dansmith: melwitt: so i don't have super strong feelings either way, would be nice if alaski were around to break any ties | 17:47 |
| mriedem | i lean toward the name change trumping the chance of the cell0 and nova_api db connection creds being the same | 17:48 |
| *** derekh has quit IRC | 17:48 | |
| mriedem | i've already started working on backporting the fixes to stable/newton in case we want those | 17:48 |
| mriedem | i tend to think that with most deployments not even at newton yet those would be good to have fixed by the time they upgrade | 17:49 |
| dansmith | I think that we're leaning towards people not doing the simple thing, and thus not just arbitrarily using the same url/creds for everything, so I want us to be more sane | 17:49 |
| *** roeyc has joined #openstack-nova | 17:49 | |
| mriedem | yeah which making the db connection arg required would fix that, but i'm not sure i want to backport that change | 17:49 |
| dansmith | yeah | 17:50 |
| mriedem | we could make db connection required in ocata, not backport anything, but put out a release note in newton saying you should be explicit when using map_cell0 | 17:50 |
| melwitt | yeah, I don't think we should do that being that we know people don't want to be forced to pass connection strings on the command line | 17:50 |
| *** CristinaPauna has quit IRC | 17:51 | |
| mriedem | melwitt: sure, but then we're also forcing them to handle it in config, i guess they can pass the --config-file option to nova-manage too | 17:52 |
| melwitt | if there's reason to have api db and cell0 db not be co-located (which has been mentioned today) then that addresses my concern about it. I had been thinking co-locating them would be most common, but if it wouldn't be, then I think it's fine not to default it that way | 17:52 |
| mriedem | and by handle it in config, i mean if they want cell0 to use the same host/user/password as the nova_api db and not the main db | 17:52 |
| melwitt | mriedem: yes, --config-file is key to setting up cells v1 anyway | 17:52 |
| dansmith | if they don't want to use the command line, then they need to have two separate places to define it anyway | 17:52 |
| dansmith | in the config I mean | 17:52 |
| openstackgerrit | Gábor Antal proposed openstack/nova: Added instance.reboot.error to the legacy notifications https://review.openstack.org/411816 | 17:53 |
| dansmith | the re-use of the config elements to avoid using command line args is super confusing and annoying to me | 17:53 |
| mdbooth | I have a test which uses threading (it's a concurrency test, this is unavoidable) | 17:53 |
| openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.reboot.error notification https://review.openstack.org/411791 | 17:53 |
| dansmith | it would be better, IMHO, if we had a yaml format for creds or something that we just had people create for each insertion they want to use, | 17:54 |
| mdbooth | I'm explicitly using threading.Thread rather than eventlet.greenthread, because the latter doesn't do what I need | 17:54 |
| dansmith | instead of "create a fake config, put just this section/keys in there, then we'll use that to create a db row instead of reading that config at runtime like it was intended"... | 17:54 |
| mdbooth | I create a thread and start it, but when the thread does threading.current_thread() it gets the main thread | 17:54 |
| mdbooth | Is this some evilness related to eventlet? | 17:55 |
| melwitt | dansmith: yeah, I guess cell0 is a special case. for all the other things, it makes sense to say "base this cell upon my old cells v1 cell conf". cell0 might probably be a whole new host/user/pass | 17:55 |
| mdbooth | threading.current_thread() has the expected behaviour for me when not run in the test harness | 17:56 |
| dansmith | melwitt: well, that case applies to like four operators.. the other billion will be creating fake configs when they want to configure or spin up new cells | 17:56 |
| melwitt | dansmith: only if they don't want to pass on command line, yeah | 17:57 |
| dansmith | melwitt: well, right, that's what I'm railing against :) | 17:57 |
| mdbooth | (Pdb) threading.current_thread.__module__ | 17:57 |
| mdbooth | 'eventlet.green.threading' | 17:57 |
| mdbooth | eventlet: j'accuse! | 17:57 |
| *** ducttape_ has joined #openstack-nova | 17:59 | |
| *** ljxiash has quit IRC | 17:59 | |
| melwitt | dansmith: yeah, it would be good to have a real way of doing that | 18:00 |
| *** ljxiash has joined #openstack-nova | 18:00 | |
| * mdbooth constructs a hideous workaround | 18:00 | |
| *** CristinaPauna has joined #openstack-nova | 18:01 | |
| *** ralonsoh has quit IRC | 18:01 | |
| jaypipes | gah, nested resource providers needs yet another rebase :) | 18:02 |
| *** esberglu has quit IRC | 18:02 | |
| *** clenimar has joined #openstack-nova | 18:02 | |
| mriedem | melwitt: dansmith: sounds like a clouds.yaml kind of thing, but a bit different | 18:02 |
| mriedem | nova-db.yaml :) | 18:02 |
| dansmith | mriedem: yeah, like that | 18:02 |
| *** esberglu has joined #openstack-nova | 18:03 | |
| *** slaweq_ has quit IRC | 18:03 | |
| *** mjura has quit IRC | 18:03 | |
| ildikov | mriedem: I'm guessing if the version negotiation with the client can get sorted out then we can do the testing in the placement_api job as you seuggested earlier, right? | 18:03 |
| *** ducttape_ has quit IRC | 18:03 | |
| mriedem | ildikov: yes, however, still need to solve the problem of how to configure nova to use cinder v3, which i think right now is going to be the job passing local.conf to d-g/devstack, and sdague is working on making that work | 18:04 |
| mriedem | post-config or whatever it's called | 18:04 |
| ildikov | yep, I remember that one too | 18:04 |
| ildikov | I think we can sync back on it the client part in Cinder is sorted out | 18:05 |
| mriedem | i can make scottda's patch work right now | 18:05 |
| *** swamireddy has quit IRC | 18:05 | |
| ildikov | mriedem: I won't say don't do it :) | 18:05 |
| *** mvk has joined #openstack-nova | 18:06 | |
| *** esberglu has quit IRC | 18:07 | |
| *** owalsh has quit IRC | 18:08 | |
| *** jdurgin1 has quit IRC | 18:09 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Use the mirror element to detect job completion https://review.openstack.org/399735 | 18:09 |
| mriedem | ildikov: scottda: done ^ | 18:12 |
| *** bvanhav_ has joined #openstack-nova | 18:12 | |
| mriedem | oops premature | 18:12 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP Support cinder v3 API https://review.openstack.org/420201 | 18:12 |
| mriedem | there | 18:12 |
| *** bvanhav has quit IRC | 18:12 | |
| openstackgerrit | Gábor Antal proposed openstack/nova: Transform instance.reboot.error notification https://review.openstack.org/411791 | 18:12 |
| ildikov | mriedem: thanks!!! | 18:13 |
| * mriedem gets lunch | 18:14 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: WIP Support cinder v3 API https://review.openstack.org/420201 | 18:14 |
| *** gszasz has quit IRC | 18:14 | |
| *** mdnadeem has quit IRC | 18:16 | |
| *** swamireddy has joined #openstack-nova | 18:17 | |
| *** weshay_lunch is now known as weshay | 18:19 | |
| *** jpena is now known as jpena|off | 18:20 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova: libvirt: Reconnect volumes and encryptors during a hard reboot https://review.openstack.org/400384 | 18:20 |
| mriedem | dansmith: so i'm kicking around ideas of how to make the multinode job run multiple cells, | 18:24 |
| mriedem | i think at least part of that is calling map_cell_and_hosts from the subnode | 18:25 |
| dansmith | mriedem: multiple cellsv2 cells you mean? | 18:25 |
| mriedem | yeah | 18:25 |
| mriedem | so host 1 is control + n-cpu, | 18:25 |
| mriedem | host 2 (subnode) is just n-cpu | 18:25 |
| mriedem | but both hosts are going to be a cell | 18:25 |
| *** esberglu has joined #openstack-nova | 18:26 | |
| mriedem | or a combo of create_cell and discover_hosts | 18:26 |
| mriedem | on the subnode | 18:26 |
| *** esberglu has quit IRC | 18:27 | |
| *** esberglu has joined #openstack-nova | 18:27 | |
| dansmith | doesn't have to be on the subnode specifically, | 18:27 |
| dansmith | but yeah, create_cell will make it more explicit I think | 18:27 |
| mriedem | so currently we just create the dbs on the api node https://github.com/openstack-dev/devstack/blob/master/lib/nova#L683 | 18:29 |
| mriedem | which is fine | 18:29 |
| melwitt | I was thinkin once we have create_cell, we'd remove map_cell_and_hosts since that's really just create_cell + discover_hosts. to make things not overlap and be simpler | 18:29 |
| dansmith | yep | 18:30 |
| mriedem | then we do the mappings later in https://github.com/openstack-dev/devstack/blob/master/lib/nova#L948 | 18:30 |
| mriedem | which is called later here https://github.com/openstack-dev/devstack/blob/master/stack.sh#L1368 | 18:30 |
| mriedem | so we only do that stuff if on the api node | 18:30 |
| mriedem | so i think i can put an else in there for when n-api isn't running (subnode) | 18:31 |
| *** satyar has quit IRC | 18:31 | |
| mriedem | and n-cpu is enabled, | 18:31 |
| mriedem | but would probably want the subnode cell db on the api node, i'm not sure if it matters for CI? | 18:31 |
| mriedem | given we're hitting OOM errors in the gate right now, | 18:31 |
| mriedem | the cell db on the subnode is probably better | 18:32 |
| dansmith | doesn't need to actually be a different place | 18:32 |
| mriedem | yeah i know, | 18:32 |
| mriedem | but devstack isn't topology aware, | 18:32 |
| mriedem | and i'm not entirely sure how all of the subnode stuff gets setup by devstack-gate | 18:32 |
| mriedem | so i'm not sure that when setting up the api node, we know about the subnode yet | 18:32 |
| mriedem | but i guess maybe that doesn't matter? | 18:33 |
| mriedem | we just have a db somewhere, and when we create the cell for the subnode we point that cell at that db connection | 18:33 |
| dansmith | well, | 18:34 |
| dansmith | the thing I ran into earlier trying to do something else was that the subnode didn't have all the stuff it needed to do certain things | 18:35 |
| dansmith | like, not all the config | 18:35 |
| dansmith | don't recall what exactly, but I couldn't start n-api on the subnode | 18:35 |
| *** salv-orlando has quit IRC | 18:35 | |
| dansmith | might've been keystone and catalog related actually | 18:35 |
| *** amoralej is now known as amoralej|off | 18:36 | |
| *** salv-orlando has joined #openstack-nova | 18:36 | |
| mriedem | this is what we have on the subnode http://logs.openstack.org/75/396775/39/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/f82fd9f/logs/subnode-2/etc/nova/nova.conf.txt.gz | 18:36 |
| *** owalsh has joined #openstack-nova | 18:38 | |
| mriedem | somehow the subnode is getting the variables for the db host | 18:40 |
| mriedem | ah d- | 18:41 |
| mriedem | *d-g my old friend | 18:41 |
| *** dave-mccowan has joined #openstack-nova | 18:43 | |
| *** lpetrut has joined #openstack-nova | 18:44 | |
| mriedem | alright time to hacky hacky | 18:44 |
| jaosorior | gibi: you around? | 18:50 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/420844 | 18:50 |
| *** sacharya has joined #openstack-nova | 18:54 | |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472 | 18:59 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: use a single ResourceTracker object instance https://review.openstack.org/398473 | 18:59 |
| jaypipes | mriedem: ^^ | 19:00 |
| mriedem | que bueno | 19:00 |
| cdent | good timing jaypipes my to do list just rolled around to those | 19:01 |
| jaypipes | coolio. | 19:01 |
| mriedem | jaypipes: were you going to also update _setup_pci_tracker to pass the compute_node object in ther? | 19:01 |
| mriedem | *there | 19:01 |
| jaypipes | mriedem: ah, shitbuckets. | 19:01 |
| jaypipes | one sec. | 19:01 |
| mriedem | otherwise it looks super hot | 19:02 |
| mriedem | like, omg | 19:02 |
| *** voelzmo has joined #openstack-nova | 19:02 | |
| *** ijw has joined #openstack-nova | 19:05 | |
| *** ijw has quit IRC | 19:05 | |
| *** ijw has joined #openstack-nova | 19:05 | |
| *** moshele has quit IRC | 19:06 | |
| *** voelzmo has quit IRC | 19:06 | |
| mriedem | melwitt: good point on https://review.openstack.org/#/c/332713/14/nova/cmd/manage.py@1432 | 19:08 |
| mriedem | i obviously didn't think about that | 19:08 |
| mriedem | heh i even said that in PS9: | 19:09 |
| mriedem | "I think I'd like to see us first check to see if a cell mapping exists with the given name (or combination of name/transport_url/database_connection) and if we already have a match, we just exit without doing anything (not a failure)." | 19:09 |
| melwitt | mriedem: yeah, I've been admittedly confused about whether we return 0 for a no-op or not among all these commands. it seems like it hasn't been consistent | 19:10 |
| melwitt | like simple_cell_setup returns non-zero if it didn't create a cell mapping, but map_cell_and_hosts returns 0 in the same case I think | 19:11 |
| mriedem | yeah bauzas has a patch up to fix that actually | 19:11 |
| mriedem | another thing with this create_cell command is you can't create 2 cells with different names but the same db/mq urls | 19:11 |
| mriedem | i'm not sure if we care about that right now, and it could be built in later if we did | 19:12 |
| mriedem | seems we should have had a unique constraint on the name/db/transport combo in the schema | 19:12 |
| *** ijw_ has joined #openstack-nova | 19:12 | |
| melwitt | yeah, I thought about that and figured we shouldn't probably allow that case anyway (different name, same urls) | 19:12 |
| dansmith | we should allow duplicate urls of each type at the very least | 19:13 |
| dansmith | not sure it makes sense to allow complete dupes though, other than for testing | 19:14 |
| *** ducttape_ has joined #openstack-nova | 19:15 | |
| mriedem | dansmith: yeah the check in create_cell right now is db AND mq urls matching an existing cell | 19:15 |
| mriedem | that's a failure | 19:15 |
| mriedem | which i had earlier said i thought should just be a noop (don't create a new cell), but guess i forgot that i asked for that before +2ing | 19:15 |
| *** ijw has quit IRC | 19:15 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/nova: Move to tooz hash ring implementation https://review.openstack.org/399022 | 19:16 |
| jaypipes | mriedem: just re-running tests locally. should be pushed within 15minutes. | 19:16 |
| mriedem | ok push it real good please | 19:16 |
| dansmith | mriedem: well, as melwitt said, returning zero for "I didn't do anything, but it's there like you asked" will be good for some people and bad for others that think they can test for existence by trying to re-create and check for error | 19:17 |
| dansmith | mriedem: so my opinion is that we not make a "create this thing" operation idempotent and instead do what you asked | 19:18 |
| mriedem | right, and since i'm frankly tired of trying to predict how all of this is going to be used, i'm going to give up | 19:18 |
| *** voelzmo has joined #openstack-nova | 19:18 | |
| *** voelzmo has quit IRC | 19:19 | |
| *** ducttape_ has quit IRC | 19:19 | |
| *** oomichi_ has joined #openstack-nova | 19:20 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/420844 | 19:20 |
| mriedem | dansmith: don't forget about this guy https://review.openstack.org/#/c/420440/ | 19:21 |
| dansmith | mriedem: jenkins says that is shit | 19:21 |
| mriedem | it said that about the next in the series too which you +2ed | 19:22 |
| dansmith | I hand-checked that one | 19:22 |
| * dansmith is bullshitting | 19:23 | |
| mriedem | i know | 19:23 |
| mriedem | my detector has been honed over years of bullshit | 19:23 |
| *** ducttape_ has joined #openstack-nova | 19:25 | |
| *** ijw has joined #openstack-nova | 19:27 | |
| *** ducttape_ has quit IRC | 19:30 | |
| *** ijw__ has joined #openstack-nova | 19:30 | |
| *** ijw_ has quit IRC | 19:30 | |
| oomichi_ | mriedem: why not "nova-manage cell_v2 list" ? The other resources' commands are just "list" without resource name in sub commands | 19:33 |
| *** ijw has quit IRC | 19:34 | |
| mriedem | oomichi_: because we could have things like list_host_mappings or something, idk | 19:34 |
| mriedem | oomichi_: and that's what we named it like for create_cell | 19:35 |
| dansmith | I commented to that effect | 19:35 |
| oomichi_ | mriedem: ah, I see "create_cell". I got it. | 19:36 |
| oomichi_ | dansmith: Thanks, I see. it is nice to keep current one for consistent command line | 19:36 |
| openstackgerrit | Jay Pipes proposed openstack/nova: placement: RT now adds proper Ironic inventory https://review.openstack.org/404472 | 19:37 |
| openstackgerrit | Jay Pipes proposed openstack/nova: rt: use a single ResourceTracker object instance https://review.openstack.org/398473 | 19:37 |
| *** paw has quit IRC | 19:40 | |
| *** bvanhav_ has quit IRC | 19:40 | |
| melwitt | omg fixing unit tests is finding my bugs -_- | 19:40 |
| *** moshele has joined #openstack-nova | 19:44 | |
| *** oomichi_ has quit IRC | 19:47 | |
| *** kfarr has joined #openstack-nova | 19:52 | |
| *** jerrygb has joined #openstack-nova | 19:57 | |
| melwitt | dansmith: are there any common mistakes/reasons why a lazy-load of flavor would try to occur even after specifying expected_attrs=['flavor']? | 19:57 |
| dansmith | melwitt: hmm, not that I can think of | 19:58 |
| slaweq | mriedem: hello | 20:00 |
| slaweq | mriedem: I made small patch https://review.openstack.org/#/c/420176/ to fix https://bugs.launchpad.net/nova/+bug/1640502 | 20:00 |
| openstack | Launchpad bug 1640502 in OpenStack Compute (nova) "Empty /etc/machine-id causes failure in unit tests" [Medium,In progress] - Assigned to Slawek Kaplonski (slaweq) | 20:00 |
| slaweq | mriedem: can You take a look on it if You will have a while? thx in advance | 20:01 |
| melwitt | dansmith: thanks, will keep digging | 20:01 |
| *** bvanhav_ has joined #openstack-nova | 20:04 | |
| *** voelzmo has joined #openstack-nova | 20:05 | |
| dansmith | mriedem: I bet we need to discover hosts somewhere after the subnode runs, which is why I'm getting this: http://logs.openstack.org/79/319379/58/check/gate-tempest-dsvm-multinode-live-migration-ubuntu-xenial/c25f7d1/logs/testr_results.html.gz | 20:10 |
| melwitt | oh, duh. this unit test is creating instances that don't have flavors as part of the setup and the old code used to look at inst['vcpus'] apparently | 20:12 |
| melwitt | so my instance get was getting some instances with None flavors | 20:13 |
| *** moshele has quit IRC | 20:13 | |
| *** edmondsw has joined #openstack-nova | 20:18 | |
| *** adisky_ has quit IRC | 20:19 | |
| *** slaweq has quit IRC | 20:19 | |
| bauzas | jaypipes: hey, about https://review.openstack.org/#/c/417961/5/nova/scheduler/host_manager.py@362 | 20:20 |
| bauzas | jaypipes: sec to discuss ? | 20:20 |
| jaypipes | bauzas: just finishing on phone call. gimme 10. | 20:20 |
| bauzas | jaypipes: fair | 20:20 |
| bauzas | jaypipes: leaving notes meanwhile | 20:20 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/420844 | 20:21 |
| *** slaweq has joined #openstack-nova | 20:21 | |
| *** voelzmo has quit IRC | 20:22 | |
| *** salv-orlando has quit IRC | 20:23 | |
| *** edmondsw has quit IRC | 20:23 | |
| *** lpetrut has quit IRC | 20:27 | |
| jaypipes | bauzas: k, done. | 20:30 |
| *** kfarr_ has joined #openstack-nova | 20:30 | |
| bauzas | jaypipes: see my comment for context in gerrit | 20:30 |
| *** ijw has joined #openstack-nova | 20:31 | |
| *** woodster_ has joined #openstack-nova | 20:32 | |
| mriedem | whew | 20:33 |
| *** jose-phi_ has quit IRC | 20:33 | |
| mriedem | dansmith: yeah probably, which is actually what i started working on | 20:33 |
| mriedem | before the call i just got off of happened | 20:33 |
| *** dave-mccowan has quit IRC | 20:34 | |
| *** rtheis has joined #openstack-nova | 20:34 | |
| bauzas | jaypipes: thoughts on that ? | 20:34 |
| *** ijw__ has quit IRC | 20:34 | |
| dansmith | mriedem: okay | 20:35 |
| bauzas | jaypipes: I honestly totally agree with you that using a reportclient is somehow sad | 20:35 |
| *** kfarr__ has joined #openstack-nova | 20:35 | |
| bauzas | jaypipes: but I wanted to be pragmatic | 20:35 |
| *** kfarr__ has quit IRC | 20:35 | |
| *** jose-phillips has joined #openstack-nova | 20:36 | |
| *** kfarr__ has joined #openstack-nova | 20:38 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient: DO NOT MERGE: check OSC https://review.openstack.org/420941 | 20:38 |
| *** kfarr__ has quit IRC | 20:39 | |
| *** kfarr_ has quit IRC | 20:40 | |
| *** kfarr has quit IRC | 20:41 | |
| *** kfarr has joined #openstack-nova | 20:41 | |
| *** kfarr has quit IRC | 20:41 | |
| mriedem | ok on the api node i create nova_subnode_cell db, and then create subnode-cell cell with the db connection to the nova_subnode_cell db and transport_url to the primary node's same mq | 20:43 |
| mriedem | transport-url is going to be the same for all of these | 20:43 |
| mriedem | then on the subnode setup after n-cpu is running i need to run discover_hosts | 20:44 |
| *** ducttape_ has joined #openstack-nova | 20:45 | |
| mriedem | now i need to remember how to awk stuff, damn | 20:45 |
| *** Guest7011 has quit IRC | 20:46 | |
| *** rtheis has quit IRC | 20:48 | |
| *** ducttape_ has quit IRC | 20:49 | |
| *** bvanhav_ has quit IRC | 20:56 | |
| *** jamielennox|away is now known as jamielennox | 20:59 | |
| jaypipes | bauzas: sorry, reading now. got pulled away | 21:01 |
| bauzas | np | 21:01 |
| comstud | mriedem: well, there's your problem | 21:07 |
| jaypipes | comstud: blast from the past! howdy. | 21:08 |
| comstud | yo :) | 21:08 |
| jaypipes | bauzas: ok, fair point on the reporting client thing. I can go along with that. | 21:08 |
| jaypipes | bauzas: with a promise to clean up ex post facto. | 21:08 |
| *** salv-orlando has joined #openstack-nova | 21:09 | |
| bauzas | jaypipes: my change is still there | 21:09 |
| jaypipes | bauzas: I still would need to see a ComputeNodeList.get_all_by_uuids() though. | 21:09 |
| bauzas | jaypipes: I mean the placementclient one | 21:09 |
| jaypipes | bauzas: understood. | 21:09 |
| bauzas | jaypipes: sure, I'll redo it | 21:09 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: hyper-v: Adds support for VMs with nested virtualization https://review.openstack.org/398509 | 21:09 |
| openstackgerrit | Claudiu Belu proposed openstack/nova: Adds CPUFeaturesField to ImageMetaProps https://review.openstack.org/420960 | 21:09 |
| *** dimtruck is now known as zz_dimtruck | 21:10 | |
| bauzas | jaypipes: the question I was wondering about the placementclient is that it would be reasonable to only provide CRUD ops in it | 21:10 |
| bauzas | jaypipes: or if I'd rather work on objectifying that client | 21:10 |
| bauzas | cutting by just the CRUD ops could possible be merged sooner | 21:10 |
| jaypipes | bauzas: no, I think just pulling get/put/post/safe_connect into a class is correct. | 21:11 |
| mriedem | comstud: it's ok, mrodden stopped by just at the right time and gave me the awk i needed | 21:11 |
| bauzas | but I'd tend to prefer having a placementclient that would have resources in it | 21:11 |
| bauzas | jaypipes: okay, then I'll continue to keep my change up-to-date then | 21:11 |
| bauzas | it'll just be a top change, not a dependent one | 21:11 |
| jaypipes | bauzas: I'd prefer composition over inheritance as I mentioned before, and keep state in separate objects (i.e. not the PlacementClient class itself but the object that contains a PlacementClient object) | 21:12 |
| *** jose-phillips has quit IRC | 21:12 | |
| bauzas | jaypipes: composition is already in the last PS | 21:12 |
| bauzas | AFAIR | 21:12 |
| bauzas | unless I forgot to upload it | 21:12 |
| *** jamesdenton has joined #openstack-nova | 21:13 | |
| *** jose-phillips has joined #openstack-nova | 21:14 | |
| bauzas | jaypipes: state of what? it's mostly stateless if you except the disabled thing | 21:14 |
| *** JoseMello has quit IRC | 21:15 | |
| jaypipes | bauzas: I mean the _resource_providers dict. | 21:15 |
| bauzas | jaypipes: in that case, it's fine, I left that | 21:15 |
| jaypipes | bauzas: I would prefer that not be in the PlacementClient. That should be in the ReportingClient, yes? | 21:15 |
| bauzas | correct | 21:15 |
| jaypipes | cool | 21:15 |
| *** rfolco has quit IRC | 21:15 | |
| bauzas | jaypipes: https://review.openstack.org/#/c/417111/7/nova/scheduler/client/report.py is pretty seamless | 21:15 |
| jaypipes | bauzas: oui. absolument. | 21:16 |
| jaypipes | bauzas: though I'd change SafeConnectDecorator to @client.safe_connect :) | 21:16 |
| bauzas | jaypipes: but I need to keep a state, hence the deco object :) | 21:18 |
| jaypipes | bauzas: you could just look for a consistently-named attribute, no? | 21:19 |
| bauzas | jaypipes: I could use a global var | 21:20 |
| jaypipes | bauzas: or a consistently-named attr :) | 21:21 |
| jaypipes | bauzas: else you could require classes to implement a get_placement_client() method. | 21:22 |
| *** cdent has quit IRC | 21:22 | |
| jaypipes | bauzas: or something like that. | 21:22 |
| mriedem | dansmith: hmm, should create_cell sync the db schema for that cell? | 21:22 |
| bauzas | jaypipes: I thought of the latter | 21:23 |
| bauzas | jaypipes: anyway, I'll see | 21:23 |
| *** jamesdenton has quit IRC | 21:24 | |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 21:24 |
| *** Matias has quit IRC | 21:24 | |
| dansmith | mriedem: hmm, I dunno.. | 21:24 |
| dansmith | maybe optional? --sync-that-mofo-too? | 21:24 |
| mriedem | yeah because the problem is running nova-manage db sync on my subnode cell | 21:25 |
| *** paw has joined #openstack-nova | 21:25 | |
| *** jamesdenton has joined #openstack-nova | 21:25 | |
| mriedem | dansmith: fuck i've been doffm'ed https://github.com/openstack/nova/blob/7cb416546a2c6d8eb36cbf9acfde776d54a26811/nova/cmd/manage.py#L630 | 21:27 |
| dansmith | mriedem: um, congrats? | 21:28 |
| mriedem | just looks like we thought about this before | 21:28 |
| mriedem | so to sync this db i have to hack the nova.conf | 21:28 |
| mriedem | otherwise it's going to be pointing at the nova db connection on the primary node | 21:29 |
| dansmith | mriedem: you could make that work for other cells easily, right? | 21:29 |
| dansmith | just list all the mappings and target them before sync like cell0 does | 21:29 |
| mriedem | yeah we just get all the cell mappings and sync them | 21:29 |
| mriedem | the note here seems wrong https://github.com/openstack-dev/devstack/blob/master/lib/nova#L687 | 21:31 |
| mriedem | nova-manage db sync is just going to sync the local main db, or whatever is in CONF.database.connection | 21:31 |
| *** dave-mccowan has joined #openstack-nova | 21:32 | |
| *** zz_dimtruck is now known as dimtruck | 21:32 | |
| dansmith | yeah | 21:32 |
| mriedem | today we're getting cell0 synced b/c of https://github.com/openstack-dev/devstack/blob/master/lib/nova#L950 | 21:32 |
| dansmith | mriedem: could also make db sync take a cell uuid or osmeting | 21:32 |
| *** smatzek has quit IRC | 21:33 | |
| mriedem | so in the cells v1 case i don't see where we even sync the cell0 db | 21:33 |
| dansmith | the cellsv1 case? | 21:33 |
| mriedem | this https://github.com/openstack-dev/devstack/blob/master/lib/nova#L951-L956 | 21:33 |
| dansmith | map_cell0 does it | 21:34 |
| mriedem | umm, | 21:34 |
| mriedem | nope | 21:34 |
| mriedem | https://github.com/openstack/nova/blob/7cb416546a2c6d8eb36cbf9acfde776d54a26811/nova/cmd/manage.py#L1157 | 21:34 |
| mriedem | it creates the cell0 mapping | 21:34 |
| mriedem | but doesn't sync it | 21:34 |
| dansmith | oh hmm, yeah | 21:34 |
| dansmith | I thought it did | 21:34 |
| *** Jeffrey4l has quit IRC | 21:34 | |
| mriedem | simple_cell_setup is the only one that does https://github.com/openstack/nova/blob/7cb416546a2c6d8eb36cbf9acfde776d54a26811/nova/cmd/manage.py#L1105 | 21:35 |
| dansmith | wait, no.. the db sync does it for cellsv1 right? | 21:35 |
| *** Jeffrey4l has joined #openstack-nova | 21:35 | |
| *** ducttape_ has joined #openstack-nova | 21:35 | |
| mriedem | as far as i can tell, db sync is going to just sync the 'main' db | 21:36 |
| dansmith | no, it does cell0 and the main one | 21:36 |
| dansmith | https://github.com/openstack/nova/blob/7cb416546a2c6d8eb36cbf9acfde776d54a26811/nova/cmd/manage.py#L635 | 21:36 |
| mriedem | oh yeah | 21:36 |
| dansmith | it queries for cell0 and does it, it just doesn't list | 21:36 |
| mriedem | so, | 21:37 |
| mriedem | i think i'm just going to address https://github.com/openstack/nova/blob/7cb416546a2c6d8eb36cbf9acfde776d54a26811/nova/cmd/manage.py#L630 | 21:37 |
| mriedem | and do the cell mapping list | 21:37 |
| mriedem | and sync those mofos | 21:37 |
| mriedem | i mean we could also do this in create_cell easily | 21:38 |
| mriedem | but it looks like we thought about a fanout sync in sync already | 21:38 |
| *** edmondsw has joined #openstack-nova | 21:40 | |
| dansmith | yeah, as long as you continue to honor --local-cell | 21:42 |
| mriedem | i am | 21:42 |
| mriedem | should have a WIP up shortly | 21:43 |
| melwitt | hm, I'm having a problem where I want to do a get of instances across all cells using target_cell and utils.spawn, but it doesn't keep the targeted cell context that way | 21:43 |
| mriedem | and then i'll depends-on that for this devstack patch | 21:43 |
| melwitt | because of the utils.spawn. I wanted to do them all at the same time and wait on all of them to combine the results from each cell | 21:43 |
| *** ducttape_ has quit IRC | 21:43 | |
| *** ducttape_ has joined #openstack-nova | 21:44 | |
| *** ducttape_ has quit IRC | 21:44 | |
| *** edmondsw has quit IRC | 21:44 | |
| dansmith | melwitt: isn't there some magic about storing the current context in eventlet TLS? | 21:46 |
| dansmith | I dunno why or what that is because we pass it everywhere, but... there's something | 21:46 |
| melwitt | dansmith: I dunno but thanks for the hint. I'll see if I can find something related to that, because for now, I'm lost | 21:47 |
| *** jamesdenton has quit IRC | 21:47 | |
| *** salv-orlando has quit IRC | 21:47 | |
| *** jamesdenton has joined #openstack-nova | 21:49 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/420844 | 21:49 |
| melwitt | it uses the get_current() context from TLS, so that implies my target_cell modified context isn't in get_current(). makes sense because I don't think anything about target_cell saves the new targeted context to TLS | 21:50 |
| *** jamesdenton has quit IRC | 21:52 | |
| *** dave-mccowan has quit IRC | 21:54 | |
| *** thorst has quit IRC | 21:55 | |
| *** aojea has joined #openstack-nova | 21:59 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fanout the db sync to all cells if --local_cell is False https://review.openstack.org/420973 | 22:00 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Add some cellsv2 setup docs https://review.openstack.org/420198 | 22:03 |
| openstackgerrit | Matt Riedemann proposed openstack/nova: Fix the generated cell0 default database name https://review.openstack.org/420439 | 22:03 |
| mriedem | dansmith: had to remove the dependencies on that bottom change b/c the stable/newton backports for the devstack/grenade changes were keeping it from merging | 22:04 |
| *** smatzek has joined #openstack-nova | 22:05 | |
| *** eharney has quit IRC | 22:08 | |
| dansmith | mriedem: ack, thansk | 22:12 |
| *** salv-orlando has joined #openstack-nova | 22:12 | |
| mriedem | man the db sync stuff in devstack is getting a bit crazy | 22:12 |
| dansmith | sync, sync, sync again, sure one more why not | 22:12 |
| mriedem | and if we're counting, we'll have 4 dbs in the multinode job: nova_api, nova_cell0, nova (cell1), nova_subnode_cell (cell2) | 22:13 |
| dansmith | well, four dbs for a multi-cell cloud.. not bad :) | 22:13 |
| mriedem | and in pike we'll have the placement db :) | 22:13 |
| mriedem | but at that point it's no longer nova so not our fault | 22:13 |
| *** marst has quit IRC | 22:15 | |
| dansmith | yeah | 22:15 |
| *** dimtruck is now known as zz_dimtruck | 22:17 | |
| mriedem | ok here is the devstack change of doom https://review.openstack.org/420976 | 22:17 |
| mriedem | i would be amazed if that worked on the first try | 22:17 |
| dansmith | mriedem: well, we need the late discover_hosts even without trying to create another cell | 22:18 |
| *** zz_dimtruck is now known as dimtruck | 22:18 | |
| dansmith | don't you want to separate those/ | 22:18 |
| mriedem | i could | 22:18 |
| mriedem | and get less cute with the list_cells awk | 22:18 |
| *** smatzek has quit IRC | 22:19 | |
| mriedem | yeah let me separate that quick, basically just if n-cpu; then discover_hosts | 22:19 |
| dansmith | yeah | 22:19 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 22:21 |
| *** jheroux has quit IRC | 22:22 | |
| *** salv-orl_ has joined #openstack-nova | 22:24 | |
| *** thorst has joined #openstack-nova | 22:24 | |
| *** salv-orlando has quit IRC | 22:26 | |
| *** thorst has quit IRC | 22:28 | |
| mriedem | ok done | 22:28 |
| *** dimtruck is now known as zz_dimtruck | 22:28 | |
| mriedem | dansmith: here is the change you'll need https://review.openstack.org/#/c/420981/1 | 22:28 |
| dansmith | thanks | 22:29 |
| *** zz_dimtruck is now known as dimtruck | 22:29 | |
| *** jerrygb has quit IRC | 22:34 | |
| mriedem | jaypipes: +2 on https://review.openstack.org/#/c/398473/ now | 22:36 |
| mriedem | thanks | 22:36 |
| *** tblakes has quit IRC | 22:39 | |
| *** jdurgin has quit IRC | 22:39 | |
| *** jaosorior has quit IRC | 22:41 | |
| *** ducttape_ has joined #openstack-nova | 22:45 | |
| *** ducttape_ has quit IRC | 22:49 | |
| mriedem | alright apparently i'm going to die in an ice storm and should head home, so doing that now, will be back online later | 22:50 |
| *** mriedem has quit IRC | 22:51 | |
| *** jdurgin has joined #openstack-nova | 22:57 | |
| *** mdrabe has quit IRC | 22:58 | |
| *** medberry is now known as med_ | 23:05 | |
| *** krtaylor has quit IRC | 23:06 | |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 23:07 |
| openstackgerrit | Gage Hugo proposed openstack/nova: Fixed warning messages when building docs https://review.openstack.org/420990 | 23:10 |
| *** jerrygb has joined #openstack-nova | 23:22 | |
| *** mriedem has joined #openstack-nova | 23:23 | |
| *** jose-phillips has quit IRC | 23:23 | |
| *** jwcroppe has quit IRC | 23:26 | |
| *** jerrygb has quit IRC | 23:26 | |
| *** jose-phillips has joined #openstack-nova | 23:30 | |
| *** jerrygb has joined #openstack-nova | 23:34 | |
| *** xyang1 has quit IRC | 23:34 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova: api-ref: Fix deprecated proxy API parameters https://review.openstack.org/416882 | 23:35 |
| *** ducttape_ has joined #openstack-nova | 23:35 | |
| *** dimtruck is now known as zz_dimtruck | 23:35 | |
| *** ircuser-1 has joined #openstack-nova | 23:43 | |
| openstackgerrit | Dan Smith proposed openstack/nova: Multicell support for instance listing https://review.openstack.org/396775 | 23:44 |
| openstackgerrit | Dan Smith proposed openstack/nova: Make api_samples tests use simple cell environment https://review.openstack.org/410473 | 23:45 |
| openstackgerrit | Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379 | 23:45 |
| *** knikolla has quit IRC | 23:47 | |
| *** knikolla has joined #openstack-nova | 23:47 | |
| *** knikolla has quit IRC | 23:47 | |
| *** knikolla has joined #openstack-nova | 23:48 | |
| *** salv-orl_ has quit IRC | 23:50 | |
| mriedem | hmm, wtf is the difference between gate-tempest-dsvm-multinode-full-ubuntu-xenial-nv and gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv now that both are running neutron | 23:53 |
| *** dave-johnston has quit IRC | 23:55 | |
| *** edmondsw has joined #openstack-nova | 23:55 | |
| mriedem | heh, turns out, not a gd thing | 23:56 |
| *** edmondsw has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!