Wednesday, 2015-07-29

*** vilobhmm has quit IRC00:00
*** ssurana1 has joined #openstack-nova00:02
*** pkoniszewski has quit IRC00:02
*** tjones has quit IRC00:03
*** tjones has joined #openstack-nova00:04
*** lucas__ has joined #openstack-nova00:04
tjonesso speaking of config options.  i noticed when i run tox -egenconfig it does not generate all of the config variables.  Is there a way to make it do so?00:04
*** davideagnello has quit IRC00:05
*** pkoniszewski has joined #openstack-nova00:05
mriedemtjones: i thought there was a bug for that but not sure where it's all landed00:06
mriedemor if there was an outstanding change, dims might know00:06
tjonesfor example - verbose is not there or debug, and others :-(00:06
tjonesdims: do you know??00:06
*** lucas__ has quit IRC00:08
dimstjones: which specific config option? (from which python module?00:08
mriedemdims: verbose/debug00:09
tjoneswell what i want are all of them00:09
mriedemthose come from oslo.log right?00:09
mriedemtjones: so new world order is the nova options come from http://git.openstack.org/cgit/openstack/nova/tree/setup.cfg#n2800:09
mriedembut i'm not sure how options from oslo / keystone libs are pulled in00:09
mriedemshould come from here http://git.openstack.org/cgit/openstack/nova/tree/etc/nova/nova-config-generator.conf#n1300:10
mriedemwhich should pull these in http://git.openstack.org/cgit/openstack/oslo.log/tree/oslo_log/_options.py00:11
tjonesah oslo.log is there.  oh crap - im on kilo right now.  im sure it's totally different…00:11
*** jasondotstar has quit IRC00:11
*** ijw has quit IRC00:12
mriedemgah!00:12
dimstjones: ok00:15
*** achanda has quit IRC00:15
tjonesdims: ok??00:16
dimstjones: kilo, i haven't touched :) the oslo-config-generator is only in master00:16
*** jasondotstar has joined #openstack-nova00:17
tjonesam i right in thinking that in kilo when you do that, all config options should be in the sample?  if so  something broke in kilo.  ill take a look and see if i can figure it out.00:17
dimstjones: y, please do, i can look tomorrow if you are still stuck00:18
tjonesi'll give it a try.  i may ping you tomorrow if i get stuck ;-)00:19
*** xek_ has joined #openstack-nova00:19
*** xek has quit IRC00:20
*** isq_ has joined #openstack-nova00:20
*** isq has quit IRC00:20
*** Piet has quit IRC00:21
mriedemtjones: i seem to remember this being a known issue in kilo where the external oslo lib options weren't pulled into nova's sample config00:22
tjonesim just not sure what else is missing - just oslo stuff?00:23
*** david-lyle has quit IRC00:24
*** ujjain has quit IRC00:24
*** mdrabe has joined #openstack-nova00:24
mriedemand probably keystonemiddleware00:26
mriedemanything that's not an option in nova itself, listed in here http://git.openstack.org/cgit/openstack/nova/tree/etc/nova/nova-config-generator.conf#n1300:26
*** ujjain has joined #openstack-nova00:26
*** ujjain has quit IRC00:26
*** ujjain has joined #openstack-nova00:26
tjonesah ok thanks00:28
*** veena has quit IRC00:29
*** tonytan4ever has joined #openstack-nova00:30
*** tsekiyam_ has joined #openstack-nova00:32
openstackgerritMeghal Gosalia proposed openstack/nova-specs: Quota Management for Dynamic Quota Resources  https://review.openstack.org/20616000:32
*** xyang1 has quit IRC00:33
openstackgerritMeghal Gosalia proposed openstack/nova-specs: Quota Management for Dynamic Quota Resources  https://review.openstack.org/20616000:33
*** mdrabe has quit IRC00:34
*** changbl has joined #openstack-nova00:34
*** mtanino has quit IRC00:34
*** gzhai has quit IRC00:34
openstackgerritMeghal Gosalia proposed openstack/nova-specs: Quota Management for Dynamic Quota Resources  https://review.openstack.org/20616000:34
*** tsekiyama has quit IRC00:35
*** tsekiyam_ has quit IRC00:36
*** shyu has joined #openstack-nova00:37
*** otter768 has joined #openstack-nova00:37
*** mwagner_lap has joined #openstack-nova00:40
*** leeantho has quit IRC00:41
*** pkoniszewski has quit IRC00:43
*** browne_ has joined #openstack-nova00:45
*** xek_ has quit IRC00:46
*** xek_ has joined #openstack-nova00:47
*** browne_ has quit IRC00:47
*** tonytan4ever has quit IRC00:48
*** pkoniszewski has joined #openstack-nova00:50
*** MentalRay_ has joined #openstack-nova00:50
*** zhenguo has joined #openstack-nova00:57
*** nic has quit IRC00:59
*** hshiina has joined #openstack-nova01:02
*** sajeesh has joined #openstack-nova01:03
*** thorst has joined #openstack-nova01:03
*** wanghua has joined #openstack-nova01:04
sajeeshping melwitt01:04
*** wanghua has quit IRC01:05
*** wanghua has joined #openstack-nova01:05
*** angdraug has quit IRC01:05
*** lucas____ has joined #openstack-nova01:05
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtISCSIVolumeDriver into it's own module  https://review.openstack.org/20230101:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtISERVolumeDriver into it's own module  https://review.openstack.org/19731101:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move volume/remotefs/quobyte modules under volume subdir  https://review.openstack.org/19322301:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtNFSVolumeDriver into it's own module  https://review.openstack.org/19685901:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtNETVolumeDriver into it's own module  https://review.openstack.org/19734101:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move the LibvirtScalityVolumeDriver into it's own module  https://review.openstack.org/19324901:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtAOEVolumeDriver into it's own module  https://review.openstack.org/19627601:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move the LibvirtQuobyteVolumeDriver into the quobyte module  https://review.openstack.org/19323601:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert SMBFS vol driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20277901:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert NFS volume driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20277801:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtFibreChannelVolumeDriver into it's own module  https://review.openstack.org/19327801:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtGlusterfsVolumeDriver into it's own module  https://review.openstack.org/19338001:06
openstackgerritMatt Riedemann proposed openstack/nova: Introduce LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20277701:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert Scality vol driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20278301:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert Quobyte driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20278201:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move the LibvirtGPFSVolumeDriver into it's own module  https://review.openstack.org/19324001:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: convert GPFS volume driver to LibvirtBaseFileSystemVolumeDriver  https://review.openstack.org/20278101:06
openstackgerritMatt Riedemann proposed openstack/nova: libvirt: move LibvirtSMBFSVolumeDriver into it's own module  https://review.openstack.org/19638001:06
mriedemrebasing that *sucked*01:06
*** tjones has quit IRC01:06
*** browne has quit IRC01:08
openstackgerritZhenyu Zheng proposed openstack/nova: Resize: Return to initial state instead of ACTIVE when resize fails  https://review.openstack.org/20237901:08
openstackgerritZhenyu Zheng proposed openstack/nova: Add instance action events for live migration  https://review.openstack.org/19537201:09
*** Jhelf has left #openstack-nova01:09
*** lucas____ has quit IRC01:10
*** jasondotstar has quit IRC01:12
*** thorst has quit IRC01:14
*** meghal has quit IRC01:14
*** humble_ has joined #openstack-nova01:14
*** armax has joined #openstack-nova01:15
*** ssurana has joined #openstack-nova01:16
*** ssurana1 has quit IRC01:16
*** jasondotstar has joined #openstack-nova01:17
*** wanghua has quit IRC01:17
*** r-daneel has quit IRC01:19
*** salv-orlando has joined #openstack-nova01:20
openstackgerritIan Cordasco proposed openstack/python-novaclient: Use keystoneclient's TCPKeepAliveAdapter  https://review.openstack.org/20474501:20
*** jasondotstar has quit IRC01:22
*** shuquan has joined #openstack-nova01:23
*** salv-orlando has quit IRC01:25
*** patrickeast has quit IRC01:25
*** annashen has joined #openstack-nova01:26
*** mriedem has quit IRC01:26
*** shuquan has quit IRC01:29
*** shuquan_ has joined #openstack-nova01:29
*** annashen has quit IRC01:31
*** ssurana has quit IRC01:32
*** lucas____ has joined #openstack-nova01:33
*** shuquan has joined #openstack-nova01:35
*** shuquan_ has quit IRC01:36
*** lucas____ has quit IRC01:37
*** shuquan_ has joined #openstack-nova01:54
*** shuquan has quit IRC01:55
*** shuquan_ is now known as shuquan01:55
*** annashen has joined #openstack-nova01:55
*** annashen has quit IRC02:00
*** annashen has joined #openstack-nova02:10
*** annashen has quit IRC02:12
*** pixelbeat_ has joined #openstack-nova02:12
*** dboik has joined #openstack-nova02:18
*** david-lyle has joined #openstack-nova02:19
*** unicell has quit IRC02:21
*** haomaiwang has quit IRC02:21
*** haomaiwa_ has joined #openstack-nova02:22
*** david-lyle has quit IRC02:23
*** tpeoples has joined #openstack-nova02:25
*** Piet has joined #openstack-nova02:26
*** annashen has joined #openstack-nova02:27
*** dboik has quit IRC02:27
*** dboik has joined #openstack-nova02:27
*** jwcroppe has quit IRC02:27
*** jwcroppe has joined #openstack-nova02:28
*** annashen has quit IRC02:32
*** jwcroppe has quit IRC02:32
*** lucas__ has joined #openstack-nova02:34
*** baoli has joined #openstack-nova02:35
*** haomaiwa_ has quit IRC02:36
*** haomaiwang has joined #openstack-nova02:36
*** achanda has joined #openstack-nova02:37
*** lucas__ has quit IRC02:39
*** boris-42 has quit IRC02:40
*** browne has joined #openstack-nova02:40
*** armax has quit IRC02:41
*** oomichi has joined #openstack-nova02:51
*** hakimo has joined #openstack-nova02:51
*** hakimo_ has quit IRC02:53
oomichialex_xu: hi, morning02:55
alex_xuoomichi: hi, o ha yo02:56
oomichialex_xu: o ha yo ;)02:56
oomichialex_xu: about https://review.openstack.org/#/c/167905/7/nova/compute/api.py02:57
oomichialex_xu: do you mean we need to remove instance.info_cache.delete() call on that ?02:57
alex_xuoomichi: yea, I confused on destroy instance will do the same thing02:58
alex_xuoomichi: but let me check the replied comment.02:58
oomichialex_xu: ok, thanks in advance :)02:59
alex_xuoomichi: np02:59
oomichialex_xu: one more thing. can you check https://review.openstack.org/#/c/192043/ also again ?02:59
*** shuquan_ has joined #openstack-nova03:00
alex_xuoomichi: yea, sure, will check that also.03:00
*** shuquan has quit IRC03:00
*** shuquan_ is now known as shuquan03:00
oomichialex_xu: thanks again03:00
alex_xuoomichi: np again03:00
eliqiaoalex_xu: hi, can you please recheck https://review.openstack.org/#/c/165671/10/nova/virt/libvirt/driver.py , I replied your comment.03:02
alex_xueliqiao: yea, will check also03:02
* alex_xu have a lot of debt03:03
*** MentalRay_ has quit IRC03:03
eliqiaoalex_xu: thanks in advance03:03
alex_xueliqiao: np, will reach it before the end of day!03:03
eliqiaoalex_xu: take it easy...:)03:04
*** tjones has joined #openstack-nova03:04
*** sajeesh has quit IRC03:06
*** dboik has quit IRC03:08
*** dboik has joined #openstack-nova03:09
*** annashen has joined #openstack-nova03:10
*** pixelbeat_ has quit IRC03:10
*** deepthi has joined #openstack-nova03:11
*** mragupat has joined #openstack-nova03:12
alex_xueliqiao: :)03:12
*** mragupat_ has joined #openstack-nova03:13
*** dboik has quit IRC03:14
*** haomaiwang has quit IRC03:14
*** mragupat has quit IRC03:16
*** baoli has quit IRC03:20
*** whenry has quit IRC03:21
*** yamahata has joined #openstack-nova03:21
*** coolsvap|away is now known as coolsvap03:26
*** annashen_ has joined #openstack-nova03:32
*** tjones has quit IRC03:33
*** david-lyle has joined #openstack-nova03:34
*** armax has joined #openstack-nova03:35
*** annashen has quit IRC03:35
*** whenry has joined #openstack-nova03:37
openstackgerritGhanshyam Mann proposed openstack/nova: Show 'locked' information in server details  https://review.openstack.org/20239103:42
*** fawadkhaliq has joined #openstack-nova03:42
*** otter768 has quit IRC03:43
*** shuquan has quit IRC03:43
*** cpschult has joined #openstack-nova03:44
*** ishant has joined #openstack-nova03:48
*** shuquan has joined #openstack-nova03:49
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/20682703:49
*** achanda has quit IRC03:55
*** annashen has joined #openstack-nova03:58
*** jhesketh has quit IRC04:01
*** hshiina has quit IRC04:02
*** jhesketh has joined #openstack-nova04:02
*** shuquan has quit IRC04:03
*** annashen has quit IRC04:03
openstackgerritMike Dorman proposed openstack/nova: Add console allowed origins setting  https://review.openstack.org/20167704:06
openstackgerritZhenyu Zheng proposed openstack/nova: Resize: Return to initial state instead of ACTIVE when resize fails  https://review.openstack.org/20237904:06
*** annashen_ has quit IRC04:07
*** dboik has joined #openstack-nova04:09
*** mragupat_ has quit IRC04:09
*** vishwanathj has joined #openstack-nova04:10
*** dboik has quit IRC04:14
*** jwcroppe has joined #openstack-nova04:14
*** cpschult has quit IRC04:16
openstackgerritVladik Romanovsky proposed openstack/nova: Adding user_id handling to keypair index, show and create api calls  https://review.openstack.org/7048504:28
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/20682704:30
*** shuquan has joined #openstack-nova04:31
openstackgerritGhanshyam Mann proposed openstack/nova-specs: Fixing lock information spec for latest updates  https://review.openstack.org/20686404:33
*** lucas__ has joined #openstack-nova04:38
*** ftersin has joined #openstack-nova04:39
*** armax has quit IRC04:42
*** lucas__ has quit IRC04:42
*** hshiina has joined #openstack-nova04:42
*** shuquan has quit IRC04:45
*** vilobhmm has joined #openstack-nova04:49
*** unicell has joined #openstack-nova04:49
*** annashen has joined #openstack-nova04:49
openstackgerritGhanshyam Mann proposed openstack/nova: Add JSON-Home document methods  https://review.openstack.org/18955104:50
*** rushiagr_away is now known as rushiagr04:50
*** davideagnello has joined #openstack-nova04:51
openstackgerritGhanshyam Mann proposed openstack/nova: Add register method for JSON-Home data- index/create  https://review.openstack.org/20488904:51
openstackgerritGhanshyam Mann proposed openstack/nova: Add template_parameter_relation JSON-Home methods  https://review.openstack.org/20537404:51
openstackgerritGhanshyam Mann proposed openstack/nova: Support for registering JSON-Home data- show/delete/update  https://review.openstack.org/20500104:51
openstackgerritGhanshyam Mann proposed openstack/nova: Add resource_template_with_parent JSON-Home methods  https://review.openstack.org/20539204:52
openstackgerritGhanshyam Mann proposed openstack/nova: Register JSON-Home data for resources with parent  https://review.openstack.org/20590504:52
openstackgerritGhanshyam Mann proposed openstack/nova: Register JSON-Home data for resources with inherits  https://review.openstack.org/20594404:52
openstackgerritGhanshyam Mann proposed openstack/nova: Register JSON-Home data for resources actions  https://review.openstack.org/20636004:52
*** shaohe_robot has quit IRC04:54
*** garyk has joined #openstack-nova04:54
*** zzz_pczesno has joined #openstack-nova04:54
*** shaohe_robot has joined #openstack-nova04:55
*** pczesno has quit IRC04:56
*** annashen_ has joined #openstack-nova04:59
*** annashen_ has quit IRC05:03
*** tpeoples has quit IRC05:07
*** xgsa has joined #openstack-nova05:09
*** annashen has quit IRC05:10
*** davideagnello has quit IRC05:12
*** rotbeard has joined #openstack-nova05:14
*** annashen has joined #openstack-nova05:16
*** jwcroppe has quit IRC05:25
*** ajayaa has quit IRC05:25
*** jwcroppe has joined #openstack-nova05:25
*** achanda has joined #openstack-nova05:26
alex_xueliqiao: it's done, I think you are right.05:28
eliqiaoalex_xu:  cool, thanks for your supporting and reviewing..05:29
alex_xueliqiao: np05:29
*** jwcroppe has quit IRC05:30
*** deepakcs has joined #openstack-nova05:31
*** lpetrut has joined #openstack-nova05:31
*** kumar has joined #openstack-nova05:36
openstackgerritVilobh Meshram proposed openstack/nova-specs: Quota Management for Dynamic Quota Resources  https://review.openstack.org/20616005:37
*** chhavi has joined #openstack-nova05:38
*** shz has quit IRC05:38
*** liusheng has quit IRC05:38
*** liusheng has joined #openstack-nova05:39
*** lucas__ has joined #openstack-nova05:40
*** otter768 has joined #openstack-nova05:44
*** lucas__ has quit IRC05:44
*** jwcroppe has joined #openstack-nova05:45
*** Nikolay_St has quit IRC05:47
*** otter768 has quit IRC05:48
*** rajesht has joined #openstack-nova05:51
*** Murali has joined #openstack-nova05:58
*** scheuran has joined #openstack-nova05:59
*** annashen_ has joined #openstack-nova06:00
*** annashen has quit IRC06:00
*** lpetrut has quit IRC06:00
*** park_heijlong_ has joined #openstack-nova06:02
*** lpetrut has joined #openstack-nova06:04
*** annashen_ has quit IRC06:04
*** kiran-r has joined #openstack-nova06:05
*** rushiagr is now known as rushiagr_away06:07
*** afazekas has joined #openstack-nova06:09
*** liusheng has quit IRC06:12
*** ajayaa has joined #openstack-nova06:13
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Updated from global requirements  https://review.openstack.org/20682706:13
*** liusheng has joined #openstack-nova06:15
*** ftersin has quit IRC06:16
*** rushiagr_away is now known as rushiagr06:16
*** nkrinner has joined #openstack-nova06:18
openstackgerritOpenStack Proposal Bot proposed openstack/nova: Imported Translations from Transifex  https://review.openstack.org/19721306:20
*** josecastroleon has joined #openstack-nova06:22
*** whenry has quit IRC06:25
*** whenry has joined #openstack-nova06:27
*** shz has joined #openstack-nova06:28
*** moshele has joined #openstack-nova06:29
*** wanghao has joined #openstack-nova06:30
*** zhenguo has quit IRC06:32
*** armax has joined #openstack-nova06:33
*** rushiagr is now known as rushiagr_away06:34
*** abhishekk has joined #openstack-nova06:34
*** Marga_ has quit IRC06:36
*** Nikolay_St has joined #openstack-nova06:37
*** zhenguo has joined #openstack-nova06:40
*** Marga_ has joined #openstack-nova06:40
*** lucas__ has joined #openstack-nova06:42
*** lpetrut has quit IRC06:45
*** lucas__ has quit IRC06:46
*** kfox1111_ has quit IRC06:47
*** sahid has joined #openstack-nova06:47
openstackgerritYongli He proposed openstack/nova: resource tracker style pci resource management  https://review.openstack.org/13132106:47
garykndipanov: you around? quick question - do you remember the nova boot command to spin up a bdm for a ephemeral disk06:47
*** browne has quit IRC06:49
*** gzhai has joined #openstack-nova06:50
*** gszasz has joined #openstack-nova06:53
*** shuquan has joined #openstack-nova06:53
*** achanda has quit IRC06:54
*** alex_klimov has joined #openstack-nova06:56
*** karimb has joined #openstack-nova06:57
*** achanda has joined #openstack-nova06:57
*** markus_z has joined #openstack-nova06:58
*** vilobhmm has quit IRC06:59
*** tongli_ has quit IRC07:00
*** annashen has joined #openstack-nova07:01
*** mpavone has joined #openstack-nova07:03
*** yamahata has quit IRC07:05
*** annashen has quit IRC07:05
*** sorrison has quit IRC07:07
garykanyone here know off hand the block device params to pass when doing a nova boot07:10
*** cfriesen_ has quit IRC07:10
*** rushiagr_away is now known as rushiagr07:10
*** salv-orlando has joined #openstack-nova07:12
*** davideagnello has joined #openstack-nova07:13
*** openstack-glance has joined #openstack-nova07:14
*** adam_g has quit IRC07:14
*** davideagnello has quit IRC07:17
garykmdbooth: around?07:20
*** rushiagr is now known as rushiagr_away07:20
*** jwcroppe has quit IRC07:22
*** karimb has quit IRC07:22
*** karimb has joined #openstack-nova07:23
*** openstack-glance is now known as kodoku07:23
*** kodoku is now known as kodoku_07:23
*** armax has quit IRC07:25
*** shuquan has quit IRC07:27
*** achanda has quit IRC07:27
*** kmARC has joined #openstack-nova07:28
markus_zbauzas: FYI https://bugs.launchpad.net/nova/+bug/147918107:29
openstackLaunchpad bug 1479181 in OpenStack Compute (nova) "Cells: Build instance doesn't work with kilo api, juno compute " [Critical,New]07:29
*** Jeffrey4l has joined #openstack-nova07:31
*** kumar has quit IRC07:31
*** shuquan has joined #openstack-nova07:33
*** rdopiera has joined #openstack-nova07:34
*** markus_z has quit IRC07:34
*** rdopiera_ has joined #openstack-nova07:35
*** romainh has joined #openstack-nova07:36
*** rdopiera_ has quit IRC07:36
*** rdopiera has quit IRC07:36
*** rdopiera has joined #openstack-nova07:36
*** shuquan_ has joined #openstack-nova07:37
*** shuquan has quit IRC07:38
*** shuquan_ is now known as shuquan07:38
*** jlanoux has joined #openstack-nova07:38
*** y_sawai has joined #openstack-nova07:42
*** lucas__ has joined #openstack-nova07:43
*** otter768 has joined #openstack-nova07:44
*** lpetrut has joined #openstack-nova07:46
*** lucas__ has quit IRC07:48
*** otter768 has quit IRC07:49
*** y_sawai has quit IRC07:49
*** markus_z has joined #openstack-nova07:51
*** jwcroppe has joined #openstack-nova07:51
*** bfic has joined #openstack-nova07:51
kashyapgaryk: Heya, are you looking for this? -- http://docs.openstack.org/openstack-ops/content/attach_block_storage.html07:54
garykkashyap: i tried to use that but was unable to spin up an ephemeral disk. get all kinds of errors that are cryptic from nova api.07:57
*** y_sawai has joined #openstack-nova07:58
kashyapgaryk: From my history on a DevStack machine, I see one of these CLIs working correctly: nova boot --flavor 1 --block_device_mapping vda=${SNAPSHOT_ID}:snap::0 --key_name oskey volume_backed07:58
kashyapWhere  SNAPSHOT_ID=$(cinder snapshot-list | awk '/bootable_snapshot/ {print $2}')07:58
garykkashyap: ok, thanks07:59
garyki was trying nova boot --image cirros-0.3.2-i386-disk  --flavor eli --ephemeral size=1 X07:59
garykand this is what i got - ERROR (BadRequest): Block Device Mapping is Invalid: Boot sequence for the instance and image/block device mapping combination is not valid. (HTTP 400)07:59
kashyapIIRC, I was testing: "Create bootable volume, boot a nova instance from it"08:00
kashyapgaryk: I'll test yours once my new DevStack instance is built08:00
openstackgerritAlex Xu proposed openstack/nova: Skip additionalProperties checks when LegacyV2CompatibleWrapper enabled  https://review.openstack.org/19385808:01
openstackgerritAlex Xu proposed openstack/nova: Fix the wrong file name for legacy v2 compatible wrapper functional test  https://review.openstack.org/20632608:01
openstackgerritAlex Xu proposed openstack/nova: Add scenario for API sample tests with legacy v2 compatible wrapper  https://review.openstack.org/20348508:01
*** annashen has joined #openstack-nova08:01
*** danpb has joined #openstack-nova08:01
*** annashen_ has joined #openstack-nova08:02
*** y_sawai has quit IRC08:02
*** matrohon has joined #openstack-nova08:03
*** annashen has quit IRC08:05
openstackgerritGiridhar Jayavelu proposed openstack/nova: Create vif with net_io resource allocation  https://review.openstack.org/19617108:05
*** annashen_ has quit IRC08:06
garykkashyap: ok, thanks08:08
*** lucas__ has joined #openstack-nova08:08
*** jistr has joined #openstack-nova08:08
openstackgerritGiridhar Jayavelu proposed openstack/nova: VMware: create vif with net_io resource allocation  https://review.openstack.org/19617108:11
*** lucas__ has quit IRC08:12
*** shuquan_ has joined #openstack-nova08:13
*** ndipanov has quit IRC08:14
openstackgerritsahid proposed openstack/nova: libvirt: add os_admin_user to use with set admin password  https://review.openstack.org/18592108:14
openstackgerritsahid proposed openstack/nova: libvirt: enhance libvirt to set admin password  https://review.openstack.org/18591008:14
openstackgerritsahid proposed openstack/nova: libvirt: rework quiesce to not share "sensitive" informations  https://review.openstack.org/19662008:14
*** ndipanov has joined #openstack-nova08:14
*** shuquan has quit IRC08:15
*** shuquan_ is now known as shuquan08:15
openstackgerritAlex Xu proposed openstack/nova: Skip additionalProperties checks when LegacyV2CompatibleWrapper enabled  https://review.openstack.org/19385808:15
openstackgerritAlex Xu proposed openstack/nova: Fix the wrong file name for legacy v2 compatible wrapper functional test  https://review.openstack.org/20632608:15
openstackgerritAlex Xu proposed openstack/nova: Add scenario for API sample tests with legacy v2 compatible wrapper  https://review.openstack.org/20348508:15
*** lucasagomes has joined #openstack-nova08:17
*** karimb has quit IRC08:17
*** dguitarbite has joined #openstack-nova08:18
deepakcsgaryk: the nova flavor limits the max size of swap and ephemeral disks, not sure if 1GB is valid for flavor eli ?08:18
garykdeepakcs: my understand in that with block devices we can create multiple ephemeral disks as long as the sum does not exceed that configured in the flavor.08:20
garykmaybe that has not been implemented yet.08:20
deepakcsgaryk: i don't see flavor eli in my local devstack setup, but all flavor's in my setup shows '0' against ephemeral column in `nova flavor-list`08:20
*** mpaolino has joined #openstack-nova08:20
kashyapIt's a custom flavor, probably.08:21
kashyapYeah, default swap size is '0'08:21
deepakcsgaryk: what is the ephermeral value for ur flavor in flavour-list ?08:21
garykdeepakcs: it is 208:22
deepakcsgaryk: then --size=2 should work, try that08:22
garykdeepakcs: yes, size = 2 and size =0 work.08:22
deepakcsgaryk: ah ok :) thats strange08:22
garyki just wanted to see if size =1 works08:22
openstackgerritAlex Xu proposed openstack/nova: Return v2 version info with v2 legacy compatible wrapper  https://review.openstack.org/20693008:22
deepakcsgaryk: ideally <=2 should work for size, iiuc08:23
garykdeepakcs: that was my understanding too. i need to debug.08:25
garykserenity now08:25
deepakcsgaryk: ok , interesting , i would like to know if u figure the root cuase :)08:25
*** jasondotstar has joined #openstack-nova08:25
garyksure, will notfiy you as soon as i find something08:25
*** bharath has joined #openstack-nova08:26
*** neiljerr` has joined #openstack-nova08:26
deepakcsgaryk: i haven't found enuf helpful doc on --ephemeral, is it documented anywhere.. the user-guide just tells how to use it, doesn't detail out the constraints/flavor relations08:26
bharathHi, may I know why are we creating a linux bridge every time when we launch an instance? I am using libvirt for nova and ovs as neutron plugin08:29
bharathwe are creating per vif every time, referring this code https://github.com/openstack/nova/blob/6b4146e944611685b593deabaa1d83d410e90318/nova/virt/libvirt/vif.py#L44508:30
garykdeepakcs: one of the many things that we need to try and improve08:31
kashyapgaryk: W/ current Git master, invoking this - nova boot --image cirros-0.3.3-x86_64-disk --flavor 2 --ephemeral size=2 vm108:32
kashyapthrows: "ERROR (ClientException): The server has either erred or is incapable of performing the requested operation"08:32
abhishekkndipanov: hi, https://review.openstack.org/#/c/192986/08:33
* kashyap first checks if it's his environment. . .08:33
ndipanovabhishekk, looking08:35
ndipanovyou have a merfe conflict08:35
*** jwcroppe has quit IRC08:35
abhishekkndipanov: yes I am pushing a patch for that08:35
abhishekkjust want to make sure should I remove assert statement in instancejobtracker.py?08:35
ndipanovabhishekk, yeah - I mean we don't normally have them in code08:36
abhishekkndipanov: ok then I will push the patch, thank you08:37
bauzasmarkus_z: heya08:39
bauzasmarkus_z: okay, you want to discuss about what ?08:39
bauzasmarkus_z: if it's a critical or not ?08:40
openstackgerritsahid proposed openstack/nova: libvirt: enhance method to return pointer_model from image prop  https://review.openstack.org/17485408:41
openstackgerritsahid proposed openstack/nova: libvirt: deprecate use_usb_tablet in favor of pointer_model  https://review.openstack.org/17624208:41
*** pixelbeat_ has joined #openstack-nova08:41
bauzasmarkus_z: yeah, it's a critical one hitting the cells job08:42
*** exploreshaifali has joined #openstack-nova08:42
*** markus_z has quit IRC08:43
*** yassine has joined #openstack-nova08:43
*** markus_z has joined #openstack-nova08:43
chhaviWith the latest nova I am getting the "ImportError: No module named storage"08:45
openstackgerritsahid proposed openstack/nova: libvirt: make host return guest information  https://review.openstack.org/19104608:45
openstackgerritsahid proposed openstack/nova: libvirt: make host return power state of guest  https://review.openstack.org/19104708:45
chhavistorage folders seems to be removed from the nova ?08:46
bauzasmarkus_z: oh, sec08:46
bauzasmarkus_z: I thought the related bug was related to CI, but it seems it doesn't08:47
bauzasmarkus_z: I'm saying that because most of the cells job runs are -1 now08:47
bauzaschecking08:47
kashyapgaryk: Hmm, strange, w/ current Nova Git, I can't even seem to boot instances -- n-api.log says it can't find an attribute 'id' -- http://paste.openstack.org/show/406178/08:48
openstackgerritAlexis Lee proposed openstack/nova: Adds resource classes  https://review.openstack.org/12899908:48
openstackgerritAlexis Lee proposed openstack/nova: Add object models for comparing resource amounts  https://review.openstack.org/12899208:48
* kashyap starts over with a new DevStack run.08:49
alex_xuoomichi: re: https://review.openstack.org/#/c/192043  this make the api have different behavior between nova network and neutron backend08:50
alex_xuoomichi: I'm not sure this what we want08:51
gmannsdague: johnthetubaguy : I updated lock info spec as per yesterday discussion- https://review.openstack.org/#/c/206864/08:51
deepakcskashyap: i am hitting the same error too (2015-07-29 08:49:05.915 TRACE nova.virt.libvirt.driver ClientException: The server has either erred or is incapable of performing the requested operation. (HTTP 500) (Request-ID: req-83be6665-2ed6-4662-b900-67d387edad9e)08:52
deepakcskashyap: as part of nova sending update _snapshot_status to Cinder08:53
kashyapdeepakcs: I'm trying to reproduce that issue08:53
*** ZZelle has quit IRC08:53
danpbl08:53
deepakcskashyap: I was able to create a nova VM successfully tho'08:53
danpbll~N08:53
*** ZZelle has joined #openstack-nova08:53
danpbhmm, that's not my mutt terminal window !08:54
bauzasokay, the CI was pretty in a bad shape yesterday night it seems...08:54
*** haypo has joined #openstack-nova08:54
*** ip0353 has joined #openstack-nova08:55
johnthetubaguygmann: cool, thanks for the update08:56
johnthetubaguygmann: I am curious though, what happens if its locked by a user, and an admin lists the instance, do they see locked? I guess thats OK08:57
garykkashyap: maybe pull them all…08:57
kashyapgaryk: :-)08:57
* kashyap just waiting for the darn `make stack` to finish. . .08:57
oomichialex_xu: do you mean nova needs to allow integer or other values as name/description for Neutron?08:58
bauzasouch ! http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiRVJST1I6IHRoZSBtYWluIHNldHVwIHNjcmlwdCBydW4gYnkgdGhpcyBqb2IgZmFpbGVkIC0gZXhpdCBjb2RlOiAxMzdcIiBhbmQgdGFnczpcImNvbnNvbGVcIiIsImZpZWxkcyI6W10sIm9mZnNldCI6MCwidGltZWZyYW1lIjoiODY0MDAiLCJncmFwaG1vZGUiOiJjb3VudCIsInRpbWUiOnsidXNlcl9pbnRlcnZhbCI6MH0sInN0YW1wIjoxNDM4MTYwMjc4NTM4fQ==08:58
*** ftersin has joined #openstack-nova08:58
garykkashyap: there are too many moving parts :). on the bright side there is the rugby world cup soon....08:59
alex_xuoomichi: I mean nova-network didn't allow None or not specific any value for name/description. That patch enable not specific any value for neutron08:59
*** doude has joined #openstack-nova09:00
kashyapgaryk: Joys of cloud09:00
alex_xuoomichi: so for consistent API behavior, we shouldn't allow not specific any value for name/descript with neutron, does make sense?09:02
openstackgerritAbhishek Kekane proposed openstack/nova: libvirt: Kill rsync/scp processes before deleting instance  https://review.openstack.org/19298609:02
*** Jeffrey4l has quit IRC09:02
*** jwcroppe has joined #openstack-nova09:02
*** markus_z has quit IRC09:02
*** annashen has joined #openstack-nova09:02
sahidhum this patch is i ready since Apr unfortunatly no core noticed it, ndipanov can you look at it please https://review.openstack.org/#/c/176360/09:03
sahidoomichi: ^09:03
johnthetubaguyalex_xu: oomichi: we need to talk about networking APIs at some point, in the long term we need a single (interoperable) network API, so you can't tell if its nova-network or neutron underneath, although it might be that certain API calls are not supported, depending on the driver or tech chosen09:03
openstackgerritDaniel Berrange proposed openstack/nova: Don't check flavor disk attributes when booting from volume  https://review.openstack.org/20463509:03
openstackgerritDaniel Berrange proposed openstack/nova: compute: improve exceptions related to disk size checks  https://review.openstack.org/20656509:03
oomichialex_xu: does that say nova should validate parameters on the same manner between nova-network and neutron, right?09:04
*** coolsvap is now known as coolsvap|away09:04
alex_xuoomichi: yes09:04
johnthetubaguysahid: I think it would be good to have a libvirt sub group to help push important bug fixes, in here, thats what other drivers are doing to try and make sure cores give attention to the appropriate patches, its not working yet, but thats the current push quite a few folks are signed up for: etherpad.openstack.org/p/liberty-nova-priorities-tracking09:05
alex_xuoomichi: same as johnthetubaguy as said09:05
oomichialex_xu: ideally we should do that, but actually some environments run on Neutron/nova-network. so it seems difficult to enfoce the validation manner to nova-network09:05
*** shyama has joined #openstack-nova09:06
johnthetubaguyoomichi: alex_xu: its possible there are differences, but it should be a discoverable capabilities difference, ideally09:06
johnthetubaguyoomichi: you mean some environments run both?09:06
bauzascorrect me if I'm wrong but isn't there any ugly if branches for knowing whether if it's Neutron or n-net ? :)09:06
oomichijohnthetubaguy: no to some environments run both09:06
chhaviWe were using the linuxscsi.echo_scsi_command in the nova, I cannot see this in the latest code,09:06
chhaviis there any other way to use this method09:07
johnthetubaguybauzas: well it should be hidden behind the network_api, ideally, but there are some. We need a consistent API experience, I know we are not doing that today :(09:07
*** annashen has quit IRC09:07
openstackgerritZhenguo Niu proposed openstack/nova: [Ironic]Not count available resources of deployed ironic node  https://review.openstack.org/20408809:07
bauzasjohnthetubaguy: that's my point09:07
oomichijohnthetubaguy: neutron allows to create security group without name, that is different from nova's one09:07
bauzasjohnthetubaguy: before moving further, we should have a good internal API for network being abstract09:07
johnthetubaguyoomichi: agreed, we need to enforce adding a name if you use the nova api though, to be consistent09:08
oomichijohnthetubaguy: I'm not sure we can enforce the feature is disabled at this time09:08
johnthetubaguyoomichi: once that security group add uuids, it should make that easier I thought09:08
johnthetubaguyoomichi: sorry, getting confused, do you mean turn off security groups API when using neutron?09:09
alex_xuoomichi: I think it's fine, with current code, if you without name, the nova return 500, it means nobody use nova security-group api like that09:09
johnthetubaguyalex_xu: it should be 400, but yes, it seems valid to reject requests without a name when using the Nova API09:10
alex_xuoomichi: but it's worth to check the old code, whether we return 500 all the time, all the 500 is some kind of regression09:10
*** liusheng has quit IRC09:10
shyamajohnthetubaguy: nova/storage folder is missing in master. have the files moved?09:10
*** liusheng has joined #openstack-nova09:11
alex_xujohnthetubaguy: yea09:11
oomichialex_xu: now nova returns 400 in this case because of https://review.openstack.org/#/c/189541/09:11
*** BobBall_AWOL is now known as BobBall09:11
johnthetubaguyshyama: I don't remember us having a nova/storage folder, if I am honest, what does github show for the older branches, does that answer your question at all?09:11
oomichialex_xu: yeah, anyway your point is right.09:12
*** coolsvap|away is now known as coolsvap09:12
kashyapgaryk: Sigh, I had a random Nova patch series in local.conf w/ variables NOVA_REPO & NOVA_BRANCH.  Re-testing w/o that.09:12
shyamathis one https://github.com/openstack/nova/blob/master/nova/storage/linuxscsi.py gives a 40409:12
sahidjohnthetubaguy: hum I see, it's good idea to highlight patches reads yes :)09:12
sahidjust thinking it seems difficult to keep this up to date, no09:12
oomichialex_xu: on current code, creating security group without name returns 400 always.09:12
danpbshyama: it was deleted yesterday09:13
shyamait is documented here http://docs.openstack.org/developer/nova/api/nova.storage.linuxscsi.html. How do i find when it was removed09:13
shyamadanpb: ok which is the commit?09:13
openstackgerritPetrut Lucian proposed openstack/nova: HyperV: Add IOUtils class for asynchronous IO operations  https://review.openstack.org/18403509:13
chhavidanpb: Is it moved somewhere else,09:14
johnthetubaguysahid: it is, but the idea is core or patch owner can strike out (or delete) stuff when the patch is merged, and the subteam can check back to keep the list full, hopefully its minimal overhead09:14
danpbcommit e6cdd1693ba843e1c8dcccbde3af20f62eb0b5a209:14
danpbAuthor: Walter A. Boring IV <walter.boring@hp.com>09:14
danpbDate:   Mon Apr 20 13:42:44 2015 -070009:14
danpb    Switch to using os-brick09:14
danpbis what deleted it09:14
openstackgerritPetrut Lucian proposed openstack/nova: HyperV: Add IOQueue class used for named pipe IO operations  https://review.openstack.org/18403609:15
alex_xuoomichi: that's good news...at least the problem become easy09:15
johnthetubaguysahid: it feels way better than cores having to look through all patches and guessing what is important for each subteam, and then missing out the really important stuff, at least it seems worth a try (this is what we started doing after the summit, and gets advertised in every nova-meeting, but I am trying to advertise it more places so more people notice it)09:15
sahidjohnthetubaguy: understand you want to make core looking at more than one area by this way ?09:17
lxslijohnthetubaguy: sahid: imho yes, it is a hassle - but it's way better than having important patches miss reviews09:17
*** dikonoor has joined #openstack-nova09:17
sahidi mean by this one, a core not too much related to libvirt can review it or something like that, right?09:17
*** gszasz has quit IRC09:18
chhavi linuxscsi.echo_scsi_command(scanpath, '- - -')09:18
*** eliqiao has quit IRC09:18
*** kodoku_ has quit IRC09:18
garykkashyap: deepakcs: i have found the issue - just need ndipanov to try and provide some clarrity here - basically the line https://github.com/openstack/nova/blame/master/nova/compute/api.py#L1222 is the culprit09:18
johnthetubaguysahid: not really, the idea is to give focus to what is reviewed, most cores should already be looking at many areas of the code, and generally they do09:18
*** markus_z has joined #openstack-nova09:19
sahidack09:19
johnthetubaguysahid: one idea, is that eventually the sub team recommending a patch *could* be trusted as a +2, so you only need one more core review, but it will take time to build up that level of trust, but that shouldn't stop us trying09:19
ndipanovgaryk, what up with it?09:20
*** eliqiao has joined #openstack-nova09:20
kashyapgaryk: Hmm. (Aside: I didn't know GitHub provides a convenient `git blame` style web interface.)09:20
*** arunman has joined #openstack-nova09:20
sahidjohnthetubaguy: sure yes09:20
kashyapgaryk: Speaking of docs, ndipanov recently wrote some block device related documentation.  Not sure if you came across.09:20
ndipanovkashyap, not merged yet sadly09:21
kashyapndipanov: Due to the recent Gate issues?  Or general sluggishness?09:21
ndipanovbut through my own fault - haven't addressed the comments09:21
*** alexpilotti has joined #openstack-nova09:21
garykkashyap: i am not sure where he put them. he is mia at the moment…09:22
kashyapgaryk: MIA?  He's right here, talking :-)09:22
garykndipanov: ah. you around09:22
johnthetubaguysahid: I missed an important point, the idea is not just for core reviews, the idea is that all reviewers start looking for reviews in that etherpad, ideally.09:22
ndipanovyes09:23
ndipanovwhy wouldn't I be09:23
garykndipanov: i have an issue with bdms - want to boot a instance with an eph disk09:23
kashyapgaryk: https://review.openstack.org/#/c/192651/09:23
ndipanovok09:23
garykndipanov:  nova boot --image cirros-0.3.2-i386-disk  --flavor eli --ephemeral size=1 X09:23
garykERROR (BadRequest): Block Device Mapping is Invalid: Boot sequence for the instance and image/block device mapping combination is not valid. (HTTP 400) (Request-ID: req-230ebb8e-0aa6-435c-8b2f-45843ad7e1cb)09:23
garykthe flavor has eph disk size 2 - so this is legit09:24
garykthe code that breaks this is - https://github.com/openstack/nova/blame/master/nova/compute/api.py#L122209:24
ndipanovwhat's the actual cli line you used?09:24
garyknova boot --image cirros-0.3.2-i386-disk  --flavor eli --ephemeral size=1 X09:24
ndipanovhmmm09:24
ndipanovthat should not blow up09:24
garykwhen i delete the line above from the git blame it works09:25
ndipanovobviously09:25
garykwhy is the boot index of interst here at all?09:25
ndipanovhmmm09:25
ndipanovwell it shouldn't be09:26
ndipanovunless you have bdms in the image09:26
garykthe boot_indexes is []09:26
garykno09:26
ndipanovORLY?09:26
ndipanovgaryk, is this devstack or?09:27
garykyes09:27
garykdevstack with the vmware driver - but it does not even get to the driver. so you cannot blame vmware :)09:27
ndipanovand python-novaclent is also recent09:28
garyksec09:28
garyk2.2309:28
openstackgerritPetrut Lucian proposed openstack/nova: HyperV: Add IOQueue class used for named pipe IO operations  https://review.openstack.org/18403609:28
garykndipanov: should be kosher for stable/kilo09:28
ndipanovah so you are running stable/kilo?09:29
*** madhaviy has joined #openstack-nova09:29
-openstackstatus- NOTICE: Currently our CI system is broken, jobs are not getting processed at all.09:29
garyki am running stable/kilo. ack09:29
*** ChanServ changes topic to "Currently our CI system is broken, jobs are not getting processed at all."09:29
garykif you want i can run on trunk. but it will be the same.09:29
ndipanovpls try master09:29
oomichigrayk: I can run it on master devstack09:30
oomichigrayk: $ nova flavor-create eph10 100 512 0 1 --ephemeral 1009:30
oomichigrayk: $ nova boot --image cirros-0.3.4-x86_64-uec  --flavor eph10 --ephemeral size=1 X09:30
oomichigrayk: that was succeeded09:30
garykok, will do.09:30
madhaviyHi there,  I have query in /nova/compute/manager/_shutdown_instance#L2246  ,during shutdown volume detach is done directly without check_detach, is there any reason behind09:33
johnthetubaguymadhaviy: I guess the detach happens after the instance is shutdown? I suspect it was assumed it was always possible at that point, but I am not 100% sure thats why its done that way09:36
*** humble_ has quit IRC09:36
johnthetubaguymadhaviy: if we couldn't do the detach at that point, I am not 100% sure when we would do the detach, not sure if there is a clean up loop for that, for example09:37
*** romainh has left #openstack-nova09:37
*** shuquan has quit IRC09:37
kashyapoomichi: ndipanov: Hmm, w/ Nova Git master, I could consistently reproduce this AttributeError "id" -- http://paste.openstack.org/show/406182/09:38
kashyapWonder what am I missing there.09:38
*** romainh has joined #openstack-nova09:39
madhaviyjohnthetubaguy: _shutdown_instance its called during deploy failure also, if volume is not attached completely, say its in creating/attaching state still, then I feel detach directly may not work09:40
johnthetubaguymadhaviy: yes, thats true, it should be more defensive, I assumed we just ignored the error response for "already detached", but I didn't check that09:41
openstackgerritPetrut Lucian proposed openstack/nova: HyperV: Add named pipe handler  https://review.openstack.org/18403709:44
ndipanovkashyap, that seems to be a glance client issue09:45
*** otter768 has joined #openstack-nova09:45
ndipanovit may be too old or smth (or an actual bug in it)09:46
openstackgerritPetrut Lucian proposed openstack/nova: HyperV: Add serial console proxy  https://review.openstack.org/18403809:46
*** markus_z has quit IRC09:46
openstackgerritPetrut Lucian proposed openstack/nova: HyperV: Add serial console handler class  https://review.openstack.org/18403909:46
kashyapndipanov: This is Glance from Git master.  It can't be newer than this.09:46
kashyapI'll file a bug09:46
openstackgerritPetrut Lucian proposed openstack/nova: HyperV: Add SerialConsoleOps class  https://review.openstack.org/18404009:46
ndipanovglancelient09:46
*** mdbooth has quit IRC09:46
ndipanovnot Glance09:46
ndipanovbut yes it might as well be a bug09:47
kashyapndipanov: Err, jumped the gun too soon.09:47
*** mdbooth has joined #openstack-nova09:47
*** alex_xu has quit IRC09:48
* kashyap tries with the unified `openstack` CLI09:48
garykoomichi: ndipanov: i have some glance issues - so still try to check :(09:48
*** alex_xu has joined #openstack-nova09:49
*** otter768 has quit IRC09:50
*** coolsvap is now known as coolsvap|away09:50
madhaviyjohnthetubaguy: so do you think adding check_detach() before detach will have more proper checks.09:53
*** eliqiao1 has joined #openstack-nova09:54
johnthetubaguymadhaviy: unsure, it feels like overkill, is there a specific problem you are seeing?09:54
*** doude has quit IRC09:55
kashyapndipanov: Hmm, that's the version I was using btw -- 0.19.0.  PyPI mirror says 0.17.2 is the current version.  /me downgrades and tests.09:56
openstackgerritJohn Garbutt proposed openstack/nova: docs: add a placeholder link to mentoring docs  https://review.openstack.org/20005309:56
*** eliqiao has quit IRC09:56
openstackgerritJohn Garbutt proposed openstack/nova: docs: split up upgrade and upgrade futures  https://review.openstack.org/20152909:56
madhaviyjohnthetubaguy: I saw an exception throw, I will get stack trace an dmay be raise a bug?09:57
kashyapgaryk: What version of python-glanceclient are you using?09:57
*** jasondotstar has quit IRC09:57
openstackgerritLucas Alvares Gomes proposed openstack/nova: Ironic: Better handle InstanceNotFound on destroy()  https://review.openstack.org/20499509:57
*** shyama has quit IRC09:58
*** coolsvap|away is now known as coolsvap09:58
openstackgerritJohn Garbutt proposed openstack/nova: Add a summary of the architecture evolution work  https://review.openstack.org/20000109:58
openstackgerritJohn Garbutt proposed openstack/nova: docs: add the scheduler evolution plans  https://review.openstack.org/19226009:59
*** yassine has quit IRC09:59
openstackgerritJohn Garbutt proposed openstack/nova: devref: update the nova architecture doc  https://review.openstack.org/17737409:59
*** madhaviy_ has joined #openstack-nova10:00
*** jwcroppe has quit IRC10:00
*** madhaviy_ has quit IRC10:00
*** alex_klimov has quit IRC10:00
*** gzhai has quit IRC10:00
*** zzz_pczesno has quit IRC10:00
*** garyk has quit IRC10:00
*** ijuwang has quit IRC10:00
*** Kennan has quit IRC10:00
*** jwang has quit IRC10:00
*** melwitt has quit IRC10:00
*** mjturek1 has quit IRC10:00
*** mitz has quit IRC10:00
*** jogo has quit IRC10:00
*** sc68cal has quit IRC10:00
*** mitz has joined #openstack-nova10:00
*** park_heijlong_ has quit IRC10:00
*** melwitt has joined #openstack-nova10:00
*** garyk has joined #openstack-nova10:00
*** jwang has joined #openstack-nova10:01
*** alex_klimov has joined #openstack-nova10:01
*** Kennan has joined #openstack-nova10:01
*** ijuwang has joined #openstack-nova10:01
*** abhishekkk has joined #openstack-nova10:01
*** madhaviy_ has joined #openstack-nova10:01
*** mjturek1 has joined #openstack-nova10:01
*** madhaviy has quit IRC10:01
*** gszasz has joined #openstack-nova10:01
*** kmARC has quit IRC10:01
*** annashen has joined #openstack-nova10:02
*** sc68cal has joined #openstack-nova10:02
*** kiran-r has quit IRC10:03
garykkashyap: glance (11.0.0.0b2, /opt/stack/glance)10:04
garykglance-store (0.7.1)10:04
garykpython-glanceclient (0.19.0)10:04
*** annashen_ has joined #openstack-nova10:04
garykkashyap: ERROR: Could not bind to 0.0.0.0:9191 after trying for 30 seconds10:04
garykreboot10:04
*** madhaviy has joined #openstack-nova10:04
kashyapgaryk: Likewise here (about the versions).  They seem buggy to me.10:04
*** sahid_ has joined #openstack-nova10:05
*** clduser_ has joined #openstack-nova10:05
*** sahid_ has quit IRC10:05
*** rushiagr_away has quit IRC10:05
*** jmccrory has quit IRC10:05
*** sahid has quit IRC10:05
*** abhishekk has quit IRC10:05
*** clduser has quit IRC10:05
*** ildikov has quit IRC10:05
*** Piet has quit IRC10:05
*** kaisers has quit IRC10:05
*** pczesno has joined #openstack-nova10:05
*** pczesno has quit IRC10:05
*** pczesno has joined #openstack-nova10:05
*** ildikov has joined #openstack-nova10:05
*** madhaviy_ has quit IRC10:05
*** sahid_ has joined #openstack-nova10:05
*** sahid_ is now known as sahid10:06
*** jogo has joined #openstack-nova10:06
*** annashen has quit IRC10:06
*** kaisers has joined #openstack-nova10:06
*** jmccrory has joined #openstack-nova10:07
*** rdo has quit IRC10:08
*** annashen_ has quit IRC10:08
*** jasondotstar has joined #openstack-nova10:09
arunmansdeague: Hi10:10
*** rdo has joined #openstack-nova10:10
arunmansdeague: I need some clarification on this bug-1478546, that you changed to 'Invalid'10:11
*** kiran-r has joined #openstack-nova10:12
arunmansdeague: I meant, this https://bugs.launchpad.net/bugs/1478546  got marked as invalid, asking for a BP10:13
openstackLaunchpad bug 1478546 in OpenStack Compute (nova) "nova GMR doesn't provide option to specify log_dir path" [Undecided,Invalid] - Assigned to Arun Mani (arun-mani)10:13
*** zhenguo has quit IRC10:13
arunmanI'm not really sure if this requires to be fixed by submitting a BP10:14
danpbarunman: the response there is saying that this is a feature request, rather than a bug report, so you need to use our feature process to submit the suggestion instead10:14
lxsliarunman: you want sdague and he's US-based so probably still asleep10:15
*** josecastroleon has quit IRC10:15
openstackgerritPetrut Lucian proposed openstack/nova: HyperV: Nova serial console access support  https://review.openstack.org/14500410:15
arunmandanpb: jus wondering how this can be considered as a feature10:15
lxsliyou want to add an extra option10:16
*** kmARC has joined #openstack-nova10:16
*** doude has joined #openstack-nova10:18
arunmanlxsli: Alrite, if BP is the only way of getting this fixed... I'll push it that way10:18
openstackgerritAndrea Rosa proposed openstack/nova: Raise nova exception for missing/empty machine-id  https://review.openstack.org/20264010:19
danpbnb, it does not need to have a spec though - merely a blueprint is sufficient imho10:19
*** romainh has quit IRC10:20
lxsli+110:23
openstackgerritMatthew Booth proposed openstack/nova: Implement compare-and-swap for instance update  https://review.openstack.org/20259310:23
lxslibauzas: good morning!10:23
bauzaslxsli: good noon !10:23
lxslibauzas: I can't find any persist-request-spec patches apart from the spec; has alaski pushed anything yet?10:23
bauzaslxsli: none I've heard of10:24
bauzaslxsli: he was testing locally10:24
lxslibauzas: that explains that then :)10:24
bauzaslxsli: well, you can begin working on that without needing to get the persisted Spec10:24
bauzaslxsli: but you would still need my top level patch10:25
lxslibauzas: yep I'll see what I can do10:25
lxslibauzas: did you see I asked a couple of qs on https://review.openstack.org/#/c/179224/16/specs/liberty/approved/check-destination-on-migrations.rst ?10:25
bauzaslxsli: remember what I've said during the midcycle, that's the main problem with delegating that one10:25
lxslibauzas: yep rebases will be plentiful10:26
bauzaslxsli: mmm, I don't get your first point10:26
bauzaslxsli: why a list ?10:27
bauzaslxsli: here the user gives a destination10:27
lxslibauzas: a user might want to allow some choice though; they can't do that right now through livemigr but maybe later10:27
bauzaslxsli: then it should amend the ReqSpec object later10:27
bauzaslxsli: for the moment, the API allows to provide *one* destination10:28
ndipanovdanpb, https://review.openstack.org/#/c/192986/ <- dat security fix10:28
lxslibauzas: it just seems like a trivial extension with high potential10:28
bauzaslxsli: not really, seems rather confusing10:28
lxslibauzas: not sure how it can be confusing but ok10:29
bauzaslxsli: here, we amend the Spec based on what's given by the user10:29
ndipanovsdague, can we land this maybe https://review.openstack.org/#/c/206249/3 ?10:29
bauzaslxsli: since the user is only allowed to provide one destination, then we should provide one item10:29
openstackgerritAndrea Rosa proposed openstack/nova: Raise nova exception for missing/empty machine-id  https://review.openstack.org/20264010:29
ndipanovdon't want to just +A it but seems to me the naming is fine10:29
lxsliwhat if the user were allowed to provide more than one?10:30
bauzaslxsli: re: the --force stuff, it's for keeping the exact existing behaviour10:30
*** shyama has joined #openstack-nova10:31
bauzaslxsli: if migrate-live or evacuate JSON bodies would accept more than a string, then we would update the Spec object to accept a ComputeNodeList for requested_destination10:31
*** xgsa has quit IRC10:33
lxslibauzas: ok, let's do it like this for now10:34
lxslibauzas: I can write my accept-multiple-force-hosts spec later :)10:34
openstackgerritMarian Horban proposed openstack/nova: libvirt:Rsync remote FS driver was added  https://review.openstack.org/14688710:35
bauzaslxsli: honestly, I'd rather be -1 to that10:35
bauzaslxsli: because we have aggregates10:35
lxslibauzas: what about my 2nd q?10:35
sdaguendipanov: ok, sure. It still seems like a bad name to me.10:35
sdaguebut if no one else agrees...10:35
bauzaslxsli: the --force option is just an alleviated way to act in urgency10:36
lxslioh hey, thanks ndipanov + sdague :)10:36
ndipanovwell what else can we call it? positive_but_zero's_cool_too?10:36
ndipanov:)10:36
ndipanovunsigned really has different connotations...10:37
bauzaslxsli: I answered to that by saying (12:30:53) bauzas: lxsli: re: the --force stuff, it's for keeping the exact existing behaviour10:38
bauzaslxsli: which means I don't want to change that by calling the scheduler10:38
*** shyu has quit IRC10:38
*** aix has quit IRC10:38
lxslibauzas: ok, I can accept an argument for not changing too much at once10:39
*** exploreshaifali_ has joined #openstack-nova10:39
lxslibauzas: do you agree we'll have to change this when/if we shift source-of-truth to scheduler?10:40
*** exploreshaifali has quit IRC10:43
bauzaslxsli: say what ?10:43
bauzassource-of-truth of what ?10:43
openstackgerritAndrea Rosa proposed openstack/nova: Raise nova exception for missing/empty machine-id  https://review.openstack.org/20264010:44
lxslisorry, that was argumentative10:44
*** dikonoor has quit IRC10:44
lxsliaiui we'd like the scheduler to be able to cache hoststate10:44
bauzaslxsli: now we invert the logic by calling the scheduler anyway when an host is given by the user10:44
bauzaslxsli: unless the user explicitely makes use of the force flag10:44
*** arunman has left #openstack-nova10:44
lxsliif instances can be forced behind the scheduler's back, the hoststate can never be fully accurate10:44
bauzaslxsli: going out to lunch10:45
bauzaslxsli: but that's life10:45
lxsliok chat later10:45
bauzaslxsli: if an user breaks the rules by forcing to a dest, then fer sur, it will raise some problems10:45
bauzasbut the host states are given by RT10:45
bauzasso, please don't overthink10:46
danpbndipanov: what about it ?10:46
lxslicall me crazy, I like to think ahead :)10:47
ndipanovdanpb, can you +2 it pls if you don't find something horribly wrong10:48
ndipanovwe really should land that fix soon10:48
kashyapgaryk: Okay, I filed it, since I was able to reproduce it twice at-least, asked the Glance folks if it legitmate, let's see what they say -- https://bugs.launchpad.net/glance/+bug/147929610:48
openstackLaunchpad bug 1479296 in Glance "On Nova instance boot, 'glanceclient/openstack/common/apiclient/base.py' throws: "AttributeError: id"" [Undecided,New]10:48
chhavidanpb: As u mentioned that nova/storage/linuxscsi.py is completely been removed by os-brick10:49
*** lennyb_ has quit IRC10:49
danpbndipanov: ah ok10:49
chhavidanpb: Now what is the preferred way to make the linuxscsi commands10:49
*** xgsa has joined #openstack-nova10:49
danpbchhavi: don't make scsi commands in nova - do it in os-brick10:49
garykkashyap: thanks10:50
chhavidanpb: Earlier we were making echo_scsi_command() in the nova drivers. Now do we need to make use of the API exposed by os-brick10:51
danpbchhavi: what exactly are you trying todo10:52
dimsdanpb: sdague: ndipanov: other nova cores who are awake :) - updated "Currently reviewable" bug list in https://etherpad.openstack.org/p/liberty-nova-priorities-tracking - we merged one in that list since yesterday10:52
*** markus_z has joined #openstack-nova10:53
garykndipanov: can you please look at https://review.openstack.org/#/c/18878910:54
chhavidanpb: I am looking for an alternative for linuxscsi.echo_scsi_command()10:55
garykjohnthetubaguy: i addressed your comment on the vnic patch - https://review.openstack.org/14712610:55
*** coolsvap is now known as coolsvap|away10:55
garykstill working on the metadata10:55
danpbchhavi: but for what purpose ?10:56
chhaviWe have been using it to scan the path to identify a scsi device10:57
dimsthanks danpb!10:58
danpbchhavi: using it from where though ? what source file / code in nova ?10:59
*** shaohe_robot has quit IRC11:00
*** mpaolino has quit IRC11:00
ndipanovgaryk, yeah it's on my (very long) list of things to look at :/11:01
*** shaohe_robot has joined #openstack-nova11:01
garykndipanov: how about moving it to the top of the list :)11:02
*** yassine has joined #openstack-nova11:04
*** annashen has joined #openstack-nova11:04
*** yassine has quit IRC11:04
*** yassine has joined #openstack-nova11:04
*** annashen has quit IRC11:09
johnthetubaguygaryk: sweet, I think I going to get some lunch right now, but will try revisit those after lunch11:12
danpbndipanov: johnthetubaguy  sdague  could one of you see if you think this single patch can be +A'd - this one patch is holding up 3 different people's patch series https://review.openstack.org/#/c/188789/1511:12
garykbon apetite11:13
johnthetubaguydanpb: its on my list, I did a first read the other day, (mostly due to it blocking those dependent patches)11:13
danpbboth myelf & jaypipes think the patch is ok, but since we've now both been involved in writing it, it doesn't feel right for us to +A it ourselves11:13
*** romainh has joined #openstack-nova11:14
johnthetubaguydanpb: ah, totally, let me take another look11:14
johnthetubaguydanpb: got out of date with that bit of the code after having to focus on specs, so having more ramp up time in there than I had hoped, it looked good, just hadn't totally convinced myself yet11:14
johnthetubaguydanpb: ah, this was the bit I wasn't sure about: https://review.openstack.org/#/c/188789/15/nova/compute/resource_tracker.py,cm11:15
johnthetubaguydanpb: I was wondering if we should copy the pattern on L246 on 667?11:16
danpbsorry, i gave you the link to the old patch version11:16
danpbthat's fixed in patchset 1811:16
johnthetubaguydanpb: ah, damm it, I keep doing that, looking at the old one11:17
danpbyeah i often make that mistake as i have 100's of firefox tabs open with reviews11:18
garykoomichi: ndipanov deepakcs kashyap: on latest master the — ephemeral size=1 works. i think that the issue is the nova client.11:18
oomichigaryk: that is a good news :)11:20
garykoomichi: yes, thank you.11:20
sdaguedanpb: seems fine to me. That being said, zuul is down, so nothing is going to flow for a bit11:21
ftersingaryk: check your novaclient for https://github.com/openstack/python-novaclient/commit/8679eedb8352630012202c12a9c9acf8757802a511:22
*** jasondot_ has joined #openstack-nova11:23
garykftersin: thanks!11:23
*** baoli has joined #openstack-nova11:24
*** deepthi has quit IRC11:24
openstackgerritGábor Takács proposed openstack/nova: Refactor os-services unit tests  https://review.openstack.org/18086911:25
*** jasondotstar has quit IRC11:25
openstackgerritPranali Deore proposed openstack/nova: Validate quota class_name  https://review.openstack.org/19211511:27
deepakcsgaryk: thanks for following up11:27
*** baoli has quit IRC11:31
*** belmoreira has joined #openstack-nova11:31
*** lucasagomes is now known as lucas-hungry11:33
openstackgerritDaniel Berrange proposed openstack/nova: Introduce VNC Security Proxy Framework  https://review.openstack.org/11548311:33
openstackgerritDaniel Berrange proposed openstack/nova: Add VeNCrypt (TLS/x509) Security Proxy Driver  https://review.openstack.org/11548411:33
*** mwagner_lap has quit IRC11:35
*** oomichi has quit IRC11:36
*** rushiagr_away has joined #openstack-nova11:37
*** madhaviy has quit IRC11:38
*** sahid has quit IRC11:38
*** madhaviy has joined #openstack-nova11:38
openstackgerritShiina, Hironori proposed openstack/nova-specs: Fix spec: Inject NMI to an instance  https://review.openstack.org/20700411:39
openstackgerritDaniel Berrange proposed openstack/nova-specs: Blueprint for Websockify security proxy framework  https://review.openstack.org/20700611:40
*** shyama has quit IRC11:46
*** otter768 has joined #openstack-nova11:46
*** cpschult has joined #openstack-nova11:47
*** jwcroppe has joined #openstack-nova11:47
*** kodoku_ has joined #openstack-nova11:47
*** thorst has joined #openstack-nova11:47
*** jwcroppe has quit IRC11:48
*** fawadkhaliq has quit IRC11:48
*** otter768 has quit IRC11:51
*** veena has joined #openstack-nova11:51
*** coolsvap|away is now known as coolsvap11:51
openstackgerritShiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI  https://review.openstack.org/20261711:52
*** cpschult has quit IRC11:53
*** shaohe_robot1 has joined #openstack-nova11:54
*** josecastroleon has joined #openstack-nova11:55
*** fawadkhaliq has joined #openstack-nova11:55
*** shaohe_robot has quit IRC11:57
*** annashen has joined #openstack-nova12:02
*** annashen_ has joined #openstack-nova12:05
*** aix has joined #openstack-nova12:07
*** annashen has quit IRC12:07
*** exploreshaifali_ has quit IRC12:07
*** annashen_ has quit IRC12:10
*** erlon has joined #openstack-nova12:16
*** shaohe_robot has joined #openstack-nova12:16
*** ByPasS has joined #openstack-nova12:16
*** marcusvrn has joined #openstack-nova12:16
*** shaohe_robot1 has quit IRC12:17
*** coolsvap is now known as coolsvap|away12:17
lyanchihHi, guys. please help to review our patch   https://review.openstack.org/#/c/202436/12:17
*** kiran-r has quit IRC12:18
*** vladikr has joined #openstack-nova12:18
*** kodoku_ has quit IRC12:19
*** salv-orl_ has joined #openstack-nova12:22
openstackgerritNikolas Hermanns proposed openstack/nova: libvirt: Add post_connection_terminated  https://review.openstack.org/17234112:22
*** oomichi has joined #openstack-nova12:23
*** xek_ is now known as xek12:23
*** salv-orlando has quit IRC12:25
oomichialex_xu: can you take a look at https://review.openstack.org/#/c/192043/10 again?12:25
oomichialex_xu: it seems difficult to make validation behavior completely same between nova-network and neutron at this time.12:25
*** salv-orlando has joined #openstack-nova12:26
alex_xuoomichi: let me check12:26
*** salv-orl_ has quit IRC12:26
*** edmondsw has joined #openstack-nova12:27
*** salv-orl_ has joined #openstack-nova12:27
*** madhaviy has quit IRC12:28
*** salv-orlando has quit IRC12:30
alex_xuoomichi: yea, that is sad, looks like we can't fix it now12:30
*** ajayaa has quit IRC12:30
*** enikher has joined #openstack-nova12:31
*** mjturek1 has quit IRC12:31
alex_xuoomichi: so for now, the fix should be no value specific return 400. The value is empty string have different behavior12:32
alex_xuoomichi: does make sense?12:32
*** kylek3h has joined #openstack-nova12:33
*** deepakcs has quit IRC12:33
scheurandanpb, could you please verify, if your -1 on https://review.openstack.org/#/c/182283 is still valid?12:34
*** salv-orlando has joined #openstack-nova12:35
scheurandanpb, I answered in the review to your comment12:35
*** ishant has quit IRC12:36
oomichialex_xu: the idea is for making the validation behavior the same as possible, right?12:36
alex_xuoomichi: yea12:37
oomichialex_xu: that seems reasonable, and I'd like to consider more12:37
*** aysyd has joined #openstack-nova12:38
alex_xuoomichi: I can image the json-schema of security-group will be hard-work :(12:38
*** salv-orl_ has quit IRC12:38
*** mpaolino has joined #openstack-nova12:38
*** lucas-hungry is now known as lucasagomes12:38
oomichialex_xu: yeah, completely right. and that was the consensus between us in kilo :-(12:39
*** mpavone has quit IRC12:39
oomichialex_xu: but I forgot it and tried to do it..12:39
alex_xuoomichi: :)12:39
openstackgerritTiago Mello proposed openstack/nova: Handle unsupported Spice feature for PowerKVM  https://review.openstack.org/20380912:43
*** mdrabe has joined #openstack-nova12:44
*** salv-orl_ has joined #openstack-nova12:44
*** mjturek1 has joined #openstack-nova12:46
*** salv-orlando has quit IRC12:47
*** VW_ has joined #openstack-nova12:47
*** mpavone has joined #openstack-nova12:47
*** salv-orl_ has quit IRC12:47
*** coolsvap|away is now known as coolsvap12:48
*** salv-orlando has joined #openstack-nova12:48
*** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. -=- master is open for Lemming development"12:50
-openstackstatus- NOTICE: zuul's disks were at capacity. Space has been freed up and jobs are being re-queued.12:50
openstackgerritRoman Podoliaka proposed openstack/nova: rebuild: make sure server is shut down before volumes are detached  https://review.openstack.org/17689212:52
openstackgerritShiina, Hironori proposed openstack/nova-specs: Fix spec: Inject NMI to an instance  https://review.openstack.org/20700412:52
*** VW_ has quit IRC12:52
*** baoli has joined #openstack-nova12:52
*** VW_ has joined #openstack-nova12:53
*** shyama has joined #openstack-nova12:53
openstackgerritShiina, Hironori proposed openstack/nova: Add a Nova API to inject an NMI  https://review.openstack.org/20261712:54
*** rfolco has joined #openstack-nova12:54
*** dims has quit IRC12:57
*** VW_ has quit IRC12:57
*** chuckcarmack has joined #openstack-nova12:57
*** rlrossit has joined #openstack-nova12:57
*** salv-orl_ has joined #openstack-nova12:57
*** annegentle has joined #openstack-nova12:57
*** dims has joined #openstack-nova12:57
*** mriedem has joined #openstack-nova12:57
*** veena has quit IRC12:58
*** salv-orlando has quit IRC13:00
*** salv-or__ has joined #openstack-nova13:00
*** scheuran has quit IRC13:02
*** scheuran has joined #openstack-nova13:02
openstackgerritKen'ichi Ohmichi proposed openstack/nova-specs: Fix bodies on consolidate-console-api  https://review.openstack.org/20702913:03
*** salv-orl_ has quit IRC13:04
*** annashen has joined #openstack-nova13:06
*** eglynn has joined #openstack-nova13:07
*** oomichi has quit IRC13:08
rlrossitdansmith: am I going crazy or are there no screen logs in this? http://logs.openstack.org/04/198904/6/check/gate-grenade-dsvm-partial-ncpu/afddd41/logs/13:08
*** salv-or__ has quit IRC13:09
*** annashen has quit IRC13:10
mriedemrlrossit: see the old and new subdirs13:10
*** salv-orlando has joined #openstack-nova13:11
mriedemrlrossit: there are no screen logs because devstack blew up on the kilo side during setup http://logs.openstack.org/04/198904/6/check/gate-grenade-dsvm-partial-ncpu/afddd41/logs/old/devstacklog.txt.gz#_2015-07-29_01_28_04_55713:11
mriedemrecheck that one13:11
rlrossitfdshqiuonewcosiueorfasdvfnloeqorive13:11
rlrossitugh13:11
*** phschwartz has quit IRC13:11
rlrossitis it friday yet?13:11
mriedemactually hold on13:11
mriedemhttps://review.openstack.org/#/c/206676/13:12
mriedemkilo will be blocked until https://review.openstack.org/#/q/I180de9d4f1f4ae385682ce40a3a695eb3acc4c43,n,z are merged13:12
*** phschwartz has joined #openstack-nova13:12
rlrossitmriedem: so don't recheck?13:12
mriedemno13:12
mriedemnot yet13:12
*** dboik has joined #openstack-nova13:12
*** dave-mccowan has joined #openstack-nova13:13
rlrossityeah I saw both changes in that chain didn't work13:13
*** coolsvap is now known as coolsvap|away13:14
*** salv-orl_ has joined #openstack-nova13:14
mnestratovmriedem: hi! could you please check if your -1 is still valid https://review.openstack.org/#/c/184153/13:16
*** sahid has joined #openstack-nova13:17
*** salv-orlando has quit IRC13:17
MuraliHi13:17
*** salv-orlando has joined #openstack-nova13:18
mriedemmnestratov: done13:18
MuraliCould any body help me. I am using the "{tenant_id}​/servers/​{server_id}​/action" REST API to reboot the instance13:18
*** rfolco has quit IRC13:18
*** rlrossit has left #openstack-nova13:19
MuraliI am getting the 404 Not Found for existing instance13:19
mnestratovmriedem: thanks13:19
*** lucas__ has joined #openstack-nova13:19
Muralihere is " curl -s -X POST -H "X-Auth-Token:a0872bc3deef4c258530df4b923e2454" -H  "Content-Type: application/json" @reboot.json http://192.168.7.73:8774/v2/db688ec0a0d844e781203329c4aa1f47/servers/77efd837-bad6-4717-93d1-63b552af64e0/reboot"13:20
mriedemMurali: see the channel topic,13:21
mriedemthis isn't a support channel13:21
*** salv-orl_ has quit IRC13:21
Muralimriedem:sorry if  interrupt any meeting13:22
openstackgerritgaryk proposed openstack/nova: Metadata: support proxying loadbalancers  https://review.openstack.org/14753913:22
openstackgerritgaryk proposed openstack/nova: Neutron: add in API method for updating VNIC index  https://review.openstack.org/14712613:22
garykjohnthetubaguy: ^ following your comments13:22
*** tpeoples has joined #openstack-nova13:22
*** eharney has joined #openstack-nova13:23
*** jswarren has joined #openstack-nova13:23
*** VW_ has joined #openstack-nova13:23
-openstackstatus- NOTICE: zuul jobs after about 07:00 UTC may need a 'recheck' to enter the queue. Look if your change is in http://status.openstack.org/zuul/ and recheck if not.13:26
*** sahid has quit IRC13:28
*** sahid has joined #openstack-nova13:28
openstackgerritAlexis Lee proposed openstack/nova: Prevent (un)pinning unknown CPUs  https://review.openstack.org/20704013:28
*** sahid has quit IRC13:28
*** sahid has joined #openstack-nova13:29
haypodims, johnthetubaguy : hi. i have a patch to reduce the technical debt of nova, https://review.openstack.org/#/c/197601/ - it prepares nova to kill the deprecation warnings on strtime() (i get a ton of such warnings when running nova tests)13:29
*** tpeoples has quit IRC13:29
haypo(i added it to https://blueprints.launchpad.net/nova/+spec/liberty-olso-sync)13:29
*** rfolco has joined #openstack-nova13:31
*** jecarey has joined #openstack-nova13:32
lxslibauzas: ndipanov makes a very good point on 128992. I don't think it applies to can_provide, but it does to consume. Does it sound sensible to you for consume to return the claimed resources?13:32
lxsliI'd check with jaypipes but he's not even in channel13:32
bauzaslxsli: I need some time before replying13:33
lxsliOK13:33
bauzaslxsli: because it needs me to consider all of that13:33
bauzaslxsli: that really depends on who the caller is13:34
lxslibauzas: well, once self.topology is updated and the original discarded, the information is lost13:34
*** annegentle has quit IRC13:36
*** rlrossit has joined #openstack-nova13:37
*** raildo has joined #openstack-nova13:38
*** lucas__ has quit IRC13:38
*** READ10 has joined #openstack-nova13:39
*** lucas__ has joined #openstack-nova13:39
*** abhishekkk has quit IRC13:39
*** ctrath has joined #openstack-nova13:42
*** nelsnelson has joined #openstack-nova13:42
lxslibauzas: another q sorry. How are we protected from concurrent calls to #consume?13:43
*** Nikolay_St has quit IRC13:43
*** jwcroppe has joined #openstack-nova13:43
*** cpschult has joined #openstack-nova13:43
*** Nikolay_St has joined #openstack-nova13:43
*** jordanP has joined #openstack-nova13:43
*** romainh has left #openstack-nova13:44
*** markvoelker has joined #openstack-nova13:44
*** otter768 has joined #openstack-nova13:47
markus_zmikal: I'd like to inform you about bug 1479181. Would you see it as "critical" too?13:48
openstackbug 1479181 in OpenStack Compute (nova) "Cells: Build instance doesn't work with kilo api, juno compute " [Critical,New] https://launchpad.net/bugs/147918113:48
*** jasondotstar has joined #openstack-nova13:48
*** bfic has quit IRC13:51
*** otter768 has quit IRC13:52
*** salv-orlando has quit IRC13:52
*** jasondot_ has quit IRC13:52
*** fawadkhaliq has quit IRC13:53
openstackgerritTiago Mello proposed openstack/nova: libvirt: Handle unsupported Spice feature for PowerKVM  https://review.openstack.org/20380913:53
alaskimarkus_z: I don't think it meets the criteria for critical since it affects a (relatively small) subset of deployments13:54
*** LukeHinds has joined #openstack-nova13:54
*** bfic has joined #openstack-nova13:55
alaskibut it seems that we need to document the fact that cells doesn't guarantee compatibility during upgrades13:56
*** tpeoples has joined #openstack-nova13:56
*** fawadkhaliq has joined #openstack-nova13:56
mosheledanp: can you review https://review.openstack.org/#/c/187052/ and https://review.openstack.org/#/c/199488/ ?13:57
markus_zalaski: I see, thanks for the feedback, I was unsure about this.13:57
*** yamahata has joined #openstack-nova13:57
mosheledanpb: can you review https://review.openstack.org/#/c/187052/ and https://review.openstack.org/#/c/199488/ ?13:57
*** mwagner_lap has joined #openstack-nova13:58
*** ferest has joined #openstack-nova13:58
*** ctrath has quit IRC13:58
*** bfic has quit IRC13:58
*** ferest has quit IRC13:59
dansmithalaski: agree on !critical and the docs13:59
*** mragupat has joined #openstack-nova13:59
garykftersin: please see my comment - context was added due to the fact that the virt driver method did not have it …13:59
markus_zalaski: Where did we document such things in the past?13:59
*** mragupat has quit IRC13:59
*** thangp has joined #openstack-nova13:59
alaskimarkus_z: well, we haven't always.  but the devref would be the right place for it14:00
*** fawadkhaliq has quit IRC14:00
*** mragupat has joined #openstack-nova14:00
*** fawadkhaliq has joined #openstack-nova14:01
*** tonytan4ever has joined #openstack-nova14:01
mriedemit's 9am and i'm already depressed14:01
mriedemcalgone take me away14:01
mriedemsorry, calgon14:02
*** ctrath has joined #openstack-nova14:03
markus_zalaski: just to double-check: We assume with cells that all nodes are on the same release-level?14:03
*** annashen has joined #openstack-nova14:03
lxslibauzas: answering ndipanov now, figured it out14:03
alaskimarkus_z: yes, all nova-cells nodes should be the same14:04
*** markvoelker has quit IRC14:04
*** anthonyper has quit IRC14:04
*** lpetrut has quit IRC14:04
bauzaslxsli: sorry had no time to look thru yety14:04
alaskimarkus_z: it wasn't until recently that we had objects being used throughout cells which is a pre-req for the smooth upgrade path.  but adding objects is what has broken things14:05
mriedemmarkus_z: cells doesn't yet support rolling upgrades14:05
lxslibauzas: np just letting you know, I don't think it's a problem after all14:05
markus_zalaski: dansmith: So, this would mean the bug is "medium" at most and we should clarify this assumption as you said.14:05
*** q3k has quit IRC14:05
markus_zmriedem: OK, I wasn't aware of that.14:05
bauzasalaski: markus_z: missed the convo, so considering juno support as non supported ?14:05
*** EmilienM has quit IRC14:05
dansmithmarkus_z: and cells is experimental14:06
*** haypo has quit IRC14:06
*** lpetrut has joined #openstack-nova14:06
mriedemdropped it to low severity14:06
*** haypo has joined #openstack-nova14:06
alaskimarkus_z: yes14:06
*** haypo has quit IRC14:06
*** haypo has joined #openstack-nova14:06
mriedemcould probably just mark as invalid14:06
dansmithif we're not going to fix it, then probably14:06
mriedemdone14:06
*** rdopiera has quit IRC14:07
*** annashen_ has joined #openstack-nova14:07
markus_zmriedem: Would keep it as low with the need to document that. Otherwise others will fall into this too.14:07
bauzasmarkus_z: if not supported, then Invalid14:07
*** y_sawai has joined #openstack-nova14:07
mriedemi agree that documenting it would be good,14:07
mriedemi'm not totally sure where14:07
markus_zdansmith: "Cells" is still on my "research-list", I didn't know that.14:07
bauzasmriedem: devref sounds the better place14:08
*** annashen has quit IRC14:08
dansmithmarkus_z: we don't do a great job of communicating it, but it does barf a warning in to the logs on startup14:08
mriedemwe barf that it's experimental14:08
mriedemwe could also barf that n-1 isn't supported14:08
mriedemor just devref14:08
*** burt has joined #openstack-nova14:08
*** garyk has quit IRC14:09
alaskiI would vote for devref14:09
mriedemand/or here http://docs.openstack.org/openstack-ops/content/scaling.html#cells_regions14:09
*** EmilienM has joined #openstack-nova14:09
alaskiexperimental implies that any guarantees are out the window imo14:09
markus_zalaski: agreed14:10
*** anthonyper has joined #openstack-nova14:10
*** MentalRay_ has joined #openstack-nova14:10
mriedemfor devref i wouldn't lump this in with the cells v2 manifesto probably,14:10
mriedemso that leaves upgrades http://docs.openstack.org/developer/nova/upgrade.html14:10
mriedemcould add a blurb there i guess?14:10
markus_zmriedem: There it is :) "Overhead: Considered experimental."14:10
bauzasa warn in logs sounds good too14:11
bauzassince when people are reading docs ?14:11
mriedemsince when do people read logs?14:11
ftersingaryk: thanks for the explanation14:11
*** annashen_ has quit IRC14:11
*** jmccrory has quit IRC14:11
bauzasmriedem: as an operator, I was reading more logs than docs :)14:11
bauzasanyway14:12
*** browne has joined #openstack-nova14:12
mriedemhttp://git.openstack.org/cgit/openstack/nova/tree/nova/cells/manager.py#n8214:12
mriedemthat's the warning in the logs today14:12
*** jmccrory has joined #openstack-nova14:13
mriedemi'd be in favor of adding there that rolling upgrades aren't supported (yet)14:13
*** q3k has joined #openstack-nova14:13
bauzas+114:13
alaskisure, I'd +2 that14:13
mriedemalright, will do14:13
markus_zOK, the devref guide mentions the experimental state too.14:14
markus_zMaybe it could contain a section what our strategy with experimental things is and what the implications are.14:14
*** annegentle has joined #openstack-nova14:14
markus_zCells won't be the last feature with this status, I guess14:14
mriedemit means use at your own risk14:15
mnestratovdansmith: hello, I fixed obj_make_compatible in https://review.openstack.org/#/c/184311/, please take a look when you have time14:15
mriedemwe log warnings when using experimental things in most cases, or should be14:15
dansmithmnestratov: thanks, it's on my list for this morning14:15
dansmithmnestratov: sorry this has been complicated for just adding a new constant :/14:15
mnestratovdansmith: that's ok. It was helpful. I figured out how object versioning works14:16
markus_zmriedem: Yeah, sorry that I created a sensation. I'm aware of this now.14:17
dansmithjust +2d, if someone else wants to hit that ^14:17
mriedemdansmith: unless it's a bong i'm not interested14:17
mriedemmarkus_z: no, it's good to ask this kind of thing, and mention that we should document it14:18
mriedemfresh perspsective and all14:18
johnthetubaguyhaypo: sounds cool, will try to read that next week I think, worried about the freeze this week14:18
* johnthetubaguy is mostly attending midcycles (in a virtual sense) today14:19
_gryfbauzas, hi, regarding https://review.openstack.org/#/c/197063 - I've included the reason why it doesn't matter for zk driver if we force a service down or not14:21
_gryfbauzas, is it ok?14:21
*** whenry has quit IRC14:21
*** markvoelker_ has joined #openstack-nova14:21
*** Murali has quit IRC14:22
kashyapDarn, it's a duplicate of what mriedem already filed - https://bugs.launchpad.net/python-glanceclient/+bug/147929614:23
openstackLaunchpad bug 1476770 in OpenStack-Gate "duplicate for #1479296 _translate_from_glance fails with "AttributeError: id" in grenade" [Undecided,In progress] - Assigned to Matt Riedemann (mriedem)14:23
bauzas_gryf: then no14:23
_gryfbauzas, no?14:23
bauzas_gryf: I don't want to have an ABC having different behaviours based on the plugin14:23
kashyapNote to self: Check for existing bugs before filing!  Anyhow, seems like there's a fix for this.14:24
bauzas_gryf: I mean, the user doesn't know which SG backend is run14:24
*** whenry has joined #openstack-nova14:24
_gryfbauzas, I see your point14:24
bauzas_gryf: so if calling the REST API, he wants to have the host fenced anyway14:24
mnestratovsdague: could you please consider reissuing your +A on https://review.openstack.org/#/c/184311/ ?14:24
_gryfbauzas, but still it doesn;t matter14:24
bauzas_gryf: why ?14:25
_gryfbauzas, currently with the shape of zk driver it's not even possible :/14:25
bauzas_gryf: if I force that host to be down, I want it down14:25
mriedemmnestratov: small issue in https://review.openstack.org/#/c/184311/14:25
bauzas_gryf: I don't care which driver is running14:25
*** mtanino has joined #openstack-nova14:25
bauzas_gryf: if you leave ZK unchanged, it would mean that calling forced_down would still see the service up if the driver is ZK, right?14:26
*** r-daneel has joined #openstack-nova14:26
_gryfbauzas, I thought that what is plugins about - they share the same interface, but they do thing differently, am i right?14:26
_gryfbauzas, yes and no14:26
_gryfbauzas, yes - if it's really up14:27
_gryfbauzas, and yes - if you query in some of the narrow window of time14:27
*** liusheng has quit IRC14:27
bauzas_gryf: so, why are you trusting ZK and not using it ? and then, why are you needing a BP for providing a way to use a 3rd-party engine ?14:27
*** ctrath has quit IRC14:28
bauzas_gryf: either we consider the 3rd-party engine as the source of truth, or we consider that we don't need that as a source of truth because ZK is self-sufficient14:28
bauzas_gryf: but here, you make a very different semantics based on which driver is14:29
_gryfbauzas, if there would be driver similar to zk (like corosync) i would trust the heartbeat. Otherwise we must do other things like that14:29
bauzas_gryf: on one hand, you consider to defer the control to an external tool, but on the other hand you don't actually do that14:29
bauzasfor the audience, we're discussing abotu https://review.openstack.org/#/c/197063/11//COMMIT_MSG,cm14:30
*** baoli has quit IRC14:31
_gryfbauzas, from consistency pointo of view  - you are right14:32
*** baoli has joined #openstack-nova14:32
*** mpaolino has quit IRC14:32
*** lpetrut has quit IRC14:32
_gryfbauzas, from functoinal - IMHO it's a bit strange to not trust tools like that (which was especially crafted for the job)14:33
bauzas_gryf: I wouldn't matter about consistency if that wasn't exposed as a WSGI resource14:33
bauzas_gryf: again, if we trust ZK, why are we needing all of that ?14:33
openstackgerritVladik Romanovsky proposed openstack/nova: Adding user_id handling to keypair index, show and create api calls  https://review.openstack.org/7048514:34
bauzas_gryf: we could just say that deployers would support ZK or implement their own drivers out-of-tree14:34
bauzasso that the upstream Nova project would propose one single heartbeat mechanism, but any downstream vendor could provide his own14:34
bauzaswe wouldn't require to have an external hook for any 3rd-party engine14:35
_gryfbauzas, well. we could say that, yes. specifically this functionality was needed to imporve performance of the db driver (which is most popular among the sg drivers)14:35
bauzas_gryf: I don't want to open that can of worms, I'm just explaining that the BP was as a whole based on the idea that we allow the SG logic to be transferred to a 3rd party tool14:36
*** lpetrut has joined #openstack-nova14:36
bauzasif that 3rd-party tool is no-op for a specific set of drivers, that doesn't make sense IMHO14:36
*** coolsvap|away is now known as coolsvap14:37
bauzasand that could lead to confusion for operators wanting to deploy their own Pacemaker14:37
bauzas_gryf: imagine the scenario14:37
bauzas_gryf: I'm an operator using ZK because I like its reactivity but I also have some specific wordloads (like HPC batches) that I want to monitor using Nagios14:38
*** markvoelker_ has quit IRC14:38
bauzas_gryf: and I'm writing some Nagios plugins for checking the batch manager14:38
bauzasbatch managers even14:38
bauzas_gryf: if I'm seeing that a batch manager is not performing correctly, then I'd mark the host down and reconfigure my cloud14:39
*** markvoelker has joined #openstack-nova14:39
_gryfbauzas, oh. ok. I see your point now.14:39
*** ajayaa has joined #openstack-nova14:39
*** ctrath has joined #openstack-nova14:39
_gryfbauzas, I'll include the changes in the base class than. Is it ok, or should I change in every each driver?14:40
bauzas_gryf: the base class is the perfect class for that IMO14:40
openstackgerritMaxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo  https://review.openstack.org/18431114:40
_gryfbauzas, cool. thanks for your time :)14:41
mnestratovmriedem: fixed14:41
bauzas_gryf: np14:41
edleafejroll: around?14:41
*** yamahata has quit IRC14:41
mriedemmnestratov: not quite, you have a pep8 error14:42
*** madhaviy has joined #openstack-nova14:43
*** mpaolino has joined #openstack-nova14:44
*** kodoku has joined #openstack-nova14:45
*** markvoelker has quit IRC14:47
*** markvoelker has joined #openstack-nova14:47
*** ZZelle has quit IRC14:48
*** cfriesen_ has joined #openstack-nova14:49
* bauzas whispers https://sbauza.wordpress.com/2014/04/05/hook-for-tox-compliancy-before-a-git-review/ 14:49
jrolledleafe: kinda :)14:51
bauzas(amended with https://review.openstack.org/#/c/205575/)14:51
danpbbauzas: does that hoook invoke it on each individual commit, or only the final release ?14:52
danpbs/release/result/14:52
openstackgerritMatt Riedemann proposed openstack/nova: cells: document upgrade limitations/assumptions  https://review.openstack.org/20707614:52
mriedemalaski: markus_z: bauzas: ^14:52
jrolledleafe: I'm actually stepping away for a bit but I should be back in 30 or so14:52
bauzasdanpb: the hook is pre-review, so only the latest patch, hence the full pep8 coverage14:52
bauzasdanpb: before submitting, I'd rather call git rebase -x with your tox modification to see the output14:53
danpbeven running full pep8 check is not sufficient14:53
danpbbecause you could have a flaw in an early patch which is fixed in a later patch14:54
danpband that'll fail jenkins14:54
bauzasdanpb: yeah, and -x unfortunately needs interactive14:54
edleafejroll: ok. ping me when you can14:55
bauzasdanpb: but that's doable to run a diff pep8 on each change14:55
bauzaswithin the hook script14:55
*** eglynn_ has joined #openstack-nova14:57
*** jwcroppe has quit IRC14:58
*** eglynn has quit IRC14:58
*** belmoreira has quit IRC14:59
*** jasondotstar has quit IRC14:59
*** tsekiyama has joined #openstack-nova14:59
*** jistr has quit IRC15:00
*** gszasz has quit IRC15:00
*** kodoku has quit IRC15:01
*** ByPasS has quit IRC15:01
*** haomaiwang has joined #openstack-nova15:01
*** mragupat has quit IRC15:01
*** rex_lee has quit IRC15:02
*** dougwig has quit IRC15:02
*** mestery has joined #openstack-nova15:02
*** jistr has joined #openstack-nova15:02
*** ByPasS has joined #openstack-nova15:02
*** haomaiwang has quit IRC15:03
*** haomaiwang has joined #openstack-nova15:03
*** alex_klimov has quit IRC15:04
*** nkrinner has quit IRC15:05
*** annashen has joined #openstack-nova15:07
openstackgerritDan Smith proposed openstack/nova: Prepare for unicode enums from Oslo  https://review.openstack.org/20708715:08
*** mpaolino has quit IRC15:11
*** annashen has quit IRC15:13
*** chhavi has quit IRC15:14
*** alex_klimov has joined #openstack-nova15:14
*** josecastroleon has quit IRC15:15
*** davideagnello has joined #openstack-nova15:16
*** davideagnello has quit IRC15:20
openstackgerritShiina, Hironori proposed openstack/python-novaclient: Add support for injecting an NMI  https://review.openstack.org/20709215:21
*** tsekiyama has quit IRC15:23
*** ddaskal has joined #openstack-nova15:23
bauzasmarkus_z: will be off for the bug triage day but enjoy it15:24
mriedemdanpb: could use your review on this https://review.openstack.org/#/c/200280/15:25
markus_zbauzas: ah, too bad :( Thanks for the heads up15:25
mriedemdanpb: since that impacts imagemeta and eventually the bdm objet15:25
mriedem*object15:25
*** rajesht has quit IRC15:25
*** tsekiyama has joined #openstack-nova15:25
mriedemmnestratov: looks like the VZ CI has a patch merge conflict here http://openstack-3rd-party-ci-logs.odin.net/80/200280/6/check/dsvm-tempest-cinder/19d0e95/logs/devstacklog.txt.gz15:27
mriedemin os-brick15:27
bauzasdanpb: FYI http://paste.openstack.org/show/406261/ made the deal for checking pep8 iteratively15:28
*** ijw has joined #openstack-nova15:29
*** lucas__ has quit IRC15:30
mnestratovmriedem: thanks15:31
*** baoli has quit IRC15:31
*** lucas__ has joined #openstack-nova15:32
mnestratovmriedem: i'll see15:32
openstackgerritMatt Riedemann proposed openstack/nova: cells: document upgrade limitations/assumptions  https://review.openstack.org/20707615:32
*** alexpilotti has quit IRC15:34
openstackgerritAndreas Scheuring proposed openstack/nova: libvirt: Add macvtap as virtual interface (vif) type to Nova's libvirt driver  https://review.openstack.org/18228315:35
*** lucas__ has quit IRC15:37
*** hshiina has quit IRC15:38
bauzasdims: can I ask you a question about oslo.config opts ?15:38
dimsbauzas: shoot15:39
bauzasdims: coolness, ty15:39
bauzasdims: (morning btw.)15:39
*** baoli has joined #openstack-nova15:39
*** gabriel-bezerra has quit IRC15:39
bauzasdims: so, given http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/allocation-ratio-to-resource-tracker.html I need to provide 2 new flags within the compute manager15:39
bauzasdims: they're currently provided by the scheduler component, but since I need to have upgrades, I'll provide 2 new opts15:40
*** diogogmt has joined #openstack-nova15:40
bauzasdims: and keep the current ones (deprecated)15:40
dimsright15:40
bauzasdims: my question is, can I have the same name for those 2 opts (ie. ram_allocation_ratio) but have them in a separate group ?15:41
bauzaslike compute?15:41
dimsbauzas: yes15:41
bauzasI would prevent any bad naming like 'compute_ram_alloc_ratio'15:41
bauzasdims: sweet !15:41
*** lucas____ has joined #openstack-nova15:41
dimsbauzas: search for DeprecatedOpt15:41
*** vishwanathj has quit IRC15:41
bauzasdims: for the moment, until my series is implemented, I can't deprecate the sched opts15:42
bauzasdims: because they're yet in use15:42
*** dannywilson has joined #openstack-nova15:42
bauzasdims: or should we consider to deprecate those even still used by Nova ?15:42
dimsbauzas: typically you add the DeprecatedOpt in one release, leave it there till you can take it out15:42
bauzasdims: the question is rather, when do we provide a DeprecatedOpt ?15:42
dimsbauzas: see example http://docs.openstack.org/developer/oslo.config/opts.html15:43
dimsthere's some in the nova code too15:43
bauzasyeah I know15:43
bauzasmy wondering is rather about *when* deprecating those15:43
bauzasie. here15:43
dimsyou register the new option/group and specify which it deprecates15:43
*** lpetrut has quit IRC15:43
bauzasI'm gonna add those 2 new opts15:43
*** artom has joined #openstack-nova15:43
*** lpetrut has joined #openstack-nova15:43
dimsyes15:44
bauzasprovide them as fields for ComputeNode, send them to the DB, get them from the DB and use them in the Scheduler before they're actually in use15:44
jrolledleafe: free when you are15:44
bauzasin the meantime, those 2 old sched opts will still be in use15:44
dimsyes15:44
openstackgerritSean Dague proposed openstack/nova: tweak graphviz formatting for readability  https://review.openstack.org/20710315:45
bauzasand will be kept until we're sure that all computes (ie. N-2) provide those 2 opts15:45
*** artom has quit IRC15:45
dimsyes15:45
bauzasdims: so, can I deprecate immediately those 2 opts even if the new opts are not yet provided to the scheduler ?15:45
*** artom has joined #openstack-nova15:45
dimsyes, bauzas15:45
openstackgerritMaxim Nestratov proposed openstack/nova: libvirt: rename parallels driver to virtuozzo  https://review.openstack.org/18431115:45
bauzasdims: ack15:45
bauzasdims: coolness for your help15:45
*** lucas____ has quit IRC15:46
dimsbauzas: once you register the new option/group with the DeprecatedOpt, both the old and the new style configs will work15:46
bauzasdims: bauzas.get_beer()++15:46
mnestratovmriedem: fixed yesterday. asked CI to recheck15:46
dimsbauzas: :) thanks15:46
*** Sukhdev_ has joined #openstack-nova15:47
bauzasdims: okay, I should probably consider that again15:47
*** ftersin has quit IRC15:47
*** otter768 has joined #openstack-nova15:48
bauzasdims: http://docs.openstack.org/developer/oslo.config/opts.html#oslo_config.cfg.DeprecatedOpt is pretty well written15:48
dimsbauzas: thanks :) kudos go to dhellmann and markmc i believe15:48
*** ijw has quit IRC15:50
edleafejroll: I'm working on an update to your patch we discussed yesterday15:51
*** alex_klimov has quit IRC15:51
edleafejroll: I'm adding a check to see if the hypervisor_type is 'ironic', and if so, skip calling get_by_host() for that host15:52
edleafejroll: that won't break existing sites using the instance affinity filters, and will keep ironic from blowing up15:52
edleafejroll: without adding a new CONF setting15:52
jrolledleafe: the scheduler caring about the compute driver doesn't seem like the best abstraction, and really this is a "hundreds of instances per host" thing, not an ironic thing15:52
*** otter768 has quit IRC15:52
jroll(also, I believe hypervisor_type is 'baremetal' but not sure)15:53
edleafejroll: it15:53
edleafejroll: it's ironic (I checked :)15:53
mriedemyou're both wrong15:53
mriedemit's both http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/hv_type.py15:53
*** patrickeast has joined #openstack-nova15:53
edleafejroll: the abstraction ugliness comes from the design decision to treat bare metal resources similar to virt resources15:54
edleafejroll: but that ship sailed long ago15:54
*** karimb has joined #openstack-nova15:54
*** fawadkhaliq has quit IRC15:54
*** garyk has joined #openstack-nova15:54
*** chhavi has joined #openstack-nova15:54
edleafejroll: I have a few more tests to clean up, but I'll push it as soon as I finish15:56
*** matrohon has quit IRC15:58
*** doude has quit IRC15:59
*** rotbeard has quit IRC15:59
*** chhavi has quit IRC16:00
*** haomaiwang has quit IRC16:01
jrolledleafe: sure, I mean if other nova folks are okay with this method, I'm okay with ti16:01
*** lpetrut has quit IRC16:01
*** gabriel-bezerra has joined #openstack-nova16:01
*** jlanoux has quit IRC16:01
*** baoli has quit IRC16:01
jrolledleafe: however, I think we'll see this with containers too16:01
*** lpetrut has joined #openstack-nova16:01
edleafejroll: I think that's a good thing.16:01
edleafejroll: it will force a re-thinking of the underlying abstraction16:02
jrolledleafe: hitting this bug with other virt drivers later is a good thing?16:02
jrollah16:02
*** haomaiwang has joined #openstack-nova16:02
jrollwell16:02
jrollit's not about the abstraction16:02
*** jnc8651 has quit IRC16:02
jrollit's about, this code can't handle hosts with hundreds of instances16:02
jrollso if I had a kvm compute node with a terabyte of ram, I bet I could break it16:02
*** baoli_ has joined #openstack-nova16:03
*** lucasagomes is now known as lucas-afk16:03
jrollit being this code16:03
alaskifwiw we turned that off in our cloud for now because it was causing an issue in once cell, and we haven't dug into the root cause yet16:03
alaskis/once/one/16:03
jrollalaski: we only turned it off in the ironic cell, right?16:03
dansmithjroll: you mean a TB of RAM and still scheduled m1.tiny instances to it :)16:03
alaskijroll: it's off everywhere I believe16:03
jrolldansmith: well, yeah :)16:03
edleafealaski: but even if you turn off tracking, it will query the instances per host on each request16:03
jrollalaski: oh, different configs for ironic cells ftl :(16:04
*** lucas____ has joined #openstack-nova16:04
jrollyou're right though16:04
*** annashen has joined #openstack-nova16:04
edleafealaski: tracking only removes the need to query per request16:04
alaskiedleafe: which seems to be fine for us.  I'm not sure if our problem was related to number of instances per host.  but it was one of about 20 issues so we just disabled and moved on to the next.  we need to get back to it16:05
*** y_sawai_ has joined #openstack-nova16:05
jrollalaski: to clarify, the tracking code still runs, that config just turns off the initial "load everything at startup"16:05
jrollwhich loads instances for all hosts, which was likely the thing16:05
edleafealaski: ok, let me know what you find when you do16:05
edleafejroll: no, that's not correct16:05
*** vilobhmm has joined #openstack-nova16:05
alaskiyeah.  that was causing the scheduler to get "stuck" doing something16:05
edleafejroll: the tracking code won't run16:06
*** moshele has quit IRC16:06
edleafejroll: the code for querying was removed from the filters and added to the host manager16:06
edleafejroll: the problem is that it happens even if you don't use those filters16:06
edleafejroll: the HM has no insight into which filters are being used16:06
jrolledleafe: setting the config for the scheduler causes the initial load to not happen, setting it for the compute makes tracking not happen at all, as I understand it16:07
*** jistr has quit IRC16:08
jrolledleafe: and what alaski is talking about only has it turned off for the scheduler, it's still on for computes16:08
edleafejroll: there is only one setting16:08
*** y_sawai has quit IRC16:08
openstackgerritxing-yang proposed openstack/nova: Add os-brick based LibvirtVolumeDriver for ScaleIO  https://review.openstack.org/19445416:08
edleafejroll: compute uses the setting from the scheduler16:08
*** annashen_ has joined #openstack-nova16:08
jrolledleafe: not if the compute has a different config file16:08
edleafe 262 CONF.import_opt('scheduler_tracks_instance_changes',16:09
*** annashen has quit IRC16:09
edleafe 263                 'nova.scheduler.host_manager')16:09
jrolledleafe: I know it's the same config16:09
alaskiright, we just turned it off on the schedulers16:09
edleafejroll: ^ from nova/compute/manager.py16:09
jrollwe have different config files on the scheduler and the compute hosts16:09
jrollthe compute hosts' config file does not have this disabled.16:09
jrollimport_opt doesn't query the scheduler for the setting16:10
openstackgerritAdelina Tuvenie proposed openstack/nova: Fixes Hyper-V boot from volume fails when using ephemeral disk  https://review.openstack.org/20711316:10
edleafejroll: why wouldn't you update the compute nodes?16:11
mriedemrlrossit: you should be good to recheck your stuff now16:11
*** thangp has quit IRC16:11
*** meghal has joined #openstack-nova16:11
jrolledleafe: me personally? because I don't operate the rest of our cloud, only the ironic bits (where we did disable it on compute hosts)16:11
alaskiedleafe: I turned if off on the scheduler because that's where the problem was.  it wasn't the messages it was something in the schedulers processing of them causing an issue16:11
*** jaypipes has joined #openstack-nova16:12
edleafealaski: ok; please let me know what you find out16:12
jrollalaski: fwiw, if that's only disabled on the scheduler, it just disables the load-every-instance-at-startup (which I'm not surprised caused problems)16:12
*** coolsvap is now known as coolsvap|away16:12
openstackgerritDan Smith proposed openstack/nova: Prepare for unicode enums from Oslo  https://review.openstack.org/20708716:12
alaskiedleafe: will do16:12
*** nic has joined #openstack-nova16:13
*** scottda has quit IRC16:13
*** briancurtin has quit IRC16:13
*** zhiyan has quit IRC16:13
*** ameade has quit IRC16:13
*** xyang has quit IRC16:13
*** wanghao has quit IRC16:13
*** LukeHinds has quit IRC16:13
*** serverascode has quit IRC16:13
*** annashen_ has quit IRC16:13
*** Piet has joined #openstack-nova16:13
alaskijroll: yeah, that was the troublemaker.  there was some unevenness in the loading, like it would process a bunch of hosts with 2-10 instances and then get to one that it said had 200 instances or something and basically just stop16:14
alaskidefinitely needs investigation16:14
jrollheh16:14
jrollyeah, we were OOM'ing because E_TOO_MANY_INSTANCES16:14
rlrossitmriedem: thanks for letting me know because I looked like 10 minutes ago and was going to give up on it until after lunch16:15
*** lucas____ has quit IRC16:15
edleafealaski: the batch size for the startup run is 10 hosts. I wanted to make that configurable, but the old 'no more CONF!' arguments won out16:15
mriedemrlrossit: do you know how to monitor a change through the queue in zuul?16:15
rlrossitmriedem: nope16:15
mriedemhttp://status.openstack.org/zuul/16:15
mriedemplug in the change number from the url16:15
*** lucas__ has joined #openstack-nova16:15
*** thangp has joined #openstack-nova16:16
rlrossitwait what do you mean by monitor?16:16
mriedeme.g. 206674 for https://review.openstack.org/#/c/206674/16:16
mriedemyou can see the changes placement in the queue and the progress of the jobs on that change16:16
mriedemif you're waiting for a thing to merge16:16
rlrossitoh yeah I do that. but I don't actively stare at it16:16
mriedemme neither, but i check in on it as i run through tabs16:17
rlrossitI thought you were saying you could like have zuul tell you when something happened16:17
mriedemi have zuul send me text messages when my code merges16:17
mriedemand tweets to my twitter account16:17
alaskiedleafe: if that's causing an issue it would be worth adding a conf, or changing the default, or optimizing.  but that seems like a reasonable default for now16:17
rlrossitI mean... I wouldn't hate that...16:17
*** leeantho has joined #openstack-nova16:17
*** lpetrut has quit IRC16:17
*** chhavi has joined #openstack-nova16:17
*** kmARC has quit IRC16:17
mriedem'@zuul says you have merged change 123 way2go LOL!'16:17
rlrossitother than the fact that I wouldn't be able to get the text within these steel walls16:17
*** unicell has quit IRC16:17
*** xyang1 has joined #openstack-nova16:18
rlrossitthe prison bars on my office block a lot of cell reception16:18
mriedemsorry, i should have had a hash tag in that tweet16:18
*** david-lyle has quit IRC16:18
alaskithere's also https://github.com/kk7ds/openstack-gerrit-dashboard16:19
mriedemyeah, that's also running16:19
mriedembut these aren't always my changes that i'm watching16:19
mriedemi use status.o.o/zuul for those16:19
*** lsmola has quit IRC16:20
mriedemahh16:20
mriedem"./dash.py -o cbehrens"16:20
mriedemRIP!16:20
alaskiheh16:20
dansmithheh16:20
alaskihe also lives on in multiple TODOs in the codebase16:20
mriedemTODO(comstud): this f'ing sucks, i hate it, let's fix it someday16:21
mriedemall over the quotas code :)16:21
mriedemooo16:22
mriedemi'm going to push a patch to dash16:22
mriedem-O is new to me16:22
*** ip0353 has quit IRC16:22
mriedemso i can watch my changes with -o and a specific other change with -c, but have to use -O OR16:22
*** belmoreira has joined #openstack-nova16:22
dansmithyeah I use: python dash.py -u danms -p openstack/nova,openstack/oslo.versionedobjects -r 10 -s -O OR -o danms16:23
mriedemdansmith: is it just OR and AND?16:23
mriedemi was going to update the doc since i had to check the code16:23
dansmithI don't remember16:24
dansmithwhat else do you want? XOR? :P16:24
mriedemsure16:24
dansmithheh16:24
mriedemhttps://review.openstack.org/Documentation/user-search.html#_boolean_operators16:24
dansmith-o danms -O NAND -p ...16:24
mriedem-OR16:24
dansmithI find it totally amusing that the sample outputs are from ~30k reviews16:25
*** Marga_ has quit IRC16:26
*** afazekas has quit IRC16:27
*** browne has quit IRC16:27
*** Marga_ has joined #openstack-nova16:28
*** dougwig has joined #openstack-nova16:29
*** yassine has quit IRC16:29
*** markus_z has quit IRC16:29
*** exploreshaifali has joined #openstack-nova16:33
*** tonytan4ever has quit IRC16:34
*** ajayaa has quit IRC16:37
*** tpeoples has quit IRC16:38
*** coolsvap|away is now known as coolsvap16:39
*** jordanP has quit IRC16:39
*** rdopiera has joined #openstack-nova16:41
*** sahid has quit IRC16:41
*** serverascode has joined #openstack-nova16:41
*** david-lyle has joined #openstack-nova16:42
*** enikher has quit IRC16:43
*** ameade has joined #openstack-nova16:43
*** alexpilotti has joined #openstack-nova16:45
*** zhiyan has joined #openstack-nova16:46
*** Marga_ has quit IRC16:47
*** baoli_ has quit IRC16:48
*** MentalRay_ has quit IRC16:48
*** baoli_ has joined #openstack-nova16:49
*** tpeoples has joined #openstack-nova16:49
*** Nikolay_St has quit IRC16:50
*** unicell has joined #openstack-nova16:50
*** BobBall is now known as BobBall_AWOL16:54
openstackgerritEd Leafe proposed openstack/nova: Remove unneeded and painful call for ironic  https://review.openstack.org/20673616:54
edleafejroll: ^^16:55
* jroll will look16:55
*** briancurtin has joined #openstack-nova16:57
openstackgerritNicolas Simonds proposed openstack/nova: libvirt: make snapshot use RBD snapshot/clone when available  https://review.openstack.org/20528216:57
*** lpetrut has joined #openstack-nova16:58
*** coolsvap is now known as coolsvap|away16:58
*** coolsvap|away is now known as coolsvap16:58
openstackgerritChuck Carmack proposed openstack/nova: Fix DeprecationWarning when using BaseException.message  https://review.openstack.org/20712716:58
*** davideagnello has joined #openstack-nova17:00
jaypipesnic: note that my review user ID is jaypipes@gmail.com. jpipes@mirantis.com is something I used for testing external CI systems...17:00
jaypipesnic: re: https://review.openstack.org/#/c/205282/ :)17:00
*** LukeHinds has joined #openstack-nova17:00
*** haomaiwang has quit IRC17:01
openstackgerritNicolas Simonds proposed openstack/nova-specs: RBD instance snapshots  https://review.openstack.org/18824417:01
nicI didn't add you, seb did17:01
nicFIXED FOR GREAT JUSTICE17:02
*** haomaiwang has joined #openstack-nova17:02
ndipanovbauzas, https://review.openstack.org/#/c/203078/17:02
nicjaypipes: although I guess I should start the whining about getting that spec squeezed in for the L release.17:03
nicFolks are saying something akin to "Ceph is unusable right now" with respect to the current state-of-the-art17:03
*** kumar has joined #openstack-nova17:04
nicWhere should I go to whine the most effectively, and at what volume?17:04
bauzasndipanov: in cells meeting, but lemme look17:04
bauzasndipanov: oh that, see my commit msg17:05
*** rex_lee has joined #openstack-nova17:05
bauzasndipanov: I wanted to make the BP implemented by the last patch, but I'm quite okay with you, we need to remove the build_req_spec() method etc.17:05
*** tjones has joined #openstack-nova17:06
ndipanovthere some other issues - go through the series and see individual comments17:06
*** scottda has joined #openstack-nova17:06
ndipanovimho this is not done until the build_request_spec dies in a fire17:06
scheuranmriedem, could you please have another look on the macvtap-vif patchset some time your afternoon? I was removing the vlan device name logic. https://review.openstack.org/#/c/182283/17:08
*** Marga_ has joined #openstack-nova17:08
scheuranmriedem, it still has to walk through the gate, so no hurry17:08
scheuranmriedem, thanks!17:08
*** annashen has joined #openstack-nova17:09
bauzasndipanov: ack, will look thru17:09
*** wanghao has joined #openstack-nova17:10
*** scheuran has quit IRC17:10
*** mjturek1 has quit IRC17:11
*** mpavone has quit IRC17:11
*** danpb has quit IRC17:12
*** annashen has quit IRC17:13
*** salv-orlando has joined #openstack-nova17:14
*** alexpilotti has quit IRC17:14
jaypipesnic: johnthetubaguy and mriedem would be my best advice.17:18
*** annashen has joined #openstack-nova17:19
*** kumar has quit IRC17:19
*** xyang has joined #openstack-nova17:19
*** dboik has quit IRC17:19
*** karimb has quit IRC17:19
*** browne has joined #openstack-nova17:19
ndipanovjaypipes, lxsli I don't think you fully understood my comment here https://review.openstack.org/#/c/128992/17:19
*** salv-orlando has quit IRC17:20
*** moshele has joined #openstack-nova17:20
mriedemnic: the correct volume is always 1117:20
jaypipesndipanov: no, I fully udnerstand your comment. I am saying that checking for capacity is not the same activity as assigning a particular NUMA node toplogy/pinning to an instance17:21
*** madhaviy has quit IRC17:21
ndipanovjaypipes, fair enough17:21
mriedemnic: so we're at feature freeze for liberty tomorrow17:21
nicI'm looking over the agenda for the meeting tomorrow, and I have no idea where to try and slot my stuff in.17:21
mriedemnic: so given the spec isn't even approved, and the deadline for spec approval was long ago, i don't see this making liberty17:21
*** rajesht has joined #openstack-nova17:21
mriedemnic: open discussion17:22
ndipanovdon't we want an interface to that side of reosurces as well17:22
ndipanovjaypipes, ^17:22
*** salv-orlando has joined #openstack-nova17:22
*** patrickeast has quit IRC17:23
*** alexpilotti has joined #openstack-nova17:23
bauzasndipanov: heya, just read your comments17:24
*** lpetrut has quit IRC17:24
bauzasndipanov: nothing really complaining about, just one sentence that I'm going thru without really understanding :D17:24
bauzasndipanov: "I wanted to comment on the previous patch - but it would be  nice to have some kind of abstraction here (like a special descriptor  that will delegate back to the one object has if the type is right and  do the conversion otherwis) instead of all of these awkwardly named  methods that special case single fields."17:24
bauzasndipanov: in https://review.openstack.org/#/c/187197/2417:24
bauzasndipanov: could you try bauzas's translator ? :)17:25
tjonesIm in rebase hell here if anyone has time for a review - https://review.openstack.org/#/c/127427/17:26
openstackgerritVilobh Meshram proposed openstack/nova-specs: Quota Management for Dynamic Quota Resources  https://review.openstack.org/20616017:26
*** rajesht has quit IRC17:26
ndipanovbauzas, well all object fields work like properties which are actually python descriptors17:27
ndipanovinstead of that inline mess with arbitrarily named methods17:27
ndipanovmaybe we could come up with a descriptor that does the conversion if needed - otherwise delegates to the standard object's field machinery17:27
*** achanda has joined #openstack-nova17:27
ndipanovthat's just a thought17:27
ndipanovnot really necessary to do it that way17:28
*** lpetrut has joined #openstack-nova17:28
bauzasndipanov: mmm, I think I see your point17:28
ndipanovbut right now it's just a bunch of weird methods that are not even uniformly named17:28
bauzasndipanov: yeah, naming is not one of my best skills17:28
*** mjturek1 has joined #openstack-nova17:29
ndipanovbauzas, also once the bp is done we can drop all forward porting methods17:29
ndipanovprimitive -> spec_obj17:29
ndipanovplease do it17:29
ndipanovno reason to keep that code around17:30
bauzasyeah I know17:30
*** tjones1 has joined #openstack-nova17:30
ndipanovif it's there people will use it17:30
ndipanovin like 3 days17:30
ndipanovtrust me :D17:30
*** aix has quit IRC17:30
bauzasndipanov: I was thinking of the compat decorator, it was pretty acceptable to keep it re: out-of-tree17:30
bauzasndipanov: since those guys would probably be like chicken with cut head17:31
ctrathmriedem: I'm looking at this: https://bugs.launchpad.net/os-brick/+bug/1479355 … Where should constants be defined?17:31
openstackLaunchpad bug 1479355 in os-brick "Define constants for connector factory protocol" [Undecided,Confirmed] - Assigned to Cale Rath (ctrath)17:31
bauzasonce they see that their filters don't work now17:31
ndipanovbauzas, absolutely not17:31
bauzasndipanov: I know17:31
*** tonytan4ever has joined #openstack-nova17:31
mriedemctrath: i believe rlrossit was already looking at that17:31
ndipanovwe have enough "stable" apis as is that we can't break17:31
bauzasndipanov: I know it's bad to keep something for out-of-tree need17:31
*** lpetrut has quit IRC17:31
mriedemrlrossit: right? ^ if so, assign it to yourself17:31
ctrathmriedem: ok.. It was not assigned17:31
bauzasndipanov: but I'm a gentleman :)17:32
ndipanovbauzas, filter interface is not one of them. Kill it17:32
*** Marga_ has quit IRC17:32
bauzasndipanov: lol17:32
bauzasndipanov: fair enough17:32
*** Marga_ has joined #openstack-nova17:32
*** tjones1 has quit IRC17:32
*** shaohe_robot has quit IRC17:32
bauzasndipanov: alaski would just probably be that worried :D17:32
bauzasndipanov: I mean RAX17:33
bauzasndipanov: but let's move on17:33
*** shaohe_robot has joined #openstack-nova17:33
*** tjones1 has joined #openstack-nova17:33
alaskiwhat are we talking about killing?  I need to read back17:33
*** tjones1 has quit IRC17:33
*** lpetrut has joined #openstack-nova17:34
bauzasalaski: no worries, just explaining that RAX is a chicken17:35
* bauzas thanks scrollback17:35
alaski"resistant to some change"17:35
bauzasalaski: no, was speaking of the filters API passing the old legacy filter_props dict17:35
bauzasalaski: since I'm changing our in-tree filters to accept a ReqSpec object now, it means that out-of-tree filters will be broken17:36
bauzasoh, that reminds me I should tag UpgradeImpact !17:36
*** armax has joined #openstack-nova17:36
ndipanovthey would be broken anyway unless they add that line17:37
alaskithat's fine.  it's not an api we guarantee to be stable17:37
bauzasalaski: so I was using the metaphor of a chicken which has the head cut17:37
ndipanovexactly17:37
bauzasyep17:37
ndipanovwe have enough "stable" APIs17:37
alaskiI would like us to get there, because I do think out of tree filters should be supported.  but we're not there yet17:37
bauzasno, that whole convo was based on keeping a compat decorator I provided17:37
ndipanovplease don't17:37
ndipanovI mean17:38
bauzasyeah yeah17:38
bauzasthat's fine, will do17:38
*** patrickeast has joined #openstack-nova17:38
bauzasand when removing the to_legacy_dict(), it will be anyway impossible to look back17:38
alaskiwe just had to adapt to https://review.openstack.org/#/c/188485/, it wasn't a big deal17:39
*** jaypipes has quit IRC17:40
bauzasalaski: yeah, and using the Spec object shouldn't be a big deal too17:40
bauzasalaski: you should just review the existing changes where I'm doing that change17:41
ndipanovbauzas, why would we remove to_legacy_dict?17:41
ndipanovwe can't until we move the compute completely to spec object17:41
ndipanovand then wait another RPC version cycle17:41
johnthetubaguynic: +1 to what mriedem said, although bug fixes are welcome, but it sounded like it was more than that I guess.17:41
bauzasndipanov: oh yeah17:41
ndipanovwhat we can and should remove ASAP is forward porting17:41
*** annashen has quit IRC17:42
ndipanovlegacy will stay with us for some time...17:42
bauzasyou mean the from_primitives() ?17:42
bauzasgotcha17:42
bauzasokay, I misunderstood your thoughts17:42
bauzasany good opportunity for me to pause and properly document the backport methods and maybe think of an abstraction17:43
*** r-daneel has quit IRC17:44
alaskibauzas: I think all we'll really need to change to deal with reqspec object is to stop using dict style attribute accesses17:44
bauzasalaski: not exactly since we remove the instance_props subdict but yeah you got the idea17:44
ctrathrlrossit: ping17:44
alaskibauzas: right, still an easy adaption17:45
*** ByPasS has quit IRC17:45
alaskiphew, apparently that is a word17:46
*** annashen has joined #openstack-nova17:46
*** dannywilson has quit IRC17:46
bauzasalaski: don't overrule me with my frenglish17:46
*** Sukhdev_ has quit IRC17:47
nicjohnthetubaguy: if feature freeze is tomorrow, and you have a completed feature, he fact that the cover sheet isn't on the TPS report shouldn't torpedo the whole thing, IMO17:48
*** ByPasS has joined #openstack-nova17:48
nicBut the process is the process for a reason, and I don't want to claim special snowflake status on my forst attempt17:48
*** otter768 has joined #openstack-nova17:49
sdaguejohnthetubaguy: gmann's spec update is here - https://review.openstack.org/#/c/20686417:50
*** Marga_ has quit IRC17:51
johnthetubaguynic: its not process for the sake of process, there is a good reason behind it (much of it described here: https://wiki.openstack.org/wiki/Nova/Liberty_Release_Schedule), we have given two weeks for all the advertised (i.e. approved) blueprint patches to be up for review, so we can try and work through all of those and maximise the number of completed blueprints (currently have 50 or so in that situation I think), while been honest to17:52
johnthetubaguy everyone else about the fact we just don't have the bandwidth to review that17:52
*** Marga_ has joined #openstack-nova17:52
johnthetubaguynic: have you got a link to your blueprint, etc, I couldn't find that in the scrollback for some reason17:52
johnthetubaguysdague: I have been having internet troubles, seems gerrit didn't submit my vote :(17:53
sdaguejohnthetubaguy: ok17:53
johnthetubaguysdague: I had a minor worry, but I still like the overall approach17:53
nicjohnthetubaguy: https://review.openstack.org/188244 and https://blueprints.launchpad.net/nova/+spec/rbd-instance-snapshots17:53
*** moshele has quit IRC17:53
johnthetubaguysdague: does my comment make sense?17:53
sdaguejohnthetubaguy: what is the minor worry?17:53
*** otter768 has quit IRC17:54
rlrossitctrath: what's up?17:54
*** jasondotstar has joined #openstack-nova17:54
*** lucas-afk is now known as lucasagomes17:55
johnthetubaguysdague: oh, I added that comment, I should really run, I have my dinner going cold at this point17:55
rlrossitctrath: yeah I'm working on https://bugs.launchpad.net/os-brick/+bug/1479355 not sure why it didn't get updated when I didn't push up a patch for it17:55
openstackLaunchpad bug 1479355 in os-brick "Define constants for connector factory protocol" [Undecided,Confirmed] - Assigned to Cale Rath (ctrath)17:55
johnthetubaguysdague: I am just worried about when locked: false doesn't apply17:55
ctrathrlrossit.. Ok17:56
mriedemctrath: https://review.openstack.org/#/c/207090/17:56
sdaguejohnthetubaguy: locked: false means there is no lock, I think that's pretty clear17:56
johnthetubaguynic: this does appear to have missed all the deadlines we have in place, its unfair to give this one an exception at this point, given the amount of blueprints that did make all the deadlines (currently around 90 or so). Most of those are about to get deferred tomorrow as well, not that that helps anyone, but the point here is to focus on the agreed project priorities17:57
ctrathrlrossit: I removed myself from the bug, but I can't find your name to assign it to you17:57
johnthetubaguysdague: sorry, I mean the other way around locked: true, but its the admin listing it, and its not actually locked for them, as its an owner lock17:57
mriedemctrath: neither can i, might be some weirdness with how the os-brick project is setup in launchpad17:57
johnthetubaguysdague: I don't think its a big deal17:57
rlrossitI got it ctrath17:57
sdaguejohnthetubaguy: sure, and when we have a lock resource, that can be distinguished17:57
ctrathok, thanks17:58
*** lucas__ has quit IRC17:58
sdagueright now the admin can break all the locks anyway17:58
rlrossitmriedem: why didn't it post the change and set it to in progress when I put the patch up?17:58
nicjohnthetubaguy: I was under no illusion that I was working on anything considered a project priority, that's why I didn't make much noise about this at the mid-cycle.  But some folks that have seen this work have said that it would be really good to land, so I'm giving it the old college try17:58
mriedemrlrossit: because ^17:58
johnthetubaguysdague: yeah, I think it just means, "there is a lock", not that "its locked for you"17:58
sdaguethere are more semantics here that are pretty hacky17:58
*** xyang1 has quit IRC17:58
sdaguejohnthetubaguy: yes, I agree, this means there is a lock17:58
rlrossitoh missed that one17:58
johnthetubaguynic: thats cool, it looks super useful, but so are the other ones we are already focused on reviewing already, its more a general fairness thing to all the others that have been deferred, I wish we had way more review capacity, but we don't right now, but we are working on it :(17:59
johnthetubaguysdague: I was just thinking its worth a note, but I am all for the change, let me add that.18:00
sdagueok18:00
sdagueyeh, the implementation is pretty clear already on that. kenichi blocked that on the spec.18:00
sdaguenot being landed18:00
*** haomaiwang has quit IRC18:01
mriedemnic: i'm going to writeup something to the ML on this topic, we discussed it a bit at the midcycle but i don't exactly remember how it landed, so will regurgitate it there18:01
sdagueso I actually think all the ducks are in a row here, just needs +2's and not mutual deadlock18:01
nicmriedem: which topic?18:01
mriedemnic: freezes18:02
*** shyama has quit IRC18:02
nicAh.  reading comprehension is not my strong suit18:02
*** haomaiwang has joined #openstack-nova18:02
*** browne has quit IRC18:02
mriedemnic: that's ok, you're american18:03
*** pixelbeat_ has quit IRC18:03
*** browne has joined #openstack-nova18:04
*** lucas__ has joined #openstack-nova18:05
*** annashen_ has joined #openstack-nova18:05
*** adam_g has joined #openstack-nova18:07
*** adam_g has quit IRC18:07
*** adam_g has joined #openstack-nova18:07
*** lucas__ has quit IRC18:09
*** lucas__ has joined #openstack-nova18:10
*** annashen_ has quit IRC18:10
*** jwcroppe has joined #openstack-nova18:11
*** lpetrut has quit IRC18:12
*** ByPasS has quit IRC18:13
*** dannywilson has joined #openstack-nova18:18
melwittmriedem: if someone forgets to cherry-pick -x for a backport, would it work to just change the commit message manually or does it have to be redone with -x? https://review.openstack.org/#/c/199569/18:18
*** coolsvap is now known as coolsvap|brb18:18
mnestratovmriedem: last pep8 issue addressed https://review.openstack.org/#/c/184311/18:18
*** ByPasS has joined #openstack-nova18:19
mriedemmelwitt: oooooo18:20
mriedemi bring hellfire down on those that don't cherry-pick -x18:20
mriedemmelwitt: it doesn't matter too much, the change id matching is the big thing18:20
dansmitheh?18:20
mriedemit would be nice to see the (cherry picked from commit) thing in there though18:20
dansmithdoes it affect that? I don't think so18:20
dansmithI thought it just does the (from commit $foo)18:21
mriedemcherry-pick -x adds the cherry pickedfrom commit hash line18:21
mriedemyeah18:21
mriedemthe hellfire comment was a joke18:21
*** annashen has quit IRC18:21
dansmithoh you mean the change-id matching is the important step to get right18:21
dansmithnot the important thing about using -x18:21
mriedemyes18:22
dansmiththen, yes18:22
* dansmith brings down hellfire on those that speak ambiguously18:22
mriedemhey18:22
mriedemhellfire block!18:22
mriedem(saving throw jokes)18:22
*** jasondotstar has quit IRC18:23
melwittI was thinking if I could add the "from commit" message in there instead of asking the person to redo the entire cherry-pick, and tell them use -x next time please18:23
mriedemmelwitt: naw, that will reset the votes and if you do that, you might as well just redo with cherry-pick -x18:23
mriedemnot sure it would matter either way18:24
*** annashen has joined #openstack-nova18:24
*** dims_ has joined #openstack-nova18:24
mriedemsounds like an unimportant bug but i guess over time it kills things with ceilometer?18:24
melwittmriedem: yeah, guess that's true. I was trying to avoid saying "redo this thing because there isn't a footnote in the commit message" but they're a similar amount of work assuming no conflicts18:25
*** dboik has joined #openstack-nova18:25
melwittmriedem: yeah cfriesen_ said ceilometer will keep creeping up the memory because of this18:26
*** annegentle has quit IRC18:26
*** annegentle has joined #openstack-nova18:26
openstackgerritLucas Alvares Gomes proposed openstack/nova: Ironic: Better handle InstanceNotFound on destroy()  https://review.openstack.org/20499518:26
*** pixelbeat_ has joined #openstack-nova18:27
*** dims has quit IRC18:28
*** lucasagomes is now known as lucas-dinner18:28
*** josecastroleon has joined #openstack-nova18:28
*** MentalRay_ has joined #openstack-nova18:30
*** openstackgerrit has quit IRC18:31
*** openstackgerrit has joined #openstack-nova18:32
*** armax has quit IRC18:32
*** pixelbeat_ has quit IRC18:33
*** jaypipes has joined #openstack-nova18:34
*** dannywilson has quit IRC18:34
*** dannywilson has joined #openstack-nova18:34
*** ByPasS has quit IRC18:34
*** chhavi has quit IRC18:35
dansmithsdague: can I run your multinode-grenade thing against a patch easily?18:35
*** thorst has quit IRC18:36
*** ByPasS has joined #openstack-nova18:36
*** ByPasS has quit IRC18:40
*** salv-orlando has quit IRC18:40
*** annashen has quit IRC18:42
*** vishwanathj has joined #openstack-nova18:42
*** ByPasS has joined #openstack-nova18:43
*** jwcroppe has quit IRC18:43
*** jwcroppe has joined #openstack-nova18:44
mnestratovdansmith: I resubmitted https://review.openstack.org/#/c/184311/ to address mriedem's comments, thus erased your +218:44
dansmithmnestratov: I refuse to review that until mriedem +2s it18:44
dansmithmnestratov: he likes to -1 my +2s too much18:44
dansmithmnestratov: so let me know when his majesty has blessed it and I'll re-+2 :)18:45
mnestratovdansmith: :)18:45
*** y_sawai has joined #openstack-nova18:45
*** fawadkhaliq has joined #openstack-nova18:45
mnestratovmriedem: your majesty, please18:46
*** vishwanathj has quit IRC18:46
mriedemwe are reviewing it now18:47
mriedemthere18:47
*** y_sawai_ has quit IRC18:48
*** jwcroppe has quit IRC18:48
sdaguedansmith: ... well, its in experimental on devstack-gate18:49
* dansmith -1s18:49
sdagueso you'd have to Depends-On that d-g change, then upload another d-g change and run check experimental18:49
dansmithsdague: heh18:49
dansmithnevermind then :)18:49
sdagueor a grenade change18:49
sdagueor convince mordred he should land the code :)18:50
sdagueand we can make it easier18:50
* dansmith scoffs in mordred's general direction18:50
*** thorst has joined #openstack-nova18:51
*** shaohe_robot1 has joined #openstack-nova18:55
*** Sukhdev_ has joined #openstack-nova18:55
*** fawadk has joined #openstack-nova18:55
*** fawadkhaliq has quit IRC18:56
*** shaohe_robot has quit IRC18:57
mnestratovany chances to look at https://review.openstack.org/#/c/186858/ today? It is +2-ed by danpb18:57
*** josecastroleon has quit IRC18:58
*** haomaiwang has quit IRC19:01
*** haomaiwa_ has joined #openstack-nova19:01
*** salv-orlando has joined #openstack-nova19:01
*** krtaylor has quit IRC19:04
*** lucas__ has quit IRC19:07
mriedemmnestratov: which does covers the case that 'disk' is in the mapping dict?19:08
mnestratovmriedem: yes19:08
mriedemsorry, which test covers19:08
mriedemthe changed test and the one above it are very similar and use some stubs, so it's hard to read how those 2 paths are differnet19:09
*** coolsvap|brb is now known as coolsvap|away19:10
*** Piet has quit IRC19:10
mnestratovtest_get_guest_config_parallels_ct19:12
mnestratovct == exe19:12
*** ZZelle has joined #openstack-nova19:13
*** IanGovett has joined #openstack-nova19:13
*** jamiec has joined #openstack-nova19:13
mnestratovabove test is for hvm, below for exe19:13
*** haypo has left #openstack-nova19:13
mriedemso this is probably just one of those cases where i have to memorize the internals of how blockinfo.get_disk_info works to know how this is actually tested19:14
mriedemi guess i was looking for an assertion that the filesystem disk wasn't added in one of these cases19:14
*** meghal has quit IRC19:16
*** fawadkhaliq has joined #openstack-nova19:16
*** artom has quit IRC19:16
*** krtaylor has joined #openstack-nova19:17
*** VW__ has joined #openstack-nova19:18
*** artom has joined #openstack-nova19:18
*** jasondotstar has joined #openstack-nova19:18
*** lpetrut has joined #openstack-nova19:18
*** jamiec has quit IRC19:19
*** IanGovett has quit IRC19:19
*** fawadk has quit IRC19:19
*** tjones has quit IRC19:21
*** VW_ has quit IRC19:22
mnestratovmriedem: thanks for review. I'll add assertion19:22
mriedemmnestratov: added some comments19:22
*** READ10 has quit IRC19:22
*** VW__ has quit IRC19:22
mriedemmnestratov: basically i don't think you should be updating the existing test,19:22
*** lucas__ has joined #openstack-nova19:22
mriedemyou should have a test that covers the case that 'disk' is in the mapping, and one where 'disk' isn't in the mapping and vm_mode is EXE19:22
mriedemand make the proper assertion that the filesystem device isn't added if 'disk' isn't in the mapping19:23
*** fawadk has joined #openstack-nova19:24
mriedemone day i hope i'll make enough money that i can add a comment in an executive blog post that says, "Empowering, enabling and energizing our enterprise customers is fundamental to who we've been, who we are and who we will be as a company."19:25
dansmithmnestratov: I agree with his majesty19:25
mnestratovmriedem: ok. just tried to change fewer lines of code19:26
*** fawadkhaliq has quit IRC19:26
dansmithmnestratov: changing few lines of production code is good. Fewer lines of test code, usually not :D19:26
mnestratovdansmith: :) me too19:26
mnestratovdansmith: I see19:27
*** dims_ has quit IRC19:27
*** davideagnello has quit IRC19:27
*** dims has joined #openstack-nova19:27
*** VW_ has joined #openstack-nova19:29
dimsmriedem: nice goal! :)19:30
mriedemdims: or, i'll just repeat that everywhere19:31
mriedemlike a slogan19:31
mriedemeventually it will be true and i'll be unstoppable19:31
dimsmriedem: print a sticker! :)19:31
dimslike the cirros default password :)19:31
*** lucas__ has quit IRC19:32
*** lucas__ has joined #openstack-nova19:33
*** vishwanathj has joined #openstack-nova19:35
*** salv-orlando has quit IRC19:35
*** ByPasS has quit IRC19:35
*** ByPasS has joined #openstack-nova19:35
dansmithdims: by the way, I have a todo item to get taylor an oslo sticker in tokyo19:36
dansmithI rudely procured only one for myself in rochester19:36
dimshaha. will do19:36
openstackgerritDan Smith proposed openstack/nova: Prepare for unicode enums from Oslo  https://review.openstack.org/20708719:36
*** rlrossit has left #openstack-nova19:41
*** annashen has joined #openstack-nova19:42
*** tjones has joined #openstack-nova19:42
dimsdansmith: mriedem: 15 fresh items in the quick hit list - https://etherpad.openstack.org/p/liberty-nova-priorities-tracking19:44
dimsjaypipes: ^^19:45
mriedemi wouldn't consider https://review.openstack.org/#/c/202616/ quick hit19:45
dimsmriedem: please add notes :) that one was added by ndipanov19:46
*** achanda has quit IRC19:46
mriedemthat's a no no19:46
*** krtaylor has quit IRC19:46
dimsmriedem: because?19:47
*** ByPasS has quit IRC19:47
*** belmoreira has quit IRC19:47
mriedemyou're not supposed to add your own changes to that list right?19:48
mriedemline 13319:48
*** Nikolay_St has joined #openstack-nova19:48
dimsah. right. i thought there was a technical concern :)19:48
mriedemwell, also that, it's a relatively complicated change19:48
mriedemat least the first part was19:48
*** HenryG has quit IRC19:49
mriedemnagyz_: so i see you're on a summit presentation about adding performance testing to tempest or something - i assume that's all out of tree?19:49
mriedemis it in public github at least?19:49
*** otter768 has joined #openstack-nova19:50
dimsmriedem: add your name to your color?19:50
*** salv-orlando has joined #openstack-nova19:51
*** HenryG has joined #openstack-nova19:52
*** ByPasS has joined #openstack-nova19:52
openstackgerritEarle F. Philhower, III proposed openstack/nova: Add os_brick-based VolumeDriver for HGST connector  https://review.openstack.org/19441219:53
*** otter768 has quit IRC19:54
*** meghal has joined #openstack-nova19:55
bauzasdims: sorry, I'm not angry but why did you changed the current reviews ?19:55
bauzasdims: tbc, I'm fine, just wondering why19:55
bauzasI mean the review requests19:55
*** vishwanathj has quit IRC19:57
dansmithmriedem: I get what you mean about it being complicated,19:57
dansmithmriedem: but it's really just making good on that comment I think19:57
dansmithmriedem: there is a lot of complexity around it, but we agreed we'd do this thing, so...19:58
*** krtaylor has joined #openstack-nova19:59
mriedemdansmith: you first19:59
*** meghal has quit IRC19:59
dansmithmriedem: hmm, I've seen this trap before19:59
dansmithmriedem: but, I just did :)19:59
*** meghal has joined #openstack-nova20:00
openstackgerritEarle F. Philhower, III proposed openstack/nova: Add os_brick-based VolumeDriver for HGST connector  https://review.openstack.org/19441220:00
mriedemdamn20:00
*** haomaiwa_ has quit IRC20:01
mriedemearlephilhower: i'm in the middle of hte last one20:01
*** meghal has quit IRC20:01
*** lpetrut has quit IRC20:01
*** meghal has joined #openstack-nova20:01
earlephilhowermriedem:  Howdy...didn't want to wait with cutoff <24hrs away.20:01
jaypipesdansmith: +W'd that PciDeviceList patch.20:01
*** edmondsw has quit IRC20:01
dansmithjaypipes: thanks homey20:01
mriedemearlephilhower: yeah, that's ok, i'll still -1 :)20:01
*** rlrossit has joined #openstack-nova20:02
*** haomaiwa_ has joined #openstack-nova20:02
*** fawadk has quit IRC20:02
*** thangp has quit IRC20:02
earlephilhowermriedem:  OK, no sweat.  I just pulled off your AOE splitoff so I'm kinda not sure what the prob is...20:03
jaypipesdansmith: no problem ma brutha.20:03
mriedemearlephilhower: oh you'll see20:04
mriedemyou'll see20:04
* earlephilhower imagines diabolical laughter...20:04
mriedemthat's implied20:05
mriedemmuwahhahahaa20:05
*** VW_ has quit IRC20:05
*** VW_ has joined #openstack-nova20:06
*** annashen_ has joined #openstack-nova20:06
*** busterswt has joined #openstack-nova20:06
mriedemearlephilhower: comments posted20:07
mriedemearlephilhower: this is kind of the biggy :) hgst=nova.virt.libvirt.volume.hgst.LibvirtQuobyteVolumeDriver20:07
*** ByPasS has quit IRC20:09
*** ByPasS has joined #openstack-nova20:09
openstackgerritDan Smith proposed openstack/nova: Add version counter to Service object  https://review.openstack.org/20173320:10
*** vishwanathj has joined #openstack-nova20:10
*** annashen_ has quit IRC20:11
dims"After reboot 285 times,can not nova reboot again"...20:11
dansmithawesome20:11
dimshttps://bugs.launchpad.net/nova/+bug/147845020:11
openstackLaunchpad bug 1478450 in OpenStack Compute (nova) "can not nova reboot vm" [Undecided,New]20:11
* earlephilhower applies head to desk20:12
*** baoli_ has quit IRC20:12
earlephilhowerfirkastirkadirka20:12
dimslol20:13
dimsnova20:13
dimsanyone remember nova/openstack/common/deadlock_monitor.py?20:15
earlephilhowermriedem:  So my CI's backlogged on the change, but I want to get it in so I ./run_tests.sh nova...test_hgst.  All good, commit!  Only thing is test is manually instantiating the right kind of object.  D'oh.20:15
mriedemearlephilhower: yeah, i guess we don't have a test that makes sure that all available volume drivers are registered properly, but we probably don't need one either20:16
*** meghal has quit IRC20:16
*** openstackgerrit has quit IRC20:16
*** meghal has joined #openstack-nova20:16
*** vladikr has quit IRC20:16
*** openstackgerrit has joined #openstack-nova20:17
openstackgerritxing-yang proposed openstack/nova: Add os-brick based LibvirtVolumeDriver for ScaleIO  https://review.openstack.org/19445420:17
*** annegentle has quit IRC20:17
openstackgerritEarle F. Philhower, III proposed openstack/nova: Add os_brick-based VolumeDriver for HGST connector  https://review.openstack.org/19441220:18
*** jasondotstar has quit IRC20:19
*** vishwanathj has quit IRC20:20
openstackgerritxing-yang proposed openstack/nova: Add os-brick based LibvirtVolumeDriver for ScaleIO  https://review.openstack.org/19445420:22
*** achanda has joined #openstack-nova20:25
*** xyang1 has joined #openstack-nova20:26
*** meghal has quit IRC20:27
*** lucas__ has quit IRC20:28
xyangmriedem: hi, can you please review the Scaleio patch again? https://review.openstack.org/#/c/194454/20:29
*** xyang1 has quit IRC20:29
rlrossitdansmith: got the most recent run done: http://logs.openstack.org/30/198730/16/check/gate-grenade-dsvm-partial-ncpu/4277d2a/logs/, and I found something suspicious...20:29
mriedemyeah, in a bit20:29
xyangmriedem: thanks!20:30
rlrossitdansmith: "Got IncompatibleObjectVersion, object name Instance at version 1.21 while processing InstanceList at version 1.16"20:30
*** y_sawai has quit IRC20:30
rlrossitInstanceList 1.16 should have Instance 1.19, and InstanceList 1.19 (the latest version which we are trying to convert) is at Instance 1.2120:31
rlrossitnot sure if it's always said that and I'm suddenly not stupid and realizing it20:31
*** lucas__ has joined #openstack-nova20:31
*** jasondotstar has joined #openstack-nova20:32
*** kaisers has quit IRC20:34
*** achanda has quit IRC20:34
*** achanda has joined #openstack-nova20:35
*** lucas__ has quit IRC20:35
*** tjones1 has joined #openstack-nova20:35
*** tjones2 has joined #openstack-nova20:36
*** salv-orlando has quit IRC20:36
*** armax has joined #openstack-nova20:39
*** armax has quit IRC20:39
*** tjones1 has quit IRC20:40
dansmithrlrossit: right so that means we're not actually converting Instance when we convert InstanceList, right?20:40
*** achanda has quit IRC20:41
*** davideagnello has joined #openstack-nova20:41
rlrossitdansmith: uhhh... sure?20:44
rlrossitdansmith: but it should be *trying* to convert Instance right?20:44
dansmithwell, presumably, but I'm not seeing any debug in your obj_make_compatible() to tell us that it's actually running20:45
dansmithso I wonder if it's something stupid and punting on the whole thing20:46
*** salv-orlando has joined #openstack-nova20:46
*** VW_ has quit IRC20:47
rlrossitdansmith: wait... who's supposed to be running obj_make_compatible()? conductor, right?20:47
*** VW_ has joined #openstack-nova20:47
dansmithyeah20:47
*** lucas__ has joined #openstack-nova20:48
rlrossitoh wait I'm dumb I was looking at ObjectListBase and saying it wasn't trying to convert Instance... so do I need to add logging to NovaObjects obj_make_compatible() now too?20:48
*** tjones has quit IRC20:49
dansmithwe know it's running L618 at least20:49
rlrossityeah but that's only for lists20:50
rlrossitwe can't see what's going on when it's trying to convert Instance20:50
rlrossitwhich if you think about it, should be out of my new stuff's hands20:50
dansmithright, but the list is what recurses20:50
*** edmondsw has joined #openstack-nova20:50
rlrossithrm20:50
dansmithso on L644, I'd want to know what child version we calculated for Instance20:50
dansmith(and ideally some indication that we went through the else on line 62520:51
*** tonytan4ever has quit IRC20:51
*** lucas__ has quit IRC20:51
dansmithand then something saying "I'm calling obj_to_primitive(instance, target_version=$foo)" on L64620:51
*** lucas__ has joined #openstack-nova20:51
dansmithand then dump the output of that to see that we actually converted it20:51
mriedemxyang: a few comments inline https://review.openstack.org/#/c/194454/20:51
*** VW_ has quit IRC20:51
dansmitheither, we're not converting it, or we're converting it to the wrong version, or we are doing both things and somehow not passing back the updated stack to the client20:52
xyangmriedem: let me check, thanks20:52
rlrossituhhh20:52
rlrossitI think I found the problem20:52
rlrossitL64620:52
dansmithwhat about it?20:53
rlrossitI should be doing self.objects[index] = self.objects[index].obj_make_compatible()?20:53
dansmithno20:53
dansmithobj_make_compatible() operates on primitive in-place20:53
dansmithso primitives[index] should be changed after that call20:53
rlrossitgotcha20:53
dansmithwhich is what you should print20:53
dansmithbut,20:53
*** dustins has joined #openstack-nova20:54
dansmithif we're ever copying that dict and returning the original and not the copy or something, then that would be same kind of result20:54
dansmithI don't think we're doing that, but answering the above 3-possibility question would tell us which thing to chase I thin20:54
dansmithrlrossit: I assume you didn't get this running locally, eh?20:55
*** Sukhdev_ has quit IRC20:55
*** ByPasS has quit IRC20:55
*** lucas__ has quit IRC20:55
rlrossitno because I couldn't figure out the zuul magic of getting my kilo change in old, and my liberty change in new20:56
*** ByPasS has joined #openstack-nova20:56
rlrossitthis is a dependency nightmare20:56
*** rmcall has joined #openstack-nova20:56
rlrossitunless I can tell grenade to not do anything20:56
*** dustins has left #openstack-nova20:56
xyangmriedem: sure, working on more tests.  So is the merge deadline by end of day tomorrow?20:56
rlrossitfor cloning20:56
dansmithrlrossit: it's easy20:56
rlrossit"easy"20:56
dansmithrlrossit: grenade has configs for that20:56
*** dustins has joined #openstack-nova20:56
rlrossitgrenade has too many configs20:56
dansmithrlrossit: you don't have to do "the zuul thing" you just have to tell grenade what you want20:57
dansmithmriedem: can't you help him?20:57
mriedemhelp rlrossit run grenade partial n-cpu locally?20:57
dansmithrlrossit: you can do this with just devstack too I think20:57
dansmithmriedem: yeah20:57
*** lucas__ has joined #openstack-nova20:57
mriedemdansmith: i've never tried20:57
*** angdraug has joined #openstack-nova20:58
mriedemxyang: yeah i think so20:58
*** meghal has joined #openstack-nova20:58
dansmithokay, well, I guess I'll just do it20:58
*** meghal has quit IRC20:58
xyangmriedem: ok, thanks20:58
*** tonytan4ever has joined #openstack-nova20:59
*** Nikolay_St has quit IRC21:00
*** haomaiwa_ has quit IRC21:01
*** lucas__ has quit IRC21:01
openstackgerritEarle F. Philhower, III proposed openstack/nova: Add os_brick-based VolumeDriver for HGST connector  https://review.openstack.org/19441221:02
*** 6JTABPDW5 has joined #openstack-nova21:02
*** nagyz_ has quit IRC21:02
*** nagyz_ has joined #openstack-nova21:03
openstackgerritEarle F. Philhower, III proposed openstack/nova: Add os_brick-based VolumeDriver for HGST connector  https://review.openstack.org/19441221:04
xyangmriedem: hemna says if I don't add path in filters, I'll need it in rootwrap.conf exe_dirs.  do you prefer the path to be added there or keep it in compute.filters?21:05
*** VW_ has joined #openstack-nova21:05
*** rdopiera has quit IRC21:08
*** dustins has quit IRC21:09
mriedemsec21:10
mriedemhttps://github.com/openstack/nova/blob/master/etc/nova/rootwrap.conf#L1321:10
mriedemah21:10
mriedemi don't really want it there no21:11
mriedemxyang: is this not in cinder somewhere? not seeing it in https://github.com/openstack/cinder/blob/master/etc/cinder/rootwrap.d/volume.filters21:11
openstackgerritDavanum Srinivas (dims) proposed openstack/nova: [WIP] default_schedule_zone should be list  https://review.openstack.org/20722121:11
mriedemxyang: the scaleio product doesn't create a link to /usr/bin or something for that executable?21:11
mriedemthat seems less than ideal21:12
openstackgerritRyan Rossiter proposed openstack/nova: Change List objects to use obj_relationships  https://review.openstack.org/19873021:12
*** ByPasS has quit IRC21:13
xyangI think I'll remove it from compute.filters21:13
xyangmriedem: we'll figure out to get a link created21:14
*** aysyd has quit IRC21:14
earlephilhowermriedem:  Sorry bout the mixup earlier, new shim version online.  Mock's pretty neat, much cleaner for the test.21:14
mriedemearlephilhower: welcome to the wonderful world of mock21:15
*** ekarlso has quit IRC21:16
mriedemxyang: well, this is a product that people have right?21:17
mriedemso probably not good to merge this in openstack if it won't work21:17
mriedemw/ older versions of that product21:17
*** ByPasS has joined #openstack-nova21:18
xyangmriedem: it works with what I have in compute.filters21:18
*** meghal has joined #openstack-nova21:18
mriedemxyang: does it work if you just have drv_cfg?21:18
xyangmriedem: if we remove the path, we need to document to add that in rootwrap.conf21:19
xyangmriedem: in the long term, we need to make sure the link is created automatically21:19
mriedemxyang: yeah, how does this work then? https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/emc/scaleio.py#L98921:20
mriedemi don't see anything in cinder's rootwrap filter21:20
xyangmriedem: let me check on our CI setup.  This is probably missed21:21
mriedemit's probably hacked into the ci setup to make it work :(21:21
*** tjones2 has quit IRC21:21
mriedemrather than having the filter in volume.filters21:21
*** angdraug has quit IRC21:24
*** tjones has joined #openstack-nova21:24
xyangmriedem: there are so many moving pieces to get this one in.  This one seems to be missed:(21:24
*** tjones1 has joined #openstack-nova21:25
mriedembusted! :)21:25
* mriedem opens cinder bug21:25
xyangmriedem: I'll a open a bug to get the cinder side fixed soon21:25
mriedemxyang: already doing that now21:26
xyangOh, already? :)21:26
mriedemyou bet21:26
*** tjones2 has joined #openstack-nova21:26
*** tjones1 has quit IRC21:26
*** tjones has quit IRC21:29
mriedemearlephilhower: congrats21:33
*** achanda has joined #openstack-nova21:34
earlephilhowermriedem:  Thanks!  I've also got an internal project that's supposed to close today, too.  Losing what little hair I've left.21:34
*** nic has quit IRC21:36
*** eglynn_ has quit IRC21:38
*** achanda has quit IRC21:39
*** annashen has quit IRC21:39
*** eglynn_ has joined #openstack-nova21:39
*** tjones2 has left #openstack-nova21:40
*** angdraug has joined #openstack-nova21:40
*** rmcall has quit IRC21:41
*** jasondotstar has quit IRC21:41
dansmithrlrossit: guess what21:42
dansmithrlrossit: http://pastebin.com/MVXsFbyB21:43
dansmithmakes it real easy to poke the problem21:43
*** exploreshaifali has quit IRC21:43
*** annashen has joined #openstack-nova21:43
*** ekarlso has joined #openstack-nova21:43
hemnaheh21:45
dansmithrlrossit: it's asking instance for version 1.2121:46
dansmithrlrossit: so something in your obj_relationships code is not picking the right child_target_version21:46
*** dboik has quit IRC21:46
*** tjones has joined #openstack-nova21:48
mriedemtest_ryan_is_lazy21:48
mriedemha21:48
dansmithrlrossit: you're missing a single break statement after L64321:50
dansmithrlrossit: which is a return in o.vo21:50
*** otter768 has joined #openstack-nova21:51
dansmithrlrossit: and it's also exactly what mriedem said he was waiting for .. that copied code to be the source of endless pain21:51
mriedemomfg21:51
*** ByPasS has quit IRC21:51
mriedemso,21:51
mriedemcan i just get a t-shirt made that says, 'i told you so!' in big print?21:51
* dansmith drops the mic and walks out21:52
dansmithmriedem: you may21:52
mriedemwell, i don't want to pay for that21:52
mriedemi'd like rlrossit to have it made21:52
mriedemlike that cupcake21:52
dansmithprobably reasonable at this point21:52
dansmithI want another cupcake for the realization that a <10 line unit test could poke the bug21:53
*** ByPasS has joined #openstack-nova21:53
dansmithit's also worth noting that edleafe suggested a break there purely for efficiency like eleventy billion patchsets ago21:53
mriedemi have a feeling that ryan has left the building21:54
dansmithmay be for the best21:54
mriedemi'm soon to follow21:54
dansmiththe amazon man comes in a few minutes and then I'm otherwise entertained for hours21:55
mriedemyou mean the ups guy?21:55
dansmithyeah21:55
*** otter768 has quit IRC21:55
openstackgerritDanny Wilson proposed openstack/nova: libvirt: add discard support for attached volumes  https://review.openstack.org/20572621:55
mriedemyou know he doesn't work directly for amazon right?21:55
mriedem:)21:55
dansmithpretty sure they'll rename themselves to "amazon delivery branch" pretty soon21:55
mriedemthey should21:55
mriedembut,21:55
mriedemdrones will replace them21:55
dansmithsurely will if they don't make smart strategic renames sooner than later, yeah21:56
*** cpschult has quit IRC21:56
rlrossitmriedem: dansmith: it's a good thing I chose to not turn off my laptop, and instead play video games with it sitting in the corner21:57
dansmithheh21:58
*** thorst has quit IRC21:58
*** READ10 has joined #openstack-nova21:59
*** burt has quit IRC21:59
*** dboik has joined #openstack-nova21:59
*** meghal has quit IRC22:00
*** 6JTABPDW5 has quit IRC22:01
rlrossitdansmith: so do I want the break in just the elif or do I need on in the if too? o.vo has it in both22:01
dansmithrlrossit: I think you want it only in the == case22:01
*** meghal has joined #openstack-nova22:01
*** meghal has quit IRC22:01
*** markvoelker has quit IRC22:02
*** meghal has joined #openstack-nova22:02
rlrossityep it's cranking in my head now, I get it22:02
*** 16WAAEHWP has joined #openstack-nova22:02
dansmithrlrossit: I dunno, I'd say write tests22:02
dansmithprove we don't need it (or do)22:02
*** Kennan2 has joined #openstack-nova22:02
rlrossitdansmith: but volleyball is in 30 minutes... I don't have time to right tests (right now)22:02
dansmithyep22:02
dansmithknowing the fix and not fixing it all night is the sweet pleasure you get22:03
openstackgerritsajeesh cs proposed openstack/nova: Nested Quota Driver: Create column allocated in Quota table  https://review.openstack.org/15132722:03
*** erlon has quit IRC22:03
dansmithi.e. that tomorrow will be productive and less head-slamming :)22:03
*** Kennan has quit IRC22:03
openstackgerritRuby Loo proposed openstack/nova: Instance destroyed if ironic node in CLEANWAIT  https://review.openstack.org/20723622:03
openstackgerritRyan Rossiter proposed openstack/nova: Change List objects to use obj_relationships  https://review.openstack.org/19873022:04
*** meghal has quit IRC22:04
rlrossitdansmith: does this mean I need to ship you another cupcake?22:04
*** READ10 has quit IRC22:04
*** mriedem has quit IRC22:04
dansmithrlrossit: I'm certainly not waiting until the next time I'm in rochester, but we can negotiate alternate payment and delivery arrangements later22:05
dansmiththe first cupcake was pretty good... not sure I'd be happy with the mess I'd receive via UPS22:05
*** apmelton has quit IRC22:06
*** annashen_ has joined #openstack-nova22:07
rlrossitdansmith: it's up and you can stare at zuul if you want, I'm personally sick of looking at it so I'm out. I'll probably check later tonight so I can sleep easy22:07
dansmithyep, all good22:07
*** rlrossit has left #openstack-nova22:07
*** apmelton has joined #openstack-nova22:08
*** achanda has joined #openstack-nova22:11
*** annashen_ has quit IRC22:12
*** busterswt has quit IRC22:12
*** d34dh0r53 has quit IRC22:13
*** ctrath has quit IRC22:13
*** VW__ has joined #openstack-nova22:13
*** d34dh0r53 has joined #openstack-nova22:13
*** LukeHinds has quit IRC22:14
*** chuckcarmack has left #openstack-nova22:15
*** d34dh0r53 has quit IRC22:15
*** lucas-dinner has quit IRC22:15
*** d34dh0r53 has joined #openstack-nova22:16
*** VW_ has quit IRC22:17
*** VW__ has quit IRC22:18
*** Sukhdev_ has joined #openstack-nova22:19
*** apmelton has quit IRC22:22
*** d34dh0r53 has quit IRC22:23
*** mjturek1 has quit IRC22:23
*** READ10 has joined #openstack-nova22:23
*** d34dh0r53 has joined #openstack-nova22:23
*** dboik has quit IRC22:25
*** apmelton has joined #openstack-nova22:25
*** d34dh0r53 has quit IRC22:27
*** d34dh0r53 has joined #openstack-nova22:27
*** jasondotstar has joined #openstack-nova22:28
edleafejroll: still around?22:28
*** armax has joined #openstack-nova22:29
jrolledleafe: yep22:29
jrollI'm always around >:)22:29
edleafejroll: are your sites using the ironic_host_manager? Could we simply override the bad behavior there?22:30
jrolledleafe: yeah, they are, I think it's a requirement for ironic22:31
jrollthat's a great point you have there :)22:31
edleafejroll: wasn't sure if it was required or just advisable :)22:31
edleafejroll: ok, lemme push something different22:32
*** d34dh0r53 has quit IRC22:32
*** d34dh0r53 has joined #openstack-nova22:33
jrolledleafe: yeah, it's required because IronicHostState22:33
*** ByPasS has quit IRC22:35
*** armax has quit IRC22:36
openstackgerritFei Long Wang proposed openstack/nova-specs: Add spec for volume backed backup  https://review.openstack.org/19091922:37
*** ByPasS has joined #openstack-nova22:37
*** mwagner_lap has quit IRC22:38
*** ijuwang has quit IRC22:40
*** ByPasS has quit IRC22:43
*** jaypipes has quit IRC22:43
*** ByPasS has joined #openstack-nova22:43
*** thorst has joined #openstack-nova22:45
*** raildo has quit IRC22:49
*** samueldmq has quit IRC22:49
*** ericksonsantos has quit IRC22:50
*** htruta has quit IRC22:50
*** ByPasS has quit IRC22:50
*** tpeoples has quit IRC22:51
*** ByPasS has joined #openstack-nova22:52
*** vishwanathj has joined #openstack-nova22:53
*** penick has joined #openstack-nova22:56
*** markvoelker_ has joined #openstack-nova22:57
*** eglynn_ has quit IRC22:58
*** ByPasS has quit IRC22:59
*** 16WAAEHWP has quit IRC23:01
openstackgerritEd Leafe proposed openstack/nova: Fix IronicHostManager to skip get_by_host() call.  https://review.openstack.org/20673623:01
*** haomaiwang has joined #openstack-nova23:02
edleafejroll: ^23:02
* jroll looks23:03
*** ByPasS has joined #openstack-nova23:04
*** jecarey has quit IRC23:04
*** alexpilotti has quit IRC23:05
*** ddaskal has quit IRC23:06
*** moshele has joined #openstack-nova23:06
*** ZZelle has quit IRC23:07
*** tonytan4ever has quit IRC23:12
*** gzhai has joined #openstack-nova23:13
*** gzhai has quit IRC23:14
*** gzhai has joined #openstack-nova23:15
*** diogogmt has quit IRC23:17
gzhaisdague: I changed according to you comments. Could you pls. take a look at https://review.openstack.org/#/c/203573/ ?23:18
*** thorst has quit IRC23:19
*** vishwanathj has quit IRC23:20
*** slong has joined #openstack-nova23:22
*** tpeoples has joined #openstack-nova23:22
openstackgerritFei Long Wang proposed openstack/nova: Fix nova backup for volume-backed instance  https://review.openstack.org/16449423:23
*** moshele has quit IRC23:24
*** annegentle has joined #openstack-nova23:28
*** annashen has quit IRC23:28
*** oomichi has joined #openstack-nova23:31
*** zhenguo has joined #openstack-nova23:31
*** ByPasS has quit IRC23:32
*** jasondotstar has quit IRC23:33
*** meghal has joined #openstack-nova23:34
*** ByPasS has joined #openstack-nova23:34
*** mwagner_lap has joined #openstack-nova23:37
*** obondarev has quit IRC23:38
*** obondarev has joined #openstack-nova23:39
*** jamielennox is now known as jamielennox|away23:39
*** penick has quit IRC23:39
openstackgerritTracy Jones proposed openstack/nova: Cleanup of compute api reboot method  https://review.openstack.org/12742723:41
*** tpeoples has quit IRC23:42
*** jasondotstar has joined #openstack-nova23:45
*** andrewbogott has left #openstack-nova23:46
*** armax has joined #openstack-nova23:47
*** rmcall has joined #openstack-nova23:47
*** otter768 has joined #openstack-nova23:51
*** Kennan2 is now known as Kennan23:53
*** otter768 has quit IRC23:56
melwittdansmith: what are the guidelines for creating an object from a primitive? when to use objects.<class>.obj_from_primitive vs objects.<class>(context, **primitive)23:57

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