Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-17
00:07:49 mikal dansmith: right, explain that bit in small words for me
00:07:56 dansmith but if you had deleted a host you would still have some instance mappings
00:08:16 mikal So deleting the host which has the instances on it doesn't cascade the delete to the instances?
00:08:17 dansmith there is no foreign key constraint on that stuff,
00:08:22 dansmith so you could have nuked some things
00:08:33 dansmith you can't delete a host yet, so .. no :)
00:09:03 mikal Heh
00:09:30 mriedem like today you can delete a compute node in the db by deleting the nova-compute service for that host
00:09:44 mriedem and i'm pretty sure that doesn't mean you can't still have instnaces in the db pointing at that node
00:09:48 dansmith right
00:09:52 mikal That's a fair point
00:09:56 dansmith just sanity checking
00:10:01 mikal I don't feel we need to hold the new thing to a higher standard than what we have now
00:14:18 mriedem mikal: i'm failing to pick up on sarcasm there
00:19:00 mikal mriedem: Oh, sorry
00:19:19 mikal mriedem: I am glad the new code is consistent in its level of terrible with the old
00:19:23 mikal mgood enough?
00:19:49 Kevin_Zheng Hi cores, any particular reason that we have limited the length of instance tag to be 60 but created a db model with max length of 80?
00:22:22 Kevin_Zheng http://git.openstack.org/cgit/openstack/nova/tree/nova/db/sqlalchemy/models.py#n1456 db model
00:22:56 mriedem Kevin_Zheng: hmm, probably a question for jaypipes
00:23:31 Kevin_Zheng mriedm: hmm ok
00:24:10 mriedem my guess is the 60 number in the API comes from the spec https://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/tag-instances.html#data-model-impact
00:24:37 Kevin_Zheng http://git.openstack.org/cgit/openstack/nova/tree/nova/objects/tag.py#n18 FYI
00:24:50 Kevin_Zheng yeah, but why 80 in db model :)
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/

Earlier   Later