Thursday, 2016-09-22

*** bdeschenes has joined #openstack-nova00:00
*** rama_y has quit IRC00:02
*** smatzek has quit IRC00:04
*** EinstCrazy has joined #openstack-nova00:09
*** bdeschenes has quit IRC00:09
*** EinstCrazy has quit IRC00:10
*** ducttape_ has joined #openstack-nova00:11
*** dpeschman has quit IRC00:12
*** TravT has quit IRC00:13
*** rmcall has quit IRC00:14
*** rmcall has joined #openstack-nova00:15
mriedemmikal: can you drop the -2 on these https://review.openstack.org/#/q/topic:bp/goal-python35 ?00:20
mriedemharlowja: i expect once we have a migration plan from cells v1 to v2 and people start kicking those tires we'll find out what out of tree patches show up and cause issues00:21
harlowjai can get u a list much sooner than that :-P00:21
mriedemlike, there won't be 2 layers of scheduling00:21
mriedemi know about the mega cells v1 patch etherpad00:21
harlowjak00:21
mriedembut a lot of those were feature parity things00:21
mriedemwhich aren't a problem with cells v200:22
harlowjahttps://gist.github.com/harlowja/581f2395f0cb68d59eba6518a4556816 is our current stuff00:22
harlowjaif u want to see any of those, let me know, ha00:23
*** kaisers1 has joined #openstack-nova00:23
harlowjaprob will be +1 soon with some sync_ instance work00:23
mriedemyeah those mostly look like feature parity things for the API00:23
*** kaisers2 has joined #openstack-nova00:23
mriedemi expect a host of other issues with cells v1 like the racey sync stuff to go away00:24
mriedemsince you don't have the 'at top' kind of issues00:24
harlowjaxright00:24
harlowja*right00:24
*** rmcall has quit IRC00:24
*** zz_dimtruck is now known as dimtruck00:24
*** rmcall has joined #openstack-nova00:25
*** kaisers has quit IRC00:25
*** kaisers3 has quit IRC00:25
*** cdelatte has quit IRC00:29
mriedemjroll: https://review.openstack.org/#/c/328157/ is open for biznass again00:31
*** hieulq_ has joined #openstack-nova00:33
*** ducttape_ has quit IRC00:34
*** Apoorva_ has joined #openstack-nova00:37
openstackgerritTuan Luong-Anh proposed openstack/nova: Using assertIsNone() instead of assertEqual(None)  https://review.openstack.org/36184700:41
*** Apoorva_ has quit IRC00:41
mikalmriedem: yep, working on it. My home internet has been offline all week. I'm fighting that so I can remove those as we speak.00:41
*** Apoorva has quit IRC00:41
mriedemkangaroo attack?00:42
*** ducttape_ has joined #openstack-nova00:42
*** markvoelker has joined #openstack-nova00:45
*** markvoelker has quit IRC00:49
mikalmriedem: no, 50 years of under investment in maintenance of the network00:52
mikalmriedem: those -2's should now be removed, complete with snarky comment about the monopoly telco here00:52
mriedemha, thanks00:53
*** maticue has quit IRC00:53
mikalSorry it took so long00:53
mikalgerrit works better when used on the internet it seem00:53
*** thorst has joined #openstack-nova00:59
*** hieulq_ has quit IRC00:59
*** ijw has quit IRC01:00
*** cheneydc has joined #openstack-nova01:00
*** Swami__ has joined #openstack-nova01:01
*** ducttape_ has quit IRC01:02
*** thorst has quit IRC01:04
*** phuongnh has joined #openstack-nova01:04
*** Swami_ has quit IRC01:05
*** annegentle has joined #openstack-nova01:05
*** sdake has quit IRC01:06
*** cheneydc has quit IRC01:07
*** yamahata has quit IRC01:15
*** baoli has joined #openstack-nova01:16
*** ijw has joined #openstack-nova01:19
*** gouthamr has quit IRC01:20
*** Swami__ has quit IRC01:20
*** baoli has quit IRC01:21
*** bkopilov has quit IRC01:26
*** bkopilov has joined #openstack-nova01:32
*** TravT has joined #openstack-nova01:32
*** EinstCrazy has joined #openstack-nova01:32
*** TravT has quit IRC01:32
*** TravT has joined #openstack-nova01:33
*** unicell has quit IRC01:34
*** TravT has quit IRC01:36
*** TravT has joined #openstack-nova01:37
*** TravT has quit IRC01:38
*** TravT_ has joined #openstack-nova01:38
*** yuntongjin has joined #openstack-nova01:41
*** r-daneel has quit IRC01:42
*** namnh has joined #openstack-nova01:42
*** TravT has joined #openstack-nova01:43
*** salv-orl_ has joined #openstack-nova01:43
*** TravT_ has quit IRC01:43
*** salv-orlando has quit IRC01:45
*** thorst has joined #openstack-nova01:45
*** markvoelker has joined #openstack-nova01:46
*** TravT_ has joined #openstack-nova01:47
*** TravT has quit IRC01:48
*** ijw has quit IRC01:48
*** woodster_ has quit IRC01:50
*** markvoelker has quit IRC01:50
*** thorst has quit IRC01:50
*** TravT_ has quit IRC01:52
*** TravT has joined #openstack-nova01:52
*** baoli has joined #openstack-nova01:52
*** TravT has quit IRC01:57
*** TravT has joined #openstack-nova01:57
*** gongysh has joined #openstack-nova01:57
*** psilvad has quit IRC01:58
*** psilvad has joined #openstack-nova01:59
*** TravT has quit IRC02:01
*** TravT has joined #openstack-nova02:02
*** tuanla has joined #openstack-nova02:02
*** ducttape_ has joined #openstack-nova02:03
*** TravT_ has joined #openstack-nova02:06
*** TravT has quit IRC02:06
*** ducttape_ has quit IRC02:08
*** namnh has quit IRC02:08
*** TravT has joined #openstack-nova02:11
*** TravT_ has quit IRC02:11
*** TravT has quit IRC02:16
*** TravT_ has joined #openstack-nova02:16
*** gongysh has quit IRC02:16
*** thorst has joined #openstack-nova02:17
*** nic has quit IRC02:17
*** gongysh has joined #openstack-nova02:20
*** ijw has joined #openstack-nova02:20
*** TravT has joined #openstack-nova02:20
*** TravT_ has quit IRC02:20
*** thorst has quit IRC02:21
*** psilvad has quit IRC02:22
*** ijw has quit IRC02:25
*** TravT has quit IRC02:25
*** TravT has joined #openstack-nova02:25
*** TravT has quit IRC02:29
*** TravT has joined #openstack-nova02:30
*** TravT has quit IRC02:34
*** TravT has joined #openstack-nova02:35
*** artom has quit IRC02:36
*** artom has joined #openstack-nova02:36
*** TravT has quit IRC02:39
*** TravT_ has joined #openstack-nova02:39
*** wuhg has joined #openstack-nova02:39
*** TravT has joined #openstack-nova02:44
*** TravT_ has quit IRC02:44
*** markvoelker has joined #openstack-nova02:46
*** tbachman has quit IRC02:47
*** thorst has joined #openstack-nova02:48
*** TravT has quit IRC02:49
*** TravT_ has joined #openstack-nova02:49
*** markvoelker has quit IRC02:51
*** thorst has quit IRC02:53
*** TravT_ has quit IRC02:53
*** TravT has joined #openstack-nova02:53
*** TravT has quit IRC02:58
*** TravT_ has joined #openstack-nova02:58
*** TravT has joined #openstack-nova03:03
*** TravT_ has quit IRC03:03
*** david-lyle has quit IRC03:03
*** jwcroppe has quit IRC03:07
*** TravT has quit IRC03:07
*** TravT has joined #openstack-nova03:08
*** jwcroppe has joined #openstack-nova03:08
*** sdake has joined #openstack-nova03:08
*** gcb has joined #openstack-nova03:09
*** ducttape_ has joined #openstack-nova03:11
*** TravT_ has joined #openstack-nova03:12
*** TravT has quit IRC03:13
*** TravT_ has quit IRC03:17
*** TravT has joined #openstack-nova03:17
*** tbachman has joined #openstack-nova03:21
*** baoli has quit IRC03:21
*** rmcall_ has joined #openstack-nova03:21
*** TravT has quit IRC03:21
*** TravT has joined #openstack-nova03:22
*** rmcall has quit IRC03:22
*** rmcall_ is now known as rmcall03:22
*** yuntongjin has quit IRC03:24
*** TravT has quit IRC03:26
*** TravT_ has joined #openstack-nova03:26
*** tjones has joined #openstack-nova03:29
*** TravT has joined #openstack-nova03:30
*** TravT_ has quit IRC03:30
*** annegentle has quit IRC03:30
*** akshai has joined #openstack-nova03:33
*** tjones has left #openstack-nova03:34
*** ducttape_ has quit IRC03:35
*** coolsvap has joined #openstack-nova03:36
*** akshai has quit IRC03:37
*** adisky has joined #openstack-nova03:40
*** hoangcx has joined #openstack-nova03:41
*** unicell has joined #openstack-nova03:41
*** unicell has quit IRC03:43
*** Sukhdev has joined #openstack-nova03:45
*** unicell has joined #openstack-nova03:46
*** mdrabe_ has quit IRC03:54
*** mdrabe_ has joined #openstack-nova03:55
*** armax has quit IRC04:05
*** tristanC has quit IRC04:11
openstackgerritDinesh Bhor proposed openstack/nova: [PY3] byte/string conversions and enable PY3 test  https://review.openstack.org/33321204:13
*** code-R has joined #openstack-nova04:13
*** tbachman has quit IRC04:14
*** kberger has quit IRC04:15
*** kberger has joined #openstack-nova04:16
openstackgerritsonu proposed openstack/nova: Add hacking checks for xrange()  https://review.openstack.org/20530704:17
*** gongysh has quit IRC04:18
*** michchap has quit IRC04:20
*** ijw has joined #openstack-nova04:25
*** ijw has quit IRC04:29
*** alee_afk is now known as alee04:30
*** mdrabe_ has quit IRC04:31
openstackgerritTony Xu proposed openstack/os-vif: Add oslo.concurrency to requirements  https://review.openstack.org/37452404:35
*** ducttape_ has joined #openstack-nova04:36
*** yamahata has joined #openstack-nova04:37
*** ducttape_ has quit IRC04:41
*** ratailor has joined #openstack-nova04:41
*** yuntongjin has joined #openstack-nova04:44
*** oanson has joined #openstack-nova04:44
*** psachin has joined #openstack-nova04:44
*** kberger has quit IRC04:45
*** kberger has joined #openstack-nova04:46
*** code-R has quit IRC04:47
*** thorst has joined #openstack-nova04:48
*** dikonoor has joined #openstack-nova04:50
*** gjayavelu has joined #openstack-nova04:54
*** dikonoor has quit IRC04:57
*** bks has joined #openstack-nova04:57
openstackgerritedan david proposed openstack/nova: WIP: User-controlled SR-IOV ports allocation  https://review.openstack.org/37415105:00
*** dikonoor has joined #openstack-nova05:04
*** claudiub has joined #openstack-nova05:06
*** jaosorior has joined #openstack-nova05:07
*** dimtruck is now known as zz_dimtruck05:22
*** ianw has quit IRC05:23
*** prateek has joined #openstack-nova05:26
*** ianw has joined #openstack-nova05:27
*** pwnall1 has quit IRC05:33
*** ecelik has joined #openstack-nova05:34
*** pbandark has joined #openstack-nova05:40
*** thorst has quit IRC05:42
*** thorst has joined #openstack-nova05:45
*** kaisers has joined #openstack-nova05:45
*** gjayavelu has quit IRC05:48
*** dikonoor has quit IRC05:49
hussain_calaski: Hello Andrew, could you please take a look at this in your free time?  Many thanks.. https://review.openstack.org/#/c/373794/05:50
*** thorst has quit IRC05:54
*** tonytan_brb has quit IRC05:56
*** Jeffrey4l has quit IRC05:57
*** mjura has joined #openstack-nova05:58
*** ducttape_ has joined #openstack-nova06:03
*** paul-carlton2 has joined #openstack-nova06:04
*** mjura has quit IRC06:05
*** mjura has joined #openstack-nova06:05
*** rcernin has joined #openstack-nova06:07
*** ducttape_ has quit IRC06:08
*** dikonoor has joined #openstack-nova06:10
*** Jeffrey4l has joined #openstack-nova06:10
*** lpetrut has joined #openstack-nova06:14
*** gongysh has joined #openstack-nova06:16
*** moshele has joined #openstack-nova06:21
*** paul-carlton2 has quit IRC06:24
*** paul-carlton2 has joined #openstack-nova06:24
*** thorst has joined #openstack-nova06:24
*** vishwanathj has quit IRC06:25
*** Dinesh_Bhor has quit IRC06:28
*** thorst has quit IRC06:30
*** sudipto_ has joined #openstack-nova06:34
*** andreas_s has joined #openstack-nova06:35
*** sudswas__ has joined #openstack-nova06:36
*** sridharg has joined #openstack-nova06:36
openstackgerritHieu LE proposed openstack/nova: Config options: improve libvirt help text (2)  https://review.openstack.org/31655006:38
*** thorst has joined #openstack-nova06:41
*** pilgrimstack has joined #openstack-nova06:46
*** thorst has quit IRC06:46
openstackgerritClint 'SpamapS' Byrum proposed openstack/nova: Remove nova.image.s3  https://review.openstack.org/37443306:51
*** sahid has joined #openstack-nova06:53
*** jichen has joined #openstack-nova06:54
openstackgerritEric Brown proposed openstack/nova: VMware: deprecate wsdl_location conf option  https://review.openstack.org/34247006:54
*** thorst has joined #openstack-nova06:57
*** lpetrut has quit IRC07:00
*** Sukhdev has quit IRC07:02
*** ducttape_ has joined #openstack-nova07:04
*** thorst has quit IRC07:05
*** avolkov has joined #openstack-nova07:07
*** ducttape_ has quit IRC07:09
*** ratailor has quit IRC07:11
*** lpetrut has joined #openstack-nova07:12
*** pilgrimstack has quit IRC07:13
*** ratailor has joined #openstack-nova07:13
*** sridharg has quit IRC07:19
*** nherciu has joined #openstack-nova07:24
*** pbandark has quit IRC07:26
*** pbandark has joined #openstack-nova07:27
*** paul-carlton2 has quit IRC07:28
*** matrohon has joined #openstack-nova07:28
*** tuanla_ has joined #openstack-nova07:30
*** paul-carlton2 has joined #openstack-nova07:30
*** sdake has quit IRC07:30
*** tuanla has quit IRC07:31
*** jlanoux has joined #openstack-nova07:31
*** jwcroppe has quit IRC07:32
*** thorst has joined #openstack-nova07:33
*** zenoway has joined #openstack-nova07:34
*** jwcroppe has joined #openstack-nova07:36
*** thorst has quit IRC07:41
*** danpb has joined #openstack-nova07:42
*** salv-orlando has joined #openstack-nova07:42
*** pilgrimstack has joined #openstack-nova07:45
*** salv-orl_ has quit IRC07:45
openstackgerritlinbing proposed openstack/nova: * Progress_watermark is design for progress_timeout, and progress_timeout   is design for stuck of libvirt no matter progress_watermark is lesser than   data_remaining or greater than data_remaining.  https://review.openstack.org/37458707:49
*** ZZelle has quit IRC07:55
*** ZZelle has joined #openstack-nova07:55
*** lpetrut has quit IRC07:56
*** tonytan4ever has joined #openstack-nova07:57
*** namnh has joined #openstack-nova07:57
*** code-R has joined #openstack-nova07:58
*** zzzeek has quit IRC08:00
*** zzzeek has joined #openstack-nova08:00
*** tonytan4ever has quit IRC08:02
*** brian_price has quit IRC08:03
openstackgerritlinbing proposed openstack/nova: * Progress_watermark is design for progress_timeout, and progress_timeout   is design for stuck of libvirt no matter progress_watermark is lesser than   data_remaining or greater than data_remaining.  https://review.openstack.org/37458708:06
openstackgerritedan david proposed openstack/nova: WIP: User-controlled SR-IOV ports allocation  https://review.openstack.org/37415108:07
*** jamielennox is now known as jamielennox|away08:09
*** yassine has joined #openstack-nova08:10
*** yassine is now known as Guest690608:10
*** thorst has joined #openstack-nova08:11
*** thorst has quit IRC08:16
*** roeyc has joined #openstack-nova08:22
*** tuanla has joined #openstack-nova08:23
*** hoangcx_ has joined #openstack-nova08:23
*** hoangcx has quit IRC08:25
*** phuongnh has quit IRC08:25
*** namnh_ has joined #openstack-nova08:25
*** phuongnh has joined #openstack-nova08:25
*** code-R has quit IRC08:25
*** namnh has quit IRC08:25
*** tuanla_ has quit IRC08:25
*** namnh_ has quit IRC08:26
*** namnh has joined #openstack-nova08:26
openstackgerritedan david proposed openstack/nova: WIP: User-controlled SR-IOV ports allocation  https://review.openstack.org/37415108:26
*** thorst has joined #openstack-nova08:28
*** paul-carlton has joined #openstack-nova08:28
openstackgerritjaveme proposed openstack/nova: Failed to evacuate instances booted from volume  https://review.openstack.org/37204508:29
*** paul-carlton2 has quit IRC08:31
*** zenoway has quit IRC08:32
*** zenoway has joined #openstack-nova08:33
*** thorst has quit IRC08:33
sfinucanlxsli: lovely - thank you :)08:35
*** sudipto_ has quit IRC08:38
*** sudswas__ has quit IRC08:38
openstackgerritStephen Finucane proposed openstack/nova: conf: Add 'deprecated_reason' to osapi opts  https://review.openstack.org/35706108:39
sfinucanraj_singh: Any recommendations for deprecated opt reno notes?08:40
*** sudswas__ has joined #openstack-nova08:40
*** derekh has joined #openstack-nova08:41
*** I has joined #openstack-nova08:41
*** zenoway has quit IRC08:41
*** I is now known as Guest5648008:42
*** thorst has joined #openstack-nova08:44
*** alexchadin has joined #openstack-nova08:45
*** thorst has quit IRC08:51
*** gszasz has joined #openstack-nova08:57
*** shihanzhang has quit IRC09:03
*** nherciu_ has joined #openstack-nova09:03
*** shihanzhang has joined #openstack-nova09:03
*** nherciu has quit IRC09:04
*** ducttape_ has joined #openstack-nova09:06
*** zenoway has joined #openstack-nova09:09
*** lpetrut has joined #openstack-nova09:11
*** ducttape_ has quit IRC09:11
*** pilgrimstack has quit IRC09:11
*** yuntongjin has quit IRC09:12
*** pilgrimstack has joined #openstack-nova09:14
*** ByPasS has quit IRC09:15
*** yamahata has quit IRC09:17
*** jamielennox|away is now known as jamielennox09:18
*** ig0r_ has joined #openstack-nova09:21
*** thorst has joined #openstack-nova09:23
*** gongysh has quit IRC09:27
*** pilgrimstack has quit IRC09:28
*** thorst has quit IRC09:28
*** roeyc has quit IRC09:28
openstackgerritTony Xu proposed openstack/os-vif: Add oslo.concurrency to requirements  https://review.openstack.org/37452409:29
* bauzas is always amazed to see features he wasn't knowing like LVM backed images for our instances... https://bugs.launchpad.net/nova/+bug/162370309:31
openstackLaunchpad bug 1623703 in OpenStack Compute (nova) "libvirt: hard reboot incorrectly create nova/instances/<uuid>/<uuid>_disk with lvm backend" [Undecided,New] - Assigned to vu tran (vu-tran)09:31
bauzaskashyap: heard of ^ ?09:31
openstackgerritStephen Finucane proposed openstack/nova: conf: Improve consistency in scheduler opts  https://review.openstack.org/34966609:31
openstackgerritStephen Finucane proposed openstack/nova: conf: Group scheduler options  https://review.openstack.org/34995909:31
openstackgerritStephen Finucane proposed openstack/nova: Remove support for deprecated driver import  https://review.openstack.org/33976009:31
openstackgerritStephen Finucane proposed openstack/nova: conf: Deprecate 'scheduler_json_file_location'  https://review.openstack.org/37412109:31
* kashyap clicks09:31
openstackgerritStephen Finucane proposed openstack/nova: conf: Attempt to resolve TODOs in scheduler.py  https://review.openstack.org/37464709:31
bauzaskashyap: not the bug, the feature09:32
kashyapbauzas: Yes, I've heard of the feature, but haven't tried it myself09:33
bauzasI wasn't knowing it was possible to use an LVM VG for that09:33
openstackgerritStephen Finucane proposed openstack/nova: conf: Attempt to resolve TODOs in scheduler.py  https://review.openstack.org/37464709:33
*** cdent has joined #openstack-nova09:33
*** mvk has quit IRC09:35
*** alexchadin has quit IRC09:36
*** yuntongjin has joined #openstack-nova09:37
*** salv-orlando has quit IRC09:37
*** salv-orlando has joined #openstack-nova09:37
bauzaskashyap: given _create_images_and_backing() is used by both hard_reboot and pre-live-mig, I guess the problem above is related to both09:38
bauzaskashyap: because in that method, we check the backing file and if not, create it, insteaf of checking the imagebackend, right?09:39
*** thorst has joined #openstack-nova09:40
kashyapbauzas: One moment, need to context-switch09:40
bauzasno worries09:40
*** code-R has joined #openstack-nova09:41
bauzaskashyap: going to take my daughter at the school, but I left a commetn09:41
kashyapbauzas: Indeed, they're both used in _hard_reboot() pre_live_migration() methods.  Need to investigate it further.09:42
kashyapSure, take your time.  I'm also in the middle of another bug09:42
bauzaskashyap: no hard call, just wanted a second eye09:42
*** alexchadin has joined #openstack-nova09:42
bauzaskashyap: thanks09:42
*** code-R_ has joined #openstack-nova09:43
*** thorst has quit IRC09:44
*** yuntongjin has quit IRC09:45
*** alexchadin has quit IRC09:46
openstackgerritTony Xu proposed openstack/os-vif: Add oslo.concurrency to requirements  https://review.openstack.org/37452409:47
*** code-R has quit IRC09:47
*** mkoderer has joined #openstack-nova09:47
*** bdeschenes has joined #openstack-nova09:50
paul-carltonjohnthetubaguy, bauzas, et al about to start work on https://bugs.launchpad.net/nova/+bug/1626230 any thoughts on my proposed approach?09:50
openstackLaunchpad bug 1626230 in OpenStack Compute (nova) "evacuate leaves instance on target compute node if it fails to spawn" [Undecided,New] - Assigned to Paul Carlton (paul-carlton2)09:50
*** zenoway has quit IRC09:51
openstackgerritBéla Vancsics proposed openstack/nova: Reduced the complexity of the _init_instance method  https://review.openstack.org/37466109:51
*** zenoway has joined #openstack-nova09:51
*** markvoelker has joined #openstack-nova09:52
johnthetubaguypaul-carlton: it sounds like we need more of the rollback logic from spawn in the rebuild case09:53
paul-carltonthe rebuild is calling spawn in the driver09:53
*** jichen has quit IRC09:53
johnthetubaguyhmm, so was does boot an instance do for the same case?09:54
paul-carltonjohnthetubaguy, we saw this scenario internally and I recreated by raising an exception in driver spawn09:54
paul-carltonI think boot works as would normal rebuild because the instance host is set to the host it is booting/rebuilding on09:55
paul-carltonjohnthetubaguy, so when it puts it into error state it is ok that the instance has files in instances dir and is maybe even running09:55
johnthetubaguyOK, I am missing how the libvirt driver cleans up I guess09:55
*** thorst has joined #openstack-nova09:56
johnthetubaguythe XenAPI driver uses an undo manager to make sure any failures don't leave files behind: https://github.com/openstack/nova/blob/master/nova/virt/xenapi/vmops.py#L61309:56
*** markvoelker has quit IRC09:56
paul-carltonjohnthetubaguy, I'm not sure it does any clean up, the point is that if a boot or rebuild fails the instance gets placed in error state but in the case of a rbuild as part of evacuate it still thinks the instance in on the source node09:57
*** tonytan4ever has joined #openstack-nova09:58
johnthetubaguyah, so its in an ERROR state, but he host is set wrongly, so we dunno where to clean up?09:59
paul-carltonjohnthetubaguy, my assessment of the libvirt driver spawn is it does not do any cleanup, if an exception occurs during spawning it raises an exception and the compute manager places the instance in error state10:01
johnthetubaguyI think I am leaning towards your solution being the simplest right now10:01
johnthetubaguyyeah, thats a shame, I guess we should keep that consistent for now10:01
johnthetubaguywe did a lot of work in XenAPI driver to make sure there are almost no cases you are left with manual cleanup when you go to an ERROR state, wasted us hours of time sorting those out10:02
johnthetubaguys/hours/man years/10:02
paul-carltonjohnthetubaguy, at that point it is down to the operator to fix the instance with a reset-state and retry of boot/rebuild10:02
*** tonytan4ever has quit IRC10:02
*** thorst has quit IRC10:04
paul-carltonjohnthetubaguy, the XenAPI approach is good but I guess the libvirt drivers approach is that it is ok for an instance to be present in libvirt etc if it fails to spawn, the operator will either fix it and reboot/rebuild or delete it10:04
openstackgerritedan david proposed openstack/nova: Use os.path.join instead of string concatenation  https://review.openstack.org/37466510:04
johnthetubaguypaul-carlton: yeah, I guess delete is the cleanup10:04
johnthetubaguypaul-carlton: so I am agreed with your approach10:05
johnthetubaguyto fix this bug10:05
*** EinstCrazy has quit IRC10:05
johnthetubaguysounds like what should have happened in the first place10:05
paul-carltonjohnthetubaguy, the problem with evacuate is we are rebuilding the instance on target but it still thinks it is on source10:05
*** EinstCrazy has joined #openstack-nova10:05
paul-carltonjohnthetubaguy, Thanks, will start work on this now10:05
*** zenoway has quit IRC10:05
*** matrohon has quit IRC10:05
*** EinstCrazy has quit IRC10:05
*** mvk has joined #openstack-nova10:06
*** EinstCrazy has joined #openstack-nova10:06
*** zenoway has joined #openstack-nova10:11
*** EinstCrazy has quit IRC10:11
*** roeyc has joined #openstack-nova10:15
*** thorst has joined #openstack-nova10:16
*** hoangcx_ has quit IRC10:16
*** matrohon has joined #openstack-nova10:18
*** edand has joined #openstack-nova10:18
*** zenoway has quit IRC10:19
mdboothpaul-carlton: Just reading that bug, I agree. Are you thinking of setting instance.host in the same save which atomically sets task_state to REBUILDING?10:19
mdboothThat looks like the point at which we switch from doing meta stuff to real stuff.10:20
openstackgerritStephen Finucane proposed openstack/nova: conf: Deprecate all topic options  https://review.openstack.org/35160810:20
openstackgerritStephen Finucane proposed openstack/nova: conf: Trivial cleanup of 'cells'  https://review.openstack.org/37467210:20
openstackgerritStephen Finucane proposed openstack/nova: conf: Trivial cleanup of compute.py  https://review.openstack.org/37467310:20
openstackgerritStephen Finucane proposed openstack/nova: conf: Trivial cleanup of console.py  https://review.openstack.org/37467410:20
*** thorst has quit IRC10:20
*** zenoway has joined #openstack-nova10:20
*** bdeschenes has quit IRC10:24
openstackgerritjaveme proposed openstack/nova: Failed to evacuate instances booted from volume  https://review.openstack.org/37204510:27
paul-carltonmdbooth, hadn't looked at it yet, about to start, I was thinking that once the target has issued the claim we are good to set the host10:28
mdboothpaul-carlton: Yeah, I think that's consistent.10:28
paul-carltonmdbooth, about to take a look, got distracted by one of our QA guys asking unrelated question10:28
mdboothThe host switch needs to be atomic, though.10:29
mdboothIn practise that means it needs to be concurrent with an atomic change, or subsequent.10:29
*** zenoway has quit IRC10:29
mdboothAnd I think the above change to task_state is the first atomic change you can hook on to.10:29
paul-carltonmdbooth, that way, if something goes wrong at that point it has effectively evacuated but failed to start so down to the operator to try and fix it on that node or move it to another node10:30
mdboothIncidentally, you might want to run this by a scheduler person. I have a vague recollection that setting the host at the end has a specific reason, and it might be schedulary.10:30
mdboothpaul-carlton: Yup, that sounds like the best way to do it to me.10:31
paul-carltonmdbooth, will do, Jay the best person?10:31
mdboothI think the point I mention above achieves that.10:31
mdboothbauzas: ^^^ ?10:31
mdboothpaul-carlton: Git history too, I guess.10:31
*** thorst has joined #openstack-nova10:32
*** oanson has quit IRC10:33
paul-carltonmdbooth, ta10:34
openstackgerritJay Pipes proposed openstack/nova: placement: handle generic PCI devices in inventory  https://review.openstack.org/37468110:34
*** zenoway has joined #openstack-nova10:34
paul-carltonjaypipes, any thoughts on above discussion re https://bugs.launchpad.net/nova/+bug/162623010:35
openstackLaunchpad bug 1626230 in OpenStack Compute (nova) "evacuate leaves instance on target compute node if it fails to spawn" [Undecided,In progress] - Assigned to Paul Carlton (paul-carlton2)10:35
openstackgerritKobi Samoray proposed openstack/nova: Fix typo in docsting in test_migrations.py  https://review.openstack.org/37468210:38
johnthetubaguyI would check with dansmith, he has looked very closely at evacuate recently10:38
mdboothpaul-carlton: https://bugs.launchpad.net/nova/+bug/141766710:40
openstackLaunchpad bug 1417667 in OpenStack Compute (nova) "migration/evacuation/rebuild/resize of instance with dedicated cpus needs to recalculate cpus on destination" [Medium,In progress] - Assigned to sahid (sahid-ferdjaoui)10:40
mdboothpaul-carlton: Specifically https://review.openstack.org/#/c/218385/10:40
*** thorst has quit IRC10:40
mdboothpaul-carlton: Nope, that's the wrong review, sorry10:42
* mdbooth finds the right one10:42
mdboothThis one: https://review.openstack.org/#/c/200485/10:43
*** zenoway has quit IRC10:44
paul-carltonmdbooth, looking now, ta10:45
paul-carltonjohnthetubaguy, will do10:45
mdboothpaul-carlton: Yup, that's the one. I don't fully understand why it's apparently important to RT that host is updated at the end not at the start, but that change is very explicit about it.10:45
*** kberger has quit IRC10:47
*** kberger has joined #openstack-nova10:48
*** sdague has joined #openstack-nova10:48
*** zenoway has joined #openstack-nova10:49
openstackgerritStephen Finucane proposed openstack/nova: conf: Group scheduler options  https://review.openstack.org/34995910:52
*** markvoelker has joined #openstack-nova10:53
*** zenoway has quit IRC10:53
cdenttonyb: thanks for chasing that issue with the pypi mirros10:53
*** zenoway has joined #openstack-nova10:53
tonybcdent: np.10:53
sdaguemorning folks10:55
openstackgerritStephen Finucane proposed openstack/nova: conf: Deprecate all topic options  https://review.openstack.org/35160810:56
openstackgerritStephen Finucane proposed openstack/nova: conf: Trivial cleanup of 'cells'  https://review.openstack.org/37467210:56
openstackgerritStephen Finucane proposed openstack/nova: conf: Trivial cleanup of compute.py  https://review.openstack.org/37467310:56
openstackgerritStephen Finucane proposed openstack/nova: conf: Trivial cleanup of console.py  https://review.openstack.org/37467410:56
openstackgerritPaul Carlton proposed openstack/nova: Add logging to NumaTopology Filter  https://review.openstack.org/37469210:56
*** markvoelker has quit IRC10:57
tonybsdague: wait morning already?11:02
*** GB21 has joined #openstack-nova11:05
*** vladikr has quit IRC11:06
*** hoonetorg has joined #openstack-nova11:06
openstackgerritStephen Finucane proposed openstack/nova: Improve help text for glance options  https://review.openstack.org/32400711:07
openstackgerritStephen Finucane proposed openstack/nova: conf: Cleanup of glance.py  https://review.openstack.org/37470111:07
*** thorst has joined #openstack-nova11:08
*** ducttape_ has joined #openstack-nova11:08
openstackgerritStephen Finucane proposed openstack/nova: conf: Attempt to resolve TODOs in scheduler.py  https://review.openstack.org/37464711:09
openstackgerritStephen Finucane proposed openstack/nova: conf: Deprecate 'scheduler_json_file_location'  https://review.openstack.org/37412111:09
*** smatzek has joined #openstack-nova11:09
*** dikonoor has quit IRC11:10
sdaguetonyb: :)11:11
*** ducttape_ has quit IRC11:12
*** pilgrimstack has joined #openstack-nova11:12
*** rtheis has joined #openstack-nova11:13
*** eantyshev has joined #openstack-nova11:15
*** thorst has quit IRC11:16
*** zenoway has quit IRC11:18
openstackgerritStephen Finucane proposed openstack/nova: conf: Move consoleauth options to a group  https://review.openstack.org/35661111:18
*** zenoway has joined #openstack-nova11:18
*** lucas-afk is now known as lucasagomes11:18
*** ihrachys has joined #openstack-nova11:18
*** jistr is now known as jistr|mtg11:20
openstackgerritJiWei proposed openstack/python-novaclient: Using assertIsNone() instead of assertIs(None, ..)  https://review.openstack.org/37472011:20
*** dikonoor has joined #openstack-nova11:21
*** bvanhav has joined #openstack-nova11:22
*** zenoway has quit IRC11:27
*** Guest56480 has quit IRC11:27
*** sudswas__ has quit IRC11:27
*** bdeschenes has joined #openstack-nova11:29
*** ratailor has quit IRC11:34
openstackgerritedan david proposed openstack/nova: Use os.path.join instead of string concatenation  https://review.openstack.org/37466511:35
*** code-R_ has quit IRC11:35
*** code-R has joined #openstack-nova11:36
*** takedakn has joined #openstack-nova11:37
*** prateek has quit IRC11:38
*** thorst has joined #openstack-nova11:43
*** dikonoor has quit IRC11:46
openstackgerritSean Dague proposed openstack/nova: Make test logging setup fixture disable future setup  https://review.openstack.org/37473311:50
openstackgerritSean Dague proposed openstack/nova: Don't print to stdout when executing hacking checks  https://review.openstack.org/37473411:50
*** takedakn has quit IRC11:50
*** markvoelker has joined #openstack-nova11:50
*** thorst has quit IRC11:52
*** coolsvap has quit IRC11:52
ftersinmdbooth: hi. i published POC of ephemeral resize: https://review.openstack.org/#/c/372721/ Look at this if it is interested for you11:54
*** zigo has quit IRC11:54
mdboothftersin: Definitely interested. Looking now.11:55
*** code-R has quit IRC11:56
*** code-R_ has joined #openstack-nova11:56
mdboothftersin: That hyper-v logic is interesting, btw. Namely resize->0 means delete.11:57
mdboothThat's generally supportable, imho, and not a design choice the driver should make.11:58
*** baoli has joined #openstack-nova11:58
mdboothMaybe out of scope for your patch, but I think we could push that logic up into compute manager.11:58
*** zigo has joined #openstack-nova11:58
*** phuongnh has quit IRC11:59
*** zigo is now known as Guest8360111:59
*** baoli_ has joined #openstack-nova11:59
*** edmondsw has joined #openstack-nova12:02
*** baoli has quit IRC12:03
*** dikonoor has joined #openstack-nova12:03
*** namnh has quit IRC12:03
*** Guest83601 has quit IRC12:03
*** psilvad has joined #openstack-nova12:04
*** lbeliveau has joined #openstack-nova12:04
*** tuanla has quit IRC12:06
*** porunov has joined #openstack-nova12:08
*** ducttape_ has joined #openstack-nova12:08
*** ducttape_ has quit IRC12:09
*** ducttape_ has joined #openstack-nova12:09
*** salv-orlando has quit IRC12:10
*** salv-orlando has joined #openstack-nova12:10
*** zigo_ has joined #openstack-nova12:12
openstackgerritPaul Carlton proposed openstack/nova: Add logging to NumaTopology Filter  https://review.openstack.org/37469212:13
*** psilvad has quit IRC12:15
*** zigo_ has quit IRC12:17
*** maticue has joined #openstack-nova12:17
*** zigo_ has joined #openstack-nova12:18
*** thorst has joined #openstack-nova12:19
openstackgerritjaveme proposed openstack/nova: Failed to evacuate instances booted from volume  https://review.openstack.org/37204512:20
ftersinmdbooth: sorry for the pause - i was having lunch. More interesting is that hyperv recreates ephemerals, not resizes them. I strongly doubt this is good.12:20
mdboothftersin: Eesh, yeah.12:20
mdboothARGH! GERRIT!12:20
openstackgerritPaul Carlton proposed openstack/nova: Add logging to NumaTopology Filter  https://review.openstack.org/37469212:21
mdboothI definitely logged in to gerrit this morning, however, on attempting to save that non-trivial comment I discover that I have been logged out at some point during the day.12:21
mdboothI now have the "Code Review - Session Expired" screen of DOOM12:21
mdboothBecause anything I do now means I lose my comment12:22
mdboothI can still see it. I could take a picture of it on my phone and type it back out again afterwards.12:22
*** alee has quit IRC12:23
*** rodrigods has quit IRC12:24
*** rodrigods has joined #openstack-nova12:24
ftersinmdboth: you probably can delete umbrella layer on the page and get into edit mode back?12:25
ftersinvia your browser developer tools12:26
*** thorst has quit IRC12:28
*** mdrabe has joined #openstack-nova12:29
ftersinmdbooth: as for the resize - i noted in the commit message that we might make a common check12:29
mdboothftersin: Good idea for next time on the browser dev tools, btw.12:30
openstackgerritSean Dague proposed openstack/nova: Work around tests that don't use nova.test as a base  https://review.openstack.org/37476112:30
*** armax has joined #openstack-nova12:30
openstackgerritSean Dague proposed openstack/nova: Don't print to stdout when executing hacking checks  https://review.openstack.org/37473412:32
openstackgerritSean Dague proposed openstack/nova: Work around tests that don't use nova.test as a base  https://review.openstack.org/37476112:32
*** tbachman has joined #openstack-nova12:33
ftersinmdbooth: since this dance with bdm is common as well, it must work with any virt driver. So that if resizing down (including to zero) is really correct for hyperv, that code must honor this.12:34
mdboothftersin: Right. I think this could be an opportunity to make this common across drivers, though.12:34
mdboothi.e. We could give libvirt the same behaviour at the same time.12:35
*** annegentle has joined #openstack-nova12:35
mdboothResize to zero, that is.12:35
*** tbachman has quit IRC12:35
*** sridharg has joined #openstack-nova12:35
*** jistr|mtg is now known as jistr12:36
*** roeyc has quit IRC12:37
ftersinmdbooth: It means that at the first we should decide what are the legal common use cases, the legal use cases with custom virt driver, and if the difference between common and custom behavoiurs exists, define how and where to handle it (rather than let a driver throw an error at the end of resizing, corrupting the instance with error state).12:41
ftersinmdbooth: as the result it seems that a spec is required for that :(12:42
mdboothftersin: I think a spec might be required anyway, although personally I think your 10 page commit message is way better than a spec :)12:43
*** Guest56480 has joined #openstack-nova12:43
*** sudswas__ has joined #openstack-nova12:43
*** rmart04 has joined #openstack-nova12:44
*** ducttape_ has quit IRC12:46
openstackgerritGábor Antal proposed openstack/nova: Use os.path.join instead of string concatenation  https://review.openstack.org/37466512:47
*** roeyc has joined #openstack-nova12:48
*** burt has joined #openstack-nova12:48
*** zigo_ has quit IRC12:48
*** ZZelle has quit IRC12:49
*** tblakes has joined #openstack-nova12:50
*** zigo_ has joined #openstack-nova12:51
*** armax has quit IRC12:51
kashyapsdague: Following up from the mailing list, want to do the final honour on this?  https://review.openstack.org/#/c/32762412:53
ftersinmdbooth: anyway i do not think this has a chance to be merged until Q release. This is a pretty big change, but core reviews are overloaded with priority features. The only chance is that feature becomes priority...12:54
*** armax has joined #openstack-nova12:56
sdaguekashyap: +A12:56
kashyapsdague: Wonderful!12:57
*** david-lyle has joined #openstack-nova12:57
ftersinmdbooth: btw about 10 pages - have you seen there a note about a bug with flat backend? Is it an incorrect behavior indeed? As i understand it may affect security12:58
mdboothftersin: I think I wrote that note :)12:58
mdboothftersin: I'm a *big* fan of *big* comments and commit messages :)12:58
mdboothYour commit message is awesome.12:59
*** thorst_ has joined #openstack-nova12:59
*** efried has joined #openstack-nova12:59
mdboothftersin: Commented, btw. I like it, but might scale it back a little.12:59
*** tonytan4ever has joined #openstack-nova12:59
mdboothAnd I think you need to rethink what happens when you delete a bdm.13:00
*** amotoki has joined #openstack-nova13:00
ftersinahh, you've just commented there. thanks13:01
openstackgerritSean Dague proposed openstack/nova: Standardize output capture for nova-manage tests  https://review.openstack.org/37479913:02
macszcan anyone tell me how to build docs locally? I added reno to one of my patches and it fails on gate-nova-releasenotes, i would like to test whats wrong locally but it seems that tox is not the way to go13:03
sdaguemacsz: tox -e releasenotes13:03
sdaguethat will build the release notes13:03
macszoh, so it is tox, thanks!13:04
sdaguemacsz: is there a patch in question up there? I can look to see if there is any quick advice13:04
openstackgerritChris Dent proposed openstack/nova: [placement] ensure that allow headers are bytes  https://review.openstack.org/37480013:04
*** thorst_ has quit IRC13:04
*** tonytan4ever has quit IRC13:05
macszsdague: it is an ultimate nit, it seems, but here is the patch: https://review.openstack.org/#/c/351608/13:05
hussain_calaski: Thanks for reviewing the patch :)13:05
mriedemmacsz: the problem is it's not rst formatted13:05
mriedemif you have bullets you need to format it13:06
mriedemmacsz: see http://docs.openstack.org/developer/reno/usage.html#formatting13:06
mriedemyou need the | to format it like rst13:06
mriedem- | that is13:06
sdagueright, otherwise it's interpretted as yaml13:06
sdaguewhich it's not valid yaml13:07
macszoh, tat's why, i appreciate your attention and thank you :)13:07
openstackgerritLudovic Beliveau proposed openstack/nova: Don't parse PCI whitelist eveytime neutron ports are created  https://review.openstack.org/33723613:07
cdentalex_xu: that bug you just submitted about 405 leading to 500, I just fixed: https://review.openstack.org/37480013:08
*** coolsvap has joined #openstack-nova13:08
sdaguemriedem: I've been letting my ocd flag free and working on a series of patches that cleanup output handling in the test suites - https://review.openstack.org/#/c/374733/13:08
sdagueand on up13:08
sdaguethey should be reasonably simple to review13:08
alex_xucdent: oops, thanks13:09
mnaserfor liberty to mitaka upgrades, am i correct to assume that "nova-manage api_db sync" will move the tables specific to api_db from the old db to the new one?13:09
*** GB21 has quit IRC13:09
alex_xucdent: one hour ago :) I almost done that13:10
*** GB21 has joined #openstack-nova13:10
cdent:)13:10
cdentalex_xu: i've been spending the morning running some requests by hand and recording issues I've found, created a few other bugs under the placement tag13:11
alex_xucdent: I'm try encode all the headers at https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/requestlog.py#L6213:11
cdentalex_xu: I don't think that's the right place to do it, if at all.13:11
*** rlrossit has joined #openstack-nova13:11
alex_xucdent: I' think about that can avoid us make similar mistake in the future13:12
*** eharney has quit IRC13:12
alex_xus/I'/i'm/13:12
*** jaosorior has quit IRC13:12
cdentI think webob handles it in most places, and if it does not, then we should put it is in its own middleware, not the request log13:12
mriedemmnaser: it might create new tables in the api db,13:12
mriedemit won't drop old tables13:12
*** jaosorior has joined #openstack-nova13:13
cdentthe reason the 405 stuff is a problem is because it is a special hack to work around Routes not providing the functionality.13:13
mriedemmnaser: we do online data migrations from the old table to the new table in nova-manage db online_db_migrations (or whatever it's called)13:13
alex_xucdent: yea, agree to put it in request log is a little strange, it just because it is the last point for now. I just want to avoid us do this like in glance https://review.openstack.org/#/c/11716/5/glance/api/v1/images.py13:13
cdentalex_xu: what I should probably do is just change wsgi-intercept to behave like od-wsgi, and then the gabbi tests will catch this sort of thing13:14
mnasermriedem, oh i see, we're doing a test upgrade from liberty to mitaka soon and hopefully tht goes smoothly13:14
edleafecdent: can you educate me on the use of iso-8859-1 instead of utf8?13:14
mriedemsdague: i'm sorry the commit message titles need to be full sentences, starting with a capital and ending with a period13:14
alex_xucdent: yea, that will be cool13:14
sdaguemriedem: (╯°□°)╯︵ ┻━┻13:14
cdentedleafe: pep3333 requires header values to be encode as iso-blah13:15
cdentwhich is weird13:15
cdentbut that's the way it goes13:15
*** hemanthm|away is now known as hemanthm13:15
edleafecdent: ok, I figured that there was a reason13:15
*** gouthamr has joined #openstack-nova13:15
cdentit's an artifact of the http spec itself13:16
cdentmany people do in fact encode everything as utf-8 and it all works just fine13:16
openstackgerritDiana Clarke proposed openstack/nova: Fix database poison warnings, part 14  https://review.openstack.org/34753113:16
*** GB21 has quit IRC13:16
*** vladikr has joined #openstack-nova13:16
*** code-R has joined #openstack-nova13:16
cdentalex_xu: re that stuff from glance: I actually think that level of explicitness is a good thing13:17
mriedemsdague: where were you seeing this? https://review.openstack.org/#/c/374734/2/nova/tests/unit/test_hacking.py13:17
mriedemjust running pep8?13:17
mriedemor only if it fails?13:17
*** jheroux has joined #openstack-nova13:17
cdentyes, it is boilerplate, but at least it is clear13:17
sdaguemriedem: running the custom hacking tests13:17
mriedemoh right py2713:17
mriedemderp13:17
sdaguehttp://logs.openstack.org/33/374733/1/check/gate-nova-python27-db-ubuntu-xenial/470a7fd/console.html#_2016-09-22_12_04_06_70661713:18
sdagueit gets rid of artifacts like those13:18
mriedemok i see, you're just going through all of the captured stdout things in the unit test runs13:20
mriedemand going hog wild13:20
*** woodster_ has joined #openstack-nova13:20
mriedemremember to touch the door knob 5 times before you leave your office13:20
*** code-R_ has quit IRC13:20
*** zz_dimtruck is now known as dimtruck13:20
alex_xucdent: ok, i'm ok with that. also gibi can catch that kind of mistake, then it will help us catch that kind of mistake. otherwise it is really a thing very easy forget by reviewer.13:20
mriedemelse the monsters will eat you13:20
alex_xus/also gibi/also hope gibi/13:20
sdaguemriedem: yeh, well some times a little spring cleaning is in order13:20
cdentalex_xu: yah13:20
openstackgerritMaciej Szankin proposed openstack/nova: conf: Deprecate all topic options  https://review.openstack.org/35160813:21
mriedemsdague: while yo'ure on this tear, i think i opened a bug for placement functional tests that aren't using some of our fixtures13:21
*** armax has quit IRC13:21
sdaguemriedem: ok, link?13:22
*** baoli_ has quit IRC13:22
mriedemsdague: https://bugs.launchpad.net/nova/+bug/162357313:22
openstackLaunchpad bug 1623573 in OpenStack Compute (nova) "placement API functional test fixtures do not do appropriate stdout and stderr handling" [Low,In progress] - Assigned to Chris Dent (cdent)13:22
sdaguethe patch after the hacking one handles this gorp - http://logs.openstack.org/33/374733/1/check/gate-nova-python27-db-ubuntu-xenial/470a7fd/console.html#_2016-09-22_12_03_03_28475713:22
*** cmartin has joined #openstack-nova13:22
*** baoli has joined #openstack-nova13:22
cdentmriedem, sdague the fix for that one is good to go: https://review.openstack.org/#/c/374256/13:23
*** hshiina has joined #openstack-nova13:23
sdaguecdent: yep, that looks reasonable to me13:23
sdague+213:23
*** tjones has joined #openstack-nova13:24
*** READ10 has joined #openstack-nova13:24
mriedem+W13:25
*** baoli has quit IRC13:27
mriedemsdague: hmm i still see 2 http://logs.openstack.org/61/374761/2/check/gate-nova-python27-db-ubuntu-xenial/343f8c1/console.html#_2016-09-22_12_43_03_33239413:28
mriedemneed to also do nova.tests.unit.db.test_db_api ?13:29
sdaguemriedem: we might13:29
sdagueyou want it in this patch, or another one?13:29
*** eharney has joined #openstack-nova13:30
*** ayoung has quit IRC13:30
openstackgerritMaciej Szankin proposed openstack/nova: Removed empty reno note  https://review.openstack.org/37481713:30
mriedemi'm fine with a later one13:30
mriedemgotta walk to the bus stop, brb13:31
*** alee has joined #openstack-nova13:33
openstackgerritMaciej Szankin proposed openstack/nova: Remove redundant check from code  https://review.openstack.org/35012213:34
*** thorst_ has joined #openstack-nova13:35
*** cleong has joined #openstack-nova13:35
*** lamt has joined #openstack-nova13:35
dansmithjohnthetubaguy: you wanna hit this bottom patch so the ones above can merge and I'll follow up with translations? https://review.openstack.org/#/c/373361/213:36
*** psachin has quit IRC13:36
*** smatzek has quit IRC13:36
cdentsdague or alex_xu: when you get a moment I could do with your brain with regard to https://bugs.launchpad.net/nova/+bug/162649313:36
openstackLaunchpad bug 1626493 in OpenStack Compute (nova) "placement api log entries have mismatched request ids" [Undecided,New] - Assigned to Chris Dent (cdent)13:36
johnthetubaguydansmith: I wasn't sure about those bits that need translating, but we could totally do that in a follow up patch13:36
cdentthe problem is somewhat more complex than my initial analysis in the bug report indicates: because oslo_log will use the context and because the local context doesn't go out of scope, the context from the last request is being used to write the initial starting log for the next request13:37
John341Can anyone advice on why my first volume is named /dev/vda (image parameters: hw_disk_bus: scsi, hw_scsi_model: virtio-scsi), then when I create a second volume and try to do nova volume-attach it fails adding it as /dev/sda. If I shutdown the instances, I can attach it, but it fails to boot with the /dev/vda and /dev/sda. This is on Liberty/CentOS7.13:38
*** READ10 has quit IRC13:38
sdaguecdent: oh, because the logging exists in middleware before the context is established?13:39
cdenta fix is to move request_log inside the request_id, auth and context middlewares here: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/deploy.py#L52 or perhaps something else I'm not aware of? alex_xu, sdague : your thoughts?13:39
dansmithjohnthetubaguy: ah, I thought you +1'd the translation comment13:39
dansmithjohnthetubaguy: we translate other messages in manage, so I assume we should here13:39
dansmithI just didn't think about it13:39
cdentsdague: yeah, pretty much13:39
sdaguecdent: well in the compute api, the logging is don't in the main code. I think that's basically the pattern that other projects are following.13:40
johnthetubaguydansmith: yeah, I think we probably should mark those for translation13:40
*** READ10 has joined #openstack-nova13:40
*** thorst_ has quit IRC13:40
cdentsdague: my concern is that the request log was raised to the outside of the middleware on purpose: so that any middleware exceptions get logged too and the entire interaction is within the start/end boundary of request log13:40
cdentbut if doing it "within" is okay, then I'll just go ahead and make the fix (it is pretty much already done) but we might want to backport it?13:41
*** burt has quit IRC13:42
*** salv-orl_ has joined #openstack-nova13:43
*** mgoddard__ has joined #openstack-nova13:43
*** salv-orlando has quit IRC13:46
*** PaulMurray has joined #openstack-nova13:46
*** mtanino has joined #openstack-nova13:46
*** ducttape_ has joined #openstack-nova13:46
openstackgerritGleb Stepanov proposed openstack/nova: Skip malformed cookies  https://review.openstack.org/34609013:46
*** mgoddard_ has quit IRC13:46
openstackgerritMaciej Szankin proposed openstack/nova: conf: Deprecate all topic options  https://review.openstack.org/35160813:47
*** paul-carlton has quit IRC13:48
*** code-R has quit IRC13:48
*** code-R has joined #openstack-nova13:48
*** jamesdenton has joined #openstack-nova13:51
*** ducttape_ has quit IRC13:51
*** thorst_ has joined #openstack-nova13:51
*** ducttape_ has joined #openstack-nova13:52
*** tanvirt has joined #openstack-nova13:52
openstackgerritChris Dent proposed openstack/nova: [placement] reorder middleware to correct logging context  https://review.openstack.org/37483313:53
*** takedakn has joined #openstack-nova13:54
gibicdent, alex_xu: I'm trying to find out what mistake shall I catch. I highlighted 30 minutes ago but I don't get it.13:54
tdurakovhello everyone13:54
* tdurakov back from vacation13:54
cdentgibi: I think alex_xu meant gabbi13:54
alex_xugibi: sorry about that...13:54
gibicdent: that would make more sense :)13:54
tdurakovwho could clarify things about https://bugs.launchpad.net/python-novaclient/+bug/1522402 ?13:54
openstackLaunchpad bug 1522402 in python-novaclient "Novaclient does not support v3 Keystone API" [High,In progress]13:54
tdurakovoh, wrong bug13:55
cdentsdague or alex_xu if you have some suggestions on how best to test https://review.openstack.org/#/c/374833/ that would be most welcome13:55
tdurakovhttps://bugs.launchpad.net/nova/+bug/1582684 - why was it fixed that way?13:55
openstackLaunchpad bug 1582684 in OpenStack Compute (nova) mitaka "nova kilo->liberty ceph configdrive upgrade fails" [High,Fix committed] - Assigned to melanie witt (melwitt)13:55
*** burgerk has joined #openstack-nova13:56
tdurakovmelwitt: hi, are you around?13:56
*** thorst_ has quit IRC13:56
*** shewless has joined #openstack-nova13:57
*** dane-fichter has joined #openstack-nova13:57
alex_xucdent: will try to think about that13:57
dane-fichterthe meeting is in #openstack-meeting right?13:57
*** pilgrimstack has quit IRC13:57
openstackgerritChris Dent proposed openstack/nova: [placement] reorder middleware to correct logging context  https://review.openstack.org/37483313:57
sdagueyeh13:57
cdentthanks alex_xu13:57
bauzasmriedem: just right before we start the meeting, are we planning a RC2 soon ?13:57
alex_xucdent: np13:57
*** takashin has joined #openstack-nova13:58
mriedembauzas: next week13:58
bauzask13:58
mriedemafter translations are done13:58
bauzasI need to have more time for looking at the bugs13:58
tdurakovmriedem: hi, do you have time to discuss one bug?13:58
sdaguemriedem: are you actually around for the meeting? or you still want me to run it13:58
*** umbSublime has joined #openstack-nova13:58
*** takedakn has quit IRC13:58
*** mgoddard__ has quit IRC13:59
*** ducttape_ has quit IRC14:00
*** matrohon has quit IRC14:00
mriedemsdague: i have to leave early14:00
mriedemso you run it14:00
mriedemhmm wtf https://review.openstack.org/#/c/373292/1/requirements.txt14:00
*** tonytan4ever has joined #openstack-nova14:00
*** r-daneel has joined #openstack-nova14:01
*** ecelik has quit IRC14:01
*** bks has quit IRC14:02
cdentmriedem: that's the more better name14:02
mriedemcdent: that's not what's in g-r though14:02
mriedemon master or stable/newton14:02
mriedemso i'm confused why the sync bot is proposing that14:03
*** takedakn has joined #openstack-nova14:03
mriedemtdurakov: if it's short, i have to leave soon14:03
mriedemplus i'm watching the nova meeting14:03
cdentmriedem it may also be a typo14:03
*** pbandark has quit IRC14:03
mriedembut, it's a bot14:03
cdent(from the global reqs change)14:03
*** sridharg has quit IRC14:03
cdentand nobody noticed because they both work14:03
tdurakovmriedem: why we fallback here https://review.openstack.org/#/c/317785/5/nova/virt/libvirt/driver.py - instead of creation rbd?14:04
openstackgerritDan Smith proposed openstack/nova: Add blocker migration to ensure for newton online migrations  https://review.openstack.org/37430714:04
openstackgerritDan Smith proposed openstack/nova: Add placeholder DB migrations for Ocata  https://review.openstack.org/37427014:04
openstackgerritDan Smith proposed openstack/nova: Remove PCI parent_addr online migration  https://review.openstack.org/37351914:04
openstackgerritDan Smith proposed openstack/nova: Make nova-manage online migrations more verbose  https://review.openstack.org/37336114:04
mriedemtdurakov: you'd have to read the bug report for history there, or see if melwitt remembers details, but it was something time warner hit when upgrading from kilo i think and there was a regression14:05
*** tonytan4ever has quit IRC14:05
tdurakovmriedem: it's still valid: changing force_config_drive to True on ceph envs causes same issue, and this fallback doesn't help14:06
*** sdake has joined #openstack-nova14:06
tdurakovbecause there is no config drive for instance before14:06
tdurakovand reboot doesn't create new14:06
*** burgerk has quit IRC14:07
mriedemcdent: hey, if you open placement bugs as like TODOs, just mark them as confirmed please, or triaged, somethign so they don't show up as new14:07
tdurakovmriedem: ok, will try to ask melwitt for details14:07
*** thorst_ has joined #openstack-nova14:07
mriedemtdurakov: ah ok14:07
cdentmriedem: I was sure if I was allowed to confirm my own bugs, but if that's case, okay.14:07
mriedemyeah if they are just todos, just mark them confirmed14:08
cdentthere not supposed to be todos, I'm hoping that some other people will pick some of them up14:08
mriedemyou're the sme on the placement api14:08
mriedemsure14:08
mriedembut i don't expect someone doing bug triage to come along and say, 'wtf are you talking about, this is obviously wrong'14:08
cdenton the requirements thing: the other reason that is wrong is its in the wrong file, no?14:09
mriedemcdent: nope that was moved to test-requirements in master14:09
cdentbecause I think it was sean already had to fix my bad with putting it in requirements, rather than test-requirements14:09
mriedemand wasn't backported14:09
mriedem^ is a stable/newton change14:09
mriedemwhich makes it doubly confusing14:09
cdentohhhh, got it14:09
cdentmissed that it was a stable14:09
jaypipesbauzas, cdent, dansmith: added PCI device (generic) to inventory management in placement stuff: https://review.openstack.org/37468114:10
jaypipesedleafe: ^14:10
*** panbalag has quit IRC14:10
cdentthanks jaypipes14:10
*** pwnall has joined #openstack-nova14:10
jaypipescdent: np. working on the allocations for those generic pci passthrough devices now.14:11
edleafejaypipes: got it in a tab. Will review shortly14:11
jaypipesedleafe: danke14:11
*** pwnall is now known as Guest7348914:11
*** rlrossit has quit IRC14:12
*** paul-carlton2 has joined #openstack-nova14:12
*** rlrossit_ has joined #openstack-nova14:15
mriedemjaypipes: can you create some ocata follow up bp for the work here? https://review.openstack.org/#/c/374681/114:15
mriedemspecless14:15
mriedemwe'll just track series against it14:15
*** thorst_ has quit IRC14:16
*** Guest73489 has quit IRC14:17
*** tonytan4ever has joined #openstack-nova14:18
*** vishwanathj has joined #openstack-nova14:19
*** wuhg has quit IRC14:21
jaypipesmriedem: good idea, will do.14:22
*** daemontool has joined #openstack-nova14:22
jaypipesmriedem: specless, right? just the blueprint...14:22
mriedemi assume there will be more changes for that, like for allocation?14:22
mriedemjaypipes: yeah14:22
mriedemwe already have that stuff in the existing specs14:22
jaypipesmriedem: yup, got it. gimme a few.14:22
mriedemjust need the ocata bp for tracking14:22
mriedemthanks14:22
*** mriedem is now known as mriedem_away14:22
*** annegentle has quit IRC14:22
*** dimtruck is now known as zz_dimtruck14:23
*** mlavalle has joined #openstack-nova14:24
*** ducttape_ has joined #openstack-nova14:26
jaypipesmriedem_away: https://blueprints.launchpad.net/nova/+spec/inventory-allocation-ocata14:26
openstackgerritJay Pipes proposed openstack/nova: placement: handle generic PCI devices in inventory  https://review.openstack.org/37468114:27
*** zzzeek has quit IRC14:27
*** thorst_ has joined #openstack-nova14:29
*** jaosorior has quit IRC14:29
openstackgerritsahid proposed openstack/nova-specs: virt: huge pages per guest numa nodes  https://review.openstack.org/37485614:30
*** tblakes has quit IRC14:30
*** zzzeek has joined #openstack-nova14:31
*** baoli has joined #openstack-nova14:31
openstackgerritAndrey Volkov proposed openstack/nova: Return build_requests instead of instances  https://review.openstack.org/37426714:32
openstackgerritAndrey Volkov proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937914:32
openstackgerritAndrey Volkov proposed openstack/nova: WIP Make schedule_and_build_instances cell aware  https://review.openstack.org/37264514:32
*** takashin has left #openstack-nova14:33
openstackgerritAndrey Volkov proposed openstack/nova: Add create cell command  https://review.openstack.org/33271314:34
openstackgerritAndrey Volkov proposed openstack/nova: Trivial: add ability to define action description  https://review.openstack.org/35578114:34
*** mat128 is now known as mat128|bbl14:35
*** tjones has left #openstack-nova14:35
BobBalljaypipes: Howdy - got time for a chat about virtual GPU?  We've got a spec up at https://review.openstack.org/#/c/280099/ where danpb suggested a ML discussion back in June (patch set 16 - comments were hidden when we reproposed for Ocata).  We did try to start one (http://lists.openstack.org/pipermail/openstack-dev/2016-June/096684.html) but clearly not at a good time and it got lost in the ML.14:35
gibisdague: hi! where is the checklist jaypipes mentioned in the nova meeting?14:36
sdaguegibi: I'm assuming he meant this - http://burndown.dague.org/14:36
jaypipesBobBall: hi! I have 25 minutes until back to back meetings. is that enough time? :)14:36
jaypipessdague: yes :)14:36
BobBallWell, I do hope so14:37
*** smatzek has joined #openstack-nova14:37
gibijaypipes, sdague: I have tables in the wiki https://wiki.openstack.org/wiki/Nova/VersionedNotificationTransformation#Tasks14:37
sdagueI think markus and johnthetubaguy adapted it for the conf work, we can try to figure out if there is a more generic adaptation14:37
gibithe check list part of the notification transformation is lot simpler14:38
*** smatzek has quit IRC14:38
*** smatzek_ has joined #openstack-nova14:38
sdaguegibi: ok, sure14:38
openstackgerritDan Smith proposed openstack/nova: Add blocker migration to ensure for newton online migrations  https://review.openstack.org/37430714:38
openstackgerritDan Smith proposed openstack/nova: Add placeholder DB migrations for Ocata  https://review.openstack.org/37427014:38
openstackgerritDan Smith proposed openstack/nova: Remove PCI parent_addr online migration  https://review.openstack.org/37351914:38
openstackgerritDan Smith proposed openstack/nova: Make nova-manage online migrations more verbose  https://review.openstack.org/37336114:38
BobBallSo the main issue is how to model virtual gpus.  This spec was proposing re-using the PCI passthrough code and modelling a virtual GPU as a fake PCI device14:38
gibiI feel there is one step per event_type14:38
sdaguethis ends up querying gerrit hourly so you don't have to maintain the outstanding list yourself14:38
gibibut having a burndown would be nice14:38
dansmithalaski: I added matt's test (which needed stable output ordering to be reliable) and the translations ^^14:38
*** smatzek_ is now known as smatzek14:39
jaypipesBobBall: https://hangouts.google.com/hangouts/_/mirantis.com/bob-vgpu14:39
gibisdague: querying gerrit is a good idea. I will look into the script14:39
sdaguegibi: cool14:39
sdagueif you need help, let me know14:39
gibisdague: thanks, I will ping you if I get stuck14:39
*** burgerk has joined #openstack-nova14:39
BobBalljaypipes: Can I join with phone?  my mic is bust14:39
jaypipesBobBall: one sec. O14:40
BobBallhang on a sec, will try a diff way.14:40
jaypipesok14:40
danpbBobBall: jaypipes  i think we have to go further than modelling vGPU as a fake PCI device14:40
danpband actually change our PCI Device code so it is not PCI specific14:41
alaskidansmith: okay. assuming tests pass I'm +2 still14:41
jaypipesdanpb: I haven't had a chance to read any proposal yet on vgpu.14:41
*** burgerk_ has joined #openstack-nova14:41
danpbeg, something that can model arbitrary device types whether PCI or not14:41
jaypipesdanpb: ++|14:41
danpbnon-PCI based vGPU is the first example14:41
dansmithalaski: you're damn right they still pass14:41
jaypipesdanpb: feel free to join the hangout above if you wish.14:41
danpbbut people are already suggesting  vTPMs,  and FPGAs neither of which are PCI based14:41
danpbso IMHO we need to generalize to represent all host devices which are available for assignment to a guest14:42
*** artom has quit IRC14:43
BobBalljaypipes: gah - sorry - just been asked to join an impromptu meeting.  Many apologies, but can we have a chat another time14:43
jaypipesdanpb: agree with you whole-heartedly14:43
openstackgerritChris Dent proposed openstack/nova: [placement] Allow both /placement and /placement/ to work  https://review.openstack.org/37487014:43
jaypipesBobBall: of course, not a problem at all.14:43
raj_singhsfinucan: Are you talking about https://review.openstack.org/#/c/356604/?14:44
danpblemme see if i can get google talk working...14:44
BobBallGreat.  OK - and as you two are in agreement on what we need to do there, perhaps a ML discussion may not be needed and we can come up with a strawman spec that might make sense14:44
BobBallbbl14:44
jaypipesciao14:44
*** burgerk has quit IRC14:44
danpbah o14:44
sfinucanraj_singh: I don't recall now :S14:44
alaskidansmith: heh. I'm not doubting, just hate when Jenkins makes me look foolish.14:44
jaypipesdanpb: we'll do a hangout laters :)14:44
_gryfjaypipes, danpb isn't it a good topic for summit discussion?14:44
jaypipes_gryf: yes14:45
danpbwe could do that14:45
sfinucanraj_singh: But not that one - something less complicated. Just a general example14:45
dansmithalaski: hah14:45
_gryfI mean, I've already scratched the surface last time14:45
danpbthough its probably more useful to start it *before* summit14:45
raj_singhsfinucan: I tweaked your patch a bit, reno was failing to parse the release notes. Looks like it went through14:45
*** dikonoor has quit IRC14:45
*** tblakes has joined #openstack-nova14:45
*** mlavalle has quit IRC14:45
_gryfdanpb, yeah, agree in 100%14:45
sdaguehttps://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:fix_tests - some output leakage cleanup for unit tests14:45
sdaguefor anyone that's wants some easy morning reviews14:46
danpbso we use summit as a place to solve hard problems, rather than merely starting the discussion and finding hard problems after summit14:46
_gryfdanpb, jaypipes so maybe it is good opportunity for a hangouts or something?14:46
sfinucanraj_singh: Yes, much better. Cheers :)14:46
jaypipessdague: on it.14:46
openstackgerritTomi Juvonen proposed openstack/nova-specs: Maintenance reason to server  https://review.openstack.org/31051014:48
*** mdrabe_ has joined #openstack-nova14:48
cdentsdague: I'm getting ready to dive in on this: https://bugs.launchpad.net/nova/+bug/1626489 but before I do I reckon I should check with at least you on what the desired and ideal state of things is.14:48
openstackLaunchpad bug 1626489 in OpenStack Compute (nova) "placement api 404 errors are logged poorly" [Undecided,Confirmed]14:48
*** mlavalle has joined #openstack-nova14:48
*** ayoung has joined #openstack-nova14:49
*** cdelatte has joined #openstack-nova14:49
sdaguecdent: the info line should be sufficient14:49
*** catintheroof has joined #openstack-nova14:50
cdentsdague: so no need to add a LOG.debug?14:50
sdagueI don't think so14:50
cdent14:50
cdentthanks14:50
*** mfisch has joined #openstack-nova14:51
*** mfisch has quit IRC14:51
*** mfisch has joined #openstack-nova14:51
*** tbachman has joined #openstack-nova14:52
*** mdrabe has quit IRC14:52
*** moshele has quit IRC14:52
*** marst has quit IRC14:54
jaypipessdague: done14:54
*** daemontool has quit IRC14:54
sdaguejaypipes: thank you14:55
*** kaisers has quit IRC14:55
*** daemontool has joined #openstack-nova14:55
*** mdrabe_ is now known as mdrabe14:55
*** TravT has quit IRC14:57
*** zenoway has joined #openstack-nova14:58
*** edand has quit IRC14:58
*** roeyc has quit IRC14:59
jaypipessdague: most welcome.14:59
*** TravT has joined #openstack-nova14:59
*** zz_dimtruck is now known as dimtruck14:59
*** rmart04 has quit IRC14:59
*** rlrossit_ has quit IRC15:00
*** Zara has left #openstack-nova15:02
*** Zara has joined #openstack-nova15:02
*** rlrossit_ has joined #openstack-nova15:03
*** mjura has quit IRC15:03
*** brian_price has joined #openstack-nova15:05
*** rcernin has quit IRC15:07
*** jlanoux has quit IRC15:08
*** arahal_ has joined #openstack-nova15:09
*** yamahata has joined #openstack-nova15:11
*** rrecio has joined #openstack-nova15:12
*** r-daneel has quit IRC15:13
*** EinstCrazy has joined #openstack-nova15:13
*** zigo_ is now known as zigo15:14
openstackgerritSergey Nikitin proposed openstack/nova-specs: Restore V3 Diagnostics info  https://review.openstack.org/35788415:14
*** andreas_s has quit IRC15:17
zigoI have one unit test failure when building Nova rc1.15:17
*** andreas_s has joined #openstack-nova15:17
zigohttp://logs.openstack.org/08/374608/1/check/gate-deb-nova-dpkg-debian-jessie/6444bc8/console.html#_2016-09-22_08_36_55_35386615:17
zigojaypipes: bauzas: ^15:17
*** lamt has quit IRC15:18
*** krotscheck has joined #openstack-nova15:20
*** paul-carlton2 has quit IRC15:21
sdaguezigo: it's actually calling out to genisoimage15:21
sdaguewhich probably means that isn't installed15:21
zigosdague: Ah, thanks!15:21
zigosdague: It's a minimal chroot using sbuild, so yeah ! :)15:21
sdaguemelwitt and I were discussing that test the other day, as she hit the same issue15:22
zigoAdding it as build-depends-indep, thanks.15:22
sdaguezigo: no prob15:22
zigosdague: It's kind of weird though to find this between b3 and rc1.15:22
zigoWell, not weird, but unexpected, I'd say.15:23
zigosdague: I really wish there was a way to make subprocess more verbose in case of error ...15:24
zigoIt's usually a black box.15:24
sdaguezigo: yeh, the oslo wrapper could maybe do that15:25
zigoexport PYTHON_SUBPROCESS_DEBUG=True15:25
zigo:)15:25
*** mriedem_away is now known as mriedem15:25
*** EinstCrazy has quit IRC15:26
sdagueah, I guess this test patch only merged 7 days ago - https://review.openstack.org/#/c/335966/15:27
*** marst has joined #openstack-nova15:27
sdaguethat's the one that tripped this15:27
mriedemjaypipes: got it, thanks15:28
bauzaszigo: what sdague said :)15:28
zigobauzas: :)15:29
zigoThe added build-depends-indep: genisoimage is already rebuilding ...15:29
openstackgerritAnusha Unnam proposed openstack/nova: Improve help text for interval_opts  https://review.openstack.org/33176115:29
zigoResult in 25 minutes.15:29
* bauzas going back errand15:29
*** lucasagomes is now known as lucas-hungry15:30
mriedemBobBall: are you good with this? https://review.openstack.org/#/c/289431/15:34
*** bks has joined #openstack-nova15:34
*** hemna_ has joined #openstack-nova15:35
*** artom has joined #openstack-nova15:35
mriedemBobBall: and was a blueprint created for this? https://review.openstack.org/#/c/333781/15:36
*** hshiina has quit IRC15:36
BobBallmriedem: Yes for https://review.openstack.org/#/c/28943115:37
bauzasPaulMurray: do you know if paul carlton is around ?15:37
BobBallSorry - had missed re-adding my +1, but the changes all look good.15:37
bauzasI'm looking at https://bugs.launchpad.net/nova/+bug/1626230 and I'm thinking of giving a "sorry, WONTFIX"15:37
openstackLaunchpad bug 1626230 in OpenStack Compute (nova) "evacuate leaves instance on target compute node if it fails to spawn" [Undecided,In progress] - Assigned to Paul Carlton (paul-carlton2)15:37
*** thorst_ has quit IRC15:37
bauzasPaulMurray: ^15:37
BobBallmriedem: For https://review.openstack.org/#/c/333781 not yet; the team member working on this one is on vacation until tomorrow.  A spec-less BP should be created next week15:37
bauzasPaulMurray: because fixing that is not really a bug, but rather needing to discuss for a feature15:38
mriedemBobBall: ok15:38
BobBallmriedem: We will ping you when a BP is ready and make a request to remove your -2.15:38
bauzasgiven we have reset-state of course15:38
*** hemna_ has quit IRC15:38
*** zenoway has quit IRC15:40
*** jlanoux has joined #openstack-nova15:41
*** burt has joined #openstack-nova15:41
*** andreas_s has quit IRC15:42
*** tri2sing has joined #openstack-nova15:43
*** jlanoux_ has joined #openstack-nova15:44
mriedemjohnthetubaguy: did you have different ideas about how quotas should be moved to the api db?15:44
mriedemfrom what is in https://review.openstack.org/#/c/372839/15:44
johnthetubaguymriedem: good question, I should take a peak15:45
*** arahal_ has quit IRC15:46
*** jlanoux has quit IRC15:46
mriedemjohnthetubaguy: i think L43 here was you? https://etherpad.openstack.org/p/ocata-nova-summit-ideas15:46
mriedemthat's why i ask15:46
johnthetubaguymriedem: just added a comment on the spec, does that make any sense?15:47
*** tongli has joined #openstack-nova15:47
mriedemjohnthetubaguy: like calling this? https://github.com/openstack/nova/blob/1dfd79495e565eda3997b0a272c594a8d2c422d4/nova/db/sqlalchemy/api.py#L377215:48
mriedemif quotas aren't in the api db, force a refresh?15:48
alaskiyeah. that's what I was expecting would happen15:50
johnthetubaguymriedem: yes, thats it, a refrehs15:50
johnthetubaguysorry on a VC, and poorly multi-tasking right now15:50
mriedemthe spec says usages and reservations won't be migrated15:50
alaskihttps://github.com/openstack/nova/blob/1dfd79495e565eda3997b0a272c594a8d2c422d4/nova/db/sqlalchemy/api.py#L365-L37015:50
johnthetubaguyI mean I would love us to have a different API structure, but that should be a different effort15:50
mriedemalaski: btw, how do things work if we don't migrate the usages and reservations?15:51
mriedembecause the API returns usage15:51
*** bks has quit IRC15:51
openstackgerritsahid proposed openstack/nova-specs: virt: huge pages per guest numa nodes  https://review.openstack.org/37485615:51
johnthetubaguyusages... I thought that was an aggregation of instance data15:52
alaskiusages is getting "migrated", just not by copying the value from cell quota_usages to api db quota_usages15:53
mriedemquota_usages is it's own table15:53
alaskiapi db quota_usages would be populated using the sync methods15:53
mriedemok15:53
mriedemso rebuilt15:53
alaskiyeah15:53
mriedemnot 'migrated'15:53
mriedemalright15:53
johnthetubaguywould need to migrate the tenant and user quota values I guess, the ones you set on the API15:54
mriedemquota limits are ok to migrate since those don't change per request15:54
mriedemyeah15:54
alaskireservations will need some thought15:54
alaskithose may need to be migrated the traditional way15:54
mriedemsounds racey15:54
johnthetubaguyyeah, they probably just stay, and start getting added in the new place?15:55
mriedembut i guess you try to read reservation from api, if not there, read from cell db15:55
johnthetubaguyyeah, that15:55
*** kfarr has joined #openstack-nova15:55
mriedemnova-manage db online_data_migrations would migrate the reservations?15:55
johnthetubaguywell, need to read both I think, for a little bit, till cell is empty15:55
mriedemhow long do reservations exist?15:55
johnthetubaguydon't they live for a very short amount of time?15:56
mriedemthat's what i thought, the life of the request15:56
johnthetubaguylike during a migrate, or during a boot15:56
johnthetubaguyso migrate is longer, but still, shouldn't be for ever15:56
mriedemif reservations only live for the lifetime of the request, then i don't think we'd need to migrate those15:56
mriedemto the api db i mean15:56
mriedemnew reservations go in the api db15:56
mriedemlookups check api and fallback to cell db15:56
mriedemand no online data migrations for reservations15:57
mriedemas they are short lived15:57
mriedemalaski: is that right? ^15:57
*** rlrossit_ has quit IRC15:57
alaskiyep15:57
mriedemgreat15:57
*** rlrossit has joined #openstack-nova15:57
johnthetubaguyyeah, +1 that15:57
*** sahid has quit IRC15:59
*** rlrossit_ has joined #openstack-nova15:59
*** code-R has quit IRC16:01
mriedemso this makes me think, we can't support multi-cell cells v2 until we've forced a migration of all cell db stuff to the API DB right?16:01
*** arahal_ has joined #openstack-nova16:01
cmartinHey if anybody is free I could use some code reviews on a bugfix https://review.openstack.org/#/c/370861/16:01
raj_singhsdague: dansmith OSIC folks are trying to test zero downtime upgrades and come up with grenade job for that. Basic idea is test old API and old compute with new control plane. johnthetubaguy mentioned you guys have some ideas on this. Just want to get your thoughts on this approach.16:01
mriedembecause if you had multiple cell dbs and had to perform say a reservation lookup, you wouldn't know which cell db to check right?16:02
*** rlrossit has quit IRC16:02
*** unicell1 has joined #openstack-nova16:03
*** unicell has quit IRC16:03
*** paul-carlton2 has joined #openstack-nova16:06
*** zenoway has joined #openstack-nova16:07
openstackgerritChris Dent proposed openstack/nova: [placement] 404 responses do not cause exception logs  https://review.openstack.org/37495516:07
alaskimriedem: that's right. we can't have multiple cells until we finish all needed migrations16:07
alaskiwell, we can't have efficient multiple cells.16:08
*** thorst_ has joined #openstack-nova16:08
*** masayukig has quit IRC16:09
mriedemyeah i guess you could loop through the cell dbs looking for a reservation...or map them somehow16:09
*** mars_toktonaliev has joined #openstack-nova16:09
*** jwcroppe has quit IRC16:10
*** zenoway has quit IRC16:11
mriedemdoes anyone know what these errors from the scheduler mean? http://logs.openstack.org/24/374324/1/gate/gate-tempest-dsvm-cells-ubuntu-xenial/81f45de/logs/screen-n-cond.txt.gz#_2016-09-21_21_20_25_06716:11
*** danpb has quit IRC16:11
*** masayukig has joined #openstack-nova16:12
mriedemedleafe: bauzas: ^?16:12
mriedemhttp://status.openstack.org//elastic-recheck/index.html#161287516:12
mriedemthat popped up later in newton16:12
*** thorst_ has quit IRC16:12
*** marst has quit IRC16:12
edleafemriedem: looking16:13
mriedemoh it's a network filter http://logs.openstack.org/24/374324/1/gate/gate-tempest-dsvm-cells-ubuntu-xenial/81f45de/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-09-21_21_20_24_44616:14
edleafemriedem: yeah, I didn't recognize: filter 'nova-no-nd-reflection'16:16
mriedemyeah it's a libvirt firewall driver thing16:16
*** prateek has joined #openstack-nova16:18
*** thorst_ has joined #openstack-nova16:18
mriedemso we probably just need to handle that 'already exists' error and ignore16:19
*** nic has joined #openstack-nova16:19
*** thorst__ has joined #openstack-nova16:20
*** rcernin has joined #openstack-nova16:20
mriedemah looks like it's only xenial jobs16:21
mriedemso something racy in newer libvirt/qemu16:21
*** thorst_ has quit IRC16:22
*** code-R has joined #openstack-nova16:24
*** jlanoux_ has quit IRC16:24
*** rlrossit has joined #openstack-nova16:25
* mriedem prepares to clone libvirt16:26
*** code-R has quit IRC16:26
*** code-R has joined #openstack-nova16:26
*** rlrossit_ has quit IRC16:28
*** alee is now known as alee_lunch16:29
paul-carlton2bauzas, re https://bugs.launchpad.net/nova/+bug/1626230, just seen your comments earlier, was travelling home from office early16:30
openstackLaunchpad bug 1626230 in OpenStack Compute (nova) "evacuate leaves instance on target compute node if it fails to spawn" [Undecided,In progress] - Assigned to Paul Carlton (paul-carlton2)16:30
mriedemshazbot, no specific error code from libvirt https://github.com/libvirt/libvirt/blob/290f2adf46295a8cd043135e37ea43d24c003b7f/src/conf/nwfilter_conf.c#L309816:31
*** baoli has quit IRC16:31
paul-carlton2you think fixing this needs a blueprint?16:31
paul-carlton2bauzas, correction you are say won't fix, why?16:32
*** bks has joined #openstack-nova16:32
paul-carlton2dansmith, johnthetubaguy suggested I talk to you about this ^^^16:32
paul-carlton2My initial thought was to move the updating of the compute host up to after the claim but before the instance is spawned16:33
*** baoli has joined #openstack-nova16:34
*** lucas-hungry is now known as lucasagomes16:35
paul-carlton2but now I'm wondering if fixing libvirt driver to tidy up instance artefacts if instance spawn fails, apparently XenAPI does this16:35
kashyapmriedem: That error doesn't seems like a race in newer version, that line you refer to seems to be added in JUL 2014 -- http://libvirt.org/git/?p=libvirt.git;a=commitdiff;h=46a811d16:36
*** lpetrut has quit IRC16:36
kashyapIt _could_ be a race still...(Shouldn't speak without fully looking into the issue)16:37
*** kumarb has joined #openstack-nova16:37
mriedemright https://github.com/libvirt/libvirt/commit/46a811db0731cedaea0153fc223faa6096cee5b516:37
mriedemadded in 1.2.716:37
*** Swami has joined #openstack-nova16:38
mriedemkashyap: it's probably a latent race in the nova network setup code forever16:38
mriedemthat's aren't new16:38
mriedemand now that we're running on xenial with libvirt 1.3.1 it's kicking us back16:38
kashyapOh, right, "latent" is the keyword here16:39
*** derekh has quit IRC16:40
*** bks has quit IRC16:40
mriedemkashyap: so i'm going to just do this http://paste.openstack.org/show/582621/16:41
* kashyap clicks16:41
*** zigo has quit IRC16:41
* kashyap was reading through the original bug referenced in the commit. It's trying to be consistent with the behavior of networks / domains16:41
*** mvk has quit IRC16:41
*** TravT has quit IRC16:42
kashyapmriedem: Ah, that's fast.  Looks reasonable to mitigate the current failures that are occuring in the Gate16:42
*** annegentle has joined #openstack-nova16:42
*** kaisers has joined #openstack-nova16:44
kashyapmriedem: BTW, thanks for those comments on that CVE fix backport.  Didn't ignore them16:44
*** zul has quit IRC16:44
*** zul has joined #openstack-nova16:45
* kashyap needs to drop off for the night, have to run a couple of errands16:45
*** takedakn1 has joined #openstack-nova16:46
*** takedakn has quit IRC16:46
*** takedakn1 is now known as takedakn16:46
*** code-R has quit IRC16:47
cfriesenFor the case of a live migration that hits the timeout, and we call guest.abort_job() at https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L6142, what is the expected next transition?16:48
*** kaisers has quit IRC16:49
cfriesendoes it go to libvirt.VIR_DOMAIN_JOB_FAILED, or libvirt.VIR_DOMAIN_JOB_CANCELLED, or something else?16:49
*** zigo has joined #openstack-nova16:51
*** zigo is now known as Guest1865616:52
*** adisky has quit IRC16:52
*** rlrossit has quit IRC16:55
*** TravT has joined #openstack-nova16:55
slaweq_hello Nova cores, can someone take a look on patch https://review.openstack.org/#/c/365756/ ?16:56
slaweq_thx in advance16:56
*** rama_y has joined #openstack-nova16:56
*** Guest18656 has quit IRC16:56
*** Jeffrey4l has quit IRC16:57
*** annegentle has quit IRC16:58
*** zigo_ has joined #openstack-nova16:59
*** browne has joined #openstack-nova17:01
openstackgerritChris Dent proposed openstack/nova: [placement] ensure that allow headers are bytes  https://review.openstack.org/37480017:01
*** rlrossit has joined #openstack-nova17:03
*** GB21 has joined #openstack-nova17:04
PaulMurraybauzas, sorry - missed you earlier. paul-carlton2 is rethinking - he's going to change the bug to "evacuate doesn't cleanup"17:04
PaulMurrayor something like that17:04
*** zigo_ has quit IRC17:07
*** tongli_ has joined #openstack-nova17:07
*** Apoorva has joined #openstack-nova17:08
*** gszasz has quit IRC17:08
*** tongli has quit IRC17:10
*** zigo_ has joined #openstack-nova17:11
*** thorst__ has quit IRC17:11
*** tongli_ has quit IRC17:12
*** tri2sing has quit IRC17:13
*** thorst_ has joined #openstack-nova17:14
*** gouthamr has quit IRC17:15
*** ijw has joined #openstack-nova17:16
*** thorst__ has joined #openstack-nova17:16
*** thorst_ has quit IRC17:18
openstackgerritChris Dent proposed openstack/nova: [placement] Allow both /placement and /placement/ to work  https://review.openstack.org/37487017:19
*** thorst__ has quit IRC17:21
*** thorst_ has joined #openstack-nova17:21
*** jichen has joined #openstack-nova17:22
*** PaulMurray has quit IRC17:23
*** slaweq_ has quit IRC17:23
*** mat128|bbl is now known as mat12817:24
*** thorst_ has quit IRC17:25
*** lpetrut has joined #openstack-nova17:26
*** code-R has joined #openstack-nova17:28
*** rlrossit has quit IRC17:29
*** gouthamr has joined #openstack-nova17:31
*** paul-carlton2 has quit IRC17:32
*** stewie925 has joined #openstack-nova17:36
*** mkoderer has quit IRC17:39
*** pradk has quit IRC17:39
*** hemanthm is now known as hemanthm|lunch17:40
*** pradk has joined #openstack-nova17:42
*** jichen has quit IRC17:43
*** lamt has joined #openstack-nova17:44
*** ihrachys has quit IRC17:45
*** codo has joined #openstack-nova17:50
codoHi so where is the starting point in the github/openstack/nova code ?17:50
*** thorst_ has joined #openstack-nova17:51
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: ignore conflict when defining network filters  https://review.openstack.org/37497517:53
auggycodo: what are you looking for specifically?17:55
codoauggy: a place in the code to start understanding17:55
auggycodo: are you looking to start contributing to Openstack?17:55
codoauggy: https://github.com/openstack/nova/blob/master/nova/ in this, where should i start from ?17:56
*** thorst__ has joined #openstack-nova17:56
*** thorst_ has quit IRC17:56
codoauggy: yeah that also17:56
codoauggy: so there may be an entry point in this code that I can start with to trace the flow ? easiest is to work towards core using apis hmm ?17:56
auggycodo: cool, just curious, what have you done so far with openstack?17:57
*** thorst__ has quit IRC17:57
codoauggy: beginner so far, but I have some experience with AWS so I understand different pieces of this IAAS17:57
auggycodo: ok, have you been running openstack?17:57
codoauggy: also, there is an api server daemon for nova (computer) right ?17:57
*** thorst_ has joined #openstack-nova17:57
*** rmart04 has joined #openstack-nova17:57
codoauggy: not yet, but I can start, I don't have access to more than a machine17:57
codoshould i get started with devstack etc ?17:58
auggycodo: i would recommend, before you start trying to understand the code, that you install devstack17:58
codoauggy: alright.17:58
auggycodo: yeah, and there are some good resources for introducing using openstack17:58
auggycodo: openstack is a HUGE project with many pieces, and nova is also pretty complicated17:58
mriedemedleafe: i'm not really following your reply here https://review.openstack.org/#/c/339760/7/nova/tests/unit/scheduler/test_scheduler.py17:58
*** moshele has joined #openstack-nova17:58
auggycodo: so you should build that context first17:58
codoauggy: went through the documentation and understood various pieces17:58
mriedemso that test is just testing that the code you removed, isn't added back in...17:58
codoauggy: such as nova/cinder/neutron17:58
mriedemso it's testing code that doesn't exist17:58
*** Guest56480 has quit IRC17:59
*** sudswas__ has quit IRC17:59
auggycodo: one way to get started with understanding the code, once you've set up a devstack and run through some different configurations is to reproduce bugs that get reported in the bugs queue17:59
artomcodo, Nova is massive and there isn't really one flow that you can trace17:59
openstackgerritSujitha proposed openstack/nova: Improve help text for glance options  https://review.openstack.org/32400717:59
codoauggy: it is ok if I have only one machine ?17:59
codoartom: ah I see17:59
auggycodo: yes, you can just run devstack on a vm17:59
auggyartom: codo: yeah that's the point i was making :)18:00
auggycodo: you don't need anything too fancy, do you have the links to the devstack docs and the openstack docs?18:00
codoauggy: i do18:00
codoI have 5.1 GB free space on my machine as on now18:00
mriedemedleafe: ignore, i didn't see that you removed it18:01
auggycodo: also, there's an awesome book that just came out https://www.amazon.com/Common-OpenStack-Deployments-Real-World-Administrators/dp/013408623618:01
codoauggy: ah thanks!18:02
auggycodo: you'll need to refer to the devstack documentation for your setup and just experiment to see what works on your system18:02
codoauggy: ok18:02
auggycodo: when you are ready to start diving into code, we have a bugs subteam where we focus on bug triage and that includes reproducing bugs18:02
auggycodo: one of the best ways to dive into code is to reproduce a problem and then to try to find the source18:03
openstackgerritSean Dague proposed openstack/nova: Fix stdout leakage during opportunistic db tests  https://review.openstack.org/37503718:03
codoalright, super interested18:03
*** code-R_ has joined #openstack-nova18:03
artomauggy, nice recruiting effort ;)18:03
sdaguemriedem: I think that's the last db related stdout leak18:03
auggycodo: https://wiki.openstack.org/wiki/Meetings/Nova/BugsTeam#Nova_Bugs_Team18:03
* stewie925 agrees with artom re: auggy18:03
*** kumarb has quit IRC18:03
auggyartom: everyone learns differently, that's one way that's worked for me ;)18:04
edleafemriedem: the patch removed the ability to import a driver via the class path. The test verified that trying to improt a class path would fail.18:04
artomauggy, that wasn't meant pejoratively :D18:04
*** rmart04 has quit IRC18:04
openstackgerritSujitha proposed openstack/nova: Improve help text for libvirt options  https://review.openstack.org/37272218:04
auggyartom: ha i didn't take it that way18:04
mriedemsdague: +218:05
mriedemsdague: i've got a series in nova-specs that should be pretty easy https://review.openstack.org/#/c/372790/18:05
mriedemstarts at ^18:05
mriedemmichael was through it once already18:05
*** code-R has quit IRC18:06
auggycodo: feel free to reach out if you have any questions and feel free to drop by a bugs team meeting!18:06
*** rmart04 has joined #openstack-nova18:06
codoauggy: will do18:06
sdaguemriedem: nice18:07
*** annegentle has joined #openstack-nova18:09
*** ig0r_ has quit IRC18:09
*** armax has joined #openstack-nova18:13
*** xyang1 has joined #openstack-nova18:13
*** ManishD has joined #openstack-nova18:13
*** rmart04 has quit IRC18:14
ManishD@Dansmith: HI, about the https://review.openstack.org/#/c/35713118:14
ManishDDansmith: We have addresses the comments you provided. please can you take a look when you get some free time18:15
dansmithokay18:15
*** yamahata has quit IRC18:15
ManishDDansmith: thanks Dansmith18:15
openstackgerritClaudiu Belu proposed openstack/nova: WIP: api: discoverable policy  https://review.openstack.org/28053918:15
openstackgerritClaudiu Belu proposed openstack/nova: numa: Fixes NUMA topology related unit tests  https://review.openstack.org/27883518:18
*** annegentle has quit IRC18:20
*** prateek has quit IRC18:20
*** codo has quit IRC18:21
*** GB21 has quit IRC18:22
*** thorst_ has quit IRC18:24
*** ayoung has quit IRC18:25
*** thorst_ has joined #openstack-nova18:25
*** mars_toktonaliev has quit IRC18:25
*** raunak has joined #openstack-nova18:26
*** slaweq_ has joined #openstack-nova18:27
*** code-R_ has quit IRC18:29
*** rlrossit has joined #openstack-nova18:29
*** code-R has joined #openstack-nova18:30
*** thorst_ has quit IRC18:30
*** lpetrut has quit IRC18:30
*** thorst_ has joined #openstack-nova18:31
*** kaisers has joined #openstack-nova18:33
*** thorst_ has quit IRC18:35
*** kaisers has quit IRC18:38
*** alee_lunch is now known as alee18:41
*** coolsvap has quit IRC18:42
*** baoli has quit IRC18:44
*** baoli has joined #openstack-nova18:45
*** baoli_ has joined #openstack-nova18:46
*** kfarr_ has joined #openstack-nova18:46
*** hemanthm|lunch is now known as hemanthm18:48
*** yamahata has joined #openstack-nova18:49
*** baoli has quit IRC18:50
mriedemBobBall: is it a known thing that the xen driver doesn't populate the diag info for cpu/memory/disk here? https://github.com/openstack/nova/blob/64cbd7c51a5a82b965dab53eccfaecba45be9c27/nova/virt/xenapi/vm_utils.py#L1745-L175218:51
*** porunov has quit IRC18:53
mriedemsame for vmware https://github.com/openstack/nova/blob/64cbd7c51a5a82b965dab53eccfaecba45be9c27/nova/virt/vmwareapi/vmops.py#L156118:54
mriedembrowne: is that a TODO that can be done? ^18:54
*** arahal_ has quit IRC18:58
mriedemsnikitin: i think we're going to have to get the vmware and xen drivers fully implementing the get_instance_diagnostics API before we can turn that on with the microversion18:59
mriedemelse it's going to be a regression for those drivers18:59
melwitttdurakov: you had some questions about https://bugs.launchpad.net/nova/+bug/1582684 ?18:59
openstackLaunchpad bug 1582684 in OpenStack Compute (nova) mitaka "nova kilo->liberty ceph configdrive upgrade fails" [High,Fix committed] - Assigned to melanie witt (melwitt)18:59
bauzasp!spy19:00
openstackgerritSarafraj Singh proposed openstack/nova: conf: Trivial cleanup of console.py  https://review.openstack.org/37467419:03
*** nherciu_ has quit IRC19:04
mriedemartom: the diff from newton for https://review.openstack.org/#/c/373354/4/specs/ocata/approved/virt-device-role-tagging.rst is pretty large19:04
*** GordonTX has joined #openstack-nova19:04
*** artom has quit IRC19:04
mriedemlooks like mostly line wrapping changes?19:04
openstackgerritSarafraj Singh proposed openstack/nova: conf: Trivial cleanup of compute.py  https://review.openstack.org/37467319:08
*** annegentle has joined #openstack-nova19:08
*** thorst_ has joined #openstack-nova19:08
*** tanvirt has quit IRC19:08
*** daemontool has quit IRC19:12
*** daemontool has joined #openstack-nova19:13
tdurakovmelwitt: hi, yes19:14
tdurakovmelwitt: I found related issue19:14
tdurakovif you change force_config_drive param19:14
*** ayoung has joined #openstack-nova19:14
tdurakovand then try to reboot instance it will fail19:14
melwitttdurakov: you mean if you change it from False to True?19:14
*** ayoung has quit IRC19:14
*** tongli has joined #openstack-nova19:14
tdurakovyes19:14
openstackgerritSarafraj Singh proposed openstack/nova: conf: Trivial cleanup of 'cells'  https://review.openstack.org/37467219:15
tdurakovmelwitt: so, wanted to figure out why it was decided that we need fallback for rbd?19:15
melwitttdurakov: hm, yeah. so upgrade from Kilo -> Liberty + change force_config_drive from False to True19:15
tdurakovmelwitt: and I'm pretty sure the bag will be valid for non-ceph events too19:16
tdurakovmelwitt: it's could be reproduced without upgrade19:16
*** ManishD has quit IRC19:16
tdurakovmelwitt: I'm working on fix, but want to be sure that there are no pitfalls I haven't noticed19:17
tdurakovs/bug19:17
*** tongli_ has joined #openstack-nova19:17
melwitttdurakov: okay. yeah ... I'm thinking we haven't had proper support for changing that value before19:17
melwitttdurakov: there's discussion on why the fall back on the review https://review.openstack.org/#/c/317785/ but it was basically because we were aiming to do a minimal, backportable fix that wouldn't involve moving people's config drives19:18
*** daemontool_ has joined #openstack-nova19:18
*** daemontool has quit IRC19:19
tdurakovoh, ok, p.s. I'll add check here that we do have raw/flat config_drive before doing fallback19:19
*** tongli has quit IRC19:19
*** devananda is now known as devananda|lunch19:20
tdurakovmelwitt: will add you to review change19:20
melwittokay19:20
*** slaweq_ has quit IRC19:22
*** daemontool_ has quit IRC19:24
openstackgerritSean Dague proposed openstack/nova: Fix stdout leakage during opportunistic db tests  https://review.openstack.org/37503719:24
sdaguemriedem: I missed an import19:25
*** daemontool_ has joined #openstack-nova19:25
mriedemsdague: have you been pinged about this upgrade failing for getting stable/newton support into d-g?19:26
mriedemhttps://review.openstack.org/#/c/362438/19:26
sdagueyeh19:26
sdaguedhellmann and I were talking about it on -dev19:27
*** thorst_ has quit IRC19:28
*** rtheis_ has joined #openstack-nova19:29
*** bdeschenes has quit IRC19:29
*** zigo_ is now known as zigo19:30
*** rtheis has quit IRC19:30
*** jvgrant has quit IRC19:31
*** jvgrant has joined #openstack-nova19:31
*** moshele has quit IRC19:33
*** READ10 has quit IRC19:34
*** rtheis_ is now known as rtheis19:35
*** bvanhav has quit IRC19:35
*** slaweq_ has joined #openstack-nova19:37
*** salv-orlando has joined #openstack-nova19:42
pumaranikarmriedem, can you please remove hold from https://review.openstack.org/#/c/360666/. remove internal_id column patch19:44
mriedemdone19:44
*** salv-orl_ has quit IRC19:45
pumaranikarmriedem, thanks :). also, please review when you get time.19:46
mriedemdansmith: do you have an issue with dropping that unused column? ^19:47
mriedemgiven we'd have to make an exception for it here https://review.openstack.org/#/c/360666/2/nova/tests/unit/db/test_migrations.py19:47
brownemriedem: no, that TODO for diag info for cpu/memory/disk still applies.  let me ask around about it19:49
*** rrecio has quit IRC19:51
*** ayoung has joined #openstack-nova19:51
*** thorst_ has joined #openstack-nova19:54
*** ijw has quit IRC19:54
*** rlrossit has quit IRC19:56
*** ayoung has quit IRC19:57
*** ijw has joined #openstack-nova19:58
*** mars_toktonaliev has joined #openstack-nova19:58
*** GordonTX has quit IRC19:59
*** artom has joined #openstack-nova19:59
*** dane-fichter has quit IRC19:59
*** mars_toktonaliev has quit IRC20:01
*** mars_toktonaliev has joined #openstack-nova20:01
*** mars_toktonaliev has quit IRC20:02
*** GordonTX has joined #openstack-nova20:03
*** ijw has quit IRC20:03
*** slaweq_ has quit IRC20:03
mriedemheh, was instances_backup a precursor to shadow tables?20:04
mriedemdansmith: sdague: alaski: ^ anyone remember?20:04
*** marst has joined #openstack-nova20:04
alaskidoesn't ring a bell20:04
sdaguemriedem: I have no idea20:04
sdaguegit log -S to the rescue20:04
sdaguehttps://review.openstack.org/#/c/54172/8/nova/db/sqlalchemy/migrate_repo/versions/138_sqlite_upgrade.sql - an artifact of table alter?20:06
sdagueis that still around somewhere?20:06
*** thorst_ has quit IRC20:06
mriedemhttps://review.openstack.org/#/c/5268/ ?20:06
*** thorst_ has joined #openstack-nova20:08
sdaguemriedem: where are you seeing instances_backup?20:08
mriedemsdague: https://github.com/openstack/nova/commit/8e825c4024f9feb18ca5e2416484ef4096c3bb0620:09
mriedemlooks like instance.internal_id was replaced with the InstanceIdMapping table20:09
mriedemdoesn't look like the ec2 api sets internal_id, so seems we could drop that column, if we want to take the table lock hit on the alter20:10
mriedemwhich, i'm not sure we do20:10
sdaguemriedem: ok, but all the instances of instances_backup is sqlite db migrations20:10
*** tbachman_ has joined #openstack-nova20:15
*** tbachman has quit IRC20:16
*** tbachman_ is now known as tbachman20:16
*** thorst_ has quit IRC20:16
*** slaweq_ has joined #openstack-nova20:16
mriedemwe need johannes to tell us how long dropping that column will take :)20:19
dansmithmriedem: yep we can't do that20:20
*** gouthamr has quit IRC20:20
dansmithit'll break everything20:20
dansmith-2'd for safety with an explanation20:21
sdaguedansmith: what about full table drop? (not for this, but for s3_images)20:22
*** kaisers has joined #openstack-nova20:22
sdaguethat seems like it should be fast20:22
dansmithsdague: if we haven't queried it in a cycle, then sure, but if it has any relations in other models that can cause them to be loaded then it would be safer to just wait20:23
*** tongli_ has quit IRC20:23
dansmiththere's really little to be gained from dropping an empty table20:23
sdaguewell, except it's easier for people to understand their environment20:23
*** manjeets has joined #openstack-nova20:24
*** code-R has quit IRC20:25
pumaranikardansmith, checking your comment. I guess I am following same thing in patchset 1. https://review.openstack.org/#/c/360666/120:26
pumaranikaris this feasible ?20:26
*** kaisers has quit IRC20:27
*** jwcroppe has joined #openstack-nova20:27
*** ijw has joined #openstack-nova20:28
mriedemah i guess that was explained here http://docs.openstack.org/developer/nova/upgrade.html?highlight=upgrade#migration-policy20:29
mriedemi totally missed it20:29
*** tongli has joined #openstack-nova20:29
sdaguethe col drop still seems like it's going to need a table lock though, right?20:30
sdagueso I'm not sure that even N+2 is safe20:30
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/37509620:30
mriedemthat's my understanding20:30
dansmithsdague: I don't think so anymore20:31
dansmithbut it's also pretty low on the list of things to clean up, so I'm fine not doing those unless we have to20:31
dansmithbut at least we can't break the N code with N+1 migrations20:31
*** smatzek has quit IRC20:32
*** annegentle has quit IRC20:34
*** annegentle has joined #openstack-nova20:34
*** ducttape_ has quit IRC20:34
*** tri2sing has joined #openstack-nova20:34
*** annegentle has quit IRC20:35
*** ihrachys has joined #openstack-nova20:35
sdaguehttps://dev.mysql.com/doc/refman/5.6/en/innodb-create-index-overview.html#innodb-online-ddl-summary-grid - so it actually looks like it's not a table lock, at least with newer mysql versions20:35
sdaguejust for reference20:35
dansmithpumaranikar: yeah20:36
dansmithsdague: right20:36
*** hamzy has quit IRC20:36
*** cdelatte has quit IRC20:37
mriedemclaudiub: have you reviewed this? https://review.openstack.org/#/c/175479/20:38
mriedemclaudiub: nvm i guess you took it over?20:39
*** mlavalle has quit IRC20:39
*** baoli_ has quit IRC20:40
*** mlavalle has joined #openstack-nova20:40
openstackgerritSean Dague proposed openstack/nova: Fix stdout leakage during opportunistic db tests  https://review.openstack.org/37503720:41
*** psilvad has joined #openstack-nova20:42
*** lucasagomes is now known as lucas-afk20:42
*** annegentle has joined #openstack-nova20:43
*** ducttape_ has joined #openstack-nova20:44
*** ducttape_ has quit IRC20:45
mriedemhmm, hyper-v ci doesn't use windows volume drivers...20:47
*** ijw has quit IRC20:48
openstackgerritJulian proposed openstack/nova: Transform instance.snapshot notifications  https://review.openstack.org/33012820:50
*** ihrachys has quit IRC20:51
*** artom has quit IRC20:51
*** artom has joined #openstack-nova20:51
*** ihrachys has joined #openstack-nova20:52
openstackgerritAndrew Laski proposed openstack/nova: DNM  https://review.openstack.org/37510620:53
*** ihrachys has quit IRC20:54
pumaranikardansmith, are we going to consider patchset 1 ? I can upload it then.20:55
*** smatzek has joined #openstack-nova20:55
openstackgerritAndrew Laski proposed openstack/nova: DNM  https://review.openstack.org/37510620:56
*** sdague has quit IRC20:56
*** ihrachys has joined #openstack-nova20:56
claudiubmriedem: hi. so yeah, the hyper-v ci on nova doesn't use windows volume drivers, only the windows ci on cinder tests the iscsi and smb windows drivers.20:57
*** ihrachys has quit IRC20:57
mriedemclaudiub: this doesn't look like it does http://64.119.130.115/cinder/315676/8/smb3_windows/devstack_config/cinder/cinder.conf.gz20:58
*** eharney has quit IRC20:58
*** rama_y has quit IRC20:58
mriedemclaudiub: anyway, i asked some questions in https://review.openstack.org/#/c/373501/1/specs/ocata/approved/hyperv-storage-qos.rst after i already approved it20:58
claudiubmriedem: because of the linux looking paths?20:59
mriedemi wanted to start reviewing the code change but then was wondering how this even works with the smbfs volume driver since i don't see that passing qos_specs back20:59
*** smatzek has quit IRC21:00
claudiubmriedem: so, the cinder volume driver doesn't have anything to do with this. an smb volume is basically mounted just like any vhd / vhdx, and you can set iops limits on those disks21:00
mriedemclaudiub: hmm, ok, the spec says this only works with SMB21:00
*** hemanthm is now known as hemanthm|afk21:01
claudiubmriedem: yeah, it does.21:01
mriedem:/21:02
claudiubmriedem: and in the implementation, only for the nova SMBVolumeDriver has the implementation for qos21:02
claudiubthe iscsi one only logs a warning21:02
mriedemyeah, so something doesn't make sense here21:02
*** psilvad has quit IRC21:02
*** psilvad has joined #openstack-nova21:02
claudiubmriedem: what exactly?21:03
mriedemso you don't need to be using the cinder smbfs volume driver for this to work?21:03
*** ihrachys has joined #openstack-nova21:03
*** ihrachys has quit IRC21:04
claudiubwell yeah, you need smb cinder volumes21:04
*** ihrachys has joined #openstack-nova21:04
claudiubany smb cinder volumes21:04
*** psilvad has quit IRC21:07
*** markvoelker has quit IRC21:07
*** Guest6906 has quit IRC21:08
*** kfarr_ has quit IRC21:09
dansmithpumaranikar: yeah, revert back to PS1 and we can review21:10
pumaranikardansmith, thanks :). can you please remove -2 ?21:12
dansmithpumaranikar: when you push it up yeah21:12
dansmithpumaranikar: I want the -2 on there until it's changed so it doesn't accidentally get merged21:12
pumaranikardansmith, ok. yes.21:12
*** GordonTX has quit IRC21:13
pumaranikarmriedem, dansmith . I guess I don't have add schema change here either https://review.openstack.org/#/c/365181/. add uniqueConstraint to instance_actions.21:13
*** kaisers has joined #openstack-nova21:15
dansmithpumaranikar: well, adding such a constraint is a problem if there is any chance that we have any dupes in the db right now21:15
dansmithpumaranikar: if you could delete any dupes as part of the migration without danger, then you could potentially do it right now21:16
dansmithI don't know what that looks like though, without going deeper21:16
openstackgerritJulian proposed openstack/nova: Transform instance.snapshot notifications  https://review.openstack.org/33012821:17
pumaranikardansmith, yes. makes sense. do you recommend then doing that in the same change?21:19
*** kaisers has quit IRC21:19
dansmithpumaranikar: you'd kinda have to in order to make sure nothing regressed between the first and secon21:20
*** jamesdenton has quit IRC21:21
openstackgerritJulian proposed openstack/nova: Transform instance.snapshot notifications  https://review.openstack.org/33012821:21
*** thorst_ has joined #openstack-nova21:22
*** cmartin has quit IRC21:23
*** tongli has quit IRC21:23
pumaranikardansmith, yeah. please let me know if you have any suggestion on it.21:24
*** burgerk__ has joined #openstack-nova21:24
*** jheroux has quit IRC21:25
*** thorst___ has joined #openstack-nova21:25
*** cleong has quit IRC21:26
*** thorst_ has quit IRC21:27
*** salv-orlando has quit IRC21:27
*** gyee has joined #openstack-nova21:27
*** salv-orlando has joined #openstack-nova21:27
*** burgerk_ has quit IRC21:28
*** burgerk__ has quit IRC21:29
*** thorst_ has joined #openstack-nova21:29
openstackgerritPushkar Umaranikar proposed openstack/nova: Remove internal_id attribute from instances table  https://review.openstack.org/36066621:29
*** thorst___ has quit IRC21:30
openstackgerritPushkar Umaranikar proposed openstack/nova: Remove internal_id attribute from instances table  https://review.openstack.org/36066621:32
*** GordonTX has joined #openstack-nova21:33
*** thorst_ has quit IRC21:33
*** rcernin has quit IRC21:34
*** GordonTX has quit IRC21:36
*** zul has quit IRC21:36
*** cdent has quit IRC21:36
*** bkopilov has quit IRC21:37
*** liverpooler has quit IRC21:37
*** NightKhaos has quit IRC21:37
*** veena has quit IRC21:37
*** pkoniszewski has quit IRC21:37
*** mirrorbo1 has quit IRC21:37
*** bradjones has quit IRC21:37
*** liangy has quit IRC21:37
*** John341 has quit IRC21:37
*** slaweq has quit IRC21:37
*** jdillaman has quit IRC21:37
*** aloga has quit IRC21:37
*** jhesketh has quit IRC21:37
*** rtheis has quit IRC21:37
*** daemontool_ has quit IRC21:37
*** xyang1 has quit IRC21:37
*** vladikr has quit IRC21:37
*** amotoki has quit IRC21:37
*** edmondsw has quit IRC21:37
*** rmcall has quit IRC21:37
*** liuyulong has quit IRC21:37
*** diana_clarke has quit IRC21:37
*** rfolco has quit IRC21:37
*** jinli has quit IRC21:37
*** irenab has quit IRC21:37
*** hemanthm|afk has quit IRC21:37
*** tobasco has quit IRC21:37
*** liusheng has quit IRC21:37
*** siva_krishnan has quit IRC21:37
*** pc_m has quit IRC21:37
*** jlvillal has quit IRC21:37
*** ekhugen has quit IRC21:37
*** mikal has quit IRC21:37
*** inara has quit IRC21:37
*** Qiming has quit IRC21:37
*** DuncanT has quit IRC21:37
*** tri2sing has quit IRC21:37
*** alee has quit IRC21:37
*** gcb has quit IRC21:37
*** rmk has quit IRC21:37
*** mriedem has quit IRC21:37
*** acabot has quit IRC21:37
*** bhagyashri has quit IRC21:37
*** vipul has quit IRC21:37
*** hussain_c has quit IRC21:37
*** rushiagr has quit IRC21:37
*** markstur has quit IRC21:37
*** Prophidys has quit IRC21:37
*** slagle has quit IRC21:37
*** d34dh0r53 has quit IRC21:37
*** jfding has quit IRC21:37
*** pkholkin has quit IRC21:37
*** greghaynes has quit IRC21:37
*** boris-42 has quit IRC21:37
*** karlamrhein has quit IRC21:37
*** ameade has quit IRC21:37
*** rmcadams has quit IRC21:37
*** ildikov has quit IRC21:37
*** BlackDex has quit IRC21:37
*** cburgess has quit IRC21:37
*** abramley has quit IRC21:37
*** bswartz has quit IRC21:37
*** kragniz has quit IRC21:37
*** chenfei has quit IRC21:37
*** kgalanov has quit IRC21:37
*** melwitt has quit IRC21:37
*** hwoarang has quit IRC21:37
*** Cassowary has quit IRC21:37
*** betherly has quit IRC21:37
*** vdrok has quit IRC21:37
*** sballe_ has quit IRC21:37
*** zhiyan has quit IRC21:37
*** nikhil has quit IRC21:37
*** morgabra has quit IRC21:37
*** mugsie has quit IRC21:37
*** Zara has quit IRC21:37
*** Qu4tre has quit IRC21:37
*** mfisch has quit IRC21:37
*** vishwanathj has quit IRC21:37
*** rodrigods has quit IRC21:37
*** shihanzhang has quit IRC21:37
*** ababich has quit IRC21:37
*** syjulian has quit IRC21:37
*** rhagarty_ has quit IRC21:37
*** clarkb has quit IRC21:37
*** migi has quit IRC21:37
*** jogo has quit IRC21:37
*** Lehtu has quit IRC21:37
*** mkoderer__ has quit IRC21:37
*** dosaboy has quit IRC21:37
*** ansiwen has quit IRC21:37
*** mdbooth has quit IRC21:37
*** stevemar has quit IRC21:37
*** ttx has quit IRC21:37
*** timello has quit IRC21:37
*** scottynomad has quit IRC21:37
*** ishant|away has quit IRC21:37
*** toure has quit IRC21:37
*** markmc has quit IRC21:37
*** doffm has quit IRC21:37
*** brad[] has quit IRC21:37
*** ajo has quit IRC21:37
*** jcook has quit IRC21:37
*** alineb has quit IRC21:37
*** gsilvis has quit IRC21:37
*** apuimedo has quit IRC21:37
*** annegentle has quit IRC21:37
*** mlavalle has quit IRC21:37
*** TravT has quit IRC21:37
*** nic has quit IRC21:37
*** burt has quit IRC21:37
*** zzzeek has quit IRC21:37
*** tonytan4ever has quit IRC21:37
*** krtaylor has quit IRC21:37
*** ircuser-1 has quit IRC21:37
*** CristinaPauna has quit IRC21:37
*** jroll has quit IRC21:37
*** yarkot has quit IRC21:37
*** tanee has quit IRC21:37
*** jamielennox has quit IRC21:37
*** aorourke has quit IRC21:37
*** gmann has quit IRC21:37
*** clif_h has quit IRC21:37
*** knikolla has quit IRC21:37
*** cargonza has quit IRC21:37
*** gus has quit IRC21:37
*** leifz has quit IRC21:37
*** dimtruck has quit IRC21:37
*** zhenguo has quit IRC21:37
*** csatari has quit IRC21:37
*** andrewbogott has quit IRC21:37
*** geekinutah has quit IRC21:37
*** yonglihe has quit IRC21:37
*** fyxim has quit IRC21:37
*** shaohe_feng has quit IRC21:37
*** clenimar has quit IRC21:37
*** McNinja has quit IRC21:37
*** haplo37_ has quit IRC21:37
*** lbragstad has quit IRC21:37
*** carl_baldwin has quit IRC21:37
*** HenryG has quit IRC21:37
*** FrankZhang has quit IRC21:37
*** tdurakov has quit IRC21:37
*** lxsli has quit IRC21:37
*** andreaf has quit IRC21:37
*** adreznec has quit IRC21:37
*** mgagne has quit IRC21:37
*** toan has quit IRC21:37
*** tri2sing has joined #openstack-nova21:38
*** Zara has joined #openstack-nova21:38
*** alee has joined #openstack-nova21:38
*** gcb has joined #openstack-nova21:38
*** rmk has joined #openstack-nova21:38
*** melwitt has joined #openstack-nova21:38
*** mriedem has joined #openstack-nova21:38
*** acabot has joined #openstack-nova21:38
*** bhagyashri has joined #openstack-nova21:38
*** vipul has joined #openstack-nova21:38
*** hussain_c has joined #openstack-nova21:38
*** rushiagr has joined #openstack-nova21:38
*** markstur has joined #openstack-nova21:38
*** Prophidys has joined #openstack-nova21:38
*** slagle has joined #openstack-nova21:38
*** jfding has joined #openstack-nova21:38
*** pkholkin has joined #openstack-nova21:38
*** greghaynes has joined #openstack-nova21:38
*** boris-42 has joined #openstack-nova21:38
*** karlamrhein has joined #openstack-nova21:38
*** ameade has joined #openstack-nova21:38
*** rmcadams has joined #openstack-nova21:38
*** ildikov has joined #openstack-nova21:38
*** BlackDex has joined #openstack-nova21:38
*** kragniz has joined #openstack-nova21:38
*** bswartz has joined #openstack-nova21:38
*** cburgess has joined #openstack-nova21:38
*** abramley has joined #openstack-nova21:38
*** chenfei has joined #openstack-nova21:38
*** kgalanov has joined #openstack-nova21:38
*** hwoarang has joined #openstack-nova21:38
*** Cassowary has joined #openstack-nova21:38
*** betherly has joined #openstack-nova21:38
*** sballe_ has joined #openstack-nova21:38
*** vdrok has joined #openstack-nova21:38
*** zhiyan has joined #openstack-nova21:38
*** nikhil has joined #openstack-nova21:38
*** morgabra has joined #openstack-nova21:38
*** mugsie has joined #openstack-nova21:38
*** Qu4tre has joined #openstack-nova21:38
*** macsz has quit IRC21:39
*** ediardo has quit IRC21:39
*** annegentle has joined #openstack-nova21:40
*** mlavalle has joined #openstack-nova21:40
*** TravT has joined #openstack-nova21:40
*** nic has joined #openstack-nova21:40
*** burt has joined #openstack-nova21:40
*** zzzeek has joined #openstack-nova21:40
*** tonytan4ever has joined #openstack-nova21:40
*** krtaylor has joined #openstack-nova21:40
*** ircuser-1 has joined #openstack-nova21:40
*** CristinaPauna has joined #openstack-nova21:40
*** jroll has joined #openstack-nova21:40
*** yarkot has joined #openstack-nova21:40
*** tanee has joined #openstack-nova21:40
*** jamielennox has joined #openstack-nova21:40
*** aorourke has joined #openstack-nova21:40
*** gmann has joined #openstack-nova21:40
*** clif_h has joined #openstack-nova21:40
*** knikolla has joined #openstack-nova21:40
*** gus has joined #openstack-nova21:40
*** leifz has joined #openstack-nova21:40
*** dimtruck has joined #openstack-nova21:40
*** zhenguo has joined #openstack-nova21:40
*** yonglihe has joined #openstack-nova21:40
*** fyxim has joined #openstack-nova21:40
*** shaohe_feng has joined #openstack-nova21:40
*** McNinja has joined #openstack-nova21:40
*** clenimar has joined #openstack-nova21:40
*** haplo37_ has joined #openstack-nova21:40
*** lbragstad has joined #openstack-nova21:40
*** carl_baldwin has joined #openstack-nova21:40
*** FrankZhang has joined #openstack-nova21:40
*** HenryG has joined #openstack-nova21:40
*** lxsli has joined #openstack-nova21:40
*** andreaf has joined #openstack-nova21:40
*** adreznec has joined #openstack-nova21:40
*** mgagne has joined #openstack-nova21:40
*** toan has joined #openstack-nova21:40
*** alee has quit IRC21:40
*** karlamrhein has quit IRC21:40
*** rmcadams has quit IRC21:40
*** ildikov has quit IRC21:40
*** kgalanov has quit IRC21:40
*** zhenguo has quit IRC21:40
*** fyxim has quit IRC21:40
*** zul has joined #openstack-nova21:40
*** cdent has joined #openstack-nova21:40
*** bkopilov has joined #openstack-nova21:40
*** liverpooler has joined #openstack-nova21:40
*** NightKhaos has joined #openstack-nova21:40
*** veena has joined #openstack-nova21:40
*** pkoniszewski has joined #openstack-nova21:40
*** mirrorbo1 has joined #openstack-nova21:40
*** bradjones has joined #openstack-nova21:40
*** liangy has joined #openstack-nova21:40
*** John341 has joined #openstack-nova21:40
*** slaweq has joined #openstack-nova21:40
*** jdillaman has joined #openstack-nova21:40
*** aloga has joined #openstack-nova21:40
*** serverascode has quit IRC21:41
*** jhesketh has joined #openstack-nova21:41
*** boris-42 has quit IRC21:41
*** ameade has quit IRC21:41
*** zhiyan has quit IRC21:41
*** nikhil has quit IRC21:41
*** rrecio has joined #openstack-nova21:41
*** GordonTX has joined #openstack-nova21:41
*** d34dh0r53 has joined #openstack-nova21:41
*** daemontool_ has joined #openstack-nova21:41
*** xyang1 has joined #openstack-nova21:41
*** vladikr has joined #openstack-nova21:41
*** amotoki has joined #openstack-nova21:41
*** edmondsw has joined #openstack-nova21:41
*** rmcall has joined #openstack-nova21:41
*** diana_clarke has joined #openstack-nova21:41
*** liuyulong has joined #openstack-nova21:41
*** rfolco has joined #openstack-nova21:41
*** jinli has joined #openstack-nova21:41
*** irenab has joined #openstack-nova21:41
*** hemanthm|afk has joined #openstack-nova21:41
*** tobasco has joined #openstack-nova21:41
*** liusheng has joined #openstack-nova21:41
*** siva_krishnan has joined #openstack-nova21:41
*** pc_m has joined #openstack-nova21:41
*** jlvillal has joined #openstack-nova21:41
*** ekhugen has joined #openstack-nova21:41
*** mikal has joined #openstack-nova21:41
*** inara has joined #openstack-nova21:41
*** Qiming has joined #openstack-nova21:41
*** thorst_ has joined #openstack-nova21:41
*** GordonTX has quit IRC21:42
*** daemontool_ has quit IRC21:42
*** mfisch has joined #openstack-nova21:42
*** vishwanathj has joined #openstack-nova21:42
*** rodrigods has joined #openstack-nova21:42
*** shihanzhang has joined #openstack-nova21:42
*** ababich has joined #openstack-nova21:42
*** syjulian has joined #openstack-nova21:42
*** rhagarty_ has joined #openstack-nova21:42
*** clarkb has joined #openstack-nova21:42
*** migi has joined #openstack-nova21:42
*** jogo has joined #openstack-nova21:42
*** Lehtu has joined #openstack-nova21:42
*** mkoderer__ has joined #openstack-nova21:42
*** dosaboy has joined #openstack-nova21:42
*** ansiwen has joined #openstack-nova21:42
*** mdbooth has joined #openstack-nova21:42
*** stevemar has joined #openstack-nova21:42
*** ttx has joined #openstack-nova21:42
*** timello has joined #openstack-nova21:42
*** scottynomad has joined #openstack-nova21:42
*** ishant|away has joined #openstack-nova21:42
*** toure has joined #openstack-nova21:42
*** markmc has joined #openstack-nova21:42
*** doffm has joined #openstack-nova21:42
*** brad[] has joined #openstack-nova21:42
*** apuimedo has joined #openstack-nova21:42
*** ajo has joined #openstack-nova21:42
*** jcook has joined #openstack-nova21:42
*** alineb has joined #openstack-nova21:42
*** gsilvis has joined #openstack-nova21:42
*** rrecio_ has joined #openstack-nova21:43
mriedemclaudiub: ok comments inline https://review.openstack.org/#/c/175479/ - we can catch up later since i know it's late21:43
*** jfding has quit IRC21:43
*** vdrok has quit IRC21:43
*** carl_baldwin has quit IRC21:44
*** betherly has quit IRC21:44
*** john51 has quit IRC21:45
*** john51 has joined #openstack-nova21:45
*** rrecio has quit IRC21:45
*** yonglihe has quit IRC21:45
*** thorst_ has quit IRC21:47
*** ediardo has joined #openstack-nova21:47
*** andrewbogott has joined #openstack-nova21:48
*** tdurakov has joined #openstack-nova21:49
*** jwcroppe has quit IRC21:49
pumaranikardansmith, back to patchset 1 for remove internal_id column. https://review.openstack.org/#/c/360666/21:50
*** geekinutah has joined #openstack-nova21:50
*** zhenguo has joined #openstack-nova21:51
pumaranikarplease review when you get time.21:51
*** unicell1 has quit IRC21:51
*** unicell has joined #openstack-nova21:52
*** unicell has quit IRC21:52
*** unicell has joined #openstack-nova21:52
*** bdeschenes has joined #openstack-nova21:53
*** macsz has joined #openstack-nova21:54
*** artom has quit IRC21:55
*** thorst_ has joined #openstack-nova21:56
*** tri2sing has quit IRC21:56
*** ayoung has joined #openstack-nova21:56
*** mvk has joined #openstack-nova21:58
*** burt has quit IRC21:58
*** thorst__ has joined #openstack-nova22:00
*** thorst_ has quit IRC22:00
*** jfding has joined #openstack-nova22:03
*** kgalanov has joined #openstack-nova22:03
*** cargonza has joined #openstack-nova22:04
*** GordonTX has joined #openstack-nova22:04
*** karlamrhein has joined #openstack-nova22:05
*** boris-42 has joined #openstack-nova22:07
*** yonglihe has joined #openstack-nova22:08
*** ildikov has joined #openstack-nova22:08
*** DuncanT has joined #openstack-nova22:08
*** carl_baldwin has joined #openstack-nova22:09
*** csatari has joined #openstack-nova22:10
*** thorst__ is now known as thorst22:11
*** artom has joined #openstack-nova22:12
*** vdrok has joined #openstack-nova22:13
*** fyxim has joined #openstack-nova22:13
*** serverascode has joined #openstack-nova22:14
*** bdeschenes has quit IRC22:14
*** mdrabe has quit IRC22:15
*** betherly has joined #openstack-nova22:15
*** thorst has quit IRC22:15
*** ameade has joined #openstack-nova22:15
openstackgerritEd Leafe proposed openstack/nova: Correct the sorting of datetimes for migrations  https://review.openstack.org/35312322:15
*** nikhil has joined #openstack-nova22:16
*** jwcroppe has joined #openstack-nova22:16
*** zhiyan has joined #openstack-nova22:16
*** rmcadams has joined #openstack-nova22:17
*** thorst has joined #openstack-nova22:17
openstackgerritClaudiu Belu proposed openstack/nova: WIP: api: discoverable policy  https://review.openstack.org/28053922:17
*** marst has quit IRC22:18
*** vishwanathj has quit IRC22:18
*** thorst has quit IRC22:18
*** xyang1 has quit IRC22:20
*** thorst has joined #openstack-nova22:20
*** tblakes has quit IRC22:23
*** lamt has quit IRC22:24
*** jwcroppe has quit IRC22:29
*** marst has joined #openstack-nova22:32
*** jwcroppe has joined #openstack-nova22:32
openstackgerritClaudiu Belu proposed openstack/nova: Fixes python 3 unit tests  https://review.openstack.org/27881022:34
*** annegentle has quit IRC22:38
*** jwcroppe has quit IRC22:39
*** alee has joined #openstack-nova22:42
*** annegentle has joined #openstack-nova22:44
*** jwcroppe has joined #openstack-nova22:46
*** mvk has quit IRC22:52
openstackgerritClaudiu Belu proposed openstack/nova: compute: fixes python 3 related unit tests  https://review.openstack.org/30026222:53
*** jwcroppe has quit IRC22:53
*** marst has quit IRC22:54
openstackgerritClaudiu Belu proposed openstack/nova: numa: Fixes NUMA topology related unit tests  https://review.openstack.org/27883522:56
*** GordonTX has quit IRC22:58
*** rama_y has joined #openstack-nova23:03
*** GordonTX has joined #openstack-nova23:03
*** kaisers has joined #openstack-nova23:04
*** annegentle has quit IRC23:04
*** maticue has quit IRC23:07
*** mvk has joined #openstack-nova23:08
*** kaisers has quit IRC23:09
*** lamt has joined #openstack-nova23:09
*** cdent has quit IRC23:12
*** ayoung has quit IRC23:13
*** ihrachys has quit IRC23:13
*** thorst has quit IRC23:19
*** thorst has joined #openstack-nova23:20
*** hamzy has joined #openstack-nova23:20
*** thorst_ has joined #openstack-nova23:24
*** thorst has quit IRC23:25
*** thorst has joined #openstack-nova23:28
*** HenryG has quit IRC23:28
*** HenryG has joined #openstack-nova23:28
*** thorst_ has quit IRC23:29
*** baoli has joined #openstack-nova23:32
openstackgerritMatt Riedemann proposed openstack/nova: Remove nova.image.s3 and configs  https://review.openstack.org/37443323:32
*** thorst has quit IRC23:32
*** GordonTX has quit IRC23:32
*** jwcroppe has joined #openstack-nova23:33
*** armax has quit IRC23:33
*** jwcroppe_ has joined #openstack-nova23:35
*** marst has joined #openstack-nova23:35
*** kuzko has quit IRC23:35
*** jwcroppe has quit IRC23:38
*** kuzko has joined #openstack-nova23:40
*** Swami has quit IRC23:42
*** kuzko has quit IRC23:45
*** kuzko has joined #openstack-nova23:49
*** TravT has quit IRC23:49
*** thorst has joined #openstack-nova23:51
*** thorst has quit IRC23:55
*** marst has quit IRC23:57

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