Friday, 2017-12-01

*** felipemonteiro has quit IRC00:01
*** rcernin has quit IRC00:02
*** rcernin has joined #openstack-nova00:02
*** chyka has quit IRC00:03
*** chyka_ has quit IRC00:04
*** yingjun has joined #openstack-nova00:06
*** rcernin has quit IRC00:07
*** rcernin has joined #openstack-nova00:07
*** penick_ has quit IRC00:08
*** penick has joined #openstack-nova00:08
*** thorst has joined #openstack-nova00:11
*** david-lyle has quit IRC00:15
*** penick has quit IRC00:15
*** thorst has quit IRC00:15
*** sdague has quit IRC00:22
*** jmlowe has quit IRC00:23
*** tetsuro has joined #openstack-nova00:23
openstackgerritMatt Riedemann proposed openstack/nova master: Fix 'force' parameter in os-quota-sets PUT schema  https://review.openstack.org/52319400:24
openstackgerritMatt Riedemann proposed openstack/nova master: Deprecate file injection  https://review.openstack.org/52202700:24
mriedemalex_xu: is this kind of what you were looking for? https://review.openstack.org/#/c/523194/2/nova/api/openstack/compute/rest_api_version_history.rst00:25
*** TuanLA has joined #openstack-nova00:25
*** david-lyle has joined #openstack-nova00:29
*** mlavalle has quit IRC00:29
*** amodi has joined #openstack-nova00:32
*** felipemonteiro_ has quit IRC00:34
tetsurojay: are you there?00:34
*** david-lyle has quit IRC00:34
*** flwang has quit IRC00:34
*** hongbin has quit IRC00:36
*** gbarros has quit IRC00:42
*** salv-orlando has joined #openstack-nova00:42
*** Tom-Tom has joined #openstack-nova00:45
*** hieulq has quit IRC00:45
*** hieulq has joined #openstack-nova00:46
*** salv-orlando has quit IRC00:46
*** READ10 has quit IRC00:50
*** Ankit has joined #openstack-nova00:52
*** armax has quit IRC00:54
*** ankit_ has joined #openstack-nova00:55
*** thorst has joined #openstack-nova00:57
*** Tom-Tom has quit IRC00:58
*** Tom-Tom has joined #openstack-nova00:58
*** phuongnh has joined #openstack-nova01:00
*** thorst has quit IRC01:02
*** Tom-Tom has quit IRC01:02
*** takashin has joined #openstack-nova01:04
*** yassine has joined #openstack-nova01:06
*** yassine is now known as Guest6787101:07
*** fragatina has quit IRC01:08
*** fragatina has joined #openstack-nova01:09
*** yangyapeng has joined #openstack-nova01:09
*** fragatin_ has joined #openstack-nova01:11
*** fragati__ has joined #openstack-nova01:12
*** fragati__ has quit IRC01:12
*** fragati__ has joined #openstack-nova01:12
*** fragati__ has quit IRC01:12
*** fragatin_ has quit IRC01:12
*** fragatina has quit IRC01:12
*** esberglu has quit IRC01:13
*** fragatina has joined #openstack-nova01:13
*** hieulq has quit IRC01:14
*** ijw has joined #openstack-nova01:14
*** ijw has quit IRC01:15
*** ijw has joined #openstack-nova01:15
*** Apoorva_ has joined #openstack-nova01:17
*** fragatina has quit IRC01:17
*** tbachman has quit IRC01:17
*** suresh12 has joined #openstack-nova01:19
openstackgerritZhenyu Zheng proposed openstack/nova master: Modify _poll_shelved_instances periodic task call _shelve_offload_instance()  https://review.openstack.org/52404701:19
*** Apoorva has quit IRC01:21
*** Apoorva_ has quit IRC01:22
*** jichen has joined #openstack-nova01:22
*** fragatina has joined #openstack-nova01:26
*** ijw has quit IRC01:27
*** qsyqian has joined #openstack-nova01:29
*** thorst has joined #openstack-nova01:30
*** fragatina has quit IRC01:30
*** thorst has quit IRC01:35
*** lyan has joined #openstack-nova01:35
*** zhurong has joined #openstack-nova01:36
openstackgerritMerged openstack/nova master: Hyper-V: fix live migration with CSVs  https://review.openstack.org/38044801:39
openstackgerritMerged openstack/nova master: Remove unused global variables  https://review.openstack.org/52412601:40
openstackgerritMerged openstack/nova master: Vzstorage: synchronize volume connect  https://review.openstack.org/50570801:40
openstackgerritMerged openstack/nova master: Fix incorrect known vcpuset when CPUPinningUnknown raised  https://review.openstack.org/51759201:41
*** Tom-Tom has joined #openstack-nova01:41
openstackgerritMerged openstack/nova master: Use NoDBTestCase for powervm driver tests  https://review.openstack.org/52011801:41
openstackgerritMerged openstack/nova master: [placement] Fix format in placement API ref  https://review.openstack.org/52148301:42
openstackgerritMerged openstack/nova master: [placement] Fix parameter order in placement API ref  https://review.openstack.org/51623301:42
openstackgerritMerged openstack/nova master: Fix TypeError of _get_project_id when project_id is None  https://review.openstack.org/52210001:42
openstackgerritClaudiu Belu proposed openstack/nova master: tests: fixes mock autospec usage  https://review.openstack.org/44750501:43
openstackgerritClaudiu Belu proposed openstack/nova master: tests: autospecs all the mock.patch usages  https://review.openstack.org/47077501:43
*** salv-orlando has joined #openstack-nova01:43
openstackgerritMerged openstack/nova master: VMware: Handle concurrent registrations of the VC extension  https://review.openstack.org/48466701:43
openstackgerritMerged openstack/nova master: [placement] Add 'CUSTOM_' prefix description in API ref  https://review.openstack.org/52149501:44
openstackgerritMerged openstack/nova master: Update description of Rebuild in server_concepts.rst  https://review.openstack.org/52208601:44
*** Tom-Tom has quit IRC01:46
*** lei-zh has joined #openstack-nova01:47
*** salv-orlando has quit IRC01:48
*** esberglu has joined #openstack-nova01:51
*** esberglu has quit IRC01:55
*** bbbbzhao_ has joined #openstack-nova01:56
*** thorst has joined #openstack-nova02:01
*** Tom-Tom has joined #openstack-nova02:05
*** thorst has quit IRC02:07
*** gmann_afk is now known as gmann02:07
gmanngibi: thanks02:08
*** lei-zh1 has joined #openstack-nova02:10
gmannmriedem: on query schema BP - https://blueprints.launchpad.net/nova/+spec/json-schema-validation-for-query-param02:10
*** liangy has joined #openstack-nova02:10
gmannmriedem: only API left is cell (v1), which anyway going to be deprecated and removed so i am thinking  to skip the schema for this02:11
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add x-openstack-request-id in API ref  https://review.openstack.org/52300702:11
*** AlexeyAbashkin has joined #openstack-nova02:12
*** vivsoni_ has joined #openstack-nova02:12
*** Ankit has quit IRC02:13
gmannmriedem: else this BP is completed.02:14
*** rcernin_ has joined #openstack-nova02:14
*** EmilienM_ has joined #openstack-nova02:15
*** thorst has joined #openstack-nova02:15
*** trungnv has joined #openstack-nova02:16
*** thorst has quit IRC02:16
*** slunkad_ has joined #openstack-nova02:16
*** AlexeyAbashkin has quit IRC02:16
*** Anticime1 has joined #openstack-nova02:18
*** hshiina has joined #openstack-nova02:18
gmannmriedem: we did for deprecated APIs but cell one is not just deprecation case but going to be removed that why we can skip this ? or its easy schema for cell i can implement if you want02:19
*** lyan has quit IRC02:20
*** annp has joined #openstack-nova02:20
*** rcernin has quit IRC02:21
*** lei-zh has quit IRC02:21
*** vivsoni has quit IRC02:21
*** Dinesh__Bhor has joined #openstack-nova02:21
*** Eran_Kuris_ has joined #openstack-nova02:21
*** zhurong has quit IRC02:22
*** qsyqian has quit IRC02:22
*** yingjun has quit IRC02:22
*** eharney has quit IRC02:22
*** coreywright has quit IRC02:22
*** markvoelker has quit IRC02:22
*** EmilienM has quit IRC02:22
*** Eran_Kuris has quit IRC02:22
*** Dinesh_Bhor has quit IRC02:22
*** edleafe has quit IRC02:22
*** Nil_ has quit IRC02:22
*** Anticimex has quit IRC02:22
*** zigo has quit IRC02:22
*** slunkad has quit IRC02:22
*** EmilienM_ is now known as EmilienM02:22
*** EmilienM has quit IRC02:22
*** EmilienM has joined #openstack-nova02:22
*** qsyqian has joined #openstack-nova02:23
*** egarbade has quit IRC02:23
*** dr_gogeta86_ has quit IRC02:24
*** toabctl has quit IRC02:25
*** egarbade has joined #openstack-nova02:26
*** dr_gogeta86 has joined #openstack-nova02:27
*** dr_gogeta86 has quit IRC02:27
*** dr_gogeta86 has joined #openstack-nova02:27
*** zigo has joined #openstack-nova02:27
*** edleafe has joined #openstack-nova02:27
*** coreywright has joined #openstack-nova02:27
*** markvoelker has joined #openstack-nova02:27
*** Nil_ has joined #openstack-nova02:27
*** toabctl has joined #openstack-nova02:27
*** cheneydc has joined #openstack-nova02:27
openstackgerritJack Lu proposed openstack/nova master: Update supported transports for iscsi connector  https://review.openstack.org/52444302:28
*** zigo is now known as Guest1326802:29
*** suresh12 has quit IRC02:29
*** nicolasbock has quit IRC02:30
*** dave-mccowan has joined #openstack-nova02:30
*** mriedem has quit IRC02:33
*** suresh12 has joined #openstack-nova02:33
*** suresh12 has quit IRC02:38
*** claudiub has quit IRC02:40
*** jmlowe has joined #openstack-nova02:42
*** amodi has quit IRC02:43
*** thorst has joined #openstack-nova02:44
*** salv-orlando has joined #openstack-nova02:44
*** thorst has quit IRC02:44
*** salv-orlando has quit IRC02:49
*** dave-mccowan has quit IRC02:51
openstackgerritMerged openstack/nova master: Remove old-style quotas code  https://review.openstack.org/51168902:53
*** itlinux has joined #openstack-nova02:53
*** liangy has quit IRC02:55
alex_xujaypipes: sir, good evening, do you want to unlock this patch https://review.openstack.org/#/c/517119/?03:01
alex_xujaypipes: or anything I can help to unlock that patch03:01
*** lei-zh1 has quit IRC03:07
*** lei-zh has joined #openstack-nova03:08
*** suresh12 has joined #openstack-nova03:08
*** yamahata has quit IRC03:09
*** suresh12 has quit IRC03:13
*** masber has joined #openstack-nova03:14
*** tetsuro has quit IRC03:17
*** felipemonteiro has joined #openstack-nova03:23
*** cheneydc has quit IRC03:24
*** thorst has joined #openstack-nova03:25
*** felipemonteiro_ has joined #openstack-nova03:26
*** takashin has quit IRC03:28
*** felipemonteiro has quit IRC03:29
*** thorst has quit IRC03:29
*** felipemonteiro_ has quit IRC03:31
*** abhishekk has joined #openstack-nova03:34
*** gbarros has joined #openstack-nova03:41
*** salv-orlando has joined #openstack-nova03:45
*** salv-orlando has quit IRC03:49
*** yingjun has joined #openstack-nova03:50
*** rcernin has joined #openstack-nova03:50
*** rcernin_ has quit IRC03:51
*** namnh has joined #openstack-nova03:52
*** links has joined #openstack-nova03:55
*** udesale has joined #openstack-nova03:55
*** kalyan has joined #openstack-nova03:57
*** coreywright has quit IRC03:57
efriedalex_xu I believe jaypipes said he was working on it locally.03:58
efriedalex_xu Never mind - he rebased it, but I think he thought he was done at that point :)04:00
openstackgerritjichenjc proposed openstack/nova master: Refactor placement version check  https://review.openstack.org/51249704:02
*** thorst has joined #openstack-nova04:04
*** fragatina has joined #openstack-nova04:05
*** fragatina has quit IRC04:05
*** fragatina has joined #openstack-nova04:06
*** trungnv has quit IRC04:06
*** fragatina has quit IRC04:07
*** fragatina has joined #openstack-nova04:07
openstackgerritjichenjc proposed openstack/nova master: [Placement] Add create inventories doc for placement  https://review.openstack.org/51221504:07
*** thorst has quit IRC04:08
*** hongbin has joined #openstack-nova04:12
*** coreywright has joined #openstack-nova04:15
*** vks1 has joined #openstack-nova04:15
*** gbarros has quit IRC04:16
*** lei-zh has quit IRC04:18
*** sree has joined #openstack-nova04:19
*** threestrands_ has joined #openstack-nova04:24
*** threestrands_ has quit IRC04:24
*** threestrands_ has joined #openstack-nova04:24
*** psachin has joined #openstack-nova04:24
*** threestrands has quit IRC04:26
openstackgerritMerged openstack/nova stable/pike: doc: fix flavor notes  https://review.openstack.org/50943804:33
*** tetsuro has joined #openstack-nova04:40
*** itlinux has quit IRC04:40
openstackgerritNguyen Van Trung proposed openstack/nova-specs master: Add doc8 to pep8 check for nova-specs project  https://review.openstack.org/51956104:43
alex_xuefried: cool, then just let us waiting for the update04:44
*** thorst has joined #openstack-nova04:44
*** salv-orlando has joined #openstack-nova04:46
*** thorst has quit IRC04:49
openstackgerritMerged openstack/nova master: Do not use “-y” for package install  https://review.openstack.org/51823804:50
*** salv-orlando has quit IRC04:50
*** ratailor has joined #openstack-nova04:58
*** hongbin has quit IRC04:59
*** lei-zh has joined #openstack-nova05:00
*** ratailor has quit IRC05:02
openstackgerritjichenjc proposed openstack/nova master: Add description for resource class creation  https://review.openstack.org/50808305:02
*** ratailor has joined #openstack-nova05:02
*** ratailor has quit IRC05:03
*** ratailor has joined #openstack-nova05:05
*** rcernin_ has joined #openstack-nova05:09
*** rcernin has quit IRC05:09
*** david-lyle has joined #openstack-nova05:13
*** itlinux has joined #openstack-nova05:17
*** janki has joined #openstack-nova05:19
*** janki has quit IRC05:20
*** thorst has joined #openstack-nova05:20
*** janki has joined #openstack-nova05:20
openstackgerritjichenjc proposed openstack/nova master: remove glance usage inside compute  https://review.openstack.org/51139205:21
openstackgerritjichenjc proposed openstack/nova master: Remove direct usage of glance.generate_image_url  https://review.openstack.org/51139705:21
*** vks1 has quit IRC05:22
*** esberglu has joined #openstack-nova05:23
*** thorst has quit IRC05:25
*** sridharg has joined #openstack-nova05:25
*** takashin has joined #openstack-nova05:25
*** esberglu has quit IRC05:27
*** trungnv has joined #openstack-nova05:30
*** gongysh has joined #openstack-nova05:32
*** gouthamr has quit IRC05:44
*** salv-orlando has joined #openstack-nova05:47
*** salv-orlando has quit IRC05:51
*** thorst has joined #openstack-nova05:54
*** vladikr has quit IRC05:55
*** vladikr has joined #openstack-nova05:55
*** gongysh has quit IRC05:56
*** fragatina has quit IRC05:57
*** thorst has quit IRC06:00
*** jichen has quit IRC06:00
*** jichen has joined #openstack-nova06:01
*** armax has joined #openstack-nova06:01
*** esberglu has joined #openstack-nova06:03
*** esberglu has quit IRC06:08
*** vks1 has joined #openstack-nova06:08
*** pcaruana has joined #openstack-nova06:10
*** yamahata has joined #openstack-nova06:12
*** salv-orlando has joined #openstack-nova06:14
*** itlinux has quit IRC06:22
*** threestrands_ has quit IRC06:24
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add migration db and object pagination support.  https://review.openstack.org/51490406:25
*** armax has quit IRC06:27
*** sree_ has joined #openstack-nova06:29
*** thorst has joined #openstack-nova06:29
*** david-lyle has quit IRC06:30
*** sree_ is now known as Guest7214106:30
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Implement query param schema for migration index  https://review.openstack.org/51864406:31
*** sree has quit IRC06:33
*** karthiks has quit IRC06:38
*** thorst has quit IRC06:38
*** lei-zh has quit IRC06:41
*** lei-zh has joined #openstack-nova06:41
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for shelve_offload instances  https://review.openstack.org/52365306:45
*** wxy_ is now known as wxy06:51
*** litao__ has joined #openstack-nova07:02
*** tojuvone has joined #openstack-nova07:08
*** thorst has joined #openstack-nova07:09
*** lei-zh has quit IRC07:10
*** lei-zh has joined #openstack-nova07:10
openstackgerritZhenyu Zheng proposed openstack/nova master: Add instance action record for backup instances  https://review.openstack.org/52367607:11
*** lei-zh has quit IRC07:11
*** lei-zh has joined #openstack-nova07:12
*** thorst has quit IRC07:13
*** Oku_OS-away is now known as Oku_OS07:16
*** hieulq has joined #openstack-nova07:18
*** Guest72141 has quit IRC07:20
*** sree has joined #openstack-nova07:20
*** andreas_s has joined #openstack-nova07:21
*** sree has quit IRC07:22
*** sree has joined #openstack-nova07:23
*** damien_r has joined #openstack-nova07:24
*** bbbbzhao_ has quit IRC07:26
*** rcernin_ has quit IRC07:29
*** lei-zh has quit IRC07:30
*** lei-zh has joined #openstack-nova07:30
*** lajoskatona has joined #openstack-nova07:32
*** adisky_ has joined #openstack-nova07:43
nsinghefried: To create keystoneauth session we are using some parameters like insecure, verify, cacert, cert, and key. And to creating sessionClient we are passing keystoneauth session for nova/glance/cinder. but these conf options from session are not used anywhere while creating session client. so is it only necessary to create session?07:44
*** thorst has joined #openstack-nova07:47
*** thorst has quit IRC07:52
*** jangutter has quit IRC07:54
*** jangutter has joined #openstack-nova07:54
*** jangutter has quit IRC07:54
*** jangutter has joined #openstack-nova07:55
*** sahid has joined #openstack-nova07:57
*** slaweq has joined #openstack-nova08:03
*** takashin has left #openstack-nova08:06
*** karthiks has joined #openstack-nova08:06
*** salv-orlando has quit IRC08:06
*** alexchadin has joined #openstack-nova08:10
*** lei-zh has quit IRC08:12
*** damien_r has quit IRC08:12
*** lei-zh has joined #openstack-nova08:12
*** pprokop has quit IRC08:12
*** yamahata has quit IRC08:13
openstackgerritTakashi NATSUME proposed openstack/nova master: [placement] Add functional tests for resource class API  https://review.openstack.org/52450608:13
*** AlexeyAbashkin has joined #openstack-nova08:15
*** pprokop has joined #openstack-nova08:15
openstackgerritJack Lu proposed openstack/nova master: Update supported transports for iscsi connector  https://review.openstack.org/52444308:17
*** sridhargaddam has joined #openstack-nova08:20
*** sridharg has quit IRC08:21
*** thorst has joined #openstack-nova08:25
*** tetsuro has quit IRC08:27
*** thorst has quit IRC08:29
*** salv-orlando has joined #openstack-nova08:33
*** migi_ is now known as migi08:33
*** sridhargaddam has quit IRC08:34
*** damien_r has joined #openstack-nova08:39
openstackgerritIldiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow  https://review.openstack.org/33028508:39
openstackgerritIldiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments  https://review.openstack.org/26758708:39
*** jpena|off is now known as jpena08:42
*** TuanLA has quit IRC08:44
*** ralonsoh has joined #openstack-nova08:44
*** TuanLA has joined #openstack-nova08:44
*** moshele has quit IRC08:46
*** esberglu has joined #openstack-nova08:48
*** qsyqian has quit IRC08:49
openstackgerritsahid proposed openstack/nova master: libvirt: slow live-migration to ensure network is ready  https://review.openstack.org/49745708:49
*** gmann is now known as gmann_afk08:54
*** thorst has joined #openstack-nova08:57
*** psachin` has joined #openstack-nova08:59
openstackgerritErik Berg proposed openstack/nova stable/pike: Fix binary name  https://review.openstack.org/52453109:00
openstackgerritMerged openstack/nova master: Reduce code complexity - linux_net.py  https://review.openstack.org/35987109:00
*** psachin has quit IRC09:00
*** gszasz has joined #openstack-nova09:00
*** thorst has quit IRC09:01
*** udesale__ has joined #openstack-nova09:01
*** udesale has quit IRC09:02
*** hshiina has quit IRC09:02
*** udesale has joined #openstack-nova09:04
*** udesale__ has quit IRC09:04
*** udesale__ has joined #openstack-nova09:05
*** udesale has quit IRC09:08
openstackgerritYikun Jiang (Kero) proposed openstack/nova master: Add cross cell sort support for get_migrations  https://review.openstack.org/51727309:16
*** lei-zh has quit IRC09:17
*** Dinesh__Bhor has quit IRC09:18
*** Dinesh__Bhor has joined #openstack-nova09:19
*** sahid has quit IRC09:19
*** thorst has joined #openstack-nova09:29
*** psachin`` has joined #openstack-nova09:29
*** psachin` has quit IRC09:30
*** lucas-afk is now known as lucasagomes09:33
*** thorst has quit IRC09:34
*** yingjun has quit IRC09:34
*** alexchadin has quit IRC09:37
*** derekh has joined #openstack-nova09:39
*** sree has quit IRC09:44
*** sahid has joined #openstack-nova09:48
*** jangutter has quit IRC09:49
*** jangutter has joined #openstack-nova09:50
*** dtantsur|afk is now known as dtantsur09:51
*** TuanLA has quit IRC09:53
*** markvoelker has quit IRC09:55
*** jaosorior has quit IRC09:55
*** alexchadin has joined #openstack-nova09:55
*** sree has joined #openstack-nova09:58
*** sree has quit IRC09:58
*** karthiks has quit IRC09:59
*** adisky_ has quit IRC10:00
*** sree has joined #openstack-nova10:00
*** abhishekk has quit IRC10:02
*** trungnv has quit IRC10:03
*** hieulq has quit IRC10:04
*** AJaeger has joined #openstack-nova10:04
*** AJaeger has left #openstack-nova10:05
*** thorst has joined #openstack-nova10:07
*** alexchadin has quit IRC10:07
*** udesale has joined #openstack-nova10:11
openstackgerritStephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project  https://review.openstack.org/51430910:11
*** thorst has quit IRC10:11
*** hoangcx has quit IRC10:12
*** udesale__ has quit IRC10:13
*** alexchadin has joined #openstack-nova10:13
*** claudiub has joined #openstack-nova10:14
stephenfinMan, Gerrit is slooowww this morning10:15
bauzasstephenfin: probably because of the close week-end ? :p10:19
bauzasor because it's snowing here10:20
bauzasFWIW, resorts are opening this week-end \o/10:20
openstackgerritStephen Finucane proposed openstack/nova master: zuul: Move legacy jobs to project  https://review.openstack.org/51430910:20
*** namnh has quit IRC10:20
*** phuongnh has quit IRC10:25
*** openstackgerrit has quit IRC10:33
*** gaurangt has quit IRC10:39
*** gaurangt has joined #openstack-nova10:40
*** andreas_s has quit IRC10:41
*** gszasz has quit IRC10:41
*** andreas_s has joined #openstack-nova10:41
*** thorst has joined #openstack-nova10:44
*** andreas_s has quit IRC10:45
*** jichen has quit IRC10:47
*** thorst has quit IRC10:48
*** andreas_s has joined #openstack-nova10:52
*** markvoelker has joined #openstack-nova10:55
*** janki has quit IRC10:56
*** andreas_s has quit IRC10:57
*** janki has joined #openstack-nova11:01
*** andreas_s has joined #openstack-nova11:04
*** andreas_s has quit IRC11:10
*** andreas_s has joined #openstack-nova11:10
*** openstackgerrit has joined #openstack-nova11:10
openstackgerritMerged openstack/nova stable/pike: Fix race in delete allocation in ServerMovingTests  https://review.openstack.org/50887211:10
*** kaisers_ has joined #openstack-nova11:13
*** thorst has joined #openstack-nova11:17
*** openstackgerrit has quit IRC11:18
*** thorst has quit IRC11:22
*** vks1 has quit IRC11:25
*** rodolof has joined #openstack-nova11:26
*** alexchadin has quit IRC11:26
*** alexchadin has joined #openstack-nova11:27
*** udesale has quit IRC11:29
*** Dinesh__Bhor has quit IRC11:30
*** alexchadin has quit IRC11:34
*** alexchadin has joined #openstack-nova11:34
*** litao__ has quit IRC11:37
*** nicolasbock has joined #openstack-nova11:43
*** sree has quit IRC11:46
*** esberglu has quit IRC11:49
*** esberglu has joined #openstack-nova11:49
*** salv-orlando has quit IRC11:49
*** salv-orlando has joined #openstack-nova11:50
*** AJaeger has joined #openstack-nova11:50
*** chyka has joined #openstack-nova11:55
*** salv-orlando has quit IRC11:55
*** thorst has joined #openstack-nova11:57
*** chyka has quit IRC12:00
*** thorst has quit IRC12:01
*** sree has joined #openstack-nova12:01
*** sahid has quit IRC12:04
efriednsingh I have noticed that too - In Nova we go through all this trouble to create a session from conf parameters, and then we create the client for the service which creates its own session.12:08
efriednsingh Eventually the vision (per mordred) is to get rid of the clients altogether and just use the ksa adapter for the API communication.12:09
*** alexchadin has quit IRC12:10
*** thorst has joined #openstack-nova12:11
*** eharney has joined #openstack-nova12:14
*** lucasagomes is now known as lucas-hungry12:14
*** sree has quit IRC12:21
*** ratailor has quit IRC12:23
*** Jack_Iv has joined #openstack-nova12:27
*** smatzek has joined #openstack-nova12:27
*** alexchadin has joined #openstack-nova12:29
nsinghefried: thank you for the info. :)12:32
*** psachin`` has quit IRC12:33
*** sree has joined #openstack-nova12:37
ildikovjohnthetubaguy: hi, daily pretty please: https://review.openstack.org/#/c/330285/12:38
*** janki has quit IRC12:38
*** openstackgerrit has joined #openstack-nova12:43
openstackgerritMerged openstack/python-novaclient master: inject file: add description of injecting multiple files.  https://review.openstack.org/51565512:43
*** sree has quit IRC12:47
*** udesale has joined #openstack-nova12:47
*** gszasz has joined #openstack-nova12:48
*** ankit_ has quit IRC12:51
*** vladikr has quit IRC12:52
*** yamamoto has quit IRC12:53
*** vladikr has joined #openstack-nova12:53
*** yangyapeng has quit IRC12:53
*** yamamoto has joined #openstack-nova12:54
*** yangyapeng has joined #openstack-nova12:57
*** cdent has joined #openstack-nova12:59
*** jpena is now known as jpena|lunch13:00
*** pchavva has joined #openstack-nova13:01
*** yangyapeng has quit IRC13:01
*** vladikr has quit IRC13:03
*** vladikr has joined #openstack-nova13:03
*** sree has joined #openstack-nova13:04
*** liverpooler has joined #openstack-nova13:04
*** vladikr has quit IRC13:05
*** vladikr has joined #openstack-nova13:05
*** lucas-hungry is now known as lucasagomes13:08
*** sree has quit IRC13:09
*** yamamoto has quit IRC13:09
*** sree has joined #openstack-nova13:12
*** sahid has joined #openstack-nova13:14
*** dtantsur is now known as dtantsur|brb13:14
*** tetsuro_ has joined #openstack-nova13:15
*** kalyan has quit IRC13:15
*** sree has quit IRC13:17
*** links has quit IRC13:17
*** efried is now known as fried_rice13:22
fried_riceFriday!13:22
*** READ10 has joined #openstack-nova13:24
openstackgerritChen Hanxiao proposed openstack/python-novaclient master: inject file: add method of showing quota value of injecting files for 'rebuild' command  https://review.openstack.org/52458813:24
*** moshele has joined #openstack-nova13:25
*** yamamoto has joined #openstack-nova13:25
*** markvoelker has quit IRC13:25
*** markvoelker has joined #openstack-nova13:25
fried_ricestephenfin You around?13:26
fried_riceor bauzas or gibi or alex_xu ?  Easy review, which will merge another patch for a backportable bug: https://review.openstack.org/#/c/524245/13:27
*** moshele has quit IRC13:30
*** yamamoto has quit IRC13:32
*** yamamoto has joined #openstack-nova13:33
*** yangyapeng has joined #openstack-nova13:34
stephenfinfried_rice: I am. Looking now13:40
fried_ricestephenfin Thanks!13:40
*** stephenfin is now known as finucannot13:40
finucannotfried_rice: Are you finding Gerrit slow today too?13:42
fried_ricefinucannot Yeah, totally.  I asked in -infra, but nobody bit yet.13:42
finucannotSo not my network. Good to know.13:42
fried_riceThat was an hour ago - more people are awake now.  I'll ask again...13:42
jaypipesfried_rice: mornin13:43
fried_ricejaypipes Howdy!13:43
*** jaypipes is now known as leakypipes13:43
cdentI’ve been out of a bed for a long time, but I make no claims to being awake.13:43
leakypipesfried_rice: what was the thing with alex from earlier?13:43
fried_riceleakypipes He was asking about https://review.openstack.org/#/c/517119/13:43
leakypipesfried_rice: ah, k13:44
fried_riceI had thought you were making local changes to it, but looking back it seemed you had only rebased it.13:44
leakypipesfried_rice: yeah, I'll work on that this mroning.13:44
fried_riceCool beans.13:44
fried_ricefinucannot FYI apparently we're "waiting for an infra-root to speak some healing words" (per AJaeger)13:45
fried_riceOh, you're in -infra too.  I'll quit relaying.13:45
*** mriedem has joined #openstack-nova13:48
AJaegerfried_rice: I'm not here in general, continue relaying, please13:48
mriedemgmann_afk: no we don't need to add query param schema for the cells APIs13:48
mriedemthose are likely gone in rocky13:48
fried_riceAJaeger But finucannot is in -infra, so he can follow the conversation there.13:49
AJaeger;)13:49
*** sree has joined #openstack-nova13:49
*** yingjun has joined #openstack-nova13:50
-openstackstatus- NOTICE: gerrit has been restarted to get it back to its normal speed.13:52
*** salv-orlando has joined #openstack-nova13:52
*** itlinux has joined #openstack-nova13:53
*** sree has quit IRC13:54
*** salv-orlando has quit IRC13:57
*** yamahata has joined #openstack-nova13:57
*** links has joined #openstack-nova13:58
*** nicolasbock has quit IRC13:59
*** yingjun has quit IRC14:01
*** yamamoto has quit IRC14:01
*** jpena|lunch is now known as jpena14:01
*** bnemec has joined #openstack-nova14:02
*** AlexeyAbashkin has quit IRC14:02
*** liverpooler has quit IRC14:03
*** edleafe is now known as figleaf14:03
openstackgerritEric Fried proposed openstack/nova stable/pike: Proper error handling by _ensure_resource_provider  https://review.openstack.org/52461814:04
fried_ricemriedem ^ pike cherry-pick, with conflicts.  Think I'll wait for zuul before doing the ocata one.14:04
mriedemok14:06
finucannotlyarwood: I assume you could review this, seeing as you only cherry-picked it (vs. writing it)? https://review.openstack.org/#/c/524157/14:07
mriedemfinucannot: not really how backport reviews work14:09
*** lyan has joined #openstack-nova14:09
*** Guest78_ has joined #openstack-nova14:09
finucannotmriedem: ok, I was curious about that :) Can you review because you wrote it? Can he review because he backported?14:09
mriedemi must say, however, that whoever did write the original must be very handsome indeed14:09
*** esberglu has quit IRC14:10
mriedemdone14:10
*** esberglu has joined #openstack-nova14:10
*** bnemec is now known as beekneemech14:11
*** erlon has joined #openstack-nova14:13
Guest78_I'm having issues unshelving an instance that has PCI passthrough aliases associated with it......the wrong hypervisor host is selected.  Is there a setting that I'm missing to include PciPassthroughFilter when scheduling for unshelve?14:14
*** salv-orlando has joined #openstack-nova14:15
*** vks1 has joined #openstack-nova14:16
*** jmlowe has quit IRC14:16
mriedemunshelve should go through the same scheduling as the initial create14:16
mriedemso if you have the filters setup for the initial create properly, unshelve should work the same14:17
*** yamamoto has joined #openstack-nova14:17
openstackgerritMatt Riedemann proposed openstack/nova master: Deprecate file injection  https://review.openstack.org/52202714:17
leakypipesGuest78_: note that this channel is for development questions, so generally that would be a question for the openstack@ mailing list. However, since it's Friday... if you just want a quick and dirty fix, you can probably muck with the data in the instance_extra table for that shelved instance and change the flavor's PCI requirements manually.14:18
Guest78_That's what I thought but that doesn't seem to be the case: 2017-12-01 13:55:52.843 2450 WARNING nova.scheduler.host_manager [req-b7876409-1149-4f8a-837d-46f0023fed10 6136e80cc67e4759843a5f95f7a9a98b 7618d5a893f04349b9feb2ca541a6fce - default default] Selected host: overcloud-novacompute-0.localdomain failed to consume from instance. Error: PCI device request [InstancePCIRequest(alias_name='GeForce10\14:18
Guest78_80',count=1,is_new=<?>,request_id=<?>,spec=[{dev_type='type-PCI',product_id='1b06',vendor_id='10de'}])] failed: PciDeviceRequestFailed: PCI device request [InstancePCIRequest(alias_name='GeForce1080',count=1,is_new=<?>,request_id=<?>,spec=[{dev_type='type-PCI',product_id='1b06',vendor_id='10de'}])] failed14:18
*** gibi is now known as giblet14:18
ildikovmriedem: hi14:19
ildikovmriedem: rebased the new attach patch and ran your test on it too, it looks good so far14:19
* giblet seriusly needs a beer and that his downstream people go and look at how CI works upstream and learn from it14:19
leakypipesgiblet: ++14:19
leakypipesgiblet: on both the beer and the learning.14:20
*** ildikov is now known as coffee_cat14:20
coffee_catgiblet: I think it's late enough in the afternoon to get a beer now :)14:20
* cdent has beer envy14:20
cdentthere’s no beer in the hosue14:20
*** liverpooler has joined #openstack-nova14:22
gibletahh I'm still in the tea phase for this afternoon14:24
*** dansmith is now known as superdan14:25
*** alexchadin has quit IRC14:25
mriedemsuperdan: just came across this in triage https://bugs.launchpad.net/nova/+bug/1735719 - seems the delete_host command should mark the compute node as mapped=0 so it can be discovered again14:25
openstackLaunchpad bug 1735719 in OpenStack Compute (nova) "Cell_v2 discover_hosts Not Working, after deleted a host from a cell " [Medium,Triaged]14:25
mriedemsince we don't have a way to directly delete compute node records14:25
mriedemcoffee_cat: ack14:26
superdanhrm, will look14:26
*** links has quit IRC14:26
*** cdent is now known as dentures14:26
*** dentures is now known as denture14:26
denturemeh14:27
*** denture is now known as cdent14:27
superdancdent: cdenture maybe?14:28
mriedemcdent: did you see that patch i started for some policy rules/file stuff for placement? how much do you remember of all that when laski was doing it a few releases ago?14:28
mriedemhttps://review.openstack.org/#/c/524425/14:28
mriedemturns out that now that people are starting to roll up to using placement, they actually care about configurable policy for the service :)14:28
*** sshwarts has quit IRC14:29
*** Jack_Iv has quit IRC14:30
cdentmriedem: I saw it briefly this morning before I rolled out but didn’t have a chance to look at it for real yet. I had thought that we were planning to put it off until we actually had some reason to have policy other than “you must be this high to ride any of this ride” to align with the “minimal config” thing?14:30
leakypipesmriedem, superdan, giblet: melwitt did some good sleuthing on the bug report associated with https://review.openstack.org/#/c/518022. I concur with her findings and think the patch is good to go.14:31
cdentsuperdan: cdenture could be workable, but it’s not quite right enough for me to be all in14:31
*** smatzek has quit IRC14:31
mriedemcdent: given it's all admin-only apis i think that's also why we put off configurable policy,14:31
mriedemthe issue that someone brought up last night was the admin role,14:31
mriedemthey don't have an "admin" role i guess14:31
mriedemso they are hacking around it with a config option14:32
*** smatzek has joined #openstack-nova14:32
mriedemmgagne: ^14:32
mriedemthis https://gist.github.com/mgagne/b43c1e085c1f1d50bebc054a7d38768814:32
mriedemanyway, not sure it's high priority, just something to have on the radar14:32
cdentah14:32
cdenthmmm14:32
mriedemsince i totally forgot about this14:32
superdancdent: hah, okay14:32
*** yamamoto has quit IRC14:33
*** Oku_OS is now known as Oku_OS-away14:34
fried_ricecdent Or "indenture" :)14:35
cdentprogress14:35
superdanoof14:36
cdentof greater concern than picking a good nick is the zomg, effort, of having to remember to change it on fridays. _So_ exhausting.14:36
superdanfriday nicks are supposed to be fun :)14:36
fried_rice"decdent" would be a little morbid14:36
*** smatzek has quit IRC14:36
mriedemsuperdan: melwitt: tssurya: i'll start throwing cellsv2 bugs in the meeting agenda for lack of a better place https://wiki.openstack.org/wiki/Meetings/NovaCellsv2#Agenda14:39
figleafcdent -> decent14:39
fried_riceMm, that's a good one.14:40
fried_riceor dcent14:40
figleafor, more appropriately, descent14:40
fried_riceheh14:40
*** slaweq has quit IRC14:41
cdentthis is reminding me of my server naming scheme from late 90s: depressing words that start with ‘d’14:41
*** slaweq has joined #openstack-nova14:41
superdanmriedem: okay14:42
cdentHas anything placement-related happened in the last 12 hours that I ought to be aware of for the update?14:42
mriedemjust the policy thing14:43
cdentthanks14:43
mriedemmikal: https://bugs.launchpad.net/nova/+bug/173566314:45
openstackLaunchpad bug 1735663 in OpenStack Compute (nova) "privsep.daemon takes 9 seconds during taking volume attaching operation" [Undecided,New]14:45
fried_ricecdent You got the _ensure bug on your list?14:46
cdentyessir14:46
fried_ricecoo14:46
fried_ricecdent And this series: https://review.openstack.org/#/c/523932/14:46
openstackgerritJay Pipes proposed openstack/nova master: handle traits with sharing providers  https://review.openstack.org/51711914:47
openstackgerritJay Pipes proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs  https://review.openstack.org/51938014:47
openstackgerritJay Pipes proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs  https://review.openstack.org/51961714:47
openstackgerritJay Pipes proposed openstack/nova master: Test allocation candidates: multiple aggregates  https://review.openstack.org/51863314:47
openstackgerritJay Pipes proposed openstack/nova master: Test helper: validate provider summaries  https://review.openstack.org/51898214:47
openstackgerritJay Pipes proposed openstack/nova master: Fix accumulated nits in refactor series  https://review.openstack.org/52118914:47
openstackgerritJay Pipes proposed openstack/nova master: Add aggregates check in allocation canditates  https://review.openstack.org/52240714:47
openstackgerritJay Pipes proposed openstack/nova master: Add functional test for get_all_with_shared()  https://review.openstack.org/52240814:47
openstackgerritJay Pipes proposed openstack/nova master: Fix missing rps in allocation candidates  https://review.openstack.org/52240914:47
openstackgerritJay Pipes proposed openstack/nova master: placement: func tests for multiple shared RPs  https://review.openstack.org/49873714:47
cdentfried_rice: that’ll get covered with the n-r-p topic14:47
fried_riceight14:48
*** mlavalle has joined #openstack-nova14:48
leakypipesfried_rice, giblet, superdan, mriedem: ok, series starting at https://review.openstack.org/#/c/517119/ is rebased and ready for reviews (mostly new func tests for alloc candidate permutations)14:48
fried_riceleakypipes ack14:48
*** vks1 has quit IRC14:49
mriedemi have to spend quality time with https://review.openstack.org/#/c/330285/ this morning14:49
* coffee_cat would really appreciate that :)14:50
leakypipeshehe14:50
leakypipescoffee_cat: why? it's only on revision 165... that's not even a middle-aged patch.14:51
coffee_catleakypipes: if you happen to have time while Zuul is chewing on all of those above you could chime in too :)14:51
mriedemguh, MOS bugs from mitaka https://bugs.launchpad.net/nova/+bug/173540714:51
openstackLaunchpad bug 1735407 in Mirantis OpenStack 9.x "[Nova] Evacuation doesn't respect anti-affinity rules" [High,New] - Assigned to Oleksiy Molchanov (omolchanov)14:51
leakypipescoffee_cat: yeah, I have it open :)14:51
coffee_catleakypipes: yeah, I appreciate all the time I can spend rebasing it :)14:52
mriedemi think at this point, any MOS bugs from mitaka can just be marked invalid with "can you recreate on master?"14:52
*** Alex_Staf has joined #openstack-nova14:52
coffee_catleakypipes: sweet, I can see I will not be bored over the weekend :)14:52
leakypipesmriedem: "can you recreated with Triple-O please?"14:52
mriedemactually this is a latent bug14:53
mriedemsince you can't evacuate more than one instance in the same request,14:53
mriedemand they are using the anti-affinity policy,14:53
mriedemwhich relies on the late affinity check on the compute,14:53
mriedemand even if that fails, we don't reschedule an evacuate14:53
mriedemso :(14:53
leakypipesfun14:54
gibletmriedem: I think there is no late check for rebuild just for boot14:54
mriedemgiblet: ah, ok,14:54
mriedemwell in this case,14:54
mriedemif they spread out the evacuates, it works - because they give time for the computes to send instance info updates back to the scheduler14:54
mriedemwhich reminds me,14:55
mriedemthere is a CLI in novaclient which just gets all instances from a given host and evacuates them in a for loop right superdan?14:55
superdanyeah14:55
mriedemif those instances were in a server group policy, we are likely destroying those14:55
*** awaugama has joined #openstack-nova14:55
*** tidwellr has joined #openstack-nova14:56
gibletwithout the late check we break the group policy, with the late check even nova doesn't reschedule we at least fail the evac instead of breaking the policy14:57
gibletthat is already better14:57
gibletof course it would be nice to reschedule14:58
*** yamamoto has joined #openstack-nova14:59
mriedemhttps://bugs.launchpad.net/mos/+bug/1735407/comments/314:59
openstackLaunchpad bug 1735407 in Mirantis OpenStack 9.x "[Nova] Evacuation doesn't respect anti-affinity rules" [High,New] - Assigned to Oleksiy Molchanov (omolchanov)14:59
mriedemmy analysis14:59
mriedemi don't really want to add a reschedule loop for evacuate14:59
*** andreas_s has quit IRC14:59
*** smatzek has joined #openstack-nova15:00
mriedemlong-term we model affinity in placement and use that, but i'm not entirely sure that will fix this case either15:00
leakypipesmriedem: ++15:00
gibletmriedem: +1 for your analysis15:00
*** yamamoto has quit IRC15:00
gibletmriedem: can we just add the post check without reschedule?15:01
*** smatzek_ has joined #openstack-nova15:01
mriedemwell i guess the affinity stuff in placement would mean we somehow 'claim' that the instance we're scheduling / evacuating is going to be on some target host,15:02
mriedemso a concurrent request gets that information when asking placement where it should go15:02
mriedemnot sure how that is all going to work out15:02
mriedemsince we don't want to model server group policies in placement15:02
*** smatzek has quit IRC15:04
*** hongbin has joined #openstack-nova15:05
mriedemgiblet: https://bugs.launchpad.net/mos/+bug/1735407/comments/415:05
openstackLaunchpad bug 1735407 in Mirantis OpenStack 9.x "[Nova] Evacuation doesn't respect anti-affinity rules" [High,New] - Assigned to Oleksiy Molchanov (omolchanov)15:05
mriedemgiblet: doing the late check might be reasonable15:05
mriedemfail the evacuate, so the user can retry15:05
gibletI vote for option 1) now and maybe consider prioritizing affinity impl with Placement in the future15:06
*** sahid has quit IRC15:06
*** david-lyle has joined #openstack-nova15:07
*** sahid has joined #openstack-nova15:07
gibletmriedem: I have to run soon but I have already looked into adding such late check before so I can take this bug if it can wait until Monday15:07
mriedemsure15:08
mriedemassigned to you15:08
gibletcool, thanks15:08
mriedemsuperdan: here is belmiro's build request cellsv1 bug https://bugs.launchpad.net/nova/+bug/173535315:09
openstackLaunchpad bug 1735353 in OpenStack Compute (nova) "build_request not deleted when using cellsV1 and local nova_api DB" [Undecided,New]15:09
*** gouthamr has joined #openstack-nova15:11
*** amodi has joined #openstack-nova15:15
mriedemcdent: figleaf: rest api error code question,15:16
mriedemhttps://bugs.launchpad.net/nova/+bug/1735329 500 is surely not correct in this case,15:16
openstackLaunchpad bug 1735329 in OpenStack Compute (nova) "console-log return 500 if n-cpu is shutdown" [Undecided,New] - Assigned to hongbin (hongbin034)15:16
mriedemi'm thinking 409?15:16
* cdent read15:17
cdents15:17
fried_ricemriedem cdent 503 "Service Unavailable" ?15:18
*** moshele has joined #openstack-nova15:18
*** damien_r has quit IRC15:19
cdent503 doesn’t really mean that, we’ve tried to discourage it15:19
mriedemwas going to say, i've been told not to use 501, 503, etc15:19
cdentmriedem: I reckon 409 is probably right15:19
mriedemthis api already returns 409 so that's a good fit15:19
mriedemhongbin: ^15:19
*** udesale has quit IRC15:20
fried_rice"conflict" seems weird for this, but whatevs.15:20
cdent503 is more for when a proxy is involved15:20
hongbino/15:20
cdentfried_rice: conflict is that the state of the resource is not aligned with what you need it to be15:20
fried_riceThat's a stretch.15:20
fried_riceIMO15:20
fried_riceBut I'm no expert.15:20
cdentthat’s what conflict really means15:21
openstackgerritDan Smith proposed openstack/nova master: Unmap compute nodes when deleting host mapping  https://review.openstack.org/52464115:21
mriedemooo15:21
*** ratailor has joined #openstack-nova15:25
mriedemnot sure how to find out our default http response timeout15:25
*** pchavva has quit IRC15:25
superdanmriedem: ^15:26
mriedemsuperdan: i know, already -1ed :)15:26
*** felipemonteiro has joined #openstack-nova15:26
mriedemor is the http response timeout configurable in uwsgi15:26
mriedem?15:26
*** dtantsur|brb is now known as dtantsur15:27
superdanmriedem: https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L70715:27
mriedemsuperdan: that's not what you want15:27
mriedemthat's the nodename15:27
mriedemComputeNode.host is the hostname15:27
mriedemit's totally confusing15:28
*** felipemonteiro_ has joined #openstack-nova15:28
superdanum15:28
*** coreywright has quit IRC15:29
superdanI thought get_all_by_host was by service hostname15:30
superdanwhy else would be it a multi-return?15:30
superdanI feel like I've made this mistake like eight times15:30
mriedemlet me see what the actual HostMapping.host field is set from15:30
openstackgerritJay Pipes proposed openstack/nova master: placement: allow filter providers in tree  https://review.openstack.org/37721515:31
openstackgerritJay Pipes proposed openstack/nova master: placement: adds REST API for nested providers  https://review.openstack.org/38480715:31
openstackgerritJay Pipes proposed openstack/nova master: placement: update client to set parent provider  https://review.openstack.org/38569315:31
mriedemhost_mapping = HostMapping(15:31
mriedem                ctxt, host=compute.host,15:31
mriedem                cell_mapping=cm)15:31
mriedemin _check_and_create_host_mappings15:31
*** felipemonteiro has quit IRC15:31
*** armax has joined #openstack-nova15:32
*** moshele has quit IRC15:33
*** ianw has quit IRC15:34
*** yamamoto has joined #openstack-nova15:35
superdanoh hypervisor_hostnme is actually the node uuid for ironic isn't it15:36
superdangawd why can't we call things what they are15:36
*** gbarros has joined #openstack-nova15:37
cdentsuperdan++15:38
*** Alex_Staf has quit IRC15:39
superdanwell, regardless, hostmapping uses cn.host15:41
superdanso that's what I want anyway15:41
*** tbachman has joined #openstack-nova15:41
*** coreywright has joined #openstack-nova15:42
mriedemsuperdan: ? so you're going to use get_by_hypervisor?15:43
superdanno I mean the .host field15:43
superdanI really thought .host was the service host15:44
superdanbecause we have get_by_host_and_node15:44
*** pchavva has joined #openstack-nova15:44
superdanI'm doing this all whilst on a call, which I really shouldn't do15:44
mriedemwhat gets passed to delete_host is the computenode.host, so you have to use ComputeNodeList.get_all_by_host, yes?15:44
mriedemok15:44
*** thorst has quit IRC15:45
mriedemthe only time this would return more than 1 node is ironic15:45
openstackgerritMerged openstack/nova stable/pike: Fix ValueError when loading old pci device record  https://review.openstack.org/52415715:45
superdanthat's what I mean, what we pass is what is in hostmapping.host, which is populated from computenode.host15:45
superdanwhich I guess is the service hostname15:46
mriedemyeah, it's what comes from CONF.host15:46
superdanI really didn't think that's what was in hostmapping15:47
*** jmlowe has joined #openstack-nova15:47
*** Tom-Tom has quit IRC15:53
*** david-lyle has quit IRC15:53
openstackgerritDan Smith proposed openstack/nova master: Unmap compute nodes when deleting host mapping  https://review.openstack.org/52464115:53
superdanmriedem: try that15:53
*** lyan has quit IRC15:56
bauzassuperdan: mriedem: yeah, so the compute service hostname is the host, and the Ironic node UUID is the hypervisor_hostname15:57
bauzasso, when we get all the nodes for a host, we should get all the ironic nodes related to the specific nova compute15:57
superdanyeah, that's not really the confusion,15:58
superdanit's what was in hostmapping.host for me,15:58
superdanand I thought it was the node15:58
bauzasah no, unfortunatel15:58
bauzasindeed15:58
*** AJaeger has left #openstack-nova15:59
bauzasand yeah, the main reason why we don't modify the field to be "node" instead of "hypervisor_hostname" is because it would be a huge change15:59
mriedemsuperdan: want to fix the alignment in the test?15:59
*** lyan has joined #openstack-nova15:59
mriedemputting comments next to the fields in the code would probably help16:00
mriedem"THIS IS THE NODENAME!!!:16:00
superdanalignment imported from the code I copied from above, but sure16:00
*** damien_r has joined #openstack-nova16:00
mriedemtwo wrongs don't make a right dan16:00
mriedemunless you're feeding bacon to a pig16:01
openstackgerritDan Smith proposed openstack/nova master: Unmap compute nodes when deleting host mapping  https://review.openstack.org/52464116:01
* finucannot heard something about bacon16:03
*** Apoorva has joined #openstack-nova16:04
*** Alex_Staf has joined #openstack-nova16:04
*** slaweq has quit IRC16:04
*** slaweq has joined #openstack-nova16:04
*** damien_r has left #openstack-nova16:05
*** sree has joined #openstack-nova16:06
*** tidwellr has quit IRC16:06
mriedemlyarwood: https://bugs.launchpad.net/nova/+bug/173578016:08
openstackLaunchpad bug 1735780 in OpenStack Compute (nova) "n-cpu logs error "is not a valid LUKS device"" [Undecided,Confirmed]16:08
mriedemhttp://logs.openstack.org/26/515426/1/check/legacy-tempest-dsvm-neutron-full/e6ac116/logs/screen-n-cpu.txt.gz#_Dec_01_14_06_49_14297716:08
*** vks1 has joined #openstack-nova16:08
*** slaweq has quit IRC16:09
*** sahid has quit IRC16:09
*** udesale has joined #openstack-nova16:10
*** Alex_Staf has quit IRC16:10
*** sree has quit IRC16:11
*** imacdonn has quit IRC16:11
*** imacdonn has joined #openstack-nova16:11
*** READ10 has quit IRC16:14
*** thorst has joined #openstack-nova16:15
lyarwoodmriedem: looking16:17
lyarwoodmriedem: so that's normal, search for cryptsetup below that and you can see we format and then open it correctly16:19
mriedemlyarwood: :(16:19
*** pchavva has quit IRC16:19
mriedemi guess that's why it's a stacktrace at debug rather than error?16:19
mriedemwhich is i think a thing i've seen with privsep before16:19
lyarwoodhttps://github.com/openstack/os-brick/blob/master/os_brick/encryptors/luks.py#L150-L16116:20
*** thorst has quit IRC16:20
lyarwoodmriedem: yeah I think so16:20
*** erlon has quit IRC16:22
*** david-lyle has joined #openstack-nova16:22
openstackgerritEdward Hope-Morley proposed openstack/nova-specs master: Add spec for extending Aggregate scheduling capabilties.  https://review.openstack.org/52466416:22
*** thorst has joined #openstack-nova16:24
*** jangutter has quit IRC16:25
cdentfried_rice: next time you make a bug like https://bugs.launchpad.net/nova/+bug/1735430 please feel free to add the placement and/or resource-tracker tags to it, for sake of search visibility (I’ve done it for that one)16:25
openstackLaunchpad bug 1735430 in OpenStack Compute (nova) pike "Report client doesn't handle RP create conflict (409) properly" [Medium,In progress] - Assigned to Eric Fried (efried)16:25
*** armax has quit IRC16:27
fried_ricecdent Okay, I did ask mriedem about that yesterday and he didn't seem to think it mattered.  The question specifically was: do we need the `placement` tag on stuff like this or just on things that are server-side?16:28
*** thorst has quit IRC16:28
cdent“need” is a big word that probably doesn’t fit here, but “nice” is handy from the visibility side. I tend to think it’s a useful tag for anything that’s interacting with the placement service (or is the service) and I use it for finding stuff for the report (which is why it came up just now)16:29
mriedemthis is kind of funny https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/attach_interfaces.py#L15016:31
mriedemhere, let me convert this 500 to a 500 for you16:31
cdentsigh16:31
mriedem@extensions.expected_errors((400, 404, 409, 500, 501))16:31
*** lajoskatona has quit IRC16:32
cdentwell that was 5 years ago, we’ve gotten a lot more verbose about what’s right since then16:32
*** yamahata has quit IRC16:33
mriedemoh i think i might know why it's doing this,16:35
mriedembecause we don't "expect" this exception over rpc, so what you get back is the InstanceAttachFailed_Remote error, which hides the original error message16:36
*** thorst has joined #openstack-nova16:41
*** slaweq has joined #openstack-nova16:42
*** slaweq_ has joined #openstack-nova16:42
*** beekneemech has quit IRC16:42
fried_riceQuick poll: CI logs closer to the coloration you would see from the console: http://184.172.12.213/16/524316/1/check/nova-out-of-tree-pvm/f0d1b76/logs/n-cpu.txt.gz16:44
cdentno16:45
*** bnemec has joined #openstack-nova16:45
fried_ricecdent Care to elaborate?16:45
cdenta) I don’t like black backgrounds, b) neon colors against black backgrounds are scary, I (for real) flinched when that page loaded16:46
superdanfried_rice: that's fairly hard to read, IMHO16:46
*** slaweq has quit IRC16:46
cdentfried_rice: if you’re after changing the colors, then perhaps something in the solarized pallette might be nice, but a) I don’t reckon the existing norm is far off from okay, b) I turn off colorization wherever possible so I’m not the target audience so ignore most of what I say on this topic because I’m an old grape in the path of the steamroller16:48
fried_riceOkay, thanks cdent and superdan.  FYI, where I'm actually going with this is the ability to interpret ANSI color codes in the HTMLification.  The idea was so they would look exactly the same as they do on the console.  I ought to have a sample of that by this afternoon.16:49
superdanfried_rice: not everyone has their console configured the same, you know16:49
*** chyka has joined #openstack-nova16:49
superdanI don't use a dark background for example16:49
cdentyeah, personally I’d prefer that the logs never have ansi codes in them16:49
cdentbut I think systemd’s journalctl doesn’t want me to feel that way16:50
superdanI also use the solarized theme for all of this kind of ting, which is super low impact, especially compared to what you've got here16:50
fried_ricecdent You can make that happen by leaving -a off of journalctl16:50
fried_ricesuperdan What is this "solarized" theme?16:50
*** thorst has quit IRC16:50
superdanfried_rice: http://ethanschoonover.com/solarized16:50
cdentfried_rice: right, but my point is that if I download a log from CI, it should _not_ be doctored16:50
cdentif the CI is including ansi, then I end up with a mess16:51
fried_ricecdent Hum, right.  See, to me that's what I *do* want, which is why I started this whole thing.16:51
cdent(see b) abve)16:51
*** fragatina has joined #openstack-nova16:52
fried_riceI only know how to make code-included logs show up colorized.  But there must be a way to just strip out the color codes.16:52
cdentansi escape sequences in log strings strikes me as some kind of weird punishment16:52
cdentprobably16:52
fried_riceOh, jeez, I feel they make it WAY more readable.16:52
*** andreas_s has joined #openstack-nova16:53
superdanthey do sometimes, but other times they get in the way depending on how you're looking at it16:53
superdanif I need to open it in an editor or something to apply a macro or something,16:53
superdanthen it's a total mess16:53
*** itlinux has quit IRC16:53
superdanif I can have the option, then cool. if I have no option, I'd lean towards clean logs16:53
*** yamamoto has quit IRC16:54
*** yamamoto has joined #openstack-nova16:56
*** claudiub has quit IRC16:57
*** weshay is now known as weshay_mtg16:57
*** andreas_s has quit IRC16:57
*** yamamoto has quit IRC17:01
*** pchavva has joined #openstack-nova17:02
*** fragatina has quit IRC17:03
*** fragatina has joined #openstack-nova17:03
openstackgerritMatthew Booth proposed openstack/nova master: Fix disk size during live migration with disk over-commit  https://review.openstack.org/52468117:03
figleafmriedem: just got back from having my teeth scraped. Reading back, yeah, 409 seems like the best choice17:06
cdentwhen you say it like that, it gives me the fingernails chalkboard sensation, or the styrofoam on cardboard things. thank you very little17:09
mriedemi had that done on tuesday17:09
mriedemso refreshing17:09
*** yamahata has joined #openstack-nova17:09
*** suresh12 has joined #openstack-nova17:09
*** Alex_Staf has joined #openstack-nova17:11
*** armax has joined #openstack-nova17:11
*** AlexeyAbashkin has joined #openstack-nova17:13
*** fried_rice is now known as fried_rolls17:14
*** Alex_Staf has quit IRC17:15
*** slaweq_ has quit IRC17:15
*** slaweq has joined #openstack-nova17:16
*** AlexeyAbashkin has quit IRC17:17
*** udesale has quit IRC17:17
*** udesale has joined #openstack-nova17:18
*** itlinux has joined #openstack-nova17:20
*** slaweq has quit IRC17:20
*** thorst has joined #openstack-nova17:21
*** tidwellr has joined #openstack-nova17:22
*** lucasagomes is now known as lucas-afk17:24
mriedemoh shelve http://lists.openstack.org/pipermail/openstack-dev/2017-December/125124.html17:25
mriedemyou'll be the death of me17:25
mriedemalaski: ^ maybe you have ideas17:25
mriedemfrom 201317:25
*** thorst has quit IRC17:26
cdentshelve-tastic17:27
*** derekh has quit IRC17:28
*** thorst has joined #openstack-nova17:28
*** penick has joined #openstack-nova17:30
*** amodi has quit IRC17:30
*** suresh12 has quit IRC17:30
mriedemfried_rolls: failure here looks real17:31
mriedemhttps://review.openstack.org/#/c/524618/17:31
mriedemhttp://logs.openstack.org/18/524618/1/check/openstack-tox-functional/81f3dd8/testr_results.html.gz17:31
mriedemcur_rp_gen = self._resource_providers[rp_uuid]['generation'] TypeError: 'NoneType' object has no attribute '__getitem__'17:31
*** ratailor has quit IRC17:32
mriedemsuperdan: want to push this through stable? else i will https://review.openstack.org/#/c/50996117:32
superdannot really, but I will17:33
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Fix incorrect known vcpuset when CPUPinningUnknown raised  https://review.openstack.org/52364417:33
*** thorst has quit IRC17:33
*** thorst has joined #openstack-nova17:34
*** AlexeyAbashkin has joined #openstack-nova17:37
*** artom has quit IRC17:37
mriedemjohnthetubaguy: bauzas: lyarwood: if you're still around, we should get these in https://review.openstack.org/#/q/I1a46ef1503be2febcd20f4594f44344d0552544617:39
mriedemfixes a regression introduced in a cve fix from a week or two ago17:39
*** bnemec has quit IRC17:41
*** AlexeyAbashkin has quit IRC17:41
*** slaweq has joined #openstack-nova17:42
*** Apoorva has quit IRC17:42
*** artom has joined #openstack-nova17:43
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Fix TypeError of _get_project_id when project_id is None  https://review.openstack.org/52468717:45
openstackgerritMatt Riedemann proposed openstack/nova stable/pike: Vzstorage: synchronize volume connect  https://review.openstack.org/52468817:46
*** udesale has quit IRC17:46
johnthetubaguymriedem: seems important, +217:50
mriedemthanks17:51
*** vks1 has quit IRC17:52
*** slaweq has quit IRC17:52
*** slaweq has joined #openstack-nova17:53
alaskimriedem: so the idea with unshelving was that it should look like nothing had happened. So you boot instance from image A, image_ref points at A. Resize/migration should work. You shelve/offload/unshelve and have an instance running again, with image_ref A. It's supposed to be like nothing happened so resize/migration should still work. What's unclear to me, since I've been out of the loop for a bit,17:54
alaskiis why resize/migration fails specifically after an unshelve.17:54
*** suresh12 has joined #openstack-nova17:56
alaskiI think the argument against updating image_ref was that something like a rebuild should use the original image, not the snapshot. But outside of that I don't recall any reasons not to update the image_ref17:57
*** slaweq has quit IRC17:57
mriedemok so create with image A, shelve to create image B, unshelve with image B, rebuild with image A again17:58
*** yamamoto has joined #openstack-nova17:58
*** suresh12 has quit IRC17:58
alaskiyeah. image B is an artifact of shelving but from a user point of view it never really exists.17:58
mriedemhmm, except it does, until we delete it17:59
alaskishelve/unshelve ideally has the vm exactly as it was before shelving, no updates to the vm at all.17:59
mriedemanywho, thanks, didn't think about the rebuild thing, but this all seems very slight of hand and confusing17:59
*** Guest67871 has quit IRC17:59
mriedemwell,18:00
mriedemwe still don't do that properly with volumes and ports completely i don't think18:00
alaskiyes, that whole feature is very much slight of hand and confusing18:00
*** dtantsur is now known as dtantsur|afk18:00
mriedemi will think on those over a 1000 calorie burrito18:00
*** sambetts is now known as sambetts|afk18:00
alaski"ideally" that's how shelve works, in practice it clearly does not. I won't claim that the ideal is achievable, so it may be better to be pragmatic and just document it well.18:01
*** david-lyle has quit IRC18:02
alaskimaybe you'll come out of your food coma with an epiphany18:02
*** david-lyle has joined #openstack-nova18:04
*** weshay_mtg is now known as weshay_bbiab18:05
*** Apoorva has joined #openstack-nova18:05
*** sree has joined #openstack-nova18:07
*** Apoorva has quit IRC18:07
*** yamamoto has quit IRC18:08
openstackgerritPeter Penchev proposed openstack/nova master: Add the Nova libvirt StorPool attachment driver.  https://review.openstack.org/14073318:08
*** sree has quit IRC18:11
*** Apoorva has joined #openstack-nova18:14
*** gaoyan has joined #openstack-nova18:16
*** gaoyan has quit IRC18:18
*** gyee has joined #openstack-nova18:18
*** amodi has joined #openstack-nova18:19
*** rabel has joined #openstack-nova18:21
*** jpena is now known as jpena|off18:21
*** slaweq has joined #openstack-nova18:21
*** gbarros has quit IRC18:22
rabelhi there. i need your thoughts on my last comment to https://review.openstack.org/#/c/402632/12 :) . it is about the datastore selection in vmware driver.18:22
*** dtruong has joined #openstack-nova18:25
*** slaweq has quit IRC18:26
cdentrabel: I suspect rgerganov is going to be best placed to answer that and he won’t be back around until Monday, but I can try to make sure he sees it18:27
*** dtruong_ has quit IRC18:28
rabelcdent: that would be great. thank you18:29
*** suresh12 has joined #openstack-nova18:29
cdent18:29
*** READ10 has joined #openstack-nova18:29
*** suresh12 has quit IRC18:29
*** suresh12_ has joined #openstack-nova18:30
*** mkoderer_ has joined #openstack-nova18:41
*** damien_r has joined #openstack-nova18:46
*** moshele has joined #openstack-nova18:48
*** rabel has quit IRC18:51
*** moshele has quit IRC18:52
*** Apoorva has quit IRC19:04
*** david-lyle has quit IRC19:05
*** moshele has joined #openstack-nova19:06
*** Apoorva has joined #openstack-nova19:07
*** damien_r has quit IRC19:07
*** gszasz has quit IRC19:13
openstackgerritMerged openstack/nova stable/pike: Log consumer uuid when retrying claims in the scheduler  https://review.openstack.org/50996119:15
openstackgerritBrianna Poulos proposed openstack/nova master: Add trusted_certs object  https://review.openstack.org/48940819:16
openstackgerritBrianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API  https://review.openstack.org/48620419:16
*** READ10 has quit IRC19:18
*** david-lyle has joined #openstack-nova19:21
*** sdaniel_ has joined #openstack-nova19:23
sdaniel_Hi, I need some help about nova services :(19:23
*** gbarros has joined #openstack-nova19:26
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.57 - Add trusted_image_certificates  https://review.openstack.org/50039619:29
*** gbarros has quit IRC19:29
openstackgerritChris Dent proposed openstack/nova master: [placement] re-use existing conf with auth token middleware  https://review.openstack.org/52340319:29
cdentmriedem: I came up with a somewhat different approach for the test of that ^ because the middleware is very kwargy instead of a attrtibutey19:30
*** gbarros has joined #openstack-nova19:32
*** fragatina has quit IRC19:33
sdaniel_I had to reconfigurate the network type from provider to self-service19:34
sdaniel_and after that know of my nova services won't come up19:34
*** gbarros has quit IRC19:34
sdaniel_(after i restarted them manually)19:34
sdaniel_I re-created the whole nova DBs and re-installed the nova packages, but nothing helped19:35
*** moshele has quit IRC19:36
*** suresh12_ has quit IRC19:36
*** suresh12 has joined #openstack-nova19:36
sdaniel_I cant restart the services19:36
sdaniel_I got this error in syslog: TypeError: __init__() got an unexpected keyword argument 'retry_on_request'19:36
*** dklyle has joined #openstack-nova19:36
*** suresh12 has quit IRC19:36
*** david-lyle has quit IRC19:37
*** suresh12 has joined #openstack-nova19:37
sdaniel_Only just nova cant restart, neutron can19:37
*** moshele has joined #openstack-nova19:41
*** leakypipes has quit IRC19:44
openstackgerritMerged openstack/nova stable/pike: Refined fix for validating image on rebuild  https://review.openstack.org/52321219:50
*** sdaniel_ has quit IRC19:51
openstackgerritMerged openstack/python-novaclient master: Move zuulv3 jobs to project repo  https://review.openstack.org/52209919:56
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.57 - Add trusted_image_certificates  https://review.openstack.org/50039619:56
*** dklyle has quit IRC19:57
openstackgerritMerged openstack/nova stable/ocata: Refined fix for validating image on rebuild  https://review.openstack.org/52342719:58
openstackgerritMerged openstack/nova stable/newton: Refined fix for validating image on rebuild  https://review.openstack.org/52343419:58
*** rodolof has quit IRC19:59
*** rodolof has joined #openstack-nova19:59
*** weshay_bbiab is now known as weshay20:06
*** sree has joined #openstack-nova20:08
*** damien_r has joined #openstack-nova20:09
*** purplerbot has quit IRC20:09
*** moshele has quit IRC20:09
*** AlexeyAbashkin has joined #openstack-nova20:11
*** weshay is now known as weshay_interview20:12
*** sree has quit IRC20:12
*** AlexeyAbashkin has quit IRC20:15
*** thorst has quit IRC20:19
*** slaweq has joined #openstack-nova20:23
*** fried_rolls is now known as fried_rice20:24
*** pcaruana has quit IRC20:25
*** itlinux has quit IRC20:25
fried_ricemriedem Yup, was just starting to look at that before "lunch".  Should have it fixed here shortly.20:25
*** itlinux has joined #openstack-nova20:26
*** slaweq has quit IRC20:27
*** penick has quit IRC20:32
*** suresh12 has quit IRC20:33
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Change 'InstancePCIRequest' spec field  https://review.openstack.org/44925720:34
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request  https://review.openstack.org/45177720:34
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions  https://review.openstack.org/46605120:34
superdanfried_rice: I hope "lunch" was something super well-deserving of quotes20:35
fried_ricesuperdan Tapping out people bigger and stronger than me.  The actual eating is happening now, at my desk.20:35
*** moshele has joined #openstack-nova20:35
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API  https://review.openstack.org/47661220:36
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API  https://review.openstack.org/48222620:36
mriedemi hope you're not eating the aforementioned tapped out people20:39
coffee_catlol :)20:40
*** penick has joined #openstack-nova20:41
cdent“tapped out people and fried_rice”, it’s what’s for dinner20:41
fried_riceYou could say I ate them for lunch.20:41
*** gbarros has joined #openstack-nova20:42
* cdent eats a bagel20:44
*** thorst has joined #openstack-nova20:50
*** ralonsoh has quit IRC20:50
*** moshele has quit IRC20:52
*** purplerbot has joined #openstack-nova20:52
openstackgerritMatt Riedemann proposed openstack/nova master: WIP: Store shelved snapshot image ref with instance after unshelve  https://review.openstack.org/52472620:54
*** moshele has joined #openstack-nova20:54
*** tidwellr has quit IRC20:54
*** tidwellr has joined #openstack-nova20:54
*** thorst has quit IRC20:55
*** sree has joined #openstack-nova20:57
openstackgerritMatt Riedemann proposed openstack/nova master: Check the return code when forcing TCG mode with libguestfs  https://review.openstack.org/52472720:58
*** moshele has quit IRC20:59
*** sree has quit IRC21:02
mriedemcoffee_cat: already found a problem which i didn't think about before :)21:05
mriedembut easy fix21:05
coffee_catmriedem: I would've been disappointed otherwise :)21:05
mriedemi don't think we supported multi-cell when this was written21:05
mriedembut now, we do21:06
mriedembigly21:06
mriedemdays since multi-cell multi-attach accidents: 021:06
coffee_catI certainly don't recall talking about multi-cell in this context before21:06
coffee_catnot sure about my feelings now that we do... :)21:07
*** ianw has joined #openstack-nova21:09
*** slaweq has joined #openstack-nova21:10
*** AlexeyAbashkin has joined #openstack-nova21:11
*** moshele has joined #openstack-nova21:11
*** AlexeyAbashkin has quit IRC21:15
*** linkmark has joined #openstack-nova21:18
*** weshay_interview is now known as weshay21:19
*** ijw has joined #openstack-nova21:19
*** thorst has joined #openstack-nova21:20
*** rcernin has joined #openstack-nova21:22
*** smatzek_ has quit IRC21:25
*** thorst has quit IRC21:25
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.56 - Enable cold migration with target host  https://review.openstack.org/40670721:28
openstackgerritJackie Truong proposed openstack/python-novaclient master: Microversion 2.57 - Add trusted_image_certificates  https://review.openstack.org/50039621:28
*** thorst has joined #openstack-nova21:31
*** damien_r has quit IRC21:34
*** ociuhandu has joined #openstack-nova21:34
*** moshele has quit IRC21:34
fried_ricesuperdan This in your wheelhouse?  https://review.openstack.org/#/c/523932/21:36
*** thorst has quit IRC21:36
openstackgerritEric Fried proposed openstack/nova stable/pike: Proper error handling by _ensure_resource_provider  https://review.openstack.org/52461821:37
fried_ricemriedem ^  giblet move the placement fixture in master presumably for a similar reason.  (Actually he duplicated it - we should remove the second one.)21:38
*** ociuhandu has quit IRC21:40
openstackgerritEric Fried proposed openstack/nova master: Trivial: Nix duplicate PlacementFixture() in test  https://review.openstack.org/52473421:41
fried_ricemriedem Fixed that ^21:41
mriedemcoffee_cat: https://review.openstack.org/#/c/330285/16521:44
coffee_catmriedem: cool, thanks21:46
coffee_catmriedem: I will get up a next version by Monday (I hope :)21:47
mriedemi don't think anything i pointed out is particularly hard to deal with21:47
mriedemi pointed out a chance to break some of this out into a separate patch too, if so desired21:47
coffee_catyeah, looks fine21:48
coffee_catif possible I would love not to maintain a longer chain21:49
mriedemi know,21:49
mriedembut at +1251 LOC,21:49
*** thorst has joined #openstack-nova21:49
*** dtruong has quit IRC21:49
coffee_catbut that's mostly tests21:49
mriedemit becomes very hard to spot regressions and test coverage21:49
mriedemi made comments in the tests too :)21:49
mriedemanyway, i would understand if you don't want to split parts of it up at this point21:50
mriedemi said that in there too21:50
coffee_catcoolio :)21:50
coffee_catI will check what you would split out and do it if it's not that tough21:50
coffee_catI will hate doing it if I choose that option and will think of you while hate doing it21:51
coffee_catso expect some hiccups :)21:52
coffee_catthanks for the first round of review, I will do my best to fix it quickly21:53
*** sdaniel has joined #openstack-nova21:54
mriedemi expect most people to think of their hate for me when they are doing things21:54
coffee_catin most cases I don't21:55
coffee_catI save it for special occasions :)21:55
coffee_catand at this point I hate the patch itself anyway...21:58
*** smatzek has joined #openstack-nova21:59
coffee_catmriedem: have a great weekend! you deserve it now :)21:59
*** smatzek has quit IRC22:00
*** smatzek has joined #openstack-nova22:00
*** smatzek has quit IRC22:01
*** smatzek has joined #openstack-nova22:01
mriedemo/22:02
*** thorst has quit IRC22:03
*** tidwellr has quit IRC22:04
*** tidwellr has joined #openstack-nova22:05
*** amodi has quit IRC22:06
*** smatzek has quit IRC22:06
*** smatzek has joined #openstack-nova22:07
*** flwang has joined #openstack-nova22:07
*** fragatina has joined #openstack-nova22:09
*** tidwellr has quit IRC22:09
*** smatzek has quit IRC22:10
*** smatzek has joined #openstack-nova22:10
*** smatzek has quit IRC22:14
*** penick has quit IRC22:14
*** smatzek has joined #openstack-nova22:14
*** smatzek has quit IRC22:19
*** penick has joined #openstack-nova22:20
*** smatzek has joined #openstack-nova22:22
*** slaweq_ has joined #openstack-nova22:23
*** slaweq_ has quit IRC22:23
*** slaweq_ has joined #openstack-nova22:24
*** smatzek has quit IRC22:24
*** rodolof has quit IRC22:24
*** smatzek has joined #openstack-nova22:24
*** slaweq_ has quit IRC22:26
*** ijw has quit IRC22:27
*** slaweq_ has joined #openstack-nova22:27
*** smatzek has quit IRC22:29
*** slaweq_ has quit IRC22:30
*** slaweq_ has joined #openstack-nova22:30
*** slaweq_ has quit IRC22:30
fried_ricesuperdan cdent Check this out: http://184.172.12.213/manual/htmlify_logs/logs/n-cpu.txt.gz?theme=solarized and http://184.172.12.213/manual/htmlify_logs/logs/n-cpu.txt.gz?theme=monochrome22:32
*** slaweq_ has joined #openstack-nova22:35
*** smatzek has joined #openstack-nova22:36
*** smatzek has quit IRC22:36
*** pchavva has quit IRC22:38
*** dtruong has joined #openstack-nova22:39
*** thorst has joined #openstack-nova22:40
*** sree has joined #openstack-nova22:40
*** slaweq_ has quit IRC22:42
*** slaweq_ has joined #openstack-nova22:43
openstackgerritMatt Riedemann proposed openstack/nova master: Fix guestfs.set_backend_settings call  https://review.openstack.org/52474822:43
*** sree has quit IRC22:45
*** thorst has quit IRC22:45
*** slaweq_ has quit IRC22:46
*** gbarros has quit IRC22:47
*** slaweq_ has joined #openstack-nova22:48
*** figleaf is now known as edleafe22:48
*** cdent has quit IRC22:48
*** felipemonteiro_ has quit IRC22:48
*** awaugama has quit IRC22:49
*** slaweq_ has quit IRC22:49
*** slaweq_ has joined #openstack-nova22:51
*** slaweq_ has quit IRC22:53
*** slaweq_ has joined #openstack-nova22:53
*** slaweq_ has quit IRC22:54
*** slaweq has quit IRC22:54
*** slaweq_ has joined #openstack-nova22:54
*** slaweq_ is now known as slaweq22:55
*** itlinux has quit IRC22:59
*** fried_rice is now known as efried22:59
*** slaweq has quit IRC23:03
*** slaweq has joined #openstack-nova23:04
*** slaweq has quit IRC23:04
*** slaweq has joined #openstack-nova23:05
*** suresh12 has joined #openstack-nova23:06
openstackgerritHongbin Lu proposed openstack/nova master: Add support for listing hosts in cellv2  https://review.openstack.org/52475523:08
*** itlinux has joined #openstack-nova23:08
mriedemsuperdan: i think we've talked about this before maybe, but is there any reason why we don't have a configurable rpc timeout for the pre_live_migration call from source compute to dest compute during live migrate?23:10
mriedemso you could turn that up specifically rather than the global rpc_response_timeout?23:10
*** thorst has joined #openstack-nova23:14
*** thorst has quit IRC23:19
*** hoonetorg has quit IRC23:19
*** slaweq has quit IRC23:21
*** lyan has quit IRC23:22
*** Guest78_ has quit IRC23:23
*** itlinux has quit IRC23:28
*** mriedem has quit IRC23:29
*** penick has quit IRC23:30
*** amodi has joined #openstack-nova23:32
*** salv-orlando has quit IRC23:37
*** salv-orlando has joined #openstack-nova23:38
*** salv-orlando has quit IRC23:42
*** thorst has joined #openstack-nova23:45
*** sdaniel has quit IRC23:47
*** thorst has quit IRC23:50
*** ijw has joined #openstack-nova23:53

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