Wednesday, 2017-01-04

EmilienMmwhahaha: i'm trying it now00:00
mwhahahamaybe we need to go dig into apache more but it seems odd to have this be the first service with such a configuration and from a deployer's standpoint it complicates things00:00
mwhahahajust some feedback on it00:00
dansmithbut it's not like it has to be on 80, right?00:01
mwhahahano it doesn't but it would be nice to have some sort of standard expectation around a standalone deployment if not 8000:02
EmilienMdansmith: we're *trying* to deploy this service in tripleo, with ssl activated. So we need to pick a port and 443 seems like the one you would choose.00:02
dansmithmwhahaha: I dunno, it seems kinda silly that we have any such defaults as it has only facilitated us avoiding using the service catalog properly in many places :)00:03
mwhahahai don't see how this is a fix for that at all, but ok00:04
*** baoli has quit IRC00:05
openstackgerritZhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail  https://review.openstack.org/40857100:08
*** dimtruck is now known as zz_dimtruck00:08
*** takashin has joined #openstack-nova00:08
*** salv-orlando has quit IRC00:14
*** zz_dimtruck is now known as dimtruck00:15
*** mlavalle has quit IRC00:22
*** jose-phillips has quit IRC00:25
*** jaugustine has quit IRC00:26
*** jaugustine has joined #openstack-nova00:26
*** chyka has quit IRC00:28
*** jaugustine has quit IRC00:30
*** Swami has quit IRC00:32
*** hongbin has quit IRC00:32
*** jwcroppe has joined #openstack-nova00:40
*** jwcroppe has quit IRC00:44
*** dimtruck is now known as zz_dimtruck00:44
*** jwcroppe has joined #openstack-nova00:45
*** claudiub|2 has quit IRC00:48
openstackgerritSTEW TY proposed openstack/nova: Transform instance.unrescue  notifications (WIP)  https://review.openstack.org/38827500:52
*** hoangcx has joined #openstack-nova00:53
*** thorst has joined #openstack-nova00:55
*** little has joined #openstack-nova00:56
*** phuongnh has joined #openstack-nova00:57
*** thorst has quit IRC00:58
openstackgerritMichael Still proposed openstack/nova: Use a service account to make vendordata requests.  https://review.openstack.org/41559700:59
openstackgerritMichael Still proposed openstack/nova: Optionally make dynamic vendordata failures fatal.  https://review.openstack.org/41639100:59
*** ijw has quit IRC01:01
*** Fdaisuke_ has joined #openstack-nova01:04
*** liusheng has joined #openstack-nova01:04
*** Fdaisuke has quit IRC01:05
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (3)  https://review.openstack.org/33576501:05
*** yingjun has joined #openstack-nova01:06
*** rama_y has quit IRC01:09
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (4)  https://review.openstack.org/33582001:12
*** trananhkma has joined #openstack-nova01:12
*** salv-orlando has joined #openstack-nova01:15
*** nic has quit IRC01:16
openstackgerritTakashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls  https://review.openstack.org/41387601:16
openstackgerritZhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail  https://review.openstack.org/41514201:18
openstackgerritHieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (5)  https://review.openstack.org/33592101:19
*** salv-orlando has quit IRC01:20
*** ijw has joined #openstack-nova01:22
*** unicell has joined #openstack-nova01:24
*** hfu has joined #openstack-nova01:27
*** ijw has quit IRC01:27
*** rfolco_ has joined #openstack-nova01:28
*** rfolco_ has quit IRC01:28
*** eliqiao has quit IRC01:29
*** claudiub|2 has joined #openstack-nova01:29
*** kevinz has joined #openstack-nova01:30
*** raunak has quit IRC01:30
*** rfolco_ has joined #openstack-nova01:31
*** rfolco_ has quit IRC01:31
openstackgerritLi Yingjun proposed openstack/nova: Notifications for flavor operations  https://review.openstack.org/39817101:33
*** eliqiao has joined #openstack-nova01:33
*** liverpooler has quit IRC01:40
*** ijw has joined #openstack-nova01:43
*** yuntongjin has joined #openstack-nova01:44
*** rfolco_ has joined #openstack-nova01:46
*** rfolco_ has quit IRC01:46
mriedemclaudiub|2: were you going to update this? https://review.openstack.org/#/c/248029/10/releasenotes/notes/hyper-v-set-boot-order-1e76b08ca6783add.yaml01:47
mriedemclaudiub|2: or did you just mean that gen 2 VMs don't support boot from ISO?01:48
mriedemjlvillal: that did the trick with the regex http://logs.openstack.org/23/415523/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial/15dd658/logs/screen-n-cpu.txt.gz?level=TRACE#_2017-01-04_00_19_32_51101:49
mriedemno longer an error01:49
*** smatzek has joined #openstack-nova01:49
*** yufei has joined #openstack-nova01:50
*** ijw has quit IRC01:50
*** dave-mccowan has joined #openstack-nova01:52
*** ijw has joined #openstack-nova01:52
*** thorst has joined #openstack-nova01:54
*** rfolco_ has joined #openstack-nova01:56
*** rfolco_ has quit IRC01:56
*** ijw has quit IRC01:57
mriedem+2 on jay's fix now01:57
*** claudiub|2 has quit IRC01:59
*** browne has quit IRC02:01
*** gongysh has joined #openstack-nova02:03
*** ijw has joined #openstack-nova02:05
*** dave-mcc_ has joined #openstack-nova02:06
*** ianw is now known as ianw_pto02:07
*** dave-mccowan has quit IRC02:08
*** ijw has quit IRC02:11
*** zz_dimtruck is now known as dimtruck02:12
*** trananhkma has quit IRC02:12
openstackgerritMatt Riedemann proposed openstack/nova: Make test_attach_interfaces work with use_neutron=True by default  https://review.openstack.org/41600102:14
openstackgerritMatt Riedemann proposed openstack/nova: Make test_nova_manage pass with CONF.use_neutron=True by default  https://review.openstack.org/41600302:14
openstackgerritMatt Riedemann proposed openstack/nova: Make test_user_data and test_multiple_create work with use_neutron=True  https://review.openstack.org/41572902:14
openstackgerritMatt Riedemann proposed openstack/nova: Make test_floating_ip* pass with CONF.use_neutron=True by default  https://review.openstack.org/41599902:14
openstackgerritMatt Riedemann proposed openstack/nova: Stub out os_vif.unplug in libvirt instance destroy tests  https://review.openstack.org/41600202:14
openstackgerritMatt Riedemann proposed openstack/nova: Make test_metadata pass with CONF.use_neutron=True by default  https://review.openstack.org/41600402:14
openstackgerritMatt Riedemann proposed openstack/nova: Make test_security_group_default_rules work with use_neutron=True by default  https://review.openstack.org/41576702:14
openstackgerritMatt Riedemann proposed openstack/nova: Make last remaining unit tests work with Neutron by default  https://review.openstack.org/41600602:14
openstackgerritMatt Riedemann proposed openstack/nova: Make test_virtual_interfaces work with CONF.use_neutron=True by default  https://review.openstack.org/41573202:14
openstackgerritMatt Riedemann proposed openstack/nova: Make test_server_usage work with CONF.use_neutron=True by default  https://review.openstack.org/41577002:14
openstackgerritMatt Riedemann proposed openstack/nova: Make several API unit tests pass with CONF.use_neutron=True by default  https://review.openstack.org/41577102:14
openstackgerritMatt Riedemann proposed openstack/nova: Make test_quota work with CONF.use_neutron=True by default  https://review.openstack.org/41570702:14
openstackgerritMatt Riedemann proposed openstack/nova: Make test_security_groups work with CONF.use_neutron=True by default  https://review.openstack.org/41574702:14
openstackgerritMatt Riedemann proposed openstack/nova: Enable Neutron by default  https://review.openstack.org/41608302:14
openstackgerritMatt Riedemann proposed openstack/nova: Make test_tenant_networks pass with CONF.use_neutron=True by default  https://review.openstack.org/41575502:14
openstackgerritMatt Riedemann proposed openstack/nova: Make test_compute pass with CONF.use_neutron=True by default  https://review.openstack.org/39997802:14
jlvillalmriedem, Thanks :)02:15
openstackgerritMatt Riedemann proposed openstack/nova: Enable Neutron by default  https://review.openstack.org/41608302:15
openstackgerritMatt Riedemann proposed openstack/nova: Make last remaining unit tests work with Neutron by default  https://review.openstack.org/41600602:15
*** salv-orlando has joined #openstack-nova02:16
*** unicell has quit IRC02:19
*** ljxiash has joined #openstack-nova02:20
*** ljxiash has quit IRC02:20
*** ljxiash has joined #openstack-nova02:21
*** thorst has quit IRC02:21
*** salv-orlando has quit IRC02:21
*** thorst has joined #openstack-nova02:21
*** smatzek has quit IRC02:22
*** thorst has quit IRC02:25
*** yuntongjin has quit IRC02:28
*** dave-mcc_ has quit IRC02:33
*** yuntongjin has joined #openstack-nova02:34
openstackgerritMatt Riedemann proposed openstack/nova: Add support matrix for attach and detach interfaces  https://review.openstack.org/40996702:36
*** dave-mccowan has joined #openstack-nova02:38
*** _Fdaisuke_ has joined #openstack-nova02:39
mtreinishmriedem: ^^^ did johnthetubaguy ever get that mapping to tempest tests (or config options) started?02:40
mriedemmtreinish: there are patches...and a bp02:41
mriedemsec02:41
mriedemmtreinish: https://blueprints.launchpad.net/nova/+spec/feature-classification-ocata02:41
mriedemi think raj_singh and other people from OSIC have been working on it moastly02:41
mriedem*mostly02:41
mriedemand sfinucan02:41
mtreinishcool, I'll take a look02:42
*** Fdaisuke_ has quit IRC02:42
*** thorst has joined #openstack-nova02:47
*** rama_y has joined #openstack-nova02:49
*** thorst has quit IRC02:49
*** moshele has joined #openstack-nova02:50
*** trananhkma has joined #openstack-nova02:54
*** tbachman_ has joined #openstack-nova02:56
*** tbachman has quit IRC02:57
*** tbachman_ is now known as tbachman02:57
*** dave-mccowan has quit IRC03:01
openstackgerritzhangyujun proposed openstack/nova: Update IronicDriver return  hypervisor_type  https://review.openstack.org/37996803:04
*** ducttape_ has quit IRC03:04
*** ducttape_ has joined #openstack-nova03:05
*** ljxiash has quit IRC03:06
*** ljxiash has joined #openstack-nova03:06
*** ducttape_ has quit IRC03:09
*** tbachman_ has joined #openstack-nova03:10
*** ljxiash has quit IRC03:11
*** tbachman has quit IRC03:11
*** tbachman_ is now known as tbachman03:11
*** raunak has joined #openstack-nova03:12
*** hongbin has joined #openstack-nova03:17
mriedemalex_xu: https://review.openstack.org/#/c/389003/ mostly looks good, i just left some suggestions for clarifications in some of the docstrings03:18
*** cfriesen has joined #openstack-nova03:21
*** mriedem has quit IRC03:22
*** coreywright has quit IRC03:26
*** nklenke has quit IRC03:33
*** thorst has joined #openstack-nova03:35
*** prateek has quit IRC03:36
*** prateek has joined #openstack-nova03:36
openstackgerritHuan Xie proposed openstack/nova: XenAPI Use os-xenapi lib for nova  https://review.openstack.org/40605903:36
*** thorst has quit IRC03:36
*** mvk has quit IRC03:38
openstackgerritHuan Xie proposed openstack/nova: XenAPI Remove useless files when use os-xenapi lib  https://review.openstack.org/41124103:40
*** prateek has quit IRC03:45
*** sudipto has joined #openstack-nova03:45
*** sudipto_ has joined #openstack-nova03:45
*** stewie925 has quit IRC03:46
*** nklenke has joined #openstack-nova03:48
*** moshele has quit IRC03:49
*** liverpooler has joined #openstack-nova03:50
*** jamielennox is now known as jamielennox|away03:51
*** yuntongjin has quit IRC03:52
*** links has joined #openstack-nova03:56
*** jamielennox|away is now known as jamielennox03:59
*** dimtruck is now known as zz_dimtruck04:00
*** ijw has joined #openstack-nova04:01
*** ducttape_ has joined #openstack-nova04:02
*** zz_dimtruck is now known as dimtruck04:04
*** raunak has quit IRC04:06
*** tojuvone has quit IRC04:08
*** baoli has joined #openstack-nova04:10
*** yingjun has quit IRC04:10
*** cfriesen has quit IRC04:13
*** cfriesen_ has joined #openstack-nova04:13
*** rama_y has quit IRC04:14
*** baoli has quit IRC04:15
*** psachin has joined #openstack-nova04:16
*** sreeram_v has joined #openstack-nova04:17
*** salv-orlando has joined #openstack-nova04:17
*** ljxiash has joined #openstack-nova04:19
*** tjones has quit IRC04:19
*** salv-orlando has quit IRC04:21
*** hongbin has quit IRC04:22
*** ducttape_ has quit IRC04:25
*** ducttape_ has joined #openstack-nova04:26
*** tlian2 has quit IRC04:26
*** ducttape_ has joined #openstack-nova04:26
openstackgerritDao Cong Tien proposed openstack/nova: Remove mox from api/openstack/compute/test_deferred_delete.py  https://review.openstack.org/32034604:27
*** sudipto_ has quit IRC04:27
*** sudipto has quit IRC04:27
*** bkopilov has quit IRC04:28
*** thorst has joined #openstack-nova04:37
*** bkopilov has joined #openstack-nova04:40
*** thorst has quit IRC04:41
openstackgerritTuan Luong-Anh proposed openstack/nova: Ironic: use portgroups  https://review.openstack.org/20616304:42
*** nicolasbock has joined #openstack-nova04:42
*** adisky_ has joined #openstack-nova04:52
*** coreywright has joined #openstack-nova04:57
*** unicell has joined #openstack-nova05:01
*** raunak has joined #openstack-nova05:04
*** ayogi has joined #openstack-nova05:04
*** sudipto has joined #openstack-nova05:08
*** sudipto_ has joined #openstack-nova05:08
*** mdnadeem has joined #openstack-nova05:10
*** ratailor has joined #openstack-nova05:11
*** raunak has quit IRC05:18
*** yuntongjin has joined #openstack-nova05:19
*** raunak has joined #openstack-nova05:19
*** raunak has quit IRC05:20
openstackgerritint32bit proposed openstack/python-novaclient: Allow multiple tag add/delete from cli  https://review.openstack.org/41463505:23
*** ijw has quit IRC05:25
*** pbandark has joined #openstack-nova05:35
*** gouthamr has joined #openstack-nova05:37
*** moshele has joined #openstack-nova05:42
*** prateek has joined #openstack-nova05:44
*** kristian__ has joined #openstack-nova05:47
*** mvk has joined #openstack-nova05:50
*** yingjun has joined #openstack-nova05:50
*** kristian__ has quit IRC05:52
openstackgerritWang Qing wu proposed openstack/nova: Introduce base PowerVM driver  https://review.openstack.org/39128805:54
openstackgerritDan Peschman proposed openstack/nova: Option to always create network metadata information as static  https://review.openstack.org/33629305:59
*** kristian__ has joined #openstack-nova06:00
*** kristian__ has quit IRC06:00
*** ekuris has joined #openstack-nova06:02
*** gouthamr has quit IRC06:06
*** diga has joined #openstack-nova06:06
*** moshele has quit IRC06:08
*** dimtruck is now known as zz_dimtruck06:08
openstackgerritDao Cong Tien proposed openstack/nova: Remove mox from api/openstack/compute/test_floating_ip_pools.py  https://review.openstack.org/32262406:09
*** mvk has quit IRC06:11
*** coreywright has quit IRC06:13
openstackgerritDao Cong Tien proposed openstack/nova: Remove mox from unit/api/openstack/compute/test_cloudpipe.py  https://review.openstack.org/32142006:17
openstackgerritZhenyu Zheng proposed openstack/nova: [WIP]Add new policy for server list/detail with all_tenants  https://review.openstack.org/41533006:18
*** amotoki has joined #openstack-nova06:22
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix parameters in os-hosts.inc  https://review.openstack.org/41644406:23
openstackgerritAlex Xu proposed openstack/nova: Adds support for versioned schema validation for query parameters  https://review.openstack.org/38900306:23
*** sridharg has joined #openstack-nova06:31
openstackgerritLi Yingjun proposed openstack/nova: Notifications for flavor operations  https://review.openstack.org/39817106:32
openstackgerritDao Cong Tien proposed openstack/nova: Stop using mox in compute/test_server_actions.py  https://review.openstack.org/32318606:32
*** claudiub|2 has joined #openstack-nova06:35
*** Kevin_Zheng has joined #openstack-nova06:35
*** lpetrut has joined #openstack-nova06:36
*** thorst has joined #openstack-nova06:38
*** tojuvone has joined #openstack-nova06:39
*** satyar has joined #openstack-nova06:41
*** thorst has quit IRC06:42
*** salv-orlando has joined #openstack-nova06:43
openstackgerritDao Cong Tien proposed openstack/nova: Remove mox from api/openstack/compute/test_extended_hypervisors.py  https://review.openstack.org/32146906:50
*** zz_dimtruck is now known as dimtruck06:51
*** pcaruana has joined #openstack-nova06:51
*** bkopilov has quit IRC06:52
*** dimtruck is now known as zz_dimtruck06:57
*** nkrinner_afk has quit IRC06:59
*** nkrinner has joined #openstack-nova06:59
*** mjura has joined #openstack-nova07:12
*** rcernin has joined #openstack-nova07:15
*** tesseract has joined #openstack-nova07:18
*** cfriesen_ has quit IRC07:19
*** moshele has joined #openstack-nova07:27
*** lpetrut has quit IRC07:27
*** coreywright has joined #openstack-nova07:28
*** mnasiadka has joined #openstack-nova07:29
openstackgerritTakashi NATSUME proposed openstack/nova: api-ref: Fix parameters and response in os-quota-sets.inc  https://review.openstack.org/41646107:32
*** bkopilov has joined #openstack-nova07:33
*** kristian__ has joined #openstack-nova07:37
*** salv-orlando has quit IRC07:39
*** kristian__ has quit IRC07:39
*** kristian__ has joined #openstack-nova07:39
*** kristian__ has quit IRC07:42
*** yingjun has quit IRC07:49
*** yingjun has joined #openstack-nova07:50
*** sahid has joined #openstack-nova07:50
*** tbachman has quit IRC07:53
*** haplo37 has quit IRC07:57
*** gongysh_ has joined #openstack-nova07:59
*** gongysh has quit IRC08:00
openstackgerritjichenjc proposed openstack/nova: correct misleading wording  https://review.openstack.org/41365808:03
*** takashin has left #openstack-nova08:03
openstackgerritZhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants  https://review.openstack.org/41533008:03
*** haplo37 has joined #openstack-nova08:06
Kevin_Zhengping alex_xu08:19
*** markus_z has joined #openstack-nova08:19
*** rmart04 has joined #openstack-nova08:21
*** nandal has joined #openstack-nova08:22
alex_xuKevin_Zheng: pong08:25
Kevin_Zhengdidnt get your point here https://review.openstack.org/#/c/408571/24/nova/tests/unit/test_api_validation.py08:26
Kevin_Zhengthe second comment08:26
Kevin_Zhengcan you help me out?08:26
alex_xuKevin_Zheng: it is related to this note https://github.com/openstack/nova/blob/df2fd4a252cecc1e1ef471c071e57526ddf65499/nova/api/validation/__init__.py#L4408:27
*** little has quit IRC08:27
alex_xuKevin_Zheng: we should put the 'req' into the position argument08:28
*** hfu has quit IRC08:29
*** little has joined #openstack-nova08:34
*** hfu has joined #openstack-nova08:35
*** gongysh_ has quit IRC08:37
*** thorst has joined #openstack-nova08:39
*** thorst has quit IRC08:43
*** tuanluong has joined #openstack-nova08:45
*** salv-orlando has joined #openstack-nova08:49
*** kristian__ has joined #openstack-nova08:50
*** gongysh has joined #openstack-nova08:51
openstackgerritAlex Xu proposed openstack/nova: Use JSON-Schema to validate query parameters for keypairs API  https://review.openstack.org/39875508:53
openstackgerritAlex Xu proposed openstack/nova: Adds support for versioned schema validation for query parameters  https://review.openstack.org/38900308:53
*** CristinaPauna has joined #openstack-nova08:54
*** zzzeek has quit IRC09:00
*** pbandark has quit IRC09:00
*** zzzeek has joined #openstack-nova09:00
*** derekh has joined #openstack-nova09:04
openstackgerritTaku Izumi proposed openstack/nova: virt/ironic: Implement rescue and unrescue  https://review.openstack.org/41648709:10
*** ababich has quit IRC09:13
*** yassine has joined #openstack-nova09:13
*** yassine is now known as Guest7208709:14
*** lpetrut has joined #openstack-nova09:21
*** lucas-afk is now known as lucasagomes09:31
*** yuntongjin has quit IRC09:31
openstackgerritTaku Izumi proposed openstack/nova: virt/ironic: Implement rescue and unrescue  https://review.openstack.org/41648709:32
*** hfu has quit IRC09:37
openstackgerritClaudiu Belu proposed openstack/nova: Adds Hyper-V OVS ViF driver  https://review.openstack.org/14004509:39
*** hfu has joined #openstack-nova09:40
*** yingjun has quit IRC09:42
littlecould anyone review my patch? https://review.openstack.org/#/c/414463/09:46
*** hfu has quit IRC09:47
*** salv-orl_ has joined #openstack-nova09:53
*** kristian__ has quit IRC09:54
*** salv-orlando has quit IRC09:56
*** kristian__ has joined #openstack-nova09:56
*** pbandark has joined #openstack-nova10:00
*** pbandark has quit IRC10:06
*** pbandark has joined #openstack-nova10:06
*** moshele has quit IRC10:06
*** moshele has joined #openstack-nova10:06
*** sudipto has quit IRC10:08
*** sudipto_ has quit IRC10:08
*** tobberydberg_ has joined #openstack-nova10:09
*** tobberydberg_ is now known as tobberydberg10:12
*** yufei has quit IRC10:15
*** ljxiash has quit IRC10:16
*** ljxiash has joined #openstack-nova10:16
*** ljxiash has quit IRC10:18
*** ljxiash has joined #openstack-nova10:18
*** yufei has joined #openstack-nova10:18
*** yufei has quit IRC10:18
*** mvk has joined #openstack-nova10:19
*** tuanluong has quit IRC10:21
*** kevinz has quit IRC10:23
*** satyar has quit IRC10:28
*** ociuhandu has quit IRC10:31
*** hoangcx has quit IRC10:35
*** armax has joined #openstack-nova10:36
*** thorst has joined #openstack-nova10:39
*** yamahata has quit IRC10:42
*** thorst has quit IRC10:44
openstackgerritLee Yarwood proposed openstack/nova-specs: WIP - BDM detach refactor  https://review.openstack.org/41650810:50
openstackgerritLee Yarwood proposed openstack/nova-specs: WIP - BDM detach refactor  https://review.openstack.org/41650810:51
*** tobberydberg has quit IRC10:53
*** cdent has joined #openstack-nova10:57
*** links has quit IRC10:57
*** sdague has joined #openstack-nova11:04
openstackgerritStephen Finucane proposed openstack/nova: move gate hooks to gate/  https://review.openstack.org/37895211:05
openstackgerritStephen Finucane proposed openstack/nova: tools: Remove 'enable-pre-commit-hook'  https://review.openstack.org/41651211:05
openstackgerritStephen Finucane proposed openstack/nova: tools: Remove 'clean-vlans'  https://review.openstack.org/41651311:05
openstackgerritStephen Finucane proposed openstack/nova: tools: Remove 'install_venv', 'install_venv_common'  https://review.openstack.org/41651411:05
*** salv-orl_ has quit IRC11:05
openstackgerritStephen Finucane proposed openstack/nova: tools: Remove 'with_venv'  https://review.openstack.org/41651511:05
openstackgerritStephen Finucane proposed openstack/nova: tools: Remove 'colorizer'  https://review.openstack.org/41651611:05
openstackgerritStephen Finucane proposed openstack/nova: tools: Remove +x flag from scripts  https://review.openstack.org/41651711:05
*** tbachman has joined #openstack-nova11:09
openstackgerritStephen Finucane proposed openstack/nova: tools/xenserver: Remove 'cleanup_sm_locks'  https://review.openstack.org/41652011:11
openstackgerritmelanie witt proposed openstack/nova: WIP Make Quotas object favor the API database  https://review.openstack.org/41094511:12
openstackgerritmelanie witt proposed openstack/nova: WIP Add online migration to move quotas to API database  https://review.openstack.org/41094611:12
openstackgerritmelanie witt proposed openstack/nova: WIP Count resources to check quota for cells  https://review.openstack.org/41652111:12
openstackgerritgaryk proposed openstack/nova: VMware: retry network lookup if network not ready  https://review.openstack.org/39177811:18
*** phuongnh has quit IRC11:20
*** ljxiash has quit IRC11:20
*** links has joined #openstack-nova11:21
*** gongysh has quit IRC11:21
openstackgerritLee Yarwood proposed openstack/nova: DNM - encryptors: Switch to os-brick encryptor classes  https://review.openstack.org/39159711:25
*** rfolco has joined #openstack-nova11:25
*** ducttape_ has quit IRC11:25
*** salv-orlando has joined #openstack-nova11:28
*** sreeram_v has quit IRC11:30
*** sreeram_v has joined #openstack-nova11:31
*** sreeram_v has quit IRC11:35
*** nandal has quit IRC11:36
*** ducttape_ has joined #openstack-nova11:36
*** claudiub|2 has quit IRC11:36
*** smatzek has joined #openstack-nova11:40
sfinucanraj_singh: Could you take a second look at https://review.openstack.org/#/c/397774/ ?11:45
*** ljxiash has joined #openstack-nova11:46
openstackgerritMoshe Levi proposed openstack/os-vif: OVS acceleration  https://review.openstack.org/39827711:47
*** hfu has joined #openstack-nova11:49
*** ljxiash has quit IRC11:50
*** hfu has quit IRC11:50
*** raunak has joined #openstack-nova11:54
*** ducttape_ has quit IRC11:54
*** diga has quit IRC11:55
rfolcocdent, around?11:59
cdentrfolco: yup11:59
rfolcocdent, I read the microversions spec, and still have questions on how to define a new microversion and reno for the new DELETE method.12:00
*** sreeram_v has joined #openstack-nova12:00
*** liusheng has quit IRC12:01
rfolcocdent, 2.40 is the latest in the history, so do I need a 2.41 for the new API call?12:01
cdentthe placement api uses a different history from the nova-compute api12:01
cdentso it's microversions are managed in nova/api/openstack/placement/rest_api_version_history.rst12:02
rfolcooh that explains why I couldn't find any resource_provider example there12:02
cdentand the canonical version list is in nova/api/openstack/placement/microversion.py12:03
cdentsome of this stuff is explained in this not yet merged review: https://review.openstack.org/#/c/411946/2/doc/source/placement_dev.rst12:03
*** claudiub|2 has joined #openstack-nova12:04
*** karthiks has quit IRC12:11
*** catintheroof has joined #openstack-nova12:14
*** ljxiash has joined #openstack-nova12:19
*** bvanhav_ has joined #openstack-nova12:21
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port virt.libvirt.test_imagebackend to Python 3  https://review.openstack.org/40343112:26
*** rtheis has joined #openstack-nova12:26
*** kristian__ has quit IRC12:26
*** liusheng has joined #openstack-nova12:29
*** cdelatte has joined #openstack-nova12:30
*** rtheis_ has joined #openstack-nova12:30
*** openstackgerrit has quit IRC12:33
*** rtheis has quit IRC12:33
*** openstackgerrit has joined #openstack-nova12:33
openstackgerritmelanie witt proposed openstack/nova: WIP Count resources to check quota for cells  https://review.openstack.org/41652112:33
openstackgerritmelanie witt proposed openstack/nova: WIP Make Quotas object favor the API database  https://review.openstack.org/41094512:33
openstackgerritmelanie witt proposed openstack/nova: WIP Add online migration to move quotas to API database  https://review.openstack.org/41094612:33
*** sudipto_ has joined #openstack-nova12:33
*** sudipto has joined #openstack-nova12:33
*** raunak has quit IRC12:34
*** raunak has joined #openstack-nova12:34
*** gabor_antal has joined #openstack-nova12:36
*** gabor_antal_ has joined #openstack-nova12:36
johnthetubaguymacsz: do we need to backport this one: https://review.openstack.org/#/c/41258912:37
openstackgerritClaudiu Belu proposed openstack/nova: Hyper-V: Adds vNUMA implementation  https://review.openstack.org/28240712:38
*** gabor_antal_ has quit IRC12:38
*** lucasagomes is now known as lucas-hungry12:39
*** rtheis has joined #openstack-nova12:40
*** thorst has joined #openstack-nova12:41
*** jaypipes has joined #openstack-nova12:42
*** rtheis_ has quit IRC12:42
openstackgerritZhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail  https://review.openstack.org/40857112:43
alex_xunova api meeting is in 15 mins at #openstack-meeting-412:46
*** salv-orlando has quit IRC12:56
*** weshay is now known as weshay_doctor12:58
*** psachin has quit IRC13:00
*** psachin has joined #openstack-nova13:00
alex_xunova api meeting is running13:00
alex_xujohnthetubaguy: sdague nova api meeting is running if you are around13:01
*** moshele has quit IRC13:03
*** rmart04_ has joined #openstack-nova13:05
*** sapcc-bot1 has quit IRC13:06
*** sapcc-bot has joined #openstack-nova13:06
*** david_1 has joined #openstack-nova13:06
*** carthaca_ has joined #openstack-nova13:06
*** dgonzalez_ has joined #openstack-nova13:06
*** rmart04 has quit IRC13:06
*** rmart04_ is now known as rmart0413:06
*** salv-orlando has joined #openstack-nova13:07
*** david_1 has quit IRC13:08
*** carthaca_ has quit IRC13:08
*** dgonzalez_ has quit IRC13:08
*** dave-mccowan has joined #openstack-nova13:08
*** moshele has joined #openstack-nova13:14
*** mnasiadka has quit IRC13:14
*** rmart04_ has joined #openstack-nova13:14
*** rmart04 has quit IRC13:15
*** rmart04_ is now known as rmart0413:15
*** edmondsw has joined #openstack-nova13:16
*** ayogi has quit IRC13:21
openstackgerritZhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail  https://review.openstack.org/40857113:24
*** jheroux has joined #openstack-nova13:27
*** tongli has joined #openstack-nova13:28
*** raunak has quit IRC13:28
*** liverpooler has quit IRC13:30
*** liverpooler has joined #openstack-nova13:31
*** ratailor has quit IRC13:31
*** ratailor has joined #openstack-nova13:32
*** sreeram_v has quit IRC13:33
*** lamt has joined #openstack-nova13:33
*** sreeram_v has joined #openstack-nova13:34
*** diga has joined #openstack-nova13:34
*** mdrabe has joined #openstack-nova13:36
*** sreeram_v has quit IRC13:38
*** Cristina_ has joined #openstack-nova13:40
*** prateek has quit IRC13:41
*** nklenke has quit IRC13:41
*** CristinaPauna has quit IRC13:44
*** efried has quit IRC13:44
*** nklenke has joined #openstack-nova13:46
*** mdnadeem has quit IRC13:50
*** lucas-hungry is now known as lucasagomes13:51
*** pbandark has quit IRC13:51
*** tlian has joined #openstack-nova13:53
openstackgerritEd Leafe proposed openstack/nova: Do not post allocations that are zero  https://review.openstack.org/40718013:54
*** ljxiash has quit IRC13:55
*** sridharg has quit IRC13:55
*** ratailor has quit IRC13:55
cdentjohnthetubaguy: are rpc calls to the compute node's actually expensive (in the sense of we've got hard numbers) enough to be damaging, or something that we talk about as expensive because there are several things that combine to make slowness an issue?13:58
johnthetubaguycdent: its more just the act of waiting for the RPC call reply13:58
cdentwhich presumably is unpredictable?13:59
johnthetubaguycdent: I guess, yeah13:59
*** diga has quit IRC14:00
*** bvanhav__ has joined #openstack-nova14:00
*** bvanhav__ is now known as bvanhav14:00
*** jwcroppe has quit IRC14:00
*** bvanhav_ has quit IRC14:00
*** Nakato has quit IRC14:00
johnthetubaguycdent: scaling wise, I don't really like having to scale up rabbit when you get more API calls, only want to scale that up when you get more instances/hypervisors, at least ideally14:00
*** Nakato has joined #openstack-nova14:00
johnthetubaguybut thats largely make believe I guess14:00
*** baoli has joined #openstack-nova14:09
*** pradk has quit IRC14:09
*** baoli has quit IRC14:09
*** eharney has joined #openstack-nova14:10
*** baoli has joined #openstack-nova14:10
*** tblakes has joined #openstack-nova14:10
*** gibi has quit IRC14:11
sdaguealex_xu: do we care in https://review.openstack.org/#/c/408571/26/nova/api/openstack/compute/schemas/servers.py that ip6 doesn't show up until a later microversion?14:12
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: [WIP] Refactor _post_live_migration  https://review.openstack.org/41302114:13
openstackgerritSivasathurappan Radhakrishnan proposed openstack/nova: [WIP] Add version field to LibvirtLiveMigrateData  https://review.openstack.org/40460714:13
*** zul has quit IRC14:14
macszjohnthetubaguy: there is no need, the change that introduced this was merged after newton release14:15
*** zul has joined #openstack-nova14:15
johnthetubaguymacsz: awesome, thanks for checking14:15
macszjohnthetubaguy: np :)14:15
*** salv-orlando has quit IRC14:17
*** salv-orlando has joined #openstack-nova14:17
*** jamesdenton has joined #openstack-nova14:19
*** hwoarang has quit IRC14:19
*** hwoarang has joined #openstack-nova14:20
*** kevinz has joined #openstack-nova14:20
*** ducttape_ has joined #openstack-nova14:20
*** jwcroppe has joined #openstack-nova14:21
*** smatzek has quit IRC14:21
*** karthiks has joined #openstack-nova14:22
*** weshay_doctor is now known as weshay14:23
*** cfriesen_ has joined #openstack-nova14:24
*** cfriesen_ has quit IRC14:25
*** psachin has quit IRC14:25
*** salv-orlando has quit IRC14:27
*** kevinz has quit IRC14:36
*** kevinz has joined #openstack-nova14:36
*** satyar has joined #openstack-nova14:36
*** efried has joined #openstack-nova14:40
openstackgerritSean Dague proposed openstack/nova: Add aggregate notification related enum values  https://review.openstack.org/39620914:44
*** mnestratov has joined #openstack-nova14:44
*** cleong has joined #openstack-nova14:46
*** edand has joined #openstack-nova14:47
*** kevinz has quit IRC14:47
*** kevinz has joined #openstack-nova14:47
*** baoli has quit IRC14:52
*** Jinxing has joined #openstack-nova14:53
*** smatzek has joined #openstack-nova14:56
*** baoli has joined #openstack-nova14:58
*** paw has quit IRC14:59
*** links has quit IRC14:59
*** yufei has joined #openstack-nova15:00
*** xyang1 has joined #openstack-nova15:00
gabor_antalHey, would someone please review this patch?  https://review.openstack.org/#/c/411816/15:03
*** hongbin has joined #openstack-nova15:03
*** rmart04 has quit IRC15:03
*** pradk has joined #openstack-nova15:04
*** mriedem has joined #openstack-nova15:05
*** baoli has quit IRC15:05
*** rmart04 has joined #openstack-nova15:06
* mriedem has frozen fingers15:08
*** baoli has joined #openstack-nova15:08
openstackgerritMatt Riedemann proposed openstack/nova: Enable Neutron by default  https://review.openstack.org/41608315:09
*** corey_ has joined #openstack-nova15:09
*** corey_ is now known as Guest4114715:09
*** mnestratov|2 has joined #openstack-nova15:10
*** GordonTX has joined #openstack-nova15:10
*** mdnadeem has joined #openstack-nova15:10
*** cleong has quit IRC15:11
*** zz_dimtruck is now known as dimtruck15:12
*** tbachman_ has joined #openstack-nova15:12
*** armax_ has joined #openstack-nova15:12
*** tbachman has quit IRC15:13
*** armax has quit IRC15:13
*** haplo37 has quit IRC15:13
*** zzzeek has quit IRC15:13
*** mnestratov has quit IRC15:13
*** armax_ is now known as armax15:13
*** tbachman_ is now known as tbachman15:13
johnthetubaguymriedem: is that a disney themed snack?15:13
mriedemsdague: sfinucan: thanks for going through that use_neutron series15:13
mriedemjohnthetubaguy: touche15:14
mriedembut not this time15:14
*** zzzeek has joined #openstack-nova15:14
*** cfriesen has joined #openstack-nova15:14
sdaguemriedem: no prob, I'm still chugging up the stack right now15:14
mriedemjohnthetubaguy: it's -25 F with wind chill here today15:14
mriedemkeeps out the riff-raff15:14
johnthetubaguymriedem: thats proper nasty15:14
johnthetubaguythe temperature that is :)15:14
*** haplo37 has joined #openstack-nova15:14
*** tongli has quit IRC15:14
*** kevinz has quit IRC15:15
kashyapmriedem: Holy moly, -25...15:15
kashyapHere I am trying to keep myself warm without heating from -115:15
mriedemcoffee helps15:16
dansmithmriedem: wow.. it's 24F here now, forecast for 17 tonight and we're all raising hell about it15:16
edleafemriedem: Yeah, it's like the arctic here in San Antonio - just under 50F!15:16
mriedemdansmith: it's a good thing all those hipsters have beards and stocking caps year round15:16
dansmith7F with the wind chill I guess15:16
sfinucanmriedem: np. I'll grab the last one once Jenkins is happy15:16
dansmithmriedem: heh15:17
mriedemedleafe: my wife was in phoenix last week complaining about it being about that temp15:17
* johnthetubaguy doesn't do hot or cold really15:17
*** Jinxing has quit IRC15:17
edleafemriedem: yeah, having to wear a light hoodie is too much to bear15:18
mriedembauzas: are you around?15:23
*** ijw has joined #openstack-nova15:23
bauzasmriedem: indeed15:23
bauzasjust doing lots of internal stuff15:23
mriedembauzas: was wondering where you're at with https://blueprints.launchpad.net/nova/+spec/resource-providers-scheduler-db-filters15:24
bauzasmriedem: the API change is waiting for reviews and I'm working on implementing the scheduler change15:24
bauzaswhich is the only change needed for ^15:25
openstackgerritsahid proposed openstack/nova: numa: add numa constraints for emulator threads policy  https://review.openstack.org/38502315:25
openstackgerritsahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested  https://review.openstack.org/38536415:25
openstackgerritsahid proposed openstack/nova: objects: add new field cpus_overhead in NUMACell  https://review.openstack.org/40087415:25
openstackgerritsahid proposed openstack/nova: numa: fit instance toplogy with cpu overhead onto cores  https://review.openstack.org/40087515:25
openstackgerritsahid proposed openstack/nova: virt: update overhead to take into account vCPUs  https://review.openstack.org/38502415:25
openstackgerritsahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU  https://review.openstack.org/40087815:25
openstackgerritsahid proposed openstack/nova: numa: take into account cpus overhead  https://review.openstack.org/40087615:25
openstackgerritsahid proposed openstack/nova: numa: update numa usage incording the cpu overhead  https://review.openstack.org/40087715:25
mriedembauzas: ok i knew about https://review.openstack.org/#/c/392569/15:25
mriedemjust haven't gone through it yet15:25
mriedembut wanted to know if you were at least doing the scheduler side stuff locally already, sounds like you are15:25
bauzasindeed15:26
mriedembtw, i was wondering, even though we're going to use the placement API for simple resource classes/filters like cpu/memory/disk, don't we still end up calling ComputeNodeList.get_all() for the other more advanced filters?15:26
mriedemor is the idea to widdle the original list of compute nodes based on what the placement API gives us15:27
mriedemregardless of the order of the scheduler filters in the config15:27
johnthetubaguy(I thought it was the later)15:27
bauzasmriedem: the latter yes15:27
mriedemok15:27
*** Jinxing has joined #openstack-nova15:27
bauzas(sorry, I'm awfully busy today, hopefully being less impacted tomorrow :/ )15:28
*** claudiub has joined #openstack-nova15:28
mriedemnp. on that note, i'm running to a meeting.15:28
*** ekuris has quit IRC15:28
*** ijw has quit IRC15:29
*** frickler has quit IRC15:29
*** Guest66666 has quit IRC15:29
*** frickler has joined #openstack-nova15:29
*** Guest66666 has joined #openstack-nova15:29
*** mjura has quit IRC15:30
*** rmart04 has quit IRC15:31
*** claudiub|2 has quit IRC15:31
*** baoli has quit IRC15:31
*** paw has joined #openstack-nova15:34
sdaguecdent: are you fine with this - https://review.openstack.org/#/c/407180/6 ?15:35
* cdent looks15:35
cdentsdague: I think the code has some stylistic issues, but functionally it's okay. I can go ahead and leave those comments now, but I wouldn't want it to block.15:38
*** READ10 has joined #openstack-nova15:38
openstackgerritChristopher Martin proposed openstack/nova: Bugfix: Changes to Instance Soft Delete  https://review.openstack.org/38675615:39
openstackgerritWang Qing wu proposed openstack/nova: WIP(2): Introduce base PowerVM driver (disk drv)  https://review.openstack.org/40940115:39
*** rama_y has joined #openstack-nova15:41
*** salv-orlando has joined #openstack-nova15:43
cdentsdague, edleafe: I've left those comments15:43
*** jaugustine has joined #openstack-nova15:45
*** mvk has quit IRC15:48
*** salv-orlando has quit IRC15:48
*** rama_y has quit IRC15:50
*** edand has quit IRC15:50
*** marst has joined #openstack-nova15:52
openstackgerritGábor Antal proposed openstack/nova: Added missing decorator for instance.create.error  https://review.openstack.org/41659715:56
*** yufei has quit IRC15:56
*** cdent has quit IRC15:57
*** sheel has quit IRC15:57
*** _ducttape_ has joined #openstack-nova15:59
*** sudipto has quit IRC16:00
*** sudipto_ has quit IRC16:00
*** yamahata has joined #openstack-nova16:00
*** cdent has joined #openstack-nova16:01
jaypipesedleafe: hey! you know I love my C code :)16:02
*** cleong has joined #openstack-nova16:02
*** ducttape_ has quit IRC16:03
*** Guest41147 has quit IRC16:03
*** corey_ has joined #openstack-nova16:08
*** corey_ is now known as Guest4830916:09
*** cleong has quit IRC16:09
*** rcernin has quit IRC16:09
cfriesenany chance of getting some reviewers for https://review.openstack.org/#/c/260339/ ?   (It's not my patch, but it's a useful fix.)  There's a note about a possible tempest race in the bug, so would we need to fix the tempest test before fixing nova?16:09
*** tbachman has quit IRC16:10
*** tbachman_ has joined #openstack-nova16:10
openstackgerritGábor Antal proposed openstack/nova: Removed unnecessary parantheses and fixed formation  https://review.openstack.org/37054916:10
*** nkrinner is now known as nkrinner_afk16:11
*** kfarr has joined #openstack-nova16:16
openstackgerritmelanie witt proposed openstack/nova: WIP Count resources to check quota for cells  https://review.openstack.org/41652116:19
*** sdague_ has joined #openstack-nova16:20
edleafecdent: Not sure what I was going for there, but yeah, it looks wrong. Will fix.16:20
cdent✔16:21
*** ijw has joined #openstack-nova16:22
*** dillaman has quit IRC16:23
*** dtp has joined #openstack-nova16:25
*** bvanhav_ has joined #openstack-nova16:26
*** bvanhav has quit IRC16:26
*** bvanhav_ is now known as bvanhav16:26
*** gibi has joined #openstack-nova16:27
*** jose-phillips has joined #openstack-nova16:27
*** ijw has quit IRC16:27
*** dillaman has joined #openstack-nova16:28
*** unicell has quit IRC16:29
openstackgerritGleb Stepanov proposed openstack/python-novaclient: Add evacuate host method  https://review.openstack.org/34917916:30
sfinucanmriedem: Boom https://review.openstack.org/#/c/41608316:30
openstackgerritGábor Antal proposed openstack/nova: Removed unnecessary parantheses and fixed formation  https://review.openstack.org/37054916:32
* sfinucan hopes to cash in his review credits during Pike16:32
sfinucan:P16:32
openstackgerritWang Qing wu proposed openstack/nova: WIP(3): Introduce base PowerVM driver (console)  https://review.openstack.org/40940216:32
*** _ducttape_ has quit IRC16:33
*** ducttape_ has joined #openstack-nova16:33
openstackgerritEd Leafe proposed openstack/nova: Do not post allocations that are zero  https://review.openstack.org/40718016:36
edleafecdent: should be better ^^16:36
* cdent clicks16:36
*** Jinxing has quit IRC16:37
cdentedleafe: decided you didn't like my concern about self.used?16:37
edleafecdent: you mean protecting against negative used values?16:38
*** mlavalle has joined #openstack-nova16:39
cdentyes, and also being explicit about the conditional being numeric16:39
edleafecdent: not sure I follow16:39
cdentI just think that self.used == 0 or self.used <= 0 is more explicit/readable than not self.used16:40
mriedemsdague: mtreinish: easy one https://review.openstack.org/#/c/416401/16:41
mriedemsfinucan: thanks16:41
edleafecdent: self.used == 0 is major ugly in Python16:41
edleafe<= 0 is another thing16:41
mtreinishmriedem: +216:41
cdentbut it is _exactly_ what you mean to be testing for there, especially based on the message raised if it is true16:41
edleafecdent: if you think we need to guard against negative values, I'll add <= 016:42
*** haplo37_ has quit IRC16:42
cdenta) just out of curiosity why is self.used == 0 ugly? b) I'm not sure if we do or not, I'm asking, not telling16:42
mriedemdansmith: there will be a thrilling cells v2 meeting in 17 minutes yes?16:43
mriedemdare i say titilating16:43
edleafecdent: 0, False, empty string/dict/list/tuple/set/etc are all boolean False, so you don't test for == 0, or == {}, or the like16:44
cdentedleafe: I understand that, and that's my point: you _are_ testing for 0 here. Not False or empty16:44
openstackgerritSylvain Bauza proposed openstack/nova: Expose a REST API for a specific list of RPs  https://review.openstack.org/39256916:45
cdentbecause the fields definition has already established that used must be an int16:45
dansmithmriedem: figured yeah16:45
edleafecdent: if you are getting unexpected parameter types here, you have much bigger problems16:45
*** edmondsw_ has joined #openstack-nova16:45
*** edmondsw_ has quit IRC16:45
bauzasjaypipes: mriedem: dansmith: others, wanna bite https://review.openstack.org/392569 ?16:45
*** haplo37_ has joined #openstack-nova16:45
cdentedleafe: hmm, I think we are miscommunicating a bit. At what stage of the process are ovo field types validated?16:46
mriedembauzas: it's in a tab16:46
cdentif it is _before_ create is called, then self.used is always an int, and therefore the most limited and explicit (and thus readable) conditional is self.used == 016:47
bauzasmriedem: just added a new test, so Ctrl-R is needed :)16:47
cdentedleafe: that's all I'm saying16:47
mriedemtdurakov: pkoniszewski: it would be awesome if we had a diagram of the call stack during live migration so i didn't have to relearn it everytime i'm reviewing something like this https://review.openstack.org/#/c/140045/51/nova/virt/hyperv/livemigrationops.py16:47
edleafecdent: if it could be an unexpected type, then you should do the type checking sepately16:47
cdentyes, we agree on that16:48
edleafecdent: but if it's the correct type, then == 0 is ugly16:48
cdentno, it's explicit16:48
*** mdnadeem has quit IRC16:48
sdague_mriedem: +A16:49
cdentedleafe: but since that's aesthetics, I guess we should really just try to answer if we need to worry about negative numbers. I can't remember the answer to that.16:49
bauzascdent: edleafe: HTH https://github.com/openstack/oslo.versionedobjects/blob/master/oslo_versionedobjects/base.py#L70-L7216:49
edleafebauzas: so it's enforced when the value is set, not when it's saved16:50
bauzasindeed16:51
bauzaseasy to test16:51
openstackgerritEd Leafe proposed openstack/nova: Do not post allocations that are zero  https://review.openstack.org/40718016:51
edleafecdent: ^^ try again :)16:51
cdentnow reason isn't perfect :)16:52
bauzascdent: edleafe: FWIW http://paste.openstack.org/show/593884/16:53
edleafebauzas: thx16:54
edleafecdent: what's wrong with the reason??16:54
cdent"allocations of 0 resources used not allowed'"16:54
cdentdoesn't account for the negative handling you've just added16:54
cdentbut the :) was there for a reason16:54
cdentI don't really care, but if we're striving for perfection...16:55
edleafecdent: ah, you meant in the error messag16:55
edleafeI thought you meant the comment that I changed16:55
*** ijw has joined #openstack-nova16:56
openstackgerritEd Leafe proposed openstack/nova: Do not post allocations that are zero  https://review.openstack.org/40718016:57
edleafecdent: and you know I can't leave something like that there. ^^16:57
cdentI have become your enabler16:57
*** rama_y has joined #openstack-nova17:00
mdboothCould we please bang out these final 3 py3 fixes for libvirt: https://review.openstack.org/#/c/403423/ https://review.openstack.org/#/c/403419/ https://review.openstack.org/#/c/403431/17:00
mdboothThey've all had multiple rounds of review and rebases17:00
*** moshele has quit IRC17:01
raj_singhsfinucan: https://review.openstack.org/#/c/397774/ looks good to me, but because I fixed bunch of stuff in this I want johnthetubaguy to review it.17:01
mdboothIt would be nice to get these done and forget about them17:01
sfinucanmdbooth: Oops, forgot about those. I'll take a look shortly17:02
mdboothsfinucan: Thanks. Looks like you commented on the first on Monday?17:02
sfinucanraj_singh: Aye, sounds fair. You know more about that than me though, so a +1 wouldn't go astray17:02
sfinucanmdbooth: I did, but I'd lost the context. Not going to hold it up for my one comment17:03
mdboothGood, because I don't know the answer :)17:03
*** salv-orlando has joined #openstack-nova17:04
*** mdnadeem has joined #openstack-nova17:04
mdboothsfinucan: They mutually conflict (in tests-py3.txt), btw, so at least one of them will have to be remunged whatever.17:06
*** ijw has quit IRC17:06
edleafecdent: did you mean 'persevering' or 'preseverating'? :)17:06
mdboothpreseverating sounds wonderful17:07
cdentheh, well played17:07
mdboothand potentially dangerous17:07
cdenthow about both?17:07
*** raunak has joined #openstack-nova17:09
*** jaugustine has quit IRC17:10
*** paw has quit IRC17:14
*** JoseMello has joined #openstack-nova17:14
openstackgerritRafael Folco proposed openstack/nova: DELETE all inventory for a resource provider  https://review.openstack.org/41666917:14
*** ijw has joined #openstack-nova17:15
*** paw has joined #openstack-nova17:16
openstackgerritDan Peschman proposed openstack/nova: No API cell up-call to delete consoleauth tokens  https://review.openstack.org/41592217:16
*** smatzek has quit IRC17:19
*** sahid has quit IRC17:19
*** mvk has joined #openstack-nova17:21
*** tesseract has quit IRC17:21
*** markus_z has quit IRC17:21
mriedemwoot https://blueprints.launchpad.net/nova/+spec/consistent-query-parameters-validation is complete17:22
mriedemalex_xu: nice work17:22
*** ijw has quit IRC17:23
*** 09SAAA525 has joined #openstack-nova17:25
openstackgerritChris Dent proposed openstack/nova: [placement] Add a bit about extraction plans to placement_dev  https://review.openstack.org/41667617:25
*** gibi has quit IRC17:27
openstackgerritRafael Folco proposed openstack/nova: DELETE all inventory for a resource provider  https://review.openstack.org/41666917:27
cdentmriedem: is this still on your radar? https://review.openstack.org/#/c/414230/ (more logging on rp update failures)17:28
cdentIf not, I can probably pick it up17:29
mriedemcdent: i'm not actively on it right now so feel free17:29
openstackgerritGábor Antal proposed openstack/nova: Transform instance.volume_attach notification  https://review.openstack.org/40199217:29
cdent✔17:29
*** sdague_ has quit IRC17:30
openstackgerritRafael Folco proposed openstack/nova: DELETE all inventory for a resource provider  https://review.openstack.org/41666917:30
*** 09SAAA525 has quit IRC17:33
openstackgerritRafael Folco proposed openstack/nova: DELETE all inventory for a resource provider  https://review.openstack.org/41666917:33
*** nicolasbock has quit IRC17:34
*** lpetrut has quit IRC17:34
*** burt has joined #openstack-nova17:35
openstackgerritChris Dent proposed openstack/nova: WIP: Add more debug logging on RP inventory update failures  https://review.openstack.org/41423017:35
*** weshay is now known as weshay_lunch17:35
openstackgerritGábor Antal proposed openstack/nova: Transform instance.volume_detach notification  https://review.openstack.org/40867617:36
openstackgerritGleb Stepanov proposed openstack/python-novaclient: Refactor novaclient code  https://review.openstack.org/41668217:37
*** mdnadeem has quit IRC17:37
*** Swami has joined #openstack-nova17:39
sfinucanmdbooth: Done (all trivial reapprovals bar the first). You just need to rope someone like japyipes in now :)17:39
* sfinucan is off to prep for the Chelsea Spurs game17:39
*** mnestratov|2 has quit IRC17:39
*** sdague has quit IRC17:39
*** Guest48309 has quit IRC17:40
*** cleong has joined #openstack-nova17:40
*** paw has quit IRC17:41
*** mdnadeem has joined #openstack-nova17:41
*** gibi has joined #openstack-nova17:42
*** mnestratov|2 has joined #openstack-nova17:44
openstackgerritBalazs Gibizer proposed openstack/nova: Transform aggregate.create notification  https://review.openstack.org/39451117:44
*** ijw has joined #openstack-nova17:44
openstackgerritBalazs Gibizer proposed openstack/nova: Transform aggregate.delete notification  https://review.openstack.org/39451217:45
*** baoli has joined #openstack-nova17:45
*** ijw has quit IRC17:45
openstackgerritBalazs Gibizer proposed openstack/nova: Add aggregate notification related enum values  https://review.openstack.org/39620917:45
openstackgerritBalazs Gibizer proposed openstack/nova: Transform aggregate.add_host notification  https://review.openstack.org/39621017:45
openstackgerritBalazs Gibizer proposed openstack/nova: Transform aggregate.remove_host notification  https://review.openstack.org/39621117:46
*** smatzek has joined #openstack-nova17:46
*** paw has joined #openstack-nova17:48
*** READ10 has quit IRC17:51
*** sean-k-mooney has joined #openstack-nova17:51
johnthetubaguygibi: I wasn't sure about this one, I am curious about your thoughts around that: https://review.openstack.org/#/c/408689/17:53
*** moshele has joined #openstack-nova17:53
*** mdnadeem has quit IRC17:53
*** yamahata has quit IRC17:54
*** derekh has quit IRC17:58
*** aleph1 is now known as agarner17:59
*** bvanhav has quit IRC18:02
*** bvanhav_ has joined #openstack-nova18:02
*** browne has joined #openstack-nova18:06
sean-k-mooneymriedem: QQ when https://review.openstack.org/#/c/411988/ is merged to relese the new version of os-vif will it automatically update the requirements repo or should i submit a patch to the requiremets repo to do that?18:09
openstackgerritDan Smith proposed openstack/nova: Multicell support for instance listing  https://review.openstack.org/39677518:10
openstackgerritDan Smith proposed openstack/nova: Fix non-parameterized service id in hypervisors sample tests  https://review.openstack.org/41047118:10
openstackgerritDan Smith proposed openstack/nova: Make the SingleCellSimple fixture a little more comprehensive  https://review.openstack.org/41047218:10
openstackgerritDan Smith proposed openstack/nova: WIP: Make api_samples tests use simple cell environment  https://review.openstack.org/41047318:10
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937918:10
mriedemsean-k-mooney: you have to submit a patch to raise the minimum required version of os-vif in global-requirements yourself18:11
mriedemthe upper-constraints update happens automatically, but if you're going to bump g-r you might as well bump u-c in the same patch - i think you'd have to anyway for the tests to pass18:12
*** jaugustine has joined #openstack-nova18:12
sean-k-mooneymriedem: thanks. im going to follow up with sfinucan again tomorow and see if we can get his outstanding patch merged so that the release can go ahead.18:13
*** efried has quit IRC18:13
sean-k-mooneyill update the patche to the release repo and to the requirements repo once that is done18:14
sean-k-mooneysorry for the delay18:14
mriedemsean-k-mooney: cool i was going to ask you if his change at the bottom of the series was really required or not18:14
sean-k-mooneywe could release without it but he is fixing an issue on an inplace upgrade where the mtu chages.18:15
sean-k-mooneyit would be nice to have but not strictly required.18:15
mriedemok we still have some time18:15
*** dharinic is now known as dharinic|lunch18:16
*** jaugustine has quit IRC18:17
sean-k-mooneyi assume the os-vif versions are fix on stable branch correct. e.g. they cannot be raised after the sable/branch is made ?18:17
*** weshay_lunch is now known as weshay18:18
mriedemsean-k-mooney: correct18:18
mriedemwell the minimum can't18:19
sean-k-mooneyand uppper contratiants ? i assume they are also fixed or can the be extended if the new version is backwards compatible?18:19
*** kfarr has quit IRC18:20
*** lucasagomes is now known as lucas-afk18:21
sean-k-mooneywe dont need to extend them currently i just wanted to know if we had a reason too could we in the future.18:21
openstackgerritSujitha proposed openstack/nova: Document testing process for zero downtime upgrade  https://review.openstack.org/39526918:23
*** GordonTX has quit IRC18:23
mriedemsean-k-mooney: i don't think upper-constraints is auto-updated on stable branches18:23
mriedemwe can bump then in the future on stable manually if needed to get like a bug fix or something18:24
*** tblakes has quit IRC18:25
*** tjones has joined #openstack-nova18:25
*** tbachman_ is now known as tbachman18:27
*** tjones has left #openstack-nova18:27
*** tblakes has joined #openstack-nova18:28
*** lpetrut has joined #openstack-nova18:29
*** jaugustine has joined #openstack-nova18:29
mriedempretty easy hyperv bp patch that needs another +2 here https://review.openstack.org/#/c/248029/18:31
*** yamahata has joined #openstack-nova18:32
openstackgerritChristopher Martin proposed openstack/nova: Remove mox from unit/compute/test_compute_api.py (1)  https://review.openstack.org/33484118:35
openstackgerritSujitha proposed openstack/nova: Adding tags field to InstancePayload  https://review.openstack.org/40722818:35
openstackgerritChris Dent proposed openstack/nova: [placement] Add an "Adding a Handler" section to placement_dev  https://review.openstack.org/41630418:38
openstackgerritChris Dent proposed openstack/nova: [placement] placement_dev info for testing and gabbi  https://review.openstack.org/41260018:38
openstackgerritChris Dent proposed openstack/nova: [placement] placement_dev info for microversion handling  https://review.openstack.org/41194618:38
openstackgerritChris Dent proposed openstack/nova: [placement] Add a bit about extraction plans to placement_dev  https://review.openstack.org/41667618:38
*** unicell has joined #openstack-nova18:38
*** nic has joined #openstack-nova18:39
*** READ10 has joined #openstack-nova18:39
*** ijw has joined #openstack-nova18:39
*** ijw has quit IRC18:44
openstackgerritJulian proposed openstack/nova: Transform keypair.create notification  https://review.openstack.org/40868918:45
*** yamahata has quit IRC18:46
*** yamahata has joined #openstack-nova18:46
openstackgerritEd Leafe proposed openstack/nova: Do not post allocations that are zero  https://review.openstack.org/40718018:47
*** moshele has quit IRC18:48
*** jaugustine has quit IRC18:48
*** jaugustine has joined #openstack-nova18:49
*** pcaruana has quit IRC18:52
*** sdague has joined #openstack-nova18:52
*** jaugustine has quit IRC18:53
*** mnestratov|2 has quit IRC18:54
sean-k-mooneyit would be nice to see https://review.openstack.org/#/c/140045/52 merged also. that patch has been around for a while.18:56
mriedemsean-k-mooney: yeah, it's a bit bigger but i'm good with it now18:57
*** mnestratov|2 has joined #openstack-nova18:57
oomichidansmith: melwitt: could you take a look at https://review.openstack.org/#/c/411816?  I am not a specialist at the area and nice to get your reviews I think18:57
sean-k-mooneyi actully taught the hyperv os-vif support was already in place in newton but i guess it missed the cut off.18:58
mriedemedleafe: is this what your patch was handling? http://logs.openstack.org/21/386221/3/check/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/14df095/logs/screen-n-cpu.txt.gz#_2017-01-04_12_16_04_61618:58
dansmithoomichi: I'm not really either but I'll look18:58
mriedemedleafe: on the ceph jobs we send 0 for disk allocatoins18:58
mriedemwhich placement hates18:58
mriedemedleafe: actually i guess https://review.openstack.org/#/c/414214/ should handle that for the ceph job18:59
oomichidansmith: thanks :)18:59
edleafemriedem: yep19:01
mriedemedleafe: actually no it looks like19:01
mriedemhttp://logs.openstack.org/14/414214/12/check/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/511c30e/logs/screen-n-cpu.txt.gz#_2017-01-03_23_17_44_75019:01
mriedem^ is the ceph job run on jaypipes' patch19:01
mriedemSending allocation for instance {'allocations': [{'resource_provider': {'uuid': 'f8ce7899-0b48-4294-8df2-057ac325f4d2'}, 'resources': {'MEMORY_MB': 64, 'VCPU': 1, 'DISK_GB': 0}}]}19:01
edleafemriedem: ok gimme a few - on a call now19:01
mriedemoh that's allocations,19:02
mriedemjay fixed inventories19:02
jaypipesya19:03
mriedemok looking at https://review.openstack.org/#/c/407180/ now19:03
jaypipesedleafe fixed allocs.19:03
mriedemwould be cool if we could define ranges on ovo integer fields like we can for IntOpts in oslo.config19:10
*** sdague_ has joined #openstack-nova19:10
openstackgerritJay Pipes proposed openstack/nova: Return uuid attribute for aggregates  https://review.openstack.org/41503119:11
*** jaugustine has joined #openstack-nova19:11
sean-k-mooneymriedem: if ovo cant do it as an integer type field im sure a bounded integer field could be introduced if needed19:11
sean-k-mooneymriedem: probably not this cycle though19:12
mriedemjaypipes: replied in https://review.openstack.org/#/c/415031/7/nova/tests/functional/api_sample_tests/test_aggregates.py - i think what alex was asking for is redundant19:14
mriedemwhich is why i removed it19:14
*** ijw has joined #openstack-nova19:15
openstackgerritRafael Folco proposed openstack/nova: DELETE all inventory for a resource provider  https://review.openstack.org/41666919:16
*** aslezil has joined #openstack-nova19:18
*** dharinic|lunch is now known as dharinic19:18
*** cmartin has joined #openstack-nova19:22
*** jaugustine has quit IRC19:22
*** jaugustine has joined #openstack-nova19:23
*** ijw has quit IRC19:24
mriedemcdent: edleafe: got some issues in https://review.openstack.org/#/c/407180/ - for whenever19:28
*** jaugustine has quit IRC19:28
*** fitoduarte has joined #openstack-nova19:28
*** ijw has joined #openstack-nova19:28
jaypipesmriedem: oh, I figured you removed it because you found a bug in using a regex substitution for uuids.19:30
mriedemnope it was just confusing me and was redundant19:30
*** fitoduarte has quit IRC19:30
mriedemwe can't actually compare the response to a known uuid in those tests without hacking the aggregate create db method19:31
mriedemto use a static uuid19:31
mriedemand i didn't want to do that in those tests19:31
mriedemi guess we could for some of the other tests that build on an existing created aggregate where we already know the uuid19:31
jaypipesmriedem: I had copied that subs['uuid'] stuff from test_servers.py :)19:33
mriedemlike most tihngs in api samples tests, who knows, things are cludged until they pass19:34
mriedemi can poke at using the actual created aggregates uuid in later test response subs in a bit19:34
*** bnemec has quit IRC19:35
*** ttx has quit IRC19:36
*** aignatov has quit IRC19:36
*** kevinbenton has quit IRC19:36
*** raginbajin has quit IRC19:36
*** Madasi has quit IRC19:36
*** rwmjones|hols has quit IRC19:36
*** jschwarz has quit IRC19:36
*** mariusv_ has quit IRC19:36
*** lifeless has quit IRC19:36
*** slunkad has quit IRC19:36
*** cburgess has quit IRC19:36
*** rmk has quit IRC19:36
*** belliott_ has quit IRC19:36
*** rodrigods has quit IRC19:36
*** jotr has quit IRC19:36
*** hughsaunders has quit IRC19:36
*** raj_singh has quit IRC19:36
*** doffm has quit IRC19:36
*** d34dh0r53 has quit IRC19:36
*** dharinic has quit IRC19:36
*** aslezil has quit IRC19:36
*** yarkot has quit IRC19:36
*** sneti has quit IRC19:36
*** mikal has quit IRC19:36
*** federico3 has quit IRC19:36
*** jamespd has quit IRC19:36
*** Vek has quit IRC19:36
*** mrda has quit IRC19:36
*** bnemec has joined #openstack-nova19:36
*** doffm has joined #openstack-nova19:36
*** belliott has joined #openstack-nova19:36
*** Vek has joined #openstack-nova19:36
*** mikal has joined #openstack-nova19:36
*** rwmjones has joined #openstack-nova19:36
*** rodrigods has joined #openstack-nova19:36
*** mariusv has joined #openstack-nova19:36
*** jschwarz has joined #openstack-nova19:36
*** jamespd has joined #openstack-nova19:36
*** rmk has joined #openstack-nova19:36
*** mrda has joined #openstack-nova19:36
*** d34dh0r53 has joined #openstack-nova19:36
*** slunkad has joined #openstack-nova19:36
*** cburgess has joined #openstack-nova19:36
*** Madasi has joined #openstack-nova19:36
*** lifeless has joined #openstack-nova19:36
*** ttx has joined #openstack-nova19:36
*** aignatov has joined #openstack-nova19:36
*** mariusv has quit IRC19:36
*** mariusv has joined #openstack-nova19:36
*** rmk has quit IRC19:36
*** rmk has joined #openstack-nova19:36
*** jotr has joined #openstack-nova19:37
*** kevinbenton has joined #openstack-nova19:37
*** federico3 has joined #openstack-nova19:37
*** hughsaunders has joined #openstack-nova19:37
*** raginbajin has joined #openstack-nova19:38
*** test222 has quit IRC19:39
mriedemedleafe: i'm not sure why Allocation.create() is even touched in https://review.openstack.org/#/c/407180/ since nothing in nova uses that method today from what i can tell19:39
*** yarkot has joined #openstack-nova19:39
mriedemeverything goes through AllocationList.create_all()19:39
mriedemwhich is why we don't handle ObjectActionError in the handler19:40
*** dharinic has joined #openstack-nova19:41
*** test222 has joined #openstack-nova19:43
*** raj_singh has joined #openstack-nova19:44
*** paw has quit IRC19:44
mriedemnice https://review.openstack.org/#/q/topic:bp/use-neutron-by-default19:48
mriedemanother bp to check off19:48
*** aslezil has joined #openstack-nova19:49
*** jaugustine has joined #openstack-nova19:49
*** corey_ has joined #openstack-nova19:50
*** sneti has joined #openstack-nova19:51
*** corey_ is now known as Guest374519:51
edleafemriedem: ok, back from meeting. Let me dig in further19:51
edleafemriedem: the long holiday wiped out all my context19:52
*** cleong has quit IRC19:53
mriedemedleafe: i think the fix is really just in the report client to filter out 0-value allocatoins19:53
mriedem*allocations19:53
mriedemand that's a fix we can backport to newton19:53
*** rfolco has quit IRC19:53
mriedemthe additional guarding and error handling stuff in the REST API could be separate, and doesn't fix thebug19:53
cdentmriedem: we can't just fix the report client because there will be other clients, eventually. It might be okay for now, but we'll need to fix it before end of cycle19:53
mriedemwell, what does 'fix' mean?19:54
mriedembecause Allocation.create() changes don't fix anything b/c it's dead code19:54
cdenti mostly mean the use of the exception19:55
cdentwe can probably remove create() now that we don't have remotable19:55
mriedemsure, that's fine, and makes the API response more clear to the real error19:55
mriedemi'd prefer to split the REST API handler change out from the report client fix19:55
mriedemb/c i want to backport the report client fix to newton19:55
edleafemriedem: that makes sense19:55
edleafemriedem: let me revert all but the report client change in that patch19:55
edleafeI can address the remaining issues in a separate patch19:56
mriedemcool thanks19:56
*** gouthamr has joined #openstack-nova19:58
*** efried has joined #openstack-nova20:00
*** efried has quit IRC20:02
openstackgerritEd Leafe proposed openstack/nova: Do not post allocations that are zero  https://review.openstack.org/40718020:02
edleafemriedem: ^^ just the report client changes20:02
*** efried has joined #openstack-nova20:02
*** efried has left #openstack-nova20:03
satyarThanks jaypipes and mriedem20:07
satyarputting the rbd_secret_uuid to cinder.conf solved the boot from volume issue20:07
jaypipessatyar: that was all mriedem :)20:07
mriedemwoot20:08
satyarmreidem i dont think we need rnd_secret_uuid in nova.conf anymore20:08
*** efried has joined #openstack-nova20:08
satyar:)20:08
*** beagles is now known as beagles_biab20:08
*** salv-orlando has quit IRC20:08
mriedemsatyar: only if you're using rbd for ephemeral storage on the computes with the libvirt driver20:08
satyarjaypipes you also helped :)20:08
mriedemand want them encrypted20:08
satyarok20:08
mriedemnova.conf is for the ephemeral storage, cinder.conf is for the persistent20:08
*** cdent has quit IRC20:09
*** efried has quit IRC20:09
satyargit it :) Thanks mridem20:09
mriedemjaypipes: dansmith: so i think we assume that we'd like resource providers and the reporting client to work for ironic in newton before upgrades to ocata, yes?20:10
*** weshay is now known as weshay_brb20:10
mriedembecause backporting https://review.openstack.org/#/c/414214/ is turning out to be a pain in the ass because of https://github.com/openstack/nova/commit/f1cd928bf5daac3b344d0c6cd095aabec14cdbff not being in newton20:10
*** liangy has joined #openstack-nova20:12
dansmithmriedem: you could backport that dependent patch easily, no?20:13
dansmithbut yeah I guess we'd want ironic pre-reporting20:13
mriedemdansmith: i could, even though it says blueprint in it and is just a refactor20:14
dansmithtbh, we (redhat) aren't really as prepared for placement being required in ocata as we'd like20:14
mriedembut i guess we just say 'this is for the next thing which we need and makes the backport simpler, no functional changes, so eat it tonyb'20:14
dansmithso at some point we'll probably need to decide how likely things are to make ocata in whole and maybe decide to punt the requirement20:14
dansmithyeah, I mean, doesn't seem like it should be that controversial, tbh20:14
mriedempunt the requirement upstream or rhosp?20:15
mriedembecause besides the scheduler bits to talk to the placement api, i think we're pretty close20:15
dansmithwell, osp can't punt if it's not punted upstream, but I meant upstream nova yeah20:15
dansmithokay20:15
dansmithI haven't been paying as much attention as I should20:15
dansmithbut that's good if so20:15
mriedemthat's fine, you're cells v2 plus vacation20:15
mriedemjaypipes can correct me if i'm wrong about status though20:16
*** efried has joined #openstack-nova20:16
dansmithafter detoxing for 2.5 weeks and feeling not so great today, I've determined that I'm allergic to work20:16
* dansmith files for disability coverage20:16
mriedemmore coffee20:16
dansmithheh20:16
mriedemor https://www.youtube.com/watch?v=RAZQ769_EUU20:17
mriedemat very loud levels20:17
dansmithwhen my headache goes away, maybe :)20:17
*** efried has quit IRC20:17
*** weshay_brb is now known as weshay20:26
*** rtheis has quit IRC20:28
mriedemcrap even with backporting that one change there are a bunch of conflicts20:30
mriedemso i probably won't try backporting that one20:30
*** dtp has quit IRC20:33
*** Guest3745 is now known as cleong20:33
*** READ10 has quit IRC20:35
*** bvanhav_ has quit IRC20:36
*** efried has joined #openstack-nova20:37
*** efried has quit IRC20:37
*** sneti has quit IRC20:38
*** lamt has quit IRC20:39
*** federico3 has quit IRC20:41
*** federico3 has joined #openstack-nova20:41
mriedemedleafe: +2 on that simpler change, will check the ceph job logs when that's done20:42
mriedemedleafe: are you going to roll the REST API error handling stuff back on top of that?20:42
edleafemriedem: kewl20:42
edleafemriedem: yeah, starting on that now20:42
jaypipesmriedem: I'd support backporting that dependent patch (which as you point out is just a refactor, not a functional change)20:42
mriedemjaypipes: yeah i'm doing that, plus 2 before it20:42
mriedemjaypipes: you went refactor crazy in ocata20:42
jaypipesmriedem: sorry man :(20:43
mriedemit's fine :)20:43
mriedemwe'll just have to bribe lyarwood20:43
jaypipesheh20:44
*** paw has joined #openstack-nova20:46
*** eandersson has joined #openstack-nova20:47
*** ijw has quit IRC20:49
*** JoseMello has quit IRC20:55
*** sneti has joined #openstack-nova21:01
*** jaugustine has quit IRC21:02
*** sreeram_v has joined #openstack-nova21:03
*** tjones has joined #openstack-nova21:03
mriedemhmm, do we want to backport https://review.openstack.org/#/c/392933/21:04
*** rtheis has joined #openstack-nova21:04
mriedembecause if not, then i don't need to backport https://review.openstack.org/#/c/395971/21:04
*** rtheis has quit IRC21:04
mriedemand without https://review.openstack.org/#/c/392933/ i'm not sure that ironic will have issues that we saw on master...21:05
mriedemi might need to just run a DNM patch on stable/newton in ironic that enables the placement-api service to see what blows up21:05
*** sreeram_v has quit IRC21:07
*** adisky_ has quit IRC21:09
*** dimtruck is now known as zz_dimtruck21:10
mriedemhttps://review.openstack.org/#/c/416734/21:10
*** cleong has quit IRC21:12
openstackgerritJay Pipes proposed openstack/nova: placement: adds REST API for nested providers  https://review.openstack.org/38480721:14
openstackgerritJay Pipes proposed openstack/nova: placement: allow filter providers in tree  https://review.openstack.org/37721521:14
openstackgerritJay Pipes proposed openstack/nova: pci: add uuid field to PciDevice object  https://review.openstack.org/41476821:14
openstackgerritJay Pipes proposed openstack/nova: placement: SRIOV PF devices as child providers  https://review.openstack.org/41476921:14
openstackgerritJay Pipes proposed openstack/nova: placement: update client to set parent provider  https://review.openstack.org/38569321:14
openstackgerritJay Pipes proposed openstack/nova: placement: add nested resource providers  https://review.openstack.org/37713821:14
*** tblakes has quit IRC21:17
*** dtp has joined #openstack-nova21:17
mriedemthis is very weird http://logs.openstack.org/21/386221/3/check/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/14df095/console.html#_2017-01-04_12_31_14_90741921:20
mriedemtiming out waiting for the instance to go ACTIVE,21:20
mriedemwhich it never does21:20
mriedemnetworking is setup, it doesn't appear to be boot from volume21:20
mriedemoh hrm neutron http://logs.openstack.org/21/386221/3/check/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/14df095/console.html#_2017-01-04_12_31_15_00998921:21
mriedemnvm that's on teardown21:22
*** _ducttape_ has joined #openstack-nova21:23
*** salv-orlando has joined #openstack-nova21:23
*** ducttape_ has quit IRC21:26
openstackgerritSarafraj Singh proposed openstack/nova: DNM: Test service token  https://review.openstack.org/41323221:26
*** jaypipes has quit IRC21:27
*** _ducttape_ has quit IRC21:27
*** ijw has joined #openstack-nova21:29
*** Jeffrey4l has quit IRC21:35
mriedemmdbooth: has anything merged related to fetch_func_sync stuff in the libvirt imagebackend code recently that might cause....issues?21:41
*** zz_dimtruck is now known as dimtruck21:42
mriedemi don't see anything21:44
mriedemmdbooth: got a weird hang where this is the last thing we see when spawning the instance in the virt driver http://logs.openstack.org/21/386221/3/check/gate-tempest-dsvm-full-devstack-plugin-ceph-ubuntu-xenial/14df095/logs/screen-n-cpu.txt.gz#_2017-01-04_12_16_05_11221:45
mriedemeventually the test times out waiting for the instance to go ACTIVE21:45
mriedemthis is on the rbd imagebackend, no volumes related21:45
*** Jeffrey4l has joined #openstack-nova21:49
*** lamt has joined #openstack-nova21:50
*** satyar has quit IRC21:52
*** jamesdenton has quit IRC21:53
mriedemgd we need some debug logging in the image fetch/cache code in the libvirt driver flow21:53
*** salv-orl_ has joined #openstack-nova21:54
mriedemonce we start creating the backing image we have like 0 debug logging of what's going on21:54
*** eharney has quit IRC21:54
*** salv-orlando has quit IRC21:56
*** thorst has quit IRC21:58
*** thorst has joined #openstack-nova22:00
cfriesenmriedem: yeah, we've got an open issue that we think might have been due to image fetch issues, but have no info to prove it.22:02
mriedemcfriesen: according to logstash this has only hit on my change in the last 7 days22:02
*** chyka has joined #openstack-nova22:02
mriedembut we haven't had logstash until the last few days as it was down for about the last 2 weeks22:03
mriedemi don't see anything in glance logs22:03
mriedemalthough,22:03
mriedemaround the time that this gets to the fetch_func code in the libvirt imagebackend, there is a 3 minute gap in g-reg logs where an image is being uploaded22:03
mriedemi wonder if the upload of a snapshot is hammering ceph or something and starving this other thread22:03
cfriesenmriedem: our issue was on Mitaka, so probably unrelated to whatever you're doing.22:04
dansmithmriedem: fsck.. alaski strikes again: http://logs.openstack.org/75/396775/19/check/gate-tempest-dsvm-cells-ubuntu-xenial/14f1bba/logs/devstacklog.txt.gz#_2017-01-04_18_28_21_34422:04
mriedemthe ghost of alaski22:04
mriedem"CellsV1 users cannot use this simplified setup command" - but what are we supposed to use? goto: ...22:05
*** thorst has quit IRC22:05
dansmithmriedem: the other commands that simple_cell_setup calls for you22:05
mriedemso i guess we have to figure out what the upgrade path is for cells v1 and then integrate that into our ci setup22:05
mriedemyeah i suppose22:05
dansmithyeah, working on it22:06
*** ducttape_ has joined #openstack-nova22:06
*** lbeliveau has joined #openstack-nova22:08
*** smatzek has quit IRC22:13
*** mnestratov|2 has quit IRC22:15
*** thorst has joined #openstack-nova22:18
*** thorst has quit IRC22:21
*** thorst has joined #openstack-nova22:22
*** jheroux has quit IRC22:27
*** cmartin has quit IRC22:28
*** dave-mccowan has quit IRC22:28
*** navid_ has joined #openstack-nova22:28
navid_hi22:29
navid_I had a question22:29
*** tblakes has joined #openstack-nova22:29
*** baoli has quit IRC22:29
navid_In nova scheduler i want to get the nova service ip address, how can i get this ip address.22:30
*** baoli has joined #openstack-nova22:30
*** tjones has quit IRC22:31
*** baoli has quit IRC22:33
tonybmriedem, dansmith: If you did all that while I was asleep I might never notice ;P22:34
mriedemtonyb: i didn't think you actually slept22:35
*** baoli has joined #openstack-nova22:36
mriedemnavid_: CONF.my_ip?22:36
*** gouthamr has quit IRC22:36
tonybmriedem: :) If I don't I get really grmpy and me team doesn't like that .... ask mikal ;P22:36
openstackgerritEd Leafe proposed openstack/nova: Remove the Allocation.create() method.  https://review.openstack.org/41675122:37
openstackgerritEd Leafe proposed openstack/nova: Add check for invalid allocation amounts  https://review.openstack.org/41675222:37
*** masayukig has quit IRC22:38
*** masayukig has joined #openstack-nova22:38
navid_@mriedem: thanks22:38
*** thorst has quit IRC22:39
*** dave-mccowan has joined #openstack-nova22:39
*** baoli has quit IRC22:40
*** edmondsw has quit IRC22:41
*** edmondsw has joined #openstack-nova22:42
navid_@mriedam: but my_ip will give you your ip, what if there are multiple nova instances22:44
*** tjones has joined #openstack-nova22:44
mriedemnavid_: my_ip is the IP of the host that the nova-scheduler service is running on22:45
mriedemyou're asking about the scheduler getting the IPs of other nova services, like nova-compute?22:45
navid_@mriedem: aha, thanks a bunch, saved me a lot of time22:45
*** edmondsw has quit IRC22:47
mriedemok i've backported the placement RT stuff to newton https://review.openstack.org/#/q/topic:bug/1651678+branch:stable/newton but waiting on https://review.openstack.org/#/c/416734/ to see if we have an issue22:47
*** edmondsw has joined #openstack-nova22:48
*** openstack has joined #openstack-nova22:55
*** amotoki has quit IRC23:00
*** dave-mccowan has quit IRC23:00
*** amotoki has joined #openstack-nova23:02
*** jaugustine has joined #openstack-nova23:04
*** armax has quit IRC23:04
*** navid_ has quit IRC23:04
openstackgerritSujitha proposed openstack/nova: Document testing process for zero downtime upgrade  https://review.openstack.org/39526923:04
*** lpetrut has quit IRC23:04
*** ijw has quit IRC23:05
*** armax has joined #openstack-nova23:05
*** burt has quit IRC23:05
*** xyang1 has quit IRC23:05
*** armax has quit IRC23:06
*** tblakes has quit IRC23:08
*** jaugustine has quit IRC23:10
*** sneti has quit IRC23:10
*** jamielennox is now known as jamielennox|away23:11
*** jamielennox|away is now known as jamielennox23:14
*** ducttape_ has quit IRC23:16
*** mdrabe has quit IRC23:16
*** ducttape_ has joined #openstack-nova23:17
*** sneti has joined #openstack-nova23:17
*** yamahata has quit IRC23:19
*** mlavalle has quit IRC23:20
*** salv-orl_ has quit IRC23:20
*** ducttape_ has quit IRC23:21
*** cdelatte has quit IRC23:23
*** ducttape_ has joined #openstack-nova23:24
*** yamahata has joined #openstack-nova23:26
*** amotoki has quit IRC23:26
*** gongysh has joined #openstack-nova23:28
*** coreywright has quit IRC23:29
openstackgerritDan Smith proposed openstack/nova: Multicell support for instance listing  https://review.openstack.org/39677523:30
openstackgerritDan Smith proposed openstack/nova: Fix non-parameterized service id in hypervisors sample tests  https://review.openstack.org/41047123:30
openstackgerritDan Smith proposed openstack/nova: Make the SingleCellSimple fixture a little more comprehensive  https://review.openstack.org/41047223:30
openstackgerritDan Smith proposed openstack/nova: WIP: Make api_samples tests use simple cell environment  https://review.openstack.org/41047323:30
openstackgerritDan Smith proposed openstack/nova: WIP Move instance creation to conductor  https://review.openstack.org/31937923:30
*** coreywright has joined #openstack-nova23:30
*** portdirect_ has joined #openstack-nova23:30
*** amotoki has joined #openstack-nova23:31
*** edmondsw has joined #openstack-nova23:32
*** pradk has quit IRC23:34
*** sneti has quit IRC23:34
*** ducttape_ has quit IRC23:35
*** thorst_ has joined #openstack-nova23:35
*** ducttape_ has joined #openstack-nova23:35
*** edmondsw has quit IRC23:37
*** gouthamr has joined #openstack-nova23:39
*** ducttape_ has quit IRC23:40
*** little has quit IRC23:40
*** little has joined #openstack-nova23:40
*** ducttape_ has joined #openstack-nova23:40
*** portdirect_ has quit IRC23:47
*** lamt has quit IRC23:48
*** gouthamr has quit IRC23:48
*** gouthamr has joined #openstack-nova23:52
*** lbeliveau has quit IRC23:54
*** ijw has joined #openstack-nova23:55

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