Tuesday, 2016-12-20

*** tjones has quit IRC00:00
*** jwcroppe has quit IRC00:03
*** thorst has quit IRC00:05
*** tyrola has quit IRC00:06
*** tyrola has joined #openstack-nova00:07
*** siva_krishnan_ has quit IRC00:07
*** siva_krish has quit IRC00:11
*** tlian has quit IRC00:11
*** yufei has joined #openstack-nova00:19
*** Zer0Byte__ has quit IRC00:19
*** nic has quit IRC00:22
*** tlian has joined #openstack-nova00:22
*** dtp has quit IRC00:26
*** yingjun has joined #openstack-nova00:29
*** catintheroof has quit IRC00:29
*** marst has quit IRC00:36
*** claudiub has quit IRC00:38
*** hongbin has quit IRC00:41
*** tjones has joined #openstack-nova00:53
*** lamt has joined #openstack-nova00:54
*** dimtruck is now known as zz_dimtruck00:55
*** ducttape_ has joined #openstack-nova00:56
*** amotoki has joined #openstack-nova00:56
*** kevinz has joined #openstack-nova00:56
openstackgerritZhenyu Zheng proposed openstack/nova: Check destination_type when booting with bdm provided  https://review.openstack.org/40237200:56
*** hoangcx has joined #openstack-nova00:58
*** jwcroppe has joined #openstack-nova01:00
*** ducttape_ has quit IRC01:01
*** tjones has left #openstack-nova01:02
*** thorst has joined #openstack-nova01:03
*** jwcroppe has quit IRC01:04
*** thorst has quit IRC01:10
*** amotoki has quit IRC01:11
*** ducttape_ has joined #openstack-nova01:11
*** weshay_relocatin is now known as weshay01:12
*** amotoki has joined #openstack-nova01:12
*** zz_dimtruck is now known as dimtruck01:16
*** unicell has quit IRC01:18
*** mnestratov|2 has joined #openstack-nova01:20
*** zhangjl has joined #openstack-nova01:22
*** Jeffrey4l has joined #openstack-nova01:23
*** mnestratov has quit IRC01:24
*** amotoki has quit IRC01:28
*** amotoki has joined #openstack-nova01:30
*** gongysh has joined #openstack-nova01:33
*** ssurana has quit IRC01:34
*** baoli has joined #openstack-nova01:40
*** ducnc has quit IRC01:44
*** ducnc has joined #openstack-nova01:44
*** baoli has quit IRC01:45
*** tuanluong has joined #openstack-nova01:46
*** raunak has quit IRC01:58
*** Sukhdev has quit IRC01:59
openstackgerritint32bit proposed openstack/nova: Add more actions to instance action logs  https://review.openstack.org/39651001:59
*** browne has quit IRC01:59
*** jwcroppe has joined #openstack-nova02:01
*** tlian has quit IRC02:01
*** phuongnh has joined #openstack-nova02:02
*** jwcroppe has quit IRC02:05
*** thorst has joined #openstack-nova02:07
*** gouthamr has quit IRC02:10
*** mdnadeem has joined #openstack-nova02:13
*** thorst has quit IRC02:14
*** thorst has joined #openstack-nova02:15
openstackgerritzhangyanxian proposed openstack/nova-specs: Fix typos in docs  https://review.openstack.org/41233802:17
*** thorst has quit IRC02:19
*** Jeffrey4l has quit IRC02:22
*** trananhkma has joined #openstack-nova02:22
*** yingjun has quit IRC02:24
*** yingjun has joined #openstack-nova02:27
*** ducttape_ has quit IRC02:27
*** ducttape_ has joined #openstack-nova02:28
*** int32bit has joined #openstack-nova02:28
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port libvirt.test_vif to Python 3  https://review.openstack.org/40341902:29
*** gcb has quit IRC02:30
*** zerda2 has quit IRC02:32
*** ducttape_ has quit IRC02:32
*** yuntongjin has joined #openstack-nova02:33
*** mriedem has quit IRC02:33
openstackgerritZhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_security_group  https://review.openstack.org/30713502:37
openstackgerritZhenyu Zheng proposed openstack/nova: Remove mox in tests/unit/objects/test_instance_info_cache  https://review.openstack.org/30715502:37
*** fragatina has joined #openstack-nova02:37
*** esberglu has joined #openstack-nova02:38
openstackgerritZhenyu Zheng proposed openstack/nova: Don't apply multi-queue to SRIOV ports  https://review.openstack.org/39754502:38
*** browne has joined #openstack-nova02:39
*** esberglu has quit IRC02:42
*** ducttape_ has joined #openstack-nova02:48
*** gcb has joined #openstack-nova02:50
*** tjones has joined #openstack-nova02:57
*** browne has quit IRC02:59
*** gongysh has quit IRC03:00
*** jwcroppe has joined #openstack-nova03:01
*** tjones has quit IRC03:02
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Skip network.test_manager.LdapDNSTestCase on Python 3  https://review.openstack.org/40642503:04
*** jwcroppe has quit IRC03:06
*** yamahata has quit IRC03:06
*** jamielennox is now known as jamielennox|away03:08
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port test_matchers.TestDictMatches.test__str__ to Python 3  https://review.openstack.org/40643703:13
*** thorst has joined #openstack-nova03:14
*** Jeffrey4l has joined #openstack-nova03:14
*** mdnadeem has quit IRC03:14
*** thorst has quit IRC03:14
*** diga has joined #openstack-nova03:15
*** yingjun has quit IRC03:16
*** raunak has joined #openstack-nova03:18
*** mriedem has joined #openstack-nova03:19
*** mdnadeem has joined #openstack-nova03:19
*** jamielennox|away is now known as jamielennox03:22
*** hfu has joined #openstack-nova03:26
openstackgerritCao Xuan Hoang proposed openstack/nova: Removes unnecessary utf-8 encoding  https://review.openstack.org/41005403:32
*** burgerk has joined #openstack-nova03:32
*** esberglu has joined #openstack-nova03:33
*** amotoki has quit IRC03:33
*** mdnadeem has quit IRC03:36
*** burgerk has quit IRC03:36
*** esberglu has quit IRC03:37
*** ducttape_ has quit IRC03:38
*** raunak has quit IRC03:41
*** raunak has joined #openstack-nova03:42
openstackgerritTuan Luong-Anh proposed openstack/python-novaclient: Replaces uuid.uuid4 with uuidutils.generate_uuid()  https://review.openstack.org/40683503:45
*** hongbin has joined #openstack-nova03:54
*** rkmrhj has joined #openstack-nova03:58
*** tjones has joined #openstack-nova04:01
*** jwcroppe has joined #openstack-nova04:02
*** namnh has joined #openstack-nova04:05
*** int32bit has quit IRC04:06
*** jwcroppe has quit IRC04:07
*** amotoki has joined #openstack-nova04:07
*** bhagyashris_ has joined #openstack-nova04:09
*** diga_ has joined #openstack-nova04:10
*** rkmrhj_ has joined #openstack-nova04:10
*** bhagyashris_ has left #openstack-nova04:10
*** rkmrhj has quit IRC04:13
*** yamahata has joined #openstack-nova04:15
*** rkmrhj has joined #openstack-nova04:15
*** psachin has joined #openstack-nova04:17
*** rkmrhj_ has quit IRC04:17
*** prateek has joined #openstack-nova04:19
*** thorst has joined #openstack-nova04:19
*** sudipto has joined #openstack-nova04:20
*** sudipto_ has joined #openstack-nova04:20
*** pbandark has joined #openstack-nova04:24
*** thorst has quit IRC04:28
*** esberglu has joined #openstack-nova04:28
*** unicell has joined #openstack-nova04:32
*** hongbin has quit IRC04:33
*** mdnadeem has joined #openstack-nova04:34
*** esberglu has quit IRC04:37
*** diga_ has quit IRC04:38
*** yuntongjin has quit IRC04:39
*** trananhkma has quit IRC04:39
*** sudipto has quit IRC04:42
*** sudipto_ has quit IRC04:42
*** yingjun has joined #openstack-nova04:42
*** rkmrhj has quit IRC04:50
*** links has joined #openstack-nova04:52
*** tjones has left #openstack-nova04:55
*** ratailor has joined #openstack-nova04:56
*** amotoki has quit IRC04:58
*** amotoki has joined #openstack-nova05:03
*** jwcroppe has joined #openstack-nova05:03
*** jwcroppe has quit IRC05:08
*** yuntongjin has joined #openstack-nova05:11
openstackgerritTuan Luong-Anh proposed openstack/python-novaclient: Replaces uuid.uuid4 with uuidutils.generate_uuid()  https://review.openstack.org/40683505:14
*** amotoki has quit IRC05:14
*** yingjun has quit IRC05:17
*** Sukhdev has joined #openstack-nova05:22
*** thorst has joined #openstack-nova05:25
*** Zer0Byte__ has joined #openstack-nova05:27
*** esberglu has joined #openstack-nova05:27
*** esberglu has quit IRC05:32
*** thorst has quit IRC05:33
*** yuntongjin has quit IRC05:41
*** dimtruck is now known as zz_dimtruck05:48
*** yingjun has joined #openstack-nova05:49
*** amotoki has joined #openstack-nova05:50
*** Sukhdev has quit IRC05:52
*** jaosorior has joined #openstack-nova05:57
*** armax has quit IRC05:59
*** hfu has quit IRC06:03
*** jwcroppe has joined #openstack-nova06:04
*** hfu has joined #openstack-nova06:05
*** sridharg has joined #openstack-nova06:08
*** jwcroppe has quit IRC06:10
openstackgerritint32bit proposed openstack/nova: Add more actions to instance action logs  https://review.openstack.org/39651006:12
*** shuquan_ has joined #openstack-nova06:12
*** hfu has quit IRC06:13
*** int32bit has joined #openstack-nova06:15
*** kylek3h_ has quit IRC06:20
*** amotoki has quit IRC06:21
*** moshele has joined #openstack-nova06:21
*** kylek3h_ has joined #openstack-nova06:22
*** esberglu has joined #openstack-nova06:23
*** claudiub has joined #openstack-nova06:26
*** amotoki has joined #openstack-nova06:27
*** esberglu has quit IRC06:27
*** int32bit has quit IRC06:29
*** thorst has joined #openstack-nova06:30
openstackgerritwei-fan@rc.jp.nec.com proposed openstack/nova: Don't create instance_extra entry for deleted instance  https://review.openstack.org/41277106:30
gmannmriedem: alex_xu oomichi johnthetubaguy  looks like port conflict for placement API on devstack with senlin - http://lists.openstack.org/pipermail/openstack-dev/2016-December/109219.html06:32
gmanni am making it to 8780, hope its free - https://review.openstack.org/#/c/412770/06:33
openstackgerritAbhishek Kekane proposed openstack/nova: Use check_string_length from oslo_utils  https://review.openstack.org/28639006:36
*** lpetrut has joined #openstack-nova06:37
*** thorst has quit IRC06:38
*** jaosorior has quit IRC06:41
*** jaosorior has joined #openstack-nova06:42
*** raunak has quit IRC06:47
openstackgerritPawel Koniszewski proposed openstack/nova: DO NOT MERGE THE STEGOSAURUS  https://review.openstack.org/41166206:50
*** mjura has joined #openstack-nova06:51
*** yingjun has quit IRC06:53
*** ayogi has joined #openstack-nova06:53
*** unicell has quit IRC06:55
*** unicell has joined #openstack-nova06:56
*** mjura has quit IRC06:57
*** kylek3h_ has quit IRC06:57
*** cfriesen has quit IRC06:57
*** mjura has joined #openstack-nova06:58
*** rcernin has quit IRC06:59
*** yuntongjin has joined #openstack-nova07:04
*** jwcroppe has joined #openstack-nova07:06
*** shuquan_ has quit IRC07:08
openstackgerritVasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt  https://review.openstack.org/36441307:08
*** sreeram_v has joined #openstack-nova07:09
*** Oku_OS-away is now known as Oku_OS07:10
*** rcernin has joined #openstack-nova07:11
*** jwcroppe has quit IRC07:11
*** tesseract has joined #openstack-nova07:11
*** tesseract is now known as Guest1497207:11
*** unicell1 has joined #openstack-nova07:13
*** unicell has quit IRC07:15
*** esberglu has joined #openstack-nova07:17
*** esberglu has quit IRC07:22
*** yingjun has joined #openstack-nova07:26
*** rcernin has quit IRC07:26
*** moshele has quit IRC07:27
*** jamielennox is now known as jamielennox|away07:28
*** kylek3h_ has joined #openstack-nova07:30
*** amotoki has quit IRC07:31
*** lpetrut has quit IRC07:32
*** moshele has joined #openstack-nova07:33
*** diga has quit IRC07:34
*** thorst has joined #openstack-nova07:35
*** Gorian has quit IRC07:38
*** Gorian has joined #openstack-nova07:39
openstackgerritAlex Xu proposed openstack/nova: Adds support for versioned schema validation for query parameters  https://review.openstack.org/38900307:39
openstackgerritAlex Xu proposed openstack/nova: Use JSON-Schema to validate query parameters for keypairs API  https://review.openstack.org/39875507:40
alex_xugmann: interesting...07:40
alex_xuoomichi: I just replied on https://review.openstack.org/#/c/412473/1/nova/api/validation/__init__.py07:41
alex_xuoomichi: it is used by later patch07:41
*** rcernin has joined #openstack-nova07:42
*** Zer0Byte__ has quit IRC07:43
*** andreas_s has joined #openstack-nova07:43
*** thorst has quit IRC07:43
*** hfu has joined #openstack-nova07:47
*** sahid has joined #openstack-nova07:49
*** ihrachys has joined #openstack-nova07:50
*** ihrachys has quit IRC07:56
*** takashin has joined #openstack-nova07:58
*** irenab has quit IRC07:59
*** hfu has quit IRC08:00
openstackgerritRocDove proposed openstack/python-novaclient: Perfect interface_list function  https://review.openstack.org/41225808:00
*** hfu has joined #openstack-nova08:00
*** ihrachys has joined #openstack-nova08:02
*** hfu has quit IRC08:03
*** mashat has joined #openstack-nova08:06
*** mashat has quit IRC08:06
*** nherciu has joined #openstack-nova08:06
*** jwcroppe has joined #openstack-nova08:07
*** hfu has joined #openstack-nova08:07
*** hfu has quit IRC08:09
*** hfu has joined #openstack-nova08:10
*** jwcroppe has quit IRC08:12
*** esberglu has joined #openstack-nova08:13
*** jvgrant has quit IRC08:13
*** jvgrant has joined #openstack-nova08:13
*** esberglu has quit IRC08:17
*** hfu has quit IRC08:18
*** pcaruana has joined #openstack-nova08:19
*** zhs__ has joined #openstack-nova08:19
*** hfu has joined #openstack-nova08:20
*** hfu has quit IRC08:21
*** ihrachys has quit IRC08:22
*** zhs_ has quit IRC08:23
*** ihrachys has joined #openstack-nova08:25
openstackgerritTakashi NATSUME proposed openstack/nova: Enable cold migration with target host(3/3)  https://review.openstack.org/40896408:26
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Microversion 2.41 - Enable cold migration with target host  https://review.openstack.org/40670708:31
*** amotoki has joined #openstack-nova08:33
openstackgerrityunfeng zhou proposed openstack/nova-specs: add CONTRIBUTING.rst  https://review.openstack.org/41288208:34
*** amoralej|off is now known as amoralej08:35
*** rcernin has quit IRC08:36
*** ihrachys has quit IRC08:38
*** fragatin_ has joined #openstack-nova08:38
*** fragatina has quit IRC08:39
*** xinliang has quit IRC08:40
*** thorst has joined #openstack-nova08:40
*** hfu has joined #openstack-nova08:40
*** jaosorior has quit IRC08:41
*** jaosorior has joined #openstack-nova08:43
*** roeyc has joined #openstack-nova08:46
*** roeyc1 has joined #openstack-nova08:48
*** thorst has quit IRC08:48
*** hfu has quit IRC08:50
*** roeyc has quit IRC08:50
*** hfu has joined #openstack-nova08:51
*** zenoway has joined #openstack-nova08:53
bauzasgmann: hello08:53
bauzasgmann: could you please ask the senlin folks to ping me on #openstack-dev ?08:53
*** zenoway has quit IRC08:54
*** zenoway has joined #openstack-nova08:54
gmannbauzas: yea, i ll ping them, i agree to disable placement service in their setup but for long term we should have diff port08:55
bauzasgmann: actually, I just saw something wrong with their config08:55
bauzaslemme upload the change08:56
bauzasgmann: https://review.openstack.org/#/c/412915/08:56
gmannbauzas: cool08:56
*** yufei has quit IRC08:56
bauzasgmann: they wanted to declare some services to start with their jobs, but AFAICS that wasn't working08:56
bauzasgmann: that's because they used a wrong variable name08:57
bauzasthat change ^ should help them08:57
gmannah, nice catch08:57
*** hfu has quit IRC08:58
*** rcernin has joined #openstack-nova08:58
*** zzzeek has quit IRC09:00
*** xinliang has joined #openstack-nova09:00
*** zzzeek has joined #openstack-nova09:00
gmannbauzas:  and that was working till now because of  this - http://git.openstack.org/cgit/openstack-infra/devstack-gate/tree/devstack-vm-gate.sh#n27109:01
*** hfu has joined #openstack-nova09:03
*** lucas-afk is now known as lucasagomes09:03
*** lpetrut has joined #openstack-nova09:03
*** kevinz has quit IRC09:05
*** kevinz has joined #openstack-nova09:05
bauzasgmann: not really09:06
bauzasgmann: that's how I catched something was wrong09:06
*** hfu has quit IRC09:06
bauzasgmann: senlin is declaring heat in their job description, but I can't find a trace of it in http://logs.openstack.org/43/412843/1/check/gate-senlin-dsvm-tempest-api/cff7c55/logs/localrc.txt.gz09:06
*** preethipy has joined #openstack-nova09:07
*** esberglu has joined #openstack-nova09:08
*** jwcroppe has joined #openstack-nova09:08
*** lpetrut has quit IRC09:09
*** lpetrut has joined #openstack-nova09:09
*** esberglu has quit IRC09:12
*** claudiub has quit IRC09:13
*** jwcroppe has quit IRC09:13
gmannbauzas: nit sure about api/functional job but i think they need for integration job and its there - http://logs.openstack.org/58/412658/2/check/gate-senlin-dsvm-tempest-integration-nv/598e094/logs/localrc.txt.gz09:14
*** yufei has joined #openstack-nova09:16
bauzasgmann: well, the integration job is non-voting :)09:18
bauzaseither way, I just pointed out something unclear, hopefully we would find a better solution soon09:19
bauzashere, I just want them to have their gate pasing :)09:19
bauzaspassing even09:19
gmannbauzas:  yea but not sure they really do not need other default services09:19
*** amotoki has quit IRC09:21
takashingmann: could you check my comment in https://review.openstack.org/#/c/323287/ ?09:24
gmanntakashin: actually i default one is something configured right and if that is changed then our tests would work09:25
gmanntakashin: so while mocking we should mock config option instead is default class name which can be changed on env to env09:26
gmanntakashin: devstack runs with default so it would not be problem there09:26
takashingmann: devstack runs with default, so it is no problem and my patch need not be changed, right?09:28
gmanntakashin:  yea, its not issue with devstack run but if you run locally by overriding the default scheduler class then it issue09:29
gmanntakashin: if you see, where ever mock needs to be done for config value, it is done via flag setting instead of default value mock09:29
gmanntakashin: if you flag that option with default class which is CellsScheduler then tests will run always with CellsScheduler which you are going to mock09:30
*** yassine has joined #openstack-nova09:32
*** amotoki has joined #openstack-nova09:33
*** oanson has quit IRC09:33
*** yassine is now known as Guest2890709:33
takashingmann: so should I add "self.flags(scheduler='nova.cells.scheduler.CellsScheduler', group='cells')" ?09:34
gmanntakashin: yes that should work09:35
*** amotoki has quit IRC09:35
takashingmann: Thanks. I will add it.09:35
gmanntakashin:  cool, thanks09:35
*** matrohon has joined #openstack-nova09:36
*** derekh has joined #openstack-nova09:36
*** yingjun has quit IRC09:39
*** lpetrut1 has joined #openstack-nova09:43
*** lpetrut has quit IRC09:43
*** lpetrut1 is now known as lpetrut09:43
*** sridharg has quit IRC09:44
*** thorst has joined #openstack-nova09:45
*** sridharg has joined #openstack-nova09:46
*** lpetrut has quit IRC09:48
*** yufei has quit IRC09:51
*** yufei has joined #openstack-nova09:51
*** sudipto_ has joined #openstack-nova09:53
*** sudipto has joined #openstack-nova09:53
*** thorst has quit IRC09:54
*** ihrachys has joined #openstack-nova09:54
*** lpetrut has joined #openstack-nova09:56
*** kevinz has quit IRC09:56
*** namnh has quit IRC10:03
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: [WIP] Add version field to LibvirtLiveMigrateData  https://review.openstack.org/40460710:06
openstackgerritTakashi NATSUME proposed openstack/nova: Stop using mox stubs in nova/tests/unit/cells  https://review.openstack.org/32328710:06
*** mnasiadka has joined #openstack-nova10:06
*** yufei has quit IRC10:08
*** jwcroppe has joined #openstack-nova10:09
*** GB21 has joined #openstack-nova10:11
*** jwcroppe has quit IRC10:14
*** irenab has joined #openstack-nova10:15
*** ociuhandu has quit IRC10:17
*** bkopilov_ has quit IRC10:17
*** sudipto has quit IRC10:21
*** sudipto_ has quit IRC10:21
*** preethipy has quit IRC10:22
*** hoangcx has quit IRC10:22
*** armax has joined #openstack-nova10:22
*** ganesan_ has joined #openstack-nova10:23
ganesan_I am unable to launch an instance on 2nd compute node10:24
ganesan_getting te following error on nova-compute.log10:24
ganesan_ libvirtError: internal error: process exited while connecting to monitor: kvm version too old10:24
ganesan_ failed to initialize KVM: Inappropriate ioctl for device10:25
pkoniszewskitdurakov: hey, finally found why LM+grenade job fails every time - https://bugs.launchpad.net/nova/+bug/1649586/comments/1310:31
openstackLaunchpad bug 1649586 in OpenStack Compute (nova) ""HTTP exception thrown: Cannot 'os-migrateLive' instance while it is in task_state migrating" in gate-grenade-dsvm-neutron-multinode-live-migration-nv" [Medium,Confirmed]10:31
pkoniszewskitdurakov: trying to fix it - https://review.openstack.org/#/c/412922/ - but this one does not work as expected10:31
johnthetubaguygmann: bauzas: thats a good reason for us to all move into apache I guess?10:32
johnthetubaguygmann: bauzas: so we fight over URLs rather than silly port numbers few people will want to use10:32
bauzasjohnthetubaguy: yeah, I'm working on only using the HTTP port10:32
bauzasthe application port seems useless10:33
tdurakovpkoniszewski: acked, I wonder, why this one wasn't found on other jobs, does only live-migration job uses subnode?10:33
johnthetubaguybauzas: we have the scripts to already run in apache, will they not do that for you?10:33
bauzasjohnthetubaguy: not really https://github.com/openstack-dev/devstack/blob/master/lib/placement#L9210:34
pkoniszewskitdurakov: only grenade jobs are affected and only LM+grenade job needs to take transport_url from cell mapping when communicating between hosts (at least this is what we have covered in grenade jobs)10:34
bauzasjohnthetubaguy: but fortunately, we only use the HTTP port by the service catalog10:34
pkoniszewskiso that's why LM+grenade job is failing, but the other multinode job is not (yet)10:34
*** claudiub has joined #openstack-nova10:34
bauzashttps://github.com/openstack-dev/devstack/blob/master/lib/placement#L13310:35
bauzasjohnthetubaguy: ^10:35
johnthetubaguybauzas: oh, yeah, devstack totally doesn't do that yet, let me find the script I am thinking about10:35
bauzasso, we create a VirtualHost https://github.com/openstack-dev/devstack/blob/master/files/apache-placement-api.template#L310:36
bauzaswhich is good but we also bind the port https://github.com/openstack-dev/devstack/blob/master/files/apache-placement-api.template#L110:36
bauzasgiven we need it for the virtualhost10:37
bauzasfortunately, given that https://github.com/openstack-dev/devstack/blob/master/files/apache-placement-api.template#L18-L25 is not in the Virtualhost, that works too for HTTP port10:37
*** armax has quit IRC10:38
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: [WIP] Refactor _post_live_migration  https://review.openstack.org/41302110:39
*** GB21 has quit IRC10:40
*** zhangjl has quit IRC10:40
johnthetubaguybauzas: hmm, maybe those are the ones I was thinking about (https://github.com/openstack-dev/devstack/blob/master/files/apache-nova-api.template) I could have swarn there was something in tree from dims about those10:42
*** coreywright has joined #openstack-nova10:42
johnthetubaguybauzas: ah... https://github.com/openstack/nova/blob/master/nova/wsgi/nova-api.py10:43
johnthetubaguybauzas: https://blueprints.launchpad.net/nova/+spec/run-nova-services-under-apache210:43
johnthetubaguybauzas: ignore me, suddenly sparked a memory10:43
bauzasjohnthetubaguy: yeah I guess cdent copied the nova files10:43
bauzasjohnthetubaguy: but given he provided an application port, he binded a new port too10:44
*** yamahata has quit IRC10:44
bauzasjohnthetubaguy: honestly, I just think we need to cut the related virtualhost and just use the location10:44
johnthetubaguybauzas: I was just thinking we don't have anything in this directory for placement, but its probably because we don't need it: https://github.com/openstack/nova/blob/master/nova/wsgi10:44
bauzasjohnthetubaguy: indeed10:45
bauzasjohnthetubaguy: we don't run the wsgi server like this for placement10:45
bauzassec10:45
bauzasfinding you the right file10:45
*** yuntongjin has quit IRC10:45
*** sreeram_v has quit IRC10:46
johnthetubaguyI guess its the same ish; https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/wsgi.py10:46
bauzasjohnthetubaguy: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/wsgi.py10:46
johnthetubaguythere was talk of getting every openstack project using mod_wsgi in the gate, etc10:46
johnthetubaguyas a pike goal, I should look at where that is going10:47
*** ganesan_ has quit IRC10:48
*** hfu has joined #openstack-nova10:48
*** jklare has quit IRC10:48
*** esberglu has joined #openstack-nova10:48
*** pooja_j has joined #openstack-nova10:48
bauzasjohnthetubaguy: AFAIR, the placement API can run in any WSGI server, not only mod_wsgi10:49
johnthetubaguytotally10:50
bauzashttps://github.com/openstack/nova/blob/master/setup.cfg#L72 does the mapping10:50
johnthetubaguyI was more about removing the use of the eventlet wsgi server10:50
johnthetubaguyin general10:50
*** thorst has joined #openstack-nova10:50
bauzasjohnthetubaguy: ah, for nova you mean10:51
bauzasjohnthetubaguy: I wonder if there were some quirks left in the nova case10:51
johnthetubaguyI mean for everyone10:51
johnthetubaguywell, its not been tested for a long while10:51
bauzasoh okay :)10:51
johnthetubaguystop the whole discussion about ports, etc10:51
*** lucasagomes is now known as lucas-bbl10:53
*** esberglu has quit IRC10:53
*** mvk has quit IRC10:53
*** asettle has quit IRC10:53
openstackgerritTimofey Durakov proposed openstack/nova: Added missed "raises:" docstrings into numa_get_constraints() method  https://review.openstack.org/41056510:54
tdurakovsfinucan: could you please +2 this one once again^10:55
*** thorst has quit IRC10:58
*** asettle has joined #openstack-nova10:58
*** gcb has quit IRC10:59
*** amotoki has joined #openstack-nova10:59
*** hfu has quit IRC11:07
*** GB21 has joined #openstack-nova11:07
*** phuongnh has quit IRC11:07
*** jwcroppe has joined #openstack-nova11:10
*** amotoki has quit IRC11:10
*** hfu has joined #openstack-nova11:14
*** jwcroppe has quit IRC11:14
*** tbachman has quit IRC11:15
*** sdague has joined #openstack-nova11:20
*** matrohon has quit IRC11:20
*** gszasz has joined #openstack-nova11:20
bauzasjohnthetubaguy: mmm, that's interesting11:25
bauzashttps://github.com/openstack-dev/devstack/blob/master/files/apache-placement-api.template means that we could call the placement API by calling directly without the /placement endpoint11:25
bauzasie. https://placement_host:8778/ is the same than https://placement_host:80/placement AFAICU11:26
bauzasthat said, there is a difference11:26
*** mvk has joined #openstack-nova11:26
bauzashttps://github.com/openstack-dev/devstack/blob/master/files/apache-placement-api.template#L4 means that we're using mod_wsgi in deamon while using /placement location is by embedded mode11:27
bauzassdague: just in case you want to know the context, http://lists.openstack.org/pipermail/openstack-dev/2016-December/109219.html is related to the placement API11:29
openstackgerritOleg Bondarev proposed openstack/nova: Live migration: wait for vif-plugged event on pre live migration  https://review.openstack.org/24691011:30
*** smatzek has joined #openstack-nova11:37
*** ducnc has quit IRC11:39
*** ducnc1 has joined #openstack-nova11:39
*** amotoki has joined #openstack-nova11:40
*** ducnc1 is now known as ducnc11:41
*** hfu has quit IRC11:41
*** yufei has joined #openstack-nova11:43
openstackgerritFeodor Tersin proposed openstack/nova: Implement ScaleIO image backend  https://review.openstack.org/40744011:43
*** ftersin has joined #openstack-nova11:45
bauzassdague: I actually need your opinion about how we use mod_wsgi11:50
*** fragatin_ has quit IRC11:51
andreykurilinrobcresswell: hi! sorry, I was too tired yesturday. Returning to our talk. To fix cinderclient, you can try to construct keystone session manually and transmit it while initialization cinderclient instead of transmission the bunch of arguments.11:51
bauzassdague: AFAICS, we leave the possibility for the operator to have all the wsgi apps running within the same default host and just have a location directive for pointing to the right service, or if the deployer uses the USE_VENV flag, then we create a virtualenv per service, right?11:51
robcresswellandreykurilin: Hey, no problem. Honestly, I think we will wait until the python-*clients fix themselves. It seems cinder has patches too.11:52
robcresswellDoesnt look like Ocata will get mv support, but oh well :/11:52
bauzasin that case, each service type runs behind the port it was originally having like if it was using eventlet11:52
bauzassdague: am I correct ?11:53
*** yufei has quit IRC11:53
*** ociuhandu has joined #openstack-nova11:53
andreykurilinrobcresswell: :(what about nova? Will you add support of nova's microversions in Ocata?)11:54
andreykurilinsince we will have a proper novaclient's release soon11:55
robcresswellandreykurilin: If there's a release of novaclient which works, and I have time post-christmas11:55
*** prateek has quit IRC11:55
*** thorst has joined #openstack-nova11:55
robcresswellI'll be able to make time11:55
robcresswellBut if the client gets held up, it is unlikely, as FF is <1 month now IIRC.11:56
pooja_jsdague: Hi11:56
johnthetubaguybauzas: for what I know, there is no evetlet when you use mod_wsgi, it uses threads instead, I think. keystone have been doing that the longest I thought11:56
bauzasjohnthetubaguy: sorry I was unclear11:56
andreykurilinrobcresswell: you checked our master branch, it works. release is scheduled to the end of this week, so everything should be ok.11:56
bauzasjohnthetubaguy: I mean, if you use eventlet, then your services are running on a dedicated port11:56
*** rtheis has joined #openstack-nova11:57
johnthetubaguybauzas: ah, yep, yep11:57
bauzasjohnthetubaguy: so the service catalog mentions those ports11:57
johnthetubaguyyep11:57
robcresswellandreykurilin: Sure, that would be good11:57
bauzasjohnthetubaguy: IIUC, when not using the venvs (which is the default opt value), all the services can be reached by the default HTTP and HTTPS ports11:57
bauzasusing the location directive, which I'm totally okay11:57
bauzasif you want to keep your service catalog entries *and* want to use mod_wsgi, then you have to use the venvs11:58
*** tuanluong has quit IRC11:58
bauzasaaaaand oops11:58
bauzass/venv/vhost/ (too much Python in my brain)11:59
bauzasooooh nevermind11:59
bauzasI didn't read correctly lib/nova12:00
bauzasUSE_VENV is just for pointing to the right python virtualenv12:00
bauzastotally unrelated to the virtualhosts12:00
andreykurilinrobcresswell: have nice holidays! I'll ping you when we have any news12:00
robcresswellandreykurilin: You too, thanks a lot for your help :)12:00
bauzasokay, so that goes to the big question I'm wondering12:01
bauzasif we can just remove the use of virtualhosts for the placement service12:01
*** thorst has quit IRC12:04
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Improve get_disk_type_from_path  https://review.openstack.org/41194212:04
*** lucas-bbl is now known as lucas-hungry12:04
openstackgerritAndrey Volkov proposed openstack/nova: Add debug possibility for nova-manage command  https://review.openstack.org/33215812:04
*** nicolasbock has joined #openstack-nova12:06
*** hfu has joined #openstack-nova12:08
openstackgerritwei-fan@rc.jp.nec.com proposed openstack/nova: Don't create instance_extra entry for deleted instance  https://review.openstack.org/41277112:08
*** GB21 has quit IRC12:09
*** jwcroppe has joined #openstack-nova12:11
*** jwcroppe has quit IRC12:16
openstackgerritAndrey Volkov proposed openstack/nova: Add ability to view quota details for project owner  https://review.openstack.org/38600812:18
*** hfu has quit IRC12:19
*** amotoki has quit IRC12:20
openstackgerritAndrey Volkov proposed openstack/nova: Add create cell command  https://review.openstack.org/33271312:21
openstackgerritAndrey Volkov proposed openstack/nova: Trivial: add ability to define action description  https://review.openstack.org/35578112:21
pooja_jsdague: I want to ask query regarding hacking project.12:23
*** ratailor has quit IRC12:25
*** lucas-hungry is now known as lucasagomes12:26
*** GB21 has joined #openstack-nova12:27
*** yuntongjin has joined #openstack-nova12:34
*** catintheroof has joined #openstack-nova12:37
*** thorst has joined #openstack-nova12:38
*** GB21 has quit IRC12:38
*** sreeram_v has joined #openstack-nova12:38
*** amotoki has joined #openstack-nova12:40
*** amotoki has quit IRC12:40
*** amotoki has joined #openstack-nova12:41
*** sreeram_v has quit IRC12:42
*** ociuhandu has quit IRC12:43
*** ociuhandu has joined #openstack-nova12:43
*** ociuhandu has quit IRC12:48
*** GB21 has joined #openstack-nova12:50
*** yingjun has joined #openstack-nova12:51
*** portdirect_ has joined #openstack-nova12:53
*** ociuhandu has joined #openstack-nova12:54
*** gouthamr has joined #openstack-nova12:55
*** matrohon has joined #openstack-nova12:55
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004212:57
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action  https://review.openstack.org/41004212:57
*** asettle has quit IRC12:59
*** ociuhandu has quit IRC12:59
pkoniszewskitdurakov: are we running live migration meeting today?13:01
*** pcaruana has quit IRC13:01
tdurakovpkoniszewski: yes13:01
tdurakovit will start in an hour13:02
*** vladikr has joined #openstack-nova13:02
*** hoangcx has joined #openstack-nova13:02
pkoniszewskitdurakov: okay, thanks!13:02
tdurakovpkoniszewski: what do you know for instance.migration_context?13:02
tdurakovpkoniszewski: when is it populated for live-migration?13:02
sdaguebauzas: looking at backscroll here, can you reframe?13:04
*** vladikr has quit IRC13:06
*** vladikr has joined #openstack-nova13:06
*** pcaruana has joined #openstack-nova13:06
*** dave-mccowan has joined #openstack-nova13:07
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix parameter type in servers-admin-action.inc  https://review.openstack.org/41307513:08
*** ociuhandu has joined #openstack-nova13:09
pkoniszewskitdurakov: currently it isn't13:10
tdurakovpkoniszewski: it's created during move_claim, right?13:10
pkoniszewskithat's right13:10
*** takashin has left #openstack-nova13:12
*** jwcroppe has joined #openstack-nova13:12
pkoniszewskitdurakov: this is the patch to add claims to the live migration flow - https://review.openstack.org/#/c/244489/13:12
tdurakovpkoniszewski: than we have a bug, because https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3976 will not work for live-migration13:12
pkoniszewskitdurakov: you might be right13:13
*** ociuhandu has quit IRC13:14
tdurakovpkoniszewski: I'm working on it right now, let's discuss that on a meeting, because I'd prefer to created mig_context at the time migration object is created, and then just pass it down to computes13:14
tdurakovpkoniszewski: but I need to double check your fix13:14
*** lamt has quit IRC13:14
pkoniszewskitdurakov: I think all we need is the patch that I mentioned13:14
tdurakovpkoniszewski: possible:) reading it right now13:15
*** Dinesh_Bhor has quit IRC13:16
*** pooja_j has quit IRC13:16
*** bhagyashris has quit IRC13:16
*** jwcroppe has quit IRC13:16
openstackgerritjichenjc proposed openstack/nova: Add document update for get console usage  https://review.openstack.org/41178913:18
*** liverpooler has joined #openstack-nova13:21
*** jaypipes has joined #openstack-nova13:23
*** hfu has joined #openstack-nova13:23
tdurakovpkoniszewski: got question on that: https://review.openstack.org/#/c/244489/49/nova/compute/manager.py@5215 - how is it possible?13:24
*** ociuhandu has joined #openstack-nova13:24
pkoniszewskitdurakov: this might happen when in the middle of live migration compute node will be deleted13:26
pkoniszewskitdurakov: but not sure how this might affect the rest of code13:26
tdurakovpkoniszewski: so it's gonna be a case when target node, that executes this code in that moment is deleted from nova, right?13:27
pkoniszewskitdurakov: we basically do the same here https://review.openstack.org/#/c/244489/49/nova/compute/manager.py@5719 in post live migraiton steps at destination13:27
pkoniszewskitdurakov: that's right, I'm not aware of any other cases13:27
bauzassdague: sorry, I'm a bit out for lunch13:28
*** bhagyashris has joined #openstack-nova13:28
*** Dinesh_Bhor has joined #openstack-nova13:28
*** pooja_j has joined #openstack-nova13:28
tdurakovpkoniszewski: do we really need to handle that exception?13:28
bauzassdague: I'm fine with your opinion in the ML13:28
tdurakovpkoniszewski: I'd let migration to fail, honestly13:28
bauzassdague: I just want to make sure that deployers like Fuel and TripleO know that too13:28
*** sudipto_ has joined #openstack-nova13:28
*** sudipto has joined #openstack-nova13:28
tdurakovit's a pre-step, and during it we found that target node isn't in environment anymore13:28
pkoniszewskitdurakov: this doesn't make things worse and there is an option that after such exception everything will work fine13:29
*** yuntongjin has quit IRC13:29
pkoniszewskitdurakov: we would need to dig into how scheduler picks up a node and how adding/removing this exception affects everything else13:29
tdurakovpkoniszewski: ++13:30
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Improve get_disk_type_from_path  https://review.openstack.org/41194213:30
*** ducttape_ has joined #openstack-nova13:30
openstackgerritPawel Koniszewski proposed openstack/nova: live-mig: Add claims and proper resource tracking  https://review.openstack.org/24448913:30
pkoniszewskinot sure it is worth it, this does not look like an issue13:30
pkoniszewskisdague: hey, can I ask you for help? I have trouble with setting up LM+grenade job with cells v2 configured13:31
tdurakovpkoniszewski: it's definitely not a blocker, would be good to add a note anyway13:31
pkoniszewskisdague: grenade configures the controller+compute node (rabbit transport url in particular) in the way that it blocks communication for subnode-2, i.e., it makes it impossible to start pre live migration steps13:32
*** jaypipes has quit IRC13:33
pkoniszewskisdague: tried to fix this in the most simple and probably dumbest way, but it doesn't work as expected - https://review.openstack.org/#/c/412922/13:33
*** pradk has joined #openstack-nova13:35
*** pradk has quit IRC13:35
*** ociuhandu has quit IRC13:38
*** ociuhandu has joined #openstack-nova13:38
*** pradk has joined #openstack-nova13:40
*** amoralej is now known as amoralej|lunch13:43
*** jheroux has joined #openstack-nova13:45
*** pbandark has quit IRC13:45
*** jwcroppe has joined #openstack-nova13:46
*** yingjun has quit IRC13:48
*** ociuhandu has quit IRC13:50
*** cleong has joined #openstack-nova13:57
*** GB21 has quit IRC13:59
*** GB21 has joined #openstack-nova14:00
*** esberglu has joined #openstack-nova14:00
*** sudipto_ has quit IRC14:00
*** sudipto has quit IRC14:00
*** catinthe_ has joined #openstack-nova14:06
*** mtanino has quit IRC14:08
*** catintheroof has quit IRC14:08
lbeliveaujaypipes: would be great if you could review this patch https://review.openstack.org/#/c/405575/14:10
*** vladikr_ has joined #openstack-nova14:11
*** ducttape_ has quit IRC14:12
*** lamt has joined #openstack-nova14:13
*** baoli has joined #openstack-nova14:18
*** baoli has quit IRC14:19
*** baoli has joined #openstack-nova14:19
openstackgerritSergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API.  https://review.openstack.org/35554014:20
*** prateek has joined #openstack-nova14:23
*** pcaruana has quit IRC14:23
openstackgerritSergey Nikitin proposed openstack/nova: Standardization of VM diagnostics info API.  https://review.openstack.org/35554014:24
*** salv-orlando has joined #openstack-nova14:24
*** ayogi has quit IRC14:25
*** jamesdenton has joined #openstack-nova14:25
*** hfu has quit IRC14:29
openstackgerritFrancesco Santoro proposed openstack/nova: add new variable to specify fast path vhostuser port type  https://review.openstack.org/41311114:30
GMAzrael_Anyone dealt with low timeouts on websockify for console access?14:30
*** links has quit IRC14:32
openstackgerritFrancesco Santoro proposed openstack/nova: add new variable to specify fast path vhostuser port type  https://review.openstack.org/41311114:36
*** mlavalle has joined #openstack-nova14:37
*** pcaruana has joined #openstack-nova14:37
*** clayton has quit IRC14:38
bauzassdague: here is the devstack change for removing the application port https://review.openstack.org/#/c/413118/14:39
bauzassdague: note that I needed to keep the virtualhost directive because the errrorlog directive can't be for a directory context14:40
bauzashttp://httpd.apache.org/docs/current/mod/core.html#errorlog14:40
*** clayton has joined #openstack-nova14:40
*** psachin has quit IRC14:42
mriedemandreykurilin: in https://review.openstack.org/#/c/280726/20/releasenotes/notes/deprecate_contrib_extensions-0ec70c070b09eedb.yaml what does this mean: "There are no modules and extensions for "deferred_delete", "host_evacuate", "host_evacuate_live" and "metadata_extensions" anymore. Previously, they contained only shell commands and shell module auto loads them (there is no ability to not do it)."14:42
sdaguebauzas: you are probably going to need to match the ServerName differently14:43
*** eharney has quit IRC14:43
bauzassdague: the name is useless, I'm matching it thanks to the WSGIServerGroup directive14:43
bauzasoops14:44
sdaguebauzas: it's not useless14:44
bauzasthe processgroup I meant14:44
sdagueapache uses ServerName to route requests14:44
bauzassdague: I mean, I just don't want to enter that virtualhost based on the Host header14:44
andreykurilinmriedem: host_evacuate.py modules conrained only shell methods. No manager which can be used by novaclient.v2.client.Client. Also stuff is done in shell layer14:44
bauzassdague: I could have named it Fdfdsfsfsrfs14:44
sdagueoh, gotcha14:44
sdaguegod, that's weird14:45
bauzassince it's bound on *:8014:45
bauzasyeah14:45
bauzasI know14:45
bauzasor I could just leave the virtualhost be *:8778 without binding the port14:45
andreykurilinmriedem: In one hour I'll start work on fixing all nits which left. I think it would be great place to fix all such unclear moments.14:45
bauzasand not provide a ServerName directive if that makes it clearer14:45
mriedemandreykurilin: i'm going through the release notes now, you can review when i have that up14:46
andreykurilinmriedem: cool14:46
*** mdnadeem has quit IRC14:47
*** ociuhandu has joined #openstack-nova14:47
*** moshele has quit IRC14:48
*** Guest67717 is now known as med_14:48
*** med_ has quit IRC14:48
*** med_ has joined #openstack-nova14:48
*** jaosorior has quit IRC14:48
openstackgerritJohn Garbutt proposed openstack/nova: WIP: Modify allocate_for_instance to skip create_ports  https://review.openstack.org/41195414:48
openstackgerritJohn Garbutt proposed openstack/nova: WIP: pass objects into _create_ports_for_instance  https://review.openstack.org/41195314:48
openstackgerritJohn Garbutt proposed openstack/nova: WIP: _validate_requested_network_ids returns objects  https://review.openstack.org/41195214:49
openstackgerritJohn Garbutt proposed openstack/nova: Objectify requests_and_created_ports  https://review.openstack.org/40539814:49
openstackgerritJohn Garbutt proposed openstack/nova: WIP: remove uses of network in _update_ports_for_instance  https://review.openstack.org/40539914:49
openstackgerritJohn Garbutt proposed openstack/nova: WIP: remove nets param from _update_ports_for_instance  https://review.openstack.org/40540014:49
openstackgerritJohn Garbutt proposed openstack/nova: WIP: create fake ordered_nets for get_instance_nw_info  https://review.openstack.org/40545214:49
openstackgerritJohn Garbutt proposed openstack/nova: WIP extract _validate_req_and_create_ports  https://review.openstack.org/40540114:49
openstackgerritJohn Garbutt proposed openstack/nova: WIP: Objectify _validate_requested_network_ids  https://review.openstack.org/41195014:49
openstackgerritJohn Garbutt proposed openstack/nova: WIP: make _filter_hypervisor_macs use objects  https://review.openstack.org/40545114:49
openstackgerritJohn Garbutt proposed openstack/nova: WIP: _validate_requested_network_ids takes objects  https://review.openstack.org/41195114:49
openstackgerritJohn Garbutt proposed openstack/nova: WIP: make allocate_for_instance take consistent args  https://review.openstack.org/41312214:49
openstackgerritJohn Garbutt proposed openstack/nova: WIP: introduce pre_allocate_for_instance method.  https://review.openstack.org/41312314:49
openstackgerritJohn Garbutt proposed openstack/nova: WIP: Call pre_allocate_for_instance before scheduler  https://review.openstack.org/41312414:49
*** GB21 has quit IRC14:49
*** cfriesen has joined #openstack-nova14:50
bauzassdague: actually, when looking at the documentation, I think I can probably safely remove the ServerName directive since httpd just lookups the virtualhosts one by one14:50
mriedemandreykurilin: "All modules of novaclient.v2.contrib are deprecated now and will be removed after OpenStack Pike."14:50
mriedemdoes that include the host_evacuate CLI?14:50
bauzassdague: if we already have a virtualhost for *:80 without a Servername directive, it won't apply the placement virtualhost to any request14:51
mriedemandreykurilin: or that just means that they were moved up from novaclient.v2.contrib to novaclient.v2 but continue to work for imports14:51
mriedeme.g. for novaclient.v2.contrib.cells14:52
bauzassdague: but the risk is that if we do remove a generic virtualhost, then the placement api could suddently be the main virtualhost for all requests going to port 8014:52
andreykurilinmriedem: the second choice14:52
*** ducttape_ has joined #openstack-nova14:54
*** zenoway has quit IRC14:55
*** zz_dimtruck is now known as dimtruck14:55
sdaguebauzas: actually, I'd say leave the ServerName14:55
bauzasheh, the fun stuff is that 'horizon' is alphabetically before 'placement'14:56
sdagueI think it's safer that way14:56
mriedembauzas: on https://review.openstack.org/#/c/407630/ should we just report a bug to restrict resource provider names?14:56
*** mnasiadka has quit IRC14:56
bauzasso the placement virtualhost directive should never be read without servername if we enable horizon at the same time14:56
bauzasbut that's a big risk14:56
*** tjones has joined #openstack-nova14:57
bauzasmriedem: a bug, not sure14:57
bauzasmriedem: but yeah, we should somehow note that14:57
bauzasand restrict the API validation14:57
openstackgerritDiana Clarke proposed openstack/nova: Missing usage next links in api-ref  https://review.openstack.org/41263214:59
*** zenoway has joined #openstack-nova14:59
*** thorst has quit IRC14:59
*** hoangcx has quit IRC15:02
*** amoralej|lunch is now known as amoralej15:02
mriedembauzas: ok i think we're missing a test in https://review.openstack.org/#/c/407630/15:02
mriedemfor invalid uuid in the uuid=in: case15:03
*** tjones has quit IRC15:03
mriedembauzas: i had also asked a question in a previous patch set about the use case for this, i.e. how is the scheduler going to use this change15:03
bauzasmriedem: excellent point15:03
*** tjones has joined #openstack-nova15:03
bauzasmriedem: really? lemme see15:03
*** eharney has joined #openstack-nova15:04
mriedemPS6 commit message15:04
bauzasmriedem: sorry, lemme explain :)15:04
*** dimtruck is now known as zz_dimtruck15:05
bauzasmriedem: for example, I want to get a list of RPs named 'foo' and 'bar'15:05
bauzasI mean, I know both of their names, I just want their values15:05
bauzasfor the moment, I can only GET /rp?name=foo and then GET /rp?name=bar15:06
*** thorst has joined #openstack-nova15:06
mriedembauzas: so the scheduler is going to get the list of RPs for the simple filters like VCPU and RAM?15:08
mriedemis that the idea?15:08
bauzasmriedem: oh you mean the last change needed ?15:08
mriedemwell i understand how the API change can be used15:08
mriedembut i was trying to understand how it ties into the bigger picture15:09
bauzasmriedem: yeah, so I'll lookup the Requestspec object for those fields, and then call the REST API using those fields indeed15:09
mriedemof integrating the scheduler and the placement service15:09
*** raildo has joined #openstack-nova15:09
bauzasand then I'll get a list of UUIDs by return15:09
mriedemwell you'll get a list of resource providers which you can get the uuids from15:09
mriedembut yes15:09
bauzasyeah, and then like I did for the requested_destination, we would only call the filters with those uuids15:10
bauzasgiven the RP UUID equals the Computenode uuid15:11
*** tjones has left #openstack-nova15:11
bauzasmriedem: but lemme write that change15:11
*** eharney has quit IRC15:12
mriedemare we conflating resource providers with resource classes?15:12
mriedemresource provider names are unique15:13
bauzasmriedem: oh, one point, https://review.openstack.org/#/c/407630/8/nova/api/openstack/placement/handlers/resource_provider.py is probably needing a microversion15:13
mriedemand i think the rp.name for a compute node is the hypervisor_hostname from the resource tracker, right?15:13
bauzasmriedem: indeed15:13
*** zenoway has quit IRC15:13
mriedembauzas: yeah i thought about a microversion15:13
bauzasokay, I'll add a microversion then15:13
bauzasand a reno note15:14
*** matrohon has quit IRC15:14
mriedemi'm still a bit lost of on how the scheduler is getting a list of resource providers by name if the scheduler doesn't have a list of compute nodes first, and i thought we weren't doing the ComputeNodeList.get_all() from the scheduler for these15:15
bauzasmriedem: to be clear, I was thinking of changing https://github.com/openstack/nova/blob/master/nova/scheduler/filter_scheduler.py#L134 by returning the list of compute nodes matching the uuids15:15
*** oanson has joined #openstack-nova15:16
mriedemwouldn't we instead want to get the inventories for a given resource class, like VCPU where inventory is available to satisfy the VCPU in the request spec?15:16
mriedemand then from those inventories, we build our list of resource providers where can_host=True and then from those, get the compute nodes to pass to the other complex filters15:17
bauzasmriedem: the placement API would return the RPs matching the requested amounts for each resource class15:17
bauzasmriedem: ie. the call to /RPs?resources=...15:17
bauzasthat would give us a list of compute nodes that match the request15:18
*** zenoway has joined #openstack-nova15:18
mriedemsure, but then why do i need to get a list of RPs by name again?15:18
mriedemi don't have the names without the compute nodes right? but i thought we were getting the RPs first based on the request spec so we could then widdle the available compute nodes to query from the DB15:18
bauzasmriedem: oh you mean the first change in the series ? ?15:18
bauzasmriedem: if so, I see why it confused you15:19
mriedemi'm trying to figure out why we need https://review.openstack.org/#/c/407630/ at all15:19
bauzasmriedem: that's because a former attempt of providing the RP API was by returning some location header calling that GET /RP?in=15:20
bauzasmriedem: but now that we agreed on using directly the resource usage in GET /RPs?resource= that's unrelated yeah15:20
bauzasI mean, I'm not using it at all, and I don't plan to use it scheduler-side15:20
bauzassorry if that confused you, I can just remove the dependency15:20
mriedembauzas: yeah so if uuid=in: and name=in: are things we don't actually need, i think i'd rather just drop that patch15:21
*** Drankis has joined #openstack-nova15:21
bauzasmriedem: we don't need it yet for the next changes, agreed15:21
mriedembecause if the scheduler doesn't need it, then it seems like just adding something into the API that we have to support but don't actually care about15:21
bauzasso we can just cut the dependency15:21
mriedemi.e. we have that comma in the RP name issue to deal with b/c of this15:22
bauzasmriedem: well, the usecase still remains IMHO, that's just a matter of prio15:22
bauzasbut yeah, I see your opinion15:22
*** jd__ has left #openstack-nova15:22
bauzaswe could restrict the API *before* merging that change, agreed15:22
*** zenoway has quit IRC15:22
mriedemIMO it's not a use case until we have a user of the functionality :)15:22
bauzasheh15:22
mriedemelse it's just technical debt15:22
bauzasagreed :)15:23
mriedemso how about we just abandon for now?15:23
mriedemi really don't want this slipping in on accident15:23
mriedemit already has a +2 on it15:23
bauzasheh :)15:23
bauzasokay, I'll -2 it15:23
mriedemthanks15:23
mriedemsorry for the runaround15:23
bauzasheh, thanks for your patience rather :)15:23
bauzassorry if I confused you15:24
bauzasbut man, I'm gonna kill mr. jay's stats if I just -2 :p15:24
openstackgerritMatt Riedemann proposed openstack/python-novaclient: Clarify some release notes prior to the 7.0.0 release  https://review.openstack.org/41313615:24
mriedemandreykurilin: ^15:24
mriedembauzas: i'll -2 if you don't15:25
mriedemor just abandon15:25
mriedemthen no stats problems15:25
bauzasI can abandon :)15:25
bauzasI mean, I can support that :)15:25
bauzasBAAAAM15:27
mriedemthanks15:27
*** armax has joined #openstack-nova15:27
andreykurilinmriedem: thanks, will look15:28
mriedemandreykurilin: and the release request https://review.openstack.org/#/c/413138/ which is a WIP until the reno update is merged15:28
andreykurilinmriedem: let's merge one more thing before release - https://review.openstack.org/#/c/412588/ . I'll try to finish it right now15:29
andreykurilinhttps://review.openstack.org/#/c/412588/1/novaclient/client.py <- this one can be critical15:30
*** ayoung has quit IRC15:30
*** zenoway has joined #openstack-nova15:32
mriedemandreykurilin: why is that critical?15:32
mriedemi know trove was doing it,15:32
mriedembut i'm not sure why it's a huge deal - because it overrides what's in the session client we create?15:32
BobBallDoes anyone know why Citrix XenServer CI was removed from the nova-ci group?  I hadn't realised it had been removed15:32
*** mtanino has joined #openstack-nova15:32
mriedemBobBall: yeah i removed it,15:33
BobBall(or it was so long ago that I've forgotten :D)15:33
mriedembecause of the vif plugging timeouts15:33
*** nklenke has joined #openstack-nova15:33
mriedemlast week15:33
mriedempinged you about it15:33
BobBallAhhhh - right - I didn't realise that had prompted a removal.15:33
mriedemand it was in the nova meeeting minutes last week15:33
mriedemwell it was -1ing a lot15:33
BobBall(or had forgotten - I do tend to forget these things)15:33
BobBallCan we be re-added?  Various fixes are now live and we haven't seen the timeout issue recently15:34
mriedemi had requested / suggested that you set vif_plugging_timeout_fatal=False15:34
mriedemthis was the bug https://bugs.launchpad.net/nova/+bug/164099315:34
openstackLaunchpad bug 1640993 in OpenStack Compute (nova) "xenserver hits vif plugging timeout with neutron CI job" [High,Confirmed]15:34
mriedemi see https://review.openstack.org/#/c/390931/ hasn't landed,15:34
mriedembut i see the latest comment in the bug says there was a change in the ci env to make it faster15:34
*** eharney has joined #openstack-nova15:34
andreykurilinmriedem: imo, it is critical because some guys uses wrong behaviour. they do not transmit auth_token via arguments, just set them after novaclient initialization. In such case, if they do not transmit auth_token via password argument (it looks like trove and horizon do it), SessionClient will be unable to authenticate.15:35
BobBallYes - using the python bindings for OVS commands rather than ovs-ofctl going through the rootwrap.  That's just a configuration change.15:35
robcresswellandreykurilin: If we're initialising things wrong, we can change that15:36
mriedemandreykurilin: wouldn't that just result a failure on their end?15:36
*** zenoway has quit IRC15:36
mriedemi.e. why would we have a warning in place saying 'don't do this' even though it's just going to break and fail anyway?15:36
BobBallmriedem: ignore me for a minute.15:37
andreykurilinrobcresswell: I told you yesturday about the change which should be done at horizon-side ;)15:37
robcresswellandreykurilin: Yep, I just didn't realise it was a necessity, until this discussion :)15:38
EmilienMbauzas: ack for https://review.openstack.org/#/c/406300/13/manifests/keystone/auth_placement.pp15:38
EmilienMbauzas: I'll address that in a separated patch. This parameter can be overriden by our users. What we need here is documentation15:38
andreykurilinmriedem: my patch adds a workarounds which should resolve a failure (re-set keystone auth plugin with right auth_token after novaclient initialization)15:39
bauzasEmilienM: yeah, that's a problem related to how we know the service ports15:39
BobBallmriedem: I've got a few things to check; will come back to you tomorrow15:40
bauzasEmilienM: given we don't have a port registry for the service catalog, we need to make sure we don't kill another project when we add a new one15:40
mriedemBobBall: ok15:40
mriedemEmilienM: http://docs.openstack.org/developer/nova/placement.html#deployment15:40
bauzasEmilienM: here, devstack was straightly using the default http port for configuring the service catalog, so I think that has to be the value for all deployments15:40
mriedemwhatever happens with the port thing for placement, ^ will need to be updated as it references 877815:41
bauzasmriedem: you missed a discussion this morning15:41
mriedembauzas: i wouldn't say i 'missed' it15:41
mriedemi'll defer to you, sdague and cdent on that port issue15:41
mriedemi know people were talking about eventlet but the placement API never used eventlet15:41
mriedemor supported it15:42
bauzasmriedem: sorry, I wasn't sure you saw it15:42
mriedemso it doesn't seem valid to talk about placement + eventlet15:42
andreykurilinmriedem: I mean http://xsnippet.org/362182/ possibly worked before switching to SessionClient and such behaviour doesn't work now for sure.  after my latest patch, it should work again15:42
mriedemandreykurilin: ok15:42
bauzasmriedem: lemme just write a change for updating http://docs.openstack.org/developer/nova/placement.html#deployment which is wrong then15:42
EmilienMmriedem: thanks, I used this doc to write the puppet-nova patch15:42
EmilienMbut it mentions "Devstack sets up the port used in the endpoint URL as 8778 which is intentionally different from the 8774 port used with the compute endpoint."15:42
EmilienMso I used this port :-)15:42
bauzasEmilienM: yeah, I'm changing thaty15:43
mriedemEmilienM: yeah, and i doc'ed what devstack did15:43
mriedemanywho15:43
EmilienManyway, we'll follow what you guys suggest. no biggies15:43
bauzasmriedem: not really :( https://github.com/openstack-dev/devstack/blob/51a225c5d7f7d8655be62cab0e4d677b6839f35a/lib/placement#L13315:43
bauzasdevstack actually did 2 things15:43
andreykurilinmriedem: should I finish my patch? or just push a patch to update doc with right way to initialize novaclient?15:44
bauzas#1 httpd was binding the port 8778 indeed15:44
bauzas#2 but the service catalog was reffering to port 8015:44
bauzasso port 8778 was not called15:44
*** dave-mccowan has quit IRC15:44
sdagueright, the extra port bit is misleading, as we never used it for anything15:44
mriedemandreykurilin: well, if we release novaclient 7.0.0 and a bunch of projects start failing because of this, then we'll either end up adding it in anyway, or those projects will have to adjust and fix their bug15:48
mriedemandreykurilin: it'd be better to preempt those failures but....15:48
*** harlowja has joined #openstack-nova15:49
*** zenoway has joined #openstack-nova15:49
andreykurilinmriedem: I prefer to fix it in failed projects (horizon and trove should work) :)15:50
mriedembauzas: sdague: on https://review.openstack.org/#/c/413118/ i guess it will run with placement-api by default now, so that's good,15:53
mriedemi know before that we just had the placement job running in experimental for devstack15:53
*** zenoway has quit IRC15:54
*** ducttape_ has quit IRC15:56
kukaczhi, I'd like to reopen a topic from yesterday - I seek for a best process how to remove instances which can't be deleted cleanly using `nova delete`, `nova force-delete` even with support of reset-state15:58
kukaczshould I do it directly on the database or is the process using nova.db.sqlalchemy perhaps more smooth to handle that?15:59
*** hongbin has joined #openstack-nova16:01
sdaguemriedem: yeh, the test results on the devstack change should be all good16:01
*** markvoelker_ has joined #openstack-nova16:01
sdagueit's green across the board so far16:02
sdague2 test runs still waiting to report16:02
*** dave-mccowan has joined #openstack-nova16:02
*** markvoelker has quit IRC16:03
*** baoli has quit IRC16:03
*** zenoway has joined #openstack-nova16:03
*** markvoelker has joined #openstack-nova16:04
*** prateek_ has joined #openstack-nova16:05
*** Oku_OS is now known as Oku_OS-away16:06
mriedemkukacz: this is probably a question better asked in the #openstack-operators channel as i'm sure someone there has dealt with this16:06
*** baoli has joined #openstack-nova16:06
mriedemi'm not sure exactly how you cleanup an instance from the db when it's no longer on the hypervisor and force-delete doesn't work16:06
openstackgerritSylvain Bauza proposed openstack/nova: Expose a REST API for a specific list of RPs  https://review.openstack.org/39256916:06
bauzasmriedem: sdague: necessary update since I abandoned the dependency ^16:07
mriedemi mean you can do it in the db, but we don't have cascading deletes on the instances table so you'll be orphaning other instances16:07
*** markvoelker_ has quit IRC16:07
mriedemkukacz: i'm pretty sure there are osops repo scripts for purging a given resource by id and it's related table records16:07
bauzasjay is not around so I can't ask him to review again16:07
kukaczmriedem: aha, great tip, thanks!16:07
*** zenoway has quit IRC16:08
kukaczmriedem: I forgot there's the operators channel16:08
mriedemandreykurilin: i think i'm ok with fixing projects that are using novaclient incorrectly after we release than add workarounds to straddle things before we release16:08
*** prateek has quit IRC16:08
andreykurilinmriedem: perfect16:09
*** bkopilov has quit IRC16:09
mriedemandreykurilin: it gets to be too much deprecation dance in the code which is hard to maintain16:09
*** ducnc has quit IRC16:10
*** ducnc has joined #openstack-nova16:10
openstackgerritSylvain Bauza proposed openstack/nova: placement: Amend doc to remove mention of port 8778  https://review.openstack.org/41315816:10
bauzasmriedem: sdague: doc amendment as we discussed ^16:10
bauzasnow, bailing out16:11
openstackgerritVladik Romanovsky proposed openstack/nova: objects: adding vlans field to virtual_interface  https://review.openstack.org/40594916:12
openstackgerritVladik Romanovsky proposed openstack/nova: db: add vlans column to virtual_interface table  https://review.openstack.org/40594816:12
openstackgerritVladik Romanovsky proposed openstack/nova: Adding vlans filed to Device tagging metadata  https://review.openstack.org/40595116:12
openstackgerritVladik Romanovsky proposed openstack/nova: objects: vlans field to NetworkInterfaceMetadata object  https://review.openstack.org/40595016:12
openstackgerritVladik Romanovsky proposed openstack/nova: libvirt: expose virtual interfaces with vlans to metadata  https://review.openstack.org/40595416:12
openstackgerritVladik Romanovsky proposed openstack/nova: neutron: update vlan in virtual_interface object  https://review.openstack.org/40595316:12
*** diga has joined #openstack-nova16:16
*** zenoway has joined #openstack-nova16:17
*** dtp has joined #openstack-nova16:19
*** bkopilov has joined #openstack-nova16:20
*** clayton has quit IRC16:21
*** mvk has quit IRC16:22
*** clayton has joined #openstack-nova16:22
*** roeyc1 has quit IRC16:24
*** zenoway has quit IRC16:25
*** fragatina has joined #openstack-nova16:26
*** prateek_ has quit IRC16:27
*** prateek has joined #openstack-nova16:27
*** baoli has quit IRC16:28
openstackgerritint32bit proposed openstack/nova: Add more actions to instance action logs  https://review.openstack.org/39651016:28
mriedemsdague: hmm http://logs.openstack.org/18/413118/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/4266e3f/logs/apache/placement-api.txt.gz16:31
*** zenoway has joined #openstack-nova16:31
mriedemno logs now16:31
*** thorst has quit IRC16:31
*** thorst has joined #openstack-nova16:31
mriedemgetting 500s in the cpu logs http://logs.openstack.org/18/413118/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/4266e3f/logs/screen-n-cpu.txt.gz#_2016-12-20_14_58_06_49116:32
*** andreas_s has quit IRC16:33
*** mdnadeem has joined #openstack-nova16:37
*** catintheroof has joined #openstack-nova16:39
*** thorst has quit IRC16:40
*** catinthe_ has quit IRC16:42
*** harlowja has quit IRC16:42
*** zenoway has quit IRC16:44
*** nic has joined #openstack-nova16:45
*** ducttape_ has joined #openstack-nova16:45
*** sridharg has quit IRC16:50
*** zenoway has joined #openstack-nova16:53
*** rtheis has quit IRC16:54
*** igordcard has joined #openstack-nova16:56
*** zz_dimtruck is now known as dimtruck16:56
*** igordcard has quit IRC16:56
sdaguemriedem: ok, I guess we've yet to make a Nova change that makes it fail is placement doesn't work?16:56
sdaguethat ServerName bit might not have been innocuous as bauzas thought16:57
*** ducttape_ has quit IRC16:57
sdaguehttp://logs.openstack.org/18/413118/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/4266e3f/logs/apache/error.txt.gz16:58
sdaguehttp://logs.openstack.org/18/413118/1/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/4266e3f/logs/apache/horizon_error.txt.gz16:58
*** igordcard has joined #openstack-nova16:58
*** igordcard has quit IRC16:58
*** igordcard has joined #openstack-nova16:59
*** igordcard has quit IRC16:59
*** fragatina has quit IRC16:59
*** fragatina has joined #openstack-nova17:00
*** igordcard has joined #openstack-nova17:00
*** igordcard has quit IRC17:01
*** bkopilov_ has joined #openstack-nova17:01
*** igordcard has joined #openstack-nova17:02
*** bkopilov has quit IRC17:02
*** edmondsw has joined #openstack-nova17:02
*** sreeram_v has joined #openstack-nova17:03
*** Guest14972 has quit IRC17:03
edmondswsdague, thoughts on my response to you in https://review.openstack.org/#/c/398610/ ?17:03
*** sreeram_v has quit IRC17:04
*** sreeram_v has joined #openstack-nova17:05
*** dimtruck is now known as zz_dimtruck17:06
*** zenoway has quit IRC17:06
*** sreeram_v has quit IRC17:09
bauzassdague: mriedem: just saw that and yeah my bad, I stupidely forgot that ServerName verifies the hostname17:11
*** Zer0Byte__ has joined #openstack-nova17:16
*** rtheis has joined #openstack-nova17:16
*** zenoway has joined #openstack-nova17:16
*** eharney has quit IRC17:17
*** eharney has joined #openstack-nova17:18
*** raunak has joined #openstack-nova17:18
*** lpetrut has quit IRC17:18
*** lucasagomes is now known as lucas-afk17:18
*** rtheis has quit IRC17:19
*** zenoway has quit IRC17:20
*** prateek has quit IRC17:21
*** yamahata has joined #openstack-nova17:23
*** eharney has quit IRC17:24
*** mdnadeem has quit IRC17:25
*** aloga has quit IRC17:28
*** aloga has joined #openstack-nova17:28
*** sahid has quit IRC17:29
openstackgerritSujitha proposed openstack/nova: Adding tags to InstancePayload  https://review.openstack.org/40722817:30
*** baoli has joined #openstack-nova17:33
*** prateek has joined #openstack-nova17:34
*** GMAzrael_ has quit IRC17:35
*** prateek_ has joined #openstack-nova17:36
*** gibi has joined #openstack-nova17:36
*** prateek_ has quit IRC17:37
*** thorst has joined #openstack-nova17:37
*** prateek_ has joined #openstack-nova17:37
*** eharney has joined #openstack-nova17:38
*** prateek has quit IRC17:39
*** gszasz has quit IRC17:39
*** slaweq_ has joined #openstack-nova17:44
*** ihrachys has quit IRC17:45
*** thorst has quit IRC17:45
*** nherciu has quit IRC17:51
*** ducttape_ has joined #openstack-nova17:51
*** harlowja has joined #openstack-nova17:53
*** lpetrut has joined #openstack-nova17:54
mriedemsdague: are you asking about nova-status?17:54
mriedemsdague: i'm +2 on your change as-is17:54
mriedemsdague: i think i'm going to build the resource provider / compute node comparison stuff in separately on top17:54
mriedemsdague: because it's a bit complicated and could probably be it's own check17:54
*** derekh has quit IRC17:55
*** pcaruana has quit IRC17:59
*** abalutoiu has joined #openstack-nova17:59
*** yamahata has quit IRC17:59
*** zenoway has joined #openstack-nova18:00
*** jamesdenton has quit IRC18:00
*** lpetrut has quit IRC18:00
*** unicell1 has quit IRC18:01
*** catinthe_ has joined #openstack-nova18:03
openstackgerritSarafraj Singh proposed openstack/nova: Add service_token for nova-neutron interaction  https://review.openstack.org/41039418:04
*** zenoway has quit IRC18:04
*** catintheroof has quit IRC18:06
*** parora has joined #openstack-nova18:06
*** zenoway has joined #openstack-nova18:07
*** thorst has joined #openstack-nova18:08
*** ducttape_ has quit IRC18:08
*** liverpooler has quit IRC18:08
*** nherciu has joined #openstack-nova18:09
*** prateek_ has quit IRC18:10
*** liverpooler has joined #openstack-nova18:10
*** salv-orlando has quit IRC18:12
*** lpetrut has joined #openstack-nova18:12
*** zz_dimtruck is now known as dimtruck18:13
*** zenoway has quit IRC18:16
*** ducttape_ has joined #openstack-nova18:19
*** Swami has joined #openstack-nova18:21
*** ssurana has joined #openstack-nova18:22
*** parora has quit IRC18:26
*** prateek has joined #openstack-nova18:26
*** unicell has joined #openstack-nova18:29
*** rcernin has quit IRC18:29
*** zenoway has joined #openstack-nova18:34
*** Drankis has quit IRC18:35
sdaguemriedem: no, I was thinking about the fact that when we totally screw up the placement api in devstack / nova, it doesn't break any tests18:35
sdaguemriedem: which... seems un-good18:36
*** ociuhandu has quit IRC18:39
*** amoralej is now known as amoralej|off18:40
*** yamahata has joined #openstack-nova18:41
*** zenoway has quit IRC18:43
*** gszasz has joined #openstack-nova18:46
*** ducttape_ has quit IRC18:47
*** mvk has joined #openstack-nova18:54
*** lbeliveau_ has joined #openstack-nova18:54
*** ihrachys has joined #openstack-nova19:04
*** fragatina has quit IRC19:05
*** Swami has quit IRC19:08
*** Swami has joined #openstack-nova19:09
*** diogogmt has joined #openstack-nova19:10
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] python3 experiment - ignore me  https://review.openstack.org/41320919:12
*** Sukhdev has joined #openstack-nova19:12
*** diga has quit IRC19:14
*** ihrachys has quit IRC19:16
*** ociuhandu has joined #openstack-nova19:22
*** whenry has joined #openstack-nova19:24
mriedemsdague: because we (1) aren't using nova-status yet and (2) aren't using the placement API in the scheduler yet19:24
mriedemsdague: but don't fret, soon in good time we'll be breaking all of the thing19:24
mriedem*things19:24
mriedemlet me start with the breaking by releasing novaclient 7.019:24
*** ducttape_ has joined #openstack-nova19:25
dimsLOL19:27
*** thorst has quit IRC19:29
*** gouthamr has quit IRC19:29
*** thorst has joined #openstack-nova19:29
*** stravis has joined #openstack-nova19:32
*** stravis has quit IRC19:33
mriedemwe've got 4 patches that have +2s which lead up to auto-creation of ironic resource classes in the resource tracker, starting here https://review.openstack.org/#/c/398470/1419:34
mriedemwould be nice to get those 4 bottom changes in, they are just refactors19:34
*** harlowja has quit IRC19:35
*** zenoway has joined #openstack-nova19:36
*** prateek has quit IRC19:37
*** catintheroof has joined #openstack-nova19:37
*** thorst has quit IRC19:38
*** nic has quit IRC19:38
*** nic has joined #openstack-nova19:39
*** catinthe_ has quit IRC19:40
*** zenoway has quit IRC19:40
*** jamesdenton has joined #openstack-nova19:41
*** weshay is now known as weshay_relocate19:44
*** zenoway has joined #openstack-nova19:46
*** gouthamr has joined #openstack-nova19:50
*** jamesdenton has quit IRC19:51
*** abalutoiu has quit IRC19:53
*** zenoway has quit IRC19:55
mriedemsdague: did you want to work http://docs.openstack.org/developer/nova/placement.html into the error messages for https://review.openstack.org/#/c/411883/ ?19:56
*** jamesdenton has joined #openstack-nova19:57
*** ducttape_ has quit IRC19:57
*** lbeliveau_ has quit IRC19:58
*** ducttape_ has joined #openstack-nova20:00
*** dharinic is now known as dharinic|lunch20:05
*** clenimar has quit IRC20:12
*** fragatina has joined #openstack-nova20:12
mriedemhrm, why can't i run a mysql query on the nova.services table and filter on the binary column?20:14
mriedemtopic is fine20:14
sdaguemriedem: what's your query?20:18
mriedemmysql> select host from nova.services where binary = 'nova-compute';20:18
mriedemERROR 1064 (42000): You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '= '  nova-compute'' at line 120:18
*** oanson has quit IRC20:18
mriedemmysql> select host from nova.services where topic = 'compute';20:19
mriedem+---------+20:19
mriedem| host    |20:19
mriedem+---------+20:19
mriedem| filters |20:19
mriedem+---------+20:19
mriedem1 row in set (0.00 sec)20:19
*** raildo has quit IRC20:20
mriedemoh binary is a reserved word20:20
*** ducttape_ has quit IRC20:21
*** fragatina has quit IRC20:22
*** fragatina has joined #openstack-nova20:23
mriedemmysql> select host from nova.services where `binary` = 'nova-compute';20:23
mriedem+---------+20:23
mriedem| host    |20:23
mriedem+---------+20:23
mriedem| filters |20:23
mriedem+---------+20:23
mriedem1 row in set (0.00 sec)20:23
mriedemi've learned something new today20:23
sdague:)20:23
mriedemthe reason i was looking that up was, when comparing number of resource providers to number of compute nodes for nova-status, i'm thinking we want to filter out disabled nova-compute service compute nodes20:24
mriedemb/c if the admin disabled the service, but there is a compute node showing up, we probably don't want to count it20:24
mriedemsince you can't schedule to it anyway20:24
*** weshay_relocate is now known as weshay20:32
*** thorst has joined #openstack-nova20:33
*** thorst has quit IRC20:33
*** stvnoyes has quit IRC20:35
*** stvnoyes has joined #openstack-nova20:35
*** lpetrut has quit IRC20:36
*** lpetrut has joined #openstack-nova20:37
*** slaweq_ has quit IRC20:37
*** slaweq_ has joined #openstack-nova20:39
*** BobBall is now known as BobBall_XMAS20:39
*** zhs__ has quit IRC20:39
*** zhs__ has joined #openstack-nova20:40
*** fragatina has quit IRC20:43
mgagnemriedem, melwitt, dansmith: RE api_database - so it appears that api_database/connection is only required by the parent/api cell, not compute. I managed to boot an instance just fine.20:43
*** jamielennox|away is now known as jamielennox20:44
mriedemmgagne: cool, that's what i figured would be the case20:44
*** slaweq_ has quit IRC20:44
mriedemi know the compute nodes don't up call to the api db20:45
mriedemthe question was if the api service in the child cell was going to hit the api db20:45
*** slaweq_ has joined #openstack-nova20:45
mgagnemriedem: I don't think there is an api service in the child cell? I mean, you can run one, I haven't tried yet.20:47
*** catintheroof has quit IRC20:47
*** whenry has quit IRC20:48
mriedemok i wasn't sure about that, it's just the cells service that's doing the traffic and syncing right?20:48
*** thorst has joined #openstack-nova20:50
*** Sukhdev has quit IRC20:50
mriedemsdague: you might enjoy the irony of this,20:51
sdaguedoes it involve and extra database?20:51
mriedemsdague: debugging a problem in kilo, glance v1, trying to list snapshots and get details for an image that we took for a tenant,20:51
*** smatzek has quit IRC20:52
mriedemthe snapshot was done, image was active, but getting 0 details back from glanceclient and osc,20:52
mriedembut lo and behold, you know which CLI did show the details from the glance v1 image headers?20:52
sdagueheh20:52
mriedemmotherfing novaclient20:52
sdague\m/20:53
mgagnemriedem: so I tried to bypass catalog and point to child cell running API service, API responds fine without [api_database]/connection20:53
mgagnerunning with cells v1 btw20:53
mriedemmgagne: responds to which requests? create server? list servers?20:54
mgagnenova list20:54
*** dharinic|lunch is now known as dharinic20:54
mgagneI don't plan on booting server with child cell API service20:54
*** thorst has quit IRC20:55
*** slaweq_ has quit IRC20:56
*** slaweq_ has joined #openstack-nova20:57
openstackgerritSarafraj Singh proposed openstack/nova: DNM: Test service token  https://review.openstack.org/41323220:58
*** fragatina has joined #openstack-nova21:08
*** gouthamr has quit IRC21:09
*** zhs_ has joined #openstack-nova21:10
*** zhs__ has quit IRC21:11
*** lpetrut has quit IRC21:11
bauzasmriedem: I stupidely forgot that when removing the WSGIScriptAlias directive from the VirtualHost, there was no way to get the WSGI app21:12
bauzasmriedem: I'm now running a new PS, but I'll probably try to modify again21:12
*** roeyc has joined #openstack-nova21:13
*** mtanino has quit IRC21:13
mriedemmgagne: ok, weird, and this is newton right?21:16
mgagnemriedem: what's weird? I'm running Mitaka.21:16
mriedemmgagne: because nova list in the api service in newton will check the api db for build_requests and the cell0 cell_mappings table entry21:16
mriedemmgagne: ah, ok, nvm then21:16
mriedemwe didn't list instances using things from the api db until newton21:17
mriedemi think the only thing in the api db in mitaka was the request spec21:17
mriedemwhich isn't involved in listing instances21:17
mgagneoh right, an other thing to look for during the next upgrade =)21:17
mriedemyeah i think that's what we were really wondering about21:17
mriedemi wonder what would happen if we enable cells v2 in a newton cells v1 CI job21:18
mriedemwell, either way we'd have the api db setup and it's just single-node in that job, so it probably wouldn't help much21:18
*** irenab_ has joined #openstack-nova21:21
*** irenab has quit IRC21:21
*** irenab_ is now known as irenab21:21
*** roeyc has quit IRC21:22
*** dtp has quit IRC21:22
*** roeyc has joined #openstack-nova21:22
*** mrda is now known as michaeld21:22
*** thorst has joined #openstack-nova21:23
*** michaeld is now known as mrda21:24
*** dave-mccowan has quit IRC21:26
*** ducttape_ has joined #openstack-nova21:30
*** fragatina has quit IRC21:30
*** thorst_ has joined #openstack-nova21:30
*** moshele has joined #openstack-nova21:31
*** thorst has quit IRC21:31
*** nic has quit IRC21:32
*** nic has joined #openstack-nova21:32
*** weshay is now known as weshay_afk21:33
*** Jeffrey4l has quit IRC21:35
*** roeyc has quit IRC21:35
*** ducttape_ has quit IRC21:35
*** thorst_ has quit IRC21:35
*** moshele has quit IRC21:35
*** nherciu has quit IRC21:36
*** ducttape_ has joined #openstack-nova21:38
*** liverpooler has quit IRC21:38
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] python3 experiment - ignore me  https://review.openstack.org/41320921:41
*** Jeffrey4l has joined #openstack-nova21:47
*** smatzek has joined #openstack-nova21:57
*** ducttape_ has quit IRC21:58
*** smatzek has quit IRC22:02
*** cleong has quit IRC22:04
*** edmondsw has quit IRC22:04
*** avolkov has quit IRC22:05
*** slaweq_ has quit IRC22:07
*** slaweq_ has joined #openstack-nova22:10
*** Sukhdev has joined #openstack-nova22:10
*** jheroux has quit IRC22:12
*** gouthamr has joined #openstack-nova22:13
*** thorst has joined #openstack-nova22:14
*** thorst has quit IRC22:16
*** armax has quit IRC22:16
*** thorst has joined #openstack-nova22:16
*** eharney has quit IRC22:21
*** mtanino has joined #openstack-nova22:21
*** fragatina has joined #openstack-nova22:23
*** tjones has joined #openstack-nova22:23
*** abalutoiu has joined #openstack-nova22:24
*** thorst has quit IRC22:25
*** jamesdenton has quit IRC22:26
*** mtanino has quit IRC22:29
melwittmgagne, mriedem: RE api database, beware you might need it in compute cells for other operations. one example is rebuild. so I'd do a sanity check of other instance actions22:31
mgagnewill test ;)22:31
melwittk22:31
*** esberglu has quit IRC22:32
*** esberglu has joined #openstack-nova22:32
mriedemyeah for the request spec22:34
mriedemgood point22:34
mriedemmove and rebuild operations specifically22:34
*** Sukhdev has quit IRC22:36
*** esberglu has quit IRC22:37
*** Sukhdev has joined #openstack-nova22:38
*** slaweq_ has quit IRC22:41
openstackgerritMatt Riedemann proposed openstack/nova: nova-status: check for cells v2 upgrade readiness  https://review.openstack.org/41152522:42
openstackgerritMatt Riedemann proposed openstack/nova: nova-status: add basic placement status checking  https://review.openstack.org/41188322:42
openstackgerritMatt Riedemann proposed openstack/nova: Add nova-status upgrade check command framework  https://review.openstack.org/41151722:42
openstackgerritMatt Riedemann proposed openstack/nova: WIP: nova-status: check for resource providers  https://review.openstack.org/41325022:42
*** roeyc has joined #openstack-nova22:42
mriedemsdague: ^ added my new wip on top of that stack22:42
mriedemgot some questions/todos inline, probably need to get jay and dan involved in those22:42
*** slaweq_ has joined #openstack-nova22:43
mriedemmelwitt: btw target_cell and the db fixtures are my best friends in writing that last one22:43
mriedem:)22:43
*** jwcroppe has quit IRC22:44
*** jwcroppe has joined #openstack-nova22:44
*** smatzek has joined #openstack-nova22:46
*** fragatina has quit IRC22:47
*** gszasz has quit IRC22:48
*** jwcroppe has quit IRC22:48
*** diogogmt has quit IRC22:50
*** esberglu has joined #openstack-nova22:50
*** ducttape_ has joined #openstack-nova22:53
*** esberglu has quit IRC22:55
*** Swami has quit IRC22:59
*** jwcroppe has joined #openstack-nova22:59
*** Guest28907 has quit IRC23:02
*** jwcroppe has quit IRC23:03
melwittmriedem: yay \o/23:04
*** salv-orlando has joined #openstack-nova23:06
*** sdague has quit IRC23:08
*** ducttape_ has quit IRC23:12
*** ducttape_ has joined #openstack-nova23:12
aunnam:q23:16
*** smatzek has quit IRC23:16
openstackgerritAnusha Unnam proposed openstack/nova: Add key_name field to InstancePayload  https://review.openstack.org/41326723:17
*** claudiub has quit IRC23:18
*** thorst has joined #openstack-nova23:22
*** ociuhandu has quit IRC23:24
*** Sukhdev_ has joined #openstack-nova23:25
*** Sukhdev has quit IRC23:27
*** ociuhandu has joined #openstack-nova23:28
*** pradk has quit IRC23:29
*** thorst has quit IRC23:30
*** smatzek has joined #openstack-nova23:33
*** mtanino has joined #openstack-nova23:34
*** salv-orlando has quit IRC23:40
*** thorst has joined #openstack-nova23:41
*** thorst has quit IRC23:41
*** thorst has joined #openstack-nova23:41
*** diogogmt has joined #openstack-nova23:42
*** mlavalle has quit IRC23:44
*** smatzek has quit IRC23:44
*** baoli has quit IRC23:45
*** lamt has quit IRC23:45
*** thorst has quit IRC23:46
*** lamt has joined #openstack-nova23:46
*** roeyc has quit IRC23:46
*** roeyc has joined #openstack-nova23:47
*** lamt has quit IRC23:51
*** thorst has joined #openstack-nova23:52
*** thorst has quit IRC23:52
*** thorst has joined #openstack-nova23:52
*** dimtruck is now known as zz_dimtruck23:52
*** tjones has quit IRC23:55
*** diogogmt has quit IRC23:56

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!