Thursday, 2015-07-02

*** tsekiyam_ has joined #openstack-nova00:02
*** hemna is now known as hemnafk00:04
*** tjones1 has quit IRC00:04
*** tsekiyama has quit IRC00:06
*** mtanino has quit IRC00:06
*** tsekiyam_ has quit IRC00:07
*** bjornar has quit IRC00:10
*** bjornar has joined #openstack-nova00:11
*** karimb has quit IRC00:19
*** yingjun has joined #openstack-nova00:29
*** kfox1111 is now known as kfox1111_away00:32
openstackgerritKevin Fox proposed openstack/nova-specs: Instance Users for Cloud Interaction  https://review.openstack.org/18661700:32
*** otter768 has quit IRC00:38
*** otter768 has joined #openstack-nova00:39
*** zzzeek has quit IRC00:40
*** Marga_ has joined #openstack-nova00:42
*** ijw_ has quit IRC00:44
*** zzzeek has joined #openstack-nova00:45
*** Marga_ has quit IRC00:45
*** Sukhdev has quit IRC00:45
*** Marga_ has joined #openstack-nova00:45
*** zzzeek has quit IRC00:47
*** otter768 has quit IRC00:49
*** annashen has joined #openstack-nova00:53
*** tpeoples has quit IRC00:55
*** tdp has joined #openstack-nova00:55
*** annashen has quit IRC00:58
*** jasondotstar has joined #openstack-nova01:02
*** tdp is now known as tpeoples01:04
openstackgerritZhenyu Zheng proposed openstack/nova: glance:check the num_retries option  https://review.openstack.org/12331801:04
openstackgerritZhenyu Zheng proposed openstack/nova: glance:check the num_retries option  https://review.openstack.org/12331801:05
*** Jeffrey4l has quit IRC01:09
*** patrickeast has quit IRC01:09
*** Jeffrey4l has joined #openstack-nova01:09
*** ferest has quit IRC01:14
tonybmikal: can you +2 this thing? https://review.openstack.org/#/c/194325/01:19
*** dims_ has quit IRC01:19
mikaltonyb: maybe, is it crazy pants?01:20
tonybmikal: I think it's okay.  I was scared yesterday but dansmith fixed the thing that was scaring me.01:20
mikaltonyb: ok01:20
tonybmikal: clearly you need to use you best judgement but having nova on the current oslo.concurrency would be nice01:21
*** dave-mccowan has quit IRC01:21
*** jwcroppe_ has quit IRC01:23
*** mnestratov|2 has joined #openstack-nova01:25
*** armax has quit IRC01:26
*** deepthi has joined #openstack-nova01:26
*** Piet has quit IRC01:26
*** jasondotstar has quit IRC01:27
*** annegentle has quit IRC01:28
*** mnestratov has quit IRC01:28
*** Piet has joined #openstack-nova01:30
*** otter768 has joined #openstack-nova01:36
*** dims has joined #openstack-nova01:39
*** Marga_ has quit IRC01:40
*** Marga_ has joined #openstack-nova01:42
*** jogo has quit IRC01:44
*** jogo has joined #openstack-nova01:45
*** ociuhandu has quit IRC01:46
*** mutoulbj has joined #openstack-nova01:49
*** mutoulbj has quit IRC01:49
*** huazhihao has joined #openstack-nova01:49
openstackgerritGhanshyam Mann proposed openstack/nova: Merge virtual interface tests between v2 and v2.1  https://review.openstack.org/19781101:51
*** wuhg has joined #openstack-nova01:52
*** annashen has joined #openstack-nova01:54
*** baoli has joined #openstack-nova01:56
*** claudiub has quit IRC01:57
*** annashen has quit IRC01:58
*** jwcroppe has joined #openstack-nova02:01
*** huazhihao has quit IRC02:04
*** jecarey has joined #openstack-nova02:04
*** huazhihao has joined #openstack-nova02:07
*** nelsnelson has joined #openstack-nova02:08
*** otter768 has quit IRC02:08
*** Marga_ has quit IRC02:10
*** piyanai has joined #openstack-nova02:12
*** mnestratov|2 has quit IRC02:13
*** baoli has quit IRC02:13
*** mnestratov|2 has joined #openstack-nova02:13
*** baoli has joined #openstack-nova02:13
*** mutoulbj has joined #openstack-nova02:14
*** Alexandra_ has joined #openstack-nova02:15
*** Jeffrey4l has quit IRC02:17
*** Jeffrey4l has joined #openstack-nova02:18
*** jwang_ has joined #openstack-nova02:18
*** achanda has joined #openstack-nova02:20
*** yamahata has quit IRC02:21
*** jwang__ has quit IRC02:21
*** xyang1 has quit IRC02:26
*** armax has joined #openstack-nova02:29
*** yingjun has quit IRC02:29
*** achanda has quit IRC02:30
*** jaypipes has quit IRC02:30
*** yingjun has joined #openstack-nova02:32
*** nelsnelson has quit IRC02:39
*** nelsnelson has joined #openstack-nova02:39
openstackgerritRong Han proposed openstack/nova: Catch NovaException from post_live_migration_at_destination, set instance vm_state error.  https://review.openstack.org/18663402:42
*** dave-mccowan has joined #openstack-nova02:51
*** unicell1 has quit IRC02:51
openstackgerritJim Rollenhagen proposed openstack/nova-specs: Ironic: Multiple compute host support  https://review.openstack.org/19445302:54
*** annashen has joined #openstack-nova02:55
*** annashen has quit IRC02:59
*** achanda has joined #openstack-nova03:00
openstackgerritGhanshyam Mann proposed openstack/nova: Fix 'OS-EXT-VIF-NET' extension missing in v2.1  https://review.openstack.org/19782203:00
*** pixelbeat has joined #openstack-nova03:06
*** pixelb has quit IRC03:07
openstackgerritjaveme proposed openstack/nova: VMware: Prefer ShutdownGuest to PowerOffVM_Task  https://review.openstack.org/18163403:12
*** achanda has quit IRC03:13
gmannalex_xu: johnthetubaguy : Please have a look - https://review.openstack.org/#/c/197822/03:16
gmannalex_xu: johnthetubaguy : there is difference between v2 and v2.1 for virtual interface API03:17
alex_xugmann: good catch!03:17
gmannalex_xu: johnthetubaguy : I will push fix for Kilo also03:17
gmannalex_xu: yea, i do not know how we missed this. anyways merging functional tests catching these kind of miss03:17
*** piyanai has quit IRC03:17
alex_xugmann: so we missing test in tempest also?03:19
alex_xugmann: good to see merging effort can find out bug03:19
gmannalex_xu: we have in Tempest, but there we tests for All extension enable03:20
gmannalex_xu: and as this is extension we could not put that as required param in schema03:20
gmannalex_xu: so there its hard to find such differences03:20
*** oomichi has joined #openstack-nova03:20
alex_xugmann: ok, I got the reason03:21
alex_xugmann: it will be fine after we get rid of extension03:21
gmannalex_xu: yea03:21
*** dims has quit IRC03:22
gmannalex_xu: catch you later, need to hurry up for lunch03:22
alex_xugmann: ok, will try to review asap03:23
*** tjones1 has joined #openstack-nova03:24
*** jecarey has quit IRC03:27
*** tjones1 has left #openstack-nova03:28
*** Jeffrey4l has quit IRC03:29
*** garyk has quit IRC03:33
*** Jeffrey4l has joined #openstack-nova03:42
*** coolsvap|away is now known as coolsvap03:45
*** unicell has joined #openstack-nova03:45
openstackgerritChangBo Guo(gcb) proposed openstack/nova: libvirt: add get_cpu_frequency method  https://review.openstack.org/19782903:45
*** Marga_ has joined #openstack-nova03:46
*** yingjun has quit IRC03:52
openstackgerritChangBo Guo(gcb) proposed openstack/nova: libvirt: simplify logic of checking PCI device's existence  https://review.openstack.org/14540003:54
*** annashen has joined #openstack-nova03:56
*** baoli has quit IRC03:57
*** annashen has quit IRC04:00
lifelesshi04:06
lifelesshttps://review.openstack.org/#/c/196934/ is likely to conflict with stuff, so I'd love a +2+A04:06
lifelessmikal: ^04:06
*** otter768 has joined #openstack-nova04:09
*** kiran-r has joined #openstack-nova04:09
*** ianbrown has quit IRC04:11
*** rajesht has joined #openstack-nova04:12
*** ildikov has quit IRC04:13
*** otter768 has quit IRC04:13
openstackgerritGhanshyam Mann proposed openstack/nova: Fix 'OS-EXT-VIF-NET' extension missing in v2.1  https://review.openstack.org/19782204:17
*** heyongli has quit IRC04:18
tonyblifeless: I was just doing that here ;P04:20
tonybI looked and didn't see an open review from you so ....04:20
tonyblifeless: glad you on it.04:20
lifelesstonyb: How did you look ?04:20
*** heyongli has joined #openstack-nova04:21
tonyblifeless: with a prestaged dashboard that is clearkly wrong.04:21
lifelessheh04:22
*** lucas__ has joined #openstack-nova04:23
tonyblifeless: the Xen CI failure is where I got stuck04:24
lifelesstonyb: looks clearly unrelated: 21:23:01     Details: {u'message': u"internal error: libxenlight failed to create new domain 'instance-0000003f'", u'created': u'2015-07-01T20:56:22Z', u'code': 500}04:25
lifeless21:23:01 tempest.api.compute.servers.test_server_actions.ServerActionsTestJSON.test_resize_server_revert[id-c03aab19-adb1-44f5-917d-c419577e9e68]04:25
*** Jeffrey4l has quit IRC04:27
*** Jeffrey4l has joined #openstack-nova04:27
*** lucas__ has quit IRC04:28
openstackgerritMichael Still proposed openstack/nova: Cleanup log lines in nova.image.glance  https://review.openstack.org/19691404:28
openstackgerritMichael Still proposed openstack/nova: log meaningful error message on download exception  https://review.openstack.org/16403904:28
tonyblifeless: Sure.  I guess I'm nervy whn we change the requitrements and then unrelated things break04:29
*** mnestratov has joined #openstack-nova04:29
lifelesstonyb: fair enough04:29
*** vilobhmm has joined #openstack-nova04:29
lifelesstonyb: my considered opinion here is that its flakiness not requirements for that case :)04:30
*** sdake has joined #openstack-nova04:30
tonyblifeless: good to know ;P04:30
*** garyk has joined #openstack-nova04:31
*** ianbrown has joined #openstack-nova04:31
*** heyongli has quit IRC04:31
*** mnestratov|2 has quit IRC04:32
*** heyongli has joined #openstack-nova04:34
tonyblifeless: Thanks for you input on the bash-completions thing.  I'm not sure I have the stomache for follow through now04:35
lifelessyw04:35
tonyblifeless: it dosn't seem like a big deael to me but ....04:36
StevenKHaha04:36
*** heyongli has quit IRC04:38
*** arnaud____ has joined #openstack-nova04:43
*** heyongli has joined #openstack-nova04:43
*** jwcroppe_ has joined #openstack-nova04:43
*** dave-mccowan has quit IRC04:44
*** jwcroppe has quit IRC04:46
*** heyongli has quit IRC04:48
*** shihanzhang has quit IRC04:48
*** shihanzhang has joined #openstack-nova04:49
*** heyongli has joined #openstack-nova04:49
*** heyongli has quit IRC04:54
*** shihanzhang has quit IRC04:54
*** nkrinner has joined #openstack-nova04:54
*** Kevin_Zheng has quit IRC04:54
*** liusheng has quit IRC04:54
*** inara has quit IRC04:55
*** yingjun has joined #openstack-nova04:56
*** annashen has joined #openstack-nova04:56
*** ajayaa has joined #openstack-nova05:00
*** annashen has quit IRC05:01
*** inara has joined #openstack-nova05:01
*** liusheng has joined #openstack-nova05:03
*** hanzhang has joined #openstack-nova05:03
*** Kevin_Zheng has joined #openstack-nova05:03
*** Dev_Jin has joined #openstack-nova05:05
*** mmedvede has quit IRC05:10
*** prashant_ has joined #openstack-nova05:11
*** vilobhmm has quit IRC05:13
*** vilobhmm has joined #openstack-nova05:13
*** prashant__ has joined #openstack-nova05:13
*** arnaud____ has quit IRC05:15
*** _gryf|afk is now known as _gryf05:16
*** prashup has joined #openstack-nova05:16
*** prashant_ has quit IRC05:17
*** prashant__ has quit IRC05:20
*** prashant_ has joined #openstack-nova05:21
*** prashup has quit IRC05:25
*** whenry_ has quit IRC05:26
*** prashant__ has joined #openstack-nova05:27
*** nelsnelson has quit IRC05:31
*** prashant_ has quit IRC05:31
*** sudipto has joined #openstack-nova05:31
*** prashant_ has joined #openstack-nova05:35
*** coolsvap is now known as coolsvap|away05:37
*** coolsvap|away is now known as coolsvap05:38
*** prashant__ has quit IRC05:39
*** nihilifer has joined #openstack-nova05:40
*** shalq has joined #openstack-nova05:43
*** prashant__ has joined #openstack-nova05:43
*** hanzhang has quit IRC05:44
*** hanzhang has joined #openstack-nova05:45
*** prashant_ has quit IRC05:47
*** shalq has quit IRC05:48
*** Longgeek has joined #openstack-nova05:52
*** irenab has quit IRC05:52
*** coolsvap is now known as coolsvap|away05:54
oomichialex_xu: hello05:55
*** prashant_ has joined #openstack-nova05:57
*** annashen has joined #openstack-nova05:57
*** Murali has joined #openstack-nova05:59
*** prashant__ has quit IRC06:00
alex_xuoomichi: hi06:02
*** annashen has quit IRC06:02
*** coolsvap|away is now known as coolsvap06:02
*** prashant__ has joined #openstack-nova06:03
*** armax has quit IRC06:04
*** wanghua has joined #openstack-nova06:04
*** prashant_ has quit IRC06:06
*** pbandzi has joined #openstack-nova06:09
*** otter768 has joined #openstack-nova06:10
*** prashant__ has quit IRC06:11
*** otter768 has quit IRC06:14
*** coolsvap is now known as coolsvap|away06:16
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex  https://review.openstack.org/19721306:18
*** humble_ has joined #openstack-nova06:23
*** Dev_Jin has quit IRC06:24
*** Dev_Jin has joined #openstack-nova06:25
*** irenab has joined #openstack-nova06:26
*** scheuran has joined #openstack-nova06:27
*** wanghua has quit IRC06:27
*** ianbrown has quit IRC06:29
*** unicell1 has joined #openstack-nova06:30
*** unicell has quit IRC06:30
openstackgerritTony Breeds proposed openstack/nova: Add support for one time tokens in consoleauth  https://review.openstack.org/18212906:33
*** mnestratov|2 has joined #openstack-nova06:35
*** lucas__ has joined #openstack-nova06:35
*** lyanchih_ has joined #openstack-nova06:36
openstackgerritTony Breeds proposed openstack/nova: Add support for one time tokens in consoleauth  https://review.openstack.org/18212906:36
*** browne has quit IRC06:36
*** mnestratov has quit IRC06:38
*** lyanchih_ has quit IRC06:38
*** lyanchih_ has joined #openstack-nova06:38
*** lucas__ has quit IRC06:39
*** belmoreira has joined #openstack-nova06:44
*** radek_ has joined #openstack-nova06:45
*** vilobhmm has quit IRC06:45
openstackgerritPaul Murray proposed openstack/nova: Convert RT compute_node to be a ComputeNode object  https://review.openstack.org/16094206:46
*** belmoreira has quit IRC06:47
*** abhishekk has joined #openstack-nova06:53
*** yingjun has quit IRC06:55
*** gszasz has joined #openstack-nova06:56
*** annashen has joined #openstack-nova06:58
*** belmoreira has joined #openstack-nova06:58
*** belmoreira has quit IRC06:58
rgerganovoomichi: hi Ken06:59
*** belmoreira has joined #openstack-nova06:59
*** ildikov has joined #openstack-nova07:00
rgerganovoomichi: about your comment on https://review.openstack.org/#/c/148509/07:00
rgerganovoomichi: Jay Pipes commented on the original spec to make the response that way it is: https://review.openstack.org/#/c/141065/2/specs/kilo/approved/consolidate-console-api.rst,cm07:00
*** otter768 has joined #openstack-nova07:00
*** annashen has quit IRC07:03
*** _kiran_ has joined #openstack-nova07:03
*** Alexandra_ has quit IRC07:04
*** kiranr has joined #openstack-nova07:04
*** otter768 has quit IRC07:05
*** _kiran_ has quit IRC07:05
*** mpaolino has joined #openstack-nova07:06
*** kiran-r has quit IRC07:07
*** prashant__ has joined #openstack-nova07:08
*** kiran-r has joined #openstack-nova07:08
*** kiranr has quit IRC07:08
*** fifieldt has quit IRC07:09
*** kiran-r has quit IRC07:13
*** jlanoux has joined #openstack-nova07:13
*** humble__ has joined #openstack-nova07:13
*** matrohon has joined #openstack-nova07:13
*** pask81 has joined #openstack-nova07:14
*** humble_ has quit IRC07:16
oomichirgerganov: hi, thanks for explanation07:16
openstackgerritGhanshyam Mann proposed openstack/nova: Fix 'OS-EXT-VIF-NET' extension missing in v2.1  https://review.openstack.org/19782207:17
oomichirgerganov: we have api-wg guideline for consistent api design.07:17
oomichirgerganov: but this case is not yet covered on that.07:17
oomichirgerganov: that is TODO now like https://github.com/openstack/api-wg/blob/master/guidelines/representation_structure.rst#singular-resources07:17
rgerganovoomichi: I see, thanks for the link07:18
oomichirgerganov: so I will ask jaypipes on the patch review07:18
rgerganovoomichi: I will really appreciate if we can move this forward07:18
*** gszasz has quit IRC07:19
oomichirgerganov: yeah, sorry about waiting for this.07:19
*** coolsvap|away has quit IRC07:20
rgerganovoomichi: I will try to catch Jay on IRC later today07:21
*** coolsvap|away has joined #openstack-nova07:23
*** aix has joined #openstack-nova07:24
oomichirgerganov: that will be helpful, thanks in advance :-)07:24
rgerganovoomichi: np :)07:25
*** doude has joined #openstack-nova07:25
oomichialex_xu: sorry, I went out to buy doughnuts.07:26
oomichialex_xu: I have a question about https://review.openstack.org/#/c/150710/13/etc/nova/policy.json07:26
oomichialex_xu: can you check it later?07:26
alex_xuoomichi: ok, let me check07:27
oomichialex_xu: thanks :)07:27
*** jcoufal has joined #openstack-nova07:30
*** huazhihao has quit IRC07:31
alex_xuoomichi: good point for rule:admin_api07:31
alex_xuoomichi: I think rule:admin_api is better07:31
*** unicell has joined #openstack-nova07:31
oomichialex_xu: thanks, and the usages are inconsistent now.07:31
alex_xuoomichi: another question I replied, it because there is another controller share same policy rule namespace07:31
alex_xuoomichi: yea, thanks for point that out07:32
*** mutoulbj_ has joined #openstack-nova07:32
*** unicell1 has quit IRC07:33
*** lyarwood_ has quit IRC07:34
*** mutoulbj has quit IRC07:34
*** lyarwood has joined #openstack-nova07:34
*** mpavone has joined #openstack-nova07:36
*** yingjun has joined #openstack-nova07:38
*** humble_ has joined #openstack-nova07:42
openstackgerritAlex Xu proposed openstack/nova: Move existing V2 to legacy-v2  https://review.openstack.org/19372507:42
*** humble__ has quit IRC07:42
*** oomichi has quit IRC07:43
*** humble__ has joined #openstack-nova07:44
*** humble_ has quit IRC07:47
*** wanghua has joined #openstack-nova07:48
*** jistr has joined #openstack-nova07:48
*** Namor_Fan has joined #openstack-nova07:49
*** kiran-r has joined #openstack-nova07:49
*** Namor_Fan has left #openstack-nova07:49
*** kiran-r has joined #openstack-nova07:49
*** humble__ has quit IRC07:51
*** xgsa2 has quit IRC07:53
*** _kiran_ has joined #openstack-nova07:53
*** kiran-r has quit IRC07:57
openstackgerritMatthew Gilliard proposed openstack/nova: Libvirt: Define system_family for libvirt guests  https://review.openstack.org/19713207:58
*** annashen has joined #openstack-nova07:59
*** afazekas has joined #openstack-nova08:01
*** _kiran_ is now known as kiran-r08:03
*** annashen has quit IRC08:04
*** prashant__ has quit IRC08:05
*** xgsa has joined #openstack-nova08:07
*** danpb has joined #openstack-nova08:07
*** xgsa has quit IRC08:09
*** jcoufal_ has joined #openstack-nova08:09
*** dosaboy has quit IRC08:10
*** dosaboy has joined #openstack-nova08:11
*** dboik has joined #openstack-nova08:11
*** garyk has quit IRC08:11
*** garyk has joined #openstack-nova08:11
*** dosaboy has quit IRC08:12
*** dboik_ has quit IRC08:12
*** lucasagomes has joined #openstack-nova08:13
*** rwsu has quit IRC08:13
*** yingjun has quit IRC08:15
*** rwsu has joined #openstack-nova08:15
*** xgsa has joined #openstack-nova08:16
*** arnaud____ has joined #openstack-nova08:16
*** matrohon has quit IRC08:18
openstackgerritFeodor Tersin proposed openstack/nova: Ignore image's min_disk and size for volume-backed instances  https://review.openstack.org/19656908:18
*** matrohon has joined #openstack-nova08:18
*** yingjun has joined #openstack-nova08:19
*** jamielennox is now known as jamielennox|away08:20
*** arnaud____ has quit IRC08:20
*** hanzhang has quit IRC08:21
*** dosaboy has joined #openstack-nova08:21
*** hanzhang has joined #openstack-nova08:21
*** pask81 has quit IRC08:22
*** eglynn has quit IRC08:22
*** eglynn has joined #openstack-nova08:23
*** derekh has joined #openstack-nova08:26
*** karimb has joined #openstack-nova08:30
*** xgsa has quit IRC08:32
*** yassine__ has joined #openstack-nova08:37
*** yassine__ has quit IRC08:37
*** andreykurilin__ has joined #openstack-nova08:37
*** yassine__ has joined #openstack-nova08:37
openstackgerritAlex Xu proposed openstack/nova: Move existing V2 to legacy-v2  https://review.openstack.org/19372508:41
*** ptm_away is now known as PaulMurray08:41
*** jamielennox|away is now known as jamielennox08:41
*** xgsa has joined #openstack-nova08:42
*** ankit_ag has joined #openstack-nova08:43
*** humble_ has joined #openstack-nova08:45
*** wanghua has quit IRC08:48
* johnthetubaguy lol, just reading independence day stuff in scroll back08:48
johnthetubaguyalex_xu: have we written an updated doc on what we are doing for policy yet?08:49
johnthetubaguyalex_xu: loosing track of how far we have got there08:49
*** aix has quit IRC08:50
*** foexle has joined #openstack-nova08:50
*** Longgeek has quit IRC08:52
*** Longgeek has joined #openstack-nova08:53
gmannalex_xu: oomichi: johnthetubaguy  - Kilo- https://review.openstack.org/#/c/197850/  Master - https://review.openstack.org/#/c/197822/08:53
*** ajayaa has quit IRC08:53
gmann^^ for v2.1 fix08:53
*** afazekas has quit IRC08:54
johnthetubaguyalex_xu: gmann: I wonder if we should bump the microversion for this change?08:54
gmannjohnthetubaguy: actually that is just difference between v2 and v2.108:55
gmannjohnthetubaguy: if we do that with microversion then v2.1 will differ for virtual interface API08:55
gmannjohnthetubaguy: somehow we missed to fixed that during v2.108:55
johnthetubaguygmann: agreed, but I still think we should bump the version for it08:56
johnthetubaguythe API is released now, so we need to follow the rules08:56
gmannjohnthetubaguy: hummm, yea I was also thinking about that as v2.1 is officially released08:57
BobBalltonyb / lifeless : That's a known issue in the libvirt+xen CI which we're trying to fix.08:57
alex_xujohnthetubaguy: I think it a bug need bump version, we promise 2.1 equal to v2. If we bump microversion, it means v2.1 isn't equal to v2 anymore08:58
gmannjohnthetubaguy: but then I thought it might break some user if they use v2.1 virtual interface API as same as v208:58
alex_xus/need bump/needn't bump/...08:58
johnthetubaguygmann: so I think the good news, is this is only really breaks for folks wanting to use v2.1 to service their v2.0 API, but that doesn't really work properly yes either08:58
johnthetubaguyalex_xu: its already different with validation though08:58
johnthetubaguyalex_xu: my problem is the people already using the API, we would be breaking our versioning rules, which would be really bad08:59
alex_xujohnthetubaguy: the different of validation just for the client using our api with wrong way08:59
*** romainh has joined #openstack-nova08:59
johnthetubaguyalex_xu: agreed, but the details of that are quite subtle, in reality,08:59
johnthetubaguyso here is what I am thinking09:00
alex_xujohnthetubaguy: if we didn't fix in 2.1, it means the user depend on that api, after he switch to v2.1, his code will break.09:00
*** annashen has joined #openstack-nova09:00
johnthetubaguywhen /v2 you need this extension to have your bug fix09:00
johnthetubaguywhen /v2.1 we should really require the new version before you get the fix09:00
johnthetubaguynow thats kinda nuts, but it seems "correct", I think09:00
*** otter768 has joined #openstack-nova09:01
alex_xuone more reason, this isn't a bug existed in v2 or v2.1. It is a bug when we port v2 to v2.109:02
*** ndipanov has joined #openstack-nova09:02
gmannjohnthetubaguy: alex_xu : but still for base v2.1 (which we expected to port same as v2), it is bug09:02
gmannalex_xu: yes, same line09:02
johnthetubaguyso yes, its a bug09:02
johnthetubaguybut we release API v2.1 already09:03
johnthetubaguythat means v2.1 doesn't have that attribute in its base version09:03
alex_xuthis isn't a bug in v2. we use microversion fix the bug which existed in v2.09:03
johnthetubaguyand we have to keep that bug09:03
johnthetubaguyalex_xu: I see it really as a bug in the v2.1 code's v2 compatibility mode09:03
johnthetubaguynow I keep getting the dividing lines wrong on this API version stuff, so I certainly want to see what folks like jaypipes think about it09:04
*** annashen has quit IRC09:04
alex_xuemm....it sounds like we need force people upgrade his client directly to 2.current when they switch to v2.1.09:05
*** oomichi has joined #openstack-nova09:05
*** otter768 has quit IRC09:06
alex_xujohnthetubaguy: ok, another question is about policy, you mean we need update this http://docs.openstack.org/developer/nova/policy_enforcement.html?09:06
johnthetubaguyalex_xu: yes, I think so09:08
alex_xujohnthetubaguy: ok, I can update that to show the current status.09:09
*** afazekas has joined #openstack-nova09:09
alex_xujohnthetubaguy: or you already have plan free feel to update it09:10
johnthetubaguyalex_xu: but things like renaming the policy to match the API URLs and having the defaults in the code09:10
johnthetubaguyalex_xu: so I don't see me getting to it soon enough, so if someone in the API subteam can get to it, that would be awesome09:10
johnthetubaguyalex_xu: its on my TODO list incase its forgotten09:10
johnthetubaguyjust the policy talk earlier reminded me!09:10
openstackgerritYuiko Takada proposed openstack/nova: Drop MANIFEST.in - it's not needed with PBR  https://review.openstack.org/19788809:11
*** claudiub has joined #openstack-nova09:11
gmannjohnthetubaguy: alex_xu : oomichi :  IMO /v2 and /v2.1 should only differ with validation. if we make much difference between /v2 and /v2.1 like this bug fix then, it will be more confusing and not much validate our statement about v2.1 same as v209:11
gmannsorry for switching to that topic again :)09:11
alex_xujohnthetubaguy: I remember we say waiting for dynamic policy before we doing any improvement in nova side.09:11
johnthetubaguyalex_xu: back to the other question, the folks accessing /v2 should never see a change right, when the move to requesting any specific version, they always get the same thing, v2.current is a bit nasty really, and a special case09:12
johnthetubaguyalex_xu: we stil need to write down what we want that long term plan to be, to help inform the dynamic policy discussion, personally I don't think we should wait, but thats me09:12
johnthetubaguygmann: in theory yes, but v2.1 has already been released without that attribute09:12
johnthetubaguygmann: now its in use, we can't violate the API contract09:13
oomichigmann: +1 without a new microversion09:13
johnthetubaguyoomichi: but that is breaking our v2.1 API contract09:13
alex_xuv2.1 equal to v2 is contract except validation09:13
oomichijohnthetubaguy: that is a little difficult balance between v2 compatible and microversion manner09:14
oomichijohnthetubaguy: yeah, I can see the point09:14
oomichijohnthetubaguy: but users who use the extension cannot switch to v2.1 without specifying a microversion09:14
alex_xujohnthetubaguy: you mean the doc needn't wait, or you mean those policy work won't wait09:14
oomichijohnthetubaguy: on current implementation.09:14
*** prashant__ has joined #openstack-nova09:14
oomichijohnthetubaguy: v2 compatibility is base feature of v2.1 api, that seems a bug09:15
*** garyk has quit IRC09:15
*** humble__ has joined #openstack-nova09:16
johnthetubaguyalex_xu: the doc should be done already, it was a post summit action, but non of us have done that yet, the policy work, I don't think it has to wait, if we get time for it, or at least we should talk through the options there09:16
johnthetubaguyalex_xu: I thought we were waiting for oslo.policy changes, rather than keystone, but I am out of date on that conversation09:16
*** garyk has joined #openstack-nova09:16
johnthetubaguyoomichi: so back to what I said before, when the user requests v2 and we use the v2.1 code base, we MUST return that missing attribute09:17
johnthetubaguyoomichi: agreed, that bit is a bug that needs fixing09:17
alex_xujohnthetubaguy: ok, got it, let check the plan first.09:17
johnthetubaguyoomichi: now the current v2.1 API is released, we said when you request a specific version we will never touch that API, as such, if you request v2.1 explicity (all the way up the microversion we add for this change), the user should still get the buggy response that has the attribute missing09:18
johnthetubaguyoomichi: now thats hard, but it feels like the correct thing here09:18
*** gszasz has joined #openstack-nova09:18
*** openstackgerrit has quit IRC09:19
*** humble_ has quit IRC09:20
*** openstackgerrit has joined #openstack-nova09:20
alex_xuanother question, will we have '/v3' in the future. or we still plan to get rid of version from the url?09:22
gmannjohnthetubaguy: but i think it is also valid that if user move to /v2.1 (without specify microversion ), it should not break them. has to be same as /v209:22
*** jasondotstar has joined #openstack-nova09:22
oomichijohnthetubaguy: yeah, I can see your point. on microversions manner, your saying is exactly right. but I feel most users of v2.1 are switched from v2.0. so this bug seems hard to them.09:22
johnthetubaguygmann: when they move to a new version, it might be completely different, in theory09:22
johnthetubaguyoomichi: I don't see that as a good enough reason to violate our API rules, but that might just be me09:23
*** afazekas has quit IRC09:24
*** dims has joined #openstack-nova09:24
gmannjohnthetubaguy: humm both things seems valid here but breaking user with that bug seems more dangerous for me :)09:27
johnthetubaguygmann: thing is, I see the current change as a bug that will break the user of the v2.1 API09:27
*** exploreshaifali has joined #openstack-nova09:28
johnthetubaguyto be clear, I am not -2 on this, I am -1 on it09:29
openstackgerritSylvain Bauza proposed openstack/nova-specs: Proposes to check destination on migrations  https://review.openstack.org/17922409:29
johnthetubaguyit feels like the wrong trade off, but I have no intention of blocking the change, if that make sense09:29
*** tojuvone has joined #openstack-nova09:30
bauzasalex_xu: thanks for your review, I was unclear - I made a new PS trying to be more concise09:30
alex_xubauzas: np09:30
bauzasalex_xu: so basically, HTT400 if force and no host09:30
bauzasalex_xu: and HTTP409 if no force and host, but the host is invalid for the scheduler (which returns a NoValidHost)09:31
johnthetubaguygmann: oomichi alex_xu to be more specific, the proposed change is a lot lot better that not doing anything09:31
alex_xubauzas: currently noValidHost reutrn 400 also09:33
*** bradjones has quit IRC09:33
alex_xujohnthetubaguy: yea!09:33
bauzasalaski: I know09:33
gmannjohnthetubaguy: i see09:33
bauzasalaski: here we're changing the behaviour anyway, so I think providing a 409 is definitely a good thing09:34
*** aix has joined #openstack-nova09:34
oomichijohnthetubaguy: who can approve patch which is -1d by PTL :-(09:34
bauzasalex_xu: because it's not a malformed request, right ?09:34
gmannoomichi: nice point :)09:34
oomichijohnthetubaguy: I cannot do that at least09:34
alex_xuhah09:35
johnthetubaguyoomichi: people really should!09:35
*** yingjun has quit IRC09:35
*** bradjones has joined #openstack-nova09:35
*** bradjones has quit IRC09:35
*** bradjones has joined #openstack-nova09:35
johnthetubaguyoomichi: its like the pope, I am not voting as PTL unless I say I am, or something like that :)09:35
alex_xubauzas: let me think about it.09:35
oomichijohnthetubaguy: omg09:35
bauzasalex_xu: MHO is that a NoValid host should be a 5xx09:36
bauzasalex_xu: I mean, a normal error when botting09:36
bauzasbooting even09:36
bauzasalex_xu: because that's a server problem09:36
bauzasalex_xu: somethink like a 50309:36
gmannjohnthetubaguy: what you thing about - https://review.openstack.org/#/c/195412/09:36
bauzasalex_xu: here, if providing a dest host is optional and the host fails, it's a 409 - because the user asked for it09:37
gmannjohnthetubaguy: mriedem nd me thought of this as bug fix09:37
bauzasalex_xu: API meeting tomorrow ?09:38
gmannjohnthetubaguy: as it is just returning the correct response for --tenant_id filter on server list09:38
alex_xubauzas: emm...question become hard...09:39
alex_xubauzas: yes, we have meeting09:39
johnthetubaguygmann: the problem is when can a user discover its fixed, ideally they could look at the microversion09:39
gmannjohnthetubaguy: but i saw 2.5 kinda same line09:39
openstackgerritLucas Alvares Gomes proposed openstack/nova: Ironic: Improve driver logs  https://review.openstack.org/19789609:39
johnthetubaguygmann: as you can tell, we are all learning right now about when to bump a microversion, its tricky, I certainly don't claim to be an expert yet, more the opposite09:39
openstackgerritQin Zhao proposed openstack/nova: Refuse to delete compute service if host is in host aggregate  https://review.openstack.org/19764809:39
*** afazekas has joined #openstack-nova09:39
gmannjohnthetubaguy: yea, that is better way to publish fix but just release notes would not help?09:40
* alex_xu go to dinner, back after few minutes09:40
bauzasalex_xu: actually, thinking about it more, I consider that NoValidHost should be 409 for all cases09:40
bauzasalex_xu: lemme give you an example09:40
johnthetubaguygmann: no, as the user couldn't tell what a random cloud is currently able to do, they could be deployed off any commit in the history of Nova, for all the user knows09:41
bauzasalex_xu: I have 3 apples, Alice is asking for 1 and you're asking for 409:41
openstackgerritBalazs Gibizer proposed openstack/nova: Add functional test for server group  https://review.openstack.org/14751409:41
bauzasalex_xu: there is a conflict here09:41
openstackgerritBalazs Gibizer proposed openstack/nova: scheduler:refactor group info in filter_properties  https://review.openstack.org/14827509:41
openstackgerritBalazs Gibizer proposed openstack/nova: api: add soft-affinity policies for server groups  https://review.openstack.org/14752009:41
openstackgerritBalazs Gibizer proposed openstack/nova: Add functional tests to soft affinity capabilities  https://review.openstack.org/14752109:41
openstackgerritBalazs Gibizer proposed openstack/nova: scheduler: add num of instance per host in a group  https://review.openstack.org/14827709:41
openstackgerritBalazs Gibizer proposed openstack/nova: object:count members per hosts in InstanceGroup  https://review.openstack.org/14751709:41
openstackgerritBalazs Gibizer proposed openstack/nova: scheduler: add soft-(anti-)affinity weighers  https://review.openstack.org/14751609:41
bauzasalex_xu: the only way that you can get some apples is to reduce the level of expectation that you have - and ask for 209:41
johnthetubaguygmann: for example, rackspace public cloud has never deployed a tagged release, AFAIK09:42
*** Jeffrey4l has quit IRC09:42
gmannjohnthetubaguy: hummm09:43
gmannjohnthetubaguy: yea with 2.5 where we fixed same kinda bug of filtering with ipv6, we should have version to publish this fix too09:45
*** gilliard has quit IRC09:46
gmannjohnthetubaguy: or better way to be have conclusion over sdague spec about when to bump version by giving this example?09:46
*** gilliard has joined #openstack-nova09:46
*** oomichi has quit IRC09:46
johnthetubaguygmann: we sure need to write down these examples at some point, it does feel a lot like v2.509:47
johnthetubaguygmann: I know I voted the wrong way around on that on initially, wasn't till I though more about the discoverability from the user point of view it made sense to bump09:48
*** apuimedo has joined #openstack-nova09:48
gmannjohnthetubaguy: ok, sure. i will push that for version bump09:49
gmannjohnthetubaguy: yes, agree that is nice point from user point of view09:49
rgerganovgilliard: hi09:50
gmannjohnthetubaguy: 1 more question, can we push spec in liberty folder? or should we wait for mita one to open?09:50
rgerganovgilliard: the discussions about the console API change that I am trying to make seems to never end :)09:50
*** claudiub has quit IRC09:50
rgerganovgilliard: can I get your opinion on the latest issue09:50
rgerganovgilliard: https://review.openstack.org/#/c/148509/09:50
openstackgerritRoman Dobosz proposed openstack/nova: Introducing new forced_down field for a Service object  https://review.openstack.org/18528009:51
openstackgerritRoman Dobosz proposed openstack/nova: New nova API call to mark nova-compute down  https://review.openstack.org/18408609:51
openstackgerritRoman Dobosz proposed openstack/nova: Service group drivers forced_down flag utilization  https://review.openstack.org/19706309:51
openstackgerritRoman Dobosz proposed openstack/nova: Refactoring of the os-services module  https://review.openstack.org/18527909:51
johnthetubaguygmann: you mean for the API change?09:51
gmannjohnthetubaguy: yes09:52
johnthetubaguygmann: if its an API bug fix, I think just go for liberty, and raise it in the nova-meeting so it doesn't get ignored09:52
gmannjohnthetubaguy: ohkk,09:52
johnthetubaguygmann: we shouldn't slow down bug fixes, I probably haven't made that totally clear in the guidelines yet09:52
johnthetubaguygmann: the spec freeze is more a feature thing09:53
gmannjohnthetubaguy: sounds good09:53
johnthetubaguyalex_xu: just incase you didn't know about my take on API bug fixes and specs ^ I think its still good to propose those, as its a bug fix09:53
gmannjohnthetubaguy: ll do that tomorrow. need to leave for today09:54
gmannjohnthetubaguy: thanks a lot for your time09:54
johnthetubaguygmann: no problem, thank you for your hard work on these things09:54
openstackgerritLucas Alvares Gomes proposed openstack/nova: Ironic: Improve driver logs  https://review.openstack.org/19789609:55
*** ajayaa has joined #openstack-nova09:55
*** Jeffrey4l has joined #openstack-nova09:55
*** nagyz_ has quit IRC09:57
*** nagyz_ has joined #openstack-nova09:57
*** annashen has joined #openstack-nova10:01
openstackgerritjichenjc proposed openstack/nova: Refactory test cases for live-migrate error case  https://review.openstack.org/17904810:02
*** annashen has quit IRC10:06
*** ociuhandu has joined #openstack-nova10:12
openstackgerritVineet Menon proposed openstack/nova-specs: Add flavor tables to API database  https://review.openstack.org/19014710:15
*** IanGovett has joined #openstack-nova10:16
*** ociuhandu has quit IRC10:19
*** mutoulbj_ has quit IRC10:19
*** mutoulbj has joined #openstack-nova10:20
*** mutoulbj has quit IRC10:25
*** piyanai has joined #openstack-nova10:27
*** ociuhandu has joined #openstack-nova10:27
*** romainh has left #openstack-nova10:31
*** doude has quit IRC10:32
*** prashant__ has quit IRC10:33
openstackgerritSylvain Bauza proposed openstack/nova: Add RequestSpec methods for primitiving into dicts  https://review.openstack.org/18719710:33
openstackgerritSylvain Bauza proposed openstack/nova: Use RequestSpec object in HostManager  https://review.openstack.org/19125110:33
openstackgerritSylvain Bauza proposed openstack/nova: Use the same pci_requests field for all filters and HostManager  https://review.openstack.org/19125010:33
openstackgerritSylvain Bauza proposed openstack/nova: Remove unnecessary method in FilterScheduler  https://review.openstack.org/18848510:33
openstackgerritSylvain Bauza proposed openstack/nova: Consider that all scheduler calls are IO Ops  https://review.openstack.org/18848610:33
*** doude has joined #openstack-nova10:33
openstackgerritSylvain Bauza proposed openstack/nova: Create RequestSpec object  https://review.openstack.org/14552810:34
openstackgerritSylvain Bauza proposed openstack/nova: Use RequestSpec object in the FilterScheduler  https://review.openstack.org/18848710:34
openstackgerritSylvain Bauza proposed openstack/nova: Add two fields to ImageMetaProps object  https://review.openstack.org/19791110:34
openstackgerritSylvain Bauza proposed openstack/nova: WIP: Modify filters for using RequestSpec object  https://review.openstack.org/19791210:34
*** rfolco has quit IRC10:35
*** lyanchih_ has quit IRC10:36
*** lyanchih has joined #openstack-nova10:40
*** rushiagr_away is now known as rushiagr10:42
*** claudiub has joined #openstack-nova10:42
*** lpetrut has joined #openstack-nova10:43
*** ftersin has quit IRC10:46
*** lyanchih has quit IRC10:50
openstackgerritEugeniya Kudryashova proposed openstack/python-novaclient: Raise exception when API does't support microversions  https://review.openstack.org/18881610:51
*** tojuvone has quit IRC10:53
*** hakimo has joined #openstack-nova10:55
*** mnestratov has joined #openstack-nova10:57
*** mnestratov|2 has quit IRC11:00
*** annashen has joined #openstack-nova11:02
*** otter768 has joined #openstack-nova11:02
*** annashen has quit IRC11:06
*** otter768 has quit IRC11:06
*** lpetrut has quit IRC11:08
openstackgerritNikola Dipanov proposed openstack/nova: libvirt: Always default device names at boot  https://review.openstack.org/18963211:09
openstackgerritNikola Dipanov proposed openstack/nova: virt: add get_device_name_for_instance to the base driver class  https://review.openstack.org/18063711:09
openstackgerritNikola Dipanov proposed openstack/nova: libvirt: implement get_device_name_for_instance  https://review.openstack.org/18063811:09
*** mwagner_lap has quit IRC11:13
*** mutoulbj has joined #openstack-nova11:14
*** sahid has joined #openstack-nova11:18
alex_xujohnthetubaguy: got it11:20
*** dprince has joined #openstack-nova11:20
*** exploreshaifali has quit IRC11:23
*** kiran-r has quit IRC11:27
*** sahid has quit IRC11:27
*** lpetrut has joined #openstack-nova11:28
neiljerramHey danpb - could you give me any kind of schedule for reviewing https://review.openstack.org/#/c/146914/ ?  Also am I correct to ask you to review first, or should I ask other cores to look in parallel?11:30
*** sahid has joined #openstack-nova11:33
*** yingjun has joined #openstack-nova11:33
*** tlbr has quit IRC11:34
*** tlbr has joined #openstack-nova11:35
openstackgerritClaudiu Belu proposed openstack/nova-specs: Adds Hyper-V vNUMA enable spec  https://review.openstack.org/19616211:37
*** thorst has joined #openstack-nova11:38
*** tojuvone has joined #openstack-nova11:38
alex_xubauzas: for 503, I'm not sure, and there is guideline at api-wg https://review.openstack.org/#/c/183698/3/guidelines/http.rst that didn't go to the direction you said11:45
*** jasondotstar has quit IRC11:50
*** ferest has joined #openstack-nova11:50
openstackgerritClaudiu Belu proposed openstack/nova-specs: Adds Hyper-V UEFI Secure Boot spec  https://review.openstack.org/19099711:52
*** jang has joined #openstack-nova11:52
*** jang has quit IRC11:52
*** mwagner_lap has joined #openstack-nova11:53
*** dprince has quit IRC11:54
*** dprince has joined #openstack-nova11:54
*** lyarwood has quit IRC12:01
*** lyarwood has joined #openstack-nova12:01
*** zhenguo has quit IRC12:01
*** matrohon has quit IRC12:02
*** annashen has joined #openstack-nova12:02
*** matrohon has joined #openstack-nova12:03
*** ferest has quit IRC12:03
*** nagyz_ has quit IRC12:04
*** nagyz_ has joined #openstack-nova12:05
*** annashen has quit IRC12:07
*** exploreshaifali has joined #openstack-nova12:08
*** rfolco has joined #openstack-nova12:13
andreykurilinjohnthetubaguy: hi! my spec is ready:) http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/novaclient-api-microversions.html and bp is pending approval:)12:17
openstackgerritVictor Stinner proposed openstack/nova: Port crypto to Python 3  https://review.openstack.org/19774812:21
*** yonglihe has joined #openstack-nova12:23
*** mpaolino has quit IRC12:24
*** aysyd has joined #openstack-nova12:25
*** sharkJing has joined #openstack-nova12:25
*** yingjun has quit IRC12:26
*** yingjun has joined #openstack-nova12:28
*** dave-mccowan has joined #openstack-nova12:30
*** yonglihe has quit IRC12:31
*** jaypipes has joined #openstack-nova12:32
*** annegentle has joined #openstack-nova12:34
*** Redcavalier has joined #openstack-nova12:34
*** matrohon has quit IRC12:35
*** matrohon has joined #openstack-nova12:35
*** jasondotstar has joined #openstack-nova12:37
*** nkrinner has quit IRC12:38
*** deepthi has quit IRC12:38
rgerganovjaypipes: hi Jay, I desperately need your opinion on https://review.openstack.org/#/c/148509/12:38
jaypipesrgerganov: gimme 2 minutes to finish up an email and I'm on i.12:39
jaypipesit.12:39
rgerganovjaypipes: thanks!12:39
*** baoli has joined #openstack-nova12:40
*** Jeffrey4l has quit IRC12:41
*** edmondsw has joined #openstack-nova12:42
*** lpetrut has left #openstack-nova12:46
openstackgerritNikola Dipanov proposed openstack/nova: Fix overloading of block device on boot by device name  https://review.openstack.org/19032412:46
openstackgerritNikola Dipanov proposed openstack/nova: API: make sure a blank volume with no size is rejected  https://review.openstack.org/18891312:46
openstackgerritNikola Dipanov proposed openstack/nova: Make Nova better at keeping track of volume sizes in BDM  https://review.openstack.org/18891412:46
openstackgerritNikola Dipanov proposed openstack/nova: compute: Make swap_volume with resize updates BDM size  https://review.openstack.org/18891512:46
openstackgerritNikola Dipanov proposed openstack/nova: snapshot: Add device_name to the snapshot bdms  https://review.openstack.org/19032312:46
openstackgerritNikola Dipanov proposed openstack/nova: Add documentation for block device mapping  https://review.openstack.org/19265112:46
jaypipesndipanov: hmm, you've been busy this morning :)12:46
*** lpetrut has joined #openstack-nova12:46
*** sudipto has quit IRC12:47
*** andreykurilin__ has quit IRC12:47
*** prashant__ has joined #openstack-nova12:47
*** ivasev has joined #openstack-nova12:49
jaypipessdague: around? want to chat with you about https://review.openstack.org/#/c/148509/24/doc/v3/api_samples/os-remote-consoles/v2.6/create-vnc-console-req.json12:50
jaypipessdague: basically, kenichi is absolutely correct. however I still don't want to add a top-level key to the response :)12:50
johnthetubaguyandreykurilin: so we might want to move the blueprint out of python-novaclient into nova, but approved where it is for the moment12:51
*** vladikr has joined #openstack-nova12:51
*** deepthi has joined #openstack-nova12:51
bauzasjaypipes: FYI I updated both my spec and the requestspec object series12:52
bauzasjaypipes: you'll see a WIP for the latest patch in the series12:52
andreykurilinjohnthetubaguy: thanks for approve. why it should be in nova?12:52
jaypipesbauzas: rock on. I'll get to that as soon as I finish rgerganov's patch review.12:52
bauzasjaypipes: no worries, take your time12:52
*** jang has joined #openstack-nova12:53
*** lyarwood has quit IRC12:53
bauzasjaypipes: I mean, I'm continuing to work on the last patch, so there is not really an issue for reviewing12:53
*** lyarwood has joined #openstack-nova12:53
jaypipesbauzas: kk12:53
sahidndipanov: can i have your eye on this https://review.openstack.org/#/c/165838/ I hope it address your complaint :)12:54
johnthetubaguyandreykurilin: its more we don't really use the blueprint in python-novaclient, usually, but lets see how that goes12:54
*** Jeffrey4l has joined #openstack-nova12:54
*** mpaolino has joined #openstack-nova12:56
*** mpaolino has quit IRC12:56
openstackgerritBalazs Gibizer proposed openstack/nova: Make on_shared_storage optional in compute manager  https://review.openstack.org/19795112:56
*** prashant__ has quit IRC12:57
jaypipesrgerganov: done12:59
*** Dev_Jin has quit IRC13:01
rgerganovjaypipes: thanks a lot, I will add example response13:01
johnthetubaguyclaudiub: hey, can you please help we understand the hyper-v specs that you have up for for the exception freeze?13:03
bauzascan see some build failures due to http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiYWZ0ZXIgNjUgbWludXRlc1wiIEFORCB0YWdzOlwiY29uc29sZS5odG1sXCIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6IjE0NDAwIiwiZ3JhcGhtb2RlIjoiY291bnQiLCJ0aW1lIjp7InVzZXJfaW50ZXJ2YWwiOjB9LCJzdGFtcCI6MTQzNTg0MjE2MzcwOCwibW9kZSI6IiIsImFuYWx5emVfZmllbGQiOiIifQ==13:03
bauzasis there any bug opened for that ?13:03
*** otter768 has joined #openstack-nova13:03
claudiubjohnthetubaguy: yep13:03
*** mmedvede has joined #openstack-nova13:03
*** annashen has joined #openstack-nova13:03
openstackgerritsahid proposed openstack/nova: libvirt: introduce GuestBlock to wrap around Block API  https://review.openstack.org/19021013:03
openstackgerritsahid proposed openstack/nova: libvirt: update swap volume to use Guest  https://review.openstack.org/19021113:03
openstackgerritsahid proposed openstack/nova: libvirt: rename GuestVCPUInfo to VCPUInfo  https://review.openstack.org/19413313:03
openstackgerritsahid proposed openstack/nova: libvirt: return whether a domain is persistent  https://review.openstack.org/19096213:03
openstackgerritsahid proposed openstack/nova: libvirt: make unit tests concise by setup guest object  https://review.openstack.org/19096313:03
openstackgerritsahid proposed openstack/nova: libvirt: introduce method to wait for block device job  https://review.openstack.org/19021413:03
openstackgerritsahid proposed openstack/nova: libvirt: update live snapshot to use Guest object  https://review.openstack.org/19021213:03
openstackgerritsahid proposed openstack/nova: libvirt: update volume delete snapshot to use Guest  https://review.openstack.org/19021313:03
openstackgerritsahid proposed openstack/nova: libvirt: save the memory state of guest  https://review.openstack.org/19699113:03
openstackgerritsahid proposed openstack/nova: libvirt: make host return guest information  https://review.openstack.org/19104613:03
openstackgerritsahid proposed openstack/nova: libvirt: make host return power state of guest  https://review.openstack.org/19104713:03
johnthetubaguyclaudiub: what is vNUMA, it seems to be quite different to what libvirt is already doing with NUMA13:03
sahidjaypipes: done !13:03
jaypipessahid: :) rock on. I'll get to that shortly.13:04
*** mutoulbj has quit IRC13:04
johnthetubaguyclaudiub: is this about exposing the guest to the NUMA architecture it is living on? I am quite confused what it is for13:04
*** eglynn has quit IRC13:04
sahideheh cool :)13:04
claudiubjohnthetubaguy: yeah13:04
johnthetubaguyclaudiub: so thats quite different to what libvirt is doing right now, well it kinda mixes two concepts together I think13:04
claudiubjohnthetubaguy: basically, VMs can be spawned in certain physical NUMA nodes. Or have the memory / vcpus split in a few NUMA nodes.13:05
*** piyanai has quit IRC13:05
*** Jeffrey4l has quit IRC13:05
johnthetubaguyclaudiub: so why does this not use the same settings as libvirt?13:06
*** matrohon has quit IRC13:06
*** Jeffrey4l has joined #openstack-nova13:06
claudiubjohnthetubaguy: I am uncertain to what settings you reffer to.13:06
*** matrohon has joined #openstack-nova13:06
claudiubjohnthetubaguy: but anyways, the VMs still have to be enabled for it, the NUMA topology still have to reported by the HyperVDriver13:07
jaypipesjohnthetubaguy: any chance we can see the instance tagging spec approved today?13:07
jaypipesjohnthetubaguy: I know snikitin would sure appreciate it :)13:07
*** otter768 has quit IRC13:07
*** annashen has quit IRC13:08
johnthetubaguyjaypipes: looking through all the exceptions again, I think I +2ed that one already13:08
johnthetubaguyclaudiub: these bits: https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L26313:09
jaypipesjohnthetubaguy: right, so perhaps you are looking for a +2 from another driver?13:10
claudiubjohnthetubaguy: oh, you mean the constraints specified in the flavor extra spec or image properties. Well, for those properties I took a look at: http://specs.openstack.org/openstack/nova-specs/specs/juno/implemented/virt-driver-numa-placement.html13:11
johnthetubaguyjaypipes: ah, yeah, thats probably right, I don't quite remember the state of that specific one13:11
claudiubjohnthetubaguy: but yeah, I should update the spec to include the newer image properties / flavor extra specs13:11
*** karmatronic has joined #openstack-nova13:12
johnthetubaguyclaudiub: yeah, not sure I remember the history on those changes13:12
claudiubjohnthetubaguy: from what I saw in the specs, danpb should have more information on the changes.13:13
*** karimb has quit IRC13:13
claudiubjohnthetubaguy: I will update the spec according to the link you've provided.13:14
jaypipesclaudiub: BTW, did I ever mention that me, my wife, dansmith and his wife, stayed at the same airbnb place right after you left in YVR? :) Was funny... I walked into the house and Grant introduced himself, and there I see an openstack/mirantis sticker, and then your biz card next to it :)13:14
claudiubjaypipes: oh really? :))13:15
*** ijuwang has joined #openstack-nova13:15
claudiubjaypipes: maybe it was fate. :D13:15
jaypipesclaudiub: yeah. was a great place and location, wasn't it?13:15
johnthetubaguyclaudiub: added some comments in the spec, honestly, not sure its going to make it at this point I am afraid13:16
*** zhenguo has joined #openstack-nova13:16
*** karmatronic has quit IRC13:17
*** zzzeek has joined #openstack-nova13:18
openstackgerritTimofey Durakov proposed openstack/nova: Split instance resize operation into 2 phases  https://review.openstack.org/19508813:18
openstackgerritTimofey Durakov proposed openstack/nova: Create class hierarchy for tasks in conductor  https://review.openstack.org/18449513:18
claudiubjaypipes: true, but we did not get to stay long at the house. Just long enough to sleep. It is a shame that we did not get to stay longer. We didn't know that we would receive a 5 year visa, so we only stayed during the conference period + 2-3 days. :(13:18
jaypipesah :(13:20
jaypipesshame.13:20
claudiubjohnthetubaguy: thanks for the comments. :) well, it did look to me that it was the same feature as the one implemented in libvirt.13:20
ndipanovclaudiub, can you link the spec for numa on hyperv13:21
claudiubjaypipes: well, afaik, I won't need visa for japan, so I'm looking forward to it, that is my #1 place to visit. :D13:21
*** READ10 has joined #openstack-nova13:21
claudiubndipanov: https://review.openstack.org/19616213:21
*** zzzeek has quit IRC13:22
jaypipesclaudiub: nice :)13:23
*** ankit_ag has quit IRC13:23
danpbdoes anyone know why tox would be complaining about lines in test-requirements.txt looking like13:23
danpbfutures>=3.0;python_version=='2.7' or python_version=='2.6'13:23
danpbit doesn't like the ';' and stuff that follows13:23
claudiubdanpb: i think it needs pbr 1.2.013:23
claudiubdanpb: i had the same question yesterday13:23
*** thangp has joined #openstack-nova13:23
claudiubdanpb: fetching link13:24
johnthetubaguyclaudiub: I think its better to make the spec just say what existing things you are adopting, and point to the code where they are already defined, I guess. Makes it easier to see what happening13:24
*** xgsa has quit IRC13:24
ndipanovclaudiub, so why don't you want to use our properties13:24
claudiubdanpb: i think this is it: https://github.com/openstack-dev/pbr/commit/4b1503849c33cb9b6c0c0cb8b3322204d396249e13:25
danpboh, i ee13:25
danpbsee13:25
*** thumpba has joined #openstack-nova13:25
ndipanovimho user specifying nodes it wants cpus to go is inferior (read: broken)13:25
*** mriedem has joined #openstack-nova13:25
ndipanovour = libvirt13:25
claudiubndipanov: So, I've found the old NUMA spec that was added in juno and I based the properties on it: http://specs.openstack.org/openstack/nova-specs/specs/juno/implemented/virt-driver-numa-placement.html13:26
ndipanovright13:26
*** abhishekk has quit IRC13:26
ndipanovso that is just worded in a confusing way then13:26
claudiubndipanov: but I don't have a preference to which properties to use, but if libvirt is using a certain set of props, we should do it as well, for consistency13:26
ndipanovyes!13:27
ndipanovso carry on then :)13:27
claudiubjohnthetubaguy: yeah, sure, thanks for the advice. :)13:27
*** amotoki has joined #openstack-nova13:28
claudiubndipanov: yep, will do. :D13:28
*** sharkJing has quit IRC13:28
johnthetubaguyclaudiub: a few comments on the uefi spec, basically the same as danpb mentioned already I think, not sure I like the user being forced to know about the version, maybe we could do some hypervisor capability thing instead, so users don't need to worry, and we just do the "right" thing. One prop also looks hypervisors specific, which doesn't see quite right13:29
*** sharkJing has joined #openstack-nova13:29
*** karmatronic has joined #openstack-nova13:29
*** xgsa has joined #openstack-nova13:29
danpbjohnthetubaguy: the hypervisor version match sucks, but the schedular already has that functionality13:30
claudiubjohnthetubaguy: yeah, looking at them. yeah, the property which is hypervisor specific... the hyperv:secure_boot, it was a type / mistake. It was supposed to be "os:secure_boot". I didn't see it until now.13:30
danpband we don't have a good extensible way to expose compute host capabilities that i know13:30
johnthetubaguydanpb: yeah, but thats only when its required, we don't actually need it in this cases13:31
danpbso i figured it was ok to rely on that version match initially with understanding that we've got a bigger problem to fix here13:31
*** lucas__ has joined #openstack-nova13:31
johnthetubaguydanpb: claudiub: its maybe worth a quick note in the alternatives about that above thought process, thats probably fair enough for now I guess ^13:31
johnthetubaguyclaudiub: ah, no worries, I figured that just slipped through13:31
claudiubjohnthetubaguy: and yeah, really hoping for the hypervisor capabilities, since that will make our lives much easier. Was thinking about doing a POC on that, if no one's working on it13:32
*** zzzeek has joined #openstack-nova13:32
danpbi thought that was already mentioned in the spec13:32
johnthetubaguyclaudiub: yeah, there is an interesting project here around flavor extra specs and image properties, and hypervisor properties, if you fancy it13:33
johnthetubaguydanpb: ah, quite possibly, I maybe missed that13:33
ndipanovclaudiub, so basically all you need to do is just expose this data in get_available_resources really13:33
ndipanovand do the work in your driver to configure it13:33
ndipanoveverything else should work as is13:33
claudiubjohnthetubaguy: yeah, it sounds like an interesting idea to work on. Will do.13:34
claudiubndipanov: yeah, that should be enough.13:34
ndipanovclaudiub, cool - glad to see this being used by other drivers13:35
ndipanov:)13:35
*** piyanai has joined #openstack-nova13:35
claudiubndipanov: :D13:35
*** mutoulbj has joined #openstack-nova13:35
*** r-daneel has joined #openstack-nova13:36
openstackgerritsahid proposed openstack/nova: libvirt: add os_admin_user to use with set admin password  https://review.openstack.org/18592113:38
openstackgerritsahid proposed openstack/nova: libvirt: enhance libvirt to set admin password  https://review.openstack.org/18591013:38
openstackgerritsahid proposed openstack/nova: libvirt: rework quiesce to not share "sensitive" informations  https://review.openstack.org/19662013:38
openstackgerritClaudiu Belu proposed openstack/nova-specs: Adds Hyper-V UEFI Secure Boot spec  https://review.openstack.org/19099713:38
*** tojuvone has quit IRC13:38
*** piyanai has quit IRC13:40
*** apuimedo has quit IRC13:40
*** romainh has joined #openstack-nova13:41
*** ZZelle has quit IRC13:42
*** ZZelle has joined #openstack-nova13:42
*** tjones1 has joined #openstack-nova13:43
*** _gryf is now known as _gryf|afk13:43
*** nihilifer has quit IRC13:45
*** xgsa1 has quit IRC13:47
openstackgerritVictor Stinner proposed openstack/nova: Port crypto to Python 3  https://review.openstack.org/19774813:50
openstackgerritVictor Stinner proposed openstack/nova: Fix Python 3 issues in nova.utils and nova.tests  https://review.openstack.org/19798213:50
*** jkugel has joined #openstack-nova13:50
*** lyanchih has joined #openstack-nova13:51
*** achanda has joined #openstack-nova13:52
*** fifieldt has joined #openstack-nova13:53
*** jecarey has joined #openstack-nova13:55
*** garyk has quit IRC13:56
*** irenab has quit IRC13:56
*** dboik has quit IRC13:57
nagyz_dansmith, you are probably right, I guess an explicit "is None" is better?13:59
*** ctrath has joined #openstack-nova14:00
dansmithnagyz_: yeah14:00
mriedemnagyz_: dansmith: i commented on that in ps114:00
mriedemNone means we're shutting down14:00
mriedemin the bug report, people said they hit this when deleting an instance, not when shutting down14:01
mriedemso i don't think the None check itself would fix the bug14:01
nagyz_yes, I've hit it when trying to delete an instance14:01
dansmithmriedem: if it's not none, it won't fail, right?14:01
*** nikhil_k1away is now known as nikhil_k14:01
dansmithmaybe something else is going on then14:01
mriedemdansmith: empty14:01
claudiubndipanov: johnthetubaguy: I think the link you've reffered to previously https://github.com/openstack/nova/blob/master/nova/virt/hardware.py#L263 does not reffer to NUMA topology, it reffers to vCPU topology exposed to the VM. vCPU topology spec: http://specs.openstack.org/openstack/nova-specs/specs/juno/implemented/virt-driver-vcpu-topology.html14:02
mriedemlike maybe they delete the instance before _events has an entry for the instance14:02
dansmithmriedem: the bug specifically says NoneType has no pop() right?14:02
mriedemdansmith: yeah from the originator14:02
*** artom has joined #openstack-nova14:02
mriedemdansmith: nagyz_ said he also hit it when deleting an instance14:02
claudiubndipanov: johnthetubaguy: danpb also mentioned this as a comment: https://review.openstack.org/#/c/196162/2/specs/liberty/approved/hyper-v-vnuma-enable.rst,cm14:02
*** artom has quit IRC14:02
*** artom has joined #openstack-nova14:02
nagyz_mriedem, dansmith let me check my logs on the compute node before the fix14:02
dansmithnagyz_: can I see a stack trace of when you hit it via delete_instance?14:02
dimsmriedem: haypo: got time for an oslo.rootwrap review? please take a look when you get a chance https://review.openstack.org/#/c/180695/14:03
dansmithcan someone +W this? it's been around a while: https://review.openstack.org/#/c/192708/1114:04
*** annegentle has quit IRC14:04
*** annashen has joined #openstack-nova14:04
*** deepthi has quit IRC14:04
danpbdansmith: ack14:06
dansmithdanpb: thanks!14:08
*** tonytan4ever has joined #openstack-nova14:09
*** annashen has quit IRC14:09
*** wxy has joined #openstack-nova14:09
*** browne has joined #openstack-nova14:11
claudiubdanpb: quick question: the spec and the flavor extra specs defined here are still valid for L, right? http://specs.openstack.org/openstack/nova-specs/specs/juno/implemented/virt-driver-numa-placement.html14:13
*** Murali has quit IRC14:14
claudiubdanpb: I mean, that is still the way to do NUMA node placement?14:14
*** wxy has left #openstack-nova14:15
*** dboik has joined #openstack-nova14:16
*** dims has quit IRC14:19
danpbclaudiub: yes, that's mostly still accurate14:20
danpbNB *nothing*  allows the tenant user to decide NUMA node placement14:20
openstackgerritVictor Stinner proposed openstack/nova: Fix Python 3 issues in nova.utils and nova.tests  https://review.openstack.org/19798214:20
openstackgerritVictor Stinner proposed openstack/nova: Port crypto to Python 3  https://review.openstack.org/19774814:20
danpball the properties are about allowing the user to express the desired *guest* NUMA topology14:20
danpbthe schedular is always in control over how guest topology is placed on host topology14:20
claudiubdanpb: oh, I see. makes sense. Thanks for the info!14:21
*** aix has quit IRC14:21
danpbsee also http://docs.openstack.org/developer/nova/testing/libvirt-numa.html14:21
*** busterswt has joined #openstack-nova14:22
*** matrohon has quit IRC14:24
*** achanda has quit IRC14:24
*** matrohon has joined #openstack-nova14:24
jkugelhi, I see there was a blueprint to add support for scsi passthrough here: https://blueprints.launchpad.net/nova/+spec/support-for-raw-device-mapping.  Does anyone know how to find out more about the implementation (or how to use?)14:26
*** ildikov has quit IRC14:26
*** agireud has joined #openstack-nova14:27
kashyapmriedem: Fixed that said test with your suggestion. Thanks for that tip, indeed it reduced the diff. Hope it looks okay now.14:27
*** wxy_ has joined #openstack-nova14:27
*** erlon has joined #openstack-nova14:27
*** hogepodge has quit IRC14:28
*** ajayaa has quit IRC14:29
*** wxy_ has quit IRC14:29
*** eharney has quit IRC14:30
*** wxy_ has joined #openstack-nova14:30
*** wxy_ has quit IRC14:30
*** wxy_ has joined #openstack-nova14:30
mriedemkashyap: ok, looking14:31
mriedemif there is anyone else in the world familiar with the vmware driver, please help to review https://review.openstack.org/#/c/165060/14:32
mriedemi've gone through ^ 3 times now pointing out some major issues14:32
*** tjones1 has left #openstack-nova14:34
*** garyk has joined #openstack-nova14:36
openstackgerritQin Zhao proposed openstack/nova: Refuse to delete compute service if host is in host aggregate  https://review.openstack.org/19764814:37
*** wxy has joined #openstack-nova14:37
*** dims has joined #openstack-nova14:38
openstackgerritClaudiu Belu proposed openstack/nova-specs: Adds Hyper-V vNUMA enable spec  https://review.openstack.org/19616214:39
*** mpaolino_ has joined #openstack-nova14:39
*** wxy_ has quit IRC14:40
*** annegentle has joined #openstack-nova14:41
johnthetubaguyclaudiub: ah, yes, good point14:41
claudiubjohnthetubaguy: I've addressed your comments on the spec, uploaded a new patchset, danpb voted +2 on the vNUMA spec: https://review.openstack.org/#/c/19616214:43
*** dims has quit IRC14:44
*** matrohon has quit IRC14:45
nagyz_dansmith, mriedem http://pastebin.com/uau91CGD14:45
nagyz_that's the error I had on one of our compute nodes14:45
*** matrohon has joined #openstack-nova14:45
nagyz_sorry it took me a while I had a meeting.14:45
nagyz_:)14:45
*** nikhil_k is now known as nikhil_k1away14:45
*** ildikov has joined #openstack-nova14:46
*** nikhil_k1away is now known as nikhil_k14:46
*** nikhil_k is now known as nikhil_k1away14:46
*** nikhil_k1away is now known as nikhil_k14:46
mriedemnagyz_: and the compute service wasn't shutting down when this happened?14:46
nagyz_correct14:47
mriedemnagyz_: do you remember if you like, deleted the instance shortly after starting to build it?14:47
nagyz_no, it was up and running for a long while14:47
mriedemthe instance was active?14:47
nagyz_then I decided to terminate it, and it went into an error14:47
nagyz_yes14:47
mriedemhrm14:47
ctrathmriedem: I'm getting a tox error, which I don't understand.  :14:48
ctrath   mox3.mox.ExpectedMethodCallsError: Verify: Expected methods never called:14:48
ctrath      0.  ExtensionManager.is_loaded('os-used-limits-for-admin') -> None14:48
ctrath14:48
openstackgerritShaoHe Feng proposed openstack/nova: Remove db layer hard-code permission checks for floating_ip_dns  https://review.openstack.org/15071014:48
mriedemnagyz_: what do you have configured for vif_plugging_timeout  in nova.conf?14:48
ctrathdiff is here: http://paste.openstack.org/show/335858/14:48
mriedemnagyz_: and vif_plugging_is_fatal 14:48
dansmithnagyz_: on a call atm14:48
nagyz_neither is set, so defaults14:48
*** wxy has left #openstack-nova14:48
nagyz_mriedem, I can dig through the logs and get the history of the instance if need be14:49
nagyz_I don't have greylog2 installed yet for this system... (or what do you guys use ;-))14:49
*** dims has joined #openstack-nova14:49
*** ijw has joined #openstack-nova14:49
ctrathmriedem: if you could, please let me know if you have suggestions14:50
mriedemnagyz_: defaults are fine, was just wondering if maybe we were making an assumption that those are set and they weren't when this happened14:50
mriedemctrath: well, something is mocked out and isn't called and it's expecting it to be called, which is maybe obvious14:50
mriedemctrath: you don't want to be testing this in b/nova/tests/functional/test_api_samples.py14:51
*** mutoulbj has quit IRC14:51
mriedemctrath: the functional api_samples tests are not meant for negative testing like this14:51
mriedemctrath: sec14:52
*** mutoulbj has joined #openstack-nova14:52
*** mtanino has joined #openstack-nova14:52
mriedemctrath: you want your unit tests to be here http://git.openstack.org/cgit/openstack/nova/tree/nova/tests/unit/api/openstack/compute/test_limits.py14:52
ctrathok.. I will change that.. thank you14:52
mriedemand probably here http://git.openstack.org/cgit/openstack/nova/tree/nova/tests/unit/api/openstack/compute/contrib/test_used_limits.py14:52
mriedemctrath: and remember that you have to change the v2 and v2.1 extension code14:53
mriedemctrath: v2 is here http://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/compute/contrib14:53
mriedemctrath: v2.1 is here http://git.openstack.org/cgit/openstack/nova/tree/nova/api/openstack/compute/plugins/v314:53
mriedemv2.1 is the api formerly known as v314:54
mriedemit's the prince of APIs14:54
ctrathmriedem: also, why are there duplicate controllers?  one in /nova/api/openstack/compute/plugins/v3 and the other in /nova/api/openstack/compute/14:54
ctrathis it just different versioning?14:54
mriedemdifferent endpoints14:54
mriedemctrath: some history here http://specs.openstack.org/openstack/nova-specs/specs/kilo/implemented/v2-on-v3-api.html14:55
ctrathmriedem: thank you14:55
*** piyanai has joined #openstack-nova14:56
mriedemctrath: also https://www.youtube.com/watch?v=Ml5R5rOUhO4&index=26&list=PLN55_jfhfDeTwqLybjnsIK2Daad5nhOwi14:56
mriedemsummit video on v2.1 ^14:56
ctraththanks.. I'll have a look14:56
*** wxy has joined #openstack-nova14:57
*** yamahata has joined #openstack-nova14:57
*** bradjones has quit IRC14:57
*** fawadkhaliq has joined #openstack-nova14:58
*** Marga_ has quit IRC14:58
*** wxy has quit IRC14:58
*** bradjones has joined #openstack-nova14:58
*** bradjones has quit IRC14:58
*** bradjones has joined #openstack-nova14:58
*** Marga_ has joined #openstack-nova14:59
*** Marga_ has quit IRC14:59
mriedemnagyz_: can you update the bug with those details?14:59
mriedemand reply to dansmith's comment in the review14:59
*** Marga_ has joined #openstack-nova14:59
dansmithI'll be off in a sec15:00
*** hogepodge has joined #openstack-nova15:01
dansmithI think the problem is likely that we're getting None in there during a failed boot or something15:01
dansmithso we should fix that15:01
dansmithNone should be reserved for shutdown I think15:01
mriedemdansmith: nagyz_ said this was delete of an active instance actually15:01
dansmiththere is an instance->error log above it15:01
nagyz_dansmith, you can see from the stacktrace that it's actually being hit during terminate_instance15:01
mriedemnagyz_: i think that's from when the delete fails15:01
*** exploreshaifali has quit IRC15:01
dansmithah, maybe15:02
mriedemhere http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/manager.py#n236815:02
dansmithso anyway, I think we need to figure out how None is getting in there15:02
nagyz_the thing is I cannot reproduce it15:03
nagyz_I had one instance hit this so far out of a couple thousand15:03
dansmithsounds suspicious15:03
mriedemthe only thing that sets _events to None is cancel_all_events and that's only called from cleanup_host on shutdown15:03
*** otter768 has joined #openstack-nova15:04
mriedemmaybe ubuntu threw some fun patches in their kilo packages :)15:05
*** annashen has joined #openstack-nova15:05
*** scheuran has quit IRC15:05
*** yingjun has quit IRC15:05
*** emagana has joined #openstack-nova15:06
mriedemregarding the originator of the bug, that's also suspicious since i think he works on the powervc driver and they do all kinds of wonky things15:06
*** dims has quit IRC15:06
nagyz_I checked and ubuntu doesn't have any patches touching this area :)15:06
*** jskinner has joined #openstack-nova15:06
*** Guest42148 has joined #openstack-nova15:07
*** afazekas has quit IRC15:08
*** otter768 has quit IRC15:08
*** Guest42148 is now known as dims_15:09
*** mpavone has quit IRC15:09
nagyz_so given that it could actually block an operation that the user wants (termination) and could get billed for as resource consumption it makes sense to fix the code path while keeping an eye on the origin as well, but without reproducing it's hard to do that15:10
*** annashen has quit IRC15:10
*** sudipto has joined #openstack-nova15:10
*** belmoreira has quit IRC15:10
nagyz_so should I change it to a stricter check ?15:10
mriedemnagyz_: i think the shutdown case (None) is pretty clear15:10
*** Piet has quit IRC15:10
dansmithif we just add the check, we'll never be able to keep an eye on the supposed other case15:10
nagyz_I have no idea how to reproduce it, honestly. we have VMs getting created and deleted pretty frequent and I've only seen one instance of this15:11
nagyz_user experience should trump curiousity :)15:11
*** stefan_amann has joined #openstack-nova15:12
mriedemlike i said, i think the None (stricter) case is more clear that it could happen as a race during shutdown15:14
mriedemin ps1 when i was reviewing it, the LOG.debug level was bothering me since i was thinking, i don't really see how this would happen, so it should be warning at least15:15
nagyz_ok15:15
nagyz_I can change these.15:15
*** matrohon has quit IRC15:16
*** matrohon has joined #openstack-nova15:16
openstackgerritSylvain Bauza proposed openstack/nova: WIP: Modify filters for using RequestSpec object  https://review.openstack.org/19791215:17
openstackgerritJim Rollenhagen proposed openstack/nova: Flavor Classes: Add cells scheduler filter.  https://review.openstack.org/19802415:20
*** foexle has quit IRC15:20
nagyz_mriedem, but if I tighten up the requirement and we might expect to hit it then now are you ok with the debug instead of a warning?15:23
mriedemnagyz_: yeah15:24
mriedemdebug is fine if checking for None15:24
mriedemsince that's what we have above on the pop_instance_event method15:24
*** jcoufal has quit IRC15:25
nagyz_right15:25
*** hemna has joined #openstack-nova15:26
openstackgerritZoltan Arnold Nagy proposed openstack/nova: Handle unexpected clear events call  https://review.openstack.org/19664715:27
*** hemna has quit IRC15:29
*** hemna has joined #openstack-nova15:30
*** rajesht has quit IRC15:30
*** hemna has quit IRC15:30
*** kfox1111_away is now known as kfox111115:31
*** burt has joined #openstack-nova15:38
*** huazhihao has joined #openstack-nova15:39
*** sudipto has quit IRC15:39
mriedemnagyz_: one little thing to update with the log message, then i'm +215:39
*** armax has joined #openstack-nova15:39
*** huazhihao has quit IRC15:41
*** angdraug has joined #openstack-nova15:41
*** achanda has joined #openstack-nova15:41
openstackgerritAdam Kacmarsky proposed openstack/nova: Disable router advertisements on bridge devices  https://review.openstack.org/19803315:41
openstackgerritJesse J. Cook proposed openstack/nova: Cells: decrement available slots  https://review.openstack.org/19742515:42
nagyz_mriedem, roger15:43
mriedemroger?15:43
mriedemwhats your vector, victor15:43
nagyz_I only saw that vector video (you mean the matrix parody, right?) in hungarian... :)15:43
nagyz_not sure about the jokes in english :P15:43
mriedemno, callback to some airplane! quotes from yesterday15:43
openstackgerritAdam Kacmarsky proposed openstack/nova: Disable router advertisements on bridge devices  https://review.openstack.org/19803315:43
mriedemnagyz_: http://www.imdb.com/title/tt0080339/15:44
openstackgerritZoltan Arnold Nagy proposed openstack/nova: Handle unexpected clear events call  https://review.openstack.org/19664715:44
mtreinishmriedem: where's the youtube link, I'm disappointed...15:44
*** unicell has quit IRC15:44
mriedemhttps://www.youtube.com/watch?v=fVq4_HhBK8Y15:44
mtreinishthere you go15:44
nagyz_lol :)15:45
nagyz_I should watch the movie I guess... :)15:45
mriedemyes15:45
mriedemand per the conversation yesterday, keep in mind it was rated PG in 198015:45
nagyz_I've found the youtube link for the hungarian matrix parody but it's not subtitled :(15:46
mriedemthe only thing i know about hungary is goulash15:47
* nagyz_ is hungry15:47
mriedemas interpreted via my mom when i was a kid15:47
mriedemwho is very not hungarian15:47
nagyz_we can find a hungarian restaurant in tokyo ;-)15:48
mriedemi'll be at disney world the week before the summit so i'm sure they'll have one at epcot15:48
mriedemalaski: ^ i got roped in15:49
nagyz_I'll be coming back from my honeymoon just before... IF we survive.15:49
openstackgerritKevin Fox proposed openstack/nova-specs: Instance Users for Cloud Interaction  https://review.openstack.org/18661715:49
nagyz_ok, gtg, group bbq. talk to you later15:49
mriedemlater15:49
*** doude has quit IRC15:50
alaskimriedem: heh.  epcot is pretty awesome15:51
mriedemalaski: i have very low expectations on this whole trip15:51
mriedemdespite the cost15:51
*** mwagner_lap has quit IRC15:51
*** jlanoux has quit IRC15:52
*** Piet has joined #openstack-nova15:54
ndipanovmriedem, what trip?15:55
mriedemndipanov: disney world15:55
ndipanovmriedem, ah15:55
ndipanovyeah15:55
*** nihilifer has joined #openstack-nova15:55
ndipanovnot looking forward to that15:56
mriedemi'm not excited to go15:56
ndipanovin a few years time15:56
ndipanovI really don't like places like taht15:56
ndipanovtho according to my wife - the one near Tokyo is supposed to be super amazing15:57
openstackgerritKevin Fox proposed openstack/nova-specs: Instance Users for Cloud Interaction  https://review.openstack.org/18661715:57
*** dboik_ has joined #openstack-nova15:58
mriedemi'm all about staying in mediocre hotel rooms for $250/night15:58
mriedemand being ripped off for each meal15:58
*** dboik has quit IRC15:58
kashyap:-) "mediocre" for $25015:59
mriedemkashyap: right, that's the joke15:59
mriedembut seriously, they don't look great15:59
*** baoli has quit IRC15:59
mriedemi'm told you're not there to enjoy the hotel room though15:59
ndipanovno you're there to enjoy mediocre entertainment16:00
*** stefan_amann has quit IRC16:00
*** dboik has joined #openstack-nova16:00
*** baoli has joined #openstack-nova16:00
*** karmatronic has quit IRC16:01
*** jistr has quit IRC16:02
*** dboik_ has quit IRC16:03
alaskiI've never paid to go so my perspective is skewed.  But epcot and the animal kingdom were pretty awesome, the main park is cool for a few minutes and then it's good to go do something else16:03
*** andrewbogott has joined #openstack-nova16:04
mriedemalaski: 7 days16:04
alaskithat's a bummer16:04
andrewbogottI’ve just discovered to my dismay that the default nova policy.json allows any role member to do things that aren’t expressed in the policy file.  Specifically, any role can delete any instance.  Is there a way to make policy.json into a whitelist rather than a blacklist?16:04
mriedemalaski: maybe after day 4 i'll just rent a car and show up on your doorstep16:04
mriedemor jaypipes'16:04
alaskiheh, yeah jaypipes is much closer16:04
mriedemjaypipes has guard dogs though16:05
*** nihilifer has quit IRC16:05
jaypipesGuard Pugs! Dangerous! Party On.16:05
*** annashen has joined #openstack-nova16:06
*** busterswt has quit IRC16:06
*** derekh has quit IRC16:06
andrewbogott(sorry for my bad manners, still in a state of shock)16:06
*** matrohon has quit IRC16:06
alaskiandrewbogott: change the default rule to "!" to make it a blacklist16:07
alaskisorry, whitelist16:07
andrewbogottI already have "default": [["rule:admin_or_owner"]] at the top16:07
andrewbogottbut that didn’t prevent a member of a different role from deleting things...16:08
alaskithey should only be able to delete if it's the same project_id, or they have role:admin16:08
alaskiif that's not the case that's a bug that needs to be opened16:08
andrewbogottWhat do yiou mean by “It’s the same project_id” ?16:08
mriedemadmin or owner16:09
mriedemowner being the same project16:09
andrewbogottoh, so any role in a project is an ‘owner’?16:09
alaskithe 'owner' part of that means anyone with the same project id as the resource being acted on can do it16:09
alaskiyes16:09
andrewbogottahah!16:09
andrewbogottOk, easy fix for this then.16:09
andrewbogottThank you!16:09
jaypipes:)16:09
bauzasmriedem: honestly, planning to visit Disney Tokyo N16:09
bauzas?16:09
mriedembauzas: disney world. FL16:09
bauzasmriedem: oh ack16:09
mriedemwhere it all began....16:10
*** lucasagomes has quit IRC16:10
bauzasmriedem: because Bauza family will be there one week before :)16:10
mriedemi hope to find disney's head in a jar somewhere16:10
*** ijw has quit IRC16:10
bauzasmriedem: ha16:10
mriedemor was he frozen? i can't remember16:10
*** annashen has quit IRC16:10
andrewbogottalaski: If I set to "default": [["role:projectadmin"]] will that exclude ‘admin’ as well?  Or does ‘admin’ get a free pass no matter what?16:11
*** lucas__ has quit IRC16:11
*** piyanai has quit IRC16:11
mriedemandrewbogott: admin is admin16:12
mriedemso yes16:12
*** ndipanov has quit IRC16:12
alaskifor some things16:12
andrewbogottWait, was that ‘yes’ to… the first part or the second part?16:12
andrewbogottI’m wondering if I should create an ‘admin_or_projectadmin’ rule, or if that’s unneeded.16:13
alaskiadmins don't get to bypass policy checks16:14
*** sahid has quit IRC16:14
andrewbogottok, great.16:14
andrewbogottthanks16:14
alaskibut there are some specific checks in Nova for whether or not someone is an admin16:14
*** amotoki_ has joined #openstack-nova16:14
openstackgerritClaudiu Belu proposed openstack/nova-specs: Adds Hyper-V vNUMA enable spec  https://review.openstack.org/19616216:14
mriedemyeah...and there are definitely some things in the v2 api that still require admin16:14
mriedemeven if the policy says they don't16:15
mriedemlike listing hosts16:15
mriedemv2.1 api should be fully configurable from policy.json16:16
openstackgerritEric Brown proposed openstack/nova: Add bandit for security static analysis testing  https://review.openstack.org/17956616:16
alaskiyeah.  then admin becomes just another role16:16
*** amotoki has quit IRC16:17
jrolljaypipes: fyi I updated the ironic/compute thing last night https://review.openstack.org/#/c/194453/16:18
openstackgerritEric Brown proposed openstack/nova: Add bandit for security static analysis testing  https://review.openstack.org/17956616:18
*** piyanai has joined #openstack-nova16:18
jaypipesjroll: coolio. O'll have a looksy in a bit.16:18
jrollawesome, no rush. thanks :)16:18
openstackgerritAdam Kacmarsky proposed openstack/nova: Disable router advertisements on bridge devices  https://review.openstack.org/19805416:23
*** Mike_D_laptop has joined #openstack-nova16:23
*** baoli has quit IRC16:23
andrewbogottHm, docs say that “” means “everyone” but clearly it actually means “default rule”16:23
andrewbogottwhich leaves me without a way to permit everyone...16:23
*** baoli has joined #openstack-nova16:25
*** annashen has joined #openstack-nova16:26
*** tjones1 has joined #openstack-nova16:26
*** eharney has joined #openstack-nova16:31
openstackgerritSylvain Bauza proposed openstack/nova: WIP: Modify filters for using RequestSpec object  https://review.openstack.org/19791216:31
*** ivasev has quit IRC16:33
bauzasalaski: FYI, I'm pretty close to the end of the ReqSpec series16:34
bauzasalaski: so you can look at the series and see how to persist16:34
bauzasalaski: the last thing is just to change the API for providing a ReqSpec object directly16:35
*** karmatronic has joined #openstack-nova16:35
*** tonytan4ever has quit IRC16:37
*** hemnafk is now known as hemna16:38
*** READ10 has quit IRC16:41
*** annashen has quit IRC16:42
*** Marga_ has quit IRC16:43
*** Marga_ has joined #openstack-nova16:43
*** annashen has joined #openstack-nova16:43
*** annashen has quit IRC16:43
*** annashen has joined #openstack-nova16:44
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Add rootwrap daemon mode support  https://review.openstack.org/18069516:46
*** lucas_ has joined #openstack-nova16:47
openstackgerritJim Rollenhagen proposed openstack/nova-specs: Ironic: Multiple compute host support  https://review.openstack.org/19445316:48
*** lucas_ has quit IRC16:48
*** lucas_ has joined #openstack-nova16:48
*** tsekiyama has joined #openstack-nova16:50
openstackgerritAdam Kacmarsky proposed openstack/nova: Disable router advertisements on bridge devices  https://review.openstack.org/19805416:51
*** gszasz has quit IRC16:51
*** lucas_ has quit IRC16:52
*** karmatronic has quit IRC16:52
*** mestery has quit IRC16:55
*** patrickeast has joined #openstack-nova16:55
*** piyanai has quit IRC16:56
*** claudiub has quit IRC16:57
*** Marga_ has quit IRC17:00
openstackgerritJeffrey Zhang proposed openstack/nova: use graduated oslo.policy  https://review.openstack.org/19806517:00
*** gyee has joined #openstack-nova17:01
*** piyanai has joined #openstack-nova17:02
*** annashen has quit IRC17:02
*** whenry_ has joined #openstack-nova17:02
*** beagles is now known as beagles_brb17:03
*** annashen has joined #openstack-nova17:03
*** kiran-r has joined #openstack-nova17:04
*** otter768 has joined #openstack-nova17:05
*** gokrokve has joined #openstack-nova17:06
*** romainh has left #openstack-nova17:07
*** ijw has joined #openstack-nova17:07
*** annashen has quit IRC17:08
openstackgerritJeffrey Zhang proposed openstack/nova: use graduated oslo.policy  https://review.openstack.org/19806517:08
PaulMurraydansmith, melwitt jaypipes this was approved but failed tests - fixed now: https://review.openstack.org/#/c/16094217:09
*** mpaolino_ has quit IRC17:09
*** otter768 has quit IRC17:09
jaypipesPaulMurray: +Wallby'd it.17:09
dansmithwell,17:09
dansmithwe had a whole conversation about that17:10
PaulMurraydansmith, melwitt jaypipes I saw from comments that you had talked about the change I undid17:10
dansmithyeah17:10
dansmithI'17:10
PaulMurraydansmith, the ERT thing is initialised as part of the calculations that come after the init17:11
dansmithwill just pretend I didn't notice17:11
openstackgerritAdam Kacmarsky proposed openstack/nova: Disable router advertisements on bridge devices  https://review.openstack.org/19805417:11
PaulMurraydansmith, so the value it has at that point is not from resources and is not from calculations - so it isn't in line with anything else that is saved17:11
PaulMurraydansmith, if it is not written out, the value for vcpu will be what comes from the virt driver, same as everyhting else copied in at that point17:12
PaulMurrayso I thought best to stick with that17:12
PaulMurraybut anyway - thanks jaypipes17:13
PaulMurraydansmith, jaypipes sorry there was such a long gap doing that patch - got a bit busy for a while17:13
*** lyanchih has quit IRC17:15
*** piyanai has quit IRC17:16
*** piyanai has joined #openstack-nova17:17
jaypipesPaulMurray: no worries.17:18
jaypipesdansmith: hmm, sorry, I approved it thinking it was already approved.17:19
dansmithit was17:19
*** jasondotstar is now known as jasondotstar|afk17:19
alaskibauzas: great.  I was working on persistence for an older revision but I'll update my work17:20
*** piyanai_ has joined #openstack-nova17:20
alaskiandrewbogott: "@" permits everyone17:20
*** piyanai has quit IRC17:21
*** piyanai_ is now known as piyanai17:21
*** agireud has quit IRC17:23
*** neiljerram_bb has joined #openstack-nova17:26
*** rajesht has joined #openstack-nova17:27
*** piyanai has quit IRC17:27
andrewbogottalaski: perfect, thank you17:27
*** piyanai has joined #openstack-nova17:28
*** ijuwang has quit IRC17:29
*** eharney has quit IRC17:29
*** rajesht has quit IRC17:31
*** achanda has quit IRC17:33
*** kiran-r has quit IRC17:36
*** lucas__ has joined #openstack-nova17:37
*** annashen has joined #openstack-nova17:39
*** beagles_brb is now known as beagles17:40
*** jasondotstar|afk has quit IRC17:45
*** eharney has joined #openstack-nova17:45
*** yassine__ has quit IRC17:46
*** mwagner_lap has joined #openstack-nova17:47
*** achanda has joined #openstack-nova17:52
*** tonytan4ever has joined #openstack-nova17:56
*** danpb has quit IRC17:56
*** thumpba has quit IRC17:58
*** fawadkhaliq has quit IRC17:58
mriedemandrewbogott: if there is anything useful that could go into the policy.json as far as comments/docs, you could push a change for that17:59
*** cfriesen has joined #openstack-nova18:02
*** logan2 has quit IRC18:03
*** piyanai has quit IRC18:08
*** piyanai has joined #openstack-nova18:09
*** amotoki_ has quit IRC18:09
*** ijw has quit IRC18:17
*** achanda has quit IRC18:18
*** annashen has quit IRC18:19
*** Mike_D_laptop has quit IRC18:19
*** browne has quit IRC18:20
*** annashen has joined #openstack-nova18:20
*** mutoulbj has quit IRC18:24
*** annashen has quit IRC18:24
*** rushiagr is now known as rushiagr_away18:26
*** earlephilhower has quit IRC18:28
*** pbandzi_ has joined #openstack-nova18:31
*** pbandzi has quit IRC18:32
*** r-daneel has quit IRC18:32
*** zhenguo has quit IRC18:33
*** wuhg has quit IRC18:34
*** packet has joined #openstack-nova18:34
*** gokrokve has quit IRC18:35
openstackgerritDan Smith proposed openstack/nova: Convert test_compute_utils to use Instance object  https://review.openstack.org/19691218:36
openstackgerritDan Smith proposed openstack/nova: Refactor test_servers to use instance objects  https://review.openstack.org/19777018:36
openstackgerritDan Smith proposed openstack/nova: WIP: Remove legacy flavor compatibility code from Instance  https://review.openstack.org/19777118:36
openstackgerritDan Smith proposed openstack/nova: Decouple instance object tests from the api fakes module  https://review.openstack.org/19776918:36
openstackgerritDan Smith proposed openstack/nova: Remove flavor migration from db_api and nova-manage  https://review.openstack.org/19691118:36
*** yamahata has quit IRC18:38
*** Longgeek has quit IRC18:43
*** Longgeek has joined #openstack-nova18:43
*** Sukhdev has joined #openstack-nova18:48
*** r-daneel has joined #openstack-nova18:49
*** logan2 has joined #openstack-nova18:49
*** Marga_ has joined #openstack-nova18:52
*** ZZelle_ has joined #openstack-nova18:52
*** Longgeek has quit IRC18:54
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/19432518:56
*** sdake_ has joined #openstack-nova18:58
*** sdake has quit IRC19:00
*** sdake has joined #openstack-nova19:01
*** Longgeek has joined #openstack-nova19:02
*** ajayaa has joined #openstack-nova19:03
*** sdake_ has quit IRC19:04
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Add rootwrap daemon mode support  https://review.openstack.org/18069519:04
*** otter768 has joined #openstack-nova19:06
*** lyarwood has quit IRC19:07
*** lyarwood has joined #openstack-nova19:07
*** beagles is now known as beagles_biab19:09
*** otter768 has quit IRC19:10
*** ajayaa has quit IRC19:12
*** lyarwood has quit IRC19:12
*** lyarwood has joined #openstack-nova19:13
*** Longgeek has quit IRC19:14
*** marcusvrn_ has joined #openstack-nova19:14
*** lpetrut has quit IRC19:16
*** claudiub has joined #openstack-nova19:16
*** ijw has joined #openstack-nova19:17
*** marcusvrn_ is now known as marcusvrn19:17
*** annashen has joined #openstack-nova19:20
*** ijw has quit IRC19:21
openstackgerritKevin Fox proposed openstack/nova-specs: Instance Users for Cloud Interaction  https://review.openstack.org/18661719:24
*** annashen has quit IRC19:25
mriedemndipanov is going to hate me19:28
*** andrewbogott has left #openstack-nova19:29
*** lpetrut has joined #openstack-nova19:31
* dansmith hopes Rikki doesn't lose that number19:33
*** lpetrut has quit IRC19:34
mriedemdansmith: are you working tomorrow?19:34
dansmithnope19:34
mriedemfor a second i thought you forgot superdan19:35
dansmithheh19:35
dansmithI might also have to miss the meeting this afternoon19:35
dansmithdepending on when people arrive19:35
mriedemi also forgot there was a meeting19:35
mriedemb/c $crickets19:35
dansmithand because this is virtual friday19:35
*** angdraug has quit IRC19:37
*** ijw has joined #openstack-nova19:37
*** exploreshaifali has joined #openstack-nova19:47
*** igor___ has joined #openstack-nova19:48
*** yamahata has joined #openstack-nova19:48
* jaypipes finally getting to jroll's spec now... guh, another long day.19:48
mtreinishmriedem: but isn't today the meeting at the mikal and friends friendly time? They don't have independence day tomorrow19:48
mtreinishthey don't have an independence day at all...19:48
*** ociuhandu has quit IRC19:49
mriedemmtreinish: neither do we19:49
*** jasondotstar|afk has joined #openstack-nova19:49
mtreinishhaha, fair enough it's Sat.19:50
jrolljaypipes: I know the feeling, it's no rush at all :)19:50
*** jecarey has quit IRC19:50
dansmithsomeone referred to it as "hadn't noticed it was separatist terrorist day on Saturday"19:53
dansmithonce guess as to where that person hails from :P19:53
mtreinishhaha19:53
mriedemsouth carolina?19:54
dansmithI was going to take offense and then I realized it was pretty accurate19:54
mriedemtoo soon?19:54
dansmithheh19:54
dansmithyes, too soon19:54
*** mtanino has quit IRC19:54
*** Guest31 has joined #openstack-nova19:54
*** Marga_ has quit IRC19:56
*** pixelbeat has quit IRC19:56
*** claudiub|2 has joined #openstack-nova19:58
*** derekh has joined #openstack-nova19:58
*** claudiub has quit IRC19:59
*** piyanai has quit IRC20:00
*** packet has quit IRC20:01
*** ijw has quit IRC20:01
openstackgerritJesse J. Cook proposed openstack/nova: Cells: decrement available slots  https://review.openstack.org/19742520:01
*** piyanai has joined #openstack-nova20:05
*** boris-42 has quit IRC20:12
*** Marga_ has joined #openstack-nova20:13
*** thangp has quit IRC20:13
*** edmondsw has quit IRC20:14
*** Longgeek has joined #openstack-nova20:14
*** FL1SK has quit IRC20:16
*** browne has joined #openstack-nova20:16
*** hogepodge has quit IRC20:17
*** FL1SK has joined #openstack-nova20:18
*** hogepodge has joined #openstack-nova20:18
*** mtanino has joined #openstack-nova20:18
*** annashen has joined #openstack-nova20:21
*** thorst has quit IRC20:22
*** jasondotstar|afk has quit IRC20:22
*** lucas__ has quit IRC20:23
*** baoli has quit IRC20:23
*** Redcavalier has quit IRC20:25
*** annashen has quit IRC20:26
*** tjones1 has quit IRC20:32
*** tpatil has joined #openstack-nova20:33
*** jasondotstar|afk has joined #openstack-nova20:34
*** beagles_biab is now known as beagles20:35
*** tonytan4ever has quit IRC20:38
*** dprince has quit IRC20:40
mikalMorning20:43
dims_good morning mikal20:44
mriedemwe meet again20:44
*** melwitt_ has joined #openstack-nova20:45
*** tjones1 has joined #openstack-nova20:46
russellbdon't be hating on SC :-p20:46
mriedemha20:46
mriedemrussellb: you must be bored20:46
russellbi have to be bored to glance a -nova scrollback?20:47
russellbat.20:47
russellbbut yes, that's pretty accurate ... friends arriving very soon so can't really get into anything serious20:47
*** jwcroppe has joined #openstack-nova20:48
*** Sukhdev has quit IRC20:48
*** jskinner has quit IRC20:49
*** jwcroppe_ has quit IRC20:50
*** derekh has quit IRC20:51
*** jwcroppe_ has joined #openstack-nova20:51
*** jwcroppe has quit IRC20:53
*** jwcroppe has joined #openstack-nova20:54
*** jwcroppe_ has quit IRC20:56
*** Sukhdev has joined #openstack-nova20:56
*** jskinner has joined #openstack-nova20:56
mikalMeeting in a couple of minutes?20:57
mriedemyes20:57
dansmithmikal: many of us are just fake working today, so please keep it quick20:58
mikalAhhh, 4th of July, yes?20:58
mikalI hadn't thought of that20:58
dansmithyeah20:58
mikalDo you get an extra day off, or just the weekend?20:58
tonybBobBall: Thanks.  Good to know.20:58
dansmithmikal: we get tomorrow off20:59
dansmithmikal: to prepare to blow things up and grill animals20:59
mikalNice20:59
dansmithit's a very serious thing and preparation is important20:59
melwitt_I have today off too but figured I'd meet20:59
mikalCould you perhaps have exploding animals to save time?20:59
dansmithmelwitt_: wow, that's cool20:59
mriedemmikal: tomorrow is when most people need to drive out of town to some shady looking tent run by a shady looking guy to buy gobs of questionable fireworkds20:59
dansmithmikal: only in SC20:59
mikalmelwitt_: you do vacation... poorly20:59
mikalOk, let's do this thing20:59
*** dboik has quit IRC21:00
*** karmatronic has joined #openstack-nova21:01
*** karmatronic has quit IRC21:01
*** karimb has joined #openstack-nova21:01
*** rfolco has quit IRC21:02
*** jskinner has quit IRC21:02
*** melwitt_ has quit IRC21:03
*** annashen has joined #openstack-nova21:04
*** melwitt_ has joined #openstack-nova21:04
*** aysyd has quit IRC21:05
*** otter768 has joined #openstack-nova21:07
*** jwcroppe_ has joined #openstack-nova21:08
*** shhfeng has joined #openstack-nova21:08
*** annegentle has quit IRC21:08
*** jwcroppe has quit IRC21:09
*** ijw has joined #openstack-nova21:09
*** otter768 has quit IRC21:11
*** r-daneel has quit IRC21:11
shhfeng alex_xu,  ping21:12
*** ssurana has joined #openstack-nova21:14
*** vladikr has quit IRC21:15
*** ssurana has left #openstack-nova21:15
*** melwitt_ has quit IRC21:16
*** jasondotstar|afk has quit IRC21:17
*** Longgeek has quit IRC21:20
*** ddutta has joined #openstack-nova21:20
*** piyanai has quit IRC21:26
*** logan2 has quit IRC21:31
*** piyanai has joined #openstack-nova21:32
openstackgerritLianhao Lu proposed openstack/nova-specs: Add new API to list compute node metric names  https://review.openstack.org/18004921:32
bauzasoh man, missed the meeting21:34
*** mriedem has quit IRC21:36
claudiub|2just a quick request: the spec exception enable hyper-v NUMA, it already has a +2 from danpb who worked on implementing it in Juno, I would appreciate reviews. :D https://review.openstack.org/#/c/196162/21:36
jrolljaypipes: hey btw, do I need a spec freeze exception for this thing?21:38
jrolljaypipes: and should I be putting this spec on priorities etherpad or anything?21:38
* jroll keeps meaning to catch the nova meeting21:38
alex_xudansmith: sorry for bother you again in your fake working time. For the comment https://review.openstack.org/#/c/168982/13, "hw:cpu_features" is just for image to require few cpu feature.For your concern, the operator may need build the full feature list, that addressed by abstract cpu capabilites definition.21:39
*** piyanai has quit IRC21:39
alex_xuat line 170 of https://review.openstack.org/#/c/168982/13/specs/liberty/approved/cpu-model-per-flavor-or-image.rst21:39
*** piyanai has joined #openstack-nova21:39
alex_xuyou can specifid full feature list in that mapping21:39
dansmithalex_xu: +1 for "fake working time" :)21:40
dansmithalex_xu: I just replied21:40
dansmithalex_xu: I get what you're trying to say, I just don't think this is a good interface for it21:40
*** Marga_ has quit IRC21:40
alex_xudansmith: heh, sorry for that :) I have to think about how to save my spec21:40
*** piyanai has quit IRC21:40
dansmithalex_xu: I know, but I just don't think this has anywhere near the agreement it needs to be excepted at this point21:40
dansmithmaybe it does and I'm missing it, but I've not seen it21:41
bauzasmmm, for my information, is July 3rd an holiday because 4th is on Sat?21:41
dansmithbauzas: yes21:41
alex_xudansmith: from the comment, jay, daniel and you all think the abstract cpu capability is right21:41
bauzasdansmith: okay good to know21:41
alex_xudansmith: emm...and there isn't sub-team can help on this, confuse on how to push this spec21:42
* bauzas is adding his name for being pinged for the meeting21:42
dansmithalex_xu: the abstract cpu model in general is the way to go, yes, but what you have is very intel-specific21:43
dansmithor rather, x86-specific21:43
alex_xudansmith: I don't think it's intel-specific, how to translsate the mapping is virt driver thing, like power platform, there isn't features, they can just use the cpu_model field21:44
alex_xudansmith: or we can go to the way, the mapping field have different format for different hypervisor, but I'm sure that is right way21:45
dansmithno, that's terrible :)21:45
alex_xus/i'm  sure/i'm not sure/...21:45
dansmithI knew what you meant :)21:45
alex_xuI always forget type 'not'...21:46
dansmithso, I put my summary on the thing a bit ago. Even if the cpu modeling was right, I still haven't looked at the object/data model/rpc impacts to this, nor the scheduler stuff21:46
dansmithso there's no way _I_ am comfortable with this going forward in the short term21:46
dansmithif there are others that are, then that's cool21:46
dansmithbut I'm not21:47
alex_xudansmith: ok, I see, anyway thanks for the feedback21:47
dansmiththe data model, rpc and scheduler bits are important too21:47
dansmithalex_xu: yep, I appreciate your persistence and sorry to be a "bad guy"21:47
alex_xudansmith: I don't want to push people too hard...but looks like if I won't, then I still can't get enough review...21:48
dansmithalex_xu: yeah, some pushing is required.. not faulting you for pushing21:49
dansmithalex_xu: it doesn't seem like there is enough interest in this to get sufficient review to me21:49
*** boris-42 has joined #openstack-nova21:49
openstackgerritAdam Kacmarsky proposed openstack/nova: Disable router advertisements on bridge devices  https://review.openstack.org/19805421:49
dansmithalex_xu: it's been two weeks since anyone commented, and that was my comment21:50
alex_xudansmith: emm...but I heard of we said this is the feature we want in the summit21:50
alex_xudansmith: yea, I really need think about how to push similar thing next time21:50
dansmith?21:50
*** ctrath has quit IRC21:51
dansmithalex_xu: well, really, I'm not the only person21:51
*** emagana has quit IRC21:51
alex_xuI mean I may to change another way, email, push in the nova meeting, get more boardly review, then good for get agreement21:51
*** claudiub|2 has quit IRC21:52
dansmithwhat is boardly review?21:52
dansmithoh, broadly?21:52
alex_xuoops21:52
dansmith"boardly" could be a word, with a very different meaning, so I was confused :D21:53
dansmithI mean, it has some meaning21:53
dansmithbut broadly -- yes :)21:53
*** Piet has quit IRC21:53
alex_xuyea...to hurry to type word21:54
dansmithno problem21:54
*** logan2 has joined #openstack-nova21:56
*** ijw has quit IRC21:56
*** derekh has joined #openstack-nova21:57
*** jkugel has left #openstack-nova21:58
*** marzif_ has joined #openstack-nova21:58
*** burt has quit IRC21:59
tjones1folks i've been rebasing this baby since april if anyone has time to take a look https://review.openstack.org/#/c/127427/22:01
*** ianbrown has joined #openstack-nova22:01
*** Kennan2 has joined #openstack-nova22:03
*** Kennan has quit IRC22:03
*** r-daneel has joined #openstack-nova22:04
*** rwsu has quit IRC22:05
*** annegentle has joined #openstack-nova22:07
openstackgerritVictor Stinner proposed openstack/nova: Fix Python 3 issues in nova.utils and nova.tests  https://review.openstack.org/19798222:08
openstackgerritVictor Stinner proposed openstack/nova: Port crypto to Python 3  https://review.openstack.org/19774822:08
*** claudiub has joined #openstack-nova22:09
*** marzif_ has quit IRC22:10
*** ijw has joined #openstack-nova22:14
*** pixelbeat has joined #openstack-nova22:15
*** eharney has quit IRC22:15
*** Longgeek has joined #openstack-nova22:17
*** tpeoples has quit IRC22:17
*** sdake has quit IRC22:22
*** sdake has joined #openstack-nova22:22
*** artom has quit IRC22:22
*** ijw_ has joined #openstack-nova22:22
tpatilalaski: Thank you for reviewing https://review.openstack.org/#/c/135387/13. We will fix all your comments soon.22:23
*** annegentle has quit IRC22:24
*** annegentle has joined #openstack-nova22:24
*** Longgeek has quit IRC22:25
*** ijw has quit IRC22:26
*** ociuhandu has joined #openstack-nova22:27
*** mmedvede has quit IRC22:28
*** exploreshaifali has quit IRC22:29
*** annegentle has quit IRC22:30
dansmithtjones1: the commit message mentions several cases that were impossible before the refactor, but no tests were added to show that those work...?22:33
tjones1ahhhh….  ummm…  ok let me look at the tests22:35
*** ijw_ has quit IRC22:35
*** ijw has joined #openstack-nova22:36
tjones1ugh thought i was done but you have a good point22:36
*** annashen has quit IRC22:36
*** briancurtin has quit IRC22:36
tjones1back to the drawing board22:36
dansmithtjones1: well, I was trying to figure out which things were supposed to be fixed, and was like "I know, I'll go look at the tests"22:38
dansmithand then.. :D22:38
* dansmith goes to pick up visitors22:38
*** briancurtin has joined #openstack-nova22:39
tjones1dansmith: have fun!22:39
*** ssurana has joined #openstack-nova22:40
*** claudiub has quit IRC22:40
*** erlon has quit IRC22:41
*** sdake has quit IRC22:43
*** Guest31 has quit IRC22:43
*** pixelbeat has quit IRC22:43
*** raorn has quit IRC22:43
*** erlon has joined #openstack-nova22:44
*** raorn has joined #openstack-nova22:44
*** sdake has joined #openstack-nova22:44
*** dboik has joined #openstack-nova22:46
*** sdake has quit IRC22:46
*** neiljerram_bb has quit IRC22:47
*** sdake has joined #openstack-nova22:50
*** dboik has quit IRC22:50
*** ijw has quit IRC22:51
*** sdake has quit IRC22:51
*** sdake has joined #openstack-nova22:52
*** ijw has joined #openstack-nova22:53
*** annashen has joined #openstack-nova22:53
*** Sukhdev has quit IRC22:56
*** sdake has quit IRC22:58
*** mtanino has quit IRC22:59
*** erlon has quit IRC23:00
*** tsekiyama has quit IRC23:01
*** tsekiyama has joined #openstack-nova23:01
*** annashen has quit IRC23:02
*** ijw has quit IRC23:04
*** otter768 has joined #openstack-nova23:07
*** ijw has joined #openstack-nova23:07
*** ijw has quit IRC23:08
*** r-daneel has quit IRC23:09
*** derekh has quit IRC23:11
*** otter768 has quit IRC23:12
*** mtanino has joined #openstack-nova23:13
*** ddutta has quit IRC23:15
*** tpatil has quit IRC23:16
*** radek_ has quit IRC23:17
*** sorrison_laptop has joined #openstack-nova23:19
openstackgerritMichael Still proposed openstack/nova: Cleanup log lines in nova.image.glance  https://review.openstack.org/19691423:20
openstackgerritMichael Still proposed openstack/nova: log meaningful error message on download exception  https://review.openstack.org/16403923:20
*** raildo_ has joined #openstack-nova23:20
*** ZZelle_ has quit IRC23:23
*** Sukhdev has joined #openstack-nova23:27
*** ddutta has joined #openstack-nova23:29
*** alexpilotti has joined #openstack-nova23:32
*** zzzeek has quit IRC23:33
*** IanGovett has quit IRC23:35
*** StevenK has quit IRC23:35
*** sdake has joined #openstack-nova23:37
*** sdake has left #openstack-nova23:38
*** alexpilotti has quit IRC23:38
*** zzzeek has joined #openstack-nova23:42
*** alexpilotti has joined #openstack-nova23:44
*** alexpilotti has quit IRC23:46
*** jskinner has joined #openstack-nova23:47
*** zhenguo has joined #openstack-nova23:48
*** browne has quit IRC23:51
*** ddutta has quit IRC23:59

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