Wednesday, 2016-12-07

*** hamzy has joined #openstack-nova00:00
*** Apoorva has quit IRC00:03
*** zz_dimtruck is now known as dimtruck00:04
*** annegentle has quit IRC00:06
*** aweeks has quit IRC00:06
*** ijw has quit IRC00:06
*** ijw has joined #openstack-nova00:07
*** baoli has quit IRC00:07
*** lamt has quit IRC00:14
*** ducttape_ has quit IRC00:18
*** thorst_ has joined #openstack-nova00:19
*** ducttape_ has joined #openstack-nova00:20
*** thorst_ has quit IRC00:22
*** thorst_ has joined #openstack-nova00:23
*** yingjun has joined #openstack-nova00:23
*** catintheroof has quit IRC00:23
*** catintheroof has joined #openstack-nova00:24
openstackgerritDan Smith proposed openstack/nova: Make servers api view load instance fault from proper cell  https://review.openstack.org/40723800:24
openstackgerritDan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method  https://review.openstack.org/36755700:24
openstackgerritDan Smith proposed openstack/nova: WIP Multicell support for instance listing  https://review.openstack.org/39677500:24
openstackgerritDan Smith proposed openstack/nova: Make RPCFixture support multiple connections  https://review.openstack.org/39641700:24
openstackgerritDan Smith proposed openstack/nova: Setup CellsV2 environment in base test  https://review.openstack.org/40638000:24
openstackgerritDan Smith proposed openstack/nova: Add SecurityGroup.identifier to prefer uuid over name  https://review.openstack.org/40626000:24
openstackgerritDan Smith proposed openstack/nova: Add SingleCellSimple fixture  https://review.openstack.org/40568400:24
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937900:24
openstackgerritDan Smith proposed openstack/nova: Fix up non-cells-aware context managers in test_db_api  https://review.openstack.org/40637900:24
*** yingjun_ has joined #openstack-nova00:25
*** claudiub has joined #openstack-nova00:28
*** catintheroof has quit IRC00:28
*** yingjun has quit IRC00:29
*** claudiub|2 has quit IRC00:31
*** thorst_ has quit IRC00:32
*** tlian has quit IRC00:35
*** tjones has quit IRC00:35
*** ducttape_ has quit IRC00:36
*** mtanino has quit IRC00:39
*** diana_clarke has quit IRC00:39
*** toure|biab is now known as toure00:47
*** rama_y has quit IRC00:47
*** READ10 has joined #openstack-nova00:50
*** tuanluong has joined #openstack-nova00:51
*** yuntongjin has joined #openstack-nova00:51
*** tovin07 has joined #openstack-nova00:52
*** diana_clarke has joined #openstack-nova00:57
*** ijw has quit IRC00:59
*** hongbin has quit IRC01:00
*** STEWIE925 has quit IRC01:01
*** Apoorva has joined #openstack-nova01:02
*** Apoorva has quit IRC01:02
*** ducttape_ has joined #openstack-nova01:04
*** Apoorva has joined #openstack-nova01:04
*** Apoorva has quit IRC01:04
*** dave-mccowan has joined #openstack-nova01:04
*** Jeffrey4l has quit IRC01:06
*** Jeffrey4l has joined #openstack-nova01:06
*** jamielennox is now known as jamielennox|away01:07
*** Jeffrey4l has quit IRC01:08
*** Jeffrey4l has joined #openstack-nova01:09
*** annegentle has joined #openstack-nova01:09
*** lamt has joined #openstack-nova01:11
*** zhangjl has joined #openstack-nova01:14
*** annegentle has quit IRC01:14
*** ducttape_ has quit IRC01:18
*** Jeffrey4l has quit IRC01:20
*** Jeffrey4l has joined #openstack-nova01:21
*** jamielennox|away is now known as jamielennox01:21
*** zhangjl has quit IRC01:24
*** kevinz has joined #openstack-nova01:28
*** david-lyle has joined #openstack-nova01:33
*** yingjun_ has left #openstack-nova01:33
*** mriedem has joined #openstack-nova01:36
*** READ10 has quit IRC01:37
*** claudiub has quit IRC01:37
*** Jeffrey4l has quit IRC01:39
*** Jeffrey4l has joined #openstack-nova01:39
mriedemdansmith: did you put up the g-r bump for https://review.openstack.org/#/c/407719/ yet?01:40
dansmithmriedem: not the nova one01:40
dansmithmriedem: the release created a u-c bump in requirements01:41
dansmiththat merges then we get an auto-propose right?01:41
openstackgerritwanghao proposed openstack/nova: Set min_disk in the image meta based on the root volume's size  https://review.openstack.org/40739701:42
mriedemno we need the min g-r version bump01:42
mriedemso we need both i guess01:42
mriedemi'm on the case01:42
*** mingyu has joined #openstack-nova01:43
*** smatzek has joined #openstack-nova01:43
mriedemi can't handle watching barbie's life in the dreamhouse right now01:43
melwittas opposed to other times ... :)01:43
mriedemmelwitt: you know it01:44
mriedemcan't wait to see what dumbass thing ken does this time01:44
melwittlol01:45
mriedemdansmith: https://review.openstack.org/40779401:47
*** nic has quit IRC01:47
*** mingyu has quit IRC01:49
*** yuntongjin has quit IRC01:49
openstackgerritGhanshyam Mann proposed openstack/nova: Rename few tests as per new config options  https://review.openstack.org/40640601:50
*** mingyu has joined #openstack-nova01:51
openstackgerritGhanshyam Mann proposed openstack/nova: Merge v21 view builder for limit API  https://review.openstack.org/40678701:51
*** ijw has joined #openstack-nova01:51
*** wally__ has joined #openstack-nova01:52
*** Guest42705 has joined #openstack-nova01:52
*** gjayavelu has quit IRC01:53
Guest42705Hi All, I want to install and configure nova-lxd on my exiting openstack(mitaka) . so that I can run the linux container using LXD, is there any user guide for this?01:53
Guest42705can anyone knows how to do it?01:54
*** Sukhdev has quit IRC01:54
*** mdnadeem has joined #openstack-nova01:57
*** ijw has quit IRC01:58
*** yuntongjin has joined #openstack-nova01:58
mriedemGuest42705: see topic, this isn't a support channel. even still, lxd isn't in nova's tree, so you'd have to find support for that somewhere else01:59
mriedemcanonical maintains it i think01:59
mriedemmaybe find zigo?01:59
Guest42705thanks a lot\02:00
*** Guest42705 has quit IRC02:03
*** yuntongjin has quit IRC02:05
*** Jeffrey4l has quit IRC02:06
*** Jeffrey4l has joined #openstack-nova02:07
*** smatzek has quit IRC02:07
*** baoli has joined #openstack-nova02:12
*** thorst_ has joined #openstack-nova02:14
jaypipescmake .. && make && ctest -VV02:16
jaypipesoopsie... wrong term02:16
*** baoli has quit IRC02:17
*** ducnc has joined #openstack-nova02:18
oomichialex_xu: hi, can you take a look at https://review.openstack.org/#/c/387727/ an https://review.openstack.org/#/c/382060/ ?02:18
*** Apoorva has joined #openstack-nova02:20
*** zhangjl has joined #openstack-nova02:22
*** thorst_ has quit IRC02:23
*** jaypipes has quit IRC02:28
gcbgmann ping02:29
*** takashin has joined #openstack-nova02:30
*** thorst_ has joined #openstack-nova02:31
gcboomichi:  can you help look at https://review.openstack.org/#/c/400697/ when you're free ? it got a +202:32
*** shangxdy has joined #openstack-nova02:32
openstackgerritint32bit proposed openstack/python-novaclient: Fix can't process the resource with name 'help'  https://review.openstack.org/40338202:32
*** ducttape_ has joined #openstack-nova02:33
*** browne has quit IRC02:33
*** Apoorva has quit IRC02:34
*** Zer0Byte__ has quit IRC02:36
openstackgerritTakashi NATSUME proposed openstack/nova-specs: Amend cold-migration-with-target-ocata spec  https://review.openstack.org/39719302:36
openstackgerritTakashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases  https://review.openstack.org/39543602:37
openstackgerritTakashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases  https://review.openstack.org/39473802:37
openstackgerritTakashi NATSUME proposed openstack/nova: Remove mox in test_block_device_mapping_v1.py  https://review.openstack.org/30874602:37
oomichigcb: ok, I am checking. I did mistake a patch related to py3 before and the other project jobs were broken because we don't have py27 *tempest* jobs on nova gate02:37
openstackgerritTakashi NATSUME proposed openstack/nova: Stop using mox stubs in nova/tests/unit/cells  https://review.openstack.org/32328702:37
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix 'id' (attachment_id) parameters  https://review.openstack.org/34986302:37
oomichigcb: I'd like to pay attention for avoiding it again02:37
*** hogepodge has quit IRC02:38
gcboomichi, thanks  I have been working on  unit tests , Python 3 porting recently  https://etherpad.openstack.org/p/nova_py3502:39
oomichigcb: oh, great. That is a long list02:40
*** code-R has joined #openstack-nova02:41
gcboomichi,  there was work session about Python 3 in summit : https://etherpad.openstack.org/p/ocata-python-3  johnthetubaguy would like to help review. and merged some ones :-)02:42
gcbthat's the short history02:42
openstackgerritTuan Luong-Anh proposed openstack/python-novaclient: Replaces uuid.uuid4 with uuidutils.generate_uuid()  https://review.openstack.org/40683502:43
gcboomichi ,  do we have any plan or discussion about port functional tests to Python 3 before ?02:43
oomichigcb: I don't have anything, I just reviewed these patches based on technical things by checking code, doc and blogs :)02:44
gcboomichi, thanks for your time :-)02:46
*** hongbin has joined #openstack-nova02:46
*** thorst_ has quit IRC02:50
*** hfu has joined #openstack-nova02:50
*** thorst_ has joined #openstack-nova02:50
*** liuyulong_ has joined #openstack-nova02:52
*** liuyulong has quit IRC02:55
*** kaisers__ has joined #openstack-nova02:57
openstackgerritwanghao proposed openstack/nova: Set min_disk in the image meta based on the root volume's size  https://review.openstack.org/40739702:59
*** kristian__ has joined #openstack-nova02:59
*** thorst_ has quit IRC02:59
oomichigcb: I am not sure the reason of "Replace six.iteritems(dict) with dict.items()" yet02:59
*** kaisers_ has quit IRC03:00
*** zhangjl has quit IRC03:02
*** kristian__ has quit IRC03:03
*** mdnadeem has quit IRC03:04
oomichigcb: according to http://python-future.org/compatible_idioms.html#iterating-through-dict-keys-values-items , direct items() without six seems inefficient on py203:04
*** fragatina has quit IRC03:06
*** hfu has quit IRC03:07
*** mingyu has quit IRC03:07
*** hfu has joined #openstack-nova03:07
*** sreeram_v has joined #openstack-nova03:08
openstackgerritDiana Clarke proposed openstack/python-novaclient: Microversion 2.39 - Simple tenant usage pagination  https://review.openstack.org/40782503:11
openstackgerritliaonanhai proposed openstack/nova: Use uuidutils instead of uuid.uuid4()  https://review.openstack.org/39477503:13
openstackgerritOpenStack Proposal Bot proposed openstack/python-novaclient: Updated from global requirements  https://review.openstack.org/40625003:16
*** zhangjl has joined #openstack-nova03:17
*** hfu has quit IRC03:19
openstackgerritxianming.mao proposed openstack/nova: Move attach operation into the try-except  https://review.openstack.org/40782803:20
*** hfu has joined #openstack-nova03:20
*** browne has joined #openstack-nova03:21
*** prateek has joined #openstack-nova03:22
*** links has joined #openstack-nova03:22
*** Jeffrey4l has quit IRC03:23
*** Jeffrey4l has joined #openstack-nova03:24
*** hfu has quit IRC03:24
*** dave-mccowan has quit IRC03:25
*** hfu has joined #openstack-nova03:26
*** dave-mccowan has joined #openstack-nova03:27
*** tbachman_ has joined #openstack-nova03:28
*** ducnc has quit IRC03:28
*** GB21 has joined #openstack-nova03:29
*** tbachman has quit IRC03:30
*** tbachman_ is now known as tbachman03:30
*** prateek has quit IRC03:32
*** GB21 has quit IRC03:36
*** hfu has quit IRC03:37
*** hfu has joined #openstack-nova03:37
*** browne has quit IRC03:37
*** Jeffrey4l has quit IRC03:38
*** Jeffrey4l has joined #openstack-nova03:38
*** Amit_M has joined #openstack-nova03:39
*** shangxdy has quit IRC03:40
*** hfu has quit IRC03:43
*** ducttape_ has quit IRC03:44
*** ducttape_ has joined #openstack-nova03:45
*** GordonTX has joined #openstack-nova03:46
*** hfu has joined #openstack-nova03:47
*** liuyulong__ has joined #openstack-nova03:47
*** ducttape_ has quit IRC03:47
*** ducttape_ has joined #openstack-nova03:47
*** GordonTX has quit IRC03:48
*** liuyulong_ has quit IRC03:49
*** yufei has joined #openstack-nova03:50
*** ducttape_ has quit IRC03:52
*** sdatko has quit IRC03:54
mriedemadreznec: efried: do you guys have more coming than this anytime soon? like spawn/delete patches? https://review.openstack.org/#/c/391288/203:56
mriedemthat's the last progress in over a month03:57
*** thorst_ has joined #openstack-nova03:57
*** hfu has quit IRC03:57
*** hfu has joined #openstack-nova04:00
*** hfu has quit IRC04:01
*** sreeram_v has quit IRC04:01
mriedemjroll: any idea if we can make progress on this using a depends-on to the ironic changes? https://review.openstack.org/#/c/364413/04:01
mriedemhasn't been updated since september04:01
*** sreeram_v has joined #openstack-nova04:01
mriedemKevin_Zheng: are you blocked on this? https://review.openstack.org/#/c/394321/04:02
*** shangxdy has joined #openstack-nova04:02
*** thorst_ has quit IRC04:04
*** sdatko has joined #openstack-nova04:05
*** dave-mccowan has quit IRC04:06
*** sreeram_v has quit IRC04:06
openstackgerritVladik Romanovsky proposed openstack/nova: tests: avoid starting compute service twice in sriov functional test  https://review.openstack.org/40757204:06
openstackgerritVladik Romanovsky proposed openstack/nova: tests: generate correct pci addresses for fake pci devices  https://review.openstack.org/40773004:06
*** mingyu has joined #openstack-nova04:08
*** Jeffrey4l has quit IRC04:11
*** Jeffrey4l has joined #openstack-nova04:11
*** baoli has joined #openstack-nova04:12
*** psachin has joined #openstack-nova04:12
*** mingyu has quit IRC04:12
*** tbachman has quit IRC04:13
*** shangxdy has quit IRC04:14
*** tbachman has joined #openstack-nova04:14
*** pbandark has joined #openstack-nova04:15
*** mdnadeem has joined #openstack-nova04:18
gmanngcb: pong, sorry i was stuck in other gate fix04:18
*** tbachman has quit IRC04:19
*** links has quit IRC04:20
*** armax has quit IRC04:21
efriedmriedem (adreznec, thorst), regarding https://review.openstack.org/#/c/391288/ - we'd been waiting for core commentary and also a little held up with our CI.  At this point, lots of vacation time, so we won't get spawn/delete done until January.  I'm good with your strategy of ensuring that something more than the skeleton makes Ocata - we'll beef up that change set.04:23
*** ducttape_ has joined #openstack-nova04:27
*** Jeffrey4l has quit IRC04:29
*** Jeffrey4l has joined #openstack-nova04:30
*** Jeffrey4l has quit IRC04:30
*** Jeffrey4l has joined #openstack-nova04:31
*** psachin has quit IRC04:32
*** kaisers__ has quit IRC04:33
*** huhaoran has joined #openstack-nova04:36
*** code-R has quit IRC04:37
*** ducttape_ has quit IRC04:39
*** huhaoran has quit IRC04:41
*** shangxdy has joined #openstack-nova04:46
*** psachin has joined #openstack-nova04:49
*** code-R has joined #openstack-nova04:53
*** GB21 has joined #openstack-nova04:55
*** links has joined #openstack-nova04:57
*** shangxdy has quit IRC04:57
*** hongbin has quit IRC04:58
*** hfu has joined #openstack-nova04:59
*** thorst_ has joined #openstack-nova05:01
*** jamespage has quit IRC05:02
*** baoli has quit IRC05:02
*** baoli has joined #openstack-nova05:03
*** jamespag` has joined #openstack-nova05:03
*** ayogi has joined #openstack-nova05:03
*** kaisers_ has joined #openstack-nova05:04
*** hfu has quit IRC05:05
*** nicolasbock has quit IRC05:07
*** baoli has quit IRC05:07
*** yamahata has quit IRC05:08
*** kaisers_ has quit IRC05:09
*** thorst_ has quit IRC05:09
*** GB21 has quit IRC05:13
*** leifz has quit IRC05:14
*** robcresswell has quit IRC05:14
*** claudiub has joined #openstack-nova05:14
*** ameade_ has joined #openstack-nova05:14
*** zhenguo_ has joined #openstack-nova05:14
*** McNinja has quit IRC05:14
*** ameade has quit IRC05:14
*** lbragstad has quit IRC05:14
*** fyxim has quit IRC05:14
*** mgagne has quit IRC05:15
*** sballe_ has quit IRC05:15
*** dimtruck has quit IRC05:15
*** zhenguo has quit IRC05:15
*** dirk has quit IRC05:15
*** macsz_ has joined #openstack-nova05:15
*** zhenguo_ is now known as zhenguo05:15
*** swamireddy has quit IRC05:15
*** morgan has quit IRC05:15
*** wasmum has quit IRC05:15
*** ildikov has quit IRC05:15
*** DuncanT has quit IRC05:15
*** DuncanT_ has joined #openstack-nova05:15
*** DuncanT_ is now known as DuncanT05:16
*** krtaylor has quit IRC05:16
*** gmann has quit IRC05:16
*** betherly has quit IRC05:16
*** jamielennox has quit IRC05:16
*** macsz has quit IRC05:16
*** ildikov has joined #openstack-nova05:16
*** ameade_ is now known as ameade05:16
*** macsz_ is now known as macsz05:17
*** robcresswell has joined #openstack-nova05:17
*** fragatina has joined #openstack-nova05:17
*** fyxim has joined #openstack-nova05:19
*** dirk has joined #openstack-nova05:19
*** betherly has joined #openstack-nova05:19
*** lbragstad has joined #openstack-nova05:20
*** prateek has joined #openstack-nova05:20
*** wasmum has joined #openstack-nova05:20
*** sballe_ has joined #openstack-nova05:22
gcbgmann, got it , please help revisit python3 related patch when you're free , thanks05:23
*** swamireddy has joined #openstack-nova05:23
*** morgan has joined #openstack-nova05:23
*** gmann has joined #openstack-nova05:24
gcboomichi:  please look at section "Common patterns" in https://wiki.openstack.org/wiki/Python305:25
gcboomichi: Note: Replacing dict.iteritems()/.itervalues() with six.iteritems(dict)/six.itervalues(dict) was preferred in the past, but there was a discussion suggesting to avoid six for this. The overhead of creating a temporary list on Python 2 is negligible.05:26
*** fragatina has quit IRC05:26
*** ratailor has joined #openstack-nova05:27
*** fragatina has joined #openstack-nova05:27
*** krtaylor has joined #openstack-nova05:28
*** Sukhdev has joined #openstack-nova05:31
*** jamielennox has joined #openstack-nova05:31
*** ratailor has quit IRC05:31
*** GB21 has joined #openstack-nova05:33
gmannalex_xu: johnthetubaguy oomichi : this seems backward incompatible right - https://review.openstack.org/#/c/402372/505:33
gmannnot sure cinder does not make volume in -use if no destination type05:34
*** Jeffrey4l has quit IRC05:35
*** Jeffrey4l has joined #openstack-nova05:35
*** kaisers_ has joined #openstack-nova05:39
*** ducttape_ has joined #openstack-nova05:40
*** kaisers_ has quit IRC05:42
*** kaisers_ has joined #openstack-nova05:42
*** ducttape_ has quit IRC05:45
*** kaisers_ has quit IRC05:47
*** ratailor has joined #openstack-nova05:47
*** shangxdy has joined #openstack-nova05:49
*** armax has joined #openstack-nova05:49
openstackgerritint32bit proposed openstack/python-novaclient: [WIP]Add minDisk and minRam filters to list flavors  https://review.openstack.org/40785505:49
*** prateek has quit IRC05:51
*** hfu has joined #openstack-nova05:53
*** oanson has joined #openstack-nova05:54
*** GordonTX has joined #openstack-nova05:56
*** GordonTX has quit IRC05:56
*** hfu has quit IRC05:57
*** hfu has joined #openstack-nova05:58
*** b_rake has joined #openstack-nova05:59
*** b_rake has quit IRC05:59
*** b_rake has joined #openstack-nova06:00
*** kristian__ has joined #openstack-nova06:00
*** kristian__ has quit IRC06:00
*** hfu has quit IRC06:01
*** kristian__ has joined #openstack-nova06:01
*** jaosorior has joined #openstack-nova06:02
*** b_rake has quit IRC06:04
*** markvoelker has quit IRC06:05
*** markvoelker has joined #openstack-nova06:05
*** kristian__ has quit IRC06:05
*** prateek has joined #openstack-nova06:07
*** thorst_ has joined #openstack-nova06:07
*** sudipto has joined #openstack-nova06:08
*** sudipto_ has joined #openstack-nova06:08
*** markvoelker has quit IRC06:10
openstackgerritint32bit proposed openstack/python-novaclient: Add minDisk and minRam filters to list flavors  https://review.openstack.org/40785506:11
openstackgerritwanghao proposed openstack/nova: Set min_disk in the image meta based on the root volume's size  https://review.openstack.org/40739706:11
Kevin_Zhengmriedem: Ah, sort of, it is mostly about should it be added to the existing extension or not06:12
*** thorst_ has quit IRC06:14
*** wingwj has joined #openstack-nova06:14
*** int32bit has joined #openstack-nova06:15
*** mrsoul has quit IRC06:15
*** mrsoul has joined #openstack-nova06:16
*** armax has quit IRC06:18
openstackgerritxianming.mao proposed openstack/nova: Wrap attach operation into the try-except  https://review.openstack.org/40782806:21
alex_xugmann: I thought it just make a API failed sliently. Then that sounds like a bug06:25
*** hfu has joined #openstack-nova06:26
*** fragatina has quit IRC06:26
*** hfu has quit IRC06:27
openstackgerritxianming.mao proposed openstack/nova: Replace the assertEqual(None,A) with assertIsNone(A)  https://review.openstack.org/40787006:28
*** nandal has joined #openstack-nova06:28
*** huhaoran has joined #openstack-nova06:30
*** cburgess has quit IRC06:30
*** sach-m has quit IRC06:31
*** rmk has quit IRC06:31
openstackgerritxianming.mao proposed openstack/nova: Wrap attachment operation with the try-except  https://review.openstack.org/40782806:31
*** hfu has joined #openstack-nova06:32
*** hfu has quit IRC06:32
*** kashyap has quit IRC06:32
*** Cristina__ has joined #openstack-nova06:33
*** hfu has joined #openstack-nova06:33
*** rmk has joined #openstack-nova06:33
*** rmk has quit IRC06:33
*** rmk has joined #openstack-nova06:33
*** cburgess has joined #openstack-nova06:33
*** sach-m has joined #openstack-nova06:34
*** Cristina_ has quit IRC06:36
*** kashyap has joined #openstack-nova06:36
*** sudipto has quit IRC06:37
*** sudipto_ has quit IRC06:37
*** diga has joined #openstack-nova06:38
openstackgerritxianming.mao proposed openstack/nova: Wrap attachment operation with the try-except  https://review.openstack.org/40782806:38
*** jgriffith is now known as jgriffith_away06:39
*** hfu has quit IRC06:40
*** voelzmo has joined #openstack-nova06:43
*** markus_z has joined #openstack-nova06:43
*** kaisers_ has joined #openstack-nova06:43
*** josecastroleon has joined #openstack-nova06:43
*** voelzmo has quit IRC06:45
*** voelzmo has joined #openstack-nova06:46
*** huhaoran has quit IRC06:46
openstackgerritAndia proposed openstack/nova: comment  https://review.openstack.org/40787706:47
openstackgerritZhenyu Zheng proposed openstack/nova: Support tag instances when boot  https://review.openstack.org/39432106:47
*** kaisers_ has quit IRC06:47
*** kristian__ has joined #openstack-nova06:48
*** cfriesen has quit IRC06:48
*** huhaoran has joined #openstack-nova06:49
*** hfu has joined #openstack-nova06:49
*** lpetrut has joined #openstack-nova06:50
*** voelzmo has quit IRC06:50
*** kristian__ has quit IRC06:51
*** kristian__ has joined #openstack-nova06:51
*** masber has joined #openstack-nova06:51
*** mingyu has joined #openstack-nova06:54
*** kristian__ has quit IRC06:55
openstackgerritAndia proposed openstack/nova: quotas of instance in soft-delete vm_state  https://review.openstack.org/40787706:56
openstackgerritTuan Luong-Anh proposed openstack/python-novaclient: Replaces uuid.uuid4 with uuidutils.generate_uuid()  https://review.openstack.org/40683506:56
openstackgerritGhanshyam Mann proposed openstack/nova: Use bdm source type values from object fields  https://review.openstack.org/40788606:58
*** Sukhdev has quit IRC07:01
*** kristian__ has joined #openstack-nova07:05
*** kaisers_ has joined #openstack-nova07:05
*** markvoelker has joined #openstack-nova07:06
*** mjura has joined #openstack-nova07:09
*** hfu has quit IRC07:09
*** moshele has joined #openstack-nova07:09
*** hfu has joined #openstack-nova07:09
*** kaisers_ has quit IRC07:10
*** markvoelker has quit IRC07:10
*** andreas_s has joined #openstack-nova07:10
*** ducttape_ has joined #openstack-nova07:11
*** jamielennox is now known as jamielennox|away07:11
*** sileht has quit IRC07:13
openstackgerritBhagyashri Shewale proposed openstack/nova: Return 400 when name is more than 200 characters  https://review.openstack.org/39899807:13
*** thorst_ has joined #openstack-nova07:13
openstackgerritAndia proposed openstack/nova: quotas of instance in soft-delete vm_state  https://review.openstack.org/40787707:14
*** sridharg has joined #openstack-nova07:15
*** liuyulong_ has joined #openstack-nova07:15
*** ducttape_ has quit IRC07:16
*** mjura has quit IRC07:16
*** carthaca_ has joined #openstack-nova07:17
*** mjura has joined #openstack-nova07:17
*** liuyulong__ has quit IRC07:18
*** fragatina has joined #openstack-nova07:18
*** thorst_ has quit IRC07:19
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: live-migration test hook for serial console  https://review.openstack.org/34747107:21
*** jaosorior has quit IRC07:22
*** jaosorior has joined #openstack-nova07:23
*** zhangjl1 has joined #openstack-nova07:27
*** zhangjl has quit IRC07:28
tdurakovmarkus_z: overall looks good^, let's just wait for ci07:29
*** nkrinner_afk is now known as nkrinner07:30
*** zhangjl1 has quit IRC07:31
*** zhangjl has joined #openstack-nova07:32
*** rcernin has joined #openstack-nova07:34
*** edand has joined #openstack-nova07:35
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Check source_type and destination_type when booting with bdm provided  https://review.openstack.org/40236407:35
*** voelzmo has joined #openstack-nova07:39
*** yamahata has joined #openstack-nova07:41
*** pcaruana has joined #openstack-nova07:41
*** voelzmo has quit IRC07:44
*** voelzmo has joined #openstack-nova07:44
*** voelzmo has quit IRC07:45
*** voelzmo has joined #openstack-nova07:46
*** voelzmo has quit IRC07:48
*** ayogi has quit IRC07:49
*** voelzmo has joined #openstack-nova07:49
*** hfu has quit IRC07:50
*** mvk has quit IRC07:51
*** code-R has quit IRC07:51
*** hfu has joined #openstack-nova07:51
gmannalex_xu: but, boot is all ok there it just cinder does not tag volume as in-use, may be cinder side issue ?07:52
*** gszasz has joined #openstack-nova07:53
*** rybridges2 has quit IRC07:54
*** mfisch has quit IRC07:54
*** kaisers has quit IRC07:54
*** voelzmo has quit IRC07:55
*** voelzmo has joined #openstack-nova07:55
*** kaisers has joined #openstack-nova07:55
alex_xugmann: I feel the commit message isn't very clear. The BDM record is created in the API layer. But in the nova-compute, the BDM record with destination_type=None will be ignored totally07:55
alex_xugmann: so nova didn't talk with cinder at all. the logic of nova is just ignored the BDM record. So I think it is failed sliently07:56
*** kristian__ has quit IRC07:56
*** fragatina has quit IRC07:58
*** kristian__ has joined #openstack-nova07:59
zigomriedem: FYI, lxd isn't in Debian (yet), therefore Nova in Debian can't support lxd. I don't do Nova in Ubuntu, I only maintain packages in Debian.08:00
*** moshele has quit IRC08:01
*** mriedem has quit IRC08:01
*** voelzmo has quit IRC08:01
*** ratailor has quit IRC08:02
*** ratailor has joined #openstack-nova08:02
gmannalex_xu: ohh08:03
*** wingwj has quit IRC08:03
*** voelzmo has joined #openstack-nova08:04
*** irenab has quit IRC08:04
*** irenab has joined #openstack-nova08:05
*** ihrachys has joined #openstack-nova08:05
*** sapcc-bot has quit IRC08:06
*** sapcc-bot has joined #openstack-nova08:06
*** markvoelker has joined #openstack-nova08:07
*** kaisers_ has joined #openstack-nova08:07
*** rybridges2 has joined #openstack-nova08:07
gmannalex_xu: so you mean server would not boot in that case?08:08
alex_xugmann: server will boot, just ignore that bdm record in the compute node, and won't create any real disk for that bdm08:09
*** wingwj has joined #openstack-nova08:09
gmannalex_xu: yea, that was my understanding.08:09
gmannalex_xu: so in that case user will be 400 with that change08:10
*** gszasz has quit IRC08:10
gmannalex_xu: i got your point that it fail silently but server  still  accessible08:10
*** gszasz has joined #openstack-nova08:10
alex_xugmann: yeah08:11
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: WIP: Microversion 2.39 - Enable cold migration woth target host  https://review.openstack.org/40670708:11
gmannalex_xu: thats why i was thinking it is backward incompatible may be....08:11
*** markvoelker has quit IRC08:11
gmannalex_xu: because in api doc also we mentioned it is optional08:11
alex_xugmann: if remember correct, it is ignored at https://github.com/openstack/nova/blob/master/nova/virt/driver.py#L5808:12
*** ayogi has joined #openstack-nova08:12
*** owalsh has joined #openstack-nova08:13
alex_xugmann: when it isn't specified, it is None?08:14
gmannalex_xu: yea - it comes from here at APi layer - https://github.com/openstack/nova/blob/master/nova/block_device.py#L18508:15
*** thorst_ has joined #openstack-nova08:16
openstackgerritZhenyu Zheng proposed openstack/nova: Don't apply multi-queue to SRIOV ports  https://review.openstack.org/39754508:16
*** xiaoyang has joined #openstack-nova08:16
*** Jeffrey4l has quit IRC08:17
*** lpetrut has quit IRC08:17
*** Jeffrey4l has joined #openstack-nova08:17
alex_xugmann: check the guide http://docs.openstack.org/developer/nova/api_microversion_dev.html it sounds a bug08:17
*** irenab has quit IRC08:18
*** stvnoyes has quit IRC08:18
*** hfu has quit IRC08:18
alex_xugmann: maybe worth bring it up in the api meeting today, then double check with johnthetubaguy and sdague08:18
*** irenab has joined #openstack-nova08:18
*** zul has quit IRC08:18
*** stvnoyes has joined #openstack-nova08:19
gmannalex_xu: yea, it will be more clear from johnthetubaguy  sdague feedback08:19
gmannalex_xu: thanks for your input also08:20
alex_xugmann: +108:20
alex_xugmann: np08:20
*** gszasz has quit IRC08:20
*** hfu has joined #openstack-nova08:20
*** magic has quit IRC08:20
gmannalex_xu: i noted today meeting, last week i just forgot due to holiday here :)08:20
alex_xugmann: heh :)08:21
*** code-R has joined #openstack-nova08:21
*** jed56 has joined #openstack-nova08:21
*** shangxdy has quit IRC08:22
openstackgerritOleg Bondarev proposed openstack/nova: Live migration: wait for vif-plugged event on pre live migration  https://review.openstack.org/24691008:22
*** magic has joined #openstack-nova08:23
*** sach-m has quit IRC08:23
*** thorst_ has quit IRC08:23
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: WIP: Microversion 2.39 - Enable cold migration woth target host  https://review.openstack.org/40670708:24
*** pbandark has quit IRC08:25
*** mvk has joined #openstack-nova08:26
*** sileht has joined #openstack-nova08:26
*** xiaoyang has quit IRC08:26
*** slaweq_ has quit IRC08:27
*** lpetrut has joined #openstack-nova08:27
*** amoralej|off is now known as amoralej08:29
openstackgerritZhenyu Zheng proposed openstack/nova: Check destination_type when booting with bdm provided  https://review.openstack.org/40237208:30
*** mingyu has quit IRC08:31
*** zul has joined #openstack-nova08:32
*** slaweq_ has joined #openstack-nova08:32
*** kristia__ has joined #openstack-nova08:34
*** sach-m has joined #openstack-nova08:35
*** kristian__ has quit IRC08:35
*** gszasz has joined #openstack-nova08:39
*** shaohe_feng has quit IRC08:41
*** eliqiao_ has quit IRC08:41
*** Yingxin has quit IRC08:41
*** ducttape_ has joined #openstack-nova08:41
*** roeyc has joined #openstack-nova08:42
*** mkoderer has joined #openstack-nova08:42
*** hfu has quit IRC08:45
openstackgerritTakashi NATSUME proposed openstack/nova: Fix misuse of assertTrue  https://review.openstack.org/40793508:45
*** josecastroleon has quit IRC08:45
*** ducttape_ has quit IRC08:46
*** lpetrut has quit IRC08:46
*** hfu has joined #openstack-nova08:46
*** Yingxin has joined #openstack-nova08:47
openstackgerritxianming.mao proposed openstack/nova: Wrap attachment operation with the try-except  https://review.openstack.org/40782808:48
pkholkinalex_xu: thanks for review!08:50
alex_xupkholkin: np, thanks for patch also!08:50
*** sorrison_laptop has joined #openstack-nova08:51
pkholkinalex_xu: np! will wait Matt for review, too08:51
alex_xupkholkin: cool08:51
*** shaohe_feng has joined #openstack-nova08:52
*** ralonsoh has joined #openstack-nova08:53
*** shangxdy has joined #openstack-nova08:54
*** josecastroleon has joined #openstack-nova08:57
*** hfu has quit IRC08:57
*** hfu has joined #openstack-nova08:58
*** daemontool_ has quit IRC08:59
*** zzzeek has quit IRC09:00
*** sorrison_laptop is now known as sorrison_09:00
*** zzzeek has joined #openstack-nova09:01
*** code-R_ has joined #openstack-nova09:01
*** mfisch has joined #openstack-nova09:01
*** eliqiao has joined #openstack-nova09:01
*** mfisch has quit IRC09:02
*** mfisch has joined #openstack-nova09:02
*** sapcc-bot has quit IRC09:02
*** sapcc-bot has joined #openstack-nova09:02
*** wingwj has quit IRC09:04
*** code-R has quit IRC09:04
*** markvoelker has joined #openstack-nova09:07
alex_xupkholkin: do you have python-novaclient side patch ready?09:09
pkholkinalex_xu: no, I worked only only nova patch yet09:10
*** yassine has joined #openstack-nova09:10
pkholkinI will start09:10
*** yassine is now known as Guest7837209:11
*** eliqiao has quit IRC09:11
*** lpetrut has joined #openstack-nova09:11
alex_xupkholkin: thanks, just want to add it to my review list. let me know if it is ready09:11
*** markvoelker has quit IRC09:12
pkholkinalex_xu: ok, thanks!09:12
bauzasmdbooth: do you think we should wait for merging https://review.openstack.org/#/c/382024/7 as per your last comment ?09:16
*** zhangjl1 has joined #openstack-nova09:17
*** sudipto has joined #openstack-nova09:19
*** sudipto_ has joined #openstack-nova09:19
*** eliqiao has joined #openstack-nova09:20
openstackgerritZhenyu Zheng proposed openstack/python-novaclient: Check source_type and destination_type when booting with bdm provided  https://review.openstack.org/40236409:20
*** zhangjl has quit IRC09:20
*** eandersson has joined #openstack-nova09:22
openstackgerritHussain Chachuliya proposed openstack/nova: Fix 500 error while passing 4-byte unicode data  https://review.openstack.org/40751409:24
*** kristia__ has quit IRC09:25
*** paul-carlton has joined #openstack-nova09:26
*** moshele has joined #openstack-nova09:26
*** wingwj has joined #openstack-nova09:27
*** wingwj has quit IRC09:28
*** eandersson has quit IRC09:28
*** lpetrut has quit IRC09:28
*** lpetrut has joined #openstack-nova09:28
openstackgerritAndia proposed openstack/nova: quotas of instance in soft-delete vm_state  https://review.openstack.org/40787709:29
*** efoley has joined #openstack-nova09:31
*** sudipto has quit IRC09:31
*** sudipto_ has quit IRC09:31
*** sam_nowitzki has joined #openstack-nova09:31
*** lucas-afk is now known as lucasagomes09:32
*** matrohon has joined #openstack-nova09:33
*** mdnadeem has quit IRC09:33
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Fix doc generation errors  https://review.openstack.org/40796809:39
*** derekh has joined #openstack-nova09:40
*** rmart04 has joined #openstack-nova09:42
openstackgerritjichenjc proposed openstack/nova: Catch VolumeEncryptionNotSupported during spawn  https://review.openstack.org/40610909:44
*** eliqiao has quit IRC09:49
*** mdnadeem has joined #openstack-nova09:50
openstackgerritHironori Shiina proposed openstack/nova: ironic: Add soft power off support to ironic driver.  https://review.openstack.org/40797709:50
mdboothbauzas: No09:51
mdboothbauzas: We can address that when it lands09:51
*** tpatzig_ has joined #openstack-nova09:52
*** dgonzalez_ has joined #openstack-nova09:52
*** code-R_ has quit IRC09:52
*** david_1 has joined #openstack-nova09:52
*** tommy_ has joined #openstack-nova09:52
*** databus23_ has joined #openstack-nova09:52
*** code-R has joined #openstack-nova09:52
*** tommy_ is now known as Guest411909:52
*** sapcc-bot has quit IRC09:52
*** sapcc-bot has joined #openstack-nova09:53
*** tpatzig_ has quit IRC09:54
*** dgonzalez_ has quit IRC09:54
*** david_1 has quit IRC09:54
*** Guest4119 has quit IRC09:54
*** databus23_ has quit IRC09:54
*** mvk has quit IRC09:54
openstackgerritjichenjc proposed openstack/nova: Catch VolumeEncryptionNotSupported during spawn  https://review.openstack.org/40610909:55
*** mvk has joined #openstack-nova09:55
*** GB21 has quit IRC09:55
bauzasmdbooth: okay, so W'ipping it09:55
*** eliqiao has joined #openstack-nova10:00
*** code-R_ has joined #openstack-nova10:03
*** hfu has quit IRC10:05
*** kaisers_ has quit IRC10:05
*** thorst_ has joined #openstack-nova10:06
*** code-R has quit IRC10:06
*** hfu has joined #openstack-nova10:06
*** huhaoran has quit IRC10:07
*** markvoelker has joined #openstack-nova10:08
*** irenab has quit IRC10:09
*** irenab has joined #openstack-nova10:09
*** sorrison_ has quit IRC10:09
*** mpaolino has joined #openstack-nova10:12
*** ducttape_ has joined #openstack-nova10:12
*** mpaolino has quit IRC10:12
*** mpaolino has joined #openstack-nova10:12
*** markvoelker has quit IRC10:13
*** carthaca_ has quit IRC10:14
*** sapcc-bot has quit IRC10:14
*** carthaca_ has joined #openstack-nova10:14
*** david_1 has joined #openstack-nova10:14
*** sapcc-bot has joined #openstack-nova10:14
*** tpatzig_ has joined #openstack-nova10:14
*** dgonzalez_ has joined #openstack-nova10:14
*** tommy_ has joined #openstack-nova10:14
*** databus23_ has joined #openstack-nova10:14
*** tommy_ is now known as Guest2229110:14
*** WillUK2010 has joined #openstack-nova10:15
*** thorst_ has quit IRC10:15
*** paul-carlton2 has quit IRC10:15
*** tpatzig_ has quit IRC10:16
*** databus23_ has quit IRC10:16
*** Guest22291 has quit IRC10:16
*** dgonzalez_ has quit IRC10:16
*** hfu has quit IRC10:16
*** david_1 has quit IRC10:16
*** ducttape_ has quit IRC10:16
*** hfu has joined #openstack-nova10:16
*** tuanluong has quit IRC10:17
mdboothftersin: Hey, I just responded to your comment on https://review.openstack.org/#/c/333263/1810:18
mdboothYou're right and I missed the implications of that line in cache() for Lvm backends10:18
mdboothHowever, given that it was already broken, I don't want to complicate the series by attempting to fix it early10:18
mdboothIt gets fixed later in the series anyway10:18
mdboothYeah, handling swap the same way as ephemerals breaks swap, but it was already mostly hosed anyway.10:19
*** GB21 has joined #openstack-nova10:19
mdboothAt least it's now consistently hosed.10:19
*** eglynn has joined #openstack-nova10:23
*** hfu has quit IRC10:23
eglynnbauzas: the placement API service is not intended to be exposed via a publicURL, amiright?10:25
eglynn(context is whether the keystone endpoint for that API should have a publicURL attribute)10:25
bauzaseglynn: well10:26
bauzaseglynn: we are providing an URL for the service catalog using a service type named "placement"10:27
bauzaseglynn: but yeah, end-users shouldn't call the placement API, just the 'compute' service10:27
eglynnyep, and this service catalog entry can have 3 URLs right ... public, internal, admin10:28
kashyapmarkus_z: When you're about, can you also review this (this was discussed at Barcelona Summit) - https://review.openstack.org/#/c/313568/ -- Plugin to setup libvirt/QEMU from tar releases10:28
bauzaseglynn: that said, we're putting publicURL endpoint https://github.com/openstack-dev/devstack/blob/master/lib/placement#L130-L13910:28
bauzaseglynn: yup10:28
eglynnso just wondering if the publicURL should be left unset if it's not intended to be used10:28
bauzaseglynn: maybe10:29
bauzaseglynn: but we can also use the policy10:29
bauzasgood point to be discussed with the scheduler team10:29
bauzaseglynn: could you please write an email thread about that?10:29
kashyapmarkus_z: No rush, though.  (Myself currently occupied with some stuff.)10:29
bauzaseglynn: /me needs to bail out10:29
eglynnbauzas: thanks, o/10:29
bauzasFWIW, folks, I need to bail out for today10:30
openstackgerritPawel Koniszewski proposed openstack/nova: Change exponential function to linear  https://review.openstack.org/40751810:30
bauzasso good bye and thanks for the fish10:30
openstackgerritPawel Koniszewski proposed openstack/nova: Move migration_downtime_steps to libvirt/migration  https://review.openstack.org/40751710:30
openstackgerritPawel Koniszewski proposed openstack/nova: Mark live_migration_downtime_steps as deprecated for removal  https://review.openstack.org/40800210:30
eglynncontext for the question is the puppetry for the TripleO integration ... https://review.openstack.org/#/c/406300/10/manifests/keystone/auth_placement.pp10:30
markus_zkashyap: yep, will review that in ~2h (our CI got hit, have to fix that first)10:30
*** DuncanT has quit IRC10:33
*** josecastroleon has quit IRC10:33
*** DuncanT has joined #openstack-nova10:33
*** WillUK2010 has quit IRC10:34
* kashyap nods10:34
*** david-lyle_ has joined #openstack-nova10:35
*** shangxdy has quit IRC10:37
*** david-lyle has quit IRC10:37
*** kristian__ has joined #openstack-nova10:38
*** kristian__ has quit IRC10:39
*** kristia__ has joined #openstack-nova10:40
*** kristia__ has quit IRC10:40
*** kristian__ has joined #openstack-nova10:40
*** yufei has quit IRC10:41
*** kristian__ has quit IRC10:41
*** kristian__ has joined #openstack-nova10:41
*** kristian__ has quit IRC10:42
tdurakovbauzas, johnthetubaguy, hi folks, could you please take a look at https://review.openstack.org/#/c/407602/10:42
*** kristian__ has joined #openstack-nova10:42
*** jaosorior has quit IRC10:42
tdurakovah, need to read recent logs first(10:44
*** paul-carlton has quit IRC10:44
*** yamahata has quit IRC10:45
*** sudipto has joined #openstack-nova10:46
*** nandal has quit IRC10:46
*** sudipto_ has joined #openstack-nova10:46
*** McNinja has joined #openstack-nova10:47
*** paul-carlton has joined #openstack-nova10:47
*** kristian__ has quit IRC10:47
*** mgagne has joined #openstack-nova10:47
*** mgagne is now known as Guest261510:47
*** leifz has joined #openstack-nova10:48
*** zz_dimtruck has joined #openstack-nova10:50
*** zz_dimtruck is now known as dimtruck10:50
*** kristian__ has joined #openstack-nova10:54
*** kristia__ has joined #openstack-nova10:55
*** ftersin_ has joined #openstack-nova10:59
*** kristian__ has quit IRC10:59
*** erhudy has quit IRC11:00
*** erhudy has joined #openstack-nova11:00
*** ildikov has quit IRC11:00
*** ildikov has joined #openstack-nova11:00
*** claudiub has quit IRC11:01
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Combine injection info in InjectionInfo  https://review.openstack.org/33779011:01
*** cdent has joined #openstack-nova11:02
*** tbachman has joined #openstack-nova11:02
ftersin_mdbooth: hi. i understand that ephemeral formatting is already broken for lvm, and it is better to use the same way for ephemeral and swap formatting11:03
mdboothSorry I didn't spot it earlier, btw.11:03
mdboothI didn't understand what you were getting at.11:03
mdboothsfinucan: ^^^ just lost 2 +2s due to merge conflict, btw. Would you mind fast approving?11:05
sfinucanmdbooth: Sure11:05
mdboothsfinucan: Thanks11:05
*** john-davidge has joined #openstack-nova11:05
sfinucanmdbooth: No +W though?11:06
mdboothDon't know if that was because of the merge conflict already?11:06
mdboothsfinucan: You could review and add your own if you like :)11:06
*** rfolco has joined #openstack-nova11:06
mdboothftersin: I'm just reordering those patches, btw, and I'm going to add the missing test you spotted.11:07
*** sdague has joined #openstack-nova11:08
sfinucanmdbooth: iirc, I think I did. Two +2s and no serious changes anyway so I'll assume the +W was intended :)11:08
mdboothsfinucan: Thanks11:08
mdboothftersin: Are you ok with that, btw?11:10
mdboothI appreciate it's not a clear cut decision.11:10
*** GB21 has quit IRC11:10
*** tbachman has quit IRC11:12
*** carthaca_ has quit IRC11:12
*** sapcc-bot has quit IRC11:12
*** david_1 has joined #openstack-nova11:12
*** dgonzalez_ has joined #openstack-nova11:12
*** tpatzig_ has joined #openstack-nova11:12
*** databus23_ has joined #openstack-nova11:12
*** carthaca_1 has joined #openstack-nova11:12
*** sapcc-bot2 has joined #openstack-nova11:12
*** thorst_ has joined #openstack-nova11:13
*** tbachman has joined #openstack-nova11:13
*** int32bit has quit IRC11:13
mdboothftersin: Hmm, on second thoughts, I don't think it will be simple to re-order those 2 commits11:13
*** tpatzig_ has quit IRC11:14
*** databus23_ has quit IRC11:14
*** dgonzalez_ has quit IRC11:14
*** david_1 has quit IRC11:14
*** zhangjl1 has quit IRC11:14
*** tommy_ has joined #openstack-nova11:15
cdent"ephemeral formatting" is a funny phrase, out of context11:15
*** tommy_ is now known as Guest9972611:15
mdboothcdent: Hehe11:15
ftersin_mdbooth: great, thanks for the test. as for reordering: i've got it now - you have to fix lvm to analyze image_id instead of ephemeral_size to get 'generating' flag correctly11:15
gcbsfinucan: congratulations, so can you convert +1 to +2 on https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/goal-python35  :-)11:16
*** Guest99726 has quit IRC11:17
ftersin_mdbooth: i understand that you want to minimize side works for the series, but the result looks weird for me: some compatibility is broken, some code which is no longer called is being refactored11:18
mdboothftersin: Which code is no longer being called?11:18
ftersin_mdbooth: _create_swap11:18
mdboothWell it is still being called, just not for Lvm11:19
ftersin_and you extend its interface with a parameter needed for lvm only11:19
mdboothWell, that's true. But when we fix it it will be needed.11:20
mdboothLvm *should* be calling that, but it isn't.11:20
mdboothAll I've done is make swap and ephemeral the same.11:20
*** thorst_ has quit IRC11:20
ftersin_i know it, it will be needed indeed. but i'm not sure about time range of 'when' :)11:20
mdboothYeah, that's why I haven't wanted to hold up other patches :/11:21
mdboothHowever, all the code's there, and we are getting through it.11:21
mdboothftersin: Is Lvm-backed with swap but no ephemeral currently important to you?11:22
*** kristia__ has quit IRC11:23
*** ociuhandu has joined #openstack-nova11:23
*** gcb has quit IRC11:24
*** GB21 has joined #openstack-nova11:24
ftersin_mdbooth: in fact i do not care about lvm, but scaleio is important for me. And without that code working properly, i cannot get scaleio driver formatting these damned ephemerals11:25
ftersin_see https://review.openstack.org/#/c/407440/11:25
mdboothftersin: Looking11:25
*** sam_nowitzki has quit IRC11:25
*** geekinutah has quit IRC11:26
*** geekinutah has joined #openstack-nova11:26
ftersin_mdbooth: However it could be left as 'known issue' and fixed later11:26
mdboothftersin: Actually, that fix 'if target != base' is probably sane11:27
mdboothWe could take that as a separate patch, no?11:27
ftersin_mdbooth: sure11:27
mdboothWould that address your problems with ScaleIO?11:28
*** ociuhandu has quit IRC11:28
*** ociuhandu has joined #openstack-nova11:29
ftersin_yep. the only question is time. if a right fix depends of an enormous series...11:29
mdboothWell that's just a little fix we could slip in at the beginning11:30
mdboothIn fact, I'd be prepared to put it in just before the 211:30
ftersin_as a branch of your series, right?11:30
*** openstackgerrit has quit IRC11:32
mdboothftersin: Well if I put it in my series, it would currently be number 211:33
mdboothSo hopefully it would be merged within a couple of days11:33
*** cdelatte has joined #openstack-nova11:34
*** sudipto_ has quit IRC11:36
*** sudipto has quit IRC11:36
*** mpaolino_ has joined #openstack-nova11:41
sfinucangcb: That's one https://review.openstack.org/#/c/31953211:41
*** openstackgerrit has joined #openstack-nova11:42
openstackgerritChris Dent proposed openstack/nova: HTTP interface for resource providers by aggregates  https://review.openstack.org/40774211:42
*** ducttape_ has joined #openstack-nova11:42
*** mpaolino has quit IRC11:43
*** tbachman has quit IRC11:44
ftersin_mdbooth: i'd be excellent, thanks11:45
*** code-R_ has quit IRC11:46
mdboothftersin: I'm going to try to incorporate your connect() calls in my ensure interface too, btw11:46
*** ducttape_ has quit IRC11:47
*** smatzek has joined #openstack-nova11:54
mdboothftersin: Do you know if there's a bug for Lvm ephemerals already, btw?11:54
*** catintheroof has joined #openstack-nova11:55
openstackgerritChris Dent proposed openstack/nova-specs: Update the generic resource pools to reflect reality  https://review.openstack.org/40756211:55
*** sahid has joined #openstack-nova11:55
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Microversion 2.39 - Enable cold migration woth target host  https://review.openstack.org/40670711:57
ftersin_mdbooth: i don't11:57
mdboothftersin: Ok. I'll have a search and create one if I can't find it.11:57
openstackgerritint32bit proposed openstack/nova: Choose VMware datastore by the least provisioned space  https://review.openstack.org/40263211:57
openstackgerritStephen Finucane proposed openstack/nova: conf: Improve help text for network options  https://review.openstack.org/34120311:57
openstackgerritStephen Finucane proposed openstack/nova: conf: Deprecate all nova-net related opts  https://review.openstack.org/39568311:57
*** ociuhandu has quit IRC11:58
openstackgerritStephen Finucane proposed openstack/nova: conf: Deprecate all nova-net related opts  https://review.openstack.org/39568311:59
openstackgerritStephen Finucane proposed openstack/nova: conf: Improve help text for network options  https://review.openstack.org/34120311:59
*** code-R has joined #openstack-nova12:00
*** paul-carlton1 has joined #openstack-nova12:01
*** kaisers_ has joined #openstack-nova12:01
*** nicolasbock has joined #openstack-nova12:01
*** zenoway has joined #openstack-nova12:03
*** paul-carlton has quit IRC12:03
*** zenoway has quit IRC12:03
cdentedleafe, bauzas: should we consider moving the scheduler meeting if it is going to be right in the middle of bauzas needing to do the school run?12:03
*** zenoway has joined #openstack-nova12:03
mdboothftersin: Hey, there might be a problem with that patch.12:03
mdboothWon't it cause ephemerals and swap to be overwritten in some cases?12:04
mdboothMaybe hard reboot?12:04
mdboothFor ScaleIO, maybe cold migration?12:04
mdboothWait, maybe not12:06
ftersin_mdbooth: scaleio is a shared block storage12:06
ftersin_its disks are accessible globally, from every host12:07
mdboothftersin: Yeah, ignore me. That call is already protected by the other self.exists() check12:08
openstackgerritSergey Nikitin proposed openstack/nova: Added constraints for 'hacking' lib for pep8  https://review.openstack.org/40805512:08
*** paul-carlton1 has quit IRC12:09
*** ratailor has quit IRC12:09
*** dave-mccowan has joined #openstack-nova12:09
*** markvoelker has joined #openstack-nova12:09
*** gibi has quit IRC12:14
*** markvoelker has quit IRC12:15
*** irenab has quit IRC12:18
*** irenab has joined #openstack-nova12:18
*** nkrinner has quit IRC12:18
*** thorst_ has joined #openstack-nova12:18
openstackgerritSergey Nikitin proposed openstack/nova: Added constraints for 'hacking' lib for pep8  https://review.openstack.org/40805512:19
*** prateek has quit IRC12:19
*** rtheis has joined #openstack-nova12:19
*** nkrinner has joined #openstack-nova12:21
openstackgerritLi Yingjun proposed openstack/nova: Adds view builders for hypervisors controller  https://review.openstack.org/33528212:22
*** thorst_ has quit IRC12:25
*** baoli has joined #openstack-nova12:26
*** baoli has quit IRC12:26
*** edmondsw_ has joined #openstack-nova12:26
*** baoli has joined #openstack-nova12:26
*** nkrinner has quit IRC12:28
*** oanson has quit IRC12:28
*** GB21 has quit IRC12:28
*** xiaoyang has joined #openstack-nova12:31
*** catintheroof has quit IRC12:32
*** jaosorior has joined #openstack-nova12:32
*** catintheroof has joined #openstack-nova12:32
*** bvanhav__ has joined #openstack-nova12:32
*** magic has quit IRC12:33
*** ociuhandu has joined #openstack-nova12:34
*** magic has joined #openstack-nova12:34
markus_zwznoinsk: got a second? I have a CI related question12:36
*** xiaoyang has quit IRC12:36
*** catintheroof has quit IRC12:37
markus_zOur s390x CI needs a custom flavor, none of the pre-configured flavors in Devstack are suitable for the tempest tests. Where should I trigger the creation of such a custom flavor in our CI?12:37
sdaguemarkus_z: local.sh ?12:38
sdagueor maybe with a devstack plugin12:38
sdagueyou could run that at post-extra phase12:39
BobBallI've used pre_test_hook for that sort of thing in the past12:39
markus_zIt's already possible in pre_test_hook?12:39
sdaguemarkus_z: if you are running devstack-gate, yeh, you can run arbitrary code there12:39
sdaguebut, you might want to just bundle it into a devstack plugin so that people could more easily do it manually12:40
markus_zsdague: yep, we're using that12:40
*** cdent has quit IRC12:40
*** xiaoyang has joined #openstack-nova12:40
*** takashin has left #openstack-nova12:41
markus_zsdague: OK, cool, then I use a short-term solution in pre_test_hook and a mid/long-term solution as plugin.12:41
*** nkrinner has joined #openstack-nova12:42
*** portdirect has joined #openstack-nova12:42
*** magic has quit IRC12:42
markus_zthanks12:43
*** magic has joined #openstack-nova12:43
alex_xunova api subteam meeting is in 15 mins at #openstack-meeting-412:45
*** xiaoyang has quit IRC12:45
*** thorst_ has joined #openstack-nova12:47
*** lucasagomes is now known as lucas-hungry12:49
openstackgerritSergey Nikitin proposed openstack/nova: Revert "reduce pep8 requirements to just hacking"  https://review.openstack.org/40806912:49
*** gcb has joined #openstack-nova12:50
*** edmondsw_ has quit IRC12:50
*** baoli has quit IRC12:51
*** catintheroof has joined #openstack-nova12:54
*** kylek3h__ has quit IRC12:56
sfinucanmarkus_z: Bit late, but we do it in the tests themselves12:58
sfinucanfor intel-nfv-ci at least12:58
*** kevinz has quit IRC12:59
markus_zsfinucan: Do you have an example? The reason for the custom flavor is the image we built. It's bigger than the default cirros which doesn't run on s390x.12:59
*** jaosorior has quit IRC12:59
*** gmann_ has joined #openstack-nova12:59
sfinucanmarkus_z: https://github.com/openstack/intel-nfv-ci-tests/blob/master/intel_nfv_ci_tests/tests/scenario/test_cpu_pinning.py#L118-L13713:00
alex_xujohnthetubaguy: sdague are you around for api meeting?13:00
*** lamt has quit IRC13:01
markus_zsfinucan: Ah, those are tests you run additionally to the existing ones in Tempest, right?13:01
*** voelzmo has quit IRC13:02
sfinucanmarkus_z: Ah, yes. Sorry - you mean all tests with a custom flavor?13:02
markus_zyes, exactly13:02
markus_zSomething for `flavor_ref` in the tempest.conf13:02
markus_zBobBall: Can you point me to your pre_test_hook solution please?13:03
sdaguealex_xu: I am, johnthetubaguy is on honeymoon iirc13:05
alex_xusdague: cool, we are at #openstack-meeting-413:06
sdaguesorry, apparently my hexxchat config didn't add -4 to my channel lists when I readded13:06
alex_xujohnthetubaguy: enjoy your honeymoon :)13:06
*** tbachman has joined #openstack-nova13:06
*** ducttape_ has joined #openstack-nova13:06
tdurakovsdague: hi, could you please approve https://review.openstack.org/#/c/407602/13:06
sdaguetdurakov: +A13:07
*** efoley_ has joined #openstack-nova13:07
tdurakovthanks!13:07
tdurakovmarkus_z: hope it will be last recheck for serial console hook)13:08
*** gibi has joined #openstack-nova13:08
*** amoralej is now known as amoralej|lunch13:08
*** efoley has quit IRC13:08
*** mriedem has joined #openstack-nova13:09
*** tblakes has joined #openstack-nova13:10
*** jaosorior has joined #openstack-nova13:10
markus_ztdurakov: :) yeah, it was odd that grenade didn't even reach step 2 of the hooks13:11
*** markvoelker has joined #openstack-nova13:12
*** code-R has quit IRC13:12
openstackgerritFrancesco Santoro proposed openstack/os-vif: os-vif: add vif_name to VIFVHostUser class  https://review.openstack.org/39022513:12
openstackgerritFrancesco Santoro proposed openstack/os-vif: os-vif: add new port profiles to enable fast path vhostuser  https://review.openstack.org/37553313:12
*** kristian__ has joined #openstack-nova13:13
*** catinthe_ has joined #openstack-nova13:14
mriedemjroll: https://review.openstack.org/#/c/407977/13:16
mriedemjust fyi13:16
*** catintheroof has quit IRC13:16
*** mdnadeem has quit IRC13:16
*** markvoelker has quit IRC13:16
*** heroux has joined #openstack-nova13:17
*** voelzmo has joined #openstack-nova13:17
jrollmriedem: O_o13:17
*** kristian__ has quit IRC13:17
mriedemgonna have the same issue here https://review.openstack.org/#/c/403745/13:17
*** jed56 has quit IRC13:17
*** markvoelker has joined #openstack-nova13:19
*** code-R has joined #openstack-nova13:19
*** claudiub has joined #openstack-nova13:20
*** matrohon has quit IRC13:21
*** code-R has quit IRC13:23
*** dinobot_ has joined #openstack-nova13:23
*** ducttape_ has quit IRC13:23
*** ducttape_ has joined #openstack-nova13:26
*** cdent has joined #openstack-nova13:26
*** kylek3h_away has joined #openstack-nova13:26
*** hfu has joined #openstack-nova13:27
*** code-R has joined #openstack-nova13:28
*** kevinz has joined #openstack-nova13:28
*** ayogi has quit IRC13:31
*** code-R has quit IRC13:32
BobBallmarkus_z: Sorry - correction - you can't use pre-test-hook as it's before devstack is actually set up; I was mis-remembering.13:33
*** jaypipes has joined #openstack-nova13:33
markus_zBobBall: Yeah, the logs look like this is the behavior13:34
markus_zBut I can use the hook to write a proper local.sh, I guess13:35
BobBallmarkus_z: potentially you could define your own gate_hook which sets DEVSTACK_GATE_TEMPEST=0 and runs devstack-vm-gate.sh then run tempest with a static command, since you know how you want to run it, you can build your tempest command from https://github.com/openstack-infra/devstack-gate/blob/master/devstack-vm-gate.sh#L76013:35
mriedempkholkin: some small things inline here https://review.openstack.org/#/c/405461/13:35
mriedempkholkin: but it's very close13:35
BobBallYeah - a local.sh should work I guess13:35
*** code-R has joined #openstack-nova13:35
mriedemBobBall: while you're here, has this started? https://blueprints.launchpad.net/nova/+spec/add-os-xenapi-library13:35
BobBallmriedem: Absolutely13:36
mriedemon the nova side?13:36
BobBallmriedem: https://review.openstack.org/#/c/406059/13:36
mriedemnothing is linked to the bp13:36
pkholkinmriedem: ops, just update api-ref)13:36
pkholkinthanks! will look13:36
mriedemBobBall: the bp link in that nova change is wrong13:36
mriedemso it's not linked to the bp in LP13:36
BobBallmriedem: Indeed.  the 'add' is missing.  Will comment on the change to that effect too.13:37
markus_zBobBall: Thanks, I'm fine with a short-term solution and then I'll take a deeper dive into the CI world.13:37
mriedemBobBall: i got it13:37
mriedempkholkin: it's a couple other small things too13:37
pkholkinmriedem: yes, looking13:38
jaypipesmorning supernovas.13:38
*** sudipto has joined #openstack-nova13:38
*** code-R has quit IRC13:38
*** sudipto_ has joined #openstack-nova13:38
BobBallmriedem: We're planning to have a series of 3 changes in Nova; one to make use of os-xenapi (so it shows what the refactor is), one to remove the now-unused files and then one to change to use the new API.  We figured reviews would be easier this way.  Thoughts?13:39
mriedemjaypipes: o/13:39
BobBallWhat about us normal non-super-but-still-novas?  hi jaypipes :)13:40
jaypipesBobBall: you're still a supernova, Bob :)13:40
* BobBall likes jaypipes 13:40
jaypipesheh13:41
*** lamt has joined #openstack-nova13:41
tbachmanhey folks — I’m doing some debugging on some sqlalchemy (unrelated project), and came across this patch: https://review.openstack.org/#/c/395253/13:42
*** ducttape_ has quit IRC13:42
tbachmanThe commit message for the revert says “Currently causing nova periodic jobs to fail."13:42
tbachmanIs anyone familiar with this change, and what was failing in nova?13:42
*** ljxiash has joined #openstack-nova13:42
* tbachman is also going to ask on openstack-osla13:43
mriedemtbachman: harlowja is your guy for that13:44
tbachmanmriedem: thx!13:44
mriedemit was blocking a new release of oslo.context i think13:44
jaypipesmdbooth: I presume you're working through rebasing your imagebackend series? :)13:44
mdboothjaypipes: Yep13:44
jaypipescoolio :)13:44
mriedemBobBall: seems ok, i don't really have a comment for how that's going to look13:45
*** kfarr has joined #openstack-nova13:45
openstackgerritZhenyu Zheng proposed openstack/nova: Don't apply multi-queue to SRIOV ports  https://review.openstack.org/39754513:46
jaypipesedleafe, cdent: how about "member_of" as the name of the "resource providers in these aggregates" querystring param?13:46
cdentjaypipes: will that need an 'aggregate' qualifier?13:47
cdentnaming is hard13:47
* cdent goes shopping13:47
jaypipescdent: I don't think so... aggregates is just our group/membership bucket.13:47
cdenttrue13:47
jaypipescdent: I don't imagine having another grouping mechanism.13:47
cdentalso true13:48
cdentI'm happy with "member_of"13:48
* cdent waits for edleafe to wake up before making the change13:49
*** rcernin has quit IRC13:49
*** tbachman has quit IRC13:50
*** tlian has joined #openstack-nova13:51
*** rcernin has joined #openstack-nova13:51
openstackgerritLudovic Beliveau proposed openstack/nova: Fix nova-serialproxy when registering cli options  https://review.openstack.org/40760213:51
lbeliveausfinucan: sdague: ^ patch was in merge conflict13:52
sfinucanlbeliveau: done13:53
*** tyrola has quit IRC13:53
lbeliveausfinican: nevermind ... just a rebase, no need to do anything, my bad :)13:53
*** Guest2615 is now known as mgagne13:54
*** mgagne has quit IRC13:54
*** mgagne has joined #openstack-nova13:54
*** ljxiash_ has joined #openstack-nova13:55
*** shangxdy has joined #openstack-nova13:55
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Mock imagebackend template funcs in ImageBackendFixture  https://review.openstack.org/40087113:56
openstackgerritDiana Clarke proposed openstack/nova: Simple tenant usage pagination  https://review.openstack.org/38609313:57
*** ljxiash has quit IRC13:58
*** lucas-hungry is now known as lucasagomes13:58
*** ducttape_ has joined #openstack-nova13:59
*** kfarr has quit IRC13:59
*** bkopilov has quit IRC13:59
*** links has quit IRC13:59
*** bkopilov has joined #openstack-nova13:59
*** bvanhav_ has joined #openstack-nova14:01
*** bvanhav__ has quit IRC14:02
*** catintheroof has joined #openstack-nova14:02
*** prateek has joined #openstack-nova14:03
*** lamt has quit IRC14:03
*** catinthe_ has quit IRC14:04
*** esberglu has joined #openstack-nova14:05
edleafecdent: jaypipes: been awake; just getting the family out the door :)14:05
*** lamt has joined #openstack-nova14:06
jaypipesedleafe: no prolemo.14:06
edleafecdent: 'member_of' is fine. Big improvement14:06
jaypipesor problemo..14:06
jaypipescdent: cool. ship it.14:06
cdentedleafe: cool, can do america14:06
pkholkinmriedem: quick comment here, everywhere in the code I found self.assertFalse(mocked.called) pattern14:06
*** prateek_ has joined #openstack-nova14:06
*** shangxdy has quit IRC14:06
*** mdrabe has joined #openstack-nova14:07
*** amoralej|lunch is now known as amoralej14:07
*** mdnadeem has joined #openstack-nova14:07
*** andymaier has joined #openstack-nova14:07
*** andymaier_ has joined #openstack-nova14:07
edleafecdent: while you're on that - this looks like something that really should be in an API utility: https://review.openstack.org/#/c/407742/2/nova/api/openstack/placement/handlers/resource_provider.py@18614:08
*** andymaier_ has quit IRC14:08
jaypipespkholkin: yes, because older versions of unittest.mock don't have an assert_not_called() method.14:08
jaypipesbauzas: lemme know when you have some time to chat about your patch that adds the POST /requests thing14:08
pkholkinjaypipes: so, this new one is better?14:08
edleafecdent: not a showstopper, of course, but a pattern to keep in mind14:08
cdentedleafe: I was going to wait on that. YAGNI and all that.14:08
*** prateek has quit IRC14:08
jaypipespkholkin: you mean the old one?14:08
openstackgerritDiana Clarke proposed openstack/python-novaclient: Microversion 2.39 - Simple tenant usage pagination  https://review.openstack.org/40782514:09
jaypipespkholkin: I always use self.assertFalse(mock_obj.called)14:09
edleafecdent: sure, but YAGNI could also mean "you ARE gonna need it" :)14:09
pkholkinjaypipes: what do you prefer?14:09
pkholkinhm14:09
pkholkinMatt asked me to use assert_not_called() instead of that14:09
cdentedleafe: I want to make sure that we don't enable complex query processing until we need complex (generic) query processing.14:09
cdentBecause we don't want complex queries except where explicitly necessary14:09
* cdent send edleafe to ingy14:10
cdentwhen we get there, we'll deal with it14:10
snikitinmriedem: Hi! Now results of CI pep8 job are false positive for some cases because of bug https://bugs.launchpad.net/nova/+bug/164805414:11
openstackLaunchpad bug 1648054 in OpenStack Compute (nova) "Upper constraints for 'hacking' lib doesn't work for pep8" [Medium,In progress] - Assigned to Sergey Nikitin (snikitin)14:11
snikitinI created a fix for it https://review.openstack.org/#/c/408069 but I'm not sure that this is a best solution. I wanted to consult with you. could you look, please?14:11
*** ducttape_ has quit IRC14:12
jaypipespkholkin: go with whatever mriedem wants I guess :)14:12
gcbmriedem:  about https://blueprints.launchpad.net/nova/+spec/goal-python35 ,  does this BP include functional tests Python 3 porting ?14:12
pkholkinjaypipes: agree )14:14
mriedempkholkin: the error message from assert_not_called is probably more useful than 'False != True'14:14
mriedemgcb: i think it's any and all things to make nova py3 compliant14:14
mriedemexcept the mox > mock conversion which is a separate blueprint14:15
*** baoli has joined #openstack-nova14:15
openstackgerritStephen Finucane proposed openstack/nova: conf: move console opt to console group  https://review.openstack.org/38964714:16
openstackgerritStephen Finucane proposed openstack/nova: console: Provide a ConsoleDriver interface  https://review.openstack.org/39217414:16
openstackgerritStephen Finucane proposed openstack/nova: Use stevedore for console driver  https://review.openstack.org/39217314:16
gcbmriedem, got it.  I have been working on porting unittests to Python 3 - https://etherpad.openstack.org/p/nova_py35. so we can start functional tests porting at the same time ?14:17
*** parora has joined #openstack-nova14:17
cdentedleafe: did you see my comments earlier about perhaps moving the scheduler meeting?14:17
edleafecdent: yes14:17
mriedemsnikitin: you might want to ask sdague. i don't like the alternative of duplicating the version constraints in both test-requirements.txt and tox.ini, although i don't think the version range on hacking changes much14:17
edleafeShould we discuss that at the next scheduler meeting? :)14:17
mriedemand with wheels, installing from test-requirements isn't as painful as it used to be14:17
*** annegentle has joined #openstack-nova14:19
*** prateek_ has quit IRC14:19
*** prateek has joined #openstack-nova14:19
*** diga has quit IRC14:20
snikitinmriedem ty! I will ping sdague14:20
cdentedleafe: I was going to put it on the agenda, and had that paradox14:20
cdentso resigned myself to here14:20
*** jbernard has quit IRC14:21
edleafecdent: well, since bauzas is the impetus for the move, and he's gone for the day, maybe discuss it tomorrow?14:21
*** mtanino has joined #openstack-nova14:22
*** parora has quit IRC14:22
cdentah, I did not know he was gone for the day14:22
openstackgerritint32bit proposed openstack/nova: Choose VMware datastore by the least provisioned space  https://review.openstack.org/40263214:22
cdentbouncers--14:22
*** READ10 has joined #openstack-nova14:23
*** gcb has quit IRC14:24
sdaguesnikitin: where is the patch in question?14:25
*** jbernard has joined #openstack-nova14:26
openstackgerritPavel Kholkin proposed openstack/nova: [proxy-api] microversion 2.39 deprecates image-metadata proxy API  https://review.openstack.org/40546114:26
pkholkinmriedem: ok, pushed new patch set with fixes ^14:26
*** mriedem has quit IRC14:28
*** krtaylor has quit IRC14:28
*** prateek_ has joined #openstack-nova14:29
*** krtaylor has joined #openstack-nova14:29
*** ljxiash_ has quit IRC14:29
*** ducttape_ has joined #openstack-nova14:30
*** Jeffrey4l has quit IRC14:30
snikitinsdague https://review.openstack.org/#/c/408069/ here14:30
*** prateek has quit IRC14:30
*** Jeffrey4l has joined #openstack-nova14:31
*** ljxiash has joined #openstack-nova14:31
*** matrohon has joined #openstack-nova14:31
openstackgerritChris Dent proposed openstack/nova: rt: pass the nodename to public methods  https://review.openstack.org/39847014:31
openstackgerritChris Dent proposed openstack/nova: HTTP interface for resource providers by aggregates  https://review.openstack.org/40774214:31
openstackgerritChris Dent proposed openstack/nova: rt: remove fluff from test_resource_tracker  https://review.openstack.org/39847114:31
openstackgerritChris Dent proposed openstack/nova: rt: pass nodename to internal methods  https://review.openstack.org/39846914:31
openstackgerritChris Dent proposed openstack/nova: placement: create aggregate map in report client  https://review.openstack.org/40730914:31
openstackgerritChris Dent proposed openstack/nova: Support filtering resource providers by aggregate membership  https://review.openstack.org/40762914:31
openstackgerritChris Dent proposed openstack/nova: placement: correct improper test case inheritance  https://review.openstack.org/40636314:31
openstackgerritChris Dent proposed openstack/nova: rt: explicitly pass compute node to _update()  https://review.openstack.org/39847214:31
openstackgerritChris Dent proposed openstack/nova: rt: use a single ResourceTracker object instance  https://review.openstack.org/39847314:31
cdentsigh, caught by the automatic rebase again14:32
*** ducttape_ has quit IRC14:32
*** ljxiash has quit IRC14:33
*** ljxiash has joined #openstack-nova14:33
*** eharney has joined #openstack-nova14:34
openstackgerritFrancesco Santoro proposed openstack/nova: [WIP] convert libvirt driver to use os-vif for fast path vhostuser  https://review.openstack.org/38506114:36
*** smatzek has quit IRC14:36
openstackgerritFeodor Tersin proposed openstack/nova: Implement ScaleIO image backend  https://review.openstack.org/40744014:36
*** gouthamr has joined #openstack-nova14:37
*** ljxiash has quit IRC14:38
sdaguesnikitin: ah14:39
*** shangxdy has joined #openstack-nova14:39
sdagueit would kind of suck to have to install everything in there to get contraints only14:39
*** parora has joined #openstack-nova14:39
*** hfu has quit IRC14:40
snikitinsdague, agree. That is why I need an advice14:41
sdagueso, this doesn't change that often. I'd personally be fine with keeping the version pin in tox.ini14:41
*** matrohon has quit IRC14:41
sdaguehacking<0.11,>=0.10.014:42
sdaguewe're really changing that at most once a cycle14:42
*** prateek_ has quit IRC14:42
*** hfu has joined #openstack-nova14:43
*** jaosorior has quit IRC14:43
*** prateek_ has joined #openstack-nova14:43
*** sridharg has quit IRC14:44
tdurakovsdague: so in case of hacking version updated by the bot, we need a manual follow up patch to update it in tox.ini too, right?14:44
sdaguetdurakov: sure14:44
*** kaisers has quit IRC14:45
snikitinsdague, mriedem is not happy with this solution :( He said "i don't like the alternative of duplicating the version constraints in both test-requirements.txt and tox.ini"14:45
openstackgerritDan Smith proposed openstack/nova: Make servers api view load instance fault from proper cell  https://review.openstack.org/40723814:45
openstackgerritDan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method  https://review.openstack.org/36755714:45
openstackgerritDan Smith proposed openstack/nova: WIP Multicell support for instance listing  https://review.openstack.org/39677514:45
openstackgerritDan Smith proposed openstack/nova: Make RPCFixture support multiple connections  https://review.openstack.org/39641714:45
openstackgerritDan Smith proposed openstack/nova: Setup CellsV2 environment in base test  https://review.openstack.org/40638014:45
openstackgerritDan Smith proposed openstack/nova: Add SecurityGroup.identifier to prefer uuid over name  https://review.openstack.org/40626014:45
openstackgerritDan Smith proposed openstack/nova: Add SingleCellSimple fixture  https://review.openstack.org/40568414:45
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937914:45
openstackgerritDan Smith proposed openstack/nova: Fix up non-cells-aware context managers in test_db_api  https://review.openstack.org/40637914:45
*** eharney has quit IRC14:46
coreycbhi all, is placement api valid to specify on enabled_apis in nova.conf?14:46
*** kaisers has joined #openstack-nova14:46
*** parora has quit IRC14:47
openstackgerritVladik Romanovsky proposed openstack/nova: tests: avoid starting compute service twice in sriov functional test  https://review.openstack.org/40757214:47
openstackgerritVladik Romanovsky proposed openstack/nova: tests: generate correct pci addresses for fake pci devices  https://review.openstack.org/40773014:47
openstackgerritDan Smith proposed openstack/nova: Make servers api view load instance fault from proper cell  https://review.openstack.org/40723814:47
openstackgerritDan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method  https://review.openstack.org/36755714:47
*** ducttape_ has joined #openstack-nova14:47
openstackgerritDan Smith proposed openstack/nova: WIP Multicell support for instance listing  https://review.openstack.org/39677514:47
openstackgerritDan Smith proposed openstack/nova: Make RPCFixture support multiple connections  https://review.openstack.org/39641714:47
openstackgerritDan Smith proposed openstack/nova: Setup CellsV2 environment in base test  https://review.openstack.org/40638014:47
openstackgerritDan Smith proposed openstack/nova: Add SecurityGroup.identifier to prefer uuid over name  https://review.openstack.org/40626014:47
openstackgerritDan Smith proposed openstack/nova: Add SingleCellSimple fixture  https://review.openstack.org/40568414:47
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937914:47
openstackgerritDan Smith proposed openstack/nova: Fix up non-cells-aware context managers in test_db_api  https://review.openstack.org/40637914:47
openstackgerritEvgeny Antyshev proposed openstack/nova: WIP: Convert volume format name from 'parallels' to 'ploop'  https://review.openstack.org/40814914:47
*** markus_z has quit IRC14:49
*** shangxdy has quit IRC14:50
tdurakovsdague, snikitin: let's compare time between installing all test-requirements and hacking only, is that profit that valuable to worry about it?14:52
openstackgerritRoman Podoliaka proposed openstack/nova-specs: WIP: Ironic: add support for instance RAID configuration  https://review.openstack.org/40815114:52
*** rama_y has joined #openstack-nova14:53
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Fix initialising of LVM ephemeral disks  https://review.openstack.org/40815214:53
mdboothftersin: ^^^14:53
*** jamespag` is now known as jamespage14:53
*** tongli has joined #openstack-nova14:53
*** cdent has quit IRC14:55
*** mriedem has joined #openstack-nova14:55
snikitinsdague tdurakov: installing of 'hacking' takes 2 sec. installting of test-req.txt takes 35 sec14:56
*** irenab has quit IRC14:56
openstackgerritEd Leafe proposed openstack/nova: Cascade deletes of RP aggregate associations  https://review.openstack.org/40770714:56
snikitinsdague, tdurakov I restored 'fast' solution. so you can comment is too https://review.openstack.org/#/c/408055/14:56
*** smatzek has joined #openstack-nova14:57
edleafecdent: ^^ tests added. Have at it.14:57
*** hfu has quit IRC14:57
*** jamesdenton has joined #openstack-nova14:57
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Add create_from_image and create_from_func to Backend  https://review.openstack.org/33324414:58
*** markus_z has joined #openstack-nova14:58
*** irenab has joined #openstack-nova14:59
sdaguesnikitin: oh, so you were going to change deps to just test-requirements, and not install requirements?15:01
sdagueyeh, that's probably not too bad15:01
sdagueit's when numpy gets installed in requirements.txt that's really terrible15:01
*** links has joined #openstack-nova15:03
moshelejaypipes:hi  kindly reminder to  review https://review.openstack.org/#/c/199488/15:04
moshelejaypipes: hi kindly reminder to review https://review.openstack.org/#/c/199488/15:04
*** voelzmo has quit IRC15:04
*** liverpooler has quit IRC15:05
*** liverpooler has joined #openstack-nova15:05
*** irenab has quit IRC15:06
*** tbachman has joined #openstack-nova15:07
*** voelzmo_ has joined #openstack-nova15:07
*** Amit_M has quit IRC15:07
*** hongbin has joined #openstack-nova15:07
*** rama_y has quit IRC15:09
*** TMM has joined #openstack-nova15:11
TMMHi all, can someone point me in the right direction for trying to debug "Filter NUMATopologyFilter returned 0 hosts" errors when trying to deploy flavors with numa node extra specs?15:11
TMMI'm kind of lost15:12
mriedempkholkin: what do you think about replacing that negative logic with positive logic using max_version here? https://review.openstack.org/#/c/405461/7/nova/api/openstack/compute/create_backup.py@6215:12
openstackgerritEric Berglund proposed openstack/nova: Introduce PowerVM driver skeleton  https://review.openstack.org/39128815:12
openstackgerritPavel Kholkin proposed openstack/python-novaclient: [proxy-api] microversion 2.39 deprecates image-metadata proxy API  https://review.openstack.org/40816615:12
mriedemif request_version.is_supported(max_version='2.38'):15:12
mriedemthat seems cleaner to me15:12
pkholkinmriedem: I had this idea, too15:13
mriedemTMM: cfriesen might be able to help but he's not around it looks like15:13
pkholkinalex_xu: mriedem: started novaclient patch for 2.39 https://review.openstack.org/#/c/408166/15:14
dansmithmriedem: thanks for pushing that stuff through last night15:14
dansmithmriedem: I realized I forgot to remove the back from my patch anyway, which I've now done15:14
*** irenab has joined #openstack-nova15:14
pkholkinmriedem: I think I can change that for positive logic15:14
mriedems/back/hack/ right?15:14
mriedempkholkin: ok15:14
TMMmriedem, thanks, I'll wait :)15:14
*** armax has joined #openstack-nova15:14
*** mdrabe has quit IRC15:18
snikitinsdague, Yes. In fact by removing 'deps = hacking' we replace it with 'deps = -r{toxinidir}/test-requirements.txt'  from [testenv]15:19
*** cdent has joined #openstack-nova15:20
*** anush has quit IRC15:21
*** irenab has quit IRC15:21
sdaguesnikitin: yeh, it still isn't great that we're installing postgresql c deps for that venv, but such it is I guess15:21
sdaguemriedem: why don't you want to just stick that one line in there?15:22
*** cdent_ has joined #openstack-nova15:22
sdaguemriedem: this really is only changing about once a year now15:22
mriedemi didn't -1 the idea15:22
mriedemand said basically the same15:23
mriedembut then you +2'ed the revert15:23
mriedemso i went with that15:23
mriedemcan we move the postgresql dep out of test-requirements?15:23
mriedemi guess it's needed for the opportunistic db tests15:23
*** daemontool has joined #openstack-nova15:23
openstackgerritPavel Kholkin proposed openstack/nova: [proxy-api] microversion 2.39 deprecates image-metadata proxy API  https://review.openstack.org/40546115:23
*** dave-mccowan has quit IRC15:23
pkholkinmriedem: done ^15:23
pkholkinalso started novaclient patch15:23
sdaguemriedem: we can't, for those reasons15:23
mriedempkholkin: you'll be pleased to find my -1 on your novaclient patch already :)(15:24
pkholkinmriedem: got comments15:24
pkholkinyes :)15:24
mriedemi'm +2 on https://review.openstack.org/#/c/405461/ now though, thanks15:24
pkholkinmriedem: that's the first patch :)15:24
mriedemreally fast turnaround on that one is helpful15:24
pkholkinsounds great!  )15:24
mriedemalex_xu: are you still happy with this? https://review.openstack.org/#/c/405461/15:25
mriedemsdague: maybe we should use extras in setup.cfg more15:25
openstackgerritSam Betts proposed openstack/nova: [WIP] Support Ironic interface attach/detach in nova virt  https://review.openstack.org/36441315:25
mriedembut that would require changes to the gate jobs that run those i suppose15:25
*** cdent has quit IRC15:25
*** cdent_ is now known as cdent15:25
pkholkinmriedem: will work on novaclient patch now15:25
mriedemi've got to run to a meeting15:26
pkholkinmriedem: after that I can do smth else15:26
pkholkinok15:26
*** catintheroof has quit IRC15:26
*** catintheroof has joined #openstack-nova15:26
snikitinmriedem, sdague Oh, I forgot! we need to merge this patch first https://review.openstack.org/#/c/407870/ It fixes 'bad' assetEquals15:26
*** pradk has quit IRC15:26
*** Jeffrey4l has quit IRC15:27
*** Jeffrey4l has joined #openstack-nova15:27
*** pradk has joined #openstack-nova15:27
*** owalsh has quit IRC15:27
*** rama_y has joined #openstack-nova15:27
*** irenab has joined #openstack-nova15:28
*** anush has joined #openstack-nova15:29
*** mdnadeem has quit IRC15:30
*** mdrabe has joined #openstack-nova15:30
*** catintheroof has quit IRC15:31
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Pass is_block_dev explicitly from create_from_func  https://review.openstack.org/33326315:32
*** baoli has quit IRC15:32
mdboothftersin: ^^^ Updated with a test15:32
openstackgerritDiana Clarke proposed openstack/python-novaclient: Microversion 2.39 - Simple tenant usage pagination  https://review.openstack.org/40782515:32
*** shangxdy has joined #openstack-nova15:32
*** annegentle has quit IRC15:33
*** jwcroppe has joined #openstack-nova15:33
*** Guest78372 has quit IRC15:35
*** irenab has quit IRC15:35
*** owalsh has joined #openstack-nova15:37
*** yassine has joined #openstack-nova15:37
*** moshele has quit IRC15:37
*** yassine is now known as Guest7867715:38
*** portdirect is now known as portdirect_lunch15:39
*** irenab has joined #openstack-nova15:39
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Refactor _create_image into storagepolicy  https://review.openstack.org/35665115:41
*** dave-mccowan has joined #openstack-nova15:41
*** irenab has quit IRC15:42
openstackgerritMatthew Booth proposed openstack/nova: libvirt: don't test cache() in test_driver  https://review.openstack.org/40686915:43
*** mlavalle has joined #openstack-nova15:45
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Introduce ImageCacheLocalDir  https://review.openstack.org/27966915:45
*** NightKhaos has quit IRC15:46
*** annegentle has joined #openstack-nova15:46
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Resize an existing disk  https://review.openstack.org/40687015:47
dansmithmelwitt: mriedem: also thanks for the reviews15:48
*** NightKhaos has joined #openstack-nova15:48
*** heroux has quit IRC15:48
*** irenab has joined #openstack-nova15:49
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Add ensure_backing_from_*  https://review.openstack.org/40687115:50
*** hogepodge has joined #openstack-nova15:50
*** Sukhdev_ has joined #openstack-nova15:51
*** rama_y has quit IRC15:51
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Don't call _create_image from finish_migration  https://review.openstack.org/33716015:52
mdboothLooking for a second +2 at the head of my queue here, btw: https://review.openstack.org/#/c/400871/15:52
*** portdirect_lunch is now known as portdirect15:52
*** jheroux has joined #openstack-nova15:52
mdboothThat's the last test-only change before the real stuff starts15:52
*** zenoway has quit IRC15:53
mriedemi don't think untestably is a word15:53
mdboothmriedem: It is now15:53
mriedemtouche15:53
*** zenoway has joined #openstack-nova15:53
* mdbooth scribbles it into his dictionary15:53
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Remove _create_image from driver  https://review.openstack.org/40687215:54
mdbooth^^^ One of my favourites15:54
*** kevinz has quit IRC15:55
mdbooth2 files changed, 27 insertions(+), 92 deletions(-)15:55
*** jdurgin1 has joined #openstack-nova15:56
ftersin_mdbooth: you need to rebase the series again :/15:56
mdboothftersin_: Really?15:57
*** ducttape_ has quit IRC15:57
*** ducttape_ has joined #openstack-nova15:57
* mdbooth slumps15:57
*** catintheroof has joined #openstack-nova15:57
*** irenab has quit IRC15:57
mdboothftersin_: Did you look at my Lvm ephemeral fix, btw?15:58
ftersin_as i see gerrit writes 'merge conflict'. i do see no reason to not trust it15:58
*** john-davidge has quit IRC15:58
*** zenoway has quit IRC15:58
mdboothI think it covers your ScaleIO use case, and addresses your concerns on the next 2 patches15:58
mriedemmdbooth: question inline for a mock.ANY in there https://review.openstack.org/#/c/400871/15:58
ftersin_i did. got stuck on test, being switched to something another...15:58
openstackgerritStephen Finucane proposed openstack/nova: tests: Replace use of CONF with monkey patching  https://review.openstack.org/40819115:59
openstackgerritStephen Finucane proposed openstack/nova: console: Move proxies to 'console/proxy'  https://review.openstack.org/40819215:59
openstackgerritStephen Finucane proposed openstack/nova: conf: Deprecate 'console_driver'  https://review.openstack.org/40819315:59
dansmithsdague: libvirt test and fixture fixes you would love to +W: https://review.openstack.org/#/c/407730/315:59
dansmithsdague: one on top of that too15:59
sdaguedansmith: ok, let me look15:59
*** parora has joined #openstack-nova16:00
*** diga has joined #openstack-nova16:00
sdagueL191 here - https://review.openstack.org/#/c/407730/3/nova/tests/unit/virt/libvirt/fakelibvirt.py can't be right can it? That doesn't look like valid format16:00
*** jheroux has quit IRC16:00
*** prateek_ has quit IRC16:01
sdagueactually, in general, the new formats in the template seem... odd16:01
dansmithsdague: it passes the test, so it has to be right?16:01
sdague%(slot)02x doesn't seem like a thing16:01
*** prateek_ has joined #openstack-nova16:01
dansmithsdague: sfinucan made a point about having checked them, surprised they worked16:01
dansmithsdague: sure it is, why not?16:01
dansmithsdague: >>> '%(foo)02x' % {'foo': 1}16:02
dansmith'01'16:02
sdagueoh... sorry16:02
*** voelzmo_ has quit IRC16:03
*** mpaolino_ has quit IRC16:03
sfinucansdague: Wonder if it would be a little easier to read if we used '.format' instead?16:03
*** tongli has quit IRC16:03
dansmithplease no16:04
* dansmith hates .format16:04
dansmith:)16:04
*** gmann_ has quit IRC16:04
*** parora has quit IRC16:04
mdboothIsn't there a third string formatting convention in python 3 now?16:04
sdaguesfinucan: it might be, I think the advanced format strings aren't obvious at least to me, but now that I've got that in my head, I can look at the rest of it16:04
mdboothIf they keep this up, I'lll start referring to it as perlython16:04
sfinucanmdbooth: Careful - you'll end up sounding like Zed Shaw :P16:05
mdboothHeh, I read that16:05
mdboothI disagree with his reasons, but agree with his conclusions16:06
mdboothI think arguing that a language is bad because it's hard for beginners is stupid16:06
*** jheroux has joined #openstack-nova16:06
mdboothIt's like saying F1 drivers should all use Nissan Micras, because that's what they learned in16:06
markus_zwould make the races more interesting16:06
mdboothmarkus_z: lol16:07
sdaguesounds like a top gear episode16:07
dansmithsdague: fyi, these two fixes are needed for me to make my cellsv2 change to the base test.py.. these started rando-failing when I made my change, so they're probably already unstable16:07
dansmithmostly the second patch I think, but..16:07
*** rama_y has joined #openstack-nova16:07
sdagueok16:07
*** irenab has joined #openstack-nova16:08
*** edand has quit IRC16:08
openstackgerritDiana Clarke proposed openstack/python-novaclient: Microversion 2.39 - Simple tenant usage pagination  https://review.openstack.org/40782516:09
*** prometheanfire has joined #openstack-nova16:09
*** jgriffith_away is now known as jgriffith16:09
sdaguedansmith: so... dev is still going to be potentially weird in that template right, as you can't get a 0xa16:09
prometheanfireanyone use/test the zfs storage backend in libvirt with nova? https://libvirt.org/storage.html#StorageBackendZFS16:09
dansmithsdague: not sure what you mean16:09
sdague0x%(dev)d16:10
mdboothprometheanfire: I doubt Nova supports it16:10
dansmithsdague: oh you mean if we had a large device number and didn't actually output hex/16:10
prometheanfiremdbooth: me too, just sounds 'neat' is all (since right now I'm doing nova-compute on zfs)16:11
sdaguedansmith: if dev is passed into the function > 9 the template will be all wrong16:11
prometheanfireit'd be nice not to double-cow...16:11
sdagueas it does similar conversions to slots16:11
dansmithsdague: that's unchanged in this patch so I dunno what the impact is16:11
*** rmart04 has quit IRC16:11
sdaguedansmith: right, agreed16:11
sdaguejust making sure we note it somewhere while staring at this16:11
dansmithvladikr: ^16:11
*** irenab has quit IRC16:12
dansmithdefinitely feels wrong to format something with 0x%d for sure16:12
*** lpetrut has quit IRC16:12
vladikrdansmith, on a call, just a sec16:12
*** armax has quit IRC16:12
*** lpetrut has joined #openstack-nova16:12
*** eharney has joined #openstack-nova16:13
*** gszasz has quit IRC16:13
*** links has quit IRC16:13
*** armax has joined #openstack-nova16:14
*** ihrachys has quit IRC16:15
*** cfriesen has joined #openstack-nova16:16
*** kaisers_ has quit IRC16:16
*** armax has quit IRC16:17
ftersin_mdbooth: i commented the test on https://review.openstack.org/#/c/40815216:18
mdboothftersin_: Looking16:18
*** pcaruana has quit IRC16:18
*** rcernin has quit IRC16:19
*** armax has joined #openstack-nova16:19
sdaguedansmith: I wonder if we should also make start_service explode if there are 2 services started with same exe and hostname16:20
*** owalsh has quit IRC16:20
dansmithsdague: we do as of my patch that depends on this16:20
dansmithsdague: because we create hostmappings for the service in the main thread16:21
sdaguedansmith: where is that patch?16:21
sdagueboth of these seem good, +A on them16:21
dansmithsdague: https://review.openstack.org/#/c/406380/8/nova/test.py@335 is where we will 'splode now16:22
sdagueI left the comment on the dev formatting in the approved patch as well, hopefully vladikr can circle around on that one as a follow up16:22
dansmithsdague: see that the second patch is embedded in that one ^16:22
dansmithsdague: yep16:22
sdaguedansmith: ok, cool16:22
dansmithsdague: thanks for hitting those16:23
dansmithand thanks to vladikr for jumping on them16:23
dansmithonce I convinced him they were broken :P16:23
*** irenab has joined #openstack-nova16:23
*** mjura has quit IRC16:23
markus_zkashyap: I have to review it tomorrow. I have to leave for today.16:24
mriedemdansmith: this isn't going to pass is it? https://review.openstack.org/#/c/396417/16:24
mriedemthe depends-on isn't on the g-r bump16:24
*** roeyc has quit IRC16:24
mriedemoh snap https://review.openstack.org/#/c/407794/16:24
mriedemnevm16:24
dansmithmriedem: the g-r bump landed16:24
dansmithright16:24
dansmithguess I should have removed the depends, but.. I didn't.16:24
prometheanfiremdbooth: gr bump?16:25
mriedemmeh16:25
mriedemglobal-requirements16:25
mriedemprometheanfire: aren't you reqs core?16:25
mriedem:)16:25
prometheanfire:D16:25
prometheanfirewhat's needed?16:25
*** armax has quit IRC16:25
mriedemnothing16:26
mriedemi just wasn't up to date yet on over night activity16:26
prometheanfirethen my job here is done :P16:26
mriedemzfs on out of here then!16:26
*** psachin has quit IRC16:26
prometheanfirelol16:26
* prometheanfire would like that :P16:26
vladikrsdague, dansmith, back, sorry16:26
*** annegentle has quit IRC16:26
prometheanfireuc-update should be merged in the next couple of hours as well16:27
dansmithvladikr: np, sdague left a comment on the bottom patch16:27
vladikrsdague, ah, yep, thanks, I'll fix it16:27
dansmithcool16:27
vladikr:)16:27
*** tbachman has quit IRC16:29
mriedemdansmith: +2 on https://review.openstack.org/#/c/396417/ - looks like easy fixes in the next one in the series16:29
mdboothftersin_: I think that's ok, btw16:29
sfinucanjohnthetubaguy: went for a wander round nova to figure out what the console_driver CONF opt was for. Seems it's only applicable to XVP - not websocketproxy16:29
*** ayogi has joined #openstack-nova16:29
sfinucanjohnthetubaguy: I don't know if we still care about XVP (i.e. has websocketproxy usurped it?), but I cleaned it up a small bit to clarify the intent anyway https://review.openstack.org/#/q/project:openstack/nova+topic:feat/deprecate-console_driver-opt16:30
* mdbooth waits for https://review.openstack.org/408152 to clear the gate before rebasing... again16:30
dansmithmriedem: yeah, already fixed locally, running tests16:30
*** nkrinner is now known as nkrinner_afk16:30
*** irenab has quit IRC16:30
*** yamahata has joined #openstack-nova16:31
*** mvk has quit IRC16:31
*** andymaier has quit IRC16:32
*** markus_z has quit IRC16:33
raj_singhsfinucan: johnthetubaguy is on his honeymoon. I don't think he cares about any of this ATM :)16:33
*** mdnadeem has joined #openstack-nova16:33
sfinucanraj_singh: I did not know that. Congrats to him, heh16:34
*** edand has joined #openstack-nova16:34
*** bvanhav__ has joined #openstack-nova16:34
*** bvanhav_ has quit IRC16:34
*** thorst_ is now known as thorst_afk16:35
robcresswello/ hey all, how do I go about generating a policy file? Or is there on in the docs somewhere that I can reference quickly?16:36
*** armax has joined #openstack-nova16:37
openstackgerritDan Smith proposed openstack/nova: Add SingleCellSimple fixture  https://review.openstack.org/40568416:37
mriedemrobcresswell: http://docs.openstack.org/developer/nova/sample_policy.html16:37
mriedemhttp://docs.openstack.org/developer/nova/_static/nova.policy.yaml.sample16:37
*** ducttape_ has quit IRC16:37
mriedemrobcresswell: tox -r -e genpolicy16:37
*** TMM has quit IRC16:37
*** paul-carlton2 has joined #openstack-nova16:38
*** ayogi has quit IRC16:38
*** prometheanfire has left #openstack-nova16:38
*** irenab has joined #openstack-nova16:38
mriedemsfinucan: xen cares about xvp i think16:38
mriedemso maybe ask BobBall16:38
robcresswellAh fantastic, thanks mriedem16:38
mriedemi wanted to kill it awhile back but found it was maybe still in use by xen folk16:39
*** tbachman has joined #openstack-nova16:39
sdaguedansmith: on https://review.openstack.org/#/c/405684/10/nova/tests/fixtures.py is there are reason those are instance methods instead of defined inside the setUp scope?16:39
mriedembtw, i think we should all go in on a rice cooker for johnthetubaguy16:39
*** edand has quit IRC16:39
sfinucanmriedem: But do they /still/ care about it, I wonder. BobBall sounds as good a person to ask as anyone16:40
dansmithsdague: no technical reason, as you can see, but.. is that a pattern?16:41
sdaguedansmith: mostly just curious whenever I see a thing which takes self, which doesn't need it16:41
*** paw has quit IRC16:42
mriedemdansmith: left a couple of nits in there16:42
dansmithsdague: that wouldn't address the self-ness as setUp scoped things would still have access, but yes those could probably be staticmethods, I just usually don't because mock can be weird on some of those things16:42
dansmithlike the trouble we had around those staticmethod db handlers in the api objects16:43
robcresswellAnother question; what's the difference between "os-keypairs", "os-keypairs:discoverable" and "os-keypairs:index"? My assumption is that ":index" is the list call, but I'm a little confused by the other two.16:43
sdaguedansmith: ok, no prob16:43
dansmithmriedem: okay16:43
mriedemrobcresswell: i think os-keypairs:discoverable makes it show up in the extension list16:43
mriedemindex is list16:44
*** umbSublime has quit IRC16:44
mriedemrobcresswell: looks like the base one is used for show/detail https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/keypairs.py#L30416:45
*** dtp has joined #openstack-nova16:45
mriedemwhich is then checked in more detail here https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/keypairs.py#L21116:46
mriedemi'm not exactly sure the difference16:46
mriedemadding descriptions in the yaml sample is a TODO for the policies16:46
robcresswellmriedem: Gotcha, thats enough for me to work on. Thanks again16:46
dansmithsdague: mriedem: I'm in the middle of a big '-x tox -epy27' run but will do that docstring thing when it's done16:47
mriedemrobcresswell: oh https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/keypairs.py#L304 is on the server extension16:48
mriedemso the ability to show keypairs for a given server16:48
mriedemrather than show details for a given keypair16:48
mriedemthat Controller class should probably be renamed to be something like ServerKeypairController or something more obvious16:48
mriedemsdague: ^ would you agree?16:49
robcresswellAH so "os_compute_api:os-keypairs:show" is the show for a given keypair then16:49
mriedemyes16:49
*** irenab has quit IRC16:49
mriedemdansmith: ok, i'm going to be heading to lunch in 10 minutes so can peruse when i'm back16:50
*** diga has quit IRC16:50
dansmithmriedem: so no cells meeting for you eh/16:51
dansmithguess it will just be me and melwitt  :)16:51
dtpi'll be there16:51
openstackgerritEd Leafe proposed openstack/nova: Cascade deletes of RP aggregate associations  https://review.openstack.org/40770716:51
edleafecdent: ^^ moved the test16:51
dansmithguess it will just be me and melwitt and dtp :)16:51
mriedemdansmith: oh didn't have it on my calendar - what about bauzas?!16:51
*** armax has quit IRC16:51
melwittheh16:51
dansmithif he shows16:51
cdentthanks edleafe will go looksee in a mo16:51
*** andreas_s has quit IRC16:52
*** tbachman has quit IRC16:52
mriedemdansmith: i think you just need to add a big irc nick reminder list for the start of your cells meeting16:52
mriedemso i don't need to be responsible16:52
dansmitheff that16:53
*** edand has joined #openstack-nova16:53
mriedemok it's on my calendar now for the next 2 months, but i'm still not coming, so there16:53
dansmithheh16:54
dansmithI have little to say anyway16:54
*** browne has joined #openstack-nova16:58
* mdbooth notes that there are currently only 14 patches in the imagebackend rebase stack16:58
mdbooth11 files changed, 3032 insertions(+), 1080 deletions(-)16:59
andreafmriedem: this issue https://bugs.launchpad.net/tempest/+bug/1646779 is happening relatively frequently in the gate (54 hits in the past 7dd) - I cannot find anything on tempest side really nor in nova/libvirt logs - so I added nova to it now - I'm looking for nova folks help to look into it16:59
openstackLaunchpad bug 1646779 in tempest "Cannot connect to libvirt" [Undecided,New]16:59
*** kfox1111 has quit IRC17:01
ftersin_mdbooth: great work. but is there a final state somewhere?17:01
*** dtp has left #openstack-nova17:01
*** lpetrut has quit IRC17:02
*** ducttape_ has joined #openstack-nova17:04
*** tbachman has joined #openstack-nova17:04
mdboothftersin_: I was just about to push it, but I have to wait until the patch which is currently in the gate merges17:04
mdboothOtherwise I'll end up kicking it out17:04
mdboothftersin_: Anyway, I need to get the end of the queue updated in gerrit, so I'm going to do a full push tomorrow morning (UK time) when most americans are asleep17:05
mdboothHopefully it won't stress CI too much17:06
*** Kevin_Zheng has quit IRC17:07
cdentmdbooth: if you ping me when you get it all up to date tomorrow morning I'll go back through what remains17:07
ftersin_mdbooth: i'm waiting for this with interest17:08
mdboothThanks17:08
mdboothResolves-Bug or Closes-Bug, btw?17:09
mdboothSuspect it's Closes-Bug, but Resolves-Bug seemed to work and there are loads of them in the history17:09
*** moshele has joined #openstack-nova17:09
mdboothmriedem: Given that I'm rebasing anyway, is it worth changing the above?17:09
*** Sukhdev_ has quit IRC17:10
mdboothhttps://wiki.openstack.org/wiki/GitCommitMessages says Closes17:11
*** rcernin has joined #openstack-nova17:11
mdboothdiana_clarke: Hey, do you have a list of all the backend patches you abandoned?17:12
mdboothI can't find them locally17:13
*** rama_y has quit IRC17:13
diana_clarkemdbooth: Is this good enough? https://review.openstack.org/#/q/owner:diana.joan.clarke%2540gmail.com+status:abandoned I could filter it more.17:14
*** lpetrut has joined #openstack-nova17:14
mdboothdiana_clarke: Yes! Your gerrit-fu is superior to mine.17:15
mdboothThanks :)17:15
*** moshele has quit IRC17:17
*** sacharya has joined #openstack-nova17:20
openstackgerritDiana Clarke proposed openstack/python-novaclient: CONF.osapi_max_limit -> CONF.api.max_limit  https://review.openstack.org/40822417:20
*** ihrachys has joined #openstack-nova17:22
*** mvk has joined #openstack-nova17:24
*** tbachman_ has joined #openstack-nova17:25
*** tbachman has quit IRC17:25
*** tbachman_ is now known as tbachman17:25
*** Zer0Byte__ has joined #openstack-nova17:27
openstackgerritDiana Clarke proposed openstack/python-novaclient: Usage missing from generated docs  https://review.openstack.org/40822717:28
*** baoli has joined #openstack-nova17:28
*** paul-carlton2 has quit IRC17:30
*** kristia__ has joined #openstack-nova17:30
BobBallmriedem, sfinucan: I don't think we use xvp ATM17:30
sfinucanBobBall: If the docs are to be believed, something else has replaced it. Any idea what that might be? http://docs.openstack.org/admin-guide/compute-remote-console-access.html#about-nova-consoleauth17:32
BobBallWe just use novnc I think?17:32
*** dhellmann has quit IRC17:33
sfinucanHmm... mriedem, how would one start with deprecating such a feature like XVP if it might not be used anymore? Mail to the operators list?17:33
*** mdnadeem has quit IRC17:34
*** efoley__ has joined #openstack-nova17:34
*** lucasagomes is now known as lucas-afk17:35
*** daemontool has quit IRC17:35
*** kristian__ has joined #openstack-nova17:36
*** kristia__ has quit IRC17:36
*** gouthamr has quit IRC17:37
*** efoley_ has quit IRC17:38
*** Apoorva has joined #openstack-nova17:38
*** gouthamr has joined #openstack-nova17:38
*** eglynn has quit IRC17:40
*** xiaoyang has joined #openstack-nova17:40
*** shangxdy has quit IRC17:41
*** sudipto_ has quit IRC17:42
*** sudipto has quit IRC17:42
*** magic has quit IRC17:43
*** paw has joined #openstack-nova17:44
cfriesenis there a straightforward way to get stats on where nova services are spending their time?  does cProfile work with nova?17:46
jaypipessdague, mriedem, dansmith: bauzas' patch here that adds ability to filter resource provider (objects) via requested resources is ready IMHO: https://review.openstack.org/#/c/386242/. reviews welcome.17:46
jaypipescdent, edleafe: you guys too, though you've each reviewed prior patches (thanks!)17:47
openstackgerritsahid proposed openstack/nova: objects: remove cpu_topology from __init__ of InstanceNUMATopology  https://review.openstack.org/40824017:48
openstackgerritsahid proposed openstack/nova: objects: remove pagesize from __init__ of InstanceNUMATopology  https://review.openstack.org/40824117:48
openstackgerritsahid proposed openstack/nova: objects: remove related pining from __init__ of InstanceNUMATopology  https://review.openstack.org/40824217:48
*** paul-carlton2 has joined #openstack-nova17:48
sahiddansmith: wip of what you are looking for about the object InstanceNUMATopology ^17:49
*** efoley__ has quit IRC17:51
*** bvanhav_ has joined #openstack-nova17:52
*** bvanhav__ has quit IRC17:52
*** sacharya_ has joined #openstack-nova17:56
*** derekh has quit IRC17:56
*** rama_y has joined #openstack-nova17:56
*** sacharya has quit IRC17:57
*** alaski has quit IRC17:58
*** dhellmann has joined #openstack-nova18:00
*** sahid has quit IRC18:01
*** yamahata has quit IRC18:01
*** david-lyle_ is now known as david-lyle18:02
*** fragatina has joined #openstack-nova18:03
*** fragatin_ has joined #openstack-nova18:04
*** ralonsoh has quit IRC18:04
*** cdent has quit IRC18:05
*** krtaylor has quit IRC18:06
*** cleong has joined #openstack-nova18:06
*** alaski has joined #openstack-nova18:06
*** sacharya_ has quit IRC18:07
BobBallsfinucan, mriedem: I'm going to look into the xvp proxy actually - gimme a couple of days?18:07
*** fragatina has quit IRC18:08
*** sacharya has joined #openstack-nova18:08
openstackgerritAnusha Unnam proposed openstack/nova: Do not merge: Add post hook script to run tempest with old n-api  https://review.openstack.org/40139518:08
*** gjayavelu has joined #openstack-nova18:09
*** sapcc-bot2 has quit IRC18:10
*** carthaca_1 has quit IRC18:10
*** carthaca_ has joined #openstack-nova18:11
*** kaisers_ has joined #openstack-nova18:12
*** sapcc-bot has joined #openstack-nova18:12
*** parora has joined #openstack-nova18:12
*** tbachman has quit IRC18:13
*** prateek_ has quit IRC18:13
*** gjayavelu has quit IRC18:13
*** slaweq_ has quit IRC18:14
*** dhellmann has quit IRC18:15
*** slaweq_ has joined #openstack-nova18:16
openstackgerritDan Smith proposed openstack/nova: Make servers api view load instance fault from proper cell  https://review.openstack.org/40723818:17
openstackgerritDan Smith proposed openstack/nova: Add schedule_and_build_instances conductor method  https://review.openstack.org/36755718:17
openstackgerritDan Smith proposed openstack/nova: WIP Multicell support for instance listing  https://review.openstack.org/39677518:17
openstackgerritDan Smith proposed openstack/nova: Setup CellsV2 environment in base test  https://review.openstack.org/40638018:17
openstackgerritDan Smith proposed openstack/nova: Add SecurityGroup.identifier to prefer uuid over name  https://review.openstack.org/40626018:17
openstackgerritDan Smith proposed openstack/nova: Add SingleCellSimple fixture  https://review.openstack.org/40568418:18
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937918:18
openstackgerritDan Smith proposed openstack/nova: Fix up non-cells-aware context managers in test_db_api  https://review.openstack.org/40637918:18
dansmithmelwitt: you don't have a strong opinion about it, and neither do I aside from not wanting to go back and do all those again... is there anything better about the proposed approach other than that it's arguably a little nicer to look at?18:19
dansmithif so, we should change it18:19
dansmithif not, meh18:19
*** thorst_afk is now known as thorst_18:19
*** anush has quit IRC18:20
*** gjayavelu has joined #openstack-nova18:20
*** slaweq_ has quit IRC18:21
*** prateek has joined #openstack-nova18:21
*** slaweq_ has joined #openstack-nova18:23
edleafejaypipes: can you clear up what the 'can_host' column indicates? Is it simply to say that a given RP is a compute node?18:24
*** parora has quit IRC18:24
*** lpetrut has quit IRC18:24
*** amoralej is now known as amoralej|off18:25
*** nic has joined #openstack-nova18:27
*** slaweq_ has quit IRC18:27
*** tbachman has joined #openstack-nova18:28
mriedemmdbooth: should be closes-bug but not worth changing in an already approved change probably18:30
*** mat128 has quit IRC18:31
mriedemsfinucan: i want to say i emailed the operators list about xvp once awhile back, but it might have been a dream18:31
mriedemcfriesen: os-profiler?18:32
mriedemcfriesen: https://review.openstack.org/#/c/254703/ + rally?18:32
*** tbachman has quit IRC18:33
*** tbachman has joined #openstack-nova18:33
openstackgerritSujitha proposed openstack/nova: Adding tags to InstancePayload  https://review.openstack.org/40722818:34
jaypipesedleafe: yes.18:37
jaypipesedleafe: we aren't yet using it correctly though.18:37
*** mat128 has joined #openstack-nova18:37
jaypipesedleafe: we will eventually use it but meh.18:38
edleafejaypipes: why should we even need it?18:38
jaypipesedleafe: when joining against resource_provider_aggregates, we look for the providers that are *not* compute_nodes to determine if those providers can supply resources for a given consumer.18:38
jaypipesedleafe: for example, without that flag, there's no way to differentiate between a compute node providing local disk and a shared storage pool providing shared disk.18:39
*** paul-carlton2 has quit IRC18:39
jaypipesedleafe: and when trying to determine appropriate providers that match a given request for resources, we need to query for compute nodes that have all the local resources we need or that have some of those resources but are associated with a provider of shared resources.18:40
*** anush has joined #openstack-nova18:41
edleafejaypipes: I'm still not seeing it. If we are already filtering on VCPU/RAM, that will remove the shared disk providers, no?18:41
edleafejaypipes: it also seems like it's making placement and RPs very nova-centric18:43
*** b_rake has joined #openstack-nova18:43
*** prateek has quit IRC18:43
mriedemdansmith: were you going to make the fakes in here static methods? https://review.openstack.org/#/c/405684/10..11/nova/tests/fixtures.py18:44
mriedemdansmith: i think you said no because mocks/fixtures with staticmethods are wonky sometimes18:45
mriedembut just checking18:45
*** irenab has joined #openstack-nova18:45
*** voelzmo has joined #openstack-nova18:45
jaypipesedleafe: so, the user just requests DISK_GB = 100. let's say we have 3 compute nodes. all of them are in the same aggregate. 2 of them have no local disk. one has local disk. the aggregate is associated with a provider of shared storage. if I query for providers that have 2 VCPU, 1024 MEMORY_MB and 100 DISK_GB, I need to find providers that *either* have all of the resources locally OR that have18:45
jaypipessome of those resources locally but are connected to a resource provider via an aggregate that satisfies the resources that are *not* on the compute node locally.18:45
*** armax has joined #openstack-nova18:46
*** mkoderer has quit IRC18:46
*** nicolasbock has quit IRC18:47
dansmithmriedem: right, I wasn't18:47
*** nicolasbock has joined #openstack-nova18:47
*** nicolasbock has quit IRC18:47
jaypipesedleafe: in that second query, I have two compute nodes that have local VCPU and MEMORY_MB but not DISK_GB, so I must join to resource_provider_aggregates to determine the providers that are associated with any aggregate those nodes are in that has an inventory of DISK_GB. in the case described above, that join would return *BOTH* the shared storage pool resource provider *AND* the compute node18:48
*** nicolasbock has joined #openstack-nova18:48
jaypipeswith local disk (because it's in the same aggregate). That isn't what we want, because we can't have the instance consume VCPU and RAM on one host and DISK_GB on another compute host.18:48
*** Nakato has quit IRC18:48
*** Nakato has joined #openstack-nova18:48
edleafejaypipes: I understand the use case, and the logic for shared storage.18:48
*** liusheng has quit IRC18:49
edleafejaypipes: It just seems so nova-centric. When (not if :) the placement engine is used for all sorts of resource placement, it seems like we'll need to do the same for other "main" types of providers.18:50
*** liusheng has joined #openstack-nova18:50
edleafejaypipes: i.e., "main" vs. "shared"18:51
jaypipesedleafe: do you have a suggestion to make it more generic?18:51
edleafejaypipes: dunno... off the top of my head, "shared" seems like a candidate18:52
*** ducttape_ has quit IRC18:52
edleafejaypipes: of course, that reverses the logic for 'can_host'18:52
jaypipesedleafe: sure, I could totes go for that.18:52
edleafejaypipes: so maybe 'primary'18:52
jaypipesedleafe: can_host isn't currently used anywhere AFAIK.18:52
jaypipesedleafe: so should be an easy change if you want to tackle it.18:53
edleafejaypipes: I saw it in reviewing bauzas's patch: https://review.openstack.org/#/c/386242/11/nova/objects/resource_provider.py@56218:54
edleafejaypipes: just made me think that I wasn't really clear on its purpose18:54
edleafejaypipes: but yeah, I can do a quick name change.18:54
edleafejaypipes: your preference for the name?18:55
*** nicolasbock has quit IRC18:56
jaypipesedleafe: shared is good I think?18:56
*** nicolasbock has joined #openstack-nova18:56
edleafejaypipes: sounds good18:57
*** ljxiash has joined #openstack-nova19:00
*** ihrachys has quit IRC19:01
mriedemsdague: if you're around, alex was +2 on this earlier, i'm +2 now, it's pretty straight forward pattern of proxy api deprecation which i know you love https://review.openstack.org/#/c/405461/19:02
mriedemwould close a bp which you know i love19:03
*** armax has quit IRC19:03
*** ljxiash has quit IRC19:05
*** armax has joined #openstack-nova19:06
*** ducttape_ has joined #openstack-nova19:08
mriedemkashyap: any idea why libvirt dies here? http://logs.openstack.org/66/401366/4/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/3deacc5/logs/libvirt/libvirtd.txt.gz#_2016-12-01_18_16_04_90719:10
mriedemseeing quite a few virtlogd i/o errors in syslog http://logs.openstack.org/66/401366/4/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/3deacc5/logs/syslog.txt.gz19:12
*** topol has left #openstack-nova19:13
*** hemna_ has joined #openstack-nova19:13
*** hemna_ has quit IRC19:15
*** fragatin_ has quit IRC19:16
*** krtaylor has joined #openstack-nova19:17
mriedemwe're not using the recently merged virtlogd console stuff though b/c that requires libvirt>=1.3.3 and we test with 1.3.119:17
*** dinobot_ has quit IRC19:19
*** ijw has joined #openstack-nova19:19
*** rama_y has quit IRC19:22
*** ociuhandu has quit IRC19:24
*** Sukhdev has joined #openstack-nova19:25
*** irenab has quit IRC19:28
*** fragatina has joined #openstack-nova19:29
*** irenab has joined #openstack-nova19:30
*** gouthamr has quit IRC19:30
*** s1061123 has quit IRC19:31
*** s1061123 has joined #openstack-nova19:31
*** jbernard has quit IRC19:32
*** gouthamr has joined #openstack-nova19:33
*** ihrachys has joined #openstack-nova19:35
openstackgerritDiana Clarke proposed openstack/python-novaclient: Microversion 2.39 - Simple tenant usage pagination  https://review.openstack.org/40782519:36
openstackgerritMatt Riedemann proposed openstack/nova: api-ref: note that os-virtual-interfaces is nova-network only  https://review.openstack.org/40827019:37
*** armax has quit IRC19:37
*** jbernard has joined #openstack-nova19:38
*** fragatina has quit IRC19:40
*** ljxiash has joined #openstack-nova19:42
*** irenab has quit IRC19:44
*** ljxiash has quit IRC19:46
sdaguemriedem: just got back to my desk, will look now19:47
*** voelzmo has quit IRC19:48
*** voelzmo has joined #openstack-nova19:48
*** ihrachys has quit IRC19:49
*** lamt has quit IRC19:49
openstackgerritMaciej Szankin proposed openstack/nova: conf: add warning for vm's max delete attempts  https://review.openstack.org/40827419:51
*** irenab has joined #openstack-nova19:51
*** dhellmann_ has joined #openstack-nova19:52
*** voelzmo has quit IRC19:53
*** jwcroppe has quit IRC19:53
openstackgerritMaciej Szankin proposed openstack/nova: conf: add warning for vm's max delete attempts  https://review.openstack.org/40827419:54
*** kristian__ has quit IRC19:54
*** jwcroppe has joined #openstack-nova19:54
*** voelzmo has joined #openstack-nova19:54
*** ihrachys has joined #openstack-nova19:55
*** ihrachys has quit IRC19:55
*** ihrachys has joined #openstack-nova19:56
*** ihrachys has quit IRC19:56
*** ihrachys has joined #openstack-nova19:57
*** fragatina has joined #openstack-nova19:58
*** jwcroppe has quit IRC19:58
*** dhellmann_ is now known as dhellmann19:59
*** cdent has joined #openstack-nova19:59
*** irenab has quit IRC20:00
*** clenimar has quit IRC20:01
*** ihrachys has quit IRC20:02
openstackgerritMaciej Szankin proposed openstack/nova: conf: remove deprecated barbican options  https://review.openstack.org/39777420:03
mriedemkashyap: mdbooth: know anything about an -9 exit code from qemu-img info?20:07
mriedemhttps://bugs.launchpad.net/nova/+bug/164618120:07
openstackLaunchpad bug 1646181 in OpenStack Compute (nova) "Fail to boot VM out of large snapshots (30GB+)" [Undecided,New]20:07
sdaguemriedem: +A20:08
mriedemsdague: thanks20:08
*** irenab has joined #openstack-nova20:08
mriedembtw, nfs causes all of the problems20:08
mriedemhttps://bugs.launchpad.net/nova/+bug/164689620:08
openstackLaunchpad bug 1646896 in OpenStack Compute (nova) "System hangs when using NFS storage backend with loopback mounts" [Undecided,New]20:08
*** anush has quit IRC20:10
sdaguedansmith: can you comment on this bug - https://bugs.launchpad.net/nova/+bug/164418620:11
openstackLaunchpad bug 1644186 in OpenStack Compute (nova) "nova service performs wrong update from version 15 to 9" [Undecided,New]20:11
*** Nakato has quit IRC20:11
dansmithsdague:  yeah20:11
*** sapcc-bot has quit IRC20:12
*** carthaca_ has quit IRC20:12
sdaguemriedem: yeh, well NFS is all kinds of bonkers at times20:12
*** Nakato has joined #openstack-nova20:12
*** sapcc-bot has joined #openstack-nova20:12
*** carthaca_ has joined #openstack-nova20:13
sdaguemriedem: for the NFS bug I would suggest Confirmed / Low and if they push a patch, cool.20:13
openstackgerritJay Pipes proposed openstack/nova: Extend get_all_by_filters to support resource criteria  https://review.openstack.org/38624220:15
openstackgerritJay Pipes proposed openstack/nova: placement: API filters providers by resources  https://review.openstack.org/40828520:15
*** rfolco has quit IRC20:18
*** irenab has quit IRC20:20
mriedembooya http://logs.openstack.org/00/352200/14/check/gate-tempest-dsvm-neutron-linuxbridge-ubuntu-xenial/6ba61d3/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-11-17_04_43_55_67320:24
*** anush has joined #openstack-nova20:24
openstackgerritPavel Kholkin proposed openstack/python-novaclient: WIP: [proxy-api] microversion 2.39 deprecates image-metadata proxy API  https://review.openstack.org/40816620:26
bauzasp!spy20:26
*** irenab has joined #openstack-nova20:27
*** Nakato has quit IRC20:28
*** Nakato has joined #openstack-nova20:29
*** voelzmo has quit IRC20:31
*** voelzmo has joined #openstack-nova20:32
*** gouthamr has quit IRC20:32
*** bvanhav_ has quit IRC20:33
*** voelzmo has quit IRC20:36
*** irenab has quit IRC20:38
*** harlowja has quit IRC20:41
openstackgerritPavel Kholkin proposed openstack/python-novaclient: remove variable '_' from tests  https://review.openstack.org/40829520:41
bauzasjaypipes: sorry, I was on PTO this afternoon20:42
bauzasjaypipes: why are you providing a separate change for providing GET ?20:42
*** TravT has joined #openstack-nova20:42
*** diogogmt has joined #openstack-nova20:43
bauzasjaypipes: I thought we had a agreement between dansmith, mriedem, melwitt, alaski and me having POST /<something>20:43
bauzasI know cdent and edleafe disagreed on that20:43
bauzasbut what I wrote tried to be a trade-off20:44
bauzasie. using POST with a body, but also using the RFC20:44
bauzasie. saying 'yup, we're getting you a 200, not a 201, which means we don't *create* a resource'20:44
mriedemi thought we were doing POST + body20:44
bauzasmriedem: what I wrote20:45
edleafebauzas: it isn't just POST vs GET, it's creating a brand new endpoint when we already have one for getting a filtered list of resource providers20:45
bauzasedleafe: that's not an endpoint, that's a REST resource20:45
bauzaswe have the same endpoint20:45
*** irenab has joined #openstack-nova20:45
edleafebauzas: fine, call it what you like20:45
edleafebauzas: we already have GET /resource_providers?<filters>20:46
bauzasedleafe: I know20:46
edleafeAll we need to do is add more filters20:46
edleafenot create an artificial resource20:46
bauzasedleafe: and for the exact reason we have the same discussion since 4 weeks now, I don't want to have GET20:46
*** lamt has joined #openstack-nova20:46
bauzashaving a long URL seems very bad to me20:47
mriedembauzas: so how do you do a POST to get info rather than create a new RP?20:47
mriedemdifferent body format?20:47
edleafemriedem: he does POST to a new resource called 'request'20:47
bauzasmriedem: https://review.openstack.org/#/c/39256920:47
edleafewhich means nothing in the placement API20:47
bauzasedleafe: no, false20:48
bauzasedleafe: again, in the same endpoint20:48
bauzaswhich *is* the placement API20:48
*** rtheis has quit IRC20:48
bauzasjust a separate resource for that placement API20:48
edleafebauzas: There is no concept of 'request' in the placement API20:48
bauzasedleafe: and ?20:48
mriedembauzas: i think he means there is no such thing as a 'requests' resource20:48
mriedemno db table or object20:48
mriedemfirst, let's all take a deep breath and chill out :)20:48
bauzasmriedem: we already have a POST method for the RP resource20:49
mriedemb/c you're talking past each other20:49
mriedembauzas: i know, to create the RP20:49
mriedemso we need a different route20:49
bauzasmriedem: exactly, hence me creating a new resource20:49
edleafewe already have a GET method to return a list of filtered RPs20:49
mriedembut /requests could be resource providers, or inventories, or allocations, or aggregates, etc etc20:49
bauzasedleafe: and I don't like it20:49
bauzasmriedem: that's fine to me20:49
mriedemi'm not a fan of passing a giant json blob as a query parameter either20:50
edleafebauzas: I'm sorry you don't like it, but that's the way HTTP works20:50
bauzasmriedem: I'm providing a Content-Location header for knowing which exact resource20:50
edleafemriedem: nothing json about it20:50
edleafejust key=val query parameters20:50
mriedemwhat happens when we send required vs preferred?20:50
mriedemare we going to namespace that?20:50
edleafemriedem: yes20:51
edleafejaypipes and I discussed that yesterday20:51
mriedemoh barf20:51
bauzasedleafe: again, I'm saying 'if you post that request, you'll get some identifier20:51
bauzashttps://review.openstack.org/#/c/392569/11/nova/api/openstack/placement/handlers/request.py@5820:51
bauzasaaaaand now, I'm seeing a totally different change against mine20:52
bauzaslovely20:52
*** READ10 has quit IRC20:52
mriedemare there length restrictions on query parameters?20:52
bauzasyup20:52
bauzasthe exact reason why I dislike that direction20:52
mriedem"maximum URL length is 2048 characters"20:52
edleafemriedem: yes, a little over 8000 characters20:52
bauzasedleafe: no20:52
bauzasnot over 8000 chars20:53
mriedemsorry i'm just looking at http://www.w3schools.com/TAgs/ref_httpmethods.asp20:53
bauzasedleafe: the exact answer is "it depends"20:53
bauzaswhich is terrible20:53
edleafeThe only exception was IE browser, which was very weak - about 2K20:53
cdentI'm sorry, there is a lot of misinformation being bounced around here.20:53
bauzasso, depending of which WSGI server you used, you gonna don't have the same20:53
auggymsg diana_clarke hi! how are you??20:53
auggyahah whoops20:53
edleafeThe only limit we care about is the server, not a browser20:54
* cdent smiiles at auggy 20:54
diana_clarkeauggy: good! :20:54
*** irenab has quit IRC20:54
mriedemwhy can't we do a POST with a 200 response?20:54
mriedemfor 200 repsonse, "POST   an entity describing or containing the result of the action; "20:54
*** b_rake has quit IRC20:54
mriedem*response20:54
* cdent raises his hand20:55
mriedemi'm posting a body of required/preferred stuff to process and want a list of available RPs back20:55
*** Nakato has quit IRC20:55
*** READ10 has joined #openstack-nova20:55
*** kristian__ has joined #openstack-nova20:56
bauzasmriedem: we can POST and return 20020:56
bauzasmriedem: that's exactly what I did20:56
bauzasmriedem: but the route is different20:56
*** ociuhandu has joined #openstack-nova20:56
*** Nakato has joined #openstack-nova20:56
bauzasmriedem: and I would hate having POST /rp returning either 201 or 20020:56
mriedemi don't like POST to /resource_providers having a different meaning for create and get, agree20:57
bauzasmriedem: https://review.openstack.org/#/c/392569/11/nova/api/openstack/placement/handlers/request.py@5320:57
*** Sukhdev has quit IRC20:57
cdentmriedem, bauzas: Do you guys mind if I step in here?20:58
*** kristian__ has quit IRC20:58
mriedembauzas: where is the header stuff in this?20:58
bauzascdent: edleafe already did, but go for it, of course20:58
*** xyang1 has joined #openstack-nova20:58
bauzasmriedem: https://review.openstack.org/#/c/392569/11/nova/api/openstack/placement/handlers/request.py@6120:58
cdentthere should be one and only one url that represents a list of resource providers. what have that already at /resource_providers20:59
bauzascdent: agreed20:59
cdentwe also already have a mechanism for retrieving a filtered list of resource providers: GET on that url20:59
bauzascdent: the exact reason why I'm returning a Content-Location header20:59
mriedembauzas: sure, i thought you meant it was processed from the request https://review.openstack.org/#/c/392569/11/nova/api/openstack/placement/handlers/request.py@8120:59
cdent(i'll get to the content-location header in a moment)21:00
mriedemi.e. i'm requesting some kind of processing on allocations, or inventories, or resource providers21:00
bauzascdent: which means there is no way to get a list of RPs by POST /request21:00
*** dimtruck is now known as zz_dimtruck21:00
bauzascdent: you get a result of a processing action21:00
*** anush has quit IRC21:00
bauzascdent: and that result is equivalent to a RP list, but that's not a RP list21:00
cdentthere has been zero evidence that the query string for filtering a list of resource providers will grow without bound when we add resources21:00
cdentthere seems to be an expectation that it will21:01
*** irenab has joined #openstack-nova21:01
mriedemi guess i don't like assuming it won't21:01
bauzasmriedem: oh no, the /request resource is rather a REST resource for the RequestSpec object21:01
cdentexcept that by the time we get to when we have large "request specs" that we want to send to the placement api we will be doing a POST to /allocations to make a claim21:01
*** ociuhandu has quit IRC21:01
mriedemlike when we get into nested resource providers and such21:01
mriedemand relationships21:01
cdentthe operation that we are doing now is _only_ to limit a list of resource providers21:01
mriedemand required/preferred logic21:01
bauzasmriedem: ie. we have a ReqSpec object as a resource for describing user wishes21:01
bauzasand we return something21:02
*** kristian__ has joined #openstack-nova21:03
* edleafe is starting to get deja vu about POST /servers/<id>/action overloading21:03
bauzascdent: again, the very main problem I have with GET /rps is that the maximum string size is different between two WSGI servers21:03
mriedemcdent: but an allocation requires a resource provider right?21:03
bauzasedleafe: that's something already there, so API WG should have a guideline, heh ? :)21:03
edleafebauzas: that's one of the insane APIs that led to the creation of the API-WG21:04
cdentmriedem: no, not when making an actual claim (this is some ways off): send placement requirements get back _the_ resource provider you claimed (in one step)21:04
mriedemi agree server actions is a bit wonky in places, i.e. the ones that have a body just for the action name but nothing inside the body21:04
bauzaslike, I'm totally okay, and I think the RFC7231 too, that a POST can be an action21:04
bauzasedleafe: ^21:04
edleafebauzas: *can* be21:04
cdentbauzas: there's nothing wrong with using POST for all sorts of things, however it simply doesn't apply in this case on two dimensions:21:05
bauzasedleafe: yeah, so what's the problem ?21:05
bauzasI'm not returning 20121:05
edleafethat's why the guideline says that in the extreme, POST instead of GET is acceptable21:05
cdentone is that the request needs to be against /resource_providers because that's what we want; two is that we are asking to GET a list of resource providers.21:05
bauzasI'm returning 200, which is as per the RFC what we should return when we call an action21:05
mriedemcdent: "send placement requirements get back _the_ resource provider you claimed (in one step)"21:05
mriedemin ^,21:05
mriedemif there are multiple resource providers that satisfy the request spec, what's the logic to pick one? just random?21:06
cdentmriedem: that's the long term vision when doing a POST to /allocations21:06
mriedemsure21:06
cdentthat's when the placement API _is_ the scheduler21:06
mriedemso we start filtering/weighing RPs21:06
mriedemand return the one RP that was picked21:06
cdentyeah, way long term21:06
mriedemfor the allocation21:06
*** Sukhdev has joined #openstack-nova21:07
bauzasmriedem: FWIW, I originally called the REST resource "claim" but that looked overkill, so I preferred naming it "request" which is less confusing and more abstract :)21:07
cdentin the meantime we're just doing limiting21:07
mriedemwhy couldn't POST /resource_provider/requests/ resolve #1 above?21:07
*** kristian__ has quit IRC21:07
mriedemi know it's icky21:08
mriedembut it's at least scoped to the RP resource21:08
bauzascdent: edleafe: so I hadn't get an answer to my precise point, don't you think it could defeat interop to get something very different if we use GET with a long query string, depending on your WSGI server ?21:08
bauzasmriedem: I'm fine with that21:08
bauzasmriedem: I'm not *that* tied to using /request21:08
mriedemanother thing is, the example request in http://docs-draft.openstack.org/18/385618/3/check/gate-nova-specs-docs-ubuntu-xenial/b089055//doc/build/html/specs/ocata/approved/resource-providers-get-by-request.html#post-resource-providers makes sense to me,21:08
*** paw has quit IRC21:09
bauzasmriedem: but cdent argued that it would mean that 'requests' could be an RP21:09
edleafebauzas: what is the worst case scenario? IOW, what's the shortest query string any WSGI server supports?21:09
mriedemso i think i'd like to see examples of what a similar request would look like with the GET and filter params21:09
cdentmriedem: that's hieararchy you are inferring there is not really how URIs work, but that's neither hear nor there. If POST is required (even though I still see no valid reason for why) then POST to /resource_providers (with a different schema than used for create) would the right thing semantically. But I still think GET is (way) more correct.21:09
mriedemGET /resource_providers/?VCPU=2&MEMORY_MB=102421:09
cdentmriedem: that's one of the proposals, jay put something slightly different in his recent change that implements his idea21:10
edleafemriedem: that spec was passed with the caveat that the approach was still to be determined21:10
cdentbut it is quite similar to that21:10
bauzasedleafe: that's not a valid answer, the valid answer is still 'it depends'21:10
mriedemedleafe: yes i know21:10
edleafemriedem: I'd be happy to create an alternate implementation21:10
sdagueis this POST more like a search? (coming up to speed late)21:10
bauzasedleafe: if some server changes the max size for the URI, then kaboom21:10
mriedemsdague: yes21:10
cdentsdague: it's just a filter21:10
*** irenab has quit IRC21:10
cdentas in it is _literally_ a filter, and is implemented as one21:11
sdagueI guess, would POST /resource_providers/search be more clear?21:11
mriedemedleafe: cdent: i'd like to compare POST vs GET with required/preferred resource classes21:11
mriedembecause i have a feeling the namespacing is going to get nasty21:11
bauzassdague: that is a reasonable approach which isn't fully RESTful but totally acceptable to me for the god sake of pragmatism21:11
edleafebauzas: unless you have actual numbers for these supposed limits, it's meaningless21:11
sdaguebecause this is also about consumers keeping the model in their head21:11
cdenthere's jay's proposal: http://lists.openstack.org/pipermail/openstack-dev/2016-December/108545.html21:11
sdagueand even if POST /resource_providers is technically ok21:11
sdagueit's different than the way we use POST to top level constructs in other places in openstack, and so ends up being conceptual speed bump21:12
*** ftersin_ has quit IRC21:12
cdentBTW: part of the reason for wanting GET is because of pragmatism: I can create, by typing, a reasonable query to see which compute nodes can give me 4 VPCU and 20GB of disk21:12
cdentno json required21:12
sdaguecdent: yeh, GET seems ideal honestly21:13
bauzascdent: but we could be stuck with that forever21:13
sdaguebauzas: well, maybe, we have microversioning21:13
*** ociuhandu has joined #openstack-nova21:13
bauzascdent: and then, once using traits and large requests, we could regret it21:13
sdagueif we think there is a good chance that GET works for a while21:13
bauzasand that's not a FUD21:13
edleafebauzas: likewise, we would be stuck with POST forever21:13
sdaguewe could have that as a primary interface21:13
sdagueand we could always add POST /resource_providers/search later in the game for really complex queries21:14
mriedemGET /resource_providers?resources=DISK_GB:40,VCPU:2,MEMORY_MB:2048&required=storage:ssd,hw:cpu:x86:avx2&preferred=virt:hyperv:gen221:14
bauzassdague: I'd prefer to go on the POST /rp/search way now21:14
edleafejay's example is fairly complex, and is all of 126 characters21:14
mriedem^ is jaypipes' more complicated example with namespaces from the ML21:14
cdentit's important to remember (as I said above) that there are two different interfaces under construction: now we are creating one to limit a list of resource providers by some characteristics. later we are creating one to fundamentally claim one21:14
bauzascdent: and ?21:15
bauzascdent: your point being?21:15
bauzascdent: that those two interfaces should be the same ?21:15
bauzasor the contrary?21:15
mriedemcan i try to interpret?21:15
bauzasmriedem: your pleasure21:16
mriedemi think cdent is saying that by the time we need to deal with large complicated claims like this, or filters/claims, that those are POSTs to /allocations,21:16
cdentbauzas: no, that the later interface will of necessity be different (it will involve a write to persistence and will have far more specific semantics)21:16
cdentalso what mriedem says21:16
mriedembut right now we don't have traits, so maybe we don't need to boil the ocean on that requirement21:16
mriedemif the simple query param filtering on GET is sufficient21:16
mriedemuntil it's not21:16
mriedemat which point, microversions mothertruckers21:16
sdaguemriedem: and when it is not, we can build an interface that is sufficient21:16
sdaguebut lets not overly complicate the current interface well ahead of time21:17
sdagueGET is definitely easier to understand and dev hack against in adhoc ways21:17
bauzaslooks like the consensus moved then21:17
*** ociuhandu has quit IRC21:17
mriedemwhen we started talking about this <30 min ago i wasn't in the know about the POST to /allocations (claims) stuff21:17
bauzaswell21:18
*** kristia__ has joined #openstack-nova21:18
mriedemif we eventually reserve the complicated request spec stuff for that, i'm fine21:18
bauzason one hand, we say we shouldn't consider traits as a thing21:18
cdentmriedem: we talked about that somewhat vaguely in portland, but only vaguely21:18
mriedemwe have no required/preferred logic built in right now21:18
bauzasbut on the other hand, we overcomplicate ourselves defining a scheduler claim interface we haven't yet agreed to have21:18
cdenteven when we do add traits, it will not be hugely complex, nor lengthy21:18
mriedemcdent: i lost track of about 60% of what was said in hillsboro wrt RPs21:18
*** WillUK2010 has joined #openstack-nova21:19
mriedemcdent: except that we'd have 10 nova DBs by queens21:19
cdentmriedem: yeah, I think everyone did. Only 10, I'm pretty sure it was 1121:19
sdagueand, more importantly, if we do hit a limit, we actually have a mechanism for addressing that21:19
mriedemcdent: well i wouldn't assume that, because NFV21:19
*** cleong has quit IRC21:19
sdagueso it's not like GET actually paints us into a corner21:19
cdentis 'because NFV' the new 'because docker'?21:20
mriedemwell, because NFV means, because crazy fucking use case21:20
*** Nakato has quit IRC21:20
sdaguewell, in NFV you specify which version of which FPGA firmware you are going to put this on paired with firmware version you need in your nics.... :)21:20
*** Nakato has joined #openstack-nova21:20
*** irenab has joined #openstack-nova21:21
mriedemyeah21:21
sdagueI do agree this might grow into crazy land, but it still feels better to do the simpler thing first, with the "here is how we can do the more complicated thing if we have to"21:21
mriedemso, the request spec gets nice and uber complicated21:21
mriedemi think i've come around to that also21:22
sdagueyeh, it becomes NP complete problem to figure out where a VM lands21:22
edleafeHmmm... my google-fu is lacking. Can't find any documentation on query string limits for WSGI servers21:22
edleafeBut this shows limits of servers and browsers: https://boutell.com/newfaq/misc/urllength.html21:23
bauzasedleafe: for example, I hate that http://serverfault.com/questions/757364/issues-with-url-length-in-apache-mod-wsgi21:24
bauzasI mean, I hate the problem it could create21:24
cdentI think this is a good problem to have21:25
edleafebauzas: thanks. That notes problems with URLs over 6K. I don't think we'll ever get close to that21:25
bauzasI know that for now, CPU, RAM and mem are not lengthy bits21:25
cdentif someone is making a multi-thousand char query to filter their list of resource providers, something is pathological and we've made some big mistakes on how we've designed and managed our systems21:25
bauzasbut I assume we were trying to address PCI requests ?21:25
bauzasand NUMA queries ?21:25
sdagueyeh, 4K or 8K I thought was the issue21:25
sdaguewe actually hit this with ec2 api when it was in tree21:26
sdaguebecause that's entirely a GET interface21:26
sdagueand some EC2 api calls > 8k of query params21:26
sdaguewhich apache can't do21:26
edleafesdague: ugh - I remember those21:26
mriedemcdent: s/someone/compute/ in NFV21:27
mriedemremember21:27
mriedemcrazy21:27
cdentI'll stick to my statement: if the system starts requiring things that are that long, the system blowing up on them is our signal that we need to do something differently or make them stop21:27
openstackgerritJim Rollenhagen proposed openstack/nova: Update ironic driver get_available_nodes docstring  https://review.openstack.org/40830821:27
jroll^ easy review if someone needs a break from the hard stuff, just a docstring update21:27
mriedemjroll: !21:28
edleafeThe request will be the resource class and amount. The complexity is in the nested resource providers21:28
*** smatzek has quit IRC21:28
mriedemyou've walked into it now21:28
edleafeThat isn't part of the URL21:28
jrolluh oh21:28
*** ociuhandu has joined #openstack-nova21:28
mriedemedleafe: yes it's nested RPs that worry me21:28
edleafejroll: wrong place at the wrong time21:28
edleafe:)21:28
bauzasedleafe: what if I'm adding more custom resource classes ?21:28
edleafemriedem: but they are never part of the API21:29
bauzasedleafe: and what if I'm wanting more than just the standard tuple ?21:29
bauzasedleafe: sorry?21:29
sdaguejroll: +221:29
jrollnice, thanks sdague21:29
mriedemedleafe: why not?21:29
bauzasedleafe: say I'm adding a custom resource class of FOOBAR21:29
mriedembauzas: you mean CUSTOM_FOOBAR?21:29
bauzasedleafe: I should be able to query the placement API against that FOOBAR21:29
edleafemriedem: because the request if for a) amounts of a class and b) traits (eventually)21:29
bauzasmriedem: yeah21:29
mriedemCUSTOM_BAREMETAL_JROLL_DONUTS21:29
bauzasyeah that, loving it21:29
jrollಠ_ಠ21:29
*** burt has quit IRC21:30
edleafehow those classes are arranged with nested RPs is internal to the placement engine21:30
sdagueso... before we solve CUSTOM_BAREMETAL_JROLL_DONUTS, I would rather be using this for CPU/MEM :)21:30
bauzasor people would want to class French cheeses21:30
bauzassdague: but we agreed on having custom resource classes for Ocata :)21:30
bauzasthat's a priority too21:30
bauzaswhich means that by Ocata, people could litterally ask for anything they like21:31
sdaguebauzas: ok, but we still aren't scheduling with it?21:31
mriedemi'm less worried about GET /resource_providers/?CUSTOM_BAREMETAL_JROLL_DONUTS=5 than nested RPs21:31
*** irenab has quit IRC21:31
bauzassdague: well, we could21:31
dansmithsdague: FWIW, I agree that anything beyond reporting and scheduling with the base resources is a distraction at this point21:31
dansmithsdague: but I'm in the minority21:31
mriedemCUSTOM_BAREMETAL_JROLL_DONUTS=5 just returns me a list of ironic nodes doesn't it?21:31
sdaguedansmith: ok, I'm in that minority as well then21:31
cdentdansmith: i'm in that minority too, so it is starting to sound like not a minority21:32
edleafemriedem: yeah, because adding CUSTOM_BAREMETAL_JROLL_DONUTS=5 increases the length by a whopping 32 characters21:32
sdaguebecause, it feels like we often end up arguing about things that can't land in this cycle, instead of the things that can, so less lands in current cycle21:32
mriedemedleafe: i'm not talking about length restrictions actually21:32
mriedembut anyway21:32
*** ociuhandu has quit IRC21:32
edleafecdent: sdague: dansmith: welcome to the minority! It's getting crowded here21:32
dansmithsdague: and, it would be nice to prove out what we have and make actual gains, before we get further down the rabbit hole21:32
bauzassdague: cdent: dansmith: do we have a Majority Whip then ?21:33
edleafemriedem: understood. But POST doesn't help that either21:33
bauzascall Frank Underwood21:33
dansmithsdague: so far we have no gains from all this, just reporting a bunch of stuff we're not using21:33
sdague3 means we're an organization right?21:33
mriedemok, so let's go with GET + filteres21:33
mriedem*filters21:33
sdaguealice's restaurant rules21:33
*** yamahata has joined #openstack-nova21:33
edleafemriedem: heh, I thought that was French21:33
mriedemif we turn out to be terribly wrong, we pivot with a microversion21:33
cdentdansmith: the limiting in the scheduler is very soon, but it required this thing we were debating just now :(21:33
dansmithcdent: yeah, I know the query thing is a dep21:34
cdentyour statement is kind of a qed: "query thing" :)21:34
mriedembauzas: so i think we're doing GET21:34
jrollso the downside of not scheduling on custom stuff is it pushes getting rid of the ironic-horribleness out another cycle, to be clear (don't have a strong opinion on whether y'all do that or not)21:34
bauzasmriedem: that's what I'm seeing :)21:34
dansmithjroll: I know21:35
dansmithbauzas: really?21:35
dansmither,21:35
dansmithmriedem: really?21:35
edleafejroll: agreed, but we set modest goals for Ocata21:35
*** Jeffrey4l has quit IRC21:35
jrollright, just making sure that's clear :)21:35
jrollI know some people are super hyped about deleting that code21:35
mriedemdansmith: that's what we were just talking about21:35
dansmithmriedem: seems like the most people prefer POST no?21:35
mriedemdansmith: no21:35
*** Jeffrey4l has joined #openstack-nova21:35
mriedemunless i'm misunderstanding21:35
dansmithhuh, okay21:35
mriedemas i said earlier, i wasn't aware fully about the eventual plan to POST requests to /allocations to make RP claims21:36
mriedemtraits aren't a thing yet, so the namespace/nesting garbage in the GET query is less of a concern for me right now21:36
openstackgerritChris Dent proposed openstack/nova: WIP [placement] start a placement_dev doc  https://review.openstack.org/40831321:36
mriedemas noted, if we worry about all of that stuff too much we won't get anything done for ocata21:36
mriedemand we can always do a POST later if needed21:37
*** catintheroof has quit IRC21:37
dansmithbummer21:37
cdentjaypipes, edleafe, bauzas (and anyone else): that placement_dev doc ^^^ could do with some feedback on direction and style before I get too much further21:37
*** irenab has joined #openstack-nova21:38
*** catintheroof has joined #openstack-nova21:38
bauzasmriedem: okay, Jay wrote a bit of that direction in a separate change, I'll squash it with my change and co-author mr. jaypipes21:38
bauzasmriedem: I still think we could need a microversion very soon, but okay, let's merge that21:39
mriedembauzas: that's this right? https://review.openstack.org/#/c/408285/21:40
bauzasI'm just done with discussing over and over this since more than 4 weeks now21:40
bauzasso, if the consensus moved and now more people are inclined with GET, then prepare your +2s21:40
mriedemsdague: to make sure, you're pro GET now right?21:40
mriedemsdague: with the understanding we can change later if needed?21:40
sdaguemriedem: I am pro GET21:40
bauzasmriedem: yup21:41
mriedembauzas: i'm prepared to go home soon21:41
mriedemand cry21:41
bauzasmriedem: jay made most of the work, hence me merging that21:41
sdagueyes, with the back pocket move to change once NFV has a turing complete query string to allocate servers21:41
bauzasmriedem: that will be tomorrow anyway21:41
*** harlowja has joined #openstack-nova21:41
dansmithso,21:41
dansmithwe all know we'll have to have a post,21:41
dansmithbut meh?21:42
dansmithI like this plan21:42
bauzasexactly21:42
cdentdansmith: [t 4fPX]21:42
purplerbot<cdent> except that by the time we get to when we have large "request specs" that we want to send to the placement api we will be doing a POST to /allocations to make a claim [2016-12-07 21:01:35] [n 4fPX]21:42
bauzascdent: wrong assumption21:42
edleafedansmith: the POST is for a completely different action21:42
dansmithyeah, I disagree with that21:42
*** catintheroof has quit IRC21:42
cdentso the idea is pretty much what you were saying: do the basics now, get the win21:42
dansmithbut doing the thing that we can do with the get via post is equally as trivial21:43
cdentupdate the scheduler to actually get some advantage from the info21:43
*** thorst_ has quit IRC21:43
bauzascdent: because scheduler claims are way out of being merged soon, but nested resource providers and custom resource classes are already a thing21:43
openstackgerritSarafraj Singh proposed openstack/nova: WIP: Objectify requests_and_created_ports  https://review.openstack.org/40539821:43
dansmithanyway, whatever, I keep saying I'm going to stop having this conversation21:43
cdentlooks let's not go back into this. we have a plan21:43
edleafedansmith: another use case for this is for things like Watcher that want to check for possible hosts before live migrating21:43
*** thorst_ has joined #openstack-nova21:44
edleafethis filtered list of hosts isn't going away21:44
cdentsdague's perspective is spot on: [t n8R]21:44
purplerbot<sdague> GET is definitely easier to understand and dev hack against in adhoc ways [2016-12-07 21:17:31] [n n8R]21:44
cdentthat provides us with flexibility21:44
cdentand explorability21:44
*** thorst_ has quit IRC21:44
cdentand doesn't lock us into anything21:44
bauzasI'm out of battery and spit21:44
bauzasoops21:45
bauzasspit is maybe not the right word21:45
cdentbest quote ever21:45
cdentno it's perfect21:45
cdent:)21:45
*** slaweq_ has joined #openstack-nova21:46
*** paw has joined #openstack-nova21:46
bauzascould I beg you for changing that to "I'm out of battery and breath"21:47
bauzasso I'm gonna leave anyway and do $stuff tomorrow21:47
openstackgerritDiana Clarke proposed openstack/nova: Simple tenant usage pagination  https://review.openstack.org/38609321:48
*** irenab has quit IRC21:48
*** jamielennox|away is now known as jamielennox21:49
cdentyeah, me too, been too long of a day21:49
*** cdent has quit IRC21:51
openstackgerritSujitha proposed openstack/nova: Adding tags to InstancePayload  https://review.openstack.org/40722821:53
*** takashin has joined #openstack-nova21:54
*** krtaylor has quit IRC21:58
*** irenab has joined #openstack-nova21:59
*** slunkad has quit IRC22:05
*** slunkad has joined #openstack-nova22:06
*** mriedem has quit IRC22:06
*** irenab has quit IRC22:07
*** dave-mccowan has quit IRC22:09
openstackgerritDiana Clarke proposed openstack/nova: Simple tenant usage pagination  https://review.openstack.org/38609322:11
*** kristia__ has quit IRC22:12
*** baoli has quit IRC22:12
*** baoli has joined #openstack-nova22:13
*** irenab has joined #openstack-nova22:14
*** baoli has quit IRC22:18
*** kaisers_ has quit IRC22:18
*** irenab has quit IRC22:19
*** jheroux has quit IRC22:23
*** READ10 has quit IRC22:25
*** eharney has quit IRC22:27
*** kylek3h_away has quit IRC22:32
*** armax has joined #openstack-nova22:33
*** tblakes has quit IRC22:35
*** slaweq_ has quit IRC22:37
*** esberglu has quit IRC22:37
*** edand has quit IRC22:38
*** esberglu has joined #openstack-nova22:38
*** krtaylor has joined #openstack-nova22:42
*** esberglu has quit IRC22:42
*** rcernin has quit IRC22:48
*** ayoung has quit IRC22:48
*** lamt has quit IRC22:48
*** jamesdenton has quit IRC22:51
*** browne has quit IRC22:53
*** mriedem has joined #openstack-nova22:55
*** unicell has joined #openstack-nova23:03
*** armax has quit IRC23:03
*** mdrabe has quit IRC23:06
*** ducttape_ has quit IRC23:07
*** WillUK2010 has quit IRC23:09
*** gjayavelu has quit IRC23:11
*** armax has joined #openstack-nova23:12
*** gjayavelu has joined #openstack-nova23:14
*** kaisers_ has joined #openstack-nova23:14
*** gjayavelu has joined #openstack-nova23:14
*** mlavalle has quit IRC23:15
*** rfolco has joined #openstack-nova23:15
*** rfolco has quit IRC23:15
*** harlowja has quit IRC23:16
*** david-lyle_ has joined #openstack-nova23:20
*** david-lyle_ has quit IRC23:20
*** xyang1 has quit IRC23:22
*** sacharya has quit IRC23:23
*** sacharya has joined #openstack-nova23:24
*** zz_dimtruck is now known as dimtruck23:26
*** jamielennox is now known as jamielennox|away23:27
*** jamielennox|away is now known as jamielennox23:28
*** WillUK2010 has joined #openstack-nova23:31
*** armax has quit IRC23:34
*** browne has joined #openstack-nova23:41
*** esberglu has joined #openstack-nova23:45
openstackgerritSarafraj Singh proposed openstack/nova: Objectify requests_and_created_ports  https://review.openstack.org/40539823:45
*** ijw has quit IRC23:46
*** ijw has joined #openstack-nova23:46
*** esberglu has quit IRC23:49
*** tbachman has quit IRC23:50
*** erlon-airlong has quit IRC23:50
*** ducttape_ has joined #openstack-nova23:56

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