Tuesday, 2015-08-04

*** haomaiwa_ has quit IRC00:01
*** edleafe has quit IRC00:01
*** haomaiwang has joined #openstack-nova00:02
*** edleafe has joined #openstack-nova00:03
*** MentalRay has quit IRC00:04
*** Vikas has joined #openstack-nova00:05
*** achanda has quit IRC00:06
VikasHi folks..I am a beginner and soon after i commited changes for review,https://review.openstack.org/#/c/208680/, got "-1" from "Citrix XenServer CI".Can anybody please suggest what is this about and what should be my next step?00:06
clarkbVikas: thats is a "third party CI" system run by people that care about nova working with xenserver. They should post logs that allow you to look and see why it failed00:07
*** pixelbeat_ has joined #openstack-nova00:07
VikasThanks Clark00:11
*** gzhai has joined #openstack-nova00:16
*** vladikr has quit IRC00:18
*** jaypipes has quit IRC00:18
gzhaisdague: Could you pls. look at my new patch? https://review.openstack.org/#/c/203573/  Thanks a lot!00:18
prometheanfireanish: well 'iscsiadm -m discovery -t st -p $cinder_vol_ip -P 1' shows the volume and lsscsi shows it attached but nova isn't starting the instance00:21
prometheanfireseems like iscsi isn't showing up in /dev/disk/by-path as nova expects00:25
*** otter768 has joined #openstack-nova00:26
*** dannywilson has quit IRC00:26
*** tsekiyam_ has joined #openstack-nova00:30
*** salv-orlando has quit IRC00:31
*** tsekiyama has quit IRC00:34
*** annashen has joined #openstack-nova00:34
anishprometheanfire: iscsiadm -m session00:35
anishwhat does that show00:35
*** haigang has joined #openstack-nova00:35
prometheanfireanish: tcp: [5] ip:3260,1 iqn.2010-10.org.openstack:volume-6313aaf5-e37e-48df-9c3f-659f0bf589f500:38
prometheanfireanish: it does show up in lsscsi00:38
prometheanfirejust not in /dev/disk/by-path00:38
prometheanfireneed to figure that part out00:38
*** annashen has quit IRC00:39
prometheanfireanish: I'm guessing I'm missing a udev rule?00:39
anishsomething like that00:39
prometheanfirerunning udev 216 atm00:40
anishone way is to check if sysfs entries are present correctly00:41
*** tpeoples has joined #openstack-nova00:42
*** mdrabe has quit IRC00:44
*** mdrabe has joined #openstack-nova00:44
*** edmondsw has quit IRC00:45
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/20870600:47
prometheanfireanish: how would sysfs set a symlink?00:48
prometheanfireanish: it does attach           [13:0:0:1]   disk    IET      VIRTUAL-DISK     0001  /dev/sdf00:49
prometheanfirethe symlink just isn't created for some reason00:49
*** mtanino has quit IRC00:51
*** kjelly has joined #openstack-nova00:52
*** erlon has quit IRC00:53
gmannsdague: alaski : Please have look on clarifying more about server lock on lock spec - https://review.openstack.org/#/c/208326/00:55
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/20870600:56
anishprometheanfire: sorry, that goes into areas I'm not very familiar with, but my money is on udev00:59
*** haomaiwang has quit IRC01:01
prometheanfireanish: can't find anything special with udev unfortunately01:01
openstackgerritTakashi NATSUME proposed openstack/nova: Fix typos of 'address', 'current' and 'concurrent'  https://review.openstack.org/20874901:01
*** haomaiwang has joined #openstack-nova01:02
*** leeantho has quit IRC01:05
*** shyu has joined #openstack-nova01:07
prometheanfireanish: can you give me the output of lsscsi for an attached volume?01:10
prometheanfireanish: https://github.com/openstack/nova/blob/stable/kilo/nova/virt/libvirt/volume.py makes me think it's looking in /dev/mapper too01:11
openstackgerritZhenyu Zheng proposed openstack/nova: Allow resize root_gb to 0 for volume-backed instances  https://review.openstack.org/20779401:11
*** praneshp has joined #openstack-nova01:11
openstackgerritZhenyu Zheng proposed openstack/nova: Add instance action events for live migration  https://review.openstack.org/19537201:12
*** exploreshaifali has quit IRC01:12
*** browne has quit IRC01:12
openstackgerritZhenyu Zheng proposed openstack/nova: CONF.allow_resize_to_same_host should check only once in controller  https://review.openstack.org/18623601:13
openstackgerritZhenyu Zheng proposed openstack/nova: network_request_obj: Clean up outdated code  https://review.openstack.org/20142501:14
anishprometheanfire: sure, but lsscsi output won't help you much01:20
anishprometheanfire: http://ix.io/k5z01:21
*** ijw has joined #openstack-nova01:21
*** lyanchih_ has joined #openstack-nova01:22
prometheanfireanish: ya :(01:23
*** ijw_ has quit IRC01:23
prometheanfireanish: where in /sys did you want me to look?01:23
prometheanfireI can see it's stuff there (while it's trying to attach01:24
*** angdraug has quit IRC01:25
*** baoli has joined #openstack-nova01:25
*** haigang has quit IRC01:26
*** zhenguo has joined #openstack-nova01:27
*** haigang has joined #openstack-nova01:27
prometheanfirewellnow, this gives me hope...01:28
prometheanfirehttp://askubuntu.com/questions/579619/device-attached-over-iscsi-not-found-at-dev-disk-by-path01:28
anishprometheanfire: verify that /sys/block is full populated01:28
anishbut this is a gentoo-udev issue01:28
anishnot really an openstack problem01:29
anishyou might get better help in #gentoo01:29
*** hshiina has joined #openstack-nova01:29
prometheanfireI'm a gentoo dev, might be a udev issue though01:29
prometheanfirething is, the udev rules for /dev/disk/by-path are the same on both01:30
anishall you need is a rule that works on /sys/class/iscsi_host/host${HOST}/device/session*/iscsi_session*/targetname01:32
anishyou could write your own01:32
*** SourabhP has quit IRC01:32
*** kiwik-chenrui has joined #openstack-nova01:32
*** salv-orlando has joined #openstack-nova01:33
prometheanfireya, thinking of doing that now01:33
*** tsekiyam_ has quit IRC01:33
*** ijw has quit IRC01:34
*** tsekiyama has joined #openstack-nova01:34
*** tsekiyama has quit IRC01:34
*** MentalRay has joined #openstack-nova01:34
prometheanfireanish: what's ${HOST} suposed to be, for me it just incriments01:34
anishprometheanfire: it's now part of systemd-udevd.service I believe, so can't find the actual rule for you01:34
anishprometheanfire: the scsi host allocated by open-iscsi01:34
*** tsekiyama has joined #openstack-nova01:34
prometheanfireok01:34
anishgenerally in the form hostN01:34
prometheanfire/sys/class/iscsi_host/host14/device/session9/iscsi_session/session9/targetname01:35
prometheanfireya01:35
prometheanfirewas missing an extra session*01:35
*** park_heijlong has quit IRC01:35
*** RuiChen has quit IRC01:35
*** wanghua has joined #openstack-nova01:37
prometheanfireanish: the ip in the device name, is that suposed to be the client or the server ip?01:38
*** tsekiyama has quit IRC01:39
anishprometheanfire: target ip01:40
prometheanfireanish: that means the guest right (the nova host)?01:41
aniship-10.192.194.163:3261-iscsi-iqn.2004-05.com.chelsio.target-lun-0 , lun 0 for target iqn.2004-05.com.chelsio.target running on 10.192.194.163 at port 3261 with tcp transport01:41
*** salv-orlando has quit IRC01:41
anishno, that means cinder host01:41
prometheanfireok01:42
*** thorst has quit IRC01:42
prometheanfiretrying to find where in /sys to pull that from01:42
*** tobe_ has joined #openstack-nova01:46
openstackgerritRui Chen proposed openstack/nova: Adds instance_uuid index for instance_system_metadata  https://review.openstack.org/20827701:46
*** pixelbeat_ has quit IRC01:46
*** park_heijlong has joined #openstack-nova01:48
*** otter768 has quit IRC01:49
*** READ10 has joined #openstack-nova01:49
*** lyanchih_ has quit IRC01:51
*** obondarev_ has quit IRC01:54
*** tdurakov has quit IRC01:54
*** pkholkin has quit IRC01:54
*** kjelly has quit IRC01:55
*** lyanchih_ has joined #openstack-nova01:58
*** haomaiwang has quit IRC02:01
*** haomaiwang has joined #openstack-nova02:02
*** baoli has quit IRC02:05
*** baoli has joined #openstack-nova02:06
*** yamahata has quit IRC02:07
*** zzzeek has quit IRC02:12
prometheanfireanish: I can't find iscsi rules anywhere on any distro02:13
*** rushiagr_away is now known as rushiagr02:13
*** praneshp has quit IRC02:20
*** praneshp has joined #openstack-nova02:21
anishprometheanfire: I'm guessing you need to look at something like rhel5 maybe, by-path has been part of udev for a while now02:23
prometheanfireI know02:25
prometheanfireI've been looking everwhere02:25
prometheanfirethis shouldn't be this hard to find02:26
*** Vikas_ has quit IRC02:31
*** lyanchih_ has quit IRC02:33
prometheanfireanish: sorry to keep bothering you, but that's an example of a valid path?02:33
*** annashen has joined #openstack-nova02:35
anishprometheanfire: there are a couple in the nova test cases02:36
prometheanfire/dev/disk/by-path/ip-10.192.194.163:3261-iscsi-iqn.2010-10.org.openstack:volume-6313aaf5-e37e-48df-9c3f-659f0bf589f5 I think02:36
anishyes02:36
anishit's missing the lun at the end02:37
*** tjones has quit IRC02:37
prometheanfirestaticly -lun-0?02:37
anish/dev/disk/by-path/ip-10.192.194.163:3261-iscsi-iqn.2004-05.com.chelsio.target-lun-002:37
anishno02:37
anishstatically -lun-X02:37
*** unicell1 has quit IRC02:37
prometheanfirewhere X is whatever I want it to be?02:37
anishX increments according to the number of luns your target is exporting02:38
*** annashen has quit IRC02:40
prometheanfireguess that's gonna be this then02:40
prometheanfire/sys/class/iscsi_host/host*/device/session*/target*/*/block/${DISK}/../../../../iscsi_session/session*/target_id02:40
*** salv-orlando has joined #openstack-nova02:40
anishprometheanfire: here https://review.openstack.org/#/c/193451/12/os_brick/tests/initiator/test_linuxscsi.py02:41
prometheanfirethanks02:41
*** browne has joined #openstack-nova02:41
*** achanda has joined #openstack-nova02:45
prometheanfirelooks like kilo expects this https://github.com/openstack/nova/blob/7ba9da9605aa0a16945e95462e95f89b113bdd09/nova/tests/unit/virt/libvirt/volume/test_volume.py#L30302:46
prometheanfirebah, that's master02:46
*** BobBall_ is now known as BobBall02:46
prometheanfirehttps://github.com/openstack/nova/blob/stable/kilo/nova/tests/unit/virt/libvirt/test_volume.py#L30802:46
*** salv-orlando has quit IRC02:49
*** vilobhmm has joined #openstack-nova02:51
openstackgerritShuquan Huang proposed openstack/nova: nova.utils._get_root_helper() should be public  https://review.openstack.org/20736602:51
*** kiran-r has joined #openstack-nova02:52
*** hakimo has joined #openstack-nova02:52
*** tobe_ has quit IRC02:54
*** dims has joined #openstack-nova02:54
*** hakimo_ has quit IRC02:55
*** bkopilov has quit IRC02:58
*** haomaiwang has quit IRC03:01
*** haomaiwang has joined #openstack-nova03:02
*** tojuvone has joined #openstack-nova03:03
*** READ10 has quit IRC03:05
*** rushiagr is now known as rushiagr_away03:06
*** lpetrut has joined #openstack-nova03:07
*** gzhai has quit IRC03:10
*** eliqiao has joined #openstack-nova03:13
*** eliqiao has left #openstack-nova03:13
openstackgerritvikas choudhary proposed openstack/nova: [WIP]Update hypervisor resources correctly in booting failed case  https://review.openstack.org/20868003:14
*** MentalRay has quit IRC03:14
*** dims has quit IRC03:15
*** ftersin has joined #openstack-nova03:17
*** vilobhmm has quit IRC03:19
*** tjones has joined #openstack-nova03:20
*** rushiagr_away is now known as rushiagr03:22
*** eliqiao has joined #openstack-nova03:29
*** MentalRay has joined #openstack-nova03:34
*** rfolco has quit IRC03:34
*** achanda has quit IRC03:37
*** eliqiao1 has joined #openstack-nova03:37
*** eliqiao has quit IRC03:39
openstackgerritvikas choudhary proposed openstack/nova: [WIP]Update hypervisor resources correctly in booting failed case  https://review.openstack.org/20868003:39
*** Sukhdev has joined #openstack-nova03:41
*** eglynn_ has joined #openstack-nova03:42
*** coolsvap|away is now known as coolsvap03:43
*** Vikas has quit IRC03:46
*** tjones has quit IRC03:46
*** eglynn_ has quit IRC03:47
*** kumar has joined #openstack-nova03:48
*** cfriesen has joined #openstack-nova03:48
*** otter768 has joined #openstack-nova03:50
*** Vikas has joined #openstack-nova03:50
*** salv-orlando has joined #openstack-nova03:50
prometheanfireanish: you know of any way to get the device mappings iscsi is using (figuring out what target is /dev/sdFOO)?03:50
prometheanfirecat't find a way to do it in sys at all03:50
prometheanfirenone of the /sys/class/iscsi_* does it03:50
*** mdrabe has quit IRC03:51
*** rushiagr is now known as rushiagr_away03:51
*** afazekas has joined #openstack-nova03:54
*** otter768 has quit IRC03:55
*** salv-orl_ has joined #openstack-nova03:57
*** salv-orlando has quit IRC03:57
*** haomaiwang has quit IRC04:01
*** salv-orl_ has quit IRC04:01
*** haomaiwang has joined #openstack-nova04:02
*** rotbeard has quit IRC04:04
prometheanfireah, lsiscsi gives it much better then munging in /sys04:04
*** unicell has joined #openstack-nova04:08
*** Vikas has quit IRC04:09
*** sc68cal has quit IRC04:10
*** deepakcs has joined #openstack-nova04:13
*** vivekd has joined #openstack-nova04:14
*** _kiran_ has joined #openstack-nova04:16
*** kiran-r has quit IRC04:18
*** sc68cal has joined #openstack-nova04:21
*** sc68cal has quit IRC04:21
*** sc68cal has joined #openstack-nova04:21
*** achanda has joined #openstack-nova04:21
*** haigang has quit IRC04:22
*** haigang has joined #openstack-nova04:25
*** kiran-r has joined #openstack-nova04:25
*** garyk has quit IRC04:27
*** _kiran_ has quit IRC04:29
*** jdandrea has quit IRC04:30
*** deepakcs has quit IRC04:33
*** achanda_ has joined #openstack-nova04:35
*** achanda has quit IRC04:35
*** annashen has joined #openstack-nova04:36
*** shyama has joined #openstack-nova04:36
*** bkopilov has joined #openstack-nova04:38
*** annashen has quit IRC04:41
*** gzhai has joined #openstack-nova04:42
*** gzhai has quit IRC04:43
*** oomichi has joined #openstack-nova04:45
openstackgerritShiina, Hironori proposed openstack/nova: libvirt: improve error handling of injecting an NMI  https://review.openstack.org/20879404:47
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Fix root device name for volume-backed instances  https://review.openstack.org/20879604:48
*** rushiagr_away is now known as rushiagr04:48
*** kiran-r has quit IRC04:48
openstackgerritShuquan Huang proposed openstack/nova: nova.utils._get_root_helper() should be public  https://review.openstack.org/20736604:58
*** lpetrut has quit IRC04:58
*** MentalRay has quit IRC04:59
*** salv-orlando has joined #openstack-nova05:00
*** kumar has quit IRC05:00
*** haomaiwang has quit IRC05:01
*** 32NAAAL0S has joined #openstack-nova05:02
*** haigang has quit IRC05:04
*** meghal has joined #openstack-nova05:06
*** meghal1 has joined #openstack-nova05:07
openstackgerritShiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI  https://review.openstack.org/20261705:08
openstackgerritShiina, Hironori proposed openstack/nova-specs: Fix spec: Inject NMI to an instance  https://review.openstack.org/20700405:09
*** changbl has joined #openstack-nova05:09
*** meghal has quit IRC05:10
*** salv-orlando has quit IRC05:13
openstackgerritShiina, Hironori proposed openstack/nova: Add a compute RPC API to inject an NMI  https://review.openstack.org/20260505:13
openstackgerritShiina, Hironori proposed openstack/nova: libvirt: improve error handling of injecting an NMI  https://review.openstack.org/20879405:13
*** fawadkhaliq has joined #openstack-nova05:14
*** ishant has joined #openstack-nova05:14
*** fawadk has joined #openstack-nova05:15
*** rotbeard has joined #openstack-nova05:17
*** tjones has joined #openstack-nova05:17
*** fawadkhaliq has quit IRC05:18
*** irenab has joined #openstack-nova05:20
prometheanfirethis is so fucking annoying05:22
*** ishant_ has joined #openstack-nova05:26
*** sc68cal has quit IRC05:26
*** garyk has joined #openstack-nova05:27
*** vivekd has quit IRC05:27
*** sc68cal has joined #openstack-nova05:29
*** ishant has quit IRC05:29
*** Nikolay_St has quit IRC05:33
*** neelabh__ has joined #openstack-nova05:35
*** tobe_ has joined #openstack-nova05:36
neelabh__ I want to fetch all instance through rest-api, please tell me the Rest-api which can call the all tenant instance..05:37
*** chhavi has joined #openstack-nova05:38
*** Sukhdev has quit IRC05:41
*** josecastroleon has joined #openstack-nova05:43
*** sudipto has joined #openstack-nova05:44
*** ig0r_ has joined #openstack-nova05:46
*** zzzeek has joined #openstack-nova05:48
*** tjones has quit IRC05:48
*** haigang has joined #openstack-nova05:50
*** afazekas has quit IRC05:50
*** tobe_ has quit IRC05:50
*** meghal1 has quit IRC05:51
*** otter768 has joined #openstack-nova05:51
*** vivekd has joined #openstack-nova05:51
*** tobe_ has joined #openstack-nova05:53
*** kmARC has joined #openstack-nova05:53
*** otter768 has quit IRC05:55
*** Murali has joined #openstack-nova05:55
*** deepakcs has joined #openstack-nova05:57
*** tobe_ has quit IRC05:58
*** kiran-r has joined #openstack-nova06:00
*** ig0r_ has quit IRC06:04
*** salv-orlando has joined #openstack-nova06:10
*** unicell1 has joined #openstack-nova06:10
*** ig0r_ has joined #openstack-nova06:11
openstackgerritgaryk proposed openstack/nova: Metadata: support proxying loadbalancers  https://review.openstack.org/14753906:12
openstackgerritgaryk proposed openstack/nova: Neutron: add in API method for updating VNIC index  https://review.openstack.org/14712606:12
*** unicell has quit IRC06:12
*** deepakcs has quit IRC06:13
openstackgerritgaryk proposed openstack/nova: VMware: improve instance names on VC  https://review.openstack.org/16660806:13
openstackgerritgaryk proposed openstack/nova: VMware: add in folder support on VC  https://review.openstack.org/16506006:13
*** abhishekk has joined #openstack-nova06:17
*** kiran-r has quit IRC06:17
*** salv-orlando has quit IRC06:21
openstackgerritShiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI  https://review.openstack.org/20261706:22
*** salv-orlando has joined #openstack-nova06:22
openstackgerritRen Qiaowei proposed openstack/nova: Remove unnecessary executable permission  https://review.openstack.org/20237606:22
*** kmARC has quit IRC06:25
*** deepakcs has joined #openstack-nova06:26
*** vilobhmm has joined #openstack-nova06:28
*** Nikolay_St has joined #openstack-nova06:29
*** praneshp has quit IRC06:29
*** vilobhmm1 has joined #openstack-nova06:30
*** park_heijlong_ has joined #openstack-nova06:31
*** belmoreira has joined #openstack-nova06:31
*** vilobhmm has quit IRC06:33
*** SourabhP has joined #openstack-nova06:34
*** takedakn has joined #openstack-nova06:34
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex  https://review.openstack.org/19721306:35
*** annashen has joined #openstack-nova06:37
openstackgerritZachary Sais proposed openstack/nova: Remove v3 from nova API code tree  https://review.openstack.org/19372506:41
*** annashen has quit IRC06:41
openstackgerritShiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI  https://review.openstack.org/20261706:44
openstackgerritShiina, Hironori proposed openstack/nova: Add a compute RPC API to inject an NMI  https://review.openstack.org/20260506:44
openstackgerritShiina, Hironori proposed openstack/nova: libvirt: improve error handling of injecting an NMI  https://review.openstack.org/20879406:44
*** Marga_ has joined #openstack-nova06:46
*** Marga_ has quit IRC06:46
*** Marga_ has joined #openstack-nova06:47
*** humble_ has joined #openstack-nova06:51
*** atuvenie has joined #openstack-nova06:52
*** rdopiera has joined #openstack-nova06:53
*** wanghua has quit IRC06:55
*** josecastroleon has quit IRC06:56
*** humble__ has joined #openstack-nova06:57
*** yuntong has quit IRC06:59
*** ankit_ag has joined #openstack-nova06:59
*** shuquan has joined #openstack-nova07:00
*** humble_ has quit IRC07:00
*** josecastroleon has joined #openstack-nova07:00
*** jlanoux has joined #openstack-nova07:00
*** achanda_ has quit IRC07:01
*** h00327910__ has joined #openstack-nova07:01
*** josecastroleon has quit IRC07:02
*** sahid has joined #openstack-nova07:04
*** sahid has quit IRC07:04
*** sahid has joined #openstack-nova07:07
*** tdurakov has joined #openstack-nova07:08
*** pkholkin has joined #openstack-nova07:08
*** jcoufal has joined #openstack-nova07:09
*** ijw has joined #openstack-nova07:09
openstackgerritSudipta Biswas proposed openstack/nova: Remove 'scheduled_at' - DB cleanup  https://review.openstack.org/19276007:10
*** alex_klimov has joined #openstack-nova07:11
*** fawadkhaliq has joined #openstack-nova07:12
deepakcsjohnthetubaguy: hi john, I notice my bp was moved from approved to pending approval (https://blueprints.launchpad.net/nova/+spec/volume-snapshot-improvements)07:13
deepakcsjohnthetubaguy: is there any background for that ?07:13
deepakcsjohnthetubaguy: the patches for that bp are already in review for quite some time now.07:13
*** scheuran has joined #openstack-nova07:14
*** sudipto has quit IRC07:14
*** fawadk has quit IRC07:14
*** josecastroleon has joined #openstack-nova07:15
*** fawadk has joined #openstack-nova07:15
*** ftersin has quit IRC07:16
prometheanfireanish: finally got it working, I hate udev so much https://gist.github.com/prometheanfire/f5d7dd35712366c79cdf07:16
prometheanfireanish: thanks for the help07:16
*** baoli has quit IRC07:17
*** Marga_ has quit IRC07:17
*** tpeoples has quit IRC07:17
*** baoli has joined #openstack-nova07:18
*** jyuso1 has joined #openstack-nova07:18
*** fawadkhaliq has quit IRC07:18
*** markus_z has joined #openstack-nova07:21
*** baoli has quit IRC07:22
*** vivekd has quit IRC07:24
*** Kevin_Zheng_ has joined #openstack-nova07:25
openstackgerritRoman Dobosz proposed openstack/nova: New nova API call to mark nova-compute down  https://review.openstack.org/18408607:27
*** moshele_ has joined #openstack-nova07:31
*** alex_klimov has quit IRC07:33
*** humble_ has joined #openstack-nova07:33
*** vilobhmm1 has quit IRC07:33
*** alex_klimov has joined #openstack-nova07:33
openstackgerritShiina, Hironori proposed openstack/nova: Add a compute RPC API to inject an NMI  https://review.openstack.org/20260507:36
*** browne has quit IRC07:36
*** apuimedo has joined #openstack-nova07:37
*** kodoku has joined #openstack-nova07:37
*** humble__ has quit IRC07:37
*** kmARC has joined #openstack-nova07:38
anishprometheanfire: glad you got it working07:40
anishbut lun-1 is wrong07:40
flip214does each compute node need its own n-api instance, or is a common one enough?07:40
anishfirst lun would be lun-007:40
anishthen lun-107:40
anishso on and so forth07:40
kodokuHi, Is it possible to add "key=value" to an image and GET this value with meta-data service ?07:40
*** lsmola has joined #openstack-nova07:41
prometheanfireanish: it's what nova was looking for07:43
prometheanfireanish: the uuid changes for each volume, so oddly, the lun is always lun-107:43
*** shuquan_ has joined #openstack-nova07:43
*** armax has quit IRC07:44
openstackgerritliaonanhai proposed openstack/nova: Add user_id info: add 'user_id' to REST os-simple-tenant-usage output  https://review.openstack.org/15152207:45
*** fawadk has quit IRC07:45
*** shuquan has quit IRC07:45
*** shuquan_ is now known as shuquan07:45
*** vivekd has joined #openstack-nova07:46
*** Marga_ has joined #openstack-nova07:46
*** cfriesen has quit IRC07:47
*** lucasagomes has joined #openstack-nova07:48
prometheanfireanish: anyway, sleep now, I'm open to patches of course :P07:49
*** otter768 has joined #openstack-nova07:51
*** chhavi has quit IRC07:52
*** moshele_ has quit IRC07:53
*** moshele has joined #openstack-nova07:53
openstackgerritSergey Nikitin proposed openstack/nova: Fixed incorrect behaviour of method _check_instance_exists  https://review.openstack.org/20303707:54
openstackgerritBartosz Fic proposed openstack/nova: libvirt: Rollback on dest host after deleting vm during live migration  https://review.openstack.org/18595807:55
*** otter768 has quit IRC07:56
flip214can somebody chime in in the discussion at https://review.openstack.org/#/c/206024/, please?07:56
*** fawadkhaliq has joined #openstack-nova07:57
*** yassine has joined #openstack-nova08:01
*** yassine has quit IRC08:01
*** yassine has joined #openstack-nova08:03
*** fawadkhaliq has quit IRC08:06
openstackgerritAlex Xu proposed openstack/python-novaclient: Add version discover and check in CLI  https://review.openstack.org/18428108:07
openstackgerritAlex Xu proposed openstack/python-novaclient: Set "latest" as default compute api version  https://review.openstack.org/18465808:07
*** jistr has joined #openstack-nova08:08
*** shuquan has quit IRC08:11
*** romainh has joined #openstack-nova08:11
*** SourabhP has quit IRC08:11
*** jcoufal has quit IRC08:14
*** shuquan has joined #openstack-nova08:14
*** tobe_ has joined #openstack-nova08:17
*** aix has joined #openstack-nova08:18
openstackgerritShiina, Hironori proposed openstack/nova: libvirt: improve error handling of injecting an NMI  https://review.openstack.org/20879408:25
*** Marga_ has quit IRC08:25
*** Marga_ has joined #openstack-nova08:26
*** danpb has joined #openstack-nova08:29
*** ijw has quit IRC08:29
*** Marga_ has quit IRC08:32
*** sayali has joined #openstack-nova08:33
*** mriedem_away has quit IRC08:35
*** pedroalvarez has quit IRC08:35
*** jwitko has quit IRC08:35
*** ByPasS has quit IRC08:35
*** hogepodge has quit IRC08:35
*** ekhugen has quit IRC08:35
*** ttx has quit IRC08:35
*** mirrorbox has quit IRC08:35
*** tteggel_ has quit IRC08:35
*** _fortis has quit IRC08:35
*** wuhao has quit IRC08:35
*** Madasi has quit IRC08:35
*** devananda has quit IRC08:35
*** rmstar has quit IRC08:35
*** Marga_ has joined #openstack-nova08:37
*** rgerganov has joined #openstack-nova08:37
*** mriedem_away has joined #openstack-nova08:37
*** pedroalvarez has joined #openstack-nova08:37
*** jwitko has joined #openstack-nova08:37
*** ByPasS has joined #openstack-nova08:37
*** hogepodge has joined #openstack-nova08:37
*** ekhugen has joined #openstack-nova08:37
*** ttx has joined #openstack-nova08:37
*** mirrorbox has joined #openstack-nova08:37
*** tteggel_ has joined #openstack-nova08:37
*** _fortis has joined #openstack-nova08:37
*** wuhao has joined #openstack-nova08:37
*** Madasi has joined #openstack-nova08:37
*** devananda has joined #openstack-nova08:37
*** rmstar has joined #openstack-nova08:37
*** annashen has joined #openstack-nova08:38
openstackgerritBartosz Fic proposed openstack/nova: libvirt: Rollback on dest host after deleting vm during live migration  https://review.openstack.org/18595808:38
openstackgerritAla Rezmerita proposed openstack/nova-specs: Inheritable admin image properties  https://review.openstack.org/20643108:40
mosheledanpb: can you review again https://review.openstack.org/#/c/187052/ I had to rebase it,  and also in  https://review.openstack.org/#/c/199488/  I addressed all your comments08:41
*** annashen has quit IRC08:42
*** Marga_ has quit IRC08:42
*** sudipto has joined #openstack-nova08:43
kodokuWhere I can found attributs of instance class ? I want to modify base.py in metadata-api file.08:44
openstackgerritAdelina Tuvenie proposed openstack/nova: Moves OVS related code to a common location  https://review.openstack.org/17972708:46
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V OVS ViF driver  https://review.openstack.org/14004508:46
*** andreykurilin has joined #openstack-nova08:49
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V vNic hot-plug implementation  https://review.openstack.org/13825508:49
tdurakovhi, nova-cores, please review this patch: https://review.openstack.org/#/c/208253/ It fixes high bug that causes jenkins, py-27 check to fail08:49
*** alexpilotti has quit IRC08:51
*** rotbeard has quit IRC08:52
openstackgerritAdelina Tuvenie proposed openstack/nova: Adds Hyper-V imagecache cleanup  https://review.openstack.org/19261808:52
*** alexpilotti has joined #openstack-nova08:52
*** jcoufal has joined #openstack-nova08:54
*** MarcF has joined #openstack-nova08:54
*** gszasz has joined #openstack-nova08:55
*** humble__ has joined #openstack-nova08:55
*** jcoufal has quit IRC08:55
*** jcoufal has joined #openstack-nova08:56
openstackgerritsahid proposed openstack/nova: libvirt: add os_admin_user to use with set admin password  https://review.openstack.org/18592108:56
openstackgerritsahid proposed openstack/nova: libvirt: enhance libvirt to set admin password  https://review.openstack.org/18591008:56
openstackgerritsahid proposed openstack/nova: libvirt: rework quiesce to not share "sensitive" informations  https://review.openstack.org/19662008:56
garykjohnthetubaguy: melwitt: i have addressed the comments from https://review.openstack.org/#/c/147126/ and the patch above.08:57
*** humble_ has quit IRC08:58
*** humble_ has joined #openstack-nova08:59
sudiptokodoku, i am joining in late. do you want to know the instance object attributes?09:01
*** kmARC has quit IRC09:01
*** kodoku_ has joined #openstack-nova09:01
kodoku_sudipto yes e.g https://review.openstack.org/#/c/197185/10/nova/api/metadata/base.py09:02
kodoku_I want to add image_id09:02
*** Jeffrey4l has quit IRC09:02
*** humble__ has quit IRC09:02
kodoku_Is it possible to add like metadata['image_id'] = self.instance.image_id   ?09:02
kodoku_or instance.image.image_id09:02
kodoku_I don't know :)09:02
*** kodoku has quit IRC09:03
sudiptokodoku_, did you take a look at nova/objects/instance.py?09:04
kodoku_yes I am reading it now ^^09:04
sudiptokodoku_, don't think you have a image_id in there. There's something called a 'image_ref'09:04
kodoku_I see image_ref09:04
kodoku_yes09:04
kodoku_I'll try with that09:04
*** jcoufal has quit IRC09:07
kodoku_sudipto works :)09:08
kodoku_thx09:08
sudiptokodoku_, np :)09:10
*** jcoufal has joined #openstack-nova09:11
openstackgerritJordan Pittier proposed openstack/nova: libvirt:on snapshot delete, use qemu-img to blockRebase if VM is stopped  https://review.openstack.org/19273609:12
*** kmARC has joined #openstack-nova09:16
*** marzif_ has joined #openstack-nova09:18
deepakcsjohnthetubaguy: req for FFE @ http://lists.openstack.org/pipermail/openstack-dev/2015-August/071297.html - hope you will agree :)09:22
johnthetubaguygaryk: hey, just looking at those patches09:22
johnthetubaguydeepakcs: the process is to put that on the etherpad, I hope that helps: https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule#Non-priority_Feature_Freeze09:23
*** ftersin has joined #openstack-nova09:24
deepakcsjohnthetubaguy: yeah, i just updated it.. thanks09:26
openstackgerritFeodor Tersin proposed openstack/nova: Fix launch from volume-backed image  https://review.openstack.org/20893209:26
ftersindanpb: ^^^^^^^^^09:26
ftersindanpb: is it right way to fix this?09:28
openstackgerritRui Chen proposed openstack/nova: Fix resource leaking when consume_from_instance raise exception  https://review.openstack.org/18216509:28
openstackgerritRui Chen proposed openstack/nova: Set task_state=None when booting instance failed  https://review.openstack.org/18714509:28
openstackgerritRui Chen proposed openstack/nova: Fix permission issue of server group API  https://review.openstack.org/19311709:28
openstackgerritRui Chen proposed openstack/nova: Fix incomplete error message of quota exceeded  https://review.openstack.org/19746009:28
*** boris-42 has quit IRC09:30
openstackgerritShiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI  https://review.openstack.org/20261709:31
*** hanzhang has joined #openstack-nova09:31
*** claudiub has joined #openstack-nova09:31
*** fawadkhaliq has joined #openstack-nova09:31
lxsliPlease could https://review.openstack.org/#/c/202640/ be re-approved after its rebase? Thanks09:33
claudiubjohnthetubaguy: hi. this previously had your +2. Just changed the folder location and that's it. Could you take a look? https://review.openstack.org/#/c/190997/09:37
*** sudipto has quit IRC09:37
johnthetubaguygaryk: got a few comments on the metadata stuff, the other bits look much better now, thank you09:39
johnthetubaguyclaudiub: sorry, I am concentrating on code reviews for the next month or so, we did say we need to fix up flavors/image props urgently, I would want to be sure there is no clash there for mitaka09:41
openstackgerritSergey Nikitin proposed openstack/nova: Added server tags support in nova-api  https://review.openstack.org/12894009:42
openstackgerritSergey Nikitin proposed openstack/nova: Fixed incorrect behaviour of method _check_instance_exists  https://review.openstack.org/20303709:42
*** marzif_ has quit IRC09:43
openstackgerritShiina, Hironori proposed openstack/nova-specs: Fix spec: Inject NMI to an instance  https://review.openstack.org/20700409:43
*** takedakn1 has joined #openstack-nova09:43
*** marzif_ has joined #openstack-nova09:43
*** MarcF has quit IRC09:44
*** neelabh__ has quit IRC09:46
*** takedakn has quit IRC09:47
*** Venkatesh has joined #openstack-nova09:52
*** otter768 has joined #openstack-nova09:52
VenkateshHi, I am new to nova development09:53
VenkateshCan someone can tell, how to start09:53
Venkatesh?09:53
*** openstackgerrit_ has joined #openstack-nova09:53
*** sudipto has joined #openstack-nova09:53
lxsliVenkatesh: hi! Have you tried running Devstack?09:54
VenkateshYes, I installed devstack,09:54
VenkateshI made some contributions to cloudkitty on stackforge09:54
*** dims has joined #openstack-nova09:55
lxsliOK - my next suggestion is to fix one of these: https://bugs.launchpad.net/nova/+bugs?field.tag=low-hanging-fruit09:55
VenkateshCan I directly understand these bugs, without code understanding of nova?09:55
lxsliVenkatesh: reading code to understand and fix a bug is a great way to gain understanding of Nova09:56
VenkateshOk09:56
*** MarcF has joined #openstack-nova09:57
lxsliThere is a little bit of overview doc I'm looking up for you09:57
VenkateshI have to take TRIAGED bugs right09:57
*** otter768 has quit IRC09:57
lxsliThat's best09:58
openstackgerritEn proposed openstack/nova: formely is not correct  https://review.openstack.org/20894809:58
*** jcoufal has quit IRC09:59
lxsliThere seems to only be one triaged bug though so you can also look at confirmed09:59
VenkateshOk, I will check ..09:59
lxsliI'd stay away from the Wishlist ones too, they're likely to need a blueprint and specification09:59
*** jcoufal has joined #openstack-nova09:59
VenkateshThanks lxsli10:01
lxsliaha! http://docs.openstack.org/developer/nova/devref/10:02
*** neelabh__ has joined #openstack-nova10:02
*** shuquan has quit IRC10:02
lxsliPeople are trying to update that at the moment so it may not be fully accurate but as a new contributor it should still be very useful10:02
*** takedakn1 has quit IRC10:03
VenkateshOk10:05
*** dheeraj-gupta-4 has joined #openstack-nova10:05
*** romainh has quit IRC10:06
*** kmARC has quit IRC10:10
openstackgerritShiina, Hironori proposed openstack/nova: Add a compute RPC API to inject an NMI  https://review.openstack.org/20260510:10
*** jcoufal has quit IRC10:10
*** lxsli has quit IRC10:10
*** lxsli has joined #openstack-nova10:11
*** shyu has quit IRC10:12
*** bjornar has quit IRC10:14
*** haigang has quit IRC10:14
*** moshele has quit IRC10:16
*** chhavi has joined #openstack-nova10:16
*** Marga_ has joined #openstack-nova10:17
VenkateshHi lxsli: confirmed bugs are already assigned to some developers10:20
*** Marga_ has quit IRC10:20
*** Marga_ has joined #openstack-nova10:20
openstackgerritKyrylo Romanenko proposed openstack/python-novaclient: Add method for better random name  https://review.openstack.org/20782910:21
*** moshele has joined #openstack-nova10:21
*** IanGovett has joined #openstack-nova10:21
*** pixelbeat_ has joined #openstack-nova10:22
*** kodoku_ has quit IRC10:24
lxsliVenkatesh: as I understand it, a bug should be "In progress" if it's being worked on10:24
Venkatesh I am also having same opinion, but some confirmed bugs have assignee10:26
lxsliCheck the comments, see if the assignee seems active10:26
lxsliEG https://bugs.launchpad.net/nova/+bug/1385489 is assigned but the assignee's patch got abandoned so he can't be that interested10:26
openstackLaunchpad bug 1385489 in OpenStack Compute (nova) "ResourceTracker._update_usage_from_migration() is inefficient due to multiple Instance.get_by_uuid() lookups" [Medium,Confirmed] - Assigned to Geronimo Orozco (gorozco)10:26
Venkateshlxsli :ok, thanks10:28
openstackgerritShiina, Hironori proposed openstack/nova: libvirt: improve error handling of injecting an NMI  https://review.openstack.org/20879410:30
lxsliVenkatesh: may I ask, are you an individual contributor or employed to work on OpenStack?10:30
ftersindims: hi, are you around?10:30
*** Marga_ has quit IRC10:31
Venkateshlxsli : I am individual contributor10:31
dimsftersin: pong10:31
garykjohnthetubaguy: ok, thanks. i'll take a look10:31
lxsliVenkatesh: ok, welcome in then :)10:31
johnthetubaguygaryk: there looked like there was some unreachable code there, the rest was just nits, let me know if that was just a missread10:32
Venkateshlxsli : Thanks10:32
ftersindims: could we join ours https://review.openstack.org/#/c/207679/ and https://review.openstack.org/#/c/208932/?10:32
*** zzzeek has quit IRC10:33
garykjohnthetubaguy: ok. np. going over it now. i think that with all of the changes you are correct10:34
ftersindims: correct link for the second review: https://review.openstack.org/#/c/208932/10:34
*** fawadk has joined #openstack-nova10:35
dimsftersin: i can drop mine10:35
dimsno worries10:35
openstackgerritSergey Nikitin proposed openstack/nova: Added server tags support in nova-api  https://review.openstack.org/12894010:35
openstackgerritSergey Nikitin proposed openstack/nova: Fixed incorrect behaviour of method _check_instance_exists  https://review.openstack.org/20303710:35
dimsftersin: done10:36
*** fawadkhaliq has quit IRC10:37
ftersindims: then i'll add 'closes' tag with your bug and fix relationship map10:38
dimsftersin: ack10:38
*** fawadkhaliq has joined #openstack-nova10:38
*** annashen has joined #openstack-nova10:38
openstackgerritFeodor Tersin proposed openstack/nova: Fix launch from volume-backed image  https://review.openstack.org/20893210:40
*** claudiub has quit IRC10:40
*** claudiub has joined #openstack-nova10:41
openstackgerritShiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI  https://review.openstack.org/20261710:41
openstackgerritSergey Nikitin proposed openstack/nova: Added server tags support in nova-api  https://review.openstack.org/12894010:42
openstackgerritSergey Nikitin proposed openstack/nova: Fixed incorrect behaviour of method _check_instance_exists  https://review.openstack.org/20303710:42
ftersindims: done as well, thanks10:42
*** fawadk has quit IRC10:43
*** annashen has quit IRC10:43
garykjohnthetubaguy: sahid: i addressed your comments in https://review.openstack.org/18020210:44
*** oomichi has quit IRC10:45
*** IanGovett has quit IRC10:46
*** IanGovett has joined #openstack-nova10:46
*** moshele has quit IRC10:51
*** jcoufal has joined #openstack-nova10:52
*** dheeraj-gupta-4 has left #openstack-nova10:53
*** sahid has quit IRC10:54
openstackgerritMatthew Booth proposed openstack/nova: Use original image size in volume image metadata  https://review.openstack.org/20852310:55
openstackgerritgaryk proposed openstack/nova: Metadata: support proxying loadbalancers  https://review.openstack.org/14753910:55
garykjohnthetubaguy: ^10:56
*** eglynn_ has joined #openstack-nova10:59
*** lucasagomes is now known as lucas-hungry11:01
*** park_heijlong_ has quit IRC11:01
*** ftersin has quit IRC11:01
*** apuimedo is now known as apuimedo|lunch11:02
*** deepakcs has quit IRC11:03
*** shihanzhang has joined #openstack-nova11:04
*** jlanoux has quit IRC11:05
openstackgerritFei Long Wang proposed openstack/nova: Fix nova backup for volume-backed instance  https://review.openstack.org/16449411:07
*** hanzhang has quit IRC11:07
openstackgerritMatthew Booth proposed openstack/nova: Don't check flavor disk size when booting from volume  https://review.openstack.org/20463511:11
openstackgerritMatthew Booth proposed openstack/nova: compute: improve exceptions related to disk size checks  https://review.openstack.org/20656511:11
*** flwang1 has joined #openstack-nova11:13
*** tojuvone has quit IRC11:14
*** coolsvap is now known as coolsvap|away11:14
openstackgerritRadoslav Gerganov proposed openstack/nova: Enable getting connection info for MKS consoles  https://review.openstack.org/20899811:16
*** smatzek has joined #openstack-nova11:22
*** Marga_ has joined #openstack-nova11:25
*** thorst has joined #openstack-nova11:27
openstackgerritSimona Iuliana Toader proposed openstack/nova: Hyper-V: Adds host maintenance implementation  https://review.openstack.org/20415711:27
*** oomichi has joined #openstack-nova11:31
*** openstackgerrit has quit IRC11:31
*** openstackgerrit_ is now known as openstackgerrit11:31
*** openstackgerrit has quit IRC11:32
*** openstackgerrit_ has joined #openstack-nova11:32
*** openstackgerrit_ is now known as openstackgerrit11:32
*** openstackgerrit_ has joined #openstack-nova11:33
*** rubasov has quit IRC11:37
openstackgerritMatthew Booth proposed openstack/nova: Don't check flavor disk size when booting from volume  https://review.openstack.org/20463511:38
openstackgerritMatthew Booth proposed openstack/nova: compute: improve exceptions related to disk size checks  https://review.openstack.org/20656511:38
*** dave-mccowan has joined #openstack-nova11:38
*** dprince has joined #openstack-nova11:39
*** Venkatesh has quit IRC11:40
johnthetubaguydanpb: is there a good time to get you and jaypipes (and vladik?) in a video conf to talk about this patch: https://review.openstack.org/#/c/149625/11:42
*** eglynn_ has quit IRC11:43
*** eglynn__ has joined #openstack-nova11:44
*** bkopilov has quit IRC11:48
openstackgerritRadoslav Gerganov proposed openstack/nova: Enable getting connection info for MKS consoles  https://review.openstack.org/20899811:48
*** fawadkhaliq has quit IRC11:48
*** jlanoux has joined #openstack-nova11:48
*** sudipto has quit IRC11:49
*** romainh has joined #openstack-nova11:50
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Allow display project-id for server groups  https://review.openstack.org/20901111:50
*** mwagner_lap has quit IRC11:50
*** moshele has joined #openstack-nova11:53
moshelejohnthetubaguy: hi11:53
*** otter768 has joined #openstack-nova11:53
openstackgerritKyrylo Romanenko proposed openstack/python-novaclient: Update pick_image function  https://review.openstack.org/20749711:54
*** Marga_ has quit IRC11:54
flip214could someone please chime in at https://review.openstack.org/#/c/206024/ and tell us whether n-api needs to run on _every_ compute node or not?11:55
moshelejohnthetubaguy:  can you also review https://review.openstack.org/#/c/187052/ it had +2 from danpb before rebase11:55
*** haigang has joined #openstack-nova11:56
*** tobe_ has quit IRC11:56
*** sahid has joined #openstack-nova11:57
*** otter768 has quit IRC11:58
danpbjohnthetubaguy: i guess more or less any time this week during normal uk hours except 3-4pm wed/fri11:58
*** haigang has quit IRC11:59
*** tobe_ has joined #openstack-nova11:59
mosheledanpb: can you review again https://review.openstack.org/#/c/187052/ ?11:59
*** jaypipes has joined #openstack-nova11:59
*** tobe_ has quit IRC12:01
*** rubasov has joined #openstack-nova12:01
openstackgerritZhenyu Zheng proposed openstack/nova: Add project-id when list server-groups  https://review.openstack.org/20901812:01
openstackgerritRui Chen proposed openstack/nova: Adds instance_uuid index for instance_system_metadata  https://review.openstack.org/20827712:02
*** garyk1 has joined #openstack-nova12:03
*** lpetrut has joined #openstack-nova12:03
oomichisdague: hi12:05
*** lucas-hungry is now known as lucasagomes12:05
oomichisdague: can we talk about https://review.openstack.org/189551 a bit now?12:05
*** garyk has quit IRC12:07
*** tojuvone has joined #openstack-nova12:11
*** Marga_ has joined #openstack-nova12:11
sdagueoomichi: sure12:14
oomichisdague: thanks, I wrote some comment on the patch.12:14
*** raildo has joined #openstack-nova12:15
oomichisdague: I'm not suere "API doc improvement" of your comment.12:15
*** kmARC has joined #openstack-nova12:15
oomichisdague: what does it mean?12:15
*** moshele has quit IRC12:15
oomichisdague: http://developer.openstack.org/api-ref.html ?12:15
*** openstackgerrit has quit IRC12:16
*** openstackgerrit has joined #openstack-nova12:17
*** moshele has joined #openstack-nova12:17
sdagueso with v2.0 we had this concept guide - https://github.com/openstack/nova/blob/57967ae3515a015e94657b4a10bcd6ef8d3dd439/doc/source/v2/2.0_server_concepts.rst12:17
sdaguewhich is basically what you need to use the API unless you are going to read the nova code12:17
sdagueit's woefully out of date12:17
sdaguethis isn't about magical autogeneration, we need really manual documentation around our API as well12:18
*** busterswt has joined #openstack-nova12:20
oomichisdague: to be honest, I know it now.12:20
oomichisdague: is that used for http://developer.openstack.org/api-ref.html also ?12:21
oomichisdague: or we are managing them separately?12:21
sdagueI'm actually not sure where api-ref is coming from atm12:21
sdagueanyway, I really really feel like we're missing a coherent approach to having reasonable documentation on our API. And I'm not keen on merging new major bits like json home until we've got a plan here.12:22
*** fawadkhaliq has joined #openstack-nova12:23
sahidgaryk1: about https://review.openstack.org/#/c/180202/ - it seems the patch is in conflict12:23
oomichisdague: http://developer.openstack.org/api-ref.html also contain bugs and I fixed them sometimes.12:23
oomichiand I found the other api doc now in our repo :-(12:23
tojuvonemark-host-down would be looking +2 to have all code merged to liberty: https://review.openstack.org/184086/12:24
*** VW_ has joined #openstack-nova12:24
*** rushiagr is now known as rushiagr_away12:24
oomichi"we're missing a coherent approach to having reasonable documentation on our API." <- yes, I also feel it.12:25
*** marcusvrn has joined #openstack-nova12:26
*** edmondsw has joined #openstack-nova12:27
sdagueoomichi: ok, so lets work on building that plan first instead of pushing big features like this. Where should we try to map that out?12:28
*** Vikas_ has joined #openstack-nova12:29
oomichisdague: "the first step is planning" is nice.12:29
*** cgoncalves has joined #openstack-nova12:30
oomichisdague: I feel most people see http://developer.openstack.org/api-ref.html instead12:30
oomichisdague: so how about removing doc/source/v2/2.0_server_concepts.rst ?12:30
*** garyk has joined #openstack-nova12:31
*** aysyd has joined #openstack-nova12:31
oomichisdague: it is better to concentrate on major api docs12:31
*** garyk1 has quit IRC12:31
garyksahid: thanks. i'll rebase - for the nth time :)12:31
sahid:)12:31
*** baoli has joined #openstack-nova12:32
*** rpodolyaka1 has joined #openstack-nova12:32
sdagueoomichi: I feel like we need the concept guide somewhere12:33
sdagueif there is a place for that in the developer.openstack.org docs, that's fine, but we need it12:33
*** baoli has quit IRC12:34
sdagueoomichi: maybe we take this out to the mailing list and try to sort out a plan there12:34
*** bapalm_ has joined #openstack-nova12:34
*** baoli has joined #openstack-nova12:34
oomichisdague: ok, the concept guide is fine, but I feel current 2.0_server_concepts.rst contains the details also. and it make out of date easily12:35
sdagueoomichi: honestly, keeping the documentation up to date needs to be part of the whole process of adding / changing the API. I feel like deleting the content because people haven't kept it up to date is the wrong approach12:36
tojuvonejohnthetubaguy: mark-host-down would be looking +2 to have all code merged to liberty: https://review.openstack.org/184086/12:37
sdagueif people need to read nova code to use the API, we've failed12:37
sdagueand that's the state we're largely in right now12:37
*** shuquan has joined #openstack-nova12:38
oomichisdague: that is right. but current way of API patch already make a single big patch, like including code changes, unit tests, functional tests, microversion history, *.json, *.json.tpl, and something.12:39
*** annashen has joined #openstack-nova12:39
*** rfolco has joined #openstack-nova12:40
oomichisdague: more changes of file will make review burdan more hard.12:40
johnthetubaguyoomichi: sdague: I think that concepts bit is possible the most important part of the API docs, in some ways, we could put a *this is not up to date* warning, but deleting it seems bad12:40
sdaguejohnthetubaguy: agree, I'm starting a mailing list thread.12:40
johnthetubaguysdague: cool12:41
johnthetubaguyoomichi: sdague: honestly, thats the bit I read when I try to find out what a new openstack API actually does12:41
sdagueoomichi: and I do understand the fact that API changes come in this giant patch and we need to come up with a better way to deal with that12:41
sdagueI think that's another related bit12:41
oomichisdague: thanks for starting, I will join it later.12:42
johnthetubaguydanpb: OK, cool, could to reach out to jaypipes to try and arrange something? I think sdague was also interested, might want to pick a time, then shoot it out on the ML?12:42
oomichisdague, johnthetubaguy: I need to go home now, so I'd like to discuss this on the ml. see you :-)12:43
openstackgerritShiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI  https://review.openstack.org/20261712:43
openstackgerritsahid proposed openstack/nova: libvirt: add os_admin_user to use with set admin password  https://review.openstack.org/18592112:44
openstackgerritsahid proposed openstack/nova: libvirt: enhance libvirt to set admin password  https://review.openstack.org/18591012:44
openstackgerritsahid proposed openstack/nova: libvirt: rework quiesce to not share "sensitive" informations  https://review.openstack.org/19662012:44
*** annashen has quit IRC12:44
*** shuquan has quit IRC12:45
*** oomichi has quit IRC12:45
*** mdrabe has joined #openstack-nova12:48
openstackgerritAlexis Lee proposed openstack/nova: Prevent (un)pinning unknown CPUs  https://review.openstack.org/20704012:48
*** ishant_ has quit IRC12:49
*** claudiub has quit IRC12:50
*** hshiina has quit IRC12:51
openstackgerritKyrylo Romanenko proposed openstack/python-novaclient: Add method for better random name  https://review.openstack.org/20782912:52
*** VW_ has quit IRC12:52
*** alexpilotti_ has joined #openstack-nova12:52
*** eliqiao1 has quit IRC12:52
*** alexpilotti has quit IRC12:52
*** alexpilotti_ is now known as alexpilotti12:52
*** VW_ has joined #openstack-nova12:52
*** zul has joined #openstack-nova12:53
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Allow display project-id for server groups  https://review.openstack.org/20901112:53
*** zul has quit IRC12:53
*** zul has joined #openstack-nova12:53
*** VW__ has joined #openstack-nova12:54
*** VW_ has quit IRC12:54
*** dustins has joined #openstack-nova12:55
*** VW__ has quit IRC12:56
*** ildikov has joined #openstack-nova12:56
*** VW_ has joined #openstack-nova12:56
*** annegentle has joined #openstack-nova12:56
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Allow display project-id for server groups  https://review.openstack.org/20901112:57
*** jaypipes has quit IRC12:57
openstackgerritgaryk proposed openstack/nova: VMware: add in vif resource limitations  https://review.openstack.org/18020213:00
*** jaypipes has joined #openstack-nova13:00
garykjohnthetubaguy: sahid: ^ rebased13:00
*** shuquan has joined #openstack-nova13:00
*** VW_ has quit IRC13:00
*** dims_ has joined #openstack-nova13:00
openstackgerritgaryk proposed openstack/nova: VMware: support limits via image meta data  https://review.openstack.org/18094213:01
openstackgerritgaryk proposed openstack/nova: Objects: add limits, reservations and shares to ImageMetaProps object  https://review.openstack.org/18832113:01
*** jdandrea has joined #openstack-nova13:02
*** dims has quit IRC13:02
openstackgerritRajesh Tailor proposed openstack/nova: Fix quota-update when nova-compute startup finish  https://review.openstack.org/17011813:03
*** dims has joined #openstack-nova13:04
*** shuquan has quit IRC13:05
*** dims_ has quit IRC13:06
*** moshele has quit IRC13:06
*** rlrossit has joined #openstack-nova13:06
*** coolsvap|away is now known as coolsvap13:06
*** vivekd has quit IRC13:07
*** moshele has joined #openstack-nova13:07
*** browne has joined #openstack-nova13:07
*** Marga_ has quit IRC13:08
*** kylek3h has joined #openstack-nova13:10
*** lpetrut has quit IRC13:10
sdaguemriedem_away: hmmm... I'm looking at this nova test that's failing everything, and it's a bear to figure out13:11
*** kmARC has quit IRC13:12
*** diogogmt has quit IRC13:12
*** rushiagr_away is now known as rushiagr13:13
*** jgriffith has quit IRC13:14
*** mriedem_away is now known as mriedem13:14
mriedemsdague: no more failing tests13:14
*** erlon has joined #openstack-nova13:15
mriedemover quota13:15
sdagueso, assertJsonEqual is sorting a list of complex structures13:15
sdagueand some times, it flips them13:15
mriedemdid you see lifeless' suggestions on the patch you had last week?13:15
sdaguehttps://jenkins01.openstack.org/job/gate-nova-python27/1457/console13:15
*** Vikas__ has joined #openstack-nova13:15
sdaguemriedem: yep, I'm not convinced that's going to change this13:16
*** diogogmt has joined #openstack-nova13:17
*** marzif_ has quit IRC13:18
*** xyang has quit IRC13:18
danpbsdague: did you see the patch for that this morning13:18
*** marzif_ has joined #openstack-nova13:18
danpbsdague:  https://review.openstack.org/#/c/208253/13:18
openstackgerritSergey Nikitin proposed openstack/nova: Fixed random failing of test_describe_instances_with_filters_tags  https://review.openstack.org/20825313:22
*** nihilifer has quit IRC13:23
*** coolsvap is now known as coolsvap_13:23
*** kmARC has joined #openstack-nova13:24
openstackgerritAlexis Lee proposed openstack/nova: Raise NovaException for missing/empty machine-id  https://review.openstack.org/20264013:24
*** rdopiera has quit IRC13:25
*** mjturek1 has joined #openstack-nova13:25
*** READ10 has joined #openstack-nova13:26
*** ig0r_ has quit IRC13:27
*** shuquan has joined #openstack-nova13:27
sdaguedanpb: nope, I did a search earlier for any patches13:28
openstackgerritvikas choudhary proposed openstack/nova: [WIP]Update hypervisor resources correctly in booting failed case  https://review.openstack.org/20868013:29
*** ig0r_ has joined #openstack-nova13:30
*** browne has quit IRC13:30
sdagueI had forgotten about the hash seed randomization, that would explain it. Because if str function moves the instanceset to the front of the list, the sort order would flip13:31
*** VW_ has joined #openstack-nova13:31
*** ijuwang has joined #openstack-nova13:31
kashyapdanpb: You intend to rebase this patch soon?  I'm setting up a new DevStack live migration env, might as well give this a spin -- https://review.openstack.org/#/c/162254/ -- "libvirt: set caps on maximum live migration time"13:32
sdaguedanpb: the reason that lists are sorted is because json doesn't have a set notion, and we serialize sets quite often13:32
*** abhishekk has quit IRC13:34
*** dims_ has joined #openstack-nova13:36
*** nelsnelson has joined #openstack-nova13:36
*** nelsnelson has quit IRC13:37
*** matrohon has joined #openstack-nova13:37
*** nelsnelson has joined #openstack-nova13:37
*** dims has quit IRC13:37
*** mestery_afk is now known as mestery13:38
*** tongli has joined #openstack-nova13:38
openstackgerritSergey Nikitin proposed openstack/nova: Added server tags support in nova-api  https://review.openstack.org/12894013:38
openstackgerritSergey Nikitin proposed openstack/nova: Fixed incorrect behaviour of method _check_instance_exists  https://review.openstack.org/20303713:38
*** bkopilov has joined #openstack-nova13:39
*** lpetrut has joined #openstack-nova13:40
*** snikitin has joined #openstack-nova13:41
*** humble__ has joined #openstack-nova13:42
*** diogogmt has quit IRC13:42
*** humble_ has quit IRC13:43
*** thumpba has joined #openstack-nova13:43
*** tpeoples has joined #openstack-nova13:44
*** ferest has joined #openstack-nova13:45
*** lpetrut has quit IRC13:47
*** lucas___ has joined #openstack-nova13:48
*** dboik has joined #openstack-nova13:49
*** mwagner_lap has joined #openstack-nova13:49
*** ctrath has joined #openstack-nova13:50
*** dboik has quit IRC13:50
*** ankit_ag has quit IRC13:50
*** dboik has joined #openstack-nova13:51
*** Marga_ has joined #openstack-nova13:51
*** burt has joined #openstack-nova13:52
*** thangp has joined #openstack-nova13:52
*** otter768 has joined #openstack-nova13:54
*** claudiub has joined #openstack-nova13:54
*** changbl has quit IRC13:56
ccesariohello... does someone know when this code https://blueprints.launchpad.net/openstack/?searchtext=vmware-better-display-names  will be approved ? This is very very important  for VMware admins13:56
*** otter768 has quit IRC13:58
openstackgerritMarian Horban proposed openstack/nova: Added processing /compute URL  https://review.openstack.org/19530313:58
*** ferest has quit IRC13:59
*** mhorban has joined #openstack-nova13:59
*** ig0r_ has quit IRC14:00
*** Marga_ has quit IRC14:01
*** Marga_ has joined #openstack-nova14:02
*** ijw has joined #openstack-nova14:04
*** jgriffith has joined #openstack-nova14:04
claudiubjohnthetubaguy: hi. I have a question. Why was this unapproved? The code was done since Kilo, but did not get enough attention. https://blueprints.launchpad.net/nova/+spec/hyperv-serial-ports14:05
mriedemccesario: the code didn't make the feature freeze cutoff14:06
*** 32NAAAL0S has quit IRC14:06
mriedemccesario: an FFE could be requested on the mailing list14:06
*** haomaiwang has joined #openstack-nova14:07
*** shuquan has quit IRC14:08
*** changbl has joined #openstack-nova14:09
*** mtanino has joined #openstack-nova14:09
*** tonytan4ever has joined #openstack-nova14:12
ccesariomriedem, hmmmm, the author ( garyk ) should have done it ?14:14
*** ndipanov has joined #openstack-nova14:15
mriedemccesario: done which part? the code or FFE?14:15
*** eglynn__ has quit IRC14:15
mriedemthe code has been up for quite awhile14:15
mriedemthe problem was lack of review14:15
*** jcoufal has quit IRC14:16
*** ferest has joined #openstack-nova14:18
openstackgerritShiina, Hironori proposed openstack/nova: Add a compute RPC API to inject an NMI  https://review.openstack.org/20260514:18
ccesariomriedem, and who can do this review ? eheheh ..14:18
mriedemccesario: i spent plenty of time on it, have you reviewed it?14:19
mriedemi was asking the vmware dev team to review it also last week because up until then none of them had14:19
mriedemsorry but i don't have much sympathy for people complaining about features not making it after the cutoff when they haven't helped with the review of the change14:19
openstackgerritShiina, Hironori proposed openstack/nova: libvirt: improve error handling of injecting an NMI  https://review.openstack.org/20879414:20
openstackgerritdane-fichter proposed openstack/nova: libvirt: Raise error when booting from encrypted volumes  https://review.openstack.org/20378414:20
*** salv-orlando has quit IRC14:20
*** ferest has quit IRC14:21
ccesariomriedem, I 'm trying test it in this momment :)14:21
*** Marga_ has quit IRC14:21
*** kjelly_ has joined #openstack-nova14:21
mriedemok, the results of that would be helpful as feedback in the change when you're done14:21
mriedemi personally don't have a vcenter cloud setup to manually test that stuff14:22
*** lmtaylor has joined #openstack-nova14:23
ccesariomriedem, give me a minutes that devstack is finishing it14:23
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/20870614:25
garykmriedem: you also started to review that stuff a few months ago …14:25
garykmriedem: most of the team has reviewed it, just expecting that with each rebase is a bit of an ak..14:26
mriedemgaryk: yeah and then i had concerns about backwards compat issues14:26
*** Piet has quit IRC14:26
garykmriedem: i explained ont hreview that there are no backward compatibility issues14:26
mriedemi don't remember other vmware devs reviewing it until i was specifically asking for that last week14:26
mriedemyeah, no one from vmware reviewed it until 7/3014:26
garykif that is the case i will chase them down to review it, will it matter? seriously?14:27
*** salv-orlando has joined #openstack-nova14:27
garyknow we need to wait another 3 months14:28
garykwe are stringling nova to death14:28
garykstrangling14:28
*** gszasz has quit IRC14:28
*** cfriesen has joined #openstack-nova14:29
*** armax has joined #openstack-nova14:29
mriedemit matters to me that the people that actually get paid to maintain the vmware driver do put in some review time on features going into it, yes14:30
*** apuimedo|lunch is now known as apuimedo14:31
garykmriedem: so i will ping you when they have reviewed and will you look then?14:31
mriedemwell, isn't that kind of the idea behind the subteam stuff posted to the priorities etherpad?14:31
*** gszasz has joined #openstack-nova14:32
garykwe posted and no one looked for months so we stopped14:32
mriedemassuming the inter-team reviews are not just a pile of +1s14:32
*** diogogmt has joined #openstack-nova14:32
garykwhy would people do that. that is kind of naive.14:32
mriedembecause people do that14:32
dansmithlol14:33
*** FL1SK has quit IRC14:33
garyki am not familiar with anone on the team doing that and actually it is insulting hearing that14:33
openstackgerritdane-fichter proposed openstack/nova: libvirt: Raise error when booting from encrypted volumes  https://review.openstack.org/20378414:33
*** jecarey has joined #openstack-nova14:33
garykmriedem: if you do see that please let me know so that i can deal with that14:34
*** cfriesen_ has joined #openstack-nova14:34
*** kurtmartin has joined #openstack-nova14:35
mriedemi'm not saying the vmware team specifically14:35
*** cfriesen has quit IRC14:35
*** kurtmartin is now known as kmartin14:35
*** alexschm has joined #openstack-nova14:35
mriedeminterested parties come along and +1 a thing frequently just because they want to see it in, not because they've done a thorough review14:35
*** dustins has quit IRC14:36
*** kmartin has quit IRC14:36
garykmriedem: but you wrote "assuming the inter-team reviews are not just a pile of +1s" when referring to the vmware subteam14:36
garykso i do not really understand14:36
ccesariomriedem, garyk  the patch it is working very good!14:37
*** jcoufal has joined #openstack-nova14:37
*** artom has joined #openstack-nova14:38
*** tonytan4ever has quit IRC14:38
garykccesario: thanks!14:38
*** Murali has quit IRC14:39
mriedemccesario: i'm particularly interested in the upgrade scenario, i.e. you have an instance before the folders change, then you apply the folders change and try to resize or migrate the instance14:39
ccesariomriedem, garyk  http://puu.sh/jp2Sj/953d2aac6c.png14:39
*** emagana has joined #openstack-nova14:39
garykmriedem: an instance has a uuid and that information is part of the vmware config spec.14:39
garykso resizing with works as it is all done with the internall uuid14:39
garykdo you want a demo for this?14:40
garyki really do not understand your concern14:40
*** annashen has joined #openstack-nova14:40
garykmriedem: that is irrelevant to the name of the instance14:40
*** dims_ has quit IRC14:41
garykmriedem: https://github.com/openstack/nova/blob/master/nova/virt/vmwareapi/vm_util.py#L21414:41
garykand all of the seraches, managementr etc is done via that14:41
*** dims has joined #openstack-nova14:41
ccesariogaryk, and in case of I upgrade Vcenter, or lost the vcenter and need "re-add" these Virtual machines in other vcenter ? do you know how do it? is it possible ?14:41
mriedemit's not the name though, it's the folder structure14:42
*** emagana has quit IRC14:42
garykyou can manually edit the VM name on your VC to 'mriedem rocks!" and it will still be successfully managed by openstack14:42
mriedemthis isn't a change that is dealing with the display name though14:42
mriedemright14:42
mriedem?14:42
garykwhat is the issue with the folder structure. that too is backward compatible. new instances will be in the new folders14:42
mriedemmy concern isn't new instances, it's old instances before this folder change14:43
garykmriedem: the bp has 2 parts: 1 folder names and 2 instance names14:43
garykyou have yet to take me to the cleaners on the 2nd patch14:43
*** Venkatesh has joined #openstack-nova14:43
mriedemb/c i don't want to get into the 2nd change until the first is sorted out14:43
garykboth make life a ton simpler for people managing this14:43
*** patrickeast has joined #openstack-nova14:43
garyksure. but i still have do not understand what your issues are with the first. i have explained that backwards compatibilioty is not an issue14:44
garykresize is done in place on the instance.14:44
*** cfriesen_ has quit IRC14:44
garykthat code was added in k14:44
garykso not sure how resize is an issue here14:44
garykccesario: openstack does not support a way of importing running VM's14:45
*** annashen has quit IRC14:45
mriedemmy concern is assumptions in the code now that the folder structure will exist for all instances, and old instances won't have this particular folder structure. like with migration. i don't know if it's an issue, but i raise it up. and i assume the vmware dev team knows more about that than i do, and we don't have upgrade testing in vmware CI for this14:45
ccesarioyou mean folder structure like this ? http://puu.sh/jp3dW/e60e66648e.png14:45
*** zzzeek has joined #openstack-nova14:45
*** lpetrut has joined #openstack-nova14:45
ccesariogaryk, well,  how t proceed in case of Vcenter appliance upgrade? Ibecause this step re-register the vms with new UUID in other appliance instance  ...14:47
garykmriedem: i have tested those flows and upgrades manually. i do not see how we can get a CI to do this for a patch that is in flux and will be till liberty (unless we get an FFE). honestly i am not sure i can apply for one at the moment14:47
*** salv-orl_ has joined #openstack-nova14:47
*** xyang has joined #openstack-nova14:48
openstackgerritZachary Sais proposed openstack/nova: Remove v3 from nova API code tree  https://review.openstack.org/19372514:48
*** emagana has joined #openstack-nova14:48
*** salv-orlando has quit IRC14:49
*** Marga_ has joined #openstack-nova14:49
garykccesario: so if i understand you correctley then you want to take a ESX that has running VM and import it to a new VC14:50
garykthat is currently not supported by openstack14:50
*** tonytan4ever has joined #openstack-nova14:50
ccesariogaryk, not exactly ....  I would do a simple Vcenter appliance Upgrade 5.5 to 6.x by example14:50
*** Piet has joined #openstack-nova14:51
garykccesario: that should be supported14:51
ccesarioI would like*14:51
ccesariobecause vcenter appliance upgrade process start a new vcenter appliance and register the vms with new UUID14:52
garykas far as i understand we have done test like that. i will follow up and let you know. i personally have not sone it14:52
*** achanda has joined #openstack-nova14:52
*** dustins has joined #openstack-nova14:53
ccesariogaryk, good, thanks by this :)14:53
*** bradjones has joined #openstack-nova14:54
*** bradjones has quit IRC14:54
*** bradjones has joined #openstack-nova14:54
openstackgerritZachary Sais proposed openstack/nova: Remove v3 from nova API code tree  https://review.openstack.org/19372514:54
*** mriedem has left #openstack-nova14:56
dansmithjohnthetubaguy: can you see my reply on this? https://review.openstack.org/#/c/197771/1014:56
*** mriedem has joined #openstack-nova14:56
*** angdraug has joined #openstack-nova14:56
moshelemriedem: hi, can you review https://review.openstack.org/#/c/187052/14:56
dansmithalaski: also, I'm not seeing where you voted -1 on that latest patch, but jenkins thinks you did14:56
openstackgerritsulochan-acharya proposed openstack/nova: xenapi: add created tag on cached images  https://review.openstack.org/20910514:56
dansmithalaski: er, gerrit14:56
*** yamahata has joined #openstack-nova14:57
alaskidansmith: I didn't, it was on 9.  I think gerrit thinks 10 is just a rebase?14:57
dansmithalaski: maybe, but I change that thing to a .get in 1014:57
dansmither14:58
dansmithcrap14:58
alaskiheh14:58
dansmithsomehow that didn't make it in14:58
dansmithffs14:58
mriedemmoshele: is johnthetubaguy ok with that?14:58
johnthetubaguydansmith: sounds like a good split14:58
*** agireud has joined #openstack-nova14:58
openstackgerritRoman Bogorodskiy proposed openstack/nova: Clean MAC addresses for direct ports on unplug  https://review.openstack.org/20911014:59
dansmithjohnthetubaguy: okay, let me fix the thing for alaski that I fixed and didn't commit (or something)14:59
openstackgerritRoman Bogorodskiy proposed openstack/nova: Clean MAC addresses for direct ports on unplug  https://review.openstack.org/20911014:59
*** MentalRay has joined #openstack-nova14:59
*** haomaiwang has quit IRC15:00
openstackgerritDan Smith proposed openstack/nova: Remove legacy flavor compatibility code from Instance  https://review.openstack.org/19777115:00
*** rpodolyaka1 has quit IRC15:00
*** patrickeast_ has joined #openstack-nova15:00
*** haomaiwang has joined #openstack-nova15:00
*** hemna has joined #openstack-nova15:00
dansmithmust have gotten caught up in the unit test fail debugging I was doing yesterday15:00
johnthetubaguymriedem: I left that one open till my second patch, as there were some +2 lost across a rebase on there15:00
johnthetubaguymriedem: I am still doing the first pass of -2s15:01
*** vilobhmm has joined #openstack-nova15:01
*** belmoreira has quit IRC15:01
mriedemok15:01
mriedemi'm trying to fix some bugs15:01
mriedemso shrug15:01
*** kiran-r has joined #openstack-nova15:01
moshelemriedem: danpb gave it +2 and it was lost in the rebase15:02
*** patrickeast has quit IRC15:02
*** patrickeast_ is now known as patrickeast15:02
moshelejohnthetubaguy: is it possible to review https://review.openstack.org/#/c/187052/ ?15:03
*** diogogmt_ has joined #openstack-nova15:03
*** achanda has quit IRC15:04
openstackgerritPetrut Lucian proposed openstack/nova: Hyper-V: Implement nova rescue  https://review.openstack.org/12715915:04
*** diogogmt has quit IRC15:05
*** diogogmt_ is now known as diogogmt15:05
dansmithjohnthetubaguy: also, if you could see my assertion on this: https://review.openstack.org/#/c/127159/15:06
dansmithjohnthetubaguy: about hyperv rescue15:06
*** kurtmartin has joined #openstack-nova15:07
dansmithI still think that we should see a clean run of hyper-v ci on it though15:07
*** coolsvap_ is now known as coolsvap|away15:08
johnthetubaguydansmith: +1 your points on both counts, I am justing going through the hyper-v blueprints now, so just about to hit those (I am working backwards in the alphabet, for reasons I don't understand)15:09
dansmithheh, okay15:09
*** rpodolyaka1 has joined #openstack-nova15:12
*** scheuran has quit IRC15:14
*** dannywilson has joined #openstack-nova15:14
*** ijw has quit IRC15:16
*** ijw has joined #openstack-nova15:16
openstackgerritVikhyat Umrao proposed openstack/nova: RBD: Reading rbd_default_features from ceph.conf  https://review.openstack.org/18653715:17
*** ijw has quit IRC15:17
*** tonytan4ever has quit IRC15:18
*** tonytan4ever has joined #openstack-nova15:18
*** dannywilson has quit IRC15:19
*** dannywilson has joined #openstack-nova15:19
*** andymaier has joined #openstack-nova15:20
*** gszasz has quit IRC15:22
*** Vikas_ has quit IRC15:22
sahidmoshele: code is good hope we can have this, thanks15:22
*** andymaier has quit IRC15:22
*** andymaier has joined #openstack-nova15:22
moshelesahid: thanks, I hope so too :)15:23
*** tsekiyama has joined #openstack-nova15:23
*** vilobhmm1 has joined #openstack-nova15:23
*** Marga_ has quit IRC15:23
*** Marga_ has joined #openstack-nova15:24
*** emagana has quit IRC15:24
*** vilobhmm has quit IRC15:25
markus_zIf you backport a commit and don't find a counterpart of changed tests, do you backport then the whole tests?15:25
mriedemmarkus_z: what's the change?15:26
*** edmondsw has quit IRC15:26
openstackgerritDan Smith proposed openstack/nova: Add version counter to Service object  https://review.openstack.org/20173315:26
markus_zmriedem: https://review.openstack.org/#/c/170780/11 the functional tests for the numa topology wasn't there then.15:26
markus_zIt feels somehow odd to omit it.15:27
*** Venkatesh has quit IRC15:27
*** johnthetubaguy has quit IRC15:27
*** browne has joined #openstack-nova15:27
*** Venkatesh has joined #openstack-nova15:28
*** johnthetubaguy has joined #openstack-nova15:28
*** praneshp has joined #openstack-nova15:28
*** rpodolyaka1 has quit IRC15:30
*** yamahata has quit IRC15:32
*** yamahata has joined #openstack-nova15:32
*** rpodolyaka1 has joined #openstack-nova15:34
moshelesahid15:35
moshele: do you know if I should request an Exception  for https://review.openstack.org/#/c/187052/ ? it doesn't have -2 yet , but it had +2 before ?15:35
*** lucas___ has quit IRC15:37
sahidmoshele: sorry i cannot predict whether you will get -2 at this stage even if it was already +2ed15:38
*** vilobhmm1 has quit IRC15:38
mriedemmarkus_z: you mean this https://review.openstack.org/#/c/170780/11/nova/tests/functional/libvirt/test_numa_servers.py ?15:41
markus_zmriedem: yes15:41
moshelesahid: ok, I added my patches to the https://etherpad.openstack.org/p/liberty-nova-non-priority-feature-freeze to request Exception   is that enough  ?15:42
mriedemmarkus_z: yeah, i guess you'd bring this back to stable/kilo if you backport https://github.com/openstack/nova/commit/52d898862dccf8a172175586b4d49faa56aa070215:42
*** tjones has joined #openstack-nova15:42
*** praneshp_ has joined #openstack-nova15:43
markus_zmriedem: Right. And I'm unsure if this is a valid thing to do.15:43
mriedemmarkus_z: it's just adding test coverage, which i think is ok to backport in this case15:43
*** praneshp has quit IRC15:43
*** praneshp_ is now known as praneshp15:43
mriedemmarkus_z: you would probably need to squash those two backports together15:43
*** rpodolyaka1 has quit IRC15:44
sahidgaryk: hum... you will be not happy but i do know understand point of this https://review.openstack.org/#/c/188321/15:44
mriedemso backport the test change first, then backport the bug fix, then git rebase and squash the 2nd into the first15:44
sahidgaryk: perhaps you have already explained that, if so please can you share to me the link15:44
markus_zmriedem: OK, thanks, I'll do it this way.15:44
*** Venkatesh has quit IRC15:44
*** Venkatesh has joined #openstack-nova15:45
dansmithalaski: did you see my query to you on this? https://review.openstack.org/#/c/202605/15:46
mriedemsahid: it's used in the follow on vmware change15:46
mriedemit's part of a bp15:46
dansmithalaski: I think that's a solid candidate for an FFE, given that it's half-merged, so I think it'd be good to make sure it's actually ready if so15:46
garyksahid: thanks. i will add comments to the review. basically it is for NFV deployments and the admin will be able to provision images with specific limitations for the disk, cpu, networ and memory performance15:46
mriedemsahid: https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/vmware-limits,n,z15:47
*** Venkatesh has quit IRC15:47
*** SourabhP has joined #openstack-nova15:47
*** yassine has quit IRC15:47
*** rpodolyaka1 has joined #openstack-nova15:47
sahidgaryk: so operators will provision images, usually when operators want to limit something they use flavor, right? - users may to update these image properties15:49
sahidi mean in a public cloud15:49
sahidmriedem: thanks will read it carefully15:49
alaskidansmith: hadn't seen that.  I'll take a look at the cells side of it15:49
dansmithalaski: thanks15:49
*** pradipta has joined #openstack-nova15:49
sahidgaryk: the spec has been approved so i do not want to be annoy, i will read it and ping you if i need more information on some points15:50
*** HenryG has quit IRC15:51
*** IanGovett1 has joined #openstack-nova15:51
garyksahid: np. reviews are importnat and if there is information lacking then i need to provide that.15:51
*** dustins has quit IRC15:52
*** SourabhP_ has joined #openstack-nova15:52
*** rdopiera has joined #openstack-nova15:53
openstackgerritDeepak C Shetty (deepakcs) proposed openstack/nova: libvirt: Nova part of volume snapshot improvements  https://review.openstack.org/17281315:53
openstackgerritDeepak C Shetty (deepakcs) proposed openstack/nova: objects: Add support for OnlineSnapshotInfo object  https://review.openstack.org/19453815:53
openstackgerritMarian Horban proposed openstack/nova: Added processing /compute URL  https://review.openstack.org/19530315:53
*** IanGovett has quit IRC15:54
*** vivekd has joined #openstack-nova15:55
*** otter768 has joined #openstack-nova15:55
*** SourabhP has quit IRC15:56
*** SourabhP_ is now known as SourabhP15:56
*** rdopiera has quit IRC15:56
*** rdopiera has joined #openstack-nova15:56
*** rdopiera has quit IRC15:57
*** rdopiera has joined #openstack-nova15:57
*** kjelly_ has quit IRC15:58
*** lmtaylor has left #openstack-nova15:59
*** otter768 has quit IRC15:59
*** haomaiwang has quit IRC16:00
*** jamiec has quit IRC16:00
*** cinerama has quit IRC16:00
*** haomaiwang has joined #openstack-nova16:00
*** HenryG has joined #openstack-nova16:01
*** jamiec has joined #openstack-nova16:01
*** cinerama has joined #openstack-nova16:02
*** vilobhmm has joined #openstack-nova16:03
*** jlanoux has quit IRC16:03
openstackgerritMarian Horban proposed openstack/nova: Helper script for running under Apache2  https://review.openstack.org/19072716:04
*** dustins has joined #openstack-nova16:05
sahidmriedem: ahaha i thought it was a link of the spec :)16:05
*** leeantho has joined #openstack-nova16:05
*** vilobhmm has quit IRC16:06
*** kmARC has quit IRC16:06
*** vilobhmm has joined #openstack-nova16:06
*** emagana has joined #openstack-nova16:07
sahidgaryk: is there a spec behind that bp/vmware-limits ?16:07
*** tpeoples has quit IRC16:08
*** rpodolyaka1 has quit IRC16:08
mriedemsahid: http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/vmware-limits.html16:08
mriedemhttps://review.openstack.org/#/c/192675/ was the spec16:08
*** HenryG has quit IRC16:08
*** jistr has quit IRC16:09
*** rpodolyaka1 has joined #openstack-nova16:09
*** unicell has joined #openstack-nova16:10
*** coolsvap|away is now known as coolsvap16:10
*** vivekd has quit IRC16:10
sahidhum... there is nothing about to expose these fields to users16:12
*** unicell1 has quit IRC16:12
*** luv has left #openstack-nova16:13
*** elmiko has quit IRC16:13
*** unicell has quit IRC16:14
mriedemisn't that through flavor extra specs?16:16
garyksahid: i am not sure thta i understand the question16:16
*** kiran-r has quit IRC16:18
*** HenryG has joined #openstack-nova16:18
sahidthe last two patches of the serie is to expose fields related to quota to users16:18
sahidby the way of image properties16:18
*** lucas__ has joined #openstack-nova16:19
sahidand i do not see the use case since users do not know much about the host16:19
sahidalso nothing about image properties is mentionned in spec16:19
sahidmriedem: no my complaint is about image properties16:19
*** sulo has joined #openstack-nova16:20
garyksahid: the admin who is creating the images will know the constraints that they would like to apply to the image16:20
*** tpeoples has joined #openstack-nova16:21
garykfor example i can create a rhel image for you that has certain disk limitation and a rhel image for mriedem that has different ones all depending on a sla16:21
sahidgaryk: don't we usually use flavor extra-spec fot that?16:21
garyksahid the problem is that flavors are explaoding and here it offers the ability for an admin to do the same from an image16:21
sahidbut what is the relation image/quota ? i mean the image needs to be configured specificly?16:22
*** annegentle has quit IRC16:22
garyksahid: images can be assigned by an admin to tenants.16:22
sahidsome tune has to be done in the image when we use quota?16:22
garykthose images will have thes eproperties configured16:22
sahidgaryk: we have same possibility with flavor16:22
garyksahid: it is not about tuning, it is about resources on the backend16:23
*** annegentle has joined #openstack-nova16:23
garykyes, but here it is the admin providing that and not the tenant choosing it.16:23
*** nic has joined #openstack-nova16:23
garykit is just a different work flow16:23
*** Marga_ has quit IRC16:23
garykplease not that the glance policy will need to be updated so a tenant cannot set these on images.16:23
garyki need to add that to the commit message16:23
garykcan we talk about this tomorrow i need to head out.16:23
*** rgerganov has quit IRC16:24
sahidgaryk: sure that will give me the time to understand more :)16:24
*** alex_klimov has quit IRC16:24
sahidhum yes we have a concept of private properties in glance or something like that16:25
*** meghal has joined #openstack-nova16:25
*** pradipta has quit IRC16:26
*** bapalm_ has quit IRC16:27
*** kiran-r has joined #openstack-nova16:27
*** armax has quit IRC16:28
*** openstackgerrit_ has quit IRC16:29
*** sayali has quit IRC16:30
*** ftersin has joined #openstack-nova16:31
*** rdopiera has quit IRC16:32
ftersinndipanov: hi. look at https://review.openstack.org/#/c/208796/ please16:32
*** matrohon has quit IRC16:38
*** tjones has quit IRC16:39
*** HenryG has quit IRC16:39
*** sudipto has joined #openstack-nova16:39
*** tjones has joined #openstack-nova16:41
*** patrickeast has quit IRC16:41
*** annashen has joined #openstack-nova16:41
*** edmondsw has joined #openstack-nova16:42
*** unicell has joined #openstack-nova16:42
*** mdrabe has quit IRC16:44
*** apuimedo has quit IRC16:44
*** mdrabe has joined #openstack-nova16:45
*** annashen has quit IRC16:46
*** ssurana has joined #openstack-nova16:46
*** _kiran_ has joined #openstack-nova16:47
ndipanovftersin, thanks16:51
*** kiran-r has quit IRC16:51
*** praneshp has quit IRC16:51
*** moshele has quit IRC16:51
*** fawadkhaliq has quit IRC16:52
ndipanovftersin, nice16:52
*** sahid has quit IRC16:55
*** cinerama has quit IRC16:57
*** patrickeast has joined #openstack-nova16:57
*** romainh has left #openstack-nova16:58
*** xyang2 has joined #openstack-nova16:59
*** haomaiwang has quit IRC17:00
*** haomaiwang has joined #openstack-nova17:00
openstackgerritEdgar Magana proposed openstack/nova: Fix few typos in nova code and docs  https://review.openstack.org/20918317:02
*** _kiran_ has quit IRC17:02
mriedemftersin: added a comment, should be a simple update, then i'd be +2 on that17:02
mriedemftersin: btw, nice commit message explanation of the problem and the fix17:03
*** tjones has quit IRC17:04
openstackgerritSimona Iuliana Toader proposed openstack/nova: Hyper-V: Adds host maintenance implementation  https://review.openstack.org/20415717:04
*** alexschm has quit IRC17:05
*** MentalRay has quit IRC17:06
*** HenryG has joined #openstack-nova17:10
*** rpodolyaka1 has quit IRC17:13
*** tjones has joined #openstack-nova17:15
*** ctrath has quit IRC17:15
*** lucasagomes is now known as lucas-dinner17:16
sudiptodansmith, I guess the security_group_rule NovaObjectDictCompat https://review.openstack.org/192760 has issues... :(17:17
dansmithsudipto: ?17:17
*** exploreshaifali has joined #openstack-nova17:18
*** danpb has quit IRC17:18
sudiptodansmith, saw this earlier in the day: http://paste.openstack.org/show/406912/17:19
sudiptobasically it appears that some of the test classes are still using the dict as opposed to nova objects...17:19
*** HenryG has quit IRC17:20
dansmithsudipto: is that related to your scheduled_at patch?17:20
dansmithor did you mean to link to the security group one?17:21
dansmithsudipto: ah, because I rechecked that one17:21
openstackgerritVladik Romanovsky proposed openstack/nova: Adding user_id handling to keypair index, show and create api calls  https://review.openstack.org/7048517:21
sudiptodansmith, opps. the security group rule one.17:21
dansmithsudipto: that's cool I just rechecked because of the failure we had yesterday17:21
sudiptodansmith, yeah i realized that :)17:22
dansmithsudipto: I'll drop my vote17:22
sudiptodansmith, yeah sure...i will fix the problem...17:22
*** tjones has quit IRC17:22
sudiptodansmith, meanwhile will fix the other objects as well. Thanks for the reviews today!17:22
*** achanda has joined #openstack-nova17:22
dansmithokay17:22
*** jogo has quit IRC17:24
*** ndipanov has quit IRC17:25
*** nic has quit IRC17:25
*** VW_ has quit IRC17:25
*** jogo has joined #openstack-nova17:26
*** VW_ has joined #openstack-nova17:26
*** salv-orl_ has quit IRC17:26
*** nic has joined #openstack-nova17:27
*** MentalRay has joined #openstack-nova17:30
*** VW_ has quit IRC17:31
*** armax has joined #openstack-nova17:31
*** VW_ has joined #openstack-nova17:32
*** kiran-r has joined #openstack-nova17:34
*** claudiub has quit IRC17:35
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Fix root device name for volume-backed instances  https://review.openstack.org/20879617:35
*** dprince has quit IRC17:37
*** garyk has quit IRC17:38
*** exploreshaifali_ has joined #openstack-nova17:39
ftersinmriedem: done, thanks17:40
*** tjones has joined #openstack-nova17:40
*** kiran-r has quit IRC17:40
ftersinndipanov: thank you too17:40
mriedemftersin: +2, but needs ndipanov - he's gone now though17:41
*** marzif_ has quit IRC17:41
*** exploreshaifali has quit IRC17:41
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: Support host type specific block volume attachment  https://review.openstack.org/20919417:42
*** SourabhP has quit IRC17:43
*** claudiub has joined #openstack-nova17:43
*** zhenguo has quit IRC17:43
*** praneshp has joined #openstack-nova17:44
sdaguemriedem: I'm now frustrated that that patch series of yours won't land. At least the unit test bug is fixed now17:44
ftersinmriedem: anyway stackforge/ec2-api is blocked by https://review.openstack.org/#/c/208932/ as well :/17:45
mriedemsdague: sorry?17:46
_gryfsdague, hi.17:46
mriedemsdague: it could be far worse, you could be getting inundated with qpid on grizzly questions/problems17:47
dansmitheww17:47
mriedemyea, no shit17:47
mriedemdouble eww17:47
*** vivekd has joined #openstack-nova17:47
*** achanda_ has joined #openstack-nova17:48
_gryfsdague, could you please take a look at https://review.openstack.org/#/c/184086/ to tell is that a right way to add new rest api call?17:48
sdagueis your answer "light it on fire"?17:48
*** david-lyle has quit IRC17:48
*** Nikolay_St has quit IRC17:49
mtreinishsdague: that seems like the right answer17:50
*** markus_z has quit IRC17:50
*** achanda has quit IRC17:50
openstackgerritVikhyat Umrao proposed openstack/nova: RBD: Reading rbd_default_features from ceph.conf  https://review.openstack.org/18653717:53
*** otter768 has joined #openstack-nova17:56
*** dprince has joined #openstack-nova17:57
*** dprince has quit IRC17:57
*** exploreshaifali_ has quit IRC17:58
*** haomaiwang has quit IRC18:00
*** haomaiwang has joined #openstack-nova18:00
*** exploreshaifali has joined #openstack-nova18:01
*** otter768 has quit IRC18:01
*** kylek3h has quit IRC18:01
*** armax has quit IRC18:02
mriedemwell, i did poke a hole in that they were supposed to package 2 patches and only patched one18:02
mriedemderp18:02
*** aix has quit IRC18:03
mriedemand do that for like, kilo -> folsom18:03
mriedemthis is what happens when i venture back into the land of packaging and builds18:03
*** VW_ has quit IRC18:04
*** VW_ has joined #openstack-nova18:05
sdague_gryf: commented18:06
dansmithis this spiking? http://goo.gl/yqRTPv18:06
mriedemdansmith: sdague said it's fixed18:06
mriedemi just rechecked your change18:06
dansmithoh18:06
dansmithdamn you beat me18:07
sdaguedansmith: that was the one that was ruining us for the last couple of weeks18:07
*** armax has joined #openstack-nova18:07
sdagueif you see it in recent stuff, please point at the log, because I think it's fixed18:07
dansmithwell, I just hit it minutes ago18:08
dansmithhttp://logs.openstack.org/71/197771/11/check/gate-nova-python27/40adc59/console.html18:08
dansmithdunno when the fix merged18:08
openstackgerritSudipta Biswas proposed openstack/nova: cleanup NovaObjectDictCompat from virt_cpu_topology  https://review.openstack.org/20920518:08
sdaguehttps://review.openstack.org/#/c/208253/18:08
sdague86 minutes ago18:08
dansmithah, mine might've been in the queue before then18:08
dansmithcheck18:08
_gryfsdague, thank you18:09
dansmithalmost definitely18:09
sdagueyeh, that's before the fix18:09
*** VW_ has quit IRC18:09
dansmithcool18:09
sdaguewe don't use assertJsonEqual on those asserts now18:09
*** dan_ has joined #openstack-nova18:09
* dansmith waits for the merry-go-round to spin18:09
*** dan_ is now known as Guest4157418:10
*** rpodolyaka1 has joined #openstack-nova18:11
*** VW_ has joined #openstack-nova18:12
*** Guest41574 is now known as dank_18:12
ftersinsdague: but why? assertJsonEqual is designed to compare unordered lists, as it is mentioned in its docstring18:12
*** ctrath has joined #openstack-nova18:15
ftersinsdague: there is no usage of it that wants to compare ordered lists (otherwise a respective test would be present for this function)18:15
openstackgerritRoman Podoliaka proposed openstack/nova: WIP: don't initialize RPC when parsing a config file  https://review.openstack.org/18646918:15
ftersinsdague: and at the last, developers want to compare json regardless of items order in lists18:17
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/20870618:17
sdagueftersin: so assertJsonEqual does not work correctly in all cases if you have [{complex dict}, {complex dict}]18:18
ftersinsdague: does it mean that we need a new function, or a special parameter for existing one?18:18
sdaguebecause the sort key for the list is the str of the items in the list18:18
sdaguebut that str depends on PYTHONHASHSEED18:18
sdagueor more importantly, is impacted by it, for dictionaries18:19
*** tongli has quit IRC18:19
sdagueso when running with hash seed randomization, the results are unpredictable18:19
*** flwang1 has quit IRC18:19
*** exploreshaifali has quit IRC18:19
ftersinbut Nikitin improved this function in PS5 of its review18:20
*** lucas__ has quit IRC18:20
sdagueftersin: it's still broken18:20
sdagueif there is another fix up, point me to it18:20
sdaguebut this was provably broken, which is why we were failing unit tests randomly18:21
ftersini understand why it doesn't work properly, i don't understand why not to fix it18:21
sdagueI didn't see PS5, but the time I saw it it was PS618:22
*** wverdugo500 has quit IRC18:22
*** gholler has joined #openstack-nova18:23
sdagueftersin: so, I think that the fix in PS5 is also valid. I'd say re-propose that as a distinct patch18:24
sdagueI think danpb might not have realized that the list sorting was intended in this function all along because of the set issue18:25
*** lucas__ has joined #openstack-nova18:27
*** lpetrut has quit IRC18:27
*** armax has quit IRC18:28
*** andymaier has quit IRC18:28
*** kurtmartin has quit IRC18:28
ftersinprobably. i hope for that18:29
*** xyang2 has quit IRC18:29
rlrossitmriedem: haven't gone back to this in a while... if you've got some time want to take a look at this? https://review.openstack.org/#/c/195713/18:30
*** lucas__ has quit IRC18:30
*** armax has joined #openstack-nova18:30
*** lucas__ has joined #openstack-nova18:30
*** HenryG has joined #openstack-nova18:31
*** patrickeast has quit IRC18:31
*** irenab has quit IRC18:32
*** dannywilson has quit IRC18:32
*** HenryG has quit IRC18:32
*** claudiub|2 has joined #openstack-nova18:33
mriedemrlrossit: sorry i'm backporting patches to qpid 0.1818:33
*** irenab has joined #openstack-nova18:34
rlrossitthat sounds like fun18:34
mriedemit's a blast18:34
*** HenryG has joined #openstack-nova18:35
*** dannywilson has joined #openstack-nova18:35
*** claudiub has quit IRC18:35
ctrathrlrossit: I reviewed that again..  Looks good to me now18:37
*** neelabh__ has quit IRC18:37
alaskidansmith: looking at https://review.openstack.org/#/c/202605 and it brings up a question on upgrades and microversions for me.  In this case exposing a new API doesn't mean that the call is usable, until computes are upgraded.18:38
alaskiI'm not sure if anyone has considered that yet18:39
*** tojuvone has quit IRC18:40
*** bapalm_ has joined #openstack-nova18:41
dansmithalaski: yeah, well I brought that up18:42
dansmithalthough that might've been in email somewhere, now that I think of it18:42
dansmiththere's two cases18:42
dansmithone is that the virt driver doesn't support it, and another is that we can't do it yet because we're pinned18:42
*** annashen has joined #openstack-nova18:42
dansmithif we're pinned, I feel like some 5xx error that implies it's not *currently* available makes sense to me18:42
dansmithand then some 4xx if the virt driver doesn't support it18:43
dansmithalaski: does that make sense?18:45
alaskiI don't really like returning 5xx, as that usually indicates an unhandled case18:45
dansmithis there a 4xx with "try again later" semantics?18:45
*** HenryG_ has joined #openstack-nova18:46
alaskinot afaik, which is unfortunate18:46
*** annashen has quit IRC18:47
dansmithI know 5xx is normally "we screwed up" but I thought that'd be the closest for "I can't do this right now" but I dunno18:47
alaskiI could see going with 501, it's just unfortunate that it's not a 4xx error18:47
dansmithwell, either way, errors to signal those two possibilities in addition to success18:48
melwittbased on the new documentation of "when do I need a microversion" I looked up what 501 means and it apparently is that. "can't do right now"18:48
dansmithwhatever numbers we choose18:48
*** HenryG has quit IRC18:48
alaskidansmith: I'm actually picking on call vs cast as well in the review because I'd like to steer away from API responses depending on driver responses18:49
dansmithalaski: well, I asked about that as well18:49
dansmithalaski: because if you're sending an NMI, things are hosed18:49
dansmithand if we just silently return 202 and never pass the NMI because it's not supported, that's like punching someone in the face while they're already in a panic to get things working18:50
dansmithso, yeah, I'm with you, but I don't really know what else to do18:50
alaskiinstance actions are the best we have right now18:50
dansmithunless we put the instance in ERROR state as a result18:50
dansmithokay, well, that works I guess18:50
alaskiinstance actions and tasks are mostly a thing because we don't return driver responses in the API18:51
dansmithyeah, if we had tasks, I'd say do it that way18:51
dansmithbut if we can communicate it with actions, then that's cool too18:51
alaskithe issue with actions is that it's not clear that someone should look at them, but the info should be there if there's a failure18:52
dansmithI just asked "how are you going to communicate that you didn't really hit the red button" in the cases where we punt by version or don't support it18:52
*** SourabhP has joined #openstack-nova18:52
dansmithyeah, well as long as it's somewhere that's better than "sure, I just NMI'd your server"18:52
*** boris-42 has joined #openstack-nova18:53
*** sudipto has quit IRC18:54
alaskidansmith: the cells stuff looks good on the patch, and the instance action stuff.  just commenting on the call/cast thing18:54
dansmithokay18:54
dansmithwhen you're done I'll comment and bow to your wisdom18:54
alaskiheh.  I don't know about that, it's more fear of seeing more methods that have to go all the way to a compute before returning an API response, and the can of worms that opens18:55
dansmithtotally agree18:56
dansmithI just think this is like a panic button, rarely if ever hit, and lying about it sucks18:56
*** exploreshaifali has joined #openstack-nova18:56
*** exploreshaifali has quit IRC18:57
dansmithwhich is why I preferred getting accurate info instead of silence... but definitely prefer not being direct and getting the info :)18:57
*** haomaiwang has quit IRC19:00
*** haomaiwang has joined #openstack-nova19:00
*** mriedem is now known as mriedem_qpid_ftw19:01
*** exploreshaifali has joined #openstack-nova19:02
*** rushiagr is now known as rushiagr_away19:03
*** exploreshaifali_ has joined #openstack-nova19:05
*** MentalRay has quit IRC19:05
* dansmith bows19:05
*** angdraug has quit IRC19:05
alaskihah19:05
*** MentalRay has joined #openstack-nova19:06
*** tonytan4ever has quit IRC19:07
*** Piet has quit IRC19:07
*** exploreshaifali has quit IRC19:07
*** coolsvap is now known as coolsvap|away19:10
*** salv-orlando has joined #openstack-nova19:10
*** vivekd has quit IRC19:10
*** claudiub|2 has quit IRC19:10
*** ftersin has quit IRC19:13
*** dprince has joined #openstack-nova19:14
*** lucas__ has quit IRC19:14
*** exploreshaifali_ has quit IRC19:15
*** xyang has quit IRC19:18
*** penick has joined #openstack-nova19:18
edleafesdague: regarding the v3 cleanup: since we didn't get the group of cores working on it at the midcycle, I think it would be best to go back to the original approach of a series of patches19:20
edleafesdague: take a look at https://review.openstack.org/#/c/193725/ and tell me that anyone can review that mess19:20
*** unicell has quit IRC19:20
*** unicell1 has joined #openstack-nova19:20
*** tjones has quit IRC19:21
*** Marga_ has joined #openstack-nova19:21
*** Marga_ has quit IRC19:24
*** Marga_ has joined #openstack-nova19:25
jrollis there a way to fail an action (e.g. delete) but put the instance back to ACTIVE?19:28
*** angdraug has joined #openstack-nova19:30
*** vivekd has joined #openstack-nova19:32
*** vivekd has quit IRC19:34
*** jcoufal has quit IRC19:39
*** lucas__ has joined #openstack-nova19:40
*** READ10 has quit IRC19:40
*** baoli has quit IRC19:42
*** thangp has quit IRC19:42
*** lpetrut has joined #openstack-nova19:43
*** thangp has joined #openstack-nova19:44
*** kurtmartin has joined #openstack-nova19:44
*** xyang2 has joined #openstack-nova19:46
*** lsmola has quit IRC19:46
*** samueldmq has quit IRC19:46
*** samueldmq has joined #openstack-nova19:47
*** ashuk has joined #openstack-nova19:48
*** thangp has quit IRC19:49
*** moshele has joined #openstack-nova19:52
*** shihanzhang has quit IRC19:53
openstackgerritEd Leafe proposed openstack/nova: Move existing V2 to legacyv2  https://review.openstack.org/19372519:53
*** rpodolyaka1 has quit IRC19:53
*** ashuk has quit IRC19:53
*** shihanzhang has joined #openstack-nova19:53
openstackgerritEd Leafe proposed openstack/nova: Move existing V2 to legacy_v2  https://review.openstack.org/19372519:54
*** ashuk has joined #openstack-nova19:54
*** ekarlso has quit IRC19:55
*** irenab has quit IRC19:55
edleafesdague: rolled it back to PS 11 from before the mid-cycle.19:56
*** armax has quit IRC19:56
*** otter768 has joined #openstack-nova19:57
*** ekarlso has joined #openstack-nova19:57
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: Better handling of SSH key comments  https://review.openstack.org/20866119:58
*** haomaiwang has quit IRC20:00
*** baoli has joined #openstack-nova20:00
*** haomaiwang has joined #openstack-nova20:00
*** chhavi has quit IRC20:00
*** baoli has quit IRC20:01
*** VW__ has joined #openstack-nova20:01
*** baoli has joined #openstack-nova20:01
*** otter768 has quit IRC20:01
sdagueedleafe: honestly, I still think big bang is a better option, just pick a day and do it. Because I really don't want to half land this20:02
sdagueit's going to be so much more work rebase chasing both sides20:02
edleafesdague: then please take a look at PS 21, and let me know if that is reviewable, much less mergeable20:03
sdagueedleafe: will do in the morning20:03
*** VW__ has quit IRC20:03
edleafesdague: thx20:03
*** thumpba has quit IRC20:03
*** VW__ has joined #openstack-nova20:04
openstackgerritDan Smith proposed openstack/nova: Remove legacy flavor compatibility code from Instance  https://review.openstack.org/19777120:04
openstackgerritDan Smith proposed openstack/nova: Remove tie between system_metadata and extra.flavor  https://review.openstack.org/20924020:04
dansmithdangit20:04
dansmithalaski: rebased the above patch unintentionally.. if you can slap the +W back on it...20:05
*** lpetrut has quit IRC20:05
*** VW_ has quit IRC20:05
alaskidansmith: got it20:05
*** tonytan4ever has joined #openstack-nova20:06
*** flwang1 has joined #openstack-nova20:06
*** VW__ has quit IRC20:06
*** patrickeast has joined #openstack-nova20:06
*** dprince has quit IRC20:07
*** VW_ has joined #openstack-nova20:07
dansmiththanks20:08
*** rpodolyaka1 has joined #openstack-nova20:09
*** lpetrut has joined #openstack-nova20:10
*** Piet has joined #openstack-nova20:10
*** marzif_ has joined #openstack-nova20:12
*** mriedem_qpid_ftw is now known as mriedem_build_qp20:13
*** MentalRay has quit IRC20:15
*** flwang1 has quit IRC20:15
*** atuvenie has quit IRC20:16
*** MentalRay has joined #openstack-nova20:16
*** dims_ has joined #openstack-nova20:21
*** Marga_ has quit IRC20:21
*** dims has quit IRC20:22
*** MentalRay has quit IRC20:29
*** meghal has quit IRC20:31
*** meghal has joined #openstack-nova20:31
*** MentalRay has joined #openstack-nova20:32
*** meghal has quit IRC20:32
*** meghal has joined #openstack-nova20:32
*** patrickeast has quit IRC20:33
*** patrickeast has joined #openstack-nova20:40
*** annashen has joined #openstack-nova20:43
openstackgerritNicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available  https://review.openstack.org/20528220:43
*** emagana has quit IRC20:45
*** atuvenie has joined #openstack-nova20:47
*** annashen has quit IRC20:48
*** echoingumesh has joined #openstack-nova20:48
*** emagana has joined #openstack-nova20:48
*** chuckcarmack has left #openstack-nova20:48
*** rpodolyaka1 has quit IRC20:48
*** flwang1 has joined #openstack-nova20:53
*** lucas__ has quit IRC20:54
*** lucas-dinner has quit IRC20:56
*** apuimedo has joined #openstack-nova20:56
*** flwang1 has quit IRC20:57
*** haomaiwang has quit IRC21:00
*** gyee has joined #openstack-nova21:00
*** haomaiwang has joined #openstack-nova21:00
*** Marga_ has joined #openstack-nova21:01
*** irenab has joined #openstack-nova21:01
*** smatzek has quit IRC21:01
*** meghal has quit IRC21:01
*** angdraug has quit IRC21:02
openstackgerritAnthony Lee proposed openstack/nova: Fix live-migrations usage of the wrong connector information  https://review.openstack.org/20277021:02
*** tjones has joined #openstack-nova21:02
*** irenab has quit IRC21:02
*** irenab has joined #openstack-nova21:03
*** flwang1 has joined #openstack-nova21:04
*** raildo has quit IRC21:04
*** baoli has quit IRC21:05
*** Vek has quit IRC21:05
*** Nikolay_St has joined #openstack-nova21:06
*** meghal has joined #openstack-nova21:07
*** bapalm_ has quit IRC21:07
*** josecastroleon has quit IRC21:08
*** patrickeast has quit IRC21:09
*** aysyd has quit IRC21:09
*** josecastroleon has joined #openstack-nova21:09
*** gyee has quit IRC21:09
*** moshele has quit IRC21:10
*** armax has joined #openstack-nova21:12
*** rlrossit has left #openstack-nova21:12
*** thorst has quit IRC21:13
*** kurtmartin is now known as kmartin21:13
*** irenab has quit IRC21:13
*** shihanzhang has quit IRC21:14
*** shihanzhang has joined #openstack-nova21:15
*** lucas__ has joined #openstack-nova21:15
*** gyee has joined #openstack-nova21:15
*** HenryG_ is now known as HenryG21:17
*** irenab has joined #openstack-nova21:17
*** patrickeast has joined #openstack-nova21:17
*** lucas__ has quit IRC21:19
*** dboik has quit IRC21:20
*** Marga_ has quit IRC21:21
*** Marga_ has joined #openstack-nova21:22
*** salv-orlando has quit IRC21:26
*** armax has quit IRC21:29
*** VW_ has quit IRC21:31
*** VW_ has joined #openstack-nova21:32
*** mwagner_lap has quit IRC21:32
*** pixelbeat has joined #openstack-nova21:34
*** pixelbeat_ has quit IRC21:36
*** thorst has joined #openstack-nova21:36
*** lucas__ has joined #openstack-nova21:36
*** VW_ has quit IRC21:37
*** alex_klimov has joined #openstack-nova21:37
*** lucas__ has quit IRC21:38
*** salv-orlando has joined #openstack-nova21:38
*** lucas__ has joined #openstack-nova21:38
*** thumpba has joined #openstack-nova21:38
*** artom has quit IRC21:39
*** artom has joined #openstack-nova21:41
*** Nikolay_St has quit IRC21:41
*** mriedem_build_qp is now known as mriedem_away21:41
*** thorst has quit IRC21:42
*** lucas__ has quit IRC21:43
*** patrickeast has quit IRC21:44
openstackgerritPengtao Huang proposed openstack/nova: change the value "all_mappings" to "block_device_mapping"  https://review.openstack.org/20851121:45
*** patrickeast has joined #openstack-nova21:46
*** xyang2 has quit IRC21:48
*** xyang2 has joined #openstack-nova21:48
*** pixelbeat has quit IRC21:48
*** thumpba has quit IRC21:48
*** tonytan4ever has quit IRC21:49
*** thumpba has joined #openstack-nova21:49
*** busterswt has quit IRC21:50
*** patrickeast has quit IRC21:53
*** agireud has quit IRC21:53
openstackgerritJohn Garbutt proposed openstack/nova: vmware: add image_cache_manager property  https://review.openstack.org/18760221:56
*** thumpba_ has joined #openstack-nova21:56
*** pixelbeat has joined #openstack-nova21:57
*** patrickeast has joined #openstack-nova21:57
*** otter768 has joined #openstack-nova21:58
*** thumpba has quit IRC21:59
*** haomaiwang has quit IRC22:00
*** burt has quit IRC22:00
*** haomaiwang has joined #openstack-nova22:00
*** edmondsw has quit IRC22:02
*** armax has joined #openstack-nova22:02
*** otter768 has quit IRC22:02
*** meghal has quit IRC22:03
*** thumpba_ has quit IRC22:03
*** thumpba has joined #openstack-nova22:05
*** thumpba has quit IRC22:08
*** thumpba has joined #openstack-nova22:09
*** thumpba has quit IRC22:09
*** thumpba has joined #openstack-nova22:10
*** prometheanfire has left #openstack-nova22:13
*** lucas__ has joined #openstack-nova22:14
*** VW_ has joined #openstack-nova22:16
*** lucas__ has quit IRC22:18
*** jecarey has quit IRC22:20
*** ctrath has quit IRC22:22
*** zul has quit IRC22:23
*** IanGovett1 has quit IRC22:27
*** thumpba has quit IRC22:31
*** meghal has joined #openstack-nova22:32
*** dustins has quit IRC22:32
*** ijw has joined #openstack-nova22:33
*** xyang2 has quit IRC22:34
*** xyang2 has joined #openstack-nova22:34
*** jaypipes has quit IRC22:35
*** atuvenie has quit IRC22:35
*** alex_klimov has quit IRC22:36
*** gholler has quit IRC22:38
*** lucas__ has joined #openstack-nova22:38
*** VW_ has quit IRC22:39
*** VW_ has joined #openstack-nova22:39
*** boris-42 has quit IRC22:40
*** lucas__ has quit IRC22:43
*** annashen has joined #openstack-nova22:44
*** VW_ has quit IRC22:44
*** MentalRay has quit IRC22:47
*** annashen has quit IRC22:48
*** penick has quit IRC22:53
*** apuimedo has quit IRC22:59
*** haomaiwang has quit IRC23:00
*** Vikas has joined #openstack-nova23:00
*** haomaiwang has joined #openstack-nova23:00
*** Marga_ has quit IRC23:01
*** zzzeek has quit IRC23:02
*** Vek has joined #openstack-nova23:03
*** xyang2 has quit IRC23:05
VikasMorning folks23:07
*** echoingumesh has quit IRC23:07
VikasOn review,https://review.openstack.org/#/c/208680/,hyper-V CI is failing.not able to find failure cause also from the logs.There is no console.html.Any suggestion will be much appreciated.Please help23:09
*** lucas__ has joined #openstack-nova23:10
*** slong has joined #openstack-nova23:11
*** tpeoples has quit IRC23:11
*** echoingumesh has joined #openstack-nova23:12
*** lucas__ has quit IRC23:15
*** diogogmt has quit IRC23:16
Vikasis it mandatory for a review request to pass all third party CI test like in my case hyper-v is failing and it has nothing to do with my changes.Can i go ahead ignoring this and ask for review of my codew changes?23:20
Vikashttps://review.openstack.org/#/c/208680/23:20
flwangdansmith: can you help review the volume backed backup spec ? https://review.openstack.org/190919  thanks a lot23:22
*** dannywilson has quit IRC23:23
*** dims_ has quit IRC23:23
openstackgerritvikas choudhary proposed openstack/nova: Update hypervisor resources correctly in booting failed case  https://review.openstack.org/20868023:24
openstackgerritDan Smith proposed openstack/nova: Adding user_id handling to keypair index, show and create api calls  https://review.openstack.org/7048523:25
*** ijw_ has joined #openstack-nova23:27
*** patrickeast has quit IRC23:27
*** lpetrut has quit IRC23:28
*** hemna has quit IRC23:28
*** dave-mccowan has quit IRC23:28
*** kmartin has quit IRC23:28
*** ijw has quit IRC23:30
*** thorst has joined #openstack-nova23:31
*** dave-mccowan has joined #openstack-nova23:32
*** ijw has joined #openstack-nova23:33
*** marcusvrn has quit IRC23:33
*** ijw_ has quit IRC23:36
*** mariojv has quit IRC23:38
*** boris-42 has joined #openstack-nova23:38
*** mariojv has joined #openstack-nova23:38
*** lucas__ has joined #openstack-nova23:40
*** otter768 has joined #openstack-nova23:41
*** lucas__ has quit IRC23:45
*** dave-mccowan has quit IRC23:47
*** dims has joined #openstack-nova23:47
*** jamielennox|away is now known as jamielennox23:50
*** zzzeek has joined #openstack-nova23:51
*** pixelbeat has quit IRC23:54
*** emagana has quit IRC23:56
*** bauzas_off has quit IRC23:56
*** zzzeek has quit IRC23:56
*** eliqiao has joined #openstack-nova23:56
*** salv-orlando has quit IRC23:57
*** bauzas has joined #openstack-nova23:58
*** shyu has joined #openstack-nova23:59

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