Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-13
21:33:55 jaypipes cdent, mriedem, dansmith, edleafe, bauzas: ok, I know it's late in the day, but I got it pushed ^^
23:01:31 openstackgerrit Merged openstack/nova master: Fix wrong log parm. https://review.openstack.org/482832
23:36:33 openstackgerrit Merged openstack/nova master: Remove mox from nova.tests.unit.virt.xenapi.test_vm_utils.py https://review.openstack.org/478749
#openstack-nova - 2017-07-14
00:47:45 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559
01:03:11 openstackgerrit zhangdaolong proposed openstack/python-novaclient master: Fix the inappropriate parameter name https://review.openstack.org/480826
01:33:37 mriedem mysql is case insensitive by default, right?
01:33:44 mriedem isn't that one of the mysql vs postgresql issues?
01:33:56 mriedem b/c https://developer.openstack.org/api-ref/compute/?expanded=add-a-single-tag-detail says "Tags are case sensitive."
01:33:59 mriedem so i'm not sure that's true
01:34:03 mriedem depending on db backend
01:34:24 melwitt mysql is case insensitive by default
01:34:33 mriedem yeah, so i guess that's a bogus statement
01:34:47 mriedem FOO and foo would end up the same
01:35:30 melwitt if there's any python caching in between that could be a case sensitive issue
01:36:28 melwitt I know that flavor names are case sensitive for that reason
01:36:55 mriedem i'm not following
01:36:55 melwitt request caching in the API by flavor ID
01:37:01 mriedem oh that
01:37:17 mriedem yeah that's also this bug https://review.openstack.org/#/c/467772/
01:37:40 melwitt yeah. a long time ago someone tried lowercasing flavor IDs before sending to the REST API and it broke people that had mixed case flavor IDs bc things weren't found in the cache that way
01:37:58 melwitt *lowercasing in novaclient
01:38:05 mriedem so if i set a tag of FOO on my server, and then check if FOO exists, it won't because the db says it's foo?
01:38:29 melwitt no that should match
01:38:48 mriedem ok
01:39:03 melwitt if there's request caching in the way you might not match FOO with Foo even though it would match in the DB is what I'm trying to say
01:39:29 mriedem what i'm hearing you saying is we should drop mysql support and only support postgresql
01:39:34 melwitt so that comment might be referring to some sort of request caching. if it's not, then it might be bogus
01:39:39 melwitt lol
01:40:08 melwitt yeah, the case insensitive default is not what I think most people would want
01:49:44 alex_xu oomichi: appreciate if you can help on the review of https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/api-no-more-extensions-pike, those patch are pretty easy, just need one more +2
02:13:19 mriedem Kevin_Zheng: some comments in https://review.openstack.org/#/c/469800/ but overall it's really nice
02:13:20 mriedem bbiab
02:13:52 Kevin_Zheng mriedem: Just saw it, thanks for the review
02:39:48 openstackgerrit Tuan Luong-Anh proposed openstack/nova master: Update URL home-page in documents according to document migration https://review.openstack.org/483757
02:42:39 openstackgerrit Matt Riedemann proposed openstack/python-novaclient master: Add functional test for resize-confirm plus quota validation https://review.openstack.org/483559
02:54:04 openstackgerrit Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/483423
03:00:46 openstackgerrit Merged openstack/nova master: Use plain routes list for os-cells endpoint instead of stevedore https://review.openstack.org/474869
03:15:49 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/483337
03:33:30 openstackgerrit Tuan Luong-Anh proposed openstack/nova master: Update URL home-page in documents according to document migration https://review.openstack.org/483757
03:34:50 openstackgerrit Zhen Qin proposed openstack/nova master: Avoid false positives of Jinja2 in Bandit scan https://review.openstack.org/479437
05:04:13 gaoyuan whoami
05:51:24 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Consistent policies https://review.openstack.org/479774
05:53:04 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Consistent policies https://review.openstack.org/479774
07:19:00 openstackgerrit Merged openstack/nova master: Fix typo https://review.openstack.org/483159
07:20:04 openstackgerrit Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955
07:23:34 bauzas stephenfin: gentle reminder for other folks that today is a French holiday in case people look at me
07:26:53 alex_xu bauzas: enjoy
07:33:32 gmann bauzas, nice. take some rest :)
07:46:43 openstackgerrit Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576
08:08:57 openstackgerrit Jan Gutter proposed openstack/nova master: Netronome SmartNIC Enablement https://review.openstack.org/483459
08:16:40 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800
08:36:43 gmann stephenfin, done, i only concern about notification one ( gibi can tell more tough ). https://review.openstack.org/#/c/478484/10
08:40:05 gibi gmann, stephenfin: the notification doc has multiple purpose.
08:40:46 gibi gmann, stephenfin: It describe how to add new notification so it helps contribution but it also talks about how now work internally (like versioning) as well as what are the existing notifications
08:40:49 gmann gibi, from user perspective also ? i mainly saw more implementation wise
08:41:03 gmann gibi, ohk
08:41:28 gibi gmann: the bottom of the doc has a list of existing notification with samples as such so it is for notification consumers too
08:41:31 gmann gibi, and version things are helpful for users like horizon etc?
08:41:44 gmann yea
08:42:01 gmann gibi, then we should divide the doc?
08:42:02 gibi gmann: yes. Not horizon today but searchlight and watcher as far as I kknow
08:42:18 gibi gmann: it would be nice to split up the doc but that should not be part of the doc migration
08:43:16 gmann gibi, yea, so IMO current one should go under contribution and then we extract consumer related part under user etc
08:43:21 gmann stephenfin, ^^
08:44:02 gmann because under reference it goes as ref and we also donot maintain those much :) but i may be wrong
08:44:23 gibi gmann: there is no single good place for the current doc so it is fine by me to put it under any section :)
08:50:21 gibi gmann, stephenfin: I summarized my view as a comment in the review
08:50:40 gmann gibi, thanks, that help
09:15:45 jangutter sean-k-mooney: would you ping me if you have a sec?
09:45:37 openstackgerrit Gábor Antal proposed openstack/nova master: Transform aggregate.update_prop notification https://review.openstack.org/462576
09:49:25 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(4/4) https://review.openstack.org/469800
09:55:08 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493
09:58:53 openstackgerrit yuanyue proposed openstack/nova master: Remove ports on create_port timeout during instance build https://review.openstack.org/483888
10:01:38 openstackgerrit jichenjc proposed openstack/nova master: Enhance support matrix document https://review.openstack.org/482013
10:06:59 openstackgerrit jichenjc proposed openstack/nova master: add description for migrate on 'resize' section https://review.openstack.org/481483
10:24:26 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257
10:24:26 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051
10:24:27 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777
10:27:14 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257
10:27:15 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777
10:28:28 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Consistent policies https://review.openstack.org/479774
10:29:03 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051
10:33:31 openstackgerrit sudhir agarwal proposed openstack/nova master: Replaces uuid.uuid4 with uuidutils.generate_uuid() https://review.openstack.org/481007
11:30:48 openstackgerrit Béla Vancsics proposed openstack/nova master: Transform servergroup.delete notification https://review.openstack.org/483902
11:59:37 openstackgerrit NidhiMittalHada proposed openstack/nova master: Nova treats hostname as pattern https://review.openstack.org/474949
12:02:43 sean-k-mooney jangutter: hi sorry just seeing this now
12:04:05 openstackgerrit Manas Mandlekar proposed openstack/nova master: An unexpected failure during live migration leaves the insatnce to migrating state forever. https://review.openstack.org/483911
12:13:18 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/477091
12:13:53 jangutter sean-k-mooney: no problem, I just got back from lunch.
12:14:34 jangutter sean-k-mooney: so, is os-vif still open for one portbinding?
12:16:04 openstackgerrit Andrey Kurilin proposed openstack/python-novaclient master: Allow tuple as for nics value https://review.openstack.org/475816
12:20:54 sean-k-mooney jangutter: what do you mean by port binding?
12:21:36 openstackgerrit Gábor Antal proposed openstack/nova master: Transform instance.resize_prep notification https://review.openstack.org/465081
12:22:05 sean-k-mooney jangutter: do you mean a new VIF_TYPE or an extention to the one of the plugins or A new port Profile. you can still submit a new Algrio portbinding to os-vif untill the end of next week i belive
12:22:38 sean-k-mooney /portbinding/port profile/
12:25:25 sean-k-mooney jangutter: looking at https://releases.openstack.org/pike/schedule.html the non client libary release freeze is next week which is the deadline that you have to meet to have changes to os-vif consumed in pike
12:28:08 jangutter sean-k-mooney: I've just finished up testing with one extra portprofile object added in upstream os-vif
12:28:47 jangutter sean-k-mooney: let me clean up the code a bit and submit it for your perusal...
12:30:22 sean-k-mooney jangutter: cool ill keep an eye out for the patch message in irc and take a lookin when you submit it. adding port profile objects is not a controversal addtion.
12:32:11 openstackgerrit Merged openstack/nova master: Use oslo.polcy DocumentedRuleDefault https://review.openstack.org/472634

Earlier   Later