Tuesday, 2017-08-29

*** edmonds__ has joined #openstack-nova00:00
*** edmondsw_ has quit IRC00:01
*** felipemonteiro__ has quit IRC00:01
*** edmondsw_ has joined #openstack-nova00:01
*** edmondsw_ has quit IRC00:01
*** edmondsw_ has joined #openstack-nova00:02
*** baoli has quit IRC00:02
*** edmonds__ has quit IRC00:04
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional recreate test for live migration pre-check fails  https://review.openstack.org/49862700:09
*** baoli has joined #openstack-nova00:10
*** mriedem has quit IRC00:11
*** rajathagasthya has quit IRC00:12
*** annegentle has quit IRC00:12
*** yamamoto_ has joined #openstack-nova00:13
*** yamamoto_ has quit IRC00:13
*** ragasthya has quit IRC00:14
*** Swami has quit IRC00:19
*** ijw has joined #openstack-nova00:19
*** sterdnotshaken1 has quit IRC00:20
*** ijw has quit IRC00:21
*** hshiina has joined #openstack-nova00:26
*** edmondsw_ has quit IRC00:26
*** edmondsw has joined #openstack-nova00:29
*** edmondsw has quit IRC00:31
*** edmondsw has joined #openstack-nova00:31
*** edmondsw has quit IRC00:36
*** kiennt has joined #openstack-nova00:38
*** Hazelesque has quit IRC00:44
*** slaweq_ has joined #openstack-nova00:45
*** Hazelesque has joined #openstack-nova00:46
*** Shunli has joined #openstack-nova00:49
*** slaweq_ has quit IRC00:50
*** thorst_afk has joined #openstack-nova00:51
*** jdurgin has joined #openstack-nova00:51
*** rajathagasthya has joined #openstack-nova00:52
*** zhurong has joined #openstack-nova00:52
*** ragasthya has joined #openstack-nova00:55
*** ljxiash has joined #openstack-nova00:55
* alex_xu stuck in an internal training00:55
*** rajathagasthya has quit IRC00:56
*** thorst_afk has quit IRC00:59
*** ljxiash has quit IRC00:59
*** gszasz has quit IRC01:00
*** baoli has quit IRC01:01
*** masber has quit IRC01:01
*** baoli has joined #openstack-nova01:02
*** masber has joined #openstack-nova01:02
*** phuongnh has joined #openstack-nova01:02
*** Nel1x has joined #openstack-nova01:03
*** gouthamr_ has joined #openstack-nova01:05
*** baoli has quit IRC01:06
*** gouthamr has quit IRC01:07
*** licanwei has joined #openstack-nova01:08
*** yamamoto_ has joined #openstack-nova01:14
*** baoli has joined #openstack-nova01:18
*** yamamoto_ has quit IRC01:20
*** litao__ has joined #openstack-nova01:27
*** gouthamr_ is now known as gouthamr01:27
*** edmondsw has joined #openstack-nova01:40
*** edmondsw has quit IRC01:44
*** slaweq_ has joined #openstack-nova01:46
*** zxy has joined #openstack-nova01:47
*** ijw has joined #openstack-nova01:48
*** slaweq_ has quit IRC01:50
*** ijw has quit IRC01:53
*** Masahisa has joined #openstack-nova01:53
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595401:59
*** thorst_afk has joined #openstack-nova02:00
*** thorst_afk has quit IRC02:06
*** yamahata has quit IRC02:08
*** ljxiash has joined #openstack-nova02:12
*** baoli has quit IRC02:16
*** yamamoto_ has joined #openstack-nova02:16
*** diga has quit IRC02:18
*** baoli has joined #openstack-nova02:19
*** baoli has quit IRC02:19
*** baoli has joined #openstack-nova02:19
*** ijw has joined #openstack-nova02:20
*** yamamoto_ has quit IRC02:22
*** tetsuro has joined #openstack-nova02:27
*** coreywright has quit IRC02:27
*** ijw has quit IRC02:28
*** gbarros has joined #openstack-nova02:30
*** ragasthya has quit IRC02:31
*** dave-mccowan has quit IRC02:32
*** yangyapeng has joined #openstack-nova02:33
*** yangyapeng has quit IRC02:33
*** yangyapeng has joined #openstack-nova02:34
*** gbarros has quit IRC02:34
*** crushil has quit IRC02:35
*** xinliang has quit IRC02:36
*** jbernard has quit IRC02:40
*** hongbin has joined #openstack-nova02:43
*** sridharg has joined #openstack-nova02:45
*** sree has joined #openstack-nova02:45
*** coreywright has joined #openstack-nova02:45
openstackgerritShunli Zhou proposed openstack/nova master: Transform servergroup.create notification  https://review.openstack.org/45402302:46
*** slaweq_ has joined #openstack-nova02:47
*** xinliang has joined #openstack-nova02:49
*** sree has quit IRC02:49
*** yamamoto has joined #openstack-nova02:49
*** gcb has joined #openstack-nova02:50
*** felipemonteiro__ has joined #openstack-nova02:51
*** slaweq_ has quit IRC02:52
*** diga has joined #openstack-nova02:53
*** masber has quit IRC02:53
*** felipemonteiro_ has joined #openstack-nova02:55
*** hongbin_ has joined #openstack-nova02:56
*** hongbin has quit IRC02:56
*** hongbin_ has quit IRC02:56
*** hongbin has joined #openstack-nova02:56
*** felipemonteiro has joined #openstack-nova02:57
*** felipemonteiro__ has quit IRC02:59
*** gouthamr has quit IRC02:59
*** felipemonteiro_ has quit IRC03:00
*** thorst_afk has joined #openstack-nova03:01
*** thorst_afk has quit IRC03:06
*** yamahata has joined #openstack-nova03:06
*** felipemonteiro has quit IRC03:11
*** yamamoto has quit IRC03:12
*** itlinux has joined #openstack-nova03:13
*** zerda3 has quit IRC03:18
*** jbernard has joined #openstack-nova03:27
*** ijw has joined #openstack-nova03:29
*** baoli has quit IRC03:30
*** annegentle has joined #openstack-nova03:34
*** ijw has quit IRC03:40
*** ijw has joined #openstack-nova03:40
*** ijw_ has joined #openstack-nova03:41
*** ijw has quit IRC03:44
*** slaweq_ has joined #openstack-nova03:48
*** annegentle has quit IRC03:48
*** annegentle has joined #openstack-nova03:48
*** hieulq has quit IRC03:49
*** hieulq has joined #openstack-nova03:49
*** udesale has joined #openstack-nova03:50
*** sree has joined #openstack-nova03:50
*** tiendc has joined #openstack-nova03:52
*** slaweq_ has quit IRC03:52
*** annegentle has quit IRC03:53
*** rajathagasthya has joined #openstack-nova03:55
*** TuanLA has joined #openstack-nova03:55
*** ljxiash has quit IRC04:02
*** thorst_afk has joined #openstack-nova04:02
*** ljxiash has joined #openstack-nova04:02
*** links has joined #openstack-nova04:03
*** ljxiash has quit IRC04:07
*** thorst_afk has quit IRC04:07
*** masber has joined #openstack-nova04:08
*** gyee has quit IRC04:09
*** hoonetorg has quit IRC04:11
*** adisky__ has joined #openstack-nova04:12
*** yamamoto has joined #openstack-nova04:13
*** hongbin has quit IRC04:13
*** Dinesh_Bhor has joined #openstack-nova04:16
*** Dinesh_Bhor has left #openstack-nova04:16
*** yamamoto has quit IRC04:19
*** itlinux has quit IRC04:19
*** claudiub|2 has joined #openstack-nova04:21
*** zhurong has quit IRC04:21
*** chyka has joined #openstack-nova04:23
*** hoonetorg has joined #openstack-nova04:23
*** lpetrut has joined #openstack-nova04:26
*** chyka has quit IRC04:27
*** felipemonteiro has joined #openstack-nova04:37
*** rajathagasthya has quit IRC04:47
*** felipemonteiro has quit IRC04:47
*** slaweq_ has joined #openstack-nova04:49
*** slaweq_ has quit IRC04:54
*** shan has joined #openstack-nova04:55
*** zhurong has joined #openstack-nova04:56
*** lpetrut has quit IRC04:56
*** lajoskatona has joined #openstack-nova05:00
*** thorst_afk has joined #openstack-nova05:03
*** kornicameister has quit IRC05:03
*** jaypipes has quit IRC05:04
*** thorst_afk has quit IRC05:08
*** cfriesen has quit IRC05:10
*** ljxiash has joined #openstack-nova05:12
*** yamamoto has joined #openstack-nova05:15
*** sridharg has quit IRC05:16
*** sridharg has joined #openstack-nova05:17
*** yamamoto has quit IRC05:21
*** Shunli has quit IRC05:22
*** Shunli has joined #openstack-nova05:22
*** ratailor has joined #openstack-nova05:24
*** TuanLA_ has joined #openstack-nova05:25
*** TuanLA has quit IRC05:28
*** ijw_ has quit IRC05:31
openstackgerritMerged openstack/nova master: Fix a typo in code comment  https://review.openstack.org/49566505:37
openstackgerritMerged openstack/nova master: Remove "dhcp_options_for_instance"  https://review.openstack.org/49270705:39
openstackgerritMerged openstack/nova master: Prevent blank line at start of migration placeholders  https://review.openstack.org/49800405:40
*** phuongnh has quit IRC05:41
*** tiendc has quit IRC05:41
*** phuongnh has joined #openstack-nova05:42
*** tiendc has joined #openstack-nova05:42
*** trinaths has joined #openstack-nova05:43
*** mdnadeem has joined #openstack-nova05:44
*** slaweq_ has joined #openstack-nova05:50
openstackgerritMerged openstack/nova master: Add placeholder migrations for Pike backports  https://review.openstack.org/49693205:50
*** slaweq_ has quit IRC05:54
*** carthaca_1 has quit IRC05:58
*** carthaca_ has joined #openstack-nova05:58
*** thorst_afk has joined #openstack-nova06:03
*** yangyape_ has joined #openstack-nova06:04
*** yangyapeng has quit IRC06:05
*** yangyape_ has quit IRC06:05
*** yangyap__ has joined #openstack-nova06:05
*** yushiro has joined #openstack-nova06:07
openstackgerritMerged openstack/nova master: Change default for [notifications]/default_publisher_id to $host  https://review.openstack.org/49809406:07
*** thorst_afk has quit IRC06:09
openstackgerritMerged openstack/nova master: Improve assertJsonEqual error reporting  https://review.openstack.org/42819906:10
*** yamamoto has joined #openstack-nova06:13
*** Oku_OS-away is now known as Oku_OS06:15
yushiroHi, I'd like to give some help for registering host aggregate in stable/ocata with devstack environment.  I deployed 2 nodes.  1st is controller which compute driver is 'ironic'.  The other is compute node which compute driver is 'libvirt.LibvirtDriver'06:16
yushiroI'd like to separate 'virtual' and 'baremetal' by using host aggregate.  However, if I specify controller's hostname as aggregate host, it returns 404(Not Found)06:17
*** ljxiash has quit IRC06:18
yushiroHere is an operation log:  http://paste.openstack.org/show/619721/06:18
*** ljxiash has joined #openstack-nova06:19
yushiroWhat can I specify as <host> for 'openstack aggregate add host <aggregate> <host>'?06:20
*** ljxiash has quit IRC06:27
*** yamamoto has quit IRC06:38
*** yamamoto has joined #openstack-nova06:39
*** yamamoto has quit IRC06:39
*** pcaruana has joined #openstack-nova06:39
*** ljxiash has joined #openstack-nova06:42
*** markus_z has joined #openstack-nova06:45
*** jaypipes has joined #openstack-nova06:45
*** ljxiash has quit IRC06:46
*** ljxiash has joined #openstack-nova06:49
*** slaweq_ has joined #openstack-nova06:51
*** andreas_s has joined #openstack-nova06:54
openstackgerritNaichuan Sun proposed openstack/nova master: (WIP)xenapi: Live migration failed in xapi pool  https://review.openstack.org/48945106:55
*** slaweq_ has quit IRC06:56
*** thorst_afk has joined #openstack-nova07:05
ratailordoes anybody know, how to debug functional tests ?07:06
*** Nel1x has quit IRC07:08
*** rcernin has joined #openstack-nova07:10
*** thorst_afk has quit IRC07:10
*** bhagyashri_s has joined #openstack-nova07:11
*** ps_jadhav has quit IRC07:13
*** pooja_jadhav has quit IRC07:13
*** bhagyashris has quit IRC07:13
*** pooja_jadhav has joined #openstack-nova07:13
*** ps_jadhav has joined #openstack-nova07:14
*** zhurong has quit IRC07:15
*** mnestratov has joined #openstack-nova07:16
*** zhurong has joined #openstack-nova07:17
openstackgerritShunli Zhou proposed openstack/nova master: Transform servergroup.create notification  https://review.openstack.org/45402307:18
*** bkopilov has quit IRC07:21
jaypipesratailor: hi Rajesh, what's up?07:22
jaypipesratailor: which functional test are you having trouble with?07:23
ratailorjaypipes, I want to debug a db migration functional test.07:23
ratailorjaypipes, FYI, I want to add a functionl test for this.. https://review.openstack.org/#/c/498334/07:24
jaypipesratailor: you mean debug the specific SQL that gets produced from, say, the nova.object methods?07:24
ratailorjaypipes, for this migration. https://review.openstack.org/#/c/498334/2/nova/db/sqlalchemy/api_migrations/migrate_repo/versions/045_mysql_host_mappings_binary_collation.py07:25
*** tesseract has joined #openstack-nova07:25
jaypipesratailor: ah, yeah.07:26
*** bkopilov has joined #openstack-nova07:26
jaypipesratailor: it's complaining that you need to add a functional test for that migration to this file: https://github.com/openstack/nova/blob/master/nova/tests/functional/db/api/test_migrations.py07:27
jaypipesratailor: just take one of the other example tests, like for migration 003: https://github.com/openstack/nova/blob/master/nova/tests/functional/db/api/test_migrations.py#L23707:28
*** jpena|off is now known as jpena07:28
ratailorjaypipes, I thought about that, but is there any way to debug it, so that I can check what all attributes are available with each object, to efficient write a test for that.07:29
jaypipesratailor: lemme take a look at your migration. one sec07:30
ratailorpython -m testtools.run discover nova.tests.functional.db.api.test_migrations07:32
ratailor jaypipes, I think this help me ^^07:32
*** ragiman has joined #openstack-nova07:33
*** yamahata has quit IRC07:37
ratailorjaypipes, Thanks for your help.07:39
*** ratailor is now known as ratailor|Lunch07:39
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Live migration failed in xapi pool  https://review.openstack.org/48945107:40
*** yamamoto has joined #openstack-nova07:40
*** priteau has joined #openstack-nova07:40
*** edmondsw has joined #openstack-nova07:40
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Live migration failed in xapi pool  https://review.openstack.org/48945107:43
*** edmondsw has quit IRC07:45
*** yamamoto has quit IRC07:45
jaypipesratailor|Lunch: k, reviewed. left instructions on the patch.07:46
*** edmondsw has joined #openstack-nova07:49
*** ralonsoh has joined #openstack-nova07:50
*** slaweq_ has joined #openstack-nova07:52
yushiroHi, I'd like to give some help for registering host aggregate in stable/ocata with devstack environment.  I deployed 2 nodes.  1st is controller which compute driver is 'ironic'.  The other is compute node which compute driver is 'libvirt.LibvirtDriver'07:56
yushiro I'd like to separate 'virtual' and 'baremetal' by using host aggregate.  However, if I specify controller's hostname as aggregate host, it returns 404(Not Found)07:56
yushiroHere is an operation log:  http://paste.openstack.org/show/619721/07:56
yushiroWhat can I specify as <host> for 'openstack aggregate add host <aggregate> <host>'?07:56
*** slaweq_ has quit IRC07:57
*** chyka has joined #openstack-nova07:59
*** chyka has quit IRC08:03
*** thorst_afk has joined #openstack-nova08:05
jaypipesyushiro: generally best to ask usage questions like this on the openstack@ mailing list or ask.openstack.org (see channel /topic). but, short answer is that you want to use the *compute host* as the host in your aggregate, not the controller.08:05
*** alex_xu has quit IRC08:06
jaypipesyushiro: your ironic baremetal nodes will be managed by one or more nova-compute services. likewise, your VMs will be managed by nova-compute services running on hypervisor hosts. It is those nova-compute host names that you will use in doing the host-aggregate assignments.08:06
*** yangyap__ has quit IRC08:07
yushirojaypipes, Thanks.  Ah, sorry.  I deployed 2 nodes( 1:controller with compute(including ir-cond, ir-api),  2:compute)08:08
openstackgerritElod Illes proposed openstack/nova master: Functional test: evacuate with no compute  https://review.openstack.org/49848208:09
yushirojaypipes, Yes, I think so too.  However, controller-hostname (furukawa-ctrl) cannot found ;( ...08:09
jaypipesyushiro: you will still need a nova-compute service on that controller node, though. if you do a nova-manage service list, what shows up?08:10
*** thorst_afk has quit IRC08:10
openstackgerritGábor Antal proposed openstack/nova master: Transform instance.resize_prep notification  https://review.openstack.org/46508108:11
yushirojaypipes, hmm, my goal is to deploy instance both ironic and nova(libvirt) in 1 controller.  Let me check a result...08:12
*** alex_xu has joined #openstack-nova08:12
*** alexchadin has joined #openstack-nova08:12
jaypipesyushiro: that won't work :) you'll need to set up a container running on the controller that has a separate hostname for the nova-compute service managing the Ironic nodes.08:13
*** slunkad has quit IRC08:13
*** slunkad has joined #openstack-nova08:14
*** yangyapeng has joined #openstack-nova08:15
*** yangyapeng has quit IRC08:16
*** shaohe_feng has quit IRC08:16
yushirojaypipes, ouch!!  oh, OK.  So, you mean it should be separated between nova-compute and ironic-conductor ?08:16
*** yangyapeng has joined #openstack-nova08:17
*** yangyapeng has quit IRC08:18
*** shaohe_feng has joined #openstack-nova08:19
jaypipesyushiro: the same nova-compute service cannot run *both* the libvirt *and* the ironic virt driver. therefore you will need to have the nova-compute service worker that manages the Ironic baremetal nodes on a separate installation than the nova-compute service running the libvirt virt driver and managing virtual resources.08:19
jaypipesyushiro: you can easily do so by just making a container that runs the nova-compute service with the ironic virt driver.08:19
jaypipesyushiro: that container can run on the same physical host that the other nova-compute service (that manages the virtual resources) is running on. But, it needs a separate hostname and separate process (because the nova.conf will be different)08:20
yushirojaypipes, Thank you.  Well, here is a result 'nova-manage host list'  http://paste.openstack.org/show/619730/   Please refer at the end of line.08:21
*** TuanLA_ has quit IRC08:21
yushirojaypipes, Let me explain again.. furukawa-ctrl includes n-cpu,n-csh,n-cond,n-api,n-cell-region,n-cell-child,placement-api,ir-api,ir-cond...,  and compute driver is 'ironic.IronicDriver',  furukawa-com includes n-cond,n-api-meta,n-cpu and compute driver is 'libvirt.LibvirtDriver'08:24
*** TuanLA has joined #openstack-nova08:24
jaypipesyushiro: you are using cellsv1 functionality?08:25
*** ljxiash has quit IRC08:26
jaypipesyushiro: you should not use cellsv1 (nova-cells service). what version of OpenStack are you running?08:26
jaypipesyushiro: and how are you installing? are you installing a distro like RDO?08:27
openstackgerritGábor Antal proposed openstack/nova master: Transform libvirt.error notification  https://review.openstack.org/48485108:27
yushirojaypipes, oops, sorry. I didn't tell you a version.  The version is stable/ocata with devstack.08:27
yushirojaypipes, I executed 'nova-manage cell_v2 list_cells' and returned   named 'cell0' which UUID is 00000000-0000-0000-0000-000000000000.08:28
jaypipesyushiro: yes, it's confusing. nova-cells service is cellsv1...08:28
jaypipesyushiro: in your localrc, did you enable NOVA_CELLS or something?08:29
*** edmondsw has quit IRC08:29
* jaypipes was almost certain devstack had removed cellsv1 stuff by now... :(08:29
*** ratailor|Lunch is now known as ratailor08:29
*** yangyapeng has joined #openstack-nova08:30
yushirojaypipes, Yes.  I added enable_service n-cell in my local.conf  because it was got an error if I didn't add this configuration.08:30
jaypipesyushiro: ok, I believe that is the primary source of the problems...08:31
jaypipesyushiro: I'm afraid you will likely need to start over with your devstack installation :(08:31
jaypipesyushiro: you probably hit an error related to "cellsv2 not setup" or similar, yes?08:31
*** ljxiash has joined #openstack-nova08:32
jaypipesyushiro: it is quite unfortunate that cellsv1 and cellsv2 are so confusing. :(08:32
yushirojaypipes, wow, really?  I didn't test 'cellsv2 not setup' but .... OK08:32
jaypipesyushiro: the nova-cells service is cellsv1 and deprecated. cellsv2 is entirely transparent and does not have separate service workers.08:33
jaypipesyushiro: what was the error you got?08:33
jaypipesyushiro: you said above "got an error if I didn't add this configuration.". what specific error did you get? do you remember?08:34
*** ljxiash is now known as Tom___08:34
*** Tom___ has quit IRC08:35
*** Tom___ has joined #openstack-nova08:35
yushirojaypipes, Sorry. I deployed this env for 2 month ago, log message has been removed ;(08:35
jaypipesyushiro: understood :) I think the error *probably* said something like "cellsv2 setup not complete" or something like that.08:37
jaypipesyushiro: I am 90% sure, however, that the source of your issue here is that cellsv1 is enabled in that environment and unfortunately, the only way to fix this is to start over and not have n-cells enabled.08:38
jaypipesyushiro: apologies for the bad news :(08:39
yushirojaypipes, no-no.  I appreciate with your kindly explanation :)  Thank you so much08:39
*** efoley has joined #openstack-nova08:40
*** yamamoto has joined #openstack-nova08:42
openstackgerritLei Zhang proposed openstack/nova master: placement: extract traits from flavor extra spec  https://review.openstack.org/49202608:42
*** shan has quit IRC08:43
*** yamamoto has quit IRC08:47
*** cdent has joined #openstack-nova08:50
*** slaweq_ has joined #openstack-nova08:53
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595408:55
*** slaweq_ has quit IRC08:58
*** shan has joined #openstack-nova08:58
*** lucas-afk is now known as lucasagomes08:59
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: cached images should be cleaned up by time  https://review.openstack.org/46595409:00
*** ratailor_ has joined #openstack-nova09:01
*** ratailor_ has quit IRC09:01
*** ratailor_ has joined #openstack-nova09:02
*** ratailor has quit IRC09:04
*** rmart04 has joined #openstack-nova09:06
*** thorst_afk has joined #openstack-nova09:06
*** thorst_afk has quit IRC09:11
openstackgerritNaichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment  https://review.openstack.org/48945109:11
*** hshiina has quit IRC09:15
*** ratailor has joined #openstack-nova09:22
*** ratailor_ has quit IRC09:26
*** Shunli has quit IRC09:26
*** gixxer1k has joined #openstack-nova09:30
*** yangyape_ has joined #openstack-nova09:31
*** hferenc has joined #openstack-nova09:31
*** amarao has joined #openstack-nova09:32
openstackgerritViktor Varga proposed openstack/nova master: Remove mox from unit/conductor/test_conductor.py (1)  https://review.openstack.org/49669109:34
openstackgerritViktor Varga proposed openstack/nova master: Ensure image metadata when unshelving BFV instance  https://review.openstack.org/49872909:34
*** yangyapeng has quit IRC09:34
*** edmondsw has joined #openstack-nova09:35
*** hferenc has quit IRC09:35
*** sambetts|afk is now known as sambetts09:36
*** hferenc has joined #openstack-nova09:37
*** yamamoto has joined #openstack-nova09:43
*** edmondsw has quit IRC09:47
*** yamamoto has quit IRC09:49
bhagyashri_sratailor: Hi, There is one alternative option to debug unit test case ref: http://superuser.openstack.org/articles/a-guide-to-testing-in-openstack/ not sure that is working or not.09:53
*** slaweq_ has joined #openstack-nova09:54
ratailorbhagyashri_s, Thanks! btw I was looking for functional tests. :009:54
ratailor:)09:54
*** sdague has joined #openstack-nova09:54
*** shan7 has joined #openstack-nova09:54
ratailorbhagyashri_s, And I got that by this way.. python -m testtools.run discover nova.tests.functional.db.api.test_migrations09:54
*** moshele has joined #openstack-nova09:55
bhagyashri_sratailor: ok. :)09:55
openstackgerritAlex Xu proposed openstack/nova master: placement: ensure RP maps to those RPs that share with it  https://review.openstack.org/48037909:55
openstackgerritAlex Xu proposed openstack/nova master: placement: functional tests for multiple shared resource providers  https://review.openstack.org/49873709:55
*** sapd has quit IRC09:56
*** sapd has joined #openstack-nova09:57
*** alexchadin has quit IRC09:58
*** slaweq_ has quit IRC09:58
*** alexchadin has joined #openstack-nova09:59
openstackgerritAlex Xu proposed openstack/nova master: placement: ensure RP maps to those RPs that share with it  https://review.openstack.org/48037910:01
*** udesale has quit IRC10:02
*** ratailor has quit IRC10:07
*** ratailor has joined #openstack-nova10:08
*** alexchadin has quit IRC10:12
*** alexchadin has joined #openstack-nova10:13
*** alexchadin has quit IRC10:13
*** hferenc has quit IRC10:13
*** alexchadin has joined #openstack-nova10:14
*** sapd has quit IRC10:17
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: deprecate image handler class path option  https://review.openstack.org/49720110:17
*** alexchadin has quit IRC10:17
*** dtantsur|afk is now known as dtantsur10:23
*** ociuhandu has quit IRC10:30
*** Tom___ has quit IRC10:32
*** sapd has joined #openstack-nova10:33
*** Tom___ has joined #openstack-nova10:34
*** lpetrut has joined #openstack-nova10:34
*** lpetrut_ has joined #openstack-nova10:34
*** zhurong has quit IRC10:35
*** Tom___ has quit IRC10:38
*** felipemonteiro has joined #openstack-nova10:44
*** yamamoto has joined #openstack-nova10:45
*** zxy has quit IRC10:47
*** yangyape_ has quit IRC10:48
*** zxy has joined #openstack-nova10:48
*** felipemonteiro has quit IRC10:48
*** yangyapeng has joined #openstack-nova10:49
*** yamamoto has quit IRC10:51
*** ociuhandu has joined #openstack-nova10:51
*** ps_jadhav has quit IRC10:54
*** TuanLA has quit IRC10:55
*** slaweq_ has joined #openstack-nova10:55
*** slaweq_ has quit IRC11:00
*** trinaths has left #openstack-nova11:02
*** moshele has quit IRC11:06
VAhlWARNING: cell0 mapping not found - not syncing cell0.  How do I recreate the cell0? OS Pike11:06
*** thorst_afk has joined #openstack-nova11:07
*** moshele has joined #openstack-nova11:08
*** thorst_afk has quit IRC11:12
*** alexchadin has joined #openstack-nova11:13
*** alexchadin has quit IRC11:16
*** tetsuro has quit IRC11:21
*** diga has quit IRC11:24
*** alexchadin has joined #openstack-nova11:24
*** diga has joined #openstack-nova11:24
*** yushiro has quit IRC11:27
*** sree has quit IRC11:28
*** kiennt has quit IRC11:28
*** tiendc has quit IRC11:31
*** shan has quit IRC11:32
*** shan7 has quit IRC11:32
openstackgerritMerged openstack/nova master: tests: De-duplicate some graphics tests  https://review.openstack.org/49796911:34
*** zhurong has joined #openstack-nova11:34
*** chyka has joined #openstack-nova11:35
openstackgerritMerged openstack/nova master: Updates to scheduling workflow doc  https://review.openstack.org/49861311:36
*** udesale has joined #openstack-nova11:36
*** ratailor has quit IRC11:38
*** chyka has quit IRC11:39
*** jaypipes has quit IRC11:42
*** yamamoto has joined #openstack-nova11:47
*** edmondsw has joined #openstack-nova11:48
*** edmondsw has quit IRC11:52
*** yamamoto has quit IRC11:52
*** gixxer1k has quit IRC11:55
*** gixxer1k has joined #openstack-nova11:55
openstackgerritJianghua Wang proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming  https://review.openstack.org/48647511:57
*** jaypipes has joined #openstack-nova11:58
*** lucasagomes is now known as lucas-hungry11:58
*** litao__ has quit IRC11:59
*** Dave has quit IRC12:02
*** dave-mccowan has joined #openstack-nova12:05
*** thorst_afk has joined #openstack-nova12:06
*** vladikr has joined #openstack-nova12:13
*** jpena is now known as jpena|lunch12:14
*** tbachman has quit IRC12:18
*** pchavva has joined #openstack-nova12:19
*** MVenesio has joined #openstack-nova12:20
*** MVenesio has quit IRC12:20
*** tbachman has joined #openstack-nova12:23
*** dtantsur is now known as dtantsur|busy12:23
*** bkopilov has quit IRC12:25
*** bkopilov has joined #openstack-nova12:25
*** gcb has quit IRC12:29
*** gcb has joined #openstack-nova12:30
*** markus_z has quit IRC12:36
*** zhurong has quit IRC12:36
*** alexchadin has quit IRC12:36
*** obre has quit IRC12:39
*** obre has joined #openstack-nova12:40
*** obre has quit IRC12:42
*** obre has joined #openstack-nova12:42
*** lucasxu has joined #openstack-nova12:42
*** obre has quit IRC12:45
*** yangyapeng has quit IRC12:46
*** obre has joined #openstack-nova12:46
*** felipemonteiro has joined #openstack-nova12:46
*** slaweq_ has joined #openstack-nova12:48
*** slaweq_ has quit IRC12:48
*** yamamoto has joined #openstack-nova12:49
*** felipemonteiro_ has joined #openstack-nova12:49
*** sree has joined #openstack-nova12:50
*** felipemonteiro__ has joined #openstack-nova12:50
*** zhurong has joined #openstack-nova12:50
*** eharney has quit IRC12:50
*** alexchadin has joined #openstack-nova12:50
*** felipemonteiro has quit IRC12:52
*** yamamoto has quit IRC12:54
*** felipemonteiro_ has quit IRC12:54
*** coreywright has quit IRC12:58
*** gabor_antal_ has joined #openstack-nova12:59
*** lucas-hungry is now known as lucasagomes13:01
*** slaweq has quit IRC13:02
*** esberglu has joined #openstack-nova13:02
*** esberglu has quit IRC13:02
*** esberglu has joined #openstack-nova13:03
*** slaweq has joined #openstack-nova13:03
openstackgerritBalazs Gibizer proposed openstack/nova master: cover migration cases with functional tests  https://review.openstack.org/49386513:04
bhagyashri_smriedem, jaypipes, dansmith: Hi, Just wanted to ask the future plans regarding resources usages related (cpu/disk/memory used) information in the compute_nodes db table, Is there any plan to remove this resource usages related information from compute_nodes db table, because till now what I observed is: resource usages are stored in allocations db table at placement side, so same thing needs to be store again in compute_nodes table as well.13:05
*** hoonetorg has quit IRC13:08
jaypipesbhagyashri_s: excellent question :) I would really like to remove the usage fields from the compute_nodes table for those resource classes. however, we still rely on those fields unfortunately for some Compute REST API results.13:09
*** hoonetorg has joined #openstack-nova13:09
jaypipesbhagyashri_s: until we rewrite those REST API pieces to use placement, we can't get rid of them.13:09
jaypipesbhagyashri_s: lemme grab some links for you... one sec13:10
claudiub|2artom: hello. around?13:11
jaypipesbhagyashri_s: tada: https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/hypervisors.py#L5413:11
*** krtaylor has joined #openstack-nova13:11
*** lyan has joined #openstack-nova13:11
jaypipesbhagyashri_s: that compute REST API endpoint (/os-hypervisors) need to just go away... :)13:12
*** niraj_singh has joined #openstack-nova13:12
jaypipesbhagyashri_s: and essentially deprecated in favor of the placement API.13:12
*** coreywright has joined #openstack-nova13:12
bhagyashri_sjaypipes: ohh ok.13:12
jaypipesbhagyashri_s: which has the /usages endpoint13:12
jaypipesbhagyashri_s: make sense?13:12
bhagyashri_sjaypipes: yes13:12
jaypipesbhagyashri_s: note that the compute API's GET /os-hypervisors endpoint is an admin-only hting13:13
jaypipesthing13:13
jaypipesas is the placement API.13:13
*** mriedem has joined #openstack-nova13:15
bhagyashri_sjaypipes: so is there any plan in future to deprecate that13:16
*** links has quit IRC13:18
*** catintheroof has joined #openstack-nova13:19
bhagyashri_sjaypipes: and as you mentioned for compute rest api endpoint like /os-hypervisors we will get info from placement API.13:19
*** dave-mccowan has quit IRC13:19
artomclaudiub|2, yeah - give me, like, 30 minutes to drive kids to daycare13:19
*** jpena|lunch is now known as jpena13:20
*** gszasz has joined #openstack-nova13:21
jaypipesbhagyashri_s: it's not been thoroughly discussed, no... probably a good thing to bring up on the mailing list and discuss in Denver PTG13:21
bhagyashri_sjaypipes: so that resource usages related columns in the compute_node table will not be needed any more13:22
mriedemgibi: i don't see much advantage in mocking out the FakeDriver here rather than just stubbing out the manager method https://review.openstack.org/#/c/498627/1/nova/tests/functional/test_servers.py13:22
*** moshele has quit IRC13:22
*** moshele has joined #openstack-nova13:23
bhagyashri_sjaypipes: ok. Thank you for info and sure will discussed in Denver PTG.13:23
bhagyashri_sjaypipes: :)13:24
*** belmoreira has joined #openstack-nova13:26
*** lucasagomes is now known as lucas-brb13:26
*** slaweq has quit IRC13:29
*** slaweq has joined #openstack-nova13:30
*** slaweq has quit IRC13:30
*** udesale has quit IRC13:30
*** mrcc has joined #openstack-nova13:31
*** armax has quit IRC13:31
*** slaweq has joined #openstack-nova13:32
lpetrutHi, we have an issue with nova assisted volume snapshots when a separate DB is being used for each cell. The API only accepts a volume id, fetching the instance id from the BDM. The issue is that the BDM object cannot be retrieved as the nova api service is looking in the wrong DB (not knowing which cell to use).13:33
*** gouthamr has joined #openstack-nova13:33
*** baoli has joined #openstack-nova13:33
*** baoli has quit IRC13:34
lpetrutI thought you guys may provide some input :) relevant nova api code snippet: https://github.com/openstack/nova/blob/5d3a11b9c9a6a5aecd46ad7ecc635215184d930e/nova/compute/api.py#L4076-L407813:34
*** baoli has joined #openstack-nova13:35
gibimriedem: mocking the manager method changes that part of nova we try to verify. While mocking the FakeDriver would only change the behavior of the test env. I also OK with way you did in your refactor patch by adding FakeDriver variants with a good class doc13:35
mriedemlpetrut: ah very possible. this doesn't grab the instance first, but if it did, the _get_instance method in the compute API would set the cell mapping in the context13:35
*** tidwellr has joined #openstack-nova13:36
mriedemlpetrut: can you report a bug?13:36
gibimriedem: ahh I see why you didn't added a FakeDriver variant.13:36
lpetrutmriedem: sure. the issue is that at this point, the instance id is unknown as it's not passed in the api call.13:36
mriedemlpetrut: yeah, which sucks, so we'll likely have to iterate the cells looking for the bdm13:36
mriedemand once found, target the context for the rest of the calls13:37
mriedemwe wouldn't have hit this in CI because only certain drivers, like NFS, test this API13:37
mriedemthere is an NFS job, but it's non-voting and in the experimental queue i think13:37
mriedemgibi: right i'd have to create a new test class to use a new virt driver13:38
openstackgerritViktor Varga proposed openstack/nova master: Ensure image metadata when unshelving BFV instance  https://review.openstack.org/49872913:38
mriedemgibi: i can see what mocking out the driver method instead looks like13:38
lpetrutmriedem: yep, that one's failing because of this: http://logs.openstack.org/21/498321/5/check/gate-tempest-dsvm-full-devstack-plugin-nfs-nv/b8bca96/logs/screen-c-vol.txt.gz?level=TRACE13:38
mriedemyeah - open a bug and we'll get a fix and backport for pike13:39
mriedemprobably not the pike GA at this point though, RC2 was last week13:39
lpetrutmriedem: sure. thanks13:39
*** burt has joined #openstack-nova13:40
mriedemlpetrut: btw, does hyperv have a backend that supports this API? or just got this from a customer?13:40
lpetrutyeah, we have the Cinder SMB driver which uses it13:40
gibimriedem: if you mock.patch.object the FakeDriver object in only in the compute manager that you need to behave differently then you can even drop the if from your fake method. But these are just small gains I agree13:41
*** dave-mccowan has joined #openstack-nova13:41
mriedemlpetrut: ok, should get the docs updated then https://developer.openstack.org/api-ref/compute/#assisted-volume-snapshots-os-assisted-volume-snapshots13:41
mriedembecause that says only the libvirt driver supports this13:41
lpetrutgood point, thanks for mentioning it13:41
*** eharney has joined #openstack-nova13:41
mriedemlpetrut: actually i only see the libvirt driver implementing the "volume_snapshot_create" method in tree13:42
mriedemare you sure you don't have something in the compute hyperv repo that's not in nova?13:42
*** crushil has joined #openstack-nova13:42
lpetrutmriedem: actually, you're right. that didn't make it upstream13:43
mriedemthe bdm not found bug is still valid13:43
*** phuongnh has quit IRC13:43
mriedemfor libvirt + nfs13:43
lpetrutyep, I'll point to the NFS CI results13:43
*** gbarros has joined #openstack-nova13:44
*** Tom___ has joined #openstack-nova13:48
*** edmondsw has joined #openstack-nova13:49
*** slaweq_ has joined #openstack-nova13:49
gibimriedem: OK. I'm convinced. The manager method your patch mocks just basically calls the driver methdon I suggested to mock, so we are not loosing real coverage with your mock13:49
gibimriedem: so I'm +113:50
*** yamamoto has joined #openstack-nova13:50
mriedemheh, and i just changed the test to mock the driver13:51
mriedemand it's passing, so i'll just leave it13:52
*** Tom___ has quit IRC13:53
gibimriedem: ohh, OK13:53
*** edmondsw has quit IRC13:53
gibimriedem: I like it both ways now13:53
mriedemjaypipes: is the derek carr on that k8s rescheduler email the same as the quarterback for the raiders?13:54
mriedembecause if so, that would be impressive13:54
*** slaweq_ has quit IRC13:54
mriedemoh also a red hat employee13:54
*** cleong has joined #openstack-nova13:55
*** yamamoto has quit IRC13:56
*** mriedem has quit IRC13:58
openstackgerritDan Smith proposed openstack/nova-specs master: WIP: Add migration-allocations spec  https://review.openstack.org/49851014:01
*** marst has joined #openstack-nova14:03
*** zhurong has quit IRC14:03
*** alexchadin has quit IRC14:04
openstackgerritStephen Finucane proposed openstack/nova master: doc: Add user index page  https://review.openstack.org/49881714:07
openstackgerritStephen Finucane proposed openstack/nova master: doc: Add configuration index page  https://review.openstack.org/49881814:07
openstackgerritStephen Finucane proposed openstack/nova master: doc: Cleanup of existing index pages  https://review.openstack.org/49881914:07
openstackgerritStephen Finucane proposed openstack/nova master: doc: Add contents pages  https://review.openstack.org/49882014:07
openstackgerritStephen Finucane proposed openstack/nova master: docs: Rename cellsv2_layout -> cellsv2-layout  https://review.openstack.org/49882114:07
*** ociuhandu has quit IRC14:07
*** alexchadin has joined #openstack-nova14:08
sean-k-mooneybeagles: hi o/ did you get an answer to your os-vif question?14:08
*** armax has joined #openstack-nova14:08
beaglessean-k-mooney, no I ended up filing a bug to avoid falling through cracks https://bugs.launchpad.net/os-vif/+bug/171359014:09
openstackLaunchpad bug 1713590 in os-vif "Plugging VFs no longer works without a readable phys_switch_id" [Undecided,New]14:09
beaglessean-k-mooney, I'm not 100% on the code path so there might be something else going on or the configuration of the test system might be missing something. From the information I had, this seemed a valid issue14:10
artomclaudiub|2, hey, back, took longer than planned, sorry14:11
claudiub|2artom: no problem. :)14:11
claudiub|2artom: i have a question regarding the test_device_tagging tempest test14:12
artomclaudiub|2, sure14:12
claudiub|2artom: what kind of image are you using when testing it?14:13
claudiub|2artom: i assume cirros, right?14:13
openstackgerritStephen Finucane proposed openstack/nova master: doc: Remove deprecated call to sphinx.util.compat  https://review.openstack.org/49882414:13
*** ijw has joined #openstack-nova14:13
sean-k-mooneybeagles: os vif does not handel pluging vf normally14:15
sean-k-mooneybeagles: its should only do this if you have ovs with hardware offlod support.14:16
artomclaudiub|2, it uses a boot volume, created from whatever's image defined in the configuration14:16
*** marst has quit IRC14:16
artomclaudiub|2, https://github.com/openstack/tempest/blob/master/tempest/api/compute/servers/test_device_tagging.py#L10214:17
sean-k-mooneybeagles: in you ml2_conf.ini i would guess you have the openvswitch mech driver listeted before the sriovnic agent correct?14:17
claudiub|2artom: i know. :)14:17
claudiub|2artom: but what do *you* use to test it with? :D14:17
artomclaudiub|2, doh, hah! Yeah, I used cirros14:17
claudiub|2hm interesting14:17
andreafzul: around? I have a change in Tempest that if merged would break some nova-lxd integration tests from the in-tree tempest plugin14:17
artomclaudiub|2, what's up?14:17
beaglessean-k-mooney, interesting ... tbh I don't know, the test system is likely gone at the moment. Where there was a patch that introduced this change to tripleo a short while ago, I think that's a safe bet14:17
*** awaugama has joined #openstack-nova14:18
andreafzul: but I don't see those tests running anywhere in nova-lxd gate, so I was wondering if I can just change tempest and propose a patch to fix nova-lxd afterward?14:18
sean-k-mooneybeagles: basically  is you want sriov and ovs in the same deployment and all your sriov host dont supprot hardware offload you have to list sriov first.14:18
*** cfriesen has joined #openstack-nova14:18
claudiub|2artom: i'm also using a cirros, but only eth0 is configured in it, even if all the interfaces are attached. the problem is that the interface which gets to be eth0 is random, and making the VM un-ssh-able14:19
*** hongbin has joined #openstack-nova14:19
*** licanwei has quit IRC14:19
claudiub|2artom: as the floating IP is associated with just one of the many private IPs the VM has.14:19
beaglessean-k-mooney,k. I'll check on that14:20
sean-k-mooneybeagles: that said we can probaly harden that could a more and retrun a specific exction rather then causing an io error by reading a file that does not exist14:20
*** kfarr has joined #openstack-nova14:20
claudiub|2artom: was wondering if you had an issue like this14:20
artomclaudiub|2, I haven't run it manually in a long time14:20
artomclaudiub|2, how long has this been going on?14:20
*** zxy has quit IRC14:21
claudiub|2artom: i've started testing the pike release, so i've only seen it nowadays14:21
artomclaudiub|2, interesting. https://review.openstack.org/#/c/463837/ merged recently, and it changes which floating ip client gets used14:21
artomBut it passed the gate, so in and of itself it's not the root cause14:22
claudiub|2hm, interesting14:23
*** marst has joined #openstack-nova14:23
*** marst_ has joined #openstack-nova14:23
artomclaudiub|2, there was also https://review.openstack.org/#/c/471116/14:24
artomBut again, passed the gate, so by itself it's not the issue14:24
zulandreaf: check with jamespage14:24
*** Tom___ has joined #openstack-nova14:24
artomclaudiub|2, I don't suppose you're running with nova-network by omission?14:24
claudiub|2artom: i don't think i've ever used nova-network. :)14:25
artomclaudiub|2, good.14:25
andreafzul: cool thanks - jamespage: this is the tempest patch https://review.openstack.org/#/c/492577/14:25
andreafjamespage: and this is nova-lxd code that would be affected http://git.openstack.org/cgit/openstack/nova-lxd/tree/nova_lxd_tempest_plugin/tests/api/compute/servers/test_create_server.py14:26
artomclaudiub|2, I'm sorry I can't be of more help, but I feel like you'll have to dig into tempest to figure out why it's doing that14:26
*** felipemonteiro__ has quit IRC14:26
artomThe test itself doesn't assign any floating IPs, it just sets validatable=True14:26
claudiub|2artom: yeah, will do. thanks for the input though. :)14:26
*** felipemonteiro__ has joined #openstack-nova14:26
artomAnd then tempest does its thing behind the scenes14:26
*** vikrant has joined #openstack-nova14:27
*** marst has quit IRC14:27
*** ijw has quit IRC14:28
*** sree has quit IRC14:28
*** mriedem has joined #openstack-nova14:28
*** sree has joined #openstack-nova14:29
*** baoli has quit IRC14:29
artomclaudiub|2, it may have something to do with the tenant network in your deployment: https://github.com/openstack/tempest/blob/master/tempest/common/compute.py#L20614:29
*** annegentle has joined #openstack-nova14:30
stephenfinmriedem, jaypipes: Fancy taking a look at these? https://review.openstack.org/#/c/498817/ and successors14:31
stephenfinThey fix some broken links that dhellmann pointed out and should be backported to Pike14:31
*** felipemonteiro__ has quit IRC14:31
mriedemstephenfin: seems like something sdague would have an opinion about14:31
mriedemregarding layout14:31
stephenfinIndeed. Any idea when he's back?14:31
mriedemthis week14:31
mriedemhe was here yesterday14:32
stephenfinOh, then I can wait for him too14:32
stephenfin(fwiw, I'm mostly just moving his stuff about)14:32
*** sree has quit IRC14:33
*** annegentle has quit IRC14:34
openstackgerritMatt Riedemann proposed openstack/nova master: Add functional recreate test for live migration pre-check fails  https://review.openstack.org/49862714:34
claudiub|2artom: hm, so basically, the fip is always associated with the tenant network. while the NIC which gets to be eth0 in the VM is random for me.14:35
*** eharney has quit IRC14:35
claudiub|2artom: while before it seems that the fip was associated with the VM, not just one port.14:36
sdaguestephenfin: https://review.openstack.org/#/c/498817/1/doc/source/index.rst I really don't want to do that until we decide that's our direction at PTG14:36
sdaguebecause we just went the other way14:36
stephenfinsdague: We don't have much of a choice though. If we don't do this, we can't hook into the likes of https://docs.openstack.org/pike/user/14:37
*** annegentle has joined #openstack-nova14:37
sdaguestephenfin: we can14:37
stephenfinWe need solid landing pages for each of those. I can duplicate stuff into the main index page, but that seems rather unhelpful :/14:37
sdaguewe don't have to take out all the deep linking from the main index page14:37
sdaguestephenfin: people are navigating in from all different directions, having every single one of them being explainatory is good14:38
stephenfinWould a simple '.. include' of each index page be a viable option?14:38
stephenfin*...option, in that case?14:38
sdaguestephenfin: I don't know, it probably won't be coherent14:38
stephenfinAye, probably not :/14:39
sdagueIt's really ok to explain things multiple ways and give multiple setups for why following a link is useful before you do it14:39
mriedemso the problem is you get this today? https://docs.openstack.org/nova/pike/user/14:39
mriedemwhich has no index14:39
sdaguemriedem: right, we should *definitely* fix that14:39
mriedemsame for https://docs.openstack.org/pike/admin/ and others i imagine14:40
sdaguewhich is this - https://review.openstack.org/#/c/498817/1/doc/source/user/index.rst14:40
*** yangyapeng has joined #openstack-nova14:40
sdaguewhich is fine14:40
mriedemoh we have https://docs.openstack.org/nova/pike/admin/14:40
sdaguewe actually have an admin index already14:40
mriedemand https://docs.openstack.org/nova/pike/reference/14:40
sdaguemy objection is stripping out the context from https://review.openstack.org/#/c/498817/1/doc/source/index.rst at the same time14:40
mriedemso yeah why not just fix the user index?14:40
stephenfinthe admin index needs work though. It's ugly as sin :)14:41
sdaguestephenfin: sure, which is fine14:41
stephenfinmriedem: because it's basically duplicating exactly what's on the main index14:41
*** lucas-brb is now known as lucasagomes14:41
stephenfinlike, I cut and paste14:41
sdagueI think the point is we need to realize that *all* of these are landing pages, for different contexts14:41
sdaguestephenfin: which is fine14:41
sdaguenova needs a coherent landing page for hitting the nova docs directly, the various guides need index pages that make sense in the context of the content they are exposed in14:42
mriedemstephenfin: why would it be the same as the main index?14:42
mriedemshouldn't /user just be what's in https://docs.openstack.org/nova/latest/#for-end-users ?14:42
mriedemfrom the main page?14:42
*** edmondsw has joined #openstack-nova14:42
stephenfinsdague: I don't know. It seems impractical to be taking a two hat approach in the long term14:42
*** mrcc has quit IRC14:42
mriedemand exclude "for operators" and "for contributors" stuff14:42
sdaguestephenfin: that's what good documentation looks like14:42
sdagueit has a context and an audience14:43
*** lajoskatona has quit IRC14:43
stephenfinBut...but...bother and hassle :(14:43
stephenfinSo if I drop the index page changes, the rest of it is reasonable enough?14:43
sdaguethe deep content isn't replicated, but the context and "why would I ever want to follow this link" is taylored to the reader you expect14:43
stephenfinat least, at first glance14:43
sdaguestephenfin: yeh, I'd be fine with that14:43
stephenfinKewl. I'll do that.14:44
stephenfinwhich isn't to say I'm enamoured with leaving the index page the way it is, but that's PTG stuff14:44
stephenfinAlso, mriedem: that's what I did :)14:44
stephenfinBut I stripped the stuff from the main index page then as duplication14:44
mriedemyour proposed user index has a bunch of operator stuff in it14:46
mriedemand nothing about the API version history or compute API reference14:46
mriedemwhich an end user would want14:46
gibicdent, mriedem: fyi, there is a resource allocation bug in resize same host when custom resources are involved: https://bugs.launchpad.net/nova/+bug/171373914:46
openstackLaunchpad bug 1713739 in OpenStack Compute (nova) "VM resize and confirm on the same host fails with custom resources" [Undecided,New]14:46
gibicdent, mriedem: regression test is being created14:46
cdentgibi: we were having _so_ much fun until you came back14:46
*** hrw has quit IRC14:47
stephenfinmriedem: Yeah, none of that is in the '/user' directory though. I was keeping only that stuff in there. A cross-reference wouldn't be any harm tho14:47
mriedemi guess that is either external or in reference directory14:48
sdaguestephenfin: we don't strictly have to only list stuff in /user14:48
mriedemguh, so...just have source/user/index.rst link back to the top level main index and be done with it14:48
gibicdent: It is not my finding :) I just suggested the certain test case to be created14:48
*** hrw has joined #openstack-nova14:48
*** itlinux has joined #openstack-nova14:49
sdaguehonestly, to avoid 404s, I'd say just write the user guide we think should be there, link whatever content is whereever, and it's fine14:49
sdaguelinks are not restricted about where they can go14:49
sdagueand the only people that will care that user isn't in the url are in this room14:49
cdentthe bug makes sense: the allocation creation routines are insufficiently custom resource class aware14:49
cdentgibi: ^14:49
*** slaweq_ has joined #openstack-nova14:50
openstackgerritSteve Noyes proposed openstack/nova master: update live migration to use v3 cinder api  https://review.openstack.org/46398714:50
*** eharney has joined #openstack-nova14:50
sean-k-mooneystephenfin: did ye bottom out on how to handel pci devices on power yesterday?14:51
*** ratailor has joined #openstack-nova14:51
gibicdent: interestingly we only see problems in the resize same host case so far14:51
openstackgerritEd Leafe proposed openstack/nova-specs master: Return Destination Objects  https://review.openstack.org/49883014:51
*** alexchadin has quit IRC14:52
edleafemriedem: dansmith: cdent: jaypipes: ^^ wrote a quick spec on the object solution we discussed yesterday. Feedback appreciated!14:52
*** yamamoto has joined #openstack-nova14:52
cdentedleafe: thanks. probably won’t have a chance to look with any rigor until tomorrow afternoon14:53
openstackgerritEd Leafe proposed openstack/nova-specs master: Return Destination Objects  https://review.openstack.org/49883014:53
jaypipesedleafe: cool. will review this evening.14:54
jaypipesedleafe: which is soon :)14:54
edleafejaypipes: thx14:54
stephenfinsean-k-mooney: I think efried and claudiub|2 opened some blueprints14:54
*** josecastroleon has joined #openstack-nova14:54
stephenfinhttps://blueprints.launchpad.net/nova/+spec/allow-pci-alias-choose-subset-of-devices and https://blueprints.launchpad.net/nova/+spec/devices-as-resources14:54
*** edmondsw has quit IRC14:54
sean-k-mooneyedleafe: ovo for scheuler info from placement make sense or at least object instead of dictionaries.14:54
*** slaweq_ has quit IRC14:55
*** sterdnotshaken has joined #openstack-nova14:55
sean-k-mooneystephenfin: there are two other spec that related to sriov bonding that would be good to group with those14:55
edleafesean-k-mooney: placement is http, so ovo isn't applicable.14:56
edleafesean-k-mooney: but yeah, from scheduler->conductor, ovo will help14:56
sean-k-mooneyedleafe: it is if you call object_to_primitive and then jsonutils.dump on it first14:56
sean-k-mooneyi would be really happy if we could start passing json serialised ovo across our apis in the future14:57
openstackgerritVladyslav Drok proposed openstack/nova master: Fix _delete_inventory log message in report client  https://review.openstack.org/49883314:57
*** edmondsw has joined #openstack-nova14:58
*** yamamoto has quit IRC14:58
mriedemedleafe: jaypipes: dansmith: we should return compute node uuid back in the dict from select_destinations too, that would make lookups for providers on the client side quicker, so the client doesn't need to lookup a compute node by host/node just to get the uuid14:58
edleafemriedem: agreed. I already did that in my first crack at these objects14:59
mriedemthe dict that's returned isn't even versioned today14:59
dansmithI wasn't really paying attention to that conversation so I'll have to catch up when I look at that14:59
mriedemthis is unrelated14:59
mriedembut to fix a bug with cleaning up allocations during live migration,14:59
jaypipesmriedem: yup, agreed. good idea.14:59
mriedemi need the compute node uuid to remove the allocations and only have the dict from select_destinations which has the host/nodename15:00
edleafemriedem: we don't really need that dict per se, just the data in it15:00
edleafeand the object will version that15:00
mriedemso let's say i added a uuid key to the dict that's returned today, what would we change for the version?15:00
mriedemthe client rpc?15:00
mriedem*scheduler client rpc? or manager?15:00
edleafeprobably the former15:01
mriedemwell it's 4.5 either way i think15:01
edleafebut there's no point if we're gonna be changing the interface to use objects15:01
*** yamahata has joined #openstack-nova15:01
*** edmondsw has quit IRC15:02
mriedemplus i couldn't backport this, and we need to backport this fix...15:02
mriedemTODO in the code it is15:02
beaglessean-k-mooney, thanks for the info btw!15:03
edleafemriedem: you mean you don't have any reference to the compute node in the migration?15:03
sean-k-mooneymriedem: well for now you would have to do a condtional check to see if the uuid is present in the dictionary and look it up if not15:03
mriedemedleafe: not when we're still in the conductor task15:03
mriedemi have the dict returned from select_destinations15:03
mriedemwhich has the host/nodename, which i can use to lookup the ComputeNode to get the UUID15:03
sean-k-mooneybeagles: no worries i was off since thursday so just go back15:03
mriedemit's just a hassle15:03
artomclaudiub|2, fips have always been associated to ports IIRC15:04
edleafemriedem: I see15:04
mriedemsean-k-mooney: i'd rather not write in dead code15:04
sean-k-mooneymriedem: agreed but it may be needed for the backport. if we go to ovo defintions and and a object based interfaces then we can do it correctly in that version for queens on15:05
claudiub|2artom: yeah, you're right. anyways. will let you know what i'll find out15:05
artomclaudiub|2, sure, thanks15:06
cdentedleafe: i lied, apparently i had something to say now15:06
*** baoli has joined #openstack-nova15:06
edleafecdent: typical15:07
mriedemsean-k-mooney: i'm just going to lookup the compute node using the host and node strings that i already have15:08
mriedemold school15:08
mriedemwe can clean it up in queens with a uuid we get back later if that happens15:08
mriedemhence the TODO15:08
cdentedleafe: well you know, I was taunted, I couldn’t help myself, I’m an easy target for bait15:08
sean-k-mooneyspeaking of ptg does anyone have the link to nova etherpads?15:11
mriedemsean-k-mooney: https://etherpad.openstack.org/p/nova-ptg-queens15:12
*** jaypipes has quit IRC15:12
sean-k-mooneymriedem: thanks :)15:12
*** lennyb has quit IRC15:12
sean-k-mooneyi just got approval at the weekend to travel so not i need to book all the things15:12
mriedemsean-k-mooney: and start coming up with 10 different scheduler filters you want to talk about15:13
mriedemlike in ATL15:13
*** Oku_OS is now known as Oku_OS-away15:13
*** yangyapeng has quit IRC15:13
*** yangyapeng has joined #openstack-nova15:13
*** felipemonteiro has joined #openstack-nova15:14
*** yangyapeng has quit IRC15:14
sean-k-mooneyhaha well perhaps. honestly alot of what i want to talk about on that front will hopefully be covered in the existing scheduler and placement sessiosn15:14
*** yangyapeng has joined #openstack-nova15:14
*** itlinux has quit IRC15:15
*** shaner has quit IRC15:17
sean-k-mooneythat said some topics i want to ensure we can model with placement and traists are, verifed boot(not the same as trusted boot), fpgas for tenants, power management of guests, and bandwith based scheduling(maily for sriov but maybe others in the future).15:18
*** felipemonteiro has quit IRC15:19
mriedemi think the huawei product team has a requirement for sriov bandwidth based scheduling15:19
mriedembut i don't really have details yet15:19
sean-k-mooneyim also interested in the pci overhal topics and live migration both across vif types and with sriov15:20
sean-k-mooneymriedem: cool we started working on that last year i think rodolfo had some patches up at one point15:20
sean-k-mooneywe should be able to model that with nested resource providers.15:21
gibisean-k-mooney, mriedem: ericsson also interested in the bandwith based scheduling, both for sriov and for ovs port as well15:21
*** gyee has joined #openstack-nova15:21
sean-k-mooneyeach pf would have a pool of bandwidth that could be consumed by the vf15:21
sean-k-mooneygibi: ovs does not support enforcing the bandwidth15:21
sean-k-mooneyactully it has bandwidth limits15:21
sean-k-mooneyit just does not support mimium bandwidth guarentees15:22
*** gmann has quit IRC15:22
gibisean-k-mooney: true, but as a first step we can trust the guest to only send/receive the requested bandwidth15:22
*** gmann has joined #openstack-nova15:22
mriedemsean-k-mooney: gibi: has anyone started a spec or etherpad for ideas or ML discussion, anything?15:22
gibimriedem: not from ericsson side15:23
sean-k-mooneywe had neutron or nova spec for this for pike15:23
gibisean-k-mooney: I think there was a neutron spec15:23
*** lennyb has joined #openstack-nova15:23
edleafecdent: taunted right back15:24
sean-k-mooneyya ill see about geting a nova one created for queens and add it to the etherpad15:24
cdentedleafe: gracias15:24
gibisean-k-mooney, mriedem: here is the neutron spec https://review.openstack.org/#/c/396297/15:24
mriedemcool thanks, i'll pass this along15:25
sean-k-mooneymriedem: we have already landed part of this i think or atleas implemented it. what was missing on the nova side was a way to ensure we dont over subscribe which needs placemnet to do cleanly15:26
sean-k-mooneymriedem: ya the sriov enforcement is implemented here https://review.openstack.org/#/c/401254/2515:27
cfriesengibi: is it reasonable to trust the guests to do that?  That seems like a private-cloud-only scenario.15:28
*** lucasagomes is now known as lucas-afk15:28
sean-k-mooneycfriesen: we dont have to turst them in all cases15:28
sean-k-mooneywith sriov the hardwer can enforce the minium bandwith gureentee15:29
sean-k-mooneywith vpp we may be able to do that in software and we could also extend ovs in the future to do it but that is a lot more work15:29
gibicfriesen: this is a temporary solution until OVS can be used to actually limit the bandwidth15:30
gibicfriesen: but you are correct that this is for private cloud only15:30
cfriesensean-k-mooney: gibi: we'd probably make use of the SRIOV case15:31
*** rmart04 has quit IRC15:32
sean-k-mooneyfor the sriov case the hardware will enforce the minimum bandwidth. provided we dont oversubscribe. eg. create 11 vf with 1G minium bandwith on a 10G nice15:32
*** tidwellr has quit IRC15:32
*** tidwellr has joined #openstack-nova15:33
*** sterdnotshaken has quit IRC15:34
*** sterdnotshaken has joined #openstack-nova15:35
*** chyka has joined #openstack-nova15:35
*** mdnadeem has quit IRC15:37
*** slaweq has quit IRC15:38
openstackgerritLucian Petrut proposed openstack/nova master: [wip] Fix nova assisted volume snapshots  https://review.openstack.org/49884515:39
*** andreas_s has quit IRC15:43
cdentedleafe: back at ya15:44
*** diga has quit IRC15:45
*** ragiman has quit IRC15:46
edleafecdent: gee thanks!15:46
cdentit’s a pleasure collaborating with you sir15:46
mriedemlpetrut: mostly right, but some comments inline15:46
edleafecdent: wish I could say the same!  :-P15:46
cdent\o/15:47
*** sree has joined #openstack-nova15:47
lpetrut_mriedem: great, thanks for the quick review15:48
*** Apoorva has joined #openstack-nova15:49
lpetrut_mriedem: about the context manager: didn't use that so that the targeted cell remains set15:49
*** lpetrut_ has quit IRC15:50
*** wangqwsh has joined #openstack-nova15:51
mriedemoh hrm15:51
mriedemlpetrut: yeah good point15:51
*** slaweq has joined #openstack-nova15:51
mriedemmaybe leave a comment in there then, in the code i mean15:51
lpetrutsure15:51
*** felipemonteiro has joined #openstack-nova15:52
*** sree has quit IRC15:52
*** sree has joined #openstack-nova15:52
*** sree has quit IRC15:53
*** sree has joined #openstack-nova15:53
*** dtantsur|busy is now known as dtantsur|afk15:53
*** edmondsw has joined #openstack-nova15:54
*** yamamoto has joined #openstack-nova15:54
*** rajathagasthya has joined #openstack-nova15:55
*** sree has quit IRC15:55
*** slaweq has quit IRC15:56
*** Apoorva has quit IRC15:56
*** ratailor has quit IRC15:56
*** Apoorva has joined #openstack-nova15:56
*** edmondsw_ has joined #openstack-nova15:58
*** edmondsw has quit IRC15:58
*** yamamoto has quit IRC16:00
*** edmondsw_ has quit IRC16:00
*** Apoorva has quit IRC16:01
*** thorst_afk has quit IRC16:01
openstackgerritElod Illes proposed openstack/nova master: Functional test: evacuate with no compute  https://review.openstack.org/49848216:02
mriedem#success gibi is now on the nova-core team16:03
openstackstatusmriedem: Added success to Success page16:03
mriedemi've successfully added success to the success page16:03
*** ssmith has joined #openstack-nova16:03
melwitt\o/16:03
ssmithmriedem: Changed that one line of code and now get this on a locked instance: "Instance a29d1e77-5191-4629-a913-ae98ed22e284 is locked (HTTP 409)"16:04
*** thorst_afk has joined #openstack-nova16:05
ssmithNo "Locked" indicator on info of cli show so we have to remember if we get that message it's likely locked.  Too bad, this is a great feature in AWS which is called "Termination Protection"16:07
*** Tom___ has quit IRC16:09
*** ijw has joined #openstack-nova16:09
*** ijw has quit IRC16:09
*** ijw has joined #openstack-nova16:09
*** tidwellr has quit IRC16:10
mriedemssmith: nova also has that flag via the ec2api :)16:10
mriedemdisable_terminate i think it's called16:10
*** tidwellr has joined #openstack-nova16:11
*** tidwellr has quit IRC16:11
mriedemssmith: which cli doesn't show if it's locked or not?16:11
gibiThank you all of you support helping me becoming a nova-core. \o/16:11
ssmithopentack server show16:11
mriedemssmith: use microversion 2.916:12
mriedemor greater16:12
*** tidwellr has joined #openstack-nova16:12
mriedemi dont know if the osc cli will print it though16:12
mriedembut the locked field comes back in the GET /servers/id response in 2.9+16:12
ssmithnova show [server] does show locked or now16:13
mriedemyeah, because nova cli negotiates for the latest microversion understood between the client and the server16:13
mriedemby default16:13
mriedemosc doesn't16:13
mriedemssmith: when you say, "Changed that one line of code and now get this on a locked instance: "Instance a29d1e77-5191-4629-a913-ae98ed22e284 is locked (HTTP 409)"" - you mean when you tried to delete the instance, right?16:13
ssmithYes, tried to delete. Get "Error: Unable to delete instance" in the UI and 409 on the cli16:15
*** tesseract has quit IRC16:15
mriedemssmith: yeah, so we'd need a change proposed on master that uses a new policy rule for this16:15
mriedemrather than something in nova.conf16:15
mriedemssmith: if this isn't something you actually want to work on, then you could propose a backlog spec and someone else could pick itup16:15
ssmithThat would be better16:15
mriedemhttps://specs.openstack.org/openstack/nova-specs/specs/backlog/index.html16:16
*** moshele has quit IRC16:16
ssmithOK16:16
*** tidwellr has quit IRC16:16
ssmithAnd just to confirm no way to get the lock status to show in "openstack server show"?16:17
mriedemssmith: there is an option with the osc cli to specify a microversion16:17
mriedemso i'd try that first16:17
openstackgerritStephen Finucane proposed openstack/nova master: docs: Rename cellsv2_layout -> cellsv2-layout  https://review.openstack.org/49882116:18
openstackgerritStephen Finucane proposed openstack/nova master: doc: Add contents pages  https://review.openstack.org/49882016:18
openstackgerritStephen Finucane proposed openstack/nova master: doc: Cleanup of existing index pages  https://review.openstack.org/49881916:18
openstackgerritStephen Finucane proposed openstack/nova master: doc: Add configuration index page  https://review.openstack.org/49881816:18
openstackgerritStephen Finucane proposed openstack/nova master: doc: Add user index page  https://review.openstack.org/49881716:18
mriedemof course i can't search for microversions in just the osc docs https://docs.openstack.org/python-openstackclient/latest/search.html16:19
mriedemsince the docs migration to use the new sphinx theme16:19
mriedemRAR16:19
*** tidwellr has joined #openstack-nova16:19
mriedemssmith: openstack help compute or something should give you the optoins16:19
mriedemit's --compute-api-version or something16:19
*** sambetts is now known as sambetts|afk16:21
mriedemssmith: --os-compute-api-version i think16:22
ssmithok16:22
*** itlinux has joined #openstack-nova16:23
mriedemi don't know if the osc cli whitelists the fields it shows though16:23
ssmithos-compute-api-version <compute-api-version>16:23
ssmith                        Compute API version, default=2.116:23
*** annegentle has quit IRC16:23
mriedemyeah, so specify 2.916:24
ssmithopenstack server show a29d1e77-5191-4629-a913-ae98ed22e284 --os-compute-api-version 2.9 WORKED16:25
mriedemcool16:25
ssmithAny way to permanently set the version?16:25
mriedemenv var16:25
mriedemOS_COMPUTE_API_VERSION=2.9 i think16:25
*** Swami has joined #openstack-nova16:26
ssmithDid a "export OS_COMPUTE_API_VERSION=2.9" which worked16:28
*** thorst_a_ has joined #openstack-nova16:28
openstackgerritMatt Riedemann proposed openstack/nova master: Cleanup allocations on invalid dest node during live migration  https://review.openstack.org/49886116:31
mriedemdansmith: cdent: gibi: here is the fix for the live migration pre-check error bug ^16:31
*** thorst_afk has quit IRC16:32
efriedsean-k-mooney that first bp was https://blueprints.launchpad.net/nova/+spec/pci-by-device-id whose spec (https://review.openstack.org/497965) has some good discussion started.16:33
*** vladikr has quit IRC16:34
efriedjaypipes and I talked about it yesterday a bit and he encouraged me to move some of the main themes over to the other one stephenfin mentioned - https://blueprints.launchpad.net/nova/+spec/devices-as-resources - whose spec seed (not yet started) is here: https://review.openstack.org/49797816:34
efried...and to start an etherpad for same topic for the PTG.16:34
*** rajathagasthya has quit IRC16:35
*** rajathagasthya has joined #openstack-nova16:36
openstackgerritLucian Petrut proposed openstack/nova master: Fix nova assisted volume snapshots  https://review.openstack.org/49884516:37
openstackgerritLajos Katona proposed openstack/nova master: WIP: Test server movings with custom resources  https://review.openstack.org/49739916:37
*** vladikr has joined #openstack-nova16:39
*** alexpilotti has joined #openstack-nova16:40
*** efried has quit IRC16:40
dansmithmriedem: so, my feeling as we neared the end of pike was that we were really not doing ourselves any favors by trying to put the allocation stuff into the existing RT calls since we need to do different things and need some context from the compute manager to do the right thing16:41
dansmithwhich is why we do some silly stuff in RT like looking at if prefix == 'old_' to do certain things16:41
*** felipemonteiro has quit IRC16:41
dansmithmriedem: for this migration uuid thing, I kinda want to add these new paths to compute manager itself, so at least we can delete RT eventually without needing to move things out of it16:42
dansmithdoes that sound legit?16:42
dansmithit might mean moving some of our existing allocation handling back out of RT as well and just clearly marking which bits are legacy pike behavior and not16:42
*** shaner has joined #openstack-nova16:44
*** penick has joined #openstack-nova16:44
cdentdansmith: I can only speak for myself, but I think that’s totally legit16:46
*** rcernin has quit IRC16:46
*** belmoreira has quit IRC16:46
mriedemdansmith: i think in general that's OK given we've already started duplicating some allocation-specific stuff in the compute manager outside of the RT16:47
dansmithyeah16:47
mriedeme.g. https://review.openstack.org/#/c/496976/16:47
dansmithI just spent 45 minutes trying to detect the condition I need from down in RT and it just doesn't make any sense I think16:47
mriedemit's also harder to debug16:48
mriedembecause of the layering16:48
dansmithyeah16:48
mriedemRT is always a new journey for me everytime i have to look into it16:48
*** itlinux has quit IRC16:48
*** pcaruana has quit IRC16:48
*** itlinux has joined #openstack-nova16:49
*** dave-mccowan has quit IRC16:50
*** thorst_a_ has quit IRC16:51
*** thorst_afk has joined #openstack-nova16:52
*** slaweq has joined #openstack-nova16:52
*** efried has joined #openstack-nova16:53
*** lpetrut has quit IRC16:54
*** links has joined #openstack-nova16:55
*** thorst_afk has quit IRC16:56
*** yamamoto has joined #openstack-nova16:56
*** amarao has quit IRC16:56
*** abalutoiu has joined #openstack-nova16:56
*** slaweq has quit IRC16:57
*** kbaegis has joined #openstack-nova16:57
*** jpena is now known as jpena|off16:59
openstackgerritMatt Riedemann proposed openstack/nova master: Refactor LiveMigrationTask._find_destination  https://review.openstack.org/49887416:59
*** baoli has quit IRC17:01
*** baoli has joined #openstack-nova17:02
*** sree has joined #openstack-nova17:02
*** yamamoto has quit IRC17:02
*** ralonsoh has quit IRC17:03
*** thorst_afk has joined #openstack-nova17:04
*** vladikr has quit IRC17:04
*** vladikr has joined #openstack-nova17:04
sean-k-mooneyefried: this one https://review.openstack.org/#/c/497965/2/specs/queens/approved/pci-by-device-id.rst17:05
sean-k-mooneyill take a look at it17:05
*** sree has quit IRC17:06
*** penick has quit IRC17:09
*** gbarros has quit IRC17:09
efriedsean-k-mooney Great, thanks.  FYI that one's not going to fly as currently conceived, but it's got some good problem descriptions and the discussion is getting us moving in the right direction.17:10
openstackgerritMerged openstack/nova master: Enhance support matrix document  https://review.openstack.org/48201317:10
mriedemso it looks like we have to allocation-related bugs with evacuate,17:11
mriedem1. if you specify a host, we don't claim b/c we bypass the scheduler https://github.com/openstack/nova/blob/16.0.0.0rc2/nova/conductor/manager.py#L74917:11
*** gbarros has joined #openstack-nova17:11
mriedem2. if you don't specify a host, we call the scheduler to find a host and create the allocations, but it rebuild fails in the compute we don't cleanup the allocations on failure17:12
*** gbarros has quit IRC17:12
sean-k-mooneyefried: yes but its a good start. i had a call with jay and other regarding smartnic offload of ovs where we also dissued how the dual role the whitelist is playing is problematic.17:13
*** wangqwsh has quit IRC17:13
efriedsean-k-mooney I would welcome thoughts on how the whitelist should work.17:13
*** baoli has quit IRC17:14
*** coreywright has quit IRC17:14
*** slaweq has joined #openstack-nova17:14
efriedI mean, we need to have one.  And it should probably be able to identify devices by classes or by specific IDs.17:14
sean-k-mooneyefried: basically i think it should just filter the list of pci devices and all other fuctionality such as tag or physnet association should be put into a different config option17:15
efriedYeah, that makes sense.17:16
sean-k-mooneydevice classes gets a little messy but may be usefull17:16
sean-k-mooneyyou proably dont want to just whitelist all net devices17:16
*** penick has joined #openstack-nova17:17
*** vladikr has quit IRC17:18
efriedIf it's to fit in with the other RP stuff, I would imagine the whitelist would be able to specify any of the qualitative or quantitative properties of the resource as reported by the driver.17:18
efried"whitelist anything of type nic with a line speed >= 10Gbps"17:18
*** efoley has quit IRC17:18
*** kbaegis has quit IRC17:19
openstackgerritBalazs Gibizer proposed openstack/nova master: Test resource allocation during soft delete  https://review.openstack.org/49515917:20
openstackgerritElod Illes proposed openstack/nova master: Functional test: evacuate with no compute  https://review.openstack.org/49848217:20
efriedCourse, trying to figure out a way to do boolean logic in a conf var...17:21
sean-k-mooneyefried: the issue with all nic >=10GB is nova now needs per class special casing of the whitelist parsing17:22
*** ijw has quit IRC17:22
efriedsean-k-mooney Well, the way Jay was talking about it, the whitelisting would be done by the driver.17:22
efriedSo the driver can define whatever whitelisting syntax it wants17:22
efriedAnd it will therefore naturally be hypervisor-appropriate17:23
efriedThat's one of the main things that I'm keeping an eye out for: that we don't wind up with a solution that works great for libvirt, but has to be kludged for e.g. HyperV or PowerVM.17:24
sean-k-mooneywell if we were talking about extending it to the classes as reproted but livirts nodedev-list in the libvirt dirver then that would be ok with me17:24
sean-k-mooneybut other wise im not sure17:24
sean-k-mooneywhat that really means is we need to all agree on a common set of resoce clasess in placement and driver specific implentaiton in the virt dirver that convert from the hyperviros defiened view into the generic form17:25
cdentqueue the super upper ontology17:26
sean-k-mooneywell i think we can proably agree that by the time the resouce lands in the condoctor/scheduler we proably dont want to still have to care about the hyperviror on the compute node17:28
*** baoli has joined #openstack-nova17:29
efriedsean-k-mooney Agree, and that's going to be a tough thing to do.17:29
efriedE.g. last time it was "assumed" that every PCI device would naturally have a PCI address.17:29
*** vladikr has joined #openstack-nova17:30
*** moshele has joined #openstack-nova17:30
sean-k-mooneyefried: i belive that is because the pci spec requires it17:31
efriedwhose spec?17:32
mriedemcdent: new one for you https://bugs.launchpad.net/nova/+bug/171378617:32
openstackLaunchpad bug 1713786 in OpenStack Compute (nova) "Allocations are not managed properly in all evacuate scenarios" [High,Triaged]17:32
sean-k-mooneyi belive the pci protocol is and ietf or ieee standard17:32
*** sterdnotshaken has quit IRC17:32
*** kbaegis has joined #openstack-nova17:32
efriedDoes the spec require the address to be 32 bits, and denoted as a string in domain:bus:slot.func format?17:33
*** sterdnotshaken has joined #openstack-nova17:33
*** yamahata has quit IRC17:33
*** coreywright has joined #openstack-nova17:33
efriedanyway, spec or no, not all hypervisors address them that way.  And also, we're trying to extend to all devices, not just PCI.17:33
efriedwhich of course makes it even harder to agree on a common set of attributes for them.17:33
*** links has quit IRC17:34
sean-k-mooneywell https://review.openstack.org/#/c/497965/2/specs/queens/approved/pci-by-device-id.rst is just for pci but we should handel other device on other busses too17:35
efriedYup.17:35
*** ssurana has joined #openstack-nova17:35
cdentmriedem: joy. so many twisting paths. i’m flying tomorrow and thursday so won’t personally be able to give that attention, but will try to make sure it is on the global radar17:36
*** slaweq has quit IRC17:36
efriedsean-k-mooney BTW, I'm not advocating for that spec to be implemented at this point.  I think Jay's rebuttal is totally valid and I would love to see a more generic solution.17:37
*** ssurana has left #openstack-nova17:37
*** moshele has quit IRC17:37
efriedThat said, I think if the generic solution as a baseline allowed inventorying, whitelisting, and aliasing via an opaque device ID, that would be a good starting point.17:37
sean-k-mooneyefried: well consideing i will need to track some nics that are not connect to the pci bus and will not have kernel netdevs in the futur more generic sound good to me17:38
efriedMove up to trying to create one-size-fits-all groupings/classes from there.17:38
cdentmriedem: how crazy would it be to have a StuffFailedSomewhereDeleteTheAllocationsAssociatedWithConsumerContainedInThisExeption ?17:38
efriedcdent flake8 failed17:38
cdenteagle eyes efried17:39
efriedHey man, I don't make the rules.  That's 86 characters.17:39
cdentwell crap, that kills that solution then17:40
* cdent sulks17:40
*** shan has joined #openstack-nova17:40
efriedcdent I haven't been following the discussion, but if you're suggesting that an exception object could contain metadata that would tell the catcher how/what to clean up, I think it's a neat idea.17:41
sean-k-mooneycdent: well its python you can always have the __init__ change then name of the class to that at runtime if your really want to punish the people debugging17:41
*** Dave has joined #openstack-nova17:42
*** lpetrut has joined #openstack-nova17:45
*** Donckers has joined #openstack-nova17:45
cdentsean-k-mooney: punishment is the name of the game17:47
cdentefried: yeah, that’s pretty much what I’m suggesting17:48
mriedemwe already have a thing similar to that17:48
mriedemInstanceFaultRollback17:48
mriedemused with a context manager17:49
mriedemit's clear as mud17:49
efriedIf you were using TaskFlow...17:49
sean-k-mooneycdent: if the exception point new how to solve the issue though would it not do it there instead of raising17:49
*** catintheroof has quit IRC17:49
cdentsean-k-mooney: yeah, that is indeed the rub17:49
*** eharney has quit IRC17:49
cdentalso the mudiness17:49
* efried stares slack-jawed at sean-k-mooney 17:49
*** vikrant has quit IRC17:50
sean-k-mooneycdent: it a good idea for thing that need config changes or other operator involement but it could just me in a message filed then that was logged17:50
mriedemdansmith: you know what we need? forced host + unshelve!17:51
*** lpetrut has quit IRC17:51
sean-k-mooneyfor exampel. "this nova needs placement to work. go install it and read the docs"17:51
cdentefried: the broader discussion was in this deeply branching nest of different ways in which allocatins need to be cleaned up was it better to clean up at the end of each branch or somewhere else17:51
* dansmith glares at mriedem 17:51
*** lpetrut has joined #openstack-nova17:51
mriedemo/17:51
mriedem~\o/~17:51
efriedcdent Taskflow.17:52
sean-k-mooneymriedem: so that you can unshvel an instance onto a specific host17:52
mriedemsean-k-mooney: not only that,17:52
mriedembut completely bypass the scheduler17:52
cdentI was hoping that the length of the exception name was a clear indicator that I thought it was mostly crazy pants; however, all the rest of the code is already crazy pants, so who knows17:52
sean-k-mooneymriedem: ah does the same happen with force boot to host e.g. skip scheduler17:53
mriedemno17:53
mriedembut it does with forced host live migration and evacuate,17:53
cdentsee above about punishment17:53
mriedemand takashi is proposing to add the same to cold migrate17:53
mriedemso i'm closing the loop on all the ways we can screw ourselves17:54
sean-k-mooneywell if we added it for both boot and unshivle then it would be consitet for all apis i guess. im assuming it supprot for resise also?17:54
*** edmondsw has joined #openstack-nova17:54
mriedemresize == cold migration17:55
sean-k-mooneyah ok17:55
mriedemsean-k-mooney: i'm sorry but i raised it up to be an asshole17:55
mriedemyou've missed that part17:55
mriedemi don't actually want to do this17:55
mriedem:)17:55
cdentmriedem: help me clear up some gaps in my brains: did we generally know in advance that these edge cases were going to come up or had we thought that something would take care of it. I’m trying to grok if there’s a thing we can close up17:55
*** yangyapeng has quit IRC17:55
mriedemcdent: in advance of what? placement or allowing these changes to the API to force a host and bypass the scheduler for evacuate and live migration?17:56
sean-k-mooneyhaha well at least i helped make it wors by bring up the other usecases17:56
mriedems/placement/claims in the scheduler/17:56
cdentclaims in the scheduler17:56
mriedemcdent: these special move operations were not considered with claims in the scheduler at all from what i can tell17:56
mriedemor probably move operations in general17:56
cdentk, thanks, good datapoint17:56
mriedemas we implemented that all as bug fixes after FF17:56
mriedemi think, whenever we did the double up thing in the scheduler anyway17:57
*** tidwellr has quit IRC17:57
*** slaweq has joined #openstack-nova17:57
*** yamamoto has joined #openstack-nova17:58
mriedemoh sorry the doubled up allocations happened the day of FF17:58
*** yangyapeng has joined #openstack-nova17:58
*** edmondsw_ has joined #openstack-nova17:58
mriedemthat can't be right17:58
mriedemoh yeah no it was FF17:58
mriedem:(17:58
dansmithmriedem: move ops in general yeah17:59
*** edmondsw has quit IRC17:59
*** moshele has joined #openstack-nova17:59
*** edmondsw_ has quit IRC17:59
dansmithI'll be pushing some more stuff up in that series in a bit18:00
*** edmondsw has joined #openstack-nova18:00
dansmithgonna try to break things into really small bits where possible per my usual,18:00
*** tidwellr has joined #openstack-nova18:00
dansmithbut hopefully to make each change a clear and understandable win18:00
mriedemok,18:00
mriedemi'm checking out the unshelve failure flows to see what we might have missed18:00
mriedemand then will start working on the evacuate bug later18:01
*** felipemonteiro has joined #openstack-nova18:01
*** slaweq has quit IRC18:01
*** lucasxu has quit IRC18:01
mriedemi guess i should start a retrospective etherpad for pike before the ptg...18:02
*** sterdnotshaken has quit IRC18:02
mriedemi'm not sure i want to even think about what we did wrong18:02
cdentsome stuff. we admit it. done18:02
mriedemthat's reserved for when i wake up at 2am18:02
*** sterdnotshaken has joined #openstack-nova18:03
*** baoli has quit IRC18:03
*** moshele has quit IRC18:03
*** yamamoto has quit IRC18:04
*** annegentle has joined #openstack-nova18:05
*** yangyapeng has quit IRC18:07
*** edmondsw has quit IRC18:07
*** baoli has joined #openstack-nova18:08
*** yangyapeng has joined #openstack-nova18:09
*** rajathagasthya has quit IRC18:09
*** yamahata has joined #openstack-nova18:09
*** ijw has joined #openstack-nova18:09
mriedemok https://etherpad.openstack.org/p/nova-pike-retrospective18:11
mriedemposting to ML18:11
*** tomtomtom has joined #openstack-nova18:12
*** sterdnotshaken has quit IRC18:12
*** sterdnotshaken has joined #openstack-nova18:12
tomtomtomhello, I'm having trouble launching instances with ephemeral volumes via ceph. anyone got any docs or pointers for such a configuration?18:13
tomtomtomcinder, nova, and ceph "seem" to be creating the volume but nova comes up with "no bootable device" each time.18:13
*** yangyapeng has quit IRC18:14
mriedemtomtomtom: https://docs.openstack.org/nova/latest/user/block-device-mapping.html ?18:15
*** liverpooler has joined #openstack-nova18:15
sean-k-mooneytomtomtom: does normal booting work wtih ceph backed root device or only fails with ephemeral disk18:18
mriedemtomtomtom: can you clarify what you mean by 'ephemeral volumes'?18:19
mriedemare you actually booting from volume?18:19
mriedemand you consider it ephemeral because delete_on_termination=True?18:19
sean-k-mooneyalternitvly do you mean you have allocated ephmeral storage in the flavor and have confiuged nova to back all vm storage with ceph volumes18:21
*** penick has quit IRC18:21
*** rajathagasthya has joined #openstack-nova18:22
*** sterdnotshaken has quit IRC18:22
*** catintheroof has joined #openstack-nova18:23
*** edmondsw has joined #openstack-nova18:23
*** adisky__ has quit IRC18:27
mriedemlooks like we're ok wrt allocations during unshelve18:27
mriedemconductor calls scheduler to pick a host, creates the allocations, and casts to compute18:27
*** sridharg has quit IRC18:27
mriedemwhere the instance claim happens18:27
*** sridharg has joined #openstack-nova18:27
* cdent makes another sacrifice18:27
mriedemno retries18:27
*** edmondsw has quit IRC18:27
mriedemMAYBE WE SHOULD BUILD RETRY LOGIC INTO UNSHELVE18:28
mriedemactually, :)18:29
* cdent sacrifices mriedem 18:29
sean-k-mooneycdent: lol18:29
sean-k-mooneycdent: you realise that that would make it your problem to fix18:29
*** sterdnotshaken has joined #openstack-nova18:30
mriedemwe make the claim, and then try to spawn the instance, if that fails we unset the instance.host/node values,18:30
mriedembut i'm not sure that we remove the allocations18:30
cdentb@ll$18:31
mriedemyeah we don't18:32
mriedemwe'd abort the claim on the exit of this https://github.com/openstack/nova/blob/16.0.0.0rc2/nova/compute/manager.py#L448518:33
mriedemhttps://github.com/openstack/nova/blob/16.0.0.0rc2/nova/compute/resource_tracker.py#L41418:33
cdentmriedem: maybe you need to reset your scanning algorithm: look for where we do, because we started from the point of not thinking about it, thus...18:33
mriedemthat method, by default, sets has_ocata_computes=False18:33
mriedemhttps://github.com/openstack/nova/blob/16.0.0.0rc2/nova/compute/resource_tracker.py#L101218:33
*** jaypipes has joined #openstack-nova18:33
mriedemwhich means we won't fix the allocations https://github.com/openstack/nova/blob/16.0.0.0rc2/nova/compute/resource_tracker.py#L104518:33
mriedemcdent: we just always relied on the periodic to heal things18:34
mriedemwhich is the same code that would have done this before ^18:34
cdenthow about we just put it back, for now?18:34
mriedembecause we want to remove it so the RT isn't trampling over thigns18:35
mriedemit just really means that we have to be very explicity about dealing with allocations everywhere18:35
mriedembut, that's probably for the best in the long run18:35
cdentthen in that case there’s no reason to express surprise that we aren’t handling things, yeah?18:36
mriedemcorrect18:36
*** lpetrut has quit IRC18:37
*** shan has quit IRC18:37
* cdent wonders if there’s a tidy validation strategy we can come up with18:39
cdentthe functional tests are great, but fairly heavy18:40
cdents/heavy/cumbersome to create18:40
*** edmondsw has joined #openstack-nova18:41
*** slaweq has joined #openstack-nova18:42
mriedemhttps://bugs.launchpad.net/nova/+bug/171379618:43
openstackLaunchpad bug 1713796 in OpenStack Compute (nova) "Failed unshelve does not remove allocations from destination node" [High,Triaged]18:43
mriedemthere are at least 2 ways unshelve can fail there which we don't cleanup the allocations18:43
*** slaweq has quit IRC18:44
mriedemmaybe we need an undo_allocations decorator for several methods in the compute manager18:45
*** edmondsw has quit IRC18:45
*** sterdnotshaken1 has joined #openstack-nova18:46
cdentare the exit conditions workable for a decorator (or, to put it another way, how is that different from the two other ideas above?)18:46
mriedemit could be messy in a decorator, probably lots of conditional logic based on the operation being performed18:47
mriedemwhich is based on the task_state18:47
mriedemso i'm not going to bother thinking about that for now18:48
mriedemregarding the functional tests, i think we need those regardless18:49
*** vladikr has quit IRC18:49
*** sterdnotshaken has quit IRC18:49
cdentoh, yeah, I wasn’t saying we should get rid of them18:49
mriedemsince we didn't have much in the way of negative functional scenario tests that were asserting allocations18:49
cdentbut I was wondering if something like a shell script that did curl based validations could be used to confirm where the bugs are18:49
*** eharney has joined #openstack-nova18:49
cdentnova < some command> […] curl to verify some allocation18:50
*** moshele has joined #openstack-nova18:54
sean-k-mooneycdent: why curl vs adding support for placement to the openstack client18:54
sean-k-mooneywe will need that evenutally anyway18:54
sean-k-mooneywe will also need it for traits18:54
mriedemsean-k-mooney: there are patches for an osc placement plugin18:54
mriedemhttps://review.openstack.org/#/q/project:openstack/osc-placement18:55
mriedemcdent: yeah i thought about something like that the other night,18:55
mriedemlike, we have a post test hook into our CI environments to do things18:55
sean-k-mooneyoh cool then ya that would be easier to use in test code then curl as it handels gettin ghte keysone auth tokens for you which is a pain18:55
mriedemcdent: assuming we cleaned everything up properly, the nodes shouldn't have any allocations against them at the end of the job18:55
mriedemi think we run the archive instances stuff through that in our jobs somewhere18:56
*** gbarros has joined #openstack-nova18:56
sean-k-mooneymriedem: i think we do someting similar in our thridpart collecd ci18:57
mriedemdansmith: don't we run archive_deleted_rows in our ci somewhere?18:57
mriedemyeah the post_test_hook.sh, i bet project-config is busted again18:58
mriedemnova/tools/hooks/post_test_hook.sh18:58
*** moshele has quit IRC18:59
sean-k-mooneysoory got confued ignore the collectd comment but what i was goning to say is im not sure if post_test_hook.sh runs after the logs are upladed or not18:59
*** lpetrut has joined #openstack-nova18:59
mriedemoh it's only in the nova-next job18:59
*** yamamoto has joined #openstack-nova18:59
*** inara has quit IRC18:59
mriedemkablam! http://logs.openstack.org/61/496861/1/check/gate-tempest-dsvm-neutron-nova-next-full-ubuntu-xenial-nv/572ceed/logs/devstack-gate-post_test_hook.txt.gz19:00
cdentsean-k-mooney: I was thinking curl because of something quick and dirty that wouldn’t necessarily be a permanently running test, rather a diagnostic tool for nowish19:00
mriedemcdent: so it would be cool to write something that queries for allocations in the post test hook at the end of a job to see if they are 0 or not19:00
cdentalso, what’s hard about getting a token?19:00
mriedemanyway, i'll post the idea to the ML and people can think about it19:00
sean-k-mooneycdent: more you just need to do that first befor just calling curl and add it to the correct header19:01
cdentmriedem: would we’d need to track instance ids throughout the test or can we simply get all instances, including deleted ones?19:01
*** cpusmith has joined #openstack-nova19:01
sean-k-mooneycdent: if you want something quick just run a mysql command directly against the db19:01
sean-k-mooneyif there are still allocations in the output you know something is messed up19:02
*** inara has joined #openstack-nova19:02
cdentsean-k-mooney: I tend do to a lot of export TOKEN=$(openstack token issue …); curl blah19:02
*** kfarr has quit IRC19:02
*** gszasz has quit IRC19:02
sean-k-mooneycdent: huh i didnt know you could do openstack token issue ...19:02
cdentbut yeah, a db query would be fine too, but…boring :)19:02
sean-k-mooneythat makes things alost eaisier19:02
*** vladikr has joined #openstack-nova19:02
*** edmondsw has joined #openstack-nova19:03
cdentalso it would mean that we are assuming that placement will always have a db backend ;)19:03
sean-k-mooneyany time i have used curl like that i have manually used curl to get the token by calling keysone myself which is a pain19:03
cdentouch. yeah. that is painful19:03
*** mnestratov has quit IRC19:03
*** ijw has quit IRC19:03
*** edmondsw has quit IRC19:04
*** ijw has joined #openstack-nova19:04
*** ssmith has quit IRC19:04
*** cpusmith_ has joined #openstack-nova19:04
*** lpetrut has quit IRC19:05
sean-k-mooneycdent: and yes i am assiming a db backend. honestly i spend enough time getting our ci test to stop going to the db directly and use the api instead so ya curl would be better if its going to last long enough for db assumtions to expire19:05
*** yamamoto has quit IRC19:05
sean-k-mooneyanyway time for food, see ye tomorow19:06
*** edmondsw has joined #openstack-nova19:07
mriedemcdent: i think we'd have to account for instances that aren't yet deleted19:07
mriedemcdent: but, the post_test_hook stuff doesn't fail the job i don't think, so this would just be an early report to start19:07
mriedemso we can kick ideas around19:07
* cdent nods19:07
*** cpusmith has quit IRC19:08
*** ijw has quit IRC19:08
*** edmondsw has quit IRC19:08
* cdent will read code on the plane19:09
*** yangyapeng has joined #openstack-nova19:10
*** moshele has joined #openstack-nova19:12
*** cdent has quit IRC19:13
*** yangyapeng has quit IRC19:14
*** slaweq has joined #openstack-nova19:15
*** armax has quit IRC19:20
mriedemsdague: do you just want the commit message updated for this? https://review.openstack.org/#/c/457636/6/lib/placement19:20
sdaguemriedem: ok, this is long enough ago, I'm going to have to regain context19:21
sdagueI think the question is... do you always want that library installed?19:22
sdaguebecause, that won't install it in the normal case19:22
sdagueonly when you specify it as LIBS_FROM_GIT19:22
mriedemyes, we are going to want it in all cases i think19:22
sdaguewhich won't give you the release version, only the master version19:22
mriedemright,19:22
sdagueok, you you'll need an else19:23
mriedemi think roman did that because he was working on getting an osc placement functional test job to run from a proposed change19:23
*** cleong has quit IRC19:23
sdaguethat does a pip_install osc-placement19:23
*** penick has joined #openstack-nova19:23
mriedemok. shouldn't the LIBS_FROM_GIT stuff be handled generically?19:23
mriedemlike, the job in project-config defines that i thought19:23
mriedemand you had me do something like this for os-traits19:24
mriedemto follow how the oslo libs are done19:24
mriedemyeah, devstack/lib/libraries19:24
mriedemhttps://github.com/openstack-dev/devstack/commit/aefc926cd45b2dc74d98f89e3a3b4cc92f2090ff19:25
mriedemso i guess i can push something to do that for osc-placement19:25
*** ijw has joined #openstack-nova19:27
*** yamamoto has joined #openstack-nova19:27
*** catintheroof has quit IRC19:30
*** abalutoiu_ has joined #openstack-nova19:30
*** ijw has quit IRC19:32
mriedemsdague: i should use pip_install_gr right?19:33
*** gcb has quit IRC19:33
sdaguemriedem: yeh, though I think it all goes through that19:33
*** lpetrut has joined #openstack-nova19:33
sdaguemriedem: most of the time the library is dragged in from requirements somewhere19:33
mriedempip_install_gr wraps pip_install19:34
sdaguebut in the edge cases, like this19:34
*** abalutoiu has quit IRC19:34
sdaguewhere it's not in anyone's requirements list19:34
sdagueyou need the fallback in code19:34
*** abalutoiu_ has quit IRC19:34
mriedemhmm, i guess i can't use pip_install_gr yet since osc-placement isn't in global-requirements19:34
sdaguemriedem: we should fix that19:34
*** ijw has joined #openstack-nova19:34
sdaguewhy isn't it in there yet?19:34
mriedemnothing is using it?19:35
sdagueah, ok, yeh chicken and egg19:35
*** dave-mccowan has joined #openstack-nova19:35
sdagueis there a release yet?19:35
*** sridharg has quit IRC19:35
*** abalutoiu has joined #openstack-nova19:35
*** catintheroof has joined #openstack-nova19:36
*** chyka has quit IRC19:38
*** gixxer1k1k has joined #openstack-nova19:38
*** ijw has quit IRC19:39
*** gixxer1k has quit IRC19:40
*** pchavva has quit IRC19:41
*** abalutoiu_ has joined #openstack-nova19:41
*** awaugama has quit IRC19:41
*** egonzalez has joined #openstack-nova19:42
*** abalutoiu has quit IRC19:46
*** xyang1 has joined #openstack-nova19:47
*** abalutoiu__ has joined #openstack-nova19:53
*** edmondsw has joined #openstack-nova19:55
*** edmondsw has quit IRC19:56
*** abalutoiu_ has quit IRC19:57
*** belmoreira has joined #openstack-nova20:04
*** edmondsw has joined #openstack-nova20:08
*** edmondsw has quit IRC20:13
*** felipemonteiro has quit IRC20:16
*** felipemonteiro has joined #openstack-nova20:16
*** felipemonteiro_ has joined #openstack-nova20:17
mriedemosc-placement release is proposed20:19
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Handle ComputeHostNotFound on startup  https://review.openstack.org/48968320:19
mriedemhttps://review.openstack.org/49892420:19
*** felipemonteiro has quit IRC20:21
*** chyka has joined #openstack-nova20:21
*** priteau has quit IRC20:24
*** annegentle has quit IRC20:26
*** ijw has joined #openstack-nova20:28
*** annegentle has joined #openstack-nova20:30
openstackgerritMatt Riedemann proposed openstack/nova master: Fix nova assisted volume snapshots  https://review.openstack.org/49884520:31
*** yangyapeng has joined #openstack-nova20:34
mriedem^ needs a +W to fix a multi-cell issue, which will go back to pike20:35
*** yangyapeng has quit IRC20:38
efriedjaypipes Is there any reason update_device_info() [names suck. feel free to suggest a better one.] needs to be separate from just get_inventory()?20:38
*** lyan has quit IRC20:41
*** armax has joined #openstack-nova20:42
*** lyan has joined #openstack-nova20:42
*** crushil has quit IRC20:43
*** cpusmith_ has quit IRC20:43
*** tidwellr has quit IRC20:48
*** moshele has quit IRC20:52
*** burt has quit IRC20:53
*** crushil has joined #openstack-nova20:53
*** crushil has quit IRC20:54
*** abalutoiu__ has quit IRC20:54
*** annegentle has quit IRC20:54
*** crushil has joined #openstack-nova20:54
dansmithmriedem: we hit that path because we literally have nothing more than the volume id to go on?20:57
*** annegentle has joined #openstack-nova20:58
mriedemdansmith: yup, there is no instance in that API20:59
dansmiththat sucks20:59
mriedemthis is the fun one that cinder calls back for FS-style backends20:59
mriedemlike glusterfs and nfs20:59
*** penick has quit IRC21:00
mriedemhttps://review.openstack.org/#/c/498510/21:00
mriedemoops21:00
mriedemhttps://developer.openstack.org/api-ref/compute/#create-assisted-volume-snapshots21:00
mriedemmaybe that create_info dict has the instance id in it21:01
mriedemthe api ref says that's optional and arbitrary21:01
*** tidwellr has joined #openstack-nova21:02
mriedemlooking at the cinder code, they don't send it either21:02
*** crushil has quit IRC21:02
dansmithawesome21:02
* dansmith hugs us21:02
mriedemhey, thank your glusterfs red hat brethren21:02
dansmithheh21:02
mriedemfrom like, 4 years ago21:02
mriedemso this only really impacts like SAP which uses NFS for everything21:03
mriedem:)21:03
*** yamamoto has quit IRC21:04
*** slaweq has quit IRC21:04
*** ijw has quit IRC21:05
*** gouthamr has quit IRC21:05
openstackgerritMichael Still proposed openstack/nova master: Move lvm handling to privsep.  https://review.openstack.org/49551621:07
openstackgerritMichael Still proposed openstack/nova master: Move xend existence probes to privsep.  https://review.openstack.org/49553821:07
openstackgerritMichael Still proposed openstack/nova master: Move shred to privsep.  https://review.openstack.org/49553721:07
openstackgerritMichael Still proposed openstack/nova master: Cleanup mount / umount and associated rmdir calls  https://review.openstack.org/49442321:07
openstackgerritMichael Still proposed openstack/nova master: WIP / Aspirational: we don't need rootwrap any more.  https://review.openstack.org/49554221:07
openstackgerritMichael Still proposed openstack/nova master: Move loopback setup and removal to privsep.  https://review.openstack.org/49566421:07
openstackgerritMichael Still proposed openstack/nova master: Move libvirt usages of chown to privsep.  https://review.openstack.org/47197221:07
openstackgerritMichael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree()  https://review.openstack.org/49232621:07
openstackgerritMichael Still proposed openstack/nova master: Move the idmapshift binary into privsep.  https://review.openstack.org/49554121:07
openstackgerritMichael Still proposed openstack/nova master: Move ploop commands to privsep.  https://review.openstack.org/49232521:07
openstackgerritMichael Still proposed openstack/nova master: Read from console ptys using privsep.  https://review.openstack.org/48948621:07
openstackgerritMichael Still proposed openstack/nova master: Avoid chowning console logs in libvirt  https://review.openstack.org/47222921:08
openstackgerritMichael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself.  https://review.openstack.org/45916621:08
openstackgerritMichael Still proposed openstack/nova master: Move execs of touch to privsep.  https://review.openstack.org/48919021:08
openstackgerritMichael Still proposed openstack/nova master: Replace dd with shred for zeroing lvm volumes.  https://review.openstack.org/49553221:08
openstackgerritMichael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep.  https://review.openstack.org/49073721:08
openstackgerritMichael Still proposed openstack/nova master: Move execs of tee to privsep.  https://review.openstack.org/48943821:08
mikalmriedem: that's the fix to those unit tests you wanted in the dd --> shred patch21:08
*** yamamoto has joined #openstack-nova21:08
mriedemaye21:08
*** artom has quit IRC21:13
*** mnestratov has joined #openstack-nova21:13
esberglumriedem: For the PowerVM Driver work, would you prefer that we create a fresh blueprint or just retarget the existing one?21:13
mriedemyou can't retarget a completed blueprint21:14
mriedemso it has to be new21:14
esbergluokay tx. Will do21:14
*** sterdnotshaken has joined #openstack-nova21:15
mriedemomg it's live https://developer.openstack.org/api-ref/placement/21:16
mriedemfinally21:16
*** sterdnotshaken1 has quit IRC21:18
*** tidwellr has quit IRC21:20
*** vladikr has quit IRC21:21
openstackgerritDan Smith proposed openstack/nova master: Move allocation manipulation out of drop_move_claim()  https://review.openstack.org/49894721:22
*** thorst_afk has quit IRC21:22
openstackgerritDan Smith proposed openstack/nova master: Make allocation cleanup honor new by-migration rules  https://review.openstack.org/49894821:22
openstackgerritDan Smith proposed openstack/nova master: Revert allocations by migration uuid  https://review.openstack.org/49894921:22
openstackgerritDan Smith proposed openstack/nova master: WIP: Pre-create migration object  https://review.openstack.org/49895021:22
mriedemdansmith: i'm going through the spec for that right now btw21:23
*** thorst_afk has joined #openstack-nova21:23
dansmithmriedem: okay I'm about to head out for a bit so take your time21:23
*** sterdnotshaken1 has joined #openstack-nova21:25
*** gixxer1k1k has quit IRC21:26
mriedemok posted21:27
*** thorst_afk has quit IRC21:27
*** sterdnotshaken has quit IRC21:28
*** crushil has joined #openstack-nova21:28
*** yamamoto has quit IRC21:30
*** yamamoto has joined #openstack-nova21:32
*** yamamoto has quit IRC21:32
*** catintheroof has quit IRC21:34
*** ijw has joined #openstack-nova21:40
*** crushil has quit IRC21:41
*** artom has joined #openstack-nova21:42
*** artom has quit IRC21:42
*** artom has joined #openstack-nova21:42
*** moshele has joined #openstack-nova21:43
*** esberglu has quit IRC21:43
*** esberglu has joined #openstack-nova21:44
*** slaweq has joined #openstack-nova21:44
*** annegentle has quit IRC21:45
*** esberglu has quit IRC21:48
*** belmoreira has quit IRC21:49
*** slaweq has quit IRC21:49
*** thorst_afk has joined #openstack-nova21:49
*** Donckers has quit IRC21:49
*** Donckers has joined #openstack-nova21:49
*** Donckers has quit IRC21:50
*** ijw has quit IRC21:51
*** crushil has joined #openstack-nova21:52
*** sterdnotshaken has joined #openstack-nova21:53
*** thorst_afk has quit IRC21:53
*** felipemonteiro_ has quit IRC21:53
*** lpetrut has quit IRC21:55
*** sterdnotshaken1 has quit IRC21:56
*** esberglu has joined #openstack-nova21:57
*** marst has joined #openstack-nova21:58
*** baoli has quit IRC22:01
*** marst_ has quit IRC22:01
*** baoli has joined #openstack-nova22:01
*** esberglu has quit IRC22:01
*** edmondsw has joined #openstack-nova22:02
*** edmondsw has quit IRC22:04
*** edmondsw has joined #openstack-nova22:04
*** sdague has quit IRC22:04
*** cfriesen has quit IRC22:06
*** xyang1 has quit IRC22:06
*** cfriesen has joined #openstack-nova22:07
*** esberglu has joined #openstack-nova22:07
*** egonzalez has quit IRC22:11
*** cfriesen has quit IRC22:12
*** itlinux has quit IRC22:14
*** ijw has joined #openstack-nova22:18
*** cfriesen has joined #openstack-nova22:18
*** lbragstad has quit IRC22:20
*** edmondsw has quit IRC22:23
*** ijw has quit IRC22:23
*** Apoorva has joined #openstack-nova22:24
*** cfriesen_ has joined #openstack-nova22:24
*** cfriesen has quit IRC22:24
*** annegentle has joined #openstack-nova22:25
*** esberglu has quit IRC22:27
*** esberglu has joined #openstack-nova22:27
*** esberglu_ has joined #openstack-nova22:28
*** sterdnotshaken has quit IRC22:29
*** sterdnotshaken1 has joined #openstack-nova22:29
*** esberglu has quit IRC22:31
*** baoli has quit IRC22:33
*** ijw has joined #openstack-nova22:34
*** yangyapeng has joined #openstack-nova22:34
*** esberglu_ has quit IRC22:35
*** lyan has quit IRC22:37
*** gouthamr has joined #openstack-nova22:38
*** ijw has quit IRC22:38
*** yangyapeng has quit IRC22:39
*** edmondsw has joined #openstack-nova22:41
*** Apoorva has quit IRC22:42
*** Apoorva has joined #openstack-nova22:42
*** edmondsw has quit IRC22:45
*** Apoorva has quit IRC22:47
openstackgerritMatt Riedemann proposed openstack/nova master: doc: link to placement api-ref and history docs from main index  https://review.openstack.org/49897722:49
*** crushil has quit IRC22:56
*** trinaths has joined #openstack-nova22:58
*** chyka has quit IRC22:58
*** edmondsw has joined #openstack-nova22:58
*** chyka has joined #openstack-nova22:59
openstackgerritmelanie witt proposed openstack/nova master: Save updated libvirt domain XML after swapping volume  https://review.openstack.org/49898323:00
*** annegentle has quit IRC23:02
*** crushil has joined #openstack-nova23:02
*** ijw has joined #openstack-nova23:02
*** edmondsw has quit IRC23:03
mriedemis swap volume ever going to be complete?23:04
melwittwhat you mean23:04
mriedemwhat do YOU mean23:06
melwittI mean what do you mean by if it's going to be complete?23:07
mriedemi'm joking that it's a never ending source of bugs23:07
mriedemafter all these years23:07
melwittoh yes23:08
*** ijw has quit IRC23:09
*** Tom has joined #openstack-nova23:10
*** cfriesen_ has quit IRC23:11
melwittto your question, I tested it with a guest with persistent domain and it worked. delete_configuration is undefining the domain, which seems like a persistent config only thing23:12
melwittI changed it to read the live config, which AFAIK is completely separate from the persistent config. but I'm going to sanity-check with mdbooth or lyarwood to be sure23:13
*** Tom has quit IRC23:15
melwittergh, but this is about uefi support. which I didn't have in my repro env. so yeah I need to look into that23:15
*** baoli has joined #openstack-nova23:16
*** Tom has joined #openstack-nova23:16
*** cfriesen_ has joined #openstack-nova23:17
*** vladikr has joined #openstack-nova23:17
melwittor wait, the uefi is just a flag for undefining the domain. so that call undefines the domain (removes the persistent config IIUC) whether there's uefi support or not23:17
*** ijw has joined #openstack-nova23:17
*** Tom has quit IRC23:21
*** mnestratov has quit IRC23:22
openstackgerritSean McCully proposed openstack/nova master: iso8601.is8601.Utc No Longer Exists  https://review.openstack.org/49828723:23
*** marst has quit IRC23:24
*** annegentle has joined #openstack-nova23:25
*** baoli has quit IRC23:25
*** yamamoto has joined #openstack-nova23:28
openstackgerritSean McCully proposed openstack/nova master: iso8601.is8601.Utc No Longer Exists  https://review.openstack.org/49828723:30
*** claudiub|2 has quit IRC23:33
*** yamahata has quit IRC23:34
*** yamahata has joined #openstack-nova23:40
*** jmlowe has joined #openstack-nova23:43
*** annegentle has quit IRC23:43
mikalmriedem: where did you want on nova-net? Is it going to heaven?23:46
*** yamamoto has quit IRC23:46
mikals/want/land/23:47
*** kbaegis has quit IRC23:59
*** moshele has quit IRC23:59

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