Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-17
00:25:00 jaypipes Kevin_Zheng: no idea... probably just an oversight.
00:25:22 mriedem snikitin: ^
00:27:20 mriedem the db model change was https://review.openstack.org/#/c/97168/
00:27:25 mriedem which was a long long slog
00:27:32 mriedem so my guess would be, the db model stuff merged,
00:27:40 mriedem and then a release or more later the REST API change merged
00:27:45 Kevin_Zheng maybe we can expand limit to 80? as we already created 80 in db
00:27:49 mriedem at which point we'd totally forgotten what we were doing
00:27:58 Kevin_Zheng haha
00:30:52 mriedem yeah even looking at patch set 1 of https://review.openstack.org/#/c/128940/127/nova/objects/tag.py
00:30:57 Kevin_Zheng looks like it start with 80 in the initial patch and nobody asked about it
00:30:59 Kevin_Zheng yeah
00:31:00 mriedem https://review.openstack.org/#/c/128940/1/nova/api/openstack/compute/schemas/v2/instance_tags.py
00:31:02 mriedem it was 60
00:31:10 mriedem ^ had 127 patch sets
00:31:34 mriedem i don't know if the 60 vs 80 has anything to do with unicode?
00:32:02 Kevin_Zheng ah maybe, he said something about converting in spec
00:32:10 mriedem Kevin_Zheng: i assume you're asking because you need to know the schema for booting an instance with tags right?
00:32:41 mriedem snikitin would be the person to ask if he remembers a good reason for it
00:34:17 Kevin_Zheng ah, good, thanks, will try to catch him
00:35:09 openstackgerrit Claudiu Belu proposed openstack/nova: Adds CPUFeaturesField to ImageMetaProps https://review.openstack.org/420960
00:35:47 jaypipes mriedem: no, 60 vs. 80 has nothing to do with unicode.
00:39:21 Kevin_Zheng jaypipes: so is it just an oversight or do we have any particular reason?
00:48:19 mriedem i think it's probably just an oversight
00:49:18 Kevin_Zheng lets sync them :)
00:50:56 mriedem well,
00:50:59 mriedem we could
00:51:13 mriedem Kevin_Zheng: is there a technical reason you need tags in the REST API to allow up to 80?
00:52:17 mriedem Kevin_Zheng: normally changing something like this in the REST API schema would be a microversion change, but this isn't making things more restrictive so it might be ok to change it without a microversion
00:52:29 Kevin_Zheng yeah
00:52:37 mriedem Kevin_Zheng: i think we should start with a post to the ML thread about it so snikitin has a chance to respond (and alex_xu)
00:52:43 mriedem Kevin_Zheng: can you start that thread?
00:52:48 Kevin_Zheng yeah sure
00:52:54 mriedem thanks
00:54:31 openstackgerrit Zhenyu Zheng proposed openstack/nova: Expand tag length limit to 80 https://review.openstack.org/421004
00:55:05 Kevin_Zheng ^ was already doing this, but we can block it until have a clean idea from ML
01:02:13 mriedem something still failing in discover_hosts on the subnode http://logs.openstack.org/76/420976/4/check/gate-tempest-dsvm-neutron-multinode-full-ubuntu-xenial-nv/1e4a55e/logs/subnode-2/devstacklog.txt.gz#_2017-01-17_00_35_47_248
01:13:37 openstackgerrit OpenStack Proposal Bot proposed openstack/nova: Updated from global requirements https://review.openstack.org/420844
01:43:49 openstackgerrit Tuan Luong-Anh proposed openstack/nova: Support Ironic interface attach/detach in nova virt https://review.openstack.org/364413
01:45:04 openstackgerrit Sivasathurappan Radhakrishnan proposed openstack/nova: Initiate network changes from source during live migration https://review.openstack.org/404607
01:57:00 openstackgerrit Anh Tran proposed openstack/nova: Remove mox stubs in api/openstack/compute/test_server_reset_state.py https://review.openstack.org/320826
02:06:37 openstackgerrit Sivasathurappan Radhakrishnan proposed openstack/nova: Initiate network changes from source during live migration https://review.openstack.org/404607
02:08:37 openstackgerrit Sivasathurappan Radhakrishnan proposed openstack/nova: Refactor _post_live_migratiion https://review.openstack.org/413021
02:18:34 openstackgerrit Anh Tran proposed openstack/nova: Remove mox stubs in api/openstack/compute/test_virtual_interfaces.py https://review.openstack.org/320325
02:20:48 openstackgerrit Dan Smith proposed openstack/nova: WIP Move instance creation to conductor https://review.openstack.org/319379
02:39:16 openstackgerrit Spencer Yu proposed openstack/nova: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/420629
03:05:11 openstackgerrit Dan Peschman proposed openstack/nova: nova-manage cell_v2 map_cell0 exit 0 https://review.openstack.org/420132
03:41:17 openstackgerrit xianming.mao proposed openstack/nova: :Add wsgi response for attach method https://review.openstack.org/411773
03:44:13 openstackgerrit Anusha Unnam proposed openstack/nova: Add keypairs field to InstancePayload https://review.openstack.org/419730
03:48:24 openstackgerrit xianming.mao proposed openstack/nova: Add wsgi response for attach method https://review.openstack.org/411773
05:05:16 openstackgerrit Yoshihiko Atsumi proposed openstack/nova: Add confirm resized server functional negative tests https://review.openstack.org/421074
06:06:13 gmann mriedem: Kevin_Zheng i think we should just do DB to 60, because it need microversion to make it 80 on API side for interoperability
07:38:28 openstackgerrit Wang Qing wu proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288
07:38:32 Dinesh_Bhor Hi all, can anyone tell me is there any document which explains what all libvirt related events generated by nova in what cases?
07:39:32 Kevin_Zheng gmann: Hmm, users would love to have longer tags, but you have the point.
07:46:45 gmann Kevin_Zheng: yea we could have done initially but now it is difficult
07:47:22 Kevin_Zheng will there be any problem if we shorten DB from 60 to 80?
07:47:28 Kevin_Zheng when upgrading
08:03:34 gmann yea it might and we need migration script also?
08:50:42 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142
08:53:22 Kevin_Zheng Sergey Nikitin thoughts it will might be much painful in DB migration than a new specless microversion in ML
09:11:44 openstackgerrit huangtianhua proposed openstack/python-novaclient: Make _console() public https://review.openstack.org/414364
09:58:33 openstackgerrit Vladik Romanovsky proposed openstack/nova: libbvirt: set vlan tag for macvtap on SRIOV VFs https://review.openstack.org/421186
10:25:28 openstackgerrit Spencer Yu proposed openstack/nova: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/420629
10:30:11 openstackgerrit Jeremy Liu proposed openstack/nova: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/420629
10:47:47 openstackgerrit Pavel Gluschak proposed openstack/nova: Added live migration test hook for VNC console https://review.openstack.org/421207
10:52:22 openstackgerrit Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209
10:54:07 openstackgerrit Davanum Srinivas (dims) proposed openstack/nova: [WIP] ignore me - trying experimental jobs https://review.openstack.org/421209
11:07:13 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142
11:12:50 openstack Launchpad bug 1657087 in OpenStack Compute (nova) "Functional versioned notification test test_create_delete_server_with_instance_update randomly fails" [Medium,Confirmed]
11:12:50 mriedem https://bugs.launchpad.net/nova/+bug/1657087 if anyone wants to investigate that
11:13:23 openstackgerrit Wang Qing wu proposed openstack/nova: Introduce base PowerVM driver https://review.openstack.org/391288
11:15:59 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add new policy for server list/detail with all_tenants https://review.openstack.org/415330
11:18:43 openstackgerrit Rikimaru Honjo proposed openstack/nova: Consolidate unit tests for shelve API https://review.openstack.org/421220
11:19:01 openstackgerrit Matt Riedemann proposed openstack/nova: Add more details when test_create_delete_server_with_instance_update fails https://review.openstack.org/421221
11:22:28 sahid hello mriedem, anychance you look at this serie when you have a moment https://review.openstack.org/#/c/409260/
11:24:12 mriedem i see you and pkoniszewski are having a debate on the top patch in the series
11:25:24 sahid yes the first one has already +2 and the last one need a third person
11:29:23 openstackgerrit Matthew Booth proposed openstack/nova: libvirt: Fix races with nfs volume mount/umount https://review.openstack.org/383859
11:38:37 openstackgerrit Zhenyu Zheng proposed openstack/nova: Strict pattern match query parameters https://review.openstack.org/420494
11:50:30 openstackgerrit Matt Riedemann proposed openstack/nova: Fanout the db sync to all cells if --local_cell is False https://review.openstack.org/420973
11:50:31 openstackgerrit Matt Riedemann proposed openstack/nova: Add --verbose option to discover_hosts command https://review.openstack.org/421228
11:53:31 openstackgerrit Matt Riedemann proposed openstack/nova: Fanout the db sync to all cells if --local_cell is False https://review.openstack.org/420973
11:53:31 openstackgerrit Matt Riedemann proposed openstack/nova: Add --verbose option to discover_hosts command https://review.openstack.org/421228
12:07:54 openstackgerrit Pavel Gluschak proposed openstack/nova: live-migration test hook for serial console https://review.openstack.org/347471
12:10:03 openstackgerrit Pavel Gluschak proposed openstack/nova: Added live migration test hook for VNC console https://review.openstack.org/421207
12:10:55 openstackgerrit Pavel Gluschak proposed openstack/nova: don't store VNC/SPICE listen addrs in migration data if disabled https://review.openstack.org/396218
12:40:28 mriedem claudiub|2: need to add hw_cpu_features to ImageMetaProps in a change before this https://review.openstack.org/#/c/398509/
12:44:07 dims "Pass 1265 Failure 60 Skip 85" on the first run of gate-tempest-dsvm-nova-py35-ubuntu-xenial (DSVM/Tempest/py35), not too shabby
12:44:18 dims Anyone interested in helping with this? ^
12:44:42 mriedem Kevin_Zheng: feel free to get a start on the pike spec for the tags length microversion, it should be a simple spec
12:44:50 mriedem and the nova-specs repo is available for posting pike specs now
12:45:12 Kevin_Zheng mriedem: thanks
12:45:23 Kevin_Zheng will do it if got time
12:48:21 claudiub|2 mriedem: yep. that is why I've put that patch on top of this: https://review.openstack.org/#/c/420960/1
13:20:12 openstackgerrit Lee Yarwood proposed openstack/nova: Compute: Reset bdm connection_info during LM rollback https://review.openstack.org/338929
13:20:13 openstackgerrit Lee Yarwood proposed openstack/nova: Live migration of instances with encrypted volumes https://review.openstack.org/389608
13:24:51 jaypipes *yawn*
13:26:56 jaypipes cdent: thx :)
13:27:21 openstackgerrit Chris Dent proposed openstack/nova: A demo script for updating resource inventory https://review.openstack.org/382613

Earlier   Later