Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-04
14:35:38 sean-k-mooney the code you referenced though in the original patch appears to be constructing a pci request
14:40:06 jangutter sean-k-mooney: agh, I forgot. VNIC_VIRTIO_FORWARDER also needs to be added to VNIC_TYPES_SRIOV.
14:40:31 jangutter sean-k-mooney: that triggers the pci_request for it.
14:40:35 sean-k-mooney jangutter: well yes and no
14:40:49 bauzas alex_xu: looks like I'm fortunate, I hitted the race
14:41:11 sean-k-mooney it does not have to be pci back it will be in your case but for intel i was hoping to use that vnic type without sriov vf
14:41:34 bauzas alex_xu: the good thing is that I had a working run, and then a failed one, so I just need to see how to bisect that
14:42:09 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add api-ref for os-quota-class-set APIs https://review.openstack.org/467938
14:42:16 sean-k-mooney at lest in some cases. i had not fully mapped out that but yes if we limit our selves to the netronome usecase you will need a pci device allocated
14:42:36 openstackgerrit Gábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/480214
14:44:17 sean-k-mooney jangutter: ignore the intel stuff for now that si still in flux and i cant going into detail anyway at this time
14:44:18 jangutter sean-k-mooney: yep, it's not necessarily that all forwarders will have a 1:1 mapping either.
14:44:51 jangutter sean-k-mooney: maybe this means a new os-vnic library....
14:46:08 sean-k-mooney sean-k-mooney: this is why i want ovs to track this and feed that info from neutron into the placement api so that we can handel these details in the ml2 driver and os-vif dirver generically
14:47:15 bauzas alex_xu: lots of fun, the working test has 309 tests, the failing test 310 :p
14:47:37 jangutter sean-k-mooney: yep, don't worry, you're not the only one thinking in that direction.
14:50:04 jangutter sean-k-mooney: ok, thanks very much. I'll get back on our TC interfaces post-haste and our refactored legacy driver should be done pretty soon.
14:50:43 bauzas alex_xu: I'm surprised, the test which fails wasn't run by the working test
14:53:33 bauzas actually nevermind
14:58:33 moshele sean-k-mooney: hi
14:59:21 moshele sean-k-mooney: is my comment https://review.openstack.org/#/c/398265/ answer your question ?
15:00:02 moshele sean-k-mooney: I have to go let talk tomorrow
15:21:29 openstackgerrit Gábor Antal proposed openstack/nova master: Transform instance.live_migration_rollback_dest notification https://review.openstack.org/480214
16:41:28 openstackgerrit Stephen Finucane proposed openstack/python-novaclient master: Remove custom autodoc implementation https://review.openstack.org/480262
16:44:34 openstackgerrit Sylvain Bauza proposed openstack/nova master: Reset the traits sync flag in the placement fixtures https://review.openstack.org/480198
16:45:48 bauzas stephenfin: guessing it's you :p
16:46:45 openstack Launchpad bug 1702304 in OpenStack Compute (nova) "MISC_SHARES_VIA_AGGREGATE not found still show up in the functional test" [High,In progress] - Assigned to Sylvain Bauza (sylvain-bauza)
16:46:45 bauzas stephenfin: don't know if you remember our convo with alex_xu but I found the root cause for https://bugs.launchpad.net/nova/+bug/1702304 so I provided a new PS https://review.openstack.org/#/c/480198/2
17:10:20 vdrok hi folks, super-easy patch if anyone around -- https://review.openstack.org/480167
18:20:00 openstackgerrit Ildiko Vancsa proposed openstack/nova master: WIP: Allow multi-attach in compute api https://review.openstack.org/271047
18:20:02 openstackgerrit Ildiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments https://review.openstack.org/267587
21:18:56 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
21:19:50 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Add Cinder v3 detach to swap_volume https://review.openstack.org/456971
21:19:51 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285
21:19:51 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Remove check_detach https://review.openstack.org/446671
23:45:30 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add 'networks' quota in quota sample files https://review.openstack.org/479227
23:49:01 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Use plain routes list for os-networks instead of stevedore https://review.openstack.org/474463
#openstack-nova - 2017-07-05
00:05:17 openstackgerrit Mathieu Gagné proposed openstack/nova master: Add ability to signal and perform online volume size change https://review.openstack.org/454322
00:39:43 Jeffrey4l hi guys, is there any consumer for nova versioned notification messages?
01:29:32 openstackgerrit Merged openstack/nova master: Don't delete neutron port when attach failed https://review.openstack.org/403568
01:35:15 Kevin_Zheng Jeffrey4l: Searchlight is going to consume those
01:37:36 Jeffrey4l Kevin_Zheng, is searchlight already finished the work?
01:37:57 Kevin_Zheng very close
01:37:59 Jeffrey4l and is there any other project ( ceilometer?) can consume versioned message?
01:39:48 Kevin_Zheng a good idea, bu no one worked on it in ceilometer
02:27:45 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Add tags to instance.create Notification https://review.openstack.org/459493
02:34:34 Nel1x what consecuences using admin_auth_url => port 5000? and not 35537 in nova => [neutron]
03:08:23 openstackgerrit Zhenyu Zheng proposed openstack/nova master: copy image id to image metadata from volume https://review.openstack.org/319905
06:23:08 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
06:35:05 masber hi, is this the right place to ask about the integration of nova with sr-iov?
06:46:40 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix arguments in calling _delete_nic_metadata https://review.openstack.org/480369
07:12:39 openstack Launchpad bug 1702304 in OpenStack Compute (nova) "MISC_SHARES_VIA_AGGREGATE not found still show up in the functional test" [Critical,In progress] - Assigned to Sylvain Bauza (sylvain-bauza)
07:12:39 bauzas alex_xu: stephenfin: follow-up on https://bugs.launchpad.net/nova/+bug/1702304 with https://review.openstack.org/#/c/480198/ needing reviews
07:12:55 alex_xu bauzas: cool, will check soon
07:13:17 openstackgerrit Alex Xu proposed openstack/nova master: placement: ensure shared RP maps with correct root RP https://review.openstack.org/480379
07:13:26 alex_xu bauzas: ^ another bug fix for placement
07:13:46 bauzas alex_xu: ack, looking
07:14:22 gibi Jeffrey4l: besides searchlight I think watcher also consumes nova versioned notifications
07:15:27 gibi Jeffrey4l: for example https://github.com/openstack/watcher/blob/3a3ce0268bbb900c0ca3fe40379beda574a4d548/watcher/decision_engine/model/notification/nova.py
07:35:00 Jeffrey4l gibi, roger. thanks a lot.
07:47:43 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.49 - Virt device tagged attach https://review.openstack.org/480392
07:54:53 openstackgerrit Chason Chan proposed openstack/python-novaclient master: switch to openstackdocstheme https://review.openstack.org/477757
08:13:00 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI https://review.openstack.org/478437
08:17:00 stephenfin bauzas: https://review.openstack.org/#/c/480198 reviewed. Moving onto the next one now
08:17:24 stephenfin wait - the next one's the same :)
08:17:45 bauzas what's the next one ? :p
08:18:09 stephenfin you pinged me last night to review https://review.openstack.org/#/c/480198, then again this morning
08:18:49 bauzas ha-ah
08:19:52 stephenfin Damn, this place is quiet without ze yanks around - I can actually read the scrollback on one page :O
08:24:35 gibi bauzas: hi! there is a doc generation improvement that only needs a second +2. If you have time could you take a look? https://review.openstack.org/#/c/475349/
08:24:50 bauzas gibi: sure
08:25:27 gibi bauzas: thanks!
08:27:46 bauzas gibi: ack my comment and I +W https://review.openstack.org/#/c/475349/1
08:28:05 bauzas based on takashi's valid point
08:35:14 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI https://review.openstack.org/478437
08:35:47 stephenfin bauzas: I don't think you +W'd that yet, if you plan to?
08:36:20 bauzas stephenfin: because I'd like to see gibi adding a follow-up
08:36:37 stephenfin aha - gotcha
08:37:37 stephenfin bauzas: or gibi could just fix it in the same patch, seeing as we're both online
08:37:44 stephenfin either-or
08:38:44 gibi bauzas, stephenfin: answered. I will propose a follow where I try to enable pep8 on these source files
08:39:42 stephenfin gibi: Eh, maybe you could leave that for a bit while we finish up doc-migration?
08:39:56 stephenfin Or at least, push it on top of https://review.openstack.org/#/c/479358/
08:40:21 stephenfin I've modified doc/source/conf.py in a few of those, and I really don't want to rebase a 15 patch series if I can help it :P
08:40:49 stephenfin but that's your call, of course
08:41:13 gibi stephenfin: I don't think that your series and my patch conflicts as I dont touch your the files you touch
08:41:43 gibi stephenfin: mine only changes doc/ext/versioned_notifications.py
08:42:02 gibi stephenfin: the followup pep8 patch might conflict with yours but then I will push that top of your series
08:42:19 stephenfin gibi: yup - I'm referring to the follow-up
08:42:40 gibi stephenfin: cool, then we are on the same page.
08:42:49 stephenfin (y)
08:43:27 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi: Exception Error logs shown in Citrix XenServer CI https://review.openstack.org/478437
08:47:29 openstackgerrit Chason Chan proposed openstack/python-novaclient master: switch to openstackdocstheme https://review.openstack.org/477757
08:51:37 openstackgerrit Merged openstack/nova master: Reset the traits sync flag in the placement fixtures https://review.openstack.org/480198
09:16:27 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Support tag instances when boot(3/4) https://review.openstack.org/459593
09:18:09 bauzas sfinucan: test courtesy ping
09:20:59 openstackgerrit Silvan Kaiser proposed openstack/nova master: [WIP] Add truncate for images if prealloc is true https://review.openstack.org/480464
09:22:31 openstackgerrit Silvan Kaiser proposed openstack/nova master: [WIP] Add truncate for images if prealloc is true https://review.openstack.org/480464
09:23:07 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/nova master: Read Neutron port 'binding_profile' during boot https://review.openstack.org/449257
09:23:28 openstackgerrit Silvan Kaiser proposed openstack/nova master: [WIP] Add truncate for images if prealloc is true https://review.openstack.org/480464

Earlier   Later