Tuesday, 2016-01-19

*** zz_dimtruck is now known as dimtruck00:02
*** frewtc4ke has left #openstack-nova00:05
boris-42dansmith: hi00:05
boris-42dansmith: are you around?00:05
*** annegentle has quit IRC00:06
*** lucas-dinner has quit IRC00:06
*** salv-orlando has quit IRC00:07
*** Jeffrey4l has joined #openstack-nova00:07
*** lucasagomes has joined #openstack-nova00:11
*** salv-orlando has joined #openstack-nova00:13
*** pradk has quit IRC00:14
*** amotoki has quit IRC00:14
*** eil397 has joined #openstack-nova00:15
*** diogogmt has quit IRC00:15
*** dslevin_ has joined #openstack-nova00:17
*** dslevin_ has quit IRC00:17
*** otter768 has quit IRC00:18
*** dslevin has quit IRC00:18
*** salv-orlando has quit IRC00:18
*** dslevin has joined #openstack-nova00:18
*** haomaiwang has joined #openstack-nova00:22
*** amotoki has joined #openstack-nova00:26
prometheanfirejroll: not all the same :P00:26
*** mgoddard__ has joined #openstack-nova00:26
jrollprometheanfire: :D00:27
*** haomaiwang has quit IRC00:27
*** mtanino_ has joined #openstack-nova00:27
*** tjones has joined #openstack-nova00:28
*** zenoway has joined #openstack-nova00:29
*** dimtruck is now known as zz_dimtruck00:29
*** mtanino has quit IRC00:29
*** mgoddard_ has quit IRC00:29
*** annegentle has joined #openstack-nova00:32
*** zenoway has quit IRC00:33
*** pratikmallya has joined #openstack-nova00:35
*** dslevin has quit IRC00:35
*** dslevin has joined #openstack-nova00:38
*** jschlueter has joined #openstack-nova00:38
*** akshai_ has joined #openstack-nova00:40
*** achanda has joined #openstack-nova00:41
*** achanda has quit IRC00:43
*** akshai has quit IRC00:43
*** achanda has joined #openstack-nova00:43
*** achanda has quit IRC00:44
*** davideagnello has quit IRC00:44
*** achanda has joined #openstack-nova00:45
*** shyama has quit IRC00:46
*** ljxiash has quit IRC00:47
*** achanda has quit IRC00:47
*** Sukhdev has quit IRC00:48
*** ducttape_ has joined #openstack-nova00:48
*** dslevin has quit IRC00:48
openstackgerritIldiko Vancsa proposed openstack/nova: Pass attachment_id to Cinder when detach a volume  https://review.openstack.org/19313400:48
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: comp: Allow volume multiattach  https://review.openstack.org/26758700:48
openstackgerritIldiko Vancsa proposed openstack/nova: Pass host when call attach to Cinder  https://review.openstack.org/25627300:48
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: libvirt: Allow multiple volume attachments  https://review.openstack.org/19313500:50
*** jnclyz12483 has joined #openstack-nova00:50
*** achanda has joined #openstack-nova00:50
*** dslevin has joined #openstack-nova00:51
ildikovjaypipes: fixes are up :) ^00:51
*** ducttape_ has quit IRC00:52
jaypipesildikov: cool. will rereview in a bit. thx!00:55
*** achanda has quit IRC00:55
ildikovjaypipes: cool, tnx00:56
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach  https://review.openstack.org/15303800:57
*** diogogmt has joined #openstack-nova00:58
*** fawadkhaliq has joined #openstack-nova01:00
*** achanda has joined #openstack-nova01:00
*** fawadkhaliq has quit IRC01:00
*** achanda has quit IRC01:01
*** yamahata has quit IRC01:01
*** Jeffrey4l has quit IRC01:01
*** dims has quit IRC01:01
*** angdraug has quit IRC01:03
*** eil397 has quit IRC01:03
*** pratikmallya has quit IRC01:03
*** achanda has joined #openstack-nova01:05
*** jinxing has joined #openstack-nova01:05
*** vishwanathj has joined #openstack-nova01:06
*** jacky-zhang has joined #openstack-nova01:06
*** vishwanathj has quit IRC01:07
*** eil397 has joined #openstack-nova01:09
*** ivase has joined #openstack-nova01:09
*** thorst has joined #openstack-nova01:11
*** vishwanathj has joined #openstack-nova01:12
*** jacky-zhang_ has joined #openstack-nova01:12
*** jnclyz12483 has quit IRC01:12
*** ZZelle_ has quit IRC01:12
*** thorst_ has joined #openstack-nova01:13
*** jacky-zhang has quit IRC01:13
openstackgerritZhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API  https://review.openstack.org/24898901:13
*** jacky-zhang_ is now known as jacky-zhang01:13
*** ivase has quit IRC01:14
*** markvoelker has quit IRC01:14
ildikovjaypipes: if you can check it now I can fix if there's anything, otherwise that will be tomorrow, I need some sleep :)01:16
*** thorst has quit IRC01:16
jaypipesildikov: it will take time, plus need to wait for a clean bill of health. let's touch base in my morning (EST timezone around 7:30am)01:16
ildikovjaypipes: ok, cool, thanks much01:17
*** achanda has quit IRC01:17
jaypipesildikov: sleep tight :)01:17
ildikovjaypipes: have a nice evening :)01:18
*** ljxiash has joined #openstack-nova01:19
*** achanda has joined #openstack-nova01:21
*** pradk has joined #openstack-nova01:23
*** haomaiwa_ has joined #openstack-nova01:23
*** dims has joined #openstack-nova01:23
*** liverpoo1er has joined #openstack-nova01:24
*** baoli has joined #openstack-nova01:24
*** liverpooler has quit IRC01:26
*** achanda has quit IRC01:26
*** haomaiwa_ has quit IRC01:28
*** baoli has quit IRC01:29
*** dslevin has quit IRC01:31
*** ljxiash_ has joined #openstack-nova01:35
*** thorst_ has quit IRC01:35
*** rfolco has joined #openstack-nova01:36
*** dslevin has joined #openstack-nova01:36
*** baoli has joined #openstack-nova01:38
*** ljxiash has quit IRC01:38
*** ccard_ has quit IRC01:42
openstackgerritTracy Jones proposed openstack/nova: Stop using mox stubs in test_configdrive  https://review.openstack.org/26547601:42
*** baoli has quit IRC01:43
*** lyanchih has joined #openstack-nova01:43
*** zhangjn has joined #openstack-nova01:45
openstackgerritTracy Jones proposed openstack/nova: change set_stubs to use stub_out in vmwareapi/stubs.py  https://review.openstack.org/26606001:45
*** rfolco has quit IRC01:46
*** ccard_ has joined #openstack-nova01:46
*** eil397 has quit IRC01:47
*** sc68cal has quit IRC01:49
*** sc68cal has joined #openstack-nova01:54
*** Jeffrey4l has joined #openstack-nova01:56
*** dslevin has quit IRC01:56
openstackgerritVladik Romanovsky proposed openstack/nova: neutron: pci_request logic considers 'direct-physical' vnic type  https://review.openstack.org/26258401:58
openstackgerritVladik Romanovsky proposed openstack/nova: pci: Add utility method for getting the MAC addr  https://review.openstack.org/26258501:58
openstackgerritVladik Romanovsky proposed openstack/nova: neutron: Update the port with a MAC address for PFs  https://review.openstack.org/26258601:58
openstackgerritVladik Romanovsky proposed openstack/nova: pci: changing the claiming and allocation logic for PF/VF assignment  https://review.openstack.org/24901801:58
openstackgerritVladik Romanovsky proposed openstack/nova: Add a vnic type for PF passthrough and a new libvirt vif driver  https://review.openstack.org/26258301:58
*** apoorvad has quit IRC02:00
*** diogogmt has quit IRC02:00
*** xiexs has quit IRC02:01
*** unicell has joined #openstack-nova02:04
*** unicell1 has quit IRC02:04
*** mtanino_ has quit IRC02:04
*** vladikr has joined #openstack-nova02:05
*** diogogmt has joined #openstack-nova02:06
*** dslevin has joined #openstack-nova02:07
*** dslev has joined #openstack-nova02:08
*** sc68cal has quit IRC02:09
*** sc68cal has joined #openstack-nova02:12
*** unicell has quit IRC02:17
*** otter768 has joined #openstack-nova02:18
*** unicell has joined #openstack-nova02:20
*** liverpooler has joined #openstack-nova02:22
*** david-lyle has joined #openstack-nova02:22
*** liverpoo1er has quit IRC02:23
*** otter768 has quit IRC02:23
*** haomaiwa_ has joined #openstack-nova02:26
*** tjones has quit IRC02:29
*** dslev has quit IRC02:31
*** thorst has joined #openstack-nova02:36
*** houming has joined #openstack-nova02:43
*** thorst has quit IRC02:44
*** davideagnello has joined #openstack-nova02:45
*** burgerk has joined #openstack-nova02:45
*** zhangjn has quit IRC02:46
*** mdnadeem has joined #openstack-nova02:48
*** ducttape_ has joined #openstack-nova02:48
*** davideagnello has quit IRC02:51
*** markvoelker has joined #openstack-nova02:52
*** ducttape_ has quit IRC02:53
openstackgerritGhanshyam Mann proposed openstack/nova: WIP-Enable all extension for server API sample tests  https://review.openstack.org/24339602:54
*** burgerk has quit IRC02:54
*** markvoelker_ has joined #openstack-nova02:56
openstackgerritZhenyu Zheng proposed openstack/nova: config options: Centralise floating ip options  https://review.openstack.org/25708702:56
openstackgerritGhanshyam Mann proposed openstack/nova: WIP-Enable all extension for server API sample tests  https://review.openstack.org/26940402:58
*** e0ne has quit IRC02:59
openstackgerritGhanshyam Mann proposed openstack/nova: Enable all extension for Flavor API sample tests  https://review.openstack.org/24339603:00
*** otter768 has joined #openstack-nova03:00
*** e0ne has joined #openstack-nova03:00
*** markvoelker has quit IRC03:00
*** markvoelker_ has quit IRC03:01
*** e0ne has quit IRC03:01
openstackgerritZhenyu Zheng proposed openstack/nova: Expose quiesce/unquiesce API  https://review.openstack.org/24898903:01
*** ducttape_ has joined #openstack-nova03:04
*** markvoelker has joined #openstack-nova03:06
*** Marga_ has joined #openstack-nova03:10
*** vishwanathj has quit IRC03:10
openstackgerritZhenyu Zheng proposed openstack/nova: Add expected_task_state for instance.save() used by block_device_mapping  https://review.openstack.org/24503803:12
*** ljxiash_ has quit IRC03:14
*** ljxiash has joined #openstack-nova03:14
openstackgerritZhenyu Zheng proposed openstack/nova: Add checks for volume status when rebuilding  https://review.openstack.org/21756703:16
*** ljxiash has quit IRC03:19
*** dims has quit IRC03:20
*** achanda has joined #openstack-nova03:21
*** Sukhdev has joined #openstack-nova03:22
*** baoli has joined #openstack-nova03:24
*** links has joined #openstack-nova03:27
*** ccard_ has quit IRC03:31
*** tjones has joined #openstack-nova03:31
*** ccard_ has joined #openstack-nova03:32
*** ducttape_ has quit IRC03:34
*** sixdoublefive321 has joined #openstack-nova03:36
*** liverpoo1er has joined #openstack-nova03:36
*** mragupat has joined #openstack-nova03:37
*** liverpooler has quit IRC03:38
*** zhangjn has joined #openstack-nova03:38
*** vladikr has quit IRC03:41
openstackgerritEli Qiao proposed openstack/nova: Libvirt: Move graphics checking and console verify early when LM  https://review.openstack.org/26504203:41
*** ljxiash has joined #openstack-nova03:41
*** ljxiash has quit IRC03:42
*** thorst has joined #openstack-nova03:42
*** ljxiash has joined #openstack-nova03:42
*** ducttape_ has joined #openstack-nova03:43
openstackgerritYingxin Cheng proposed openstack/nova: Use stevedore for scheduler host manager  https://review.openstack.org/24647603:45
*** sixdoublefive321 has quit IRC03:45
*** ljxiash has quit IRC03:47
*** davideagnello has joined #openstack-nova03:48
*** zhangjn has quit IRC03:49
*** thorst has quit IRC03:50
*** yamahata has joined #openstack-nova03:52
*** davideagnello has quit IRC03:52
*** zz_dimtruck is now known as dimtruck03:52
*** ducttape_ has quit IRC03:54
*** tjones has quit IRC03:54
*** achanda has quit IRC04:00
*** jacky-zhang has quit IRC04:01
*** dimtruck is now known as zz_dimtruck04:03
*** shyama has joined #openstack-nova04:03
*** lyanchih has quit IRC04:04
*** cfarquhar has quit IRC04:13
*** ljxiash has joined #openstack-nova04:14
*** cfarquhar has joined #openstack-nova04:14
*** cfarquhar has quit IRC04:14
*** cfarquhar has joined #openstack-nova04:14
*** yamahata has quit IRC04:15
*** ljxiash has quit IRC04:18
openstackgerritGhanshyam Mann proposed openstack/nova: Enable all extension for server PUT API sample tests  https://review.openstack.org/26942604:19
*** diana_clarke has quit IRC04:24
*** [1]Thelo has joined #openstack-nova04:29
*** Marga_ has quit IRC04:30
*** achanda has joined #openstack-nova04:30
*** Marga_ has joined #openstack-nova04:30
*** jacky-zhang has joined #openstack-nova04:31
*** baoli has quit IRC04:31
*** zz_dimtruck is now known as dimtruck04:32
*** Thelo has quit IRC04:32
*** [1]Thelo is now known as Thelo04:32
*** zhangjn has joined #openstack-nova04:33
*** diana_clarke has joined #openstack-nova04:34
*** achanda has quit IRC04:38
*** achanda has joined #openstack-nova04:39
*** markvoelker has quit IRC04:39
*** markvoelker has joined #openstack-nova04:40
*** dimtruck is now known as zz_dimtruck04:41
*** ljxiash has joined #openstack-nova04:42
*** Marga_ has quit IRC04:44
*** chhavi has joined #openstack-nova04:44
*** annegentle has quit IRC04:45
*** ljxiash has quit IRC04:46
*** thorst has joined #openstack-nova04:49
*** akshai_ has quit IRC04:51
openstackgerritNicolas Simonds proposed openstack/nova: libvirt: Fix/implement revert-resize for RBD-backed images  https://review.openstack.org/18739504:52
openstackgerritNicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available  https://review.openstack.org/20528204:52
*** thorst has quit IRC04:55
*** pradk has quit IRC04:58
*** haomaiwa_ has quit IRC04:59
*** zhangjn has quit IRC05:05
*** haomaiwang has joined #openstack-nova05:05
*** SridharG has joined #openstack-nova05:07
*** ljxiash has joined #openstack-nova05:07
*** mragupat has quit IRC05:07
*** vilobhmm11 has joined #openstack-nova05:07
*** teju has joined #openstack-nova05:08
*** lyanchih has joined #openstack-nova05:21
*** gongysh has joined #openstack-nova05:23
*** zhangjn has joined #openstack-nova05:23
*** lyanchih has quit IRC05:29
*** EinstCra_ has joined #openstack-nova05:30
*** Jeffrey4l has quit IRC05:31
*** ljxiash has quit IRC05:32
*** EinstCr__ has joined #openstack-nova05:32
*** EinstCrazy has quit IRC05:32
*** baoli has joined #openstack-nova05:32
*** ljxiash has joined #openstack-nova05:32
*** EinstCra_ has quit IRC05:34
*** ljxiash_ has joined #openstack-nova05:35
*** ljxiash has quit IRC05:37
*** ljxiash_ has quit IRC05:37
*** ljxiash has joined #openstack-nova05:37
*** otter768 has quit IRC05:38
*** amotoki has quit IRC05:39
*** liverpooler has joined #openstack-nova05:40
*** liverpoo1er has quit IRC05:42
*** salv-orlando has joined #openstack-nova05:42
*** amotoki has joined #openstack-nova05:44
*** jaosorior has joined #openstack-nova05:45
*** zhangjn has quit IRC05:46
*** EinstCr__ is now known as EinstCrazy05:47
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: libvirt: Allow multiple volume attachments  https://review.openstack.org/19313505:48
openstackgerritIldiko Vancsa proposed openstack/nova: Pass attachment_id to Cinder when detach a volume  https://review.openstack.org/19313405:48
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: comp: Allow volume multiattach  https://review.openstack.org/26758705:48
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach  https://review.openstack.org/15303805:48
openstackgerritIldiko Vancsa proposed openstack/nova: Pass host when call attach to Cinder  https://review.openstack.org/25627305:48
*** vivekd has joined #openstack-nova05:49
*** Jeffrey4l has joined #openstack-nova05:52
*** thorst has joined #openstack-nova05:53
*** edtubill has joined #openstack-nova05:53
openstackgerritGhanshyam Mann proposed openstack/nova: WIP-Enable all extension for server actions sample tests  https://review.openstack.org/26944105:57
*** ildikov has quit IRC05:58
*** edtubill has quit IRC05:58
*** lyanchih has joined #openstack-nova05:59
*** jamielennox is now known as jamielennox|away05:59
*** thorst has quit IRC06:00
openstackgerritAbhishek Kekane proposed openstack/nova: Python3: Replace dict.iteritems with six.iteritems  https://review.openstack.org/26801806:03
*** yangyapeng has joined #openstack-nova06:05
*** esracelik has joined #openstack-nova06:07
openstackgerritEli Qiao proposed openstack/nova: Add new DB API method to retrieve migration for instance  https://review.openstack.org/25727006:11
*** SridharG has quit IRC06:13
*** takashin has joined #openstack-nova06:14
*** vivekd has quit IRC06:15
*** salv-orlando has quit IRC06:15
*** liverpoo1er has joined #openstack-nova06:15
*** liverpooler has quit IRC06:17
openstackgerritGhanshyam Mann proposed openstack/nova: Enable all extension for server PUT API sample tests  https://review.openstack.org/26942606:17
openstackgerritGhanshyam Mann proposed openstack/nova: WIP-Enable all extension for server actions sample tests  https://review.openstack.org/26944106:18
*** liverpoo1er has quit IRC06:20
*** liverpooler has joined #openstack-nova06:20
*** haomaiwang has quit IRC06:21
*** 6A4ABP35J has joined #openstack-nova06:24
*** salv-orlando has joined #openstack-nova06:24
*** achanda_ has joined #openstack-nova06:25
*** achanda has quit IRC06:28
*** sahid has joined #openstack-nova06:29
*** ljxiash has quit IRC06:30
*** amotoki_ has joined #openstack-nova06:32
*** ljxiash has joined #openstack-nova06:33
*** amotoki has quit IRC06:34
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Zanata  https://review.openstack.org/26560106:35
*** amotoki_ has quit IRC06:35
*** EinstCrazy has quit IRC06:37
*** EinstCrazy has joined #openstack-nova06:37
*** davideagnello has joined #openstack-nova06:37
*** davideagnello has quit IRC06:38
*** vivekd has joined #openstack-nova06:39
openstackgerritGhanshyam Mann proposed openstack/nova: Enable all extension for server actions sample tests  https://review.openstack.org/26944106:41
*** achanda_ has quit IRC06:43
*** atuvenie has joined #openstack-nova06:44
*** liverpooler has quit IRC06:44
*** amotoki has joined #openstack-nova06:44
*** liverpooler has joined #openstack-nova06:44
*** SridharG has joined #openstack-nova06:45
*** nkrinner has joined #openstack-nova06:45
openstackgerritYingxin Cheng proposed openstack/nova: Use stevedore for scheduler driver  https://review.openstack.org/25476806:45
*** annegentle has joined #openstack-nova06:46
*** snikitin has joined #openstack-nova06:46
*** lpetrut has joined #openstack-nova06:48
*** annegentle has quit IRC06:50
*** Jeffrey4l has quit IRC06:52
*** thorst has joined #openstack-nova06:57
*** amotoki has quit IRC07:00
*** thorst has quit IRC07:04
*** salv-orlando has quit IRC07:05
openstackgerritsahid proposed openstack/nova: hardware: check whether realtime capable in API  https://review.openstack.org/26416507:07
*** 6A4ABP35J has quit IRC07:09
*** haomaiwang has joined #openstack-nova07:09
*** paul-carlton2 has joined #openstack-nova07:10
*** markvoelker_ has joined #openstack-nova07:11
*** diana_clarke has quit IRC07:14
*** markvoelker has quit IRC07:15
*** ljxiash_ has joined #openstack-nova07:15
*** ljxiash_ has quit IRC07:15
*** shyama has quit IRC07:15
*** ljxiash_ has joined #openstack-nova07:15
*** mjura has joined #openstack-nova07:15
*** shyama has joined #openstack-nova07:16
*** tdurakov_ has joined #openstack-nova07:16
*** amotoki has joined #openstack-nova07:17
*** scheuran has joined #openstack-nova07:17
*** zenoway has joined #openstack-nova07:18
*** ljxiash has quit IRC07:18
openstackgerritLisaLi proposed openstack/nova: Attach/detach encryptors during swap_volume  https://review.openstack.org/25280907:18
openstackgerritGhanshyam Mann proposed openstack/nova: Enable all extension for server PUT API sample tests  https://review.openstack.org/26942607:18
*** tdurakov_ has quit IRC07:19
*** idonotknow has joined #openstack-nova07:20
idonotknowquit07:20
*** zenoway has quit IRC07:21
*** zenoway has joined #openstack-nova07:22
openstackgerritWen Zhi Yu proposed openstack/nova: Add exception handling in _cleanup_allocated_network  https://review.openstack.org/26946207:24
*** zhangjn has joined #openstack-nova07:25
*** shyama has quit IRC07:25
*** shyama has joined #openstack-nova07:26
*** zenoway has quit IRC07:26
*** cfriesen has quit IRC07:26
*** cfriesen has joined #openstack-nova07:27
*** sshen_ has quit IRC07:27
*** sshen has joined #openstack-nova07:27
*** mpavone has joined #openstack-nova07:28
*** haomaiwang has quit IRC07:34
*** haomaiwang has joined #openstack-nova07:36
*** ptm_away has joined #openstack-nova07:38
*** EinstCrazy has quit IRC07:39
*** EinstCrazy has joined #openstack-nova07:39
*** otter768 has joined #openstack-nova07:39
*** PaulMurray has quit IRC07:42
*** ptm_away is now known as PaulMurray07:42
*** Jeffrey4l has joined #openstack-nova07:42
*** tdurakov_ has joined #openstack-nova07:43
openstackgerritGhanshyam Mann proposed openstack/nova: Enable all extension for server actions sample tests  https://review.openstack.org/26944107:44
*** otter768 has quit IRC07:44
PaulMurraypkoniszewski, Hi are you around?07:46
*** jlanoux has joined #openstack-nova07:46
PaulMurrayMorning jlanoux07:47
jlanouxhi PaulMurray07:47
PaulMurrayjlanoux, its most likely I can't make the live migration meeting this afternoon07:49
jlanouxPaulMurray: ok07:49
PaulMurrayjlanoux, would you ask pkoniszewski or tdurakov to chair it for me please?07:50
*** amotoki has quit IRC07:50
*** lpetrut has quit IRC07:50
*** ajayaa has joined #openstack-nova07:51
PaulMurrayI can check back later, but I want to make sure I catch one of them this morning07:51
eliqiao_PaulMurray: jlanoux I will attend the meeting, I can check it with pknoiszewski or tdurakov_07:52
*** eliqiao_ is now known as eliqiao07:52
PaulMurrayeliqiao, thanks - that's very helpful07:52
jlanouxPaulMurray: eliqiao sorted out!07:53
tdurakov_PaulMurray, jlanoux, eliqiao, hi07:54
eliqiaoThat's great, tdurakov_ is here. :)07:54
*** zhangjn has quit IRC07:55
*** Jeffrey4l has quit IRC07:56
*** garyk has quit IRC07:56
openstackgerritZhenyu Zheng proposed openstack/nova: Split-network-plane-for-live-migration  https://review.openstack.org/24500507:56
tdurakov_read doc about meeting infra, guess I could manage with meeting07:57
*** markvoelker_ has quit IRC07:57
*** yangyapeng has quit IRC07:57
*** EinstCrazy has quit IRC07:57
*** jinxing has quit IRC07:57
*** garyk has joined #openstack-nova07:58
*** vilobhmm11 has quit IRC07:58
*** jinxing has joined #openstack-nova08:00
*** rcernin has joined #openstack-nova08:01
*** EinstCrazy has joined #openstack-nova08:02
*** EinstCrazy has quit IRC08:02
*** EinstCrazy has joined #openstack-nova08:02
*** tdurakov_ is now known as tdurakov08:04
*** jacky-zhang_ has joined #openstack-nova08:04
*** jacky-zhang has quit IRC08:05
*** jacky-zhang_ is now known as jacky-zhang08:05
*** thorst has joined #openstack-nova08:05
*** salv-orlando has joined #openstack-nova08:06
*** zhangjn has joined #openstack-nova08:06
*** lpetrut has joined #openstack-nova08:06
*** mdnadeem has quit IRC08:08
*** ildikov has joined #openstack-nova08:09
*** PaulMurray has quit IRC08:09
*** thorst has quit IRC08:10
*** markus_z has joined #openstack-nova08:10
*** salv-orlando has quit IRC08:11
*** rdopiera has joined #openstack-nova08:12
openstackgerritWang Bo proposed openstack/nova: Fix codes using dict.iteritems() for python3  https://review.openstack.org/26948508:13
*** rgerganov has joined #openstack-nova08:13
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Add wrapper classes for return-request-id-to-caller  https://review.openstack.org/25880208:14
*** Jeffrey4l has joined #openstack-nova08:14
*** paul-carlton2 has quit IRC08:14
gongyshalex_xu, hi08:17
alex_xugongysh: hi08:17
*** irenab has joined #openstack-nova08:18
*** jichen has joined #openstack-nova08:19
gongyshalex_xu,  i am recommit that patch08:20
alex_xugongysh: yea, but just found it merge conflict08:20
openstackgerritEli Qiao proposed openstack/nova: Libvirt: Move graphics checking and console verify early when LM  https://review.openstack.org/26504208:22
alex_xugongysh: anyway I will check that later08:23
*** jianghuaw_ has joined #openstack-nova08:24
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Add wrapper classes for return-request-id-to-caller  https://review.openstack.org/25880208:25
gongyshalex_xu,  I am looking for the reason that causes the jekin failure. but I don't know why. so I recheck that patch08:26
*** paul-carlton2 has joined #openstack-nova08:27
*** matrohon has joined #openstack-nova08:27
*** lpetrut has quit IRC08:28
*** zakora has joined #openstack-nova08:30
*** zhangjn has quit IRC08:30
*** lyanchih has quit IRC08:32
*** vivekd has quit IRC08:32
openstackgerritPawel Koniszewski proposed openstack/nova: Add way to pause VM during live migration  https://review.openstack.org/24592108:33
openstackgerritBalazs Gibizer proposed openstack/nova: doc: add devref about versioned notifications  https://review.openstack.org/26378808:33
openstackgerritBalazs Gibizer proposed openstack/nova: Add service status notification  https://review.openstack.org/24567808:33
openstackgerritBalazs Gibizer proposed openstack/nova: Generate doc for versioned notifications  https://review.openstack.org/26515108:33
openstackgerritBalazs Gibizer proposed openstack/nova: Add infra for versioned notifications  https://review.openstack.org/24702408:33
openstackgerritBalazs Gibizer proposed openstack/nova: Adds json sample for the versioned notifications  https://review.openstack.org/24816708:33
openstackgerritBalazs Gibizer proposed openstack/nova: Make emitting versioned notifications configurable  https://review.openstack.org/24756408:33
*** zhangjn has joined #openstack-nova08:34
openstackgerritGhanshyam Mann proposed openstack/nova: Enable all extension for server actions sample tests  https://review.openstack.org/26944108:36
*** mdnadeem has joined #openstack-nova08:36
*** markvoelker has joined #openstack-nova08:38
*** davideagnello has joined #openstack-nova08:38
*** zhangjn has quit IRC08:39
*** daemontool has joined #openstack-nova08:41
*** vivekd has joined #openstack-nova08:42
*** Sukhdev has quit IRC08:43
*** markvoelker has quit IRC08:43
*** davideagnello has quit IRC08:44
*** lyanchih has joined #openstack-nova08:46
*** markvoelker has joined #openstack-nova08:46
*** tdurakov has quit IRC08:50
*** dedery has joined #openstack-nova08:50
openstackgerritPawel Koniszewski proposed openstack/nova: Add way to pause VM during live migration  https://review.openstack.org/24592108:51
*** tdurakov has joined #openstack-nova08:52
*** irenab has quit IRC08:52
*** zhangjn has joined #openstack-nova08:52
*** irenab has joined #openstack-nova08:53
*** zhangjn has quit IRC08:54
*** dedery has quit IRC08:55
openstackgerritEli Qiao proposed openstack/nova: Object: Make object target version check sorted  https://review.openstack.org/26950008:58
openstackgerritTang Chen proposed openstack/nova: Add a REST API to trigger crash dump in an instance.  https://review.openstack.org/20261708:59
*** irenab has quit IRC08:59
*** vivekd has quit IRC08:59
eliqiaohi tdurakov, there?09:01
tdurakoveliqiao, yep09:01
*** markvoelker has quit IRC09:02
*** daemontool has quit IRC09:02
eliqiaotdurakov: it's re https://review.openstack.org/#/c/252283/87, most of the patch seems good to me, thanks for you efforts09:02
eliqiaotdurakov: but I'd like to know where can I get the live-migration experimental test results?09:03
*** e0ne has joined #openstack-nova09:03
markus_zjohnthetubaguy: Is mitaka-2 still open? This blueprint could be finished easily. It has two small changes pushed since 6 weeks. https://blueprints.launchpad.net/nova/+spec/libvirt-aio-mode09:03
eliqiaotdurakov: I don't find it after last 'check experimental'09:03
*** daemontool has joined #openstack-nova09:03
*** vivekd has joined #openstack-nova09:04
*** irenab has joined #openstack-nova09:04
*** ankit_ag has joined #openstack-nova09:05
tdurakoveliqiao, here is the problem, job is skipped by infra, as it's doesn't changed any nova code, only tests. I've pushed patch to change this: https://review.openstack.org/#/c/267669/09:05
*** efoley has joined #openstack-nova09:06
tdurakovso, if you change smth in nova package, except tests everything works fine09:06
eliqiaotdurakov: ah, thanks, good to know it.09:07
*** thorst has joined #openstack-nova09:07
tdurakoveliqiao, thank you for +109:09
*** efoley_ has joined #openstack-nova09:09
eliqiaotdurakov: yw, thanks for your hard work. we need your patch.09:09
*** yassine__ has joined #openstack-nova09:11
*** amotoki has joined #openstack-nova09:11
*** efoley has quit IRC09:11
*** dedery has joined #openstack-nova09:12
openstackgerritDina Belova proposed openstack/nova: Integrate OSprofiler and Nova  https://review.openstack.org/25470309:13
*** efoley_ is now known as efoley09:13
chhaviildikov: regarding https://review.openstack.org/#/c/193135/09:13
ildikovchhavi: hi09:13
chhaviin this we are setting the multiattach property in the connectioninfo09:13
chhavifor any nova drivers to support multiattach do we need this property to be set in connection-info09:14
chhaviand also te virt_driver.capabilities or this is only required for libvirt09:14
ildikovchhavi: as I saw it's not the only volume info that is passed via connection_info09:14
BobBallmriedem_away: That seems to be a race condition we're struggling to track down... Will have a look at your logs to see if there are clues...09:15
*** thorst has quit IRC09:15
ildikovcurrently only Libvirt supports multiattach, but the plan would be to enable it for other drivers too09:15
chhaviyeah i am looking to enable this for our drivers so want to understand09:15
ildikovchhavi: the capabilities is needed to discover which driver supports multiattach and which does not as we don't have a full coverage09:15
ildikovchhavi: I got a comment that connection_info is not the best place, but to be honest I don't know what would be better09:16
*** rotbeard has joined #openstack-nova09:16
chhaviwhy we need to store this in connection info, can't we directly fetch from the cinder volume09:17
*** zenoway has joined #openstack-nova09:18
chhaviand does virt driver has any impact if the volume is multiattach ?09:18
*** ndipanov has quit IRC09:18
*** irenab has quit IRC09:19
openstackgerritAndrey Pavlov proposed openstack/nova: remove EC2 legacy code  https://review.openstack.org/26642509:19
*** markvoelker has joined #openstack-nova09:19
*** jistr has joined #openstack-nova09:20
*** matrohon has quit IRC09:20
chhavijust trying to understand how virt driver will be using this property09:20
ildikovchhavi: I don't think that in the virt driver we should call out to Cinder, when we have the info where we call to the virt driver09:20
ildikovthe information is needed as you usually need to disable caching in the hypervisor to support multiattach, if that's possible09:21
chhavijust thinking since we already did the check in the manager layer, before we pass the call to driver for attach/detach09:21
chhavithen do we really need to again verify in the virt driver09:21
ildikovlike iin case of Libvirt you need to add the shareable flag to the disk config309:21
*** ndipanov has joined #openstack-nova09:21
*** irenab has joined #openstack-nova09:21
ildikov*config09:21
ildikovit's not about verification, but to add the required config to the hypervisor setup for the instance09:22
ildikovotherwise it will mess up the volume09:22
ildikovor it might09:22
chhaviok so thats specific to libvirt, since some hypervisor does not care about and allows multiattach09:23
markus_zbauzas: Good morning, do you have a minute? I don't quit get the config option "scheduler_json_config_location".09:23
*** markvoelker has quit IRC09:24
markus_zbauzas: Is that an additional way to configure the scheduler? Does it have a specific structure?09:25
ildikovchhavi: if it does not allow, then you will not be able to implement that support for it09:26
ildikovchhavi: if it enables you will most probably need to set an extra config option to enable it09:27
ildikovotherwise the optimization that the hypervisor is doing will mess up the write operations on your volume from the multiple instances, which tries to access it in parallel09:27
ildikovyou will need to triple check the hypervisor you want to add support for09:28
*** danpb has joined #openstack-nova09:28
*** Marga_ has joined #openstack-nova09:29
*** jacky-zhang has quit IRC09:29
*** e0ne has quit IRC09:34
chhavihas these been updated in the hypervisor support matrix doc09:36
*** matrohon has joined #openstack-nova09:36
johnthetubaguymarkus_z: technically still open, but the feature freeze is separate, that happens on Thursday, independently of mitaka-209:37
*** jacky-zhang has joined #openstack-nova09:37
*** aix has joined #openstack-nova09:39
*** daemontool has quit IRC09:39
*** daemontool has joined #openstack-nova09:39
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Add return-request-id-to-caller function(1/5)  https://review.openstack.org/26139909:40
*** haomaiwang has quit IRC09:40
*** markvoelker has joined #openstack-nova09:40
*** otter768 has joined #openstack-nova09:40
markus_zjohnthetubaguy: ah, I thought it goes hand in hand.09:40
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Add return-request-id-to-caller function(2/5)  https://review.openstack.org/26140109:40
markus_zjohnthetubaguy: What can I do to get attention to that libvirt change? Adding it to the priorities tracking etherpad looks wrong imo.09:41
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Add return-request-id-to-caller function(3/5)  https://review.openstack.org/26140509:41
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Add return-request-id-to-caller function(4/5)  https://review.openstack.org/26140709:41
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Add return-request-id-to-caller function(5/5)  https://review.openstack.org/26141009:42
johnthetubaguymarkus_z: there is a libvirt sub group in there I thought, thats correct ish09:42
ildikovchhavi: it is updated here: https://review.openstack.org/#/c/193135/86/doc/source/support-matrix.ini09:42
*** lpetrut has joined #openstack-nova09:43
chhaviok09:43
tdurakovjohnthetubaguy, hi, last week we were talking about https://review.openstack.org/#/c/252283/ so, could we merge it?09:43
*** haomaiwang has joined #openstack-nova09:43
*** derekh has joined #openstack-nova09:43
markus_zjohnthetubaguy: ok, I'll try it there, thanks09:43
*** boris-42 has quit IRC09:43
*** jacky-zhang has quit IRC09:43
*** irenab has quit IRC09:44
*** aix_ has joined #openstack-nova09:44
*** markvoelker has quit IRC09:45
*** otter768 has quit IRC09:45
*** irenab has joined #openstack-nova09:45
johnthetubaguytdurakov: if you be great if we could have a live-migrate CI run against that patch, maybe make a folow on patch that triggers that test by some change of a comment in the main body of the code?09:45
johnthetubaguytdurakov: this week I am going to have to focus on the release, and the feature freeze, so its unlikley I will get to that I am afraid09:46
johnthetubaguybauzas: are you around today, to help getting the release sorted?09:46
bauzasmarkus_z: hi09:46
tdurakovjohnthetubaguy, i understand, have you seen sdague recently?09:46
bauzasjohnthetubaguy: heya09:46
*** aix_ has quit IRC09:46
johnthetubaguytdurakov: no, been out yesterday09:46
bauzasjohnthetubaguy: et al. I'll be off for tomorrow since I'll be teaching an OpenStack class09:46
bauzasoff tomorrow I mean09:47
*** rmart04 has joined #openstack-nova09:47
BobBalljohnthetubaguy: I've identified another race in the logconsole code which is very hard to avoid: xapi uses libxc to start a domain, which can start to write to the console log before XAPI returns that the domain has started (and we can identify the domain ID so we can explicitly preserve the console log).  The only way I can think of avoiding the race is to only delete logs that haven't been modified for the last 5 minutes, to09:47
johnthetubaguybauzas: ah, no worries, I was out yesterday cause I was traveling back from a contest, and needed some shut eye09:47
bauzasand then, Thurs and Frid, I'll be available but will be following a conf09:47
johnthetubaguybauzas: gotcha09:47
bauzasjohnthetubaguy: sure, np, I was about to ping you about the above09:47
tdurakovjohnthetubaguy, my patch in project config(which fixes this skip) waits for sdague review09:48
bauzasgiven that we cut m209:48
bauzasjohnthetubaguy: so, when do you want to cut ?09:48
bauzasjohnthetubaguy: have you seen mriedem_away's patch for stable point release ?09:48
johnthetubaguybauzas: the main thing I was thinking, is lets try work out what blockers we have, and what release notes we might be missing09:48
*** amotoki has quit IRC09:48
johnthetubaguybauzas: I haven't09:48
*** e0ne has joined #openstack-nova09:48
bauzassec09:48
bauzasjohnthetubaguy: I agree with you, we should look at the UpgradeImpact, DocImpact and features09:49
bauzasfor the notes09:49
bauzaswe had a CI issue with reno, but my fix was merged, so it should be back okay09:49
bauzasjohnthetubaguy: stable point release patch https://review.openstack.org/#/c/268709/09:50
BobBallmriedem_away: FYI https://bugs.launchpad.net/nova/+bug/153561609:51
openstackLaunchpad bug 1535616 in OpenStack Compute (nova) "XenAPI: Console logging race condition" [Undecided,New]09:51
openstackgerritSergey Nikitin proposed openstack/nova: Added server tags controller  https://review.openstack.org/12894009:51
openstackgerritSergey Nikitin proposed openstack/nova: Added server tags support in nova-api  https://review.openstack.org/26893209:51
markus_zbauzas: wrt reno: There is this ML post about DocImpact statements in Nova. I did a short analysis back then, maybe it helps: http://lists.openstack.org/pipermail/openstack-dev/2016-January/083738.html09:51
*** pbrooko has joined #openstack-nova09:52
bauzasmarkus_z: yeah I saw that thread but I missed the outcome09:52
johnthetubaguyBobBall: seems worth a try, that stuff always scared me09:53
bauzasmarkus_z: assuming you already know http://docs.openstack.org/developer/nova/code-review.html#when-a-release-note-is-needed09:53
markus_zbauzas: yepp, I know that, thanks.09:54
pkholkinjohnthetubaguy, sdague, alaski: Hello guys! Please review these patches when you have time https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/new-oslodb-enginefacade, the bp seems to be implemented :)09:54
bauzasso, I said about DocImpact "some specific changes that have a DocImpact tag but require further action from an deployer perspective"09:54
pkholkinfeel free to ask me and snikitin any questions09:54
pkholkinmdbooth: you too)09:55
bauzasmarkus_z: I agree with the thread which is that not all the DocImpact need a reno file09:55
markus_zbauzas: yepp, true09:55
markus_zbauzas: have to go to a meeting, will take a while, can talk about that later09:56
*** markus_z is now known as markus_z_meeting09:56
*** e0ne has quit IRC09:57
bauzasmarkus_z_meeting: sure, I have a few things09:57
johnthetubaguymarkus_z: bauzas: those look like for most of those things we should add a reno note09:58
bauzasjohnthetubaguy: fair to say09:58
bauzasjohnthetubaguy: could you please also see which BPs are landed in M2 so we would need a reno file ?09:59
bauzas(meaning the ones impacting the REST API)09:59
bauzasjohnthetubaguy: that should match the DocImpact09:59
*** Jeffrey4l has quit IRC10:00
johnthetubaguybauzas: did that already, launchpad should be up to date10:02
johnthetubaguybauzas: at least it was on friday just gone10:02
johnthetubaguybauzas: totally just realised I have to be in london today, and well, I am not right now...10:03
*** salv-orlando has joined #openstack-nova10:07
*** daemontool has quit IRC10:08
*** daemontool has joined #openstack-nova10:08
*** mdavidson has joined #openstack-nova10:08
openstackgerritGhanshyam Mann proposed openstack/nova: Enable all extension for server actions sample tests  https://review.openstack.org/26944110:11
*** daemontool has quit IRC10:11
openstackgerritGhanshyam Mann proposed openstack/nova: Enable all extension for server PUT API sample tests  https://review.openstack.org/26942610:11
openstackgerritGhanshyam Mann proposed openstack/nova: Enable all extension for Flavor API sample tests  https://review.openstack.org/24339610:11
openstackgerritGhanshyam Mann proposed openstack/nova: Enable all extension for server API tests part-1  https://review.openstack.org/26953310:11
*** daemontool has joined #openstack-nova10:11
openstackgerritGhanshyam Mann proposed openstack/nova: WIP-Enable all extension for server API sample tests  https://review.openstack.org/26940410:12
*** thorst has joined #openstack-nova10:12
*** kbyrne has quit IRC10:12
*** salv-orlando has quit IRC10:13
snikitinalex_xu: Alex, thanks for comments in instance tags patches! Glad to see that someone still review it :)10:13
*** markvoelker has joined #openstack-nova10:16
*** daemontool has quit IRC10:17
*** ankit_ag has quit IRC10:17
*** daemontool has joined #openstack-nova10:17
*** ankit_ag has joined #openstack-nova10:17
openstackgerritBob Ball proposed openstack/nova: XenAPI: Fix race on rotate_xen_guest_logs  https://review.openstack.org/26953610:17
*** Marga_ has quit IRC10:18
*** amotoki has joined #openstack-nova10:18
*** Jeffrey4l has joined #openstack-nova10:18
*** daemontool has quit IRC10:19
*** daemontool has joined #openstack-nova10:19
*** thorst has quit IRC10:20
*** ociuhandu has quit IRC10:21
*** markvoelker has quit IRC10:21
dederyhi all, can this fix https://review.openstack.org/#/c/262341 - Fix create snapshot failure on VMs with SRIOV - be reviewed? thanks!10:23
*** aix has quit IRC10:23
*** aix has joined #openstack-nova10:26
*** jinxing has quit IRC10:29
*** Jeffrey4l has quit IRC10:31
*** atuvenie has quit IRC10:32
rgerganovjohnthetubaguy: hi, could you approve https://review.openstack.org/#/c/228778/ ? it has a lot of +1 and it's essentially a bug fix10:36
*** daemontool has quit IRC10:38
openstackgerritLee Yarwood proposed openstack/nova: Ignore Cinder exceptions when terminating a volume's connection  https://review.openstack.org/26917210:38
*** davideagnello has joined #openstack-nova10:40
*** daemontool has joined #openstack-nova10:42
*** shyama has quit IRC10:43
*** shyama has joined #openstack-nova10:44
openstackgerritSimona Iuliana Toader proposed openstack/nova: Hyper-V: Adds Hyper-V UEFI Secure Boot  https://review.openstack.org/20958110:44
*** ildikov has quit IRC10:44
*** atuvenie has joined #openstack-nova10:44
*** markvoelker has joined #openstack-nova10:44
*** davideagnello has quit IRC10:46
*** aix has quit IRC10:46
dederyndipanov: thanks. I'll change according to the comment. I take it that you agree with the basic change from detach_interface to detach_pci_device?10:48
*** dims has joined #openstack-nova10:48
*** rfolco has joined #openstack-nova10:48
dederyndipanov: I didn't see any risk there so i'm quite comfortable with this change.10:48
openstackgerritEli Qiao proposed openstack/nova: Object: Make object target version check sorted  https://review.openstack.org/26950010:48
gibijohnthetubaguy: Hi! I lost couple of +2 form the versioned notification chain due to merge conflicts, could you please check them again?10:49
*** markvoelker has quit IRC10:49
ndipanovdedery, I think so - will look more closely before +2 ing though10:50
*** shihanzhang has quit IRC10:50
*** vishalcdac07 has joined #openstack-nova10:50
*** shihanzhang has joined #openstack-nova10:51
gibinova cores: the versioned notification commit chain already has +2 from johnthetubaguy so It needs a second +2. Could you check it please? The chain starts here: https://review.openstack.org/#/c/247024/10:52
openstackgerritZhenyu Zheng proposed openstack/nova: Split-network-plane-for-live-migration  https://review.openstack.org/24500510:53
*** Marga_ has joined #openstack-nova10:53
*** shyama has quit IRC10:53
*** shyama has joined #openstack-nova10:54
*** ihrachys has joined #openstack-nova10:54
*** lyanchih has quit IRC10:55
*** yassine__ has quit IRC10:56
*** aix has joined #openstack-nova10:56
*** tdurakov has quit IRC10:56
*** vishalcdac07 has quit IRC10:57
*** erlon has joined #openstack-nova11:00
*** irenab has quit IRC11:02
*** irenab has joined #openstack-nova11:03
*** zakora has quit IRC11:04
*** zakora has joined #openstack-nova11:06
*** aysyd has joined #openstack-nova11:06
chhaviildikov11:07
chhavicheck_attach: should also check for the attaching state11:08
chhaviin case multiattach there are chances of user sending the concurrent request11:09
*** ildikov has joined #openstack-nova11:10
ildikovchhavi: sorry, networking issues11:10
alex_xusnikitin: np :)11:10
chhaviildikov: https://review.openstack.org/#/c/267587/8/nova/volume/cinder.py11:11
chhaviin this check_attach should also check for the attaching status11:11
ildikovchhavi: if you ask from multiattach perspective it doesn't matter11:14
ildikovit's either 'attached' or 'detached', but in case of multiattach both are valid11:14
*** markvoelker has joined #openstack-nova11:14
*** Marga_ has quit IRC11:15
chhaviit does, if the volume status is 'attaching' it will fail11:15
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova: Added new scheduler filter: AggregateTypeExtraSpecsAffinityFilter  https://review.openstack.org/18927911:15
chhaviconsider if the user sends 2 simultaneous request then for the first request it will pass, and when the second request comes there are chances that volume status changed to 'attaching' then the second request will fail11:16
ildikovthis is why it has to be 'available' or 'available' or 'in-use'11:16
ildikovso that check is correct11:16
*** tdurakov has joined #openstack-nova11:16
ildikovin all the other cases, like 'attaching' or error the check will fail11:17
*** thorst has joined #openstack-nova11:17
*** zakora has quit IRC11:19
*** markvoelker has quit IRC11:20
*** sfinucan has joined #openstack-nova11:20
*** vishalcdac07 has joined #openstack-nova11:21
chhaviildikov:  multiattach should allow 'attaching' state,  because now the volume can be attached to multiple instances11:21
*** jaosorior has quit IRC11:22
*** zakora has joined #openstack-nova11:22
ildikovchhavi: as the outcome of the 'attaching' state can be error too IIRC I would not be 100% comfortable with allowing that11:22
vishalcdac07what is current status of blueprint https://blueprints.launchpad.net/nova/+spec/bulk-delete-servers11:22
*** jaosorior has joined #openstack-nova11:22
ildikovchhavi: if we face with issues because of not allowing to attach in that state we can modify the check as optimization11:23
chhaviildikov: we have seen this issue during our unit test while performing concurrent operations11:23
chhaviildikov: is there a way to pull all the multiattach nova blueprint changes11:24
chhaviinstead of pulling them separately11:24
vishalcdac07It is showing the status "Complete" on launchpad but could not find respective API reference in http://developer.openstack.org/api-ref-compute-v2.1.html11:24
*** thorst has quit IRC11:25
ildikovchhavi: if you checkout the last patch in the chain it will download the full chain for you if that's the question11:25
chhaviildikov: do u mean this one11:26
chhavihttps://review.openstack.org/#/c/193134/11:26
ildikovchhavi: it's the best to checkout this one: https://review.openstack.org/#/c/193135/11:28
ildikovthe last one is really in WIP11:28
chhaviok let me try11:28
*** rvasilets has joined #openstack-nova11:28
rvasiletsHi, guys. please help me to find information about what nova compute, scheduler, conductor, api are doing. I mean i don't understand architecture of nova. Is it possible to understand without reading the code?11:31
*** markvoelker has joined #openstack-nova11:32
rvasiletsI have found like this http://docs.openstack.org/kilo/config-reference/content/section_conductor.html But its to small information here11:32
* danpb would suggest this architecture diagram, but is sceptical that it has ever helped anyone understand openstack http://docs.openstack.org/icehouse/training-guides/content/figures/5/a/figures/openstack-arch-havana-logical-v1.jpg11:32
*** pbrooko has quit IRC11:32
rvasiletsOk. But diagram wouldn't help me to understand what is doing scheduler or how it works with queue11:34
rvasiletsetc11:34
rvasiletsAs I see its old architecture from havana)11:35
danpbat a high level, the api service accepts REST requests from clients11:35
danpbit puts data into the database and/or RPC calls onto the message bus11:36
*** ericksonsantos has joined #openstack-nova11:36
danpbschedular handles RPC calls for booting VMs and decides which hosts to run them on11:36
danpbcompute runs on each virtualization host and actually manages the hypervisor & vms on that host11:36
*** markvoelker has quit IRC11:36
danpbconductor is responsible for updating the DB on behalf of the compute service11:37
*** ociuhandu has joined #openstack-nova11:37
*** gszasz has joined #openstack-nova11:38
*** e0ne has joined #openstack-nova11:38
*** cdent has joined #openstack-nova11:39
*** pbrooko has joined #openstack-nova11:41
*** otter768 has joined #openstack-nova11:41
alex_xugongysh: looks like just need rebase. I see the merge conflict from gerrit11:42
*** smatzek has joined #openstack-nova11:43
markus_z_meetingrvasilets: That one is a recent architecture diagram: http://docs.openstack.org/developer/nova/architecture.html11:44
*** otter768 has quit IRC11:46
markus_z_meetingrvasilets: Also some information here: http://docs.openstack.org/developer/nova/index.html#architecture-concepts11:48
rvasiletsmarkus_z_meeting, looks like the last link is the best)11:49
*** markus_z_meeting is now known as markus_z11:49
*** ljxiash_ has quit IRC11:50
markus_zrvasilets: Just repeat this mantra multiple times a day: "there is no easy way to understand nova" ;)11:50
markus_zdanpb: Do you see a chance to have a look at the libvirt AIO mode changes https://review.openstack.org/#/q/topic:bp/libvirt-aio-mode,n,z ?11:51
markus_zdanpb: These are two (small) changes to an approved bp but they got little attention in the last weeks.11:51
danpbmarkus_z: ok11:52
markus_zthanks11:52
*** irenab has quit IRC11:53
*** gmann_ has joined #openstack-nova11:53
markus_zbauzas: wrt the DocImpact discussion we started, I personally don't have questions for that as I think I understood it.11:55
*** jlanoux has quit IRC11:55
markus_zbauzas: The mentioned reviews in the ML post happened in the transitioning phase when we introduce reno in Nova.11:56
markus_zbauzas: And IIUC some of them need a follow up patch with a reno file to be consistent in this release.11:56
*** yassine__ has joined #openstack-nova11:57
*** irenab has joined #openstack-nova11:57
* alex_xu reminders api meeting in 2mins11:58
*** tdurakov has quit IRC12:00
*** irenab has quit IRC12:03
*** daemontool has quit IRC12:05
*** daemontool has joined #openstack-nova12:06
*** aix has quit IRC12:06
*** EinstCrazy has quit IRC12:06
*** gongysh has quit IRC12:07
*** baoli has quit IRC12:07
*** salv-orlando has joined #openstack-nova12:08
*** vivekd_ has joined #openstack-nova12:11
*** markvoelker has joined #openstack-nova12:11
*** vivekd has quit IRC12:11
*** vivekd_ is now known as vivekd12:11
*** vivekd_ has joined #openstack-nova12:12
*** salv-orlando has quit IRC12:13
*** vivekd has quit IRC12:16
*** vivekd_ is now known as vivekd12:16
*** markvoelker has quit IRC12:16
markus_zdims: IIUC we don't use "fix comitted" anymore in bug reports. See https://bugs.launchpad.net/nova/+bug/1533678 for example.12:18
openstackLaunchpad bug 1533678 in OpenStack Compute (nova) "Jenkins py34 gate failing with 'wmi is not defined' error" [Undecided,Fix released]12:18
markus_zdims: JFYI12:18
*** aix has joined #openstack-nova12:18
dimsmarkus_z : ack12:19
dimsthanks12:19
markus_zthanks for making it cleaner!12:19
*** rgerganov has quit IRC12:19
openstackgerritTomoki Sekiyama proposed openstack/nova: libvirt: Make behavior of os_require_quiesce consistent  https://review.openstack.org/16781912:19
*** ljxiash has joined #openstack-nova12:22
*** ljxiash has quit IRC12:22
*** ljxiash has joined #openstack-nova12:22
dimssdague : johnthetubaguy : for those dhcp lease failures, this review seems to help (https://review.openstack.org/#/c/268796/1/nova/cmd/dhcpbridge.py) the theory was that if the dhcp script was already running then dnsmasq  then it does not get run (see "The script is not invoked concurrently" in http://www.thekelleys.org.uk/dnsmasq/docs/dnsmasq-man.html)12:23
dimssdague : johnthetubaguy : i still see occasional failures in say "gate-grenade-dsvm" which seems to be because the "new" nova-cpu recomputes the mac for some reason and tries to use that for dhcp_release and the mac that was used to lease the dhcp on the old side is not used for some reason12:24
dimsso that's a valid failure12:26
*** raildo-afk is now known as raildo12:27
*** tdurakov has joined #openstack-nova12:27
sdaguedims: nice find12:28
sdagueso the theory is that dnsmasq will only ever run the script once, and the fact that it takes a while means things will get lost?12:29
tdurakovsdague, hi Sean, got time to discuss live-migration job?12:29
openstackgerritJacky_lei_zhang proposed openstack/nova: Fix support customer ip protocol when create security-rule using nova-cli  https://review.openstack.org/26668712:31
*** takashin has left #openstack-nova12:31
sdaguetdurakov: in an hour or so12:31
tdurakovsdague, ok, I'll ping you in an hour12:32
*** jacky-zhang has joined #openstack-nova12:32
*** ildikov has quit IRC12:35
*** pbrooko has quit IRC12:35
sdaguedims: that fix means there is still a race right, now it's just smaller?12:36
*** pbrooko has joined #openstack-nova12:36
sdagueI almost wonder if we should replace dhcpbridge.py in config with a shell script that spins that off, to reduce the race further12:37
*** lpetrut has quit IRC12:39
*** davideagnello has joined #openstack-nova12:42
openstackgerritSean Dague proposed openstack/nova: Make project_id optional in v2.1 urls  https://review.openstack.org/23307612:43
openstackgerritBalazs Gibizer proposed openstack/nova: doc: add devref about versioned notifications  https://review.openstack.org/26378812:45
openstackgerritBalazs Gibizer proposed openstack/nova: Generate doc for versioned notifications  https://review.openstack.org/26515112:45
openstackgerritBalazs Gibizer proposed openstack/nova: Adds json sample for the versioned notifications  https://review.openstack.org/24816712:45
*** links has quit IRC12:46
*** davideagnello has quit IRC12:47
sdaguedims: is it actually dropping actions, or are they backing up?12:47
*** thorst has joined #openstack-nova12:48
*** markvoelker has joined #openstack-nova12:49
dimssdague : from what i can tell, it's dropping actions12:50
*** houming has quit IRC12:50
dimssdague : if you see my review, i just fork() to get the same effect12:50
sdagueright12:50
sdagueso the docs seem to indicate the activity will be queued up12:50
*** haomaiwang has quit IRC12:51
*** mdnadeem has quit IRC12:51
sdaguecould we print out sys.argv ?12:52
*** mpavone has quit IRC12:52
dimssdague : ack will update the review12:53
sdagueI kind of wonder if we're scrubbing away content there12:53
sdagueI bet the nova objects code changes here slowed things down a bit, which is why this got worse12:53
dimsy and the switch from db to conductor too12:54
*** markvoelker has quit IRC12:54
sdagueyeh, though that was a long time ago12:54
dimsright12:54
*** gmann_ has quit IRC12:54
sdagueyeh, I'm really curious now if we are getting a more complicated sys.argv12:55
sdagueI think that needs to be a different patch than your fork patch12:56
sdaguebecause we need to expose the collision12:56
sdaguemaybe even add a sleep in there to force it12:56
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] Avoid blocking dnsmasq by forking off RPC calls  https://review.openstack.org/26879612:57
*** ildikov has joined #openstack-nova12:58
*** mpavone has joined #openstack-nova13:01
*** mpavone has quit IRC13:01
*** jlanoux has joined #openstack-nova13:02
johnthetubaguyalex_xu: sdague: sorry, was on a train so missed the api meeting (again), looks like good discussion, let me know if you need anything from me, especially around mitaka-213:02
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] Trying to force failures in dhcp leases  https://review.openstack.org/26961613:02
johnthetubaguybauzas: sorry, I am back online now, hows it looking so far?13:03
*** irenab has joined #openstack-nova13:03
bauzasjohnthetubaguy: I'm just beginning to look at all the changes13:03
alex_xujohnthetubaguy: thanks :)13:03
* bauzas working with Pomodora :)13:03
sdaguejohnthetubaguy: mostly just https://review.openstack.org/233076 which is the no_project_id patch13:04
sdaguePS21 has a pretty comprehensive set of notes on it13:04
*** atuvenie has quit IRC13:06
*** lpetrut has joined #openstack-nova13:06
ccarmackalex_xu: was the API meeting today?13:06
eliqiaoccarmack: just finished13:07
alex_xuccarmack: yes, it just finished13:07
johnthetubaguybauzas: :)13:07
alex_xuccarmack: people agree to change rebuild13:07
johnthetubaguyndipanov: that PCI bug, I guess we should try get that into mitaka-2 if we can, given its a nasty regression?13:07
alex_xuccarmack: it probably need spec also, but I guess that will be quick update.13:07
alex_xus/need spec/need update spec/13:07
ccarmackalex_xu: so, when someone does rebuild, they need to change the description?13:08
johnthetubaguybauzas: I am listing blockers in the usual location in here: https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking13:08
alex_xuccarmack: if the name changed, I guess the meaning of description will be changed also. in that case, I think user have requirement change the description when name changed.13:08
alex_xuccarmack: does sounds make sense?13:09
johnthetubaguycdent: are you OK with being the cross project liaison, I added you into the wiki, I think13:09
*** mdnadeem has joined #openstack-nova13:10
*** ducttape_ has joined #openstack-nova13:10
*** markvoelker has joined #openstack-nova13:10
*** daemontool_ has joined #openstack-nova13:10
sfinucanmarkus_z: got a moment?13:10
ildikovjaypipes: good morning13:10
markus_zsfinucan: one sec, writing a review comment to your vnc change :)13:10
*** daemontool has quit IRC13:11
alex_xusdague: just +2 for project_id13:11
sfinucanmarkus_z: If it's about 'novnc.py' then we can talk here. If not, carry on ;)13:11
*** lpetrut1 has joined #openstack-nova13:11
ildikovjaypipes: I fixed your comment and I hope finally Jenkins will be nice to me too, the patch ran into a few known issues13:11
*** lpetrut has quit IRC13:11
*** lpetrut1 is now known as lpetrut13:11
markus_zsfinucan: It's the "vnc.py". What's up?13:11
ccarmackalex_xu: I'm not sure, maybe the name changes to "myinstance-rebuild" or something13:12
cdentjohnthetubaguy: yup13:12
sfinucanmarkus_z: quick one. You suggested also centralizing these options https://github.com/openstack/nova/blob/master/nova/cmd/novncproxy.py13:12
markus_zsfinucan: ah, yes, I remember13:12
sfinucanwhat about these? https://github.com/openstack/nova/blob/master/nova/cmd/novnc.py13:12
markus_zsfinucan: uaah, what's this again. Let me check it.13:13
*** alejandrito has joined #openstack-nova13:14
johnthetubaguycdent: awesome thank you, I noticed you made the meeting last week, thanks for that :)13:14
*** markvoelker has quit IRC13:15
cdentjohnthetubaguy: yup, I've put it in my list of things to make a habit13:15
alex_xuccarmack: if the new name is meanful, the description should be changed to match the new name. that is the case I thought13:15
cdentjohnthetubaguy: and have openstack-specs in my review queue13:15
alex_xus/meanful/meaningful/?13:15
johnthetubaguycdent: sweet, let me know if you need anything for that, I will try do the reverse also :)13:15
cdentroger, will do13:16
ccarmackalex_xu: ok13:16
alex_xuccarmack: if we didn't support change that in rebuild, user need another request to update description13:16
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova: Added new scheduler filter: AggregateTypeExtraSpecsAffinityFilter  https://review.openstack.org/18927913:17
markus_zsfinucan: These options are common to all proxys: https://github.com/openstack/nova/blob/master/nova/cmd/baseproxy.py13:17
ccarmackalex_xu, ok, but we are requiring the user to change the description, right?13:17
markus_zsfinucan: They are in a module which suggest they are solely for vnc, but they aren't.13:17
openstackgerritSayali Lunkad proposed openstack/nova: Allows to set display_name as instance_name  https://review.openstack.org/26335313:18
markus_zsfinucan: I would expect them in a "nova/conf/proxy.py" module or similar13:18
alex_xuccarmack: optional to change13:18
*** ekarlso has quit IRC13:18
*** ekarlso has joined #openstack-nova13:18
ccarmackalex_xu: ah, now I understand13:18
sfinucanmarkus_z: Sounds about right13:18
markus_zsfinucan: cool :)13:19
alex_xuccarmack: thanks13:19
sfinucanI won't move those now (there are enough items in the backlog) but I can put it into the 'for consideration' section?13:19
sfinucani.e. to move them13:19
ccarmackalex_xu, johnthetubaguy: Is there a code freeze tomorrow for non-priority patches?13:19
markus_zsfinucan: What would be the reasoning for that?13:19
markus_zsfinucan: Ah, you mean to its own [group]?13:20
alex_xuccarmack: Thursday13:20
sfinucanmarkus_z: yes, exactly13:20
alex_xuccarmack: we still have chance to cache the freeze13:20
jaypipesildikov: k, will look into it.13:20
sfinucanand into 'nova/conf/proxy.py' instead of 'nova/conf/vnc.py'13:20
alex_xuas it isn't big change13:20
jaypipesildikov: and good morning to you :)13:20
openstackgerritSayali Lunkad proposed openstack/nova: Allows to set display_name as instance_name  https://review.openstack.org/26335313:20
ildikovjaypipes: cool, tnx much13:20
markus_zsfinucan: Yes, that's fair. Just add it to the etherpad and leave a reasoning.13:21
sdaguedims: if you are busy, I can also crank out the testing patch, I'm really curious now13:21
ildikovjaypipes: well, after 4 hours of sleep it still feels like morning :)13:21
jaypipesildikov: :)13:21
ccarmackalex_xu: ok, I'll upload the spec update today13:21
alex_xuccarmack: thanks13:21
alex_xuccarmack: ping me if that ready13:21
ildikovjaypipes: time schedule is not really my strong skill nowadays...13:21
ccarmackalex_xu: starting now13:22
markus_zsfinucan: Let me know if my comment here makes sense to you: https://review.openstack.org/#/c/266273/213:22
alex_xuccarmack: gmann suggest add other bit about update will return description13:22
jaypipesildikov: I hear ya.13:22
ccarmackalex_xu: yes, I didn't know about that when I wrote the spec13:23
ndipanovjohnthetubaguy, sorry was away13:23
ndipanovso there are 2 pci bugs atm13:23
sfinucanmarkus_z: It does, yes. I'm still not entirely sure what that 'xvpvncproxy_base_url' option is really for, however.13:23
*** rgerganov has joined #openstack-nova13:23
sfinucanwhich means I probably shouldn't be writing the docs for it, but oh well :)13:23
sfinucanI'll try find the owner of that code and ping them for clarification13:23
ndipanovone that was there but got exposed by the PF code, and one that we introduced13:23
markus_zsfinucan: That's the URL each compute node returns when the REST API call happens.13:24
ndipanovboth of them are killing the PCI CI13:24
ndipanovjohnthetubaguy, I'll post a patch for both today - yes it would be awesome to get those in m-213:24
*** shakamunyi has joined #openstack-nova13:24
markus_zsfinucan: http://developer.openstack.org/api-ref-compute-v2.1.html#getSPICEConsole13:24
johnthetubaguyndipanov: cools, I added one of those patches to here: https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking will try track them, the best we can13:25
markus_zsfinucan: The token is added by the nova-console-auth service13:25
openstackgerritSean Dague proposed openstack/nova: Make project_id optional in v2.1 urls  https://review.openstack.org/23307613:26
sdaguealex_xu: there was a reno issue, fixed13:26
markus_zsfinucan: The default which has the localhost address only makes sense when you have an all-in-one machine and try to establish a SPICE connection also on that machine.13:26
alex_xusdague: oops..my fault13:27
sdaguegmann: how does - https://review.openstack.org/#/c/243396 work with the legacy v2 code?13:27
dimssdague : sure, please go ahead13:27
ndipanovjohnthetubaguy, cool13:27
*** ducttape_ has quit IRC13:27
dimssdague : i had this up if you want to weak that (https://review.openstack.org/#/c/269616/)13:28
dimss/weak/tweak/13:28
ansiwenrgerganov: hi13:28
rgerganovansiwen, hi13:28
openstackgerritBo Chi proposed openstack/nova: Disassociate floating ip when remove fixed ip  https://review.openstack.org/25733913:28
flip214I'd appreciate some feedback (and/or +2's ;) on https://review.openstack.org/#/c/256292. All dependencies are already merged, and this is small, too.13:28
*** ivase has joined #openstack-nova13:29
*** ivase has joined #openstack-nova13:29
ansiwenrgerganov: there is a CI problem with https://review.openstack.org/#/c/16802413:30
*** dslev has joined #openstack-nova13:30
ansiwenrgerganov: is it a glitch or a real problem?13:30
rgerganovansiwen, yeah, I think it is intermittent, waiting for the next run13:30
ansiwenrgerganov: has it been retriggered?13:30
sfinucanmarkus_z: Hmmm. Looking at this documentation from IBM, it seems _host + _port + a path => _base_url https://goo.gl/UlmabQ13:30
*** edmondsw has joined #openstack-nova13:30
markus_zsfinucan: And lastly, one of the many possible REST API consumers for that is Horizon, which happens to be usually on the same node as the vncproxy service. Which means "localhost" is a valid value there.13:30
openstackgerritSean Dague proposed openstack/nova: DNM: see what dhcp args look like with collisions  https://review.openstack.org/26962613:31
ansiwenrgerganov: not sure if "vmware-recheck-patch" works, never saw that13:31
rgerganovansiwen, vmware-recheck-patch should work13:31
sdaguedims: ok, I did a thing a little different didn't realize that ws up13:31
sfinucanIn which case, back to why that's not autogenerated instead. I don't want to state something I don't know to be true13:31
ansiwenrgerganov: ok, cool. is it possible to the the queue?13:32
*** ankit_ag has quit IRC13:33
markus_zsfinucan: that link is odd, it wants me to download a html, which I'd rather not do.13:33
ildikovjaypipes: and the tests are green \o/, I hope I have at least a few hours before the next merge conflict... :)13:33
rgerganovansiwen, the CI queue is not publicly visible13:33
ansiwenrgerganov: ok13:33
sfinucanmarkus_z: Yeah, I don't know what's happening there :S (Found via 'xvpvncproxy_host site:ibm.com' search)13:34
sfinucanBut the main point is the two are _very_ closely related and I don't get why the distinction was necessary. Perhaps different services use different values or something...13:35
markus_zsfinucan: http://ibm.co/1ZJO0E2 (some day in the future we will provide permalinks with slugs... hope dies last)13:37
markus_zsfinucan: ok, got it. Yeah, the setup described there is valid but not the only possible one.13:38
tdurakovsdague, are you around?13:38
sdaguetdurakov: yeh, what's up?13:38
tdurakovsdague, could you review https://review.openstack.org/#/c/267669/13:39
bauzasjohnthetubaguy: FWIW just created a new etherpad for checking all patches needing a reno file https://etherpad.openstack.org/p/nova-milestone-reno13:41
bauzasjohnthetubaguy: now, I'll look at each of those and see if a reno file is there13:42
*** otter768 has joined #openstack-nova13:42
*** vivekd_ has joined #openstack-nova13:42
*** vivekd has quit IRC13:45
*** vivekd_ is now known as vivekd13:45
sdaguetdurakov: +213:45
*** ababich has joined #openstack-nova13:45
*** EinstCrazy has joined #openstack-nova13:46
tdurakovthank you sdague, so this should work, right?)13:46
sfinucanmarkus_z: OK, I've taken the middle ground and said which options are related but not why. If you (or anyone else) has input here I'm good with either suggestions or you just changing it yourself :)13:46
sdaguetdurakov: I think so13:46
sfinucanseems safer than saying something I don't know to be true13:46
sdagueI don't seen Andreas on right now, we should circle with him13:46
*** rvasilets has left #openstack-nova13:46
johnthetubaguybauzas: cool, thanks13:47
tdurakovsdague, acked, will ping him in openstack-infra13:47
*** otter768 has quit IRC13:47
bauzasjohnthetubaguy: so, I'm just iterating over those and creating patches if needed13:47
bauzasmarkus_z: you could be interested in ^13:47
*** jaosorior has quit IRC13:48
*** jaosorior has joined #openstack-nova13:48
*** daemontool__ has joined #openstack-nova13:49
*** daemontool_ has quit IRC13:50
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/26616013:51
*** haomaiwang has joined #openstack-nova13:51
*** markvoelker has joined #openstack-nova13:52
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements  https://review.openstack.org/26964513:52
*** idonotknow has quit IRC13:52
openstackgerritSylvain Bauza proposed openstack/nova: Reno for Xen rename  https://review.openstack.org/26964713:53
*** daemontool__ is now known as daemontool13:54
openstackgerritNikola Dipanov proposed openstack/nova: Make sure that we always have a parent_addr set  https://review.openstack.org/26965013:54
ndipanovlennyb, any luck testing my patch yesterday?13:54
ndipanovlennyb, I realize there is another failure13:54
*** burgerk has joined #openstack-nova13:55
*** liusheng has quit IRC13:55
ababichsdague: Hi, couldn't you look at https://review.openstack.org/#/c/255855/3 , have a little comment for you13:56
tdurakovPaulMurray paul-carlton2 eliqiao shaohe_feng pkoniszewski jlanoux yuntongjin mdbooth kashyap kmartin claudiub mikal live-migration meeting will start in 4 minutes13:56
*** Marga_ has joined #openstack-nova13:57
*** markvoelker has quit IRC13:57
*** mdrabe has joined #openstack-nova13:57
*** nelsnelson has joined #openstack-nova13:58
*** amotoki has quit IRC13:58
sdagueababich: ok, commented again13:58
*** markvoelker has joined #openstack-nova14:00
*** nelsnels_ has quit IRC14:00
ababich sdague: Thanks! Will add a proper comment now14:00
*** markvoelker_ has joined #openstack-nova14:01
*** EinstCrazy has quit IRC14:02
*** efoley_ has joined #openstack-nova14:02
*** EinstCrazy has joined #openstack-nova14:02
*** efoley has quit IRC14:04
*** markvoelker has quit IRC14:05
*** daemontool has quit IRC14:06
*** daemontool has joined #openstack-nova14:07
*** rlrossit has joined #openstack-nova14:07
*** irenab_ has joined #openstack-nova14:07
*** ccesario has joined #openstack-nova14:08
*** irenab has quit IRC14:08
*** irenab_ is now known as irenab14:08
*** annegentle has joined #openstack-nova14:09
*** doug-fish has joined #openstack-nova14:10
openstackgerritEd Leafe proposed openstack/nova: Add better help text to scheduler options (1)  https://review.openstack.org/25551214:10
openstackgerritEd Leafe proposed openstack/nova: Clean up nova/conf/scheduler.py  https://review.openstack.org/25538314:10
openstackgerritEd Leafe proposed openstack/nova: Add better help text to scheduler options (2)  https://review.openstack.org/25551314:10
openstackgerritEd Leafe proposed openstack/nova: Add better help text to scheduler options (3)  https://review.openstack.org/25551414:10
openstackgerritEd Leafe proposed openstack/nova: Add better help text to scheduler options (4)  https://review.openstack.org/25551514:10
openstackgerritEd Leafe proposed openstack/nova: Add better help text to scheduler options (5)  https://review.openstack.org/25551614:10
openstackgerritEd Leafe proposed openstack/nova: Add better help text to scheduler options (6)  https://review.openstack.org/25551714:10
openstackgerritEd Leafe proposed openstack/nova: Add better help text to scheduler options (7)  https://review.openstack.org/25551814:10
*** EinstCrazy has quit IRC14:10
*** EinstCrazy has joined #openstack-nova14:10
*** akshai has joined #openstack-nova14:10
*** ctrath has joined #openstack-nova14:11
*** shyama has quit IRC14:13
*** shyama has joined #openstack-nova14:14
*** tpatzig has joined #openstack-nova14:14
markus_zbauzas: I was afk, which one could be interesting for me?14:15
*** shyama has quit IRC14:15
bauzasmarkus_z: since we discussed on reno, I created an etherpad for tracking which patches need a reno file14:15
bauzashttps://etherpad.openstack.org/p/nova-milestone-reno14:16
*** vivekd has quit IRC14:16
*** shyama has joined #openstack-nova14:16
bauzasI'm just adding new changes in a fix_reno_13b2 branch14:16
*** vivekd has joined #openstack-nova14:16
*** lucasagomes is now known as lucas-afk14:16
*** EinstCrazy has quit IRC14:17
openstackgerritAnna Babich proposed openstack/python-novaclient: Functional tests for flavors with public and non-public access  https://review.openstack.org/26006614:17
markus_zbauzas: ah, ok, I take a quick look14:17
edleafealex_xu: sorry I missed the meeting. Too early for me! But I've read the scrollback, and will help review the patches you mentioned.14:17
*** EinstCrazy has joined #openstack-nova14:18
tpatzigjohnthetubaguy: as already shortly discussed in the last meeting: https://review.openstack.org/#/c/267673/14:18
*** EinstCrazy has quit IRC14:18
openstackgerritChuck Carmack proposed openstack/nova-specs: REST API changes for user settable server description  https://review.openstack.org/26966414:18
tpatzigi can also move it to newton14:18
*** EinstCrazy has joined #openstack-nova14:18
alex_xuedleafe: thanks14:18
markus_zbauzas: If you find time, the "scheduler config options help text" series looks ready to merge from my point of view. https://review.openstack.org/#/c/255518/514:19
ccarmackalex_xu: updated spec ^14:19
bauzasheh :)14:19
alex_xuccarmack: checking now14:19
*** ljxiash has quit IRC14:19
*** ljxiash_ has joined #openstack-nova14:19
bauzasmarkus_z: like I said previously, I'm mostly just have time for the milestone cut, not for other reviews honestly :(14:20
*** Daisy has joined #openstack-nova14:20
markus_zbauzas: totally fair14:20
bauzasmarkus_z: but that's something we can review f2f during the midcycle14:20
bauzasmarkus_z: I'd love to help those patches be merged quickly14:20
markus_zmilestone cut has priority, just wanted to make you aware of it14:21
openstackgerritNikola Dipanov proposed openstack/nova: Make sure that we always have a parent_addr set  https://review.openstack.org/26919114:22
bauzasmarkus_z: you will be at the midcycle ?14:22
openstackgerritSylvain Bauza proposed openstack/nova: Reno for lock policy  https://review.openstack.org/26966714:22
markus_zbauzas: yepp, finally got the approval :)14:22
bauzas\o/14:23
*** jichen has quit IRC14:23
bauzasmarkus_z: so, don't worry, we could just take time to do a review time14:23
*** baoli has joined #openstack-nova14:23
markus_zbauzas: yepp, cool14:23
dimssdague : no worries14:25
openstackgerritDaniel Berrange proposed openstack/os-vif: Add formal classes for each of the types of VIF backend config  https://review.openstack.org/26967014:27
openstackgerritDaniel Berrange proposed openstack/os-vif: Fix definition of subnet object to not be untyped strings  https://review.openstack.org/26967114:27
*** annegentle has quit IRC14:27
alex_xusdague: johnthetubaguy would you like take a look at this https://review.openstack.org/269664 it adds one more bit enable update description in rebuild action. As I think if the name is changed when rebuild, a meaningful description which matching new name may required by user14:27
*** pratikmallya has joined #openstack-nova14:27
ccarmackalex_xu: does the spec look ok?14:27
ansiwenrgerganov: can you explain teruhisa, why you wanted to merge your patch first?14:27
sdaguealex_xu: honestly, I'd leave that up to the user14:27
*** diogogmt has quit IRC14:28
alex_xuccarmack: yea, looks like ok, I just add one comment if we can make more clear usecase for it14:28
*** baoli_ has joined #openstack-nova14:28
sdaguealex_xu: yeh, your wording is good14:28
sdagueI'm reading now, and it does make it more clear14:28
markus_zbauzas: The git queries you did in the milestone etherpad start from the mitaka-1 milestone, right?14:28
alex_xusdague: ok, thanks :)14:28
*** vladikr has joined #openstack-nova14:29
bauzasmarkus_z: yup14:29
*** pratikma_ has joined #openstack-nova14:29
openstackgerritDaniel Berrange proposed openstack/nova: WIP: network: add helper module for converting to os-vif object model  https://review.openstack.org/26967214:29
markus_zbauzas: https://review.openstack.org/#/c/165750/ for example was merged until mitaka-1 but doesn't have a reno file14:29
*** daemontool_ has joined #openstack-nova14:30
*** [1]Thelo has joined #openstack-nova14:30
markus_zbauzas: I would expect a reno file with a follow up patch now, or is that wrong?14:31
*** baoli has quit IRC14:31
*** mriedem_away is now known as mriedem14:31
*** daemontool has quit IRC14:31
*** pbrooko has quit IRC14:31
bauzasmarkus_z: meh14:32
*** pratikmallya has quit IRC14:32
bauzasmarkus_z: does that sound my revision range is wrong ?14:32
rlrossitnot sure if someone more knowledgable than I can answer, but... is there any reason why numa_topology in the compute node object (https://github.com/openstack/nova/blob/master/nova/objects/compute_node.py#L74) is a string and not a NUMATopology (or NUMATopologyLimits) object?14:32
bauzasrlrossit: old story14:32
bauzasrlrossit: tl;dr it was a mistake14:33
*** Thelo has quit IRC14:33
*** [1]Thelo is now known as Thelo14:33
ndipanovbauzas, no that's not true14:33
ndipanovbecause it's quite difficult to migrate fields in objects14:33
markus_zbauzas: It would be right if the changes which merged before mitaka-1 had all the proper reno files I guess.14:33
ndipanovbefore we have objects used everywhere for a certain model14:33
alex_xuccarmack: would you mind update that a little bit, that goods for other review to understand the use-case14:33
rlrossitndipanov: so did NUMATopology come after numa_topology?14:33
ccarmackalex_xu: yup, doing it now14:34
rlrossitand the obj_make_compatible of that would be a nightmare?14:34
ndipanovrlrossit, so it came before all code used objects to access compute_node14:34
markus_zbauzas: Right now it excludes those and the ops don't get any notification for them14:34
alex_xuccarmack: thanks14:34
*** daemontool_ has quit IRC14:34
bauzasmarkus_z: it was probably down under the radar when I looked at those14:34
*** daemontool_ has joined #openstack-nova14:34
openstackgerritChuck Carmack proposed openstack/nova-specs: REST API changes for user settable server description  https://review.openstack.org/26966414:35
ndipanovand as you know rlrossit we can't easily change the type of fields14:35
ndipanovon objects14:35
ndipanovso put those 2 together14:35
rlrossitndipanov: +1 yup14:35
ccarmacksdague, alex_xu ^14:35
rlrossitndipanov: so should I add a note in there saying 1) why it's a string, and 2) put a TODO to change that over to numa_topology if ComputeNode ever makes its way to 2.0?14:35
ndipanovrlrossit, yeah taht would make sense14:36
markus_zbauzas: So maybe just a new section with the range of mitaka-0 to mitaka-1?14:36
bauzasndipanov: well, I remember that story you know14:36
alex_xuccarmack: done14:36
openstackgerritAnna Babich proposed openstack/python-novaclient: Functional tests for os-services  https://review.openstack.org/25585514:36
bauzasndipanov: it doesn't really explain why we provided a string initially14:36
bauzasbut that's old story14:36
*** daemontool_ has quit IRC14:37
ndipanovbauzas, what are you talking about? string is a good decision (as opposed to a different table)14:37
*** pbrooko has joined #openstack-nova14:37
ndipanovwhat is bad that it's a string field and not an object field14:37
ccarmackmriedem wants me to update the change log, one more update14:37
rlrossitndipanov: I would expect to see a DictField14:37
*** daemontool_ has joined #openstack-nova14:37
bauzasndipanov: a string in the DB model is fine, a string as an object field is another point14:38
ndipanovright14:38
*** teju has quit IRC14:38
*** vivekd_ has joined #openstack-nova14:39
*** baoli_ has quit IRC14:39
ndipanovthat might have been because the first version of the patches was not using objects at all14:39
ndipanovthat may have been a mistake...14:40
openstackgerritJacky_lei_zhang proposed openstack/nova: Fix support customer ip protocol when create security-rule using nova-cli  https://review.openstack.org/26668714:40
ndipanovand by the time we realized that's a mistake - object changes were merged already14:41
*** busterswt has joined #openstack-nova14:41
*** vivekd has quit IRC14:42
*** vivekd_ is now known as vivekd14:42
rlrossitndipanov: is that field supposed to be a NUMATopology, or a NUMATopologyLimits? I'm changing over https://review.openstack.org/#/c/265521/7/nova/tests/unit/compute/test_resource_tracker.py and it's yelling at me saying numa_topology is coming across as NUMATopologyLimits14:42
*** davideagnello has joined #openstack-nova14:43
ndipanovNUMATopology14:43
*** pbrooko_ has joined #openstack-nova14:43
mriedemccarmack: so you're just updating the spec to add the rebuilt api - i'm assuming that came up when reviewing the actual code change?14:44
markus_zmriedem: Would you maybe take a look at the libvirt AIO mode changes? https://review.openstack.org/#/q/topic:bp/libvirt-aio-mode,n,z14:44
ccarmackmriedem: yes14:44
mriedemmarkus_z: i'll add it to the queue14:45
ccarmackmriedem: I'm ready to commit the spec with the revision table update14:45
markus_zthanks14:46
*** burt has joined #openstack-nova14:46
*** pbrooko has quit IRC14:46
*** PaulMurray has joined #openstack-nova14:46
*** thangp has joined #openstack-nova14:47
rlrossitdims: I don't want to be pushy, but could you take a look at https://review.openstack.org/#/c/266960/ and let me know what you think? I want to use that as reference for pushing Neutron on it, and I want to get it in M still14:47
PaulMurraytdurakov, thanks14:47
tdurakovPaulMurray, np14:47
openstackgerritChuck Carmack proposed openstack/nova-specs: REST API changes for user settable server description  https://review.openstack.org/26966414:48
*** davideagnello has quit IRC14:48
ccarmackmriedem, alex_xu ^14:48
*** jacky-zhang has quit IRC14:48
*** mrkz has joined #openstack-nova14:49
*** jacky-zhang has joined #openstack-nova14:50
*** notmorgan has quit IRC14:51
openstackgerritRyan Rossiter proposed openstack/nova: Add note to ComputeNode.numa_topology  https://review.openstack.org/26968514:52
rlrossitndipanov: ^ let me know what you think14:52
*** ducttape_ has joined #openstack-nova14:53
ndipanovrlrossit, looks good14:54
*** eharney has quit IRC14:54
*** diogogmt has joined #openstack-nova14:54
*** knikolla has joined #openstack-nova14:55
alex_xuccarmack: yea, thanks mriedem point that out.14:55
* alex_xu end the work of day, will review the code tomorrow14:55
*** esracelik has quit IRC14:56
bauzasrlrossit: +2'd, mriedem you could be interested in this nit https://review.openstack.org/#/c/269685/114:58
mriedem-100014:58
bauzasoh snap14:58
rlrossitwow everyone loves it14:58
bauzasjinxed again by mriedem14:58
bauzasI have a curse on me !14:58
*** akshai has quit IRC14:59
*** jacky-zhang has quit IRC14:59
*** akshai has joined #openstack-nova15:00
*** notmorgan has joined #openstack-nova15:00
*** notmorgan has quit IRC15:01
*** notmorgan has joined #openstack-nova15:01
*** notmorgan has quit IRC15:01
*** notmorgan has joined #openstack-nova15:01
mriedemndipanov: +2 on https://review.openstack.org/#/c/269191/15:02
ildikovndipanov: hi. friendly reminder: https://review.openstack.org/#/c/193134/ :)15:02
mriedemwith grumblings inline about why object fields that are nullable=True aren't also default=None15:02
*** baoli has joined #openstack-nova15:03
*** tdurakov has quit IRC15:03
*** Daisy has quit IRC15:03
*** baoli has quit IRC15:03
*** pumaranikar has joined #openstack-nova15:03
*** annegentle has joined #openstack-nova15:04
*** baoli has joined #openstack-nova15:04
mriedemdims: rlrossit: question about object fields, is there any good reason why we shouldn't make the default value on a field None when nullable=True?15:04
*** nkrinner has quit IRC15:04
mriedemto avoid lazy-load failures?15:04
*** annegent_ has joined #openstack-nova15:04
*** mragupat has joined #openstack-nova15:04
mriedemjaypipes: ^15:05
*** tdurakov has joined #openstack-nova15:06
*** baoli has quit IRC15:07
*** spinderbok has joined #openstack-nova15:07
*** rgerganov has quit IRC15:07
*** spinderbok has quit IRC15:07
*** baoli has joined #openstack-nova15:08
*** annegentle has quit IRC15:08
bauzasmriedem: I remember some convo with dansmith15:10
bauzasmriedem: about defaulting attributes15:10
mriedemuh oh, you invoked his name who must not be spoken while on vacatoin15:11
bauzasmriedem: IIRC, he was pretty against defaulting if not needed15:11
*** dslev has quit IRC15:11
mriedembauzas: i assume there is a good reason, i'd just like to know what it is15:11
bauzasmriedem: oh man, hopefully I didn't said that in front of a mirror 5 times15:11
mriedemit's 3 times in a dark bathroom15:11
mriedemso you're probably ok15:12
bauzasmeh15:12
*** diogogmt has quit IRC15:12
bauzasanyway, I'd bet the answer is in the logs :)15:12
*** daemontool_ has quit IRC15:12
bauzasthat helps, heh ?15:12
*** eharney has joined #openstack-nova15:13
bauzasnah, enough kidding, IIRC it's because nullifying a field doesn't mean it's set15:13
bauzasit's just for coercing15:13
*** tdurakov has quit IRC15:13
ndipanovmriedem, thanks!15:13
*** vivekd has quit IRC15:13
*** tonytan4ever has joined #openstack-nova15:14
*** PaulMurray has quit IRC15:14
*** PaulMurray has joined #openstack-nova15:14
*** zz_dimtruck is now known as dimtruck15:14
*** tdurakov has joined #openstack-nova15:14
mriedembauzas: sure, but15:14
mriedemif it's nullable, it's optional, which means do we care if it's set?15:15
mriedemit's like calling dict.get() and getting None back if the key isn't in the dict15:15
mriedemsaves us from shooting ourselves in the foot15:15
bauzasI see your point15:16
mriedemi'm assuming the reason is we fail in obvious ways if the field isn't set, which is better than failing in non-obvious ways when the field isn't set and defaults to None15:16
*** tdurakov has quit IRC15:16
bauzasyeah, I agree there are 2 things : defaulting fields automatically, and defaulting a field which can be nullable15:16
*** annegent_ is now known as annegentle15:16
bauzaswhile I disagree the former, I tend to agree the latter15:17
*** tdurakov has joined #openstack-nova15:17
bauzasmriedem: happy with patching that and CC me ?15:17
*** dslevin has quit IRC15:18
mriedemin o.vo?15:18
mriedemi'll just add it to my todo list for things to talk to superdan about15:19
*** edtubill has joined #openstack-nova15:19
*** mdnadeem has quit IRC15:19
mriedemright after who is our favorite twilight hunk15:19
*** lucas-afk is now known as lucasagomes15:19
bauzasmriedem: yeah in o.vo15:20
*** ijw has joined #openstack-nova15:21
*** ijw has quit IRC15:21
*** ijw has joined #openstack-nova15:21
* tonyb isn't sure he wants to read the scrollback 15:24
*** daemontool_ has joined #openstack-nova15:24
*** annegentle has quit IRC15:26
*** annegentle has joined #openstack-nova15:26
*** jinxing has joined #openstack-nova15:27
*** salv-orlando has joined #openstack-nova15:27
*** pradk has joined #openstack-nova15:27
*** EinstCrazy has quit IRC15:27
*** PaulMurray has quit IRC15:28
*** burgerk has quit IRC15:28
*** EinstCrazy has joined #openstack-nova15:28
*** achanda has joined #openstack-nova15:29
ndipanovmriedem, so the problem was that default kwarg doesn't really work on fields afair15:29
*** EinstCrazy has quit IRC15:30
mriedemildikov: i have meetings but put some comments in https://review.openstack.org/#/c/193134/ - i'm not done reviewing that yet though15:30
ndipanovbecause you can't really do it in the constructor since there is nothing stopping you from creating an object in python, setting a few values and calling save()15:30
*** angdraug has joined #openstack-nova15:30
openstackgerritVladik Romanovsky proposed openstack/nova: Add tag column to vifs and bdm  https://review.openstack.org/26401415:30
mriedemhowever, the changes to the api response w/o a microversion are concerning15:30
*** knikolla has quit IRC15:31
ildikovmriedem: thanks for checking15:31
ndipanovwhich would overwrite whatever is in the db with defaults15:31
ildikovmriedem: changes to the api response?15:31
*** SridharG has quit IRC15:32
*** akshai has quit IRC15:32
openstackgerritTomi Juvonen proposed openstack/python-novaclient: Added support for Nova microversion 2.16  https://review.openstack.org/25827015:32
*** efoley__ has joined #openstack-nova15:32
*** efoley_ has quit IRC15:35
sdaguedims: can you respin https://review.openstack.org/#/c/268796/ without the WIP?15:38
sdaguewe probably need some explanation as to why we're doing that15:39
dimssdague : ack, will add detailed commit message15:39
dimsyep15:39
sdagueyeh, as well as comments in the code15:39
sdagueit's super non obvious15:39
openstackgerritDaniel Berrange proposed openstack/os-vif: Fix definition of subnet object to not be untyped strings  https://review.openstack.org/26967115:39
*** achanda has quit IRC15:40
*** tdurakov has quit IRC15:40
*** READ10 has joined #openstack-nova15:40
*** tdurakov has joined #openstack-nova15:41
bauzasjohnthetubaguy: thoughts about providing a reno change for https://review.openstack.org/#/c/249013/ and https://review.openstack.org/#/c/211753/ ?15:42
bauzasjohnthetubaguy: apart saying "please run nova-manage db sync", I don't really see a comment there15:43
*** tdurakov has quit IRC15:43
bauzasI mean, it's not really needing a reno file15:43
*** otter768 has joined #openstack-nova15:43
*** tdurakov has joined #openstack-nova15:43
*** tjones has joined #openstack-nova15:43
*** PaulMurray has joined #openstack-nova15:43
ildikovmriedem: I answered your API related concerns15:44
*** yamahata has joined #openstack-nova15:44
sdaguedims: this still bugs me a little, because the part in the fork is actually a smaller amount of the runtime15:44
*** akshai has joined #openstack-nova15:45
ildikovmriedem: I think I had issues, during the modification of the translation function in the API and tempest was quite sensitive about a few things there15:46
dimssdague : wanted to avoid the print_leases in fork() guessing someone is already using that in their scripts or something15:46
cdentmeh to the N and O names.15:46
*** mtanino has joined #openstack-nova15:46
sdaguedims: ok, however the content in the fork is a cast15:47
sdagueso it should be really fast15:47
*** otter768 has quit IRC15:47
*** garyk has quit IRC15:49
*** dedery has quit IRC15:49
*** nkrinner has joined #openstack-nova15:50
tobascohm i need to access is_volume_backed_instance() defined in compute/api.py from scheduler/utils.py but i can't import it because that will cause a loop and just fail, i also don't want to move it or recreate a duplicate function in like compute/utils.py, my options are pretty much none...15:50
*** eharney has quit IRC15:50
openstackgerritwuhao proposed openstack/nova: Add 'all_state' value to 'notify_on_state_change' option  https://review.openstack.org/26972015:51
*** hemna has joined #openstack-nova15:51
*** markvoelker_ has quit IRC15:51
openstackgerritDina Belova proposed openstack/nova: Integrate OSprofiler and Nova  https://review.openstack.org/25470315:51
*** angdraug has quit IRC15:52
rlrossitmriedem: I think it's because of lazy-loading. Because not set means "I need to go to the db to know what this is", and then if it feels like loading it from the database and it's null, then it's set.15:52
rlrossitwhere if we set it to default null, then it won't try to look it up in the db15:52
openstackgerritting wang proposed openstack/python-novaclient: Add debug testenv in tox  https://review.openstack.org/26621915:53
*** efoley__ is now known as efoley15:53
mriedemildikov: yeah i meant like here https://review.openstack.org/#/c/193134/77/nova/api/openstack/compute/volumes.py - but now looking at that again, i guess it's not changing the response15:53
mriedemildikov: it's taking the first attachment from the list and using the instance_uuid and mountpoint from that15:54
ildikovmriedem: yes15:54
*** salv-orlando has quit IRC15:54
*** Marga_ has quit IRC15:54
ildikovmriedem: it gives back the same response only the internally used structure is changed15:54
sdaguemriedem: can you weigh in on https://review.openstack.org/#/c/268796/2/nova/cmd/dhcpbridge.py - I'm feeling really mixed right now, as I'm not sure I understand how it will actually fix things15:54
mriedemsdague: heh, idk either - turn on the vish signal?15:56
ndipanovildikov, commented15:57
*** burgerk has joined #openstack-nova15:57
ndipanovjohnthetubaguy, danpb https://review.openstack.org/#/c/269191/15:57
sdaguemriedem: I'm sure he drank away all those brain cells15:58
*** tpatzig has left #openstack-nova15:58
sdagueI would have :)15:58
*** houming has joined #openstack-nova15:58
*** bradjones has joined #openstack-nova15:58
*** bradjones has quit IRC15:58
*** bradjones has joined #openstack-nova15:58
mriedemndipanov: you and i had the same thought here https://review.openstack.org/#/c/193134/ in the rest api15:59
*** achanda has joined #openstack-nova15:59
mriedemwith the keys order, but it does use an OrderedDict15:59
mriedemsdague: dims: so i guess the thinking in https://review.openstack.org/#/c/268796/ is we just fork off the calls to like add and remove a lease, so we're not blocking on those in a multi-tenant env and getting out of sync in the network manager code?16:01
sdagueyeh16:01
ildikovmriedem: thanks, I'm writing the same comment to the patch16:01
mriedemi can't imagine dhcpbridge is blocking for long since it's doing an rpc cast for those operations16:01
*** baoli has quit IRC16:01
*** lucasagomes is now known as lucas-brb16:01
sdagueright, I think the bigger issue is the amount of time spent in running the script overall16:02
*** eharney has joined #openstack-nova16:02
sdaguemriedem: it is a race16:02
sdagueand only seems to happen when 2 dhcp actions are hitting within a few msecs of each other16:02
ndipanovmriedem, hmmm16:03
*** achanda has quit IRC16:03
*** boris-42 has joined #openstack-nova16:03
*** PaulMurray has quit IRC16:04
*** diogogmt has joined #openstack-nova16:04
markus_zbauzas: I added a query to https://etherpad.openstack.org/p/nova-milestone-reno and took a look at the changes16:04
dimsmriedem : right16:04
*** aix has quit IRC16:05
markus_zbauzas: I have to go earlier today, I think you will get my additions.16:05
openstackgerritSayali Lunkad proposed openstack/nova: Allows to set display_name as instance_name  https://review.openstack.org/26335316:06
ndipanovmriedem, responded16:06
*** jinxing has quit IRC16:06
*** mlavalle has joined #openstack-nova16:07
mlavallejohnthetubaguy: ping16:07
gibimikal: hi! I see your comments on https://review.openstack.org/#/c/247024/. Do I understand correclty that you have started fixing them or shall I go and respin the patch?16:07
gibimikal: btw, thanks for the review16:07
alaskimriedem: https://review.openstack.org/#/c/266060 is a quick one for you16:10
*** mjura has quit IRC16:11
*** markus_z has quit IRC16:11
ndipanovlennyb, here?16:12
tjonesthanks alaski for the reviews on ^^16:12
tjonesi hope to get rid of the last stubs.Set today16:13
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Avoid blocking dnsmasq by forking off RPC calls  https://review.openstack.org/26879616:13
openstackgerritChris Dent proposed openstack/nova: Tidy up scheduler_evolution.rst  https://review.openstack.org/26256116:13
alaskitjones: np.  It's nice to see mox fade away16:13
tjones:-)16:14
*** zenoway has quit IRC16:15
dimssdague : mriedem : i've added as much info as i could to https://review.openstack.org/26879616:15
*** gongysh has joined #openstack-nova16:16
*** gtt116__ has joined #openstack-nova16:18
openstackgerritTracy Jones proposed openstack/nova: Stop using mox stubs in test_configdrive  https://review.openstack.org/26547616:19
*** smatzek has quit IRC16:19
*** zenoway has joined #openstack-nova16:19
mriedemildikov: ndipanov: went through it again, clarified the rest api part16:22
mriedemthe new dict that's coming out of the volume api is causing confusion, but the code is working16:22
*** gtt116_ has quit IRC16:22
*** RuiChen has quit IRC16:23
ildikovmriedem: I added tests for that code to be sure16:23
*** RuiChen has joined #openstack-nova16:23
*** mirrorbo1 is now known as mirrorbox16:23
ildikovmriedem: I'm going through the comments now16:23
*** zenoway has quit IRC16:23
*** smatzek has joined #openstack-nova16:24
mriedemwell, tempest would also fail if you changed the response, as you found out16:24
*** mirrorbox has quit IRC16:24
*** mirrorbox has joined #openstack-nova16:24
*** fawadkhaliq has joined #openstack-nova16:24
openstackgerritMatt Riedemann proposed openstack/nova: change set_stubs to use stub_out in vmwareapi/stubs.py  https://review.openstack.org/26606016:26
mriedemalaski: got it16:26
*** pbrooko_ has quit IRC16:27
*** Jeffrey4l has joined #openstack-nova16:27
*** lucas-brb is now known as lucasagomes16:28
*** bdrich has joined #openstack-nova16:28
*** Jeffrey4l has quit IRC16:28
*** garyk has joined #openstack-nova16:29
*** Jeffrey4l has joined #openstack-nova16:29
tjonesmriedem: thanks for fixing  my alignment issue16:29
mriedemyw16:29
mriedemthanks for suffering my ocd16:29
tjones:-)16:29
*** diana_clarke has joined #openstack-nova16:30
*** mdrabe has quit IRC16:30
tjones2 down - 2 to go…16:30
tjoneswith removing mox i mean16:31
bdrichHi, looking for some help. Running a test which spins up vms, however the vms are not loading all data. For instance, "availability zones". I have logs for viewing, if anyone would like to see them.16:31
mriedem#success nova liberty 12.0.1 released http://lists.openstack.org/pipermail/openstack-announce/2016-January/000922.html16:34
openstackstatusmriedem: Added success to Success page16:34
*** baoli has joined #openstack-nova16:35
ndipanovmriedem, respnded to your response to my response to your respooooaaaa16:35
*** markvoelker has joined #openstack-nova16:36
mriedem:)16:36
*** irenab has quit IRC16:36
ndipanovmriedem, https://review.openstack.org/#/c/230919/ <- check this one when you get a chance16:36
*** teju has joined #openstack-nova16:37
*** salv-orlando has joined #openstack-nova16:37
mriedemdo we need that given we have https://review.openstack.org/#/c/252565/ ?16:38
edleafemriedem: not sure why https://review.openstack.org/#/c/269240/ is failing. The log at http://logs.openstack.org/40/269240/2/check/gate-grenade-dsvm/ff1377b/logs/old/screen-n-cpu.txt.gz?level=WARNING show that the compute node record was not found. No idea why that would be, though.16:39
edleafemriedem: would appreciate your log-reading skills16:39
*** alejandrito has quit IRC16:41
mriedemedleafe: did you check the end of grenade.sh.txt?16:41
mriedemoh nvm16:42
mriedemhttp://logs.openstack.org/40/269240/2/check/gate-grenade-dsvm/ff1377b/console.html#_2016-01-19_15_37_50_52516:42
*** irenab has joined #openstack-nova16:42
mriedemedleafe: test_volume_boot_pattern + ssh failed is like the top bug in the system16:42
mriedemso just recheck16:42
*** gszasz has quit IRC16:42
mriedemhttp://status.openstack.org/elastic-recheck/index.html#135557316:42
edleafemriedem: ok, thanks16:42
edleafemriedem: already rechecked once, with the same result. Wanted to make sure I wasn't missing something.16:43
mriedemedleafe: only other thing i'd check on that type of change is the n-net logs, but i didn't see any errors there16:44
*** Marga_ has joined #openstack-nova16:44
mriedemif you messed up the config options there would be lots of failures in all of the jobs probably (all of the n-net jobs that is)16:44
edleafemriedem: yeah, I looked through all of those - didn't want to be embarrassed by you finding a bug that I missed again :)16:45
*** hemna has quit IRC16:45
mriedemin my 1:1 today i was told that at times i use a hammer when i should be using soft warm mittens and giving hugs16:45
mriedemedleafe: so never fear16:45
*** mgoddard_ has joined #openstack-nova16:46
mriedemit's a new year and a new, cuddlier mriedem16:46
*** mdrabe has joined #openstack-nova16:46
*** annegentle has quit IRC16:46
kashyapmriedem: When you get a sec later, for this bug - https://bugs.launchpad.net/nova/+bug/1524898, is it possible to run that test job with QEMU 2.3 ?  QEMU 2.0 is 2 years older.16:46
openstackLaunchpad bug 1524898 in OpenStack Compute (nova) "Volume based live migration aborted unexpectedly" [High,Confirmed]16:46
kashyapMy guess is 'not at the moment'16:47
*** zakora has quit IRC16:47
edleafemriedem: somehow I find that much scarier16:47
*** mgoddard__ has quit IRC16:49
dimsmriedem : alaski : who is responsible for "Citrix XenServer CI"? seeing some "Hash Sum mismatch" errors16:49
mriedemkashyap: we have a fedora (21?) job in the experimental queue, i'm not sure what version of qemu is on that though16:50
*** annegentle has joined #openstack-nova16:50
mriedemthat's the only way i know of though to test newer libvirt/qemu is on the fc job16:50
mriedemdims: BobBall16:50
kashyapmriedem: Afraid, F21 is End-Of-Life :-)16:50
mriedemsomeone at one point was working on an fc22 job16:50
mriedemor maybe fc2316:50
mriedemchecking16:50
kashyapOkay, I'll check with Infra tomorrow16:50
*** irenab has quit IRC16:51
*** nelsnelson has quit IRC16:51
mriedemkashyap: oh, but16:51
*** salv-orlando has quit IRC16:51
kashyap?16:51
mriedemthat's a test that fails in a multi-node live migration job16:51
*** yamahata has quit IRC16:51
mriedemwhich is only ubuntu images right now16:51
mriedemso no dice with fc being in that job16:52
openstackgerritVladik Romanovsky proposed openstack/nova: neutron: pci_request logic considers 'direct-physical' vnic type  https://review.openstack.org/26258416:52
openstackgerritVladik Romanovsky proposed openstack/nova: pci: Add utility method for getting the MAC addr  https://review.openstack.org/26258516:52
openstackgerritVladik Romanovsky proposed openstack/nova: neutron: Update the port with a MAC address for PFs  https://review.openstack.org/26258616:52
openstackgerritVladik Romanovsky proposed openstack/nova: pci: changing the claiming and allocation logic for PF/VF assignment  https://review.openstack.org/24901816:52
openstackgerritVladik Romanovsky proposed openstack/nova: Add a vnic type for PF passthrough and a new libvirt vif driver  https://review.openstack.org/26258316:52
openstackgerritVladik Romanovsky proposed openstack/nova: Make sure that we always have a parent_addr set  https://review.openstack.org/26919116:52
openstackgerritVladik Romanovsky proposed openstack/nova: pci: use context when performing oprations of pci devices  https://review.openstack.org/26976416:52
kashyapmriedem: Doesn't have to be Fedora.  If those Ubuntu imges can manage to have newest stable QEMU that'd be useful16:52
*** nelsnelson has joined #openstack-nova16:52
mriedemyou'd have to probably hook in the cloud archive or something16:52
BobBalldims: Which changeset?16:53
kashyapOkay, I made a note to follow up who can do that.16:53
*** rcernin has quit IRC16:53
mriedemhttp://ubuntu-cloud.archive.canonical.com/ubuntu/dists/trusty-updates/liberty/main/binary-i386/Packages16:53
dimsBobBall : https://review.openstack.org/#/c/268796/16:54
mriedemqemu-kvm Version: 1:2.3+dfsg-5ubuntu9~cloud016:54
*** nkrinner has quit IRC16:54
dimsBobBall : direct link http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/96/268796/3/3473/index.html16:54
mriedemkashyap: so if you could get the cloud archive enabled you could update libvirt/qemu16:54
*** apoorvad has joined #openstack-nova16:54
BobBalldims: I believe that's triggered when the ubuntu archive is in the process of being updated?16:55
*** matrohon has quit IRC16:55
kashyapmriedem: Noted, I don't know who can do that - maybe Ubuntu Virt maintainers (the name that pops up in my head is Serge Hallyn)16:55
*** annegentle has quit IRC16:55
mriedemkashyap: you might be better off starting a thread in the ML16:55
dimsBobBall : so just a recheck?16:55
BobBalldims: i.e. we download the metadata first, it gets updated, then we download some of the package listings and check the new hash against the old metadata16:55
BobBalldims: Yup16:55
dimsBobBall : ack thanks16:55
mriedemkashyap: sdague would probably have quick answers to why we don't use the cloud archive in the gate16:55
*** teju has quit IRC16:55
kashyapmriedem: Okay, will go do that16:55
*** hemna has joined #openstack-nova16:57
*** vishwanathj has joined #openstack-nova16:57
*** scheuran has quit IRC16:57
*** david-lyle has quit IRC16:58
*** houming has quit IRC16:59
*** dslevin has joined #openstack-nova16:59
*** gongysh has quit IRC16:59
*** mgoddard_ has quit IRC17:00
*** mgoddard has joined #openstack-nova17:00
*** SridharG has joined #openstack-nova17:00
*** claudiub has joined #openstack-nova17:00
*** e0ne has quit IRC17:01
*** apoorvad has quit IRC17:02
*** sahid has quit IRC17:03
*** salv-orlando has joined #openstack-nova17:03
*** david-lyle has joined #openstack-nova17:06
*** jlanoux has quit IRC17:07
kashyap(I made a typo above: latest stable QEMU is 2.5, not 2.3)17:08
*** rcernin has joined #openstack-nova17:09
*** yamahata has joined #openstack-nova17:09
*** PaulMurray has joined #openstack-nova17:11
*** Daisy has joined #openstack-nova17:15
mriedemndipanov: i'm not sure what to do about check_detach here https://review.openstack.org/#/c/193134/77/nova/volume/cinder.py17:17
mriedemi added some notes, it looks like if you pass the wrong attachment_id, cinder doesn't error out17:17
mriedemwould be nice if some cinder people, cough hemna scottda cough, would look at that17:17
hemnahey17:17
hemnasorry had the day off yesterday17:17
* hemna checks17:18
mriedemyeah, bankers hours at HPE17:18
mriedemhemna: the check_detach discussion here https://review.openstack.org/#/c/193134/77/nova/volume/cinder.py17:18
hemna:)17:18
*** yassine__ has quit IRC17:18
ildikovmriedem: another question to check_detach17:18
mriedemalthough really i was just looking at attachment not found17:19
*** Daisy has quit IRC17:19
ildikovmriedem: if I change that VolumeUnattached exception there there are tests I need to change which was not added by me meaning that it changes already existing behavior17:19
ildikovmriedem: not behavior just the exception17:19
ildikovmriedem: is it acceptable or should I leave it for now?17:19
mriedemildikov: maybe just put a todo in this change to add a better exception at some point17:20
ildikovmriedem: ok, I will change it back then, tnx17:20
mriedemwhere is the actual detach_volume method for the lvm backend in cinder?17:21
*** shyama has quit IRC17:21
*** ssurana has joined #openstack-nova17:21
*** Jeffrey4l has quit IRC17:22
scottdaSorry, bankers hours as well...I'll have a look in a little while.17:23
*** derekh has quit IRC17:23
dimsmriedem : sdague : seen this? message:"libvirtError: unsupported configuration: guest and host CPU are not compatible:" AND voting:117:25
dimsmriedem : sdague : http://logs.openstack.org/96/268796/3/check/gate-tempest-dsvm-multinode-full/7587dc3/logs/screen-n-cpu.txt.gz?level=ERROR#_2016-01-19_16_54_33_33017:25
*** diogogmt has quit IRC17:25
mriedemdims: yeah17:25
hemnaso fwiw, the multi attach attribute has been part of the Cinder API since https://github.com/openstack/cinder/commit/10d54216871fde27172fbcb1a3c9bfec59b824c217:25
mriedemdims: https://bugs.launchpad.net/openstack-gate/+bug/153579917:26
openstackLaunchpad bug 1535799 in devstack "internap doesn't support the same cpu flags as other providers" [Undecided,In progress]17:26
dimsmriedem : ah thanks, you are ahead of me :)17:26
openstackgerritChris Dent proposed openstack/nova: Add ResourceClassList versioned object  https://review.openstack.org/26349817:26
openstackgerritChris Dent proposed openstack/nova: Add resource_classes table to api database  https://review.openstack.org/26349617:26
openstackgerritChris Dent proposed openstack/nova: Versioned objects support for ResourceClass  https://review.openstack.org/26349717:26
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova: Added new scheduler filter: AggregateTypeExtraSpecsAffinityFilter  https://review.openstack.org/18927917:30
mriedemso i guess only like 3 volume drivers actually use the attachment in detach_volume17:30
mriedemhemna: yeah we know multiattach has been in cinder for a long time, the question i had in check_detach there was how wise it is to not validate the instance uuid vs the attachments in the volume17:31
hemnaI put my $0.02 in17:31
mriedemok17:31
hemnaat this point we shouldn't boil the ocean17:31
hemnamriedem, you are correct17:33
mriedemok, i tend to agree with you on leaving it in17:35
mriedemand cleaning up later17:35
hemnayah subsequent patch if anything17:35
mriedemi tend to think check_detach should be using begin_detaching instead of it's own status checking, we could avoid that - let cinder do the status checking since it's holding the current data17:36
*** diogogmt has joined #openstack-nova17:36
mriedembut that's a later nova cleanup imo17:36
mriedemand nova can deal with validating the instance uuid vs the volume attachments list17:36
*** davideagnello has joined #openstack-nova17:37
*** dslevin has quit IRC17:37
ildikovmriedem: that looks like a good way to go17:37
*** carolynv_ has quit IRC17:37
*** apoorvad has joined #openstack-nova17:37
*** Marga_ has quit IRC17:37
johnthetubaguyndipanov: ah, good good, that fix is +W now, were there two bugs you said?17:37
*** Marga_ has joined #openstack-nova17:38
johnthetubaguybauzas: looking17:38
ndipanovyes - the next fix after that - but I think vladikr is going to resping17:38
ndipanovjohnthetubaguy, ^17:38
*** sfinucan has quit IRC17:39
johnthetubaguybauzas: yeah, agreed there isn't too much to say for those two, probably skip them17:39
*** _RuiChen has joined #openstack-nova17:40
johnthetubaguymriedem: just wanting to ping about how things are looking CI wise, in terms of tagging mitaka-2?17:40
johnthetubaguymriedem: I am trying to track any blockers in here, in case you spot any: https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking17:40
*** RuiChen has quit IRC17:42
*** pratikma_ has quit IRC17:42
mriedemfor thursday?17:43
mriedemthere is a fix going through devstack for a problem in the multinode jobs17:43
mriedemotherwise i haven't heard of anything17:43
johnthetubaguymriedem: well, usually try to pick today or tomorrow for the release17:44
*** otter768 has joined #openstack-nova17:44
mriedemoh blockers as in like rc potential?17:44
johnthetubaguymriedem: the PCI breakage ndipanov was talking about, kinda feels bad enough to wait for17:44
mriedemyeah17:44
mriedemgot it17:44
johnthetubaguyyeah, more stuff like that, sometimes the CI breakages point at them I guess17:44
ndipanovjohnthetubaguy, well it's on it's way to the gate now17:45
johnthetubaguyndipanov: ack, yeah, glad to see that17:45
johnthetubaguyndipanov: did you say there was another one?17:45
ndipanovjohnthetubaguy, and the other thing is something vladik picked up on when he was doing functional tests for the PF feature17:45
*** nic has joined #openstack-nova17:45
ndipanovand I think it's only hit when actually using it17:45
johnthetubaguyndipanov: ah, OK17:46
ndipanovbut moshele commented on one of the patches that there is _another_ CI fail17:46
ndipanovbut I was not able to grab any of the CI folks today17:47
johnthetubaguyah, OK, will keep an eye out for that17:47
mriedemmarkus_z isn't around, but i thought we had another bug triage person this week17:47
mriedemwhich would be auggy17:48
auggythat's me!17:48
*** jistr has quit IRC17:48
mriedemauggy: i guess be on the lookup for critical regressions that we should hold up m-2 for17:49
*** lpetrut has quit IRC17:49
*** otter768 has quit IRC17:49
*** ZZelle has quit IRC17:49
auggyshould i bring them to people's attention here?17:49
*** ZZelle has joined #openstack-nova17:50
*** burgerk_ has joined #openstack-nova17:50
auggyit's been pretty slow on the bug front, mostly "i got an api error that told me to file a bug" and i'm still working on reproducing some of the others to verify them17:50
*** burgerk has quit IRC17:51
mriedemtjones: can you see why the vmware nsx ci isn't reporting back on https://review.openstack.org/#/c/168024/ ? i rechecked that yesterday17:51
openstackgerritAndrey Pavlov proposed openstack/nova: remove EC2 legacy code  https://review.openstack.org/26642517:51
*** angdraug has joined #openstack-nova17:52
*** unicell has quit IRC17:53
tjonessure17:53
tjones@mriedem sure i'll check17:53
*** cdent has quit IRC17:53
johnthetubaguyauggy: that sounds great, mostly tracking the blocking things in here: https://etherpad.openstack.org/p/mitaka-nova-priorities-tracking17:54
*** cdent has joined #openstack-nova17:54
mriedemauggy: really anything that like merges this week and all of a sudden you can't boot an instance or start the compute service17:54
mriedemmajor regressions which we should hold the milestone release for17:55
auggyok cool, haven't seen anything like that17:55
*** ljxiash_ has quit IRC17:55
auggymost of these so far when i've tried to repro and then searched for more info have just looked like config issues17:55
auggybug triaging is very educational!17:56
*** rmart04_ has joined #openstack-nova17:59
*** rmart04_ has quit IRC17:59
*** rmart04 has quit IRC18:00
ildikovmriedem: is there a way to test the lower version of detach_volume with rpc api without rewriting _test_compute_api?18:00
*** mrkz has quit IRC18:02
mriedemildikov: in the past i've just written a new test case using mock for the version wrinkles rather than trying to shoe-horn the change into the common code18:02
*** busterswt has quit IRC18:02
*** mrkz has joined #openstack-nova18:02
openstackgerritEd Leafe proposed openstack/nova: doc: remove detail about extensions  https://review.openstack.org/26737018:02
mriedemildikov: e.g. https://review.openstack.org/#/c/192396/7/nova/tests/unit/network/test_rpcapi.py18:03
mriedembrb18:03
mlavallejohnthetubaguy: ping18:04
*** diogogmt has quit IRC18:04
mlavallealaski: tomorrow, what time is the cells v2 meeting? 1700 or 2100 UTC?18:05
ildikovmriedem: thanks for the pointer, only item on the list that's missing18:05
*** diogogmt has joined #openstack-nova18:05
*** zenoway has joined #openstack-nova18:06
*** fawadkhaliq has quit IRC18:08
alaskimlavalle: tomorrow is 170018:08
mlavallealaski: thanks. Looking to get up to speed on the porgress so far18:09
mlavalleprogress^^^^18:09
*** pratikmallya has joined #openstack-nova18:10
*** zenoway has quit IRC18:11
openstackgerritDaniel Berrange proposed openstack/nova: WIP: os-vif: convert libvirt driver to use os-vif library  https://review.openstack.org/26967218:12
*** PaulMurray has quit IRC18:12
*** BobBall is now known as BobBall_AWOL18:13
*** danpb has quit IRC18:16
tjonesmriedem: NSX CI is running but behind.  we bumped up your job so it should run next18:17
*** e0ne has joined #openstack-nova18:17
mriedemok18:17
mriedemthanks18:17
*** diana_clarke has quit IRC18:17
*** eil397 has joined #openstack-nova18:18
*** pradk has quit IRC18:19
*** bdrich has quit IRC18:19
openstackgerritBalazs Gibizer proposed openstack/nova: doc: add devref about versioned notifications  https://review.openstack.org/26378818:20
openstackgerritBalazs Gibizer proposed openstack/nova: Add service status notification  https://review.openstack.org/24567818:20
openstackgerritBalazs Gibizer proposed openstack/nova: Generate doc for versioned notifications  https://review.openstack.org/26515118:20
openstackgerritBalazs Gibizer proposed openstack/nova: Add infra for versioned notifications  https://review.openstack.org/24702418:20
openstackgerritBalazs Gibizer proposed openstack/nova: Adds json sample for the versioned notifications  https://review.openstack.org/24816718:20
openstackgerritBalazs Gibizer proposed openstack/nova: Make emitting versioned notifications configurable  https://review.openstack.org/24756418:21
*** knikolla has joined #openstack-nova18:21
*** salv-orlando has quit IRC18:23
*** efoley has quit IRC18:25
*** rotbeard has quit IRC18:25
*** pradk has joined #openstack-nova18:26
*** pradk has quit IRC18:26
*** unicell has joined #openstack-nova18:27
*** rdopiera has quit IRC18:28
*** baoli has quit IRC18:29
*** knikolla has quit IRC18:29
*** paul-carlton2 has quit IRC18:29
*** zhiyan has quit IRC18:33
*** tpeoples has quit IRC18:34
*** auggy has quit IRC18:34
*** belliott has quit IRC18:35
*** pumarani- has quit IRC18:35
*** zhiyan has joined #openstack-nova18:36
*** tpeoples has joined #openstack-nova18:36
*** belliott has joined #openstack-nova18:36
*** auggy has joined #openstack-nova18:37
*** annegentle has joined #openstack-nova18:38
*** angdraug has quit IRC18:38
*** mgoddard_ has joined #openstack-nova18:39
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: libvirt: Allow multiple volume attachments  https://review.openstack.org/19313518:41
openstackgerritIldiko Vancsa proposed openstack/nova: Pass attachment_id to Cinder when detach a volume  https://review.openstack.org/19313418:41
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: comp: Allow volume multiattach  https://review.openstack.org/26758718:41
*** salv-orlando has joined #openstack-nova18:41
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach  https://review.openstack.org/15303818:41
openstackgerritIldiko Vancsa proposed openstack/nova: Pass host when call attach to Cinder  https://review.openstack.org/25627318:41
*** rlrossit has quit IRC18:41
*** mgoddard has quit IRC18:42
ildikovmriedem: new version is up ^18:45
*** rlrossit_ has joined #openstack-nova18:46
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: libvirt: Allow multiple volume attachments  https://review.openstack.org/19313518:46
openstackgerritIldiko Vancsa proposed openstack/nova: Pass attachment_id to Cinder when detach a volume  https://review.openstack.org/19313418:46
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: comp: Allow volume multiattach  https://review.openstack.org/26758718:46
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach  https://review.openstack.org/15303818:46
openstackgerritIldiko Vancsa proposed openstack/nova: Pass host when call attach to Cinder  https://review.openstack.org/25627318:46
mriedemildikov: yup, was just reading through the next one on passing the host18:46
mriedemand trying to sort out what's in the connector dict18:46
mriedemand feeling sad that we don't validate any of that information18:47
*** vishwanathj has quit IRC18:47
ildikovmriedem: I only rebased as your tempest patch failed again with merge issues18:47
ildikovmriedem: I hear ya18:48
ildikovat least for the Cinder volume I can check the API docs and hope it's correct18:48
ildikovconnector is more tricky :(18:48
*** pradk has joined #openstack-nova18:49
vladikrjaypipes, when you'll have a moment, please look at the libosinfo series. It has danpb's +2 https://review.openstack.org/#/q/topic:bp/libvirt-hardware-policy-from-libosinfo18:51
*** boris-42 has quit IRC18:53
*** salv-orlando has quit IRC18:54
mriedemildikov: yeah for libvirt the connector comes from os-brick18:55
mriedemthe ComputeDriver docstring does say what should be in the dict18:55
ildikovmriedem: ah, ok, got it18:55
ildikovmriedem: I got the hints from hemnafk for that patch :)18:56
*** chhavi has quit IRC18:59
*** alex_xu has quit IRC19:04
*** gonzalo2kx has joined #openstack-nova19:04
*** salv-orlando has joined #openstack-nova19:04
*** jichen has joined #openstack-nova19:04
*** lpetrut has joined #openstack-nova19:06
*** alex_xu has joined #openstack-nova19:07
*** eliqiao has quit IRC19:11
*** eliqiao has joined #openstack-nova19:13
*** rmart04 has joined #openstack-nova19:13
*** vilobhmm11 has joined #openstack-nova19:14
*** bdrich has joined #openstack-nova19:14
*** rmart04_ has joined #openstack-nova19:15
*** tdurakov has quit IRC19:15
*** tdurakov has joined #openstack-nova19:16
mriedemhttps://github.com/openstack/nova/blob/master/nova/objects/migrate_data.py#L165-L16619:17
*** angdraug has joined #openstack-nova19:17
mriedemhow does subtracting a dict from a set work?19:17
mriedemoh derp19:18
rlrossit_silly mriedem19:18
*** rmart04 has quit IRC19:18
*** rmart04_ is now known as rmart0419:18
*** ihrachys has quit IRC19:18
jaypipesvladikr: sorry, meeting hell day... will try to get to that series later tonight (I noted it yesterday but couldn't get to it)19:19
sdaguemriedem: although, that use of inline set is so unused, it's confusing when it shows up19:19
vladikrjaypipes, thanks!19:19
mriedemsdague: +119:19
*** SridharG has quit IRC19:22
*** jaosorior has quit IRC19:25
*** jaosorior has joined #openstack-nova19:25
*** rmart04 has quit IRC19:26
*** diana_clarke has joined #openstack-nova19:27
*** dedery has joined #openstack-nova19:33
mriedemdoffm: https://review.openstack.org/#/c/206735/19:34
mriedemdoffm: nova api showing neutron quotas is a long-standing bug that no one was ever working on (ctrath had a spec up for it)19:34
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [python3] byte/string conversions and enable more tests  https://review.openstack.org/26104519:35
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [python3] Port more code to python3 and enable tests  https://review.openstack.org/26208319:35
mriedemdoffm: https://bugs.launchpad.net/nova/+bug/118635419:35
openstackLaunchpad bug 1186354 in OpenStack Compute (nova) "Limits API doesn't work with Neutron" [Medium,In progress] - Assigned to Cale Rath (ctrath)19:35
*** paul-carlton2 has joined #openstack-nova19:43
*** otter768 has joined #openstack-nova19:45
openstackgerritNicolas Simonds proposed openstack/nova: Compute API: omit disk/container formats when creating images of snapshots  https://review.openstack.org/22606919:47
*** dslev has joined #openstack-nova19:47
sdaguedims: do you have logs that pinpoint the failure due to the dhcpbridge issue?19:47
sdagueso I can convince myself the fork is going to help19:47
dimssdague : i'll dig one up19:47
*** krtaylor has quit IRC19:49
sdaguedims: thanks19:49
*** otter768 has quit IRC19:50
jaypipesvladikr: done.19:50
vladikrjaypipes, thanks19:50
*** eharney has quit IRC19:52
*** salv-orlando has quit IRC19:54
*** zenoway has joined #openstack-nova19:55
*** PsionTheory has joined #openstack-nova19:56
*** rfolco has quit IRC19:56
*** ctrath has quit IRC19:58
*** aysyd has quit IRC19:59
openstackgerritVladik Romanovsky proposed openstack/nova: neutron: pci_request logic considers 'direct-physical' vnic type  https://review.openstack.org/26258420:02
openstackgerritVladik Romanovsky proposed openstack/nova: pci: Add utility method for getting the MAC addr  https://review.openstack.org/26258520:02
openstackgerritVladik Romanovsky proposed openstack/nova: neutron: Update the port with a MAC address for PFs  https://review.openstack.org/26258620:02
openstackgerritVladik Romanovsky proposed openstack/nova: pci: store context when creating pci devices  https://review.openstack.org/26976420:02
openstackgerritVladik Romanovsky proposed openstack/nova: pci: changing the claiming and allocation logic for PF/VF assignment  https://review.openstack.org/24901820:02
openstackgerritVladik Romanovsky proposed openstack/nova: Add a vnic type for PF passthrough and a new libvirt vif driver  https://review.openstack.org/26258320:02
*** tonytan4ever has quit IRC20:02
*** lucasagomes is now known as lucas-pto20:03
*** krtaylor has joined #openstack-nova20:03
*** salv-orlando has joined #openstack-nova20:03
*** mrkz has quit IRC20:04
*** dave-mccowan has joined #openstack-nova20:04
*** e0ne has quit IRC20:04
*** dave-mcc_ has joined #openstack-nova20:06
*** mrkz has joined #openstack-nova20:06
bauzasmriedem: you okay if I forward port your relnote for the OSSA issues ? :D20:06
*** jaosorior has quit IRC20:08
mriedemforward port?20:08
*** salv-orlando has quit IRC20:09
*** dave-mccowan has quit IRC20:09
*** abitha has joined #openstack-nova20:11
*** ociuhandu has quit IRC20:11
bauzasmriedem: https://review.openstack.org/#/c/267178/20:11
bauzascorrect me if I'm wrong, but all of the changes are also in master, right?20:11
mriedembauzas: yeah, those were already announced though20:12
mriedemseparately20:12
bauzasmriedem: but they're missing a relnote hence my question20:13
tonybbauzas: Yeah they're there (on master)20:13
mriedembauzas: oh missing the release note on master, i get it20:13
mriedemyeah go nuts20:13
tonybbauzas: yeah you're right, they'll need somethign like hat review on master20:14
bauzasmriedem: tonyb: yeah not speaking about the patches themselves, just the reno files20:14
bauzasI'm just adding a bunch of reno files now we're about to cutr20:15
bauzascut20:15
bauzashttps://etherpad.openstack.org/p/mitaka-nova-priorities-tracking and https://etherpad.openstack.org/p/nova-milestone-reno20:15
sdagueauggy: I wonder if we can respin this so that we instantiate the test class without inheriting from it - https://review.openstack.org/#/c/263440/720:15
*** tonytan4ever has joined #openstack-nova20:15
sdaguethat should also handle some of the testscenarios oddness20:15
openstackgerritRyan Rossiter proposed openstack/nova: Remove DictCompat from ComputeNode  https://review.openstack.org/26552120:17
*** diana_clarke has quit IRC20:17
*** ctrath has joined #openstack-nova20:18
*** Sukhdev has joined #openstack-nova20:18
*** salv-orlando has joined #openstack-nova20:19
dimssdague : see some of the time drifts between when DHCPACK is in syslog and we have an entry in dhcpbridge - https://www.evernote.com/l/AZymPWaeEkRJmLKtkO9gx1f3-kWa25PPcb0 - picked from a recent run - http://logs.openstack.org/30/263430/5/check/gate-tempest-dsvm-full/d67d96f/logs/20:21
*** e0ne has joined #openstack-nova20:22
*** claudiub has quit IRC20:22
*** e0ne has quit IRC20:22
*** tdurakov has quit IRC20:23
sdagueok, will look at post TC meeting20:23
auggysdague: sure i can try to figure out how to do that20:24
auggyi tried to do it as an instance within the class but couldn't get it working20:24
sdagueauggy: yeh... I'm realizing it might not be super straight forward ... let me figure out if there is a good way to handle the bootstrapping20:25
auggysdague: this was more me just trying to get something working but i figured we'd need to fix that inheritance issue20:25
sdagueauggy: yeh, the reset is good, the inheritance makes me nervous20:26
*** gtt116_ has joined #openstack-nova20:26
sdagues/reset/rest/20:27
auggysdague: also let me know if you notice any gaps in test coverage that could be important20:27
auggyi mainly just went off the logical branches in the method20:27
openstackgerritRyan Rossiter proposed openstack/nova: Use extra_data_func to get fingerprints of objects  https://review.openstack.org/26391520:28
sdagueyeh, I think everything there is good, one of the important things I want to get tested after that is the messages when things fail20:28
sdaguebecause they are often pretty surprising20:28
auggyyes, i found that and put some comments in the tests20:28
auggythere were some things that should have been caught well before it got to where it got in the code20:28
auggyand i think that's contributing to the unhelpful errors20:29
*** gtt116__ has quit IRC20:29
auggysdague: i'll revisit how i tried to this initially without inheritance and let you know the results i had, it's possible it's something really obvious i just missed20:29
sdagueyeh, test class instantiation is actually a bit tricky20:30
*** diana_clarke has joined #openstack-nova20:32
*** salv-orlando has quit IRC20:32
*** READ10 has quit IRC20:34
*** ZZelle_ has joined #openstack-nova20:34
*** Yingxin has quit IRC20:36
*** diogogmt has quit IRC20:36
*** tdurakov has joined #openstack-nova20:36
*** Yingxin has joined #openstack-nova20:36
bauzasmriedem: meh, nevermind my point, I can't forward port https://review.openstack.org/#/c/267178/1/releasenotes/notes/12.0.1-cve-bugs-7b04b2e34a3e9a70.yaml20:37
bauzasmriedem: because of your prelude <320:37
mriedembauzas: well, you could and then just modify it20:38
mriedemor just write a new change with the same change id maybe, but different prelude20:38
mriedemor no prelude20:38
bauzasmriedem: yeah, was thinking about that (forward port and cut the prelude)20:38
bauzasbut it's a bit weirdo IMHO20:38
bauzasjust for some doc strings :)20:38
auggysdague: this is the error i got when i tried instantiating it and calling the _compare_result method on the instance https://gist.github.com/missaugustina/59d378bfe465690eed3920:39
*** xyang1 has joined #openstack-nova20:39
mriedembauzas: or just copy the text and write a new change in master20:39
mriedemdoesn't really matter to me20:39
bauzasyeah, was just for bugging you :p20:39
mriedem<head explodes>20:40
*** diogogmt has joined #openstack-nova20:40
bauzas:)20:41
*** boris-42 has joined #openstack-nova20:43
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: libvirt: Allow multiple volume attachments  https://review.openstack.org/19313520:47
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: comp: Allow volume multiattach  https://review.openstack.org/26758720:47
openstackgerritIldiko Vancsa proposed openstack/nova: WIP: REST API changes to support multi-attach  https://review.openstack.org/15303820:47
openstackgerritIldiko Vancsa proposed openstack/nova: Pass host when call attach to Cinder  https://review.openstack.org/25627320:47
sdagueauggy: ok, I got something super hacky20:48
auggysdague: alright i'm all eyes20:48
sdaguehttp://paste.openstack.org/show/484340/20:48
auggysdague: alright that looks pretty much like what people on stack overflow were telling people to do in a similar situation20:49
auggyi was tempted to give it a try but i didn't want to be laughed at20:49
auggyso i'm glad you put yourself out there :D haha20:49
auggyi'll give it a try and see, i'm pretty sure that's gonna fix it20:49
auggyi understand that the api test class is derived from some python test class that does all this fancy shmancy stuff under the hood so thanks for helping to make some sense out of that :)20:50
ildikovmriedem: got the tests fixed or at least checnged and locally still green :)20:51
ildikovmriedem: thank you for all the reviews and help with this! I really appreciate!20:52
sdagueyou can totally blame me20:52
sdaguewe can also ask lifeless if he knows a better way, he knows test loader internals deeply20:52
lifelesshi, I have a queue right now, but in 20m I should be free20:53
*** dslev has quit IRC20:55
auggylifeless: its cool, we were just blaming you for something20:55
lifelessauggy: its all my fault20:56
*** dave-mcc_ has quit IRC20:57
auggysdague: lifeless: i aspire some day to know the magical test loader incantations as well as you all do20:57
auggyyay that worked20:58
auggysdague: lifeless: i will update my current patch shortly (with comments) so review at your leisure20:58
*** daemontool_ has quit IRC20:59
auggyhttps://review.openstack.org/#/c/263440/20:59
auggy(not updated yet just posted for reference)20:59
*** jdurgin1 has joined #openstack-nova20:59
rlrossit_dims: any chance you could take a look at https://review.openstack.org/#/c/266960/ ? I would like to get this in before FF (and I feel terrible for begging)21:00
*** annegentle has quit IRC21:00
*** cdent has quit IRC21:02
*** jschlueter has quit IRC21:02
*** jschlueter has joined #openstack-nova21:03
dimsrlrossit_ : +1 from me. that's the max karma for me :)21:03
lifelessauggy: sdague: so whats the question you have?21:05
auggylifeless: we're creating test coverage for some of the functional tests in Api Samples Base Class21:06
auggyi'm working on the first stab at this for _compare_results21:06
openstackgerritNicolas Simonds proposed openstack/nova: Fix deleted non-public flavor lookups for non-admin users  https://review.openstack.org/21180921:06
auggyand we want to instantiate the Api Samples Base Class in order to call the _compare_results method on it, but have to navigate around some of the test magic21:06
openstackgerritVladik Romanovsky proposed openstack/nova: virt: introduce libosinfo library to set hardware policy  https://review.openstack.org/14962521:06
openstackgerritVladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring network model  https://review.openstack.org/14962721:06
openstackgerritVladik Romanovsky proposed openstack/nova: libvirt: adding a class to retrieve hardware properties  https://review.openstack.org/14962621:06
auggyso sdague proposed a way to do it but we want to make sure there isn't a better way21:06
openstackgerritVladik Romanovsky proposed openstack/nova: libvirt: use osinfo when configuring the disk bus  https://review.openstack.org/14962821:06
auggylifeless: http://paste.openstack.org/show/484340/21:07
openstackgerritAugustina Ragwitz proposed openstack/nova: Add test coverage to functional api tests _compare_result method  https://review.openstack.org/26344021:07
auggylifeless: ^^21:07
vladikrjaypipes, I've updated the libosinfo patches, took a while to figure out the PropertyMock bit..21:08
openstackgerritChuck Carmack proposed openstack/nova: REST API changes for user settable server description  https://review.openstack.org/25495021:08
tjonesalaski: would you mind taking a look at https://review.openstack.org/#/c/265476/.  I added more complete param verification but got feedback to remove it.  So i would like your take since i added them due to a comment you made on the completeness (or lack of) the coverage.21:09
lifelessauggy: sdague: so, thats a patch - a better way for /what/ - like, what do you have in mind / what bits don't you like ?21:09
alaskitjones: sure21:09
*** dslevin has joined #openstack-nova21:09
auggylifeless: well it just feels kind of hacky, i think we want to make sure that it's a good way to deal with the problem of instantiating a test class and then testing the methods in it21:09
*** diana_clarke has quit IRC21:10
lifelessok, so the goal is: exercise the test helpers that are methods on (a test class)21:10
lifelessyou can instantiate a test class with Class('methodname')21:10
auggyyes, and make sure they have test coverage so we can refactor them21:10
lifelessok, so exercise it without changing the existing code, as a prelude to changing it21:11
sdaguelifeless: yes, exactly that21:11
lifelessmethodname has to exist, but won't be run - tests are instantiated, then run later21:11
lifelessso t = Class('anymethodatall')21:11
lifelessis valud21:11
lifelessbah21:11
lifelessvalid21:11
sdaguelifeless: oh, ok21:11
sdagueI just looked at the unittest code, saw it wanted runTests, made that method21:12
lifelessit won't run any code at that point - __init__ on TestCase is very minimal21:12
auggyi thought i tried that and had the same problem with it not finding the "test" method21:12
auggybut i'll see if that works21:12
lifelessyes, you need to use an existing method, but it can be one of the existing tests21:12
lifelessor a helper method21:12
sdagueok, good to know21:12
lifelessor whatever. You won't be actually running the test, so - shrug.21:12
sdagueright, was just trying to do this in the least weird way.21:12
lifelessif you do want to be able to run it, there's some helpful scaffolding + some juidicious mocking, but IIRC your tests don't actually run it anyhow21:13
sdagueright, this is not going to run the test class, it's just going to run a set of internal methods which are going to later come out of the test hierarchy and into their own class (which is not a test class)21:13
lifelessyeah21:14
lifelessI'd use Class('somemethodname')21:14
sdagueok, cool21:14
lifelesssubclassing it is indeed weird with the intent you have21:14
auggyok i'll give that a try21:14
sdagueso Class('setUp')21:14
lifelesssince you'd also end up running all the base classes tests21:14
lifelessand I don't think you want that21:14
lifelessI added a small review of some of the tests too21:14
sdaguelifeless: ok, you said any method :)21:14
lifelessa patch set back or so21:15
*** tdurakov has quit IRC21:15
lifelesssdague: setUp is fine. Just be sure to have a comment :)21:15
lifelessI'd put it in a helpper21:15
sdagueyeh, agreed21:15
lifelessdef get_SampleTestInstance(self):21:15
lifeless   # ....21:15
lifeless  return api...('setUp')21:15
lifelessbbiab21:16
sdagueis runTest part of python testing interface, all test runners will have that? the current pass method there might be as clean as anything honestly.21:16
lifelessno, its a TemplateMethod pattern21:16
lifelessbasically there to confuse the hell out of everyone21:17
*** tdurakov has joined #openstack-nova21:17
auggylifeless: yeah i saw your comment, thanks for that suggestion21:17
openstackgerritMatt Riedemann proposed openstack/nova: vmware: cleanup unused volume connector 'instance' key  https://review.openstack.org/26986321:17
*** gyee has joined #openstack-nova21:17
*** tdurakov has quit IRC21:17
*** eharney has joined #openstack-nova21:18
*** paul-carlton2 has quit IRC21:18
auggysdague: so i went ahead and did what lifeless suggested and i'm getting the error about not having the api version set21:20
auggyhow do i set attributes on an object when calling it that way?21:20
auggyAttributeError: 'ApiSampleTestBase' object has no attribute 'api_major_version'21:21
*** vladikr has quit IRC21:21
*** ijw_ has joined #openstack-nova21:21
auggyi'll update my gist21:21
*** ijw has quit IRC21:21
sdagueauggy: we still need that21:22
auggyi'm guessing it's something in setUp, so i'll poke around21:22
sdagueapi_major_version is used inside the comparitors21:23
*** tdurakov has joined #openstack-nova21:23
*** ijw has joined #openstack-nova21:23
auggysdague: https://gist.github.com/missaugustina/59d378bfe465690eed3921:23
sdaguehttps://github.com/openstack/nova/blob/770666f8766074fc7df99c87a0ed9fc86e249b9f/nova/tests/functional/api_samples_test_base.py#L35521:23
sdaguewhich is also why I had to mock the self.compute out21:23
*** e0ne has joined #openstack-nova21:23
auggyok so i'll see if i can do self.ast.api_major_version = 221:24
sdagueI think we probably actually should stick with the wrapper, because that's only defined in the subclass with testscenarios iirc21:24
*** jichen has quit IRC21:25
auggyok, i was able to set the attribute and the tests passed21:25
sdagueok, then that's fine21:25
*** Sukhdev has quit IRC21:25
*** smatzek has quit IRC21:26
auggyi am just a little confused by that syntax, so i want to try to put some comments in to clarify for future people encountering it21:26
*** ijw_ has quit IRC21:26
sdagueauggy: ++21:27
sdagueI also think it needs to be "v2"21:27
auggyyeah that was me shorthanding21:27
auggyi'll update my gist with the line i added21:28
auggyalright updated - https://gist.github.com/missaugustina/59d378bfe465690eed3921:28
auggyi'll update the rest of the code and also see about using the ExpectedException thing lifeless recommended21:29
*** mriedem has quit IRC21:29
auggyand add comments around what we're doing so it's clear to others21:29
sdagueso I think I'd add all the modification to self.ast to the get_.... method21:29
sdagueso you have the class fully formed from it21:29
*** mdnadeem has joined #openstack-nova21:29
auggyok that makes sense21:30
sdaguebut, then I think we're good to go21:31
auggyyeah i think i just wasn't sure what all was going on there but then realized oh it's just being assigned to a varaible21:31
sdagueyep21:31
auggyanyways yeah don't mind me, the magic was blinding me21:32
sdagueheh, no worries21:32
sdagueok, time to call it a day, I'll look in the morning21:33
*** e0ne has quit IRC21:33
auggysdague: no problem21:34
auggysdague: i'll be on a plane in the afternoon hopefully with wifi en route to amsterdam21:34
sdagueok, cool21:35
auggyi'm going to spend a couple days in amsterdam visiting friends and getting time zone adjusted pre-midcycle21:35
auggyi was so jet lagged during the summit, i was like no, not making that mistake again21:35
*** ihrachys has joined #openstack-nova21:35
sdaguelike I said, I think this is close, so I think after doing this, it's probably ready to go21:35
auggyok cool, let me know if there are other methods that need coverage and i can start working on those too21:35
sdaguewill do, I can leave those as comments for follow on patch21:36
auggyalright have a good evening and thanks for the help!21:36
*** dslevin has quit IRC21:38
*** baoli has joined #openstack-nova21:38
*** baoli_ has joined #openstack-nova21:39
*** thangp has left #openstack-nova21:40
*** mdnadeem has quit IRC21:41
*** penick has joined #openstack-nova21:42
*** tdurakov has quit IRC21:42
*** baoli has quit IRC21:43
*** smatzek has joined #openstack-nova21:44
*** dslev has joined #openstack-nova21:45
*** otter768 has joined #openstack-nova21:46
*** ihrachys_ has joined #openstack-nova21:46
*** ihrachys has quit IRC21:46
*** baoli_ has quit IRC21:49
*** baoli has joined #openstack-nova21:49
*** baoli has quit IRC21:50
*** otter768 has quit IRC21:50
*** baoli has joined #openstack-nova21:50
*** diana_clarke has joined #openstack-nova21:52
*** vladik-train has joined #openstack-nova21:53
*** angdraug has quit IRC21:55
*** cburgess has quit IRC21:56
lifelessauggy: just assign to it21:57
lifelesssdague: testscenarios doesn't subclass itself21:57
*** cburgess has joined #openstack-nova21:57
lifelesssdague: it does a copy + attribute assignment21:57
auggylifeless: yeah thanks, i figured it out :) i was just overthinking cuz i do that21:57
auggyi was thinking that code was returning a ref to a function21:57
*** eil397 has quit IRC21:57
auggynot an instance21:57
lifelessif only it was haskell, you'd be right :)21:58
auggysee you UNDERSTAND MEEE21:58
*** salv-orlando has joined #openstack-nova21:58
*** rk4n has joined #openstack-nova21:59
*** Guest51217 is now known as med_22:00
*** med_ has quit IRC22:00
*** med_ has joined #openstack-nova22:00
*** pratikmallya has quit IRC22:01
*** baoli has quit IRC22:01
*** baoli has joined #openstack-nova22:01
*** lpetrut has quit IRC22:01
*** vladik-train_ has joined #openstack-nova22:04
*** vladik-train has quit IRC22:04
*** alejandrito has joined #openstack-nova22:05
*** vladik-train_ has joined #openstack-nova22:06
*** vladik-train_ has left #openstack-nova22:06
*** vladik-train_ has joined #openstack-nova22:06
rlrossit_dims: thanks for the review :). I just need your approval as leverage to hit mriedem harder :)22:09
*** su_zhang has joined #openstack-nova22:11
*** Sukhdev has joined #openstack-nova22:11
*** smatzek has quit IRC22:12
*** eliqiao has quit IRC22:12
*** ivase has quit IRC22:13
*** baoli has quit IRC22:13
*** ivase has joined #openstack-nova22:13
*** eliqiao has joined #openstack-nova22:13
*** baoli has joined #openstack-nova22:14
*** nic has left #openstack-nova22:18
*** ivase has quit IRC22:18
*** nic has joined #openstack-nova22:18
*** thorst has quit IRC22:19
*** pratikmallya has joined #openstack-nova22:19
*** smatzek has joined #openstack-nova22:20
*** rcernin has quit IRC22:21
*** thorst has joined #openstack-nova22:22
*** Sukhdev has quit IRC22:22
*** rlrossit_ has quit IRC22:23
*** Sukhdev has joined #openstack-nova22:23
*** akshai has quit IRC22:24
*** baoli has quit IRC22:25
*** baoli has joined #openstack-nova22:26
*** thorst has quit IRC22:26
*** ctrath has quit IRC22:28
*** irenab has joined #openstack-nova22:28
*** baoli has quit IRC22:31
*** vladik-train_ has quit IRC22:33
*** ctrath has joined #openstack-nova22:34
*** zenoway has quit IRC22:34
*** jamielennox|away is now known as jamielennox22:35
*** akshai has joined #openstack-nova22:35
*** baoli has joined #openstack-nova22:36
*** davidlenwell has quit IRC22:37
openstackgerritRodolfo Alonso Hernandez proposed openstack/nova: Added new scheduler filter: AggregateTypeExtraSpecsAffinityFilter  https://review.openstack.org/18927922:38
*** tonytan4ever has quit IRC22:38
*** smatzek has quit IRC22:38
*** davidlenwell has joined #openstack-nova22:38
*** pratikmallya has quit IRC22:40
*** thorst has joined #openstack-nova22:40
openstackgerritChuck Carmack proposed openstack/nova: config options: centralize 'spice' options  https://review.openstack.org/26427122:41
*** eil397 has joined #openstack-nova22:42
*** bdrich has quit IRC22:43
andrewbogottCan I get a +2 for https://review.openstack.org/#/c/261108/ ?  Or is there some other process I should follow other than just periodically asking here?22:45
*** thorst has quit IRC22:45
*** zenoway has joined #openstack-nova22:46
*** baoli_ has joined #openstack-nova22:46
edleafeandrewbogott: send cash usually helps :)22:46
*** baoli_ has quit IRC22:48
*** baoli_ has joined #openstack-nova22:49
*** baoli has quit IRC22:49
openstackgerritDavid Edery proposed openstack/nova: Fix create snapshot failure on VMs with SRIOV  https://review.openstack.org/26234122:50
*** zenoway has quit IRC22:50
*** alejandrito has quit IRC22:51
*** rk4n has quit IRC22:55
*** dedery has quit IRC22:58
*** garyk has quit IRC22:58
*** garyk has joined #openstack-nova23:00
*** baoli_ has quit IRC23:00
*** baoli has joined #openstack-nova23:01
*** rk4n has joined #openstack-nova23:04
andrewbogottedleafe: so just keep nagging, huh?23:04
*** diana_clarke has quit IRC23:04
edleafeandrewbogott: works as well as anything else! :)23:07
*** dslev has quit IRC23:07
*** edtubill has quit IRC23:08
*** edtubill has joined #openstack-nova23:08
*** edtubill has quit IRC23:08
*** mragupat has quit IRC23:08
*** mdrabe has quit IRC23:09
*** annegentle has joined #openstack-nova23:10
openstackgerritChuck Carmack proposed openstack/nova: config options: centralize 'spice' options  https://review.openstack.org/26991023:12
*** mrkz has quit IRC23:12
*** baoli has quit IRC23:12
*** dimtruck is now known as zz_dimtruck23:12
*** ihrachys has joined #openstack-nova23:14
*** burt has quit IRC23:15
*** rk4n has quit IRC23:15
*** baoli has joined #openstack-nova23:15
*** ajayaa has quit IRC23:15
openstackgerritClint 'SpamapS' Byrum proposed openstack/nova: Log retries at INFO level per guidelines  https://review.openstack.org/26991523:16
*** ihrachys_ has quit IRC23:16
*** ihrachys has quit IRC23:20
*** baoli has quit IRC23:25
*** thorst has joined #openstack-nova23:28
*** rk4n has joined #openstack-nova23:28
*** alex_xu has quit IRC23:30
*** alex_xu has joined #openstack-nova23:31
*** rk4n has quit IRC23:32
*** rk4n has joined #openstack-nova23:33
*** xyang1 has quit IRC23:37
*** doug-fish has quit IRC23:39
openstackgerritVilobh Meshram proposed openstack/nova: ServiceGroup Refactoring: Add is_up to Service Obj  https://review.openstack.org/26992923:39
*** thorst has quit IRC23:41
*** thorst has joined #openstack-nova23:42
*** thorst has quit IRC23:46
*** diana_clarke has joined #openstack-nova23:46
*** otter768 has joined #openstack-nova23:47
*** pratikmallya has joined #openstack-nova23:48
*** doug-fish has joined #openstack-nova23:49
*** ducttape_ has quit IRC23:50
*** irenab has quit IRC23:51
*** pumaranikar has quit IRC23:51
*** otter768 has quit IRC23:51
*** pumaranikar has joined #openstack-nova23:52
*** doug-fish has quit IRC23:54
*** pumaranikar has quit IRC23:58

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