Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-03-27
14:41:09 sfinucan hence the rename
14:41:16 sfinucan See https://review.openstack.org/450266
14:41:47 sean-k-mooney sfinucan: yes but my question is when do we need it?
14:41:50 sfinucan I'm going on the assumption that there are still instances where that won't be passed by neutron, though I don't know how I could go find out
14:41:52 johnthetubaguy I think its to support old neutron versions, I guess. At somepoint we can probably pull that, if we got microversion like API system
14:41:54 sfinucan Aha, jinx :)
14:42:31 sfinucan Yeah, I wouldn't even know where to start in investigating that
14:42:47 sfinucan ...besides just handing it off to neutron guys
14:43:02 sean-k-mooney ya as far as i know the openvsiwtch,ovn and odl drivers now pass the bridge as of newton
14:43:31 sean-k-mooney ill take a quick look and confirm
14:45:04 sfinucan sean-k-mooney: Awesome. If so we could drop that option entirely instead
14:46:13 sean-k-mooney sfinucan: im not seeing it in odl at first glance but it would be a small change to enable.
14:46:52 openstackgerrit Matthew Booth proposed openstack/nova master: Improve the readability of check_instance_state https://review.openstack.org/440536
14:53:12 bauzas mriedem: dansmith: gentle reminder for https://review.openstack.org/#/c/391060/
14:53:19 bauzas mriedem: do you want me to log a bug against that ?
14:54:01 dansmith ack
14:59:00 mriedem i thought i asked for a bug before, but can't remember
14:59:06 mriedem it at least needs a release note i think
14:59:10 mriedem but it is a bug imo
15:00:48 bauzas mriedem: I provided a note
15:00:51 jaypipes edleafe: pkgutils.walk_packages() thing looks cool. feel free to push a patch to os-traits on top of my refactor for the future one.
15:00:57 bauzas mriedem: but I can also provide a bug report
15:01:06 artom derekh, heya, bit of a weird blast from the past for you
15:01:18 sean-k-mooney sfinucan: it looks like only the ovs ml2 driver passes the bridge https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/mech_driver/mech_openvswitch.py#L101-L115
15:02:07 mriedem dansmith: sdague: am i reading this correctly that in grenade, we don't attach a volume on the old side and then detach it on the new side? https://github.com/openstack-dev/grenade/blob/master/projects/60_nova/resources.sh
15:02:17 mriedem i see we add a floating ip on the old side
15:02:28 sean-k-mooney sfinucan: we would have to modify odl/ovn to also set it to remove that option unfortunetly.
15:02:32 artom derekh, remember https://bugs.launchpad.net/nova/+bug/1622545 ? Do you by any chance have similar dumps for older openstack versions? The one you provided in comment #6 has been very useful for our downstream testing, and we'd like to backport the fix all the way back to Juno
15:02:32 openstack Launchpad bug 1622545 in OpenStack Compute (nova) "archive_deleted_rows isn't archiving instances" [Medium,Fix released] - Assigned to Dan Smith (danms)
15:03:16 mriedem johnthetubaguy: i've left comments in your cinder v3 spec https://review.openstack.org/#/c/373203/
15:03:17 sean-k-mooney sfinucan: or rather there ml2 drivers not ovn/odl themselves
15:03:39 dansmith mriedem: did you check cinder? might be in there
15:03:56 mriedem ooo looking
15:04:07 derekh artom: Hi, afraid not, this was a redeploy and the data from the old installation is long gone
15:04:14 dansmith mriedem: well, we have a BFV
15:04:27 artom derekh, figured. Oh well, it was worth a shot - thanks :)
15:04:33 mriedem ah https://github.com/openstack-dev/grenade/blob/master/projects/70_cinder/resources.sh#L130
15:04:37 sdague mriedem: we have a bvf on the cinder side, but not an explicit attach after iirc
15:04:58 mriedem ok,
15:05:12 mriedem for johnthetubaguy's cinder v3 spec i think we want a grenade scenario where we attach on the old side and detach on the new side,
15:05:13 derekh artom: its always worth a shot, sorry I don't have anything useful
15:05:24 mriedem because the detach flow is going to depend on how we attached the volume in the first place
15:07:45 mriedem johnthetubaguy: on https://review.openstack.org/#/c/373203/ my main concern was around the swap volume steps setting bdm.attachment_id,
15:07:57 mriedem which would trigger the new detach flow before we actually wanted that i think
15:08:11 mriedem johnthetubaguy: otherwise just some questions and clarifications but i'd be +2
15:08:14 johnthetubaguy mriedem: yeah, +1 the grenade these, that sounds good
15:08:42 johnthetubaguy mriedem: oh, thats the new swap flow, so it only applies when attachement_id is already set in the BDM right?
15:08:44 mriedem and i think on the shared connections section you could be more clear that we're not dealing with that in this spec, and why we are not of course
15:09:34 mriedem johnthetubaguy: it doesn't say that https://review.openstack.org/#/c/373203/21/specs/pike/approved/cinder-new-attach-apis.rst@265
15:09:53 johnthetubaguy mriedem: yeah, it should say that
15:10:27 johnthetubaguy mriedem: basically, like all the other ones, keep the old flow if the attachment_id is not there
15:11:01 mriedem ok i didn't see that in the swap volume part, really threw me off
15:11:16 mriedem feel free to point out where i'm blind, otherwise we should probably clarify
15:11:22 sfinucan sean-k-mooney: Hmm, OK can you leave that in the review (if you haven't done so already)_
15:11:46 sfinucan Not sure if that's worth doing. It might be. We should probably add a TODO in the code in any case
15:11:50 mriedem sdague: dansmith: johnthetubaguy: also, fyi, i'm traveling to CA for orientation this week, leaving in less than an hour, getting back home thursday night.
15:12:00 mriedem so i probably won't be around much the next few days
15:12:02 sdague ok, cool
15:12:04 sdague enjoy
15:12:12 dansmith mriedem: ack
15:12:19 mriedem i will not enjoy tucking my shirt in
15:12:25 mriedem or you know, wearing pants
15:12:30 dansmith heh
15:19:13 johnthetubaguy mriedem: good luck with orientation!
15:19:31 johnthetubaguy mriedem: sounds like a very good clarification we need in the spec
15:19:38 sdague don't forget your compass
15:22:08 bauzas mriedem: North is in that direction =>
15:22:42 sean-k-mooney sfinucan: sure, i was planning on requing it as part of the migration to os-vif objects so that would be a logical time to implement it.
15:23:16 mriedem don't worry guys, i've seen The Edge several times,
15:23:27 mriedem i know how to survive in the wild with a pine needle compass
15:24:04 openstackgerrit Balazs Gibizer proposed openstack/nova master: use context mgr in instance.delete https://review.openstack.org/443764
15:25:27 bauzas mriedem: I hugely watched McGyver when I was a child
15:25:46 bauzas so, nothing is scaring me about the wild
15:27:22 bauzas mriedem: just make sure you got you swiss army knife and you'll be good
15:29:17 mriedem i'm less worried about the bears than i am the tight connection i have
15:35:21 dims mriedem : safe travels!
15:37:50 openstackgerrit Jens Rosenboom proposed openstack/nova master: WIP: Fix network metadata information - part 3/3 https://review.openstack.org/450297
15:42:58 openstackgerrit Maciej Szankin proposed openstack/nova-specs master: Add remove-nova-cert spec https://review.openstack.org/448882
15:45:36 openstackgerrit Sujitha proposed openstack/nova master: Add description to policies in aggregates.py https://review.openstack.org/445230
15:50:19 openstackgerrit Sujitha proposed openstack/nova master: Add description to policies in cells_scheduler.py https://review.openstack.org/448260
16:19:30 openstackgerrit Ildiko Vancsa proposed openstack/nova master: Add Cinder V3 Detach calls https://review.openstack.org/438750
16:26:11 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Rename '[neutron] ovs_bridge' https://review.openstack.org/450266
16:26:11 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Move 'floating_ips' opts into 'network' https://review.openstack.org/431039
16:26:12 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Deprecate 'default_floating_pool' https://review.openstack.org/431038
16:26:12 openstackgerrit Stephen Finucane proposed openstack/nova master: conf: Add neutron.default_floating_pool https://review.openstack.org/431037
16:34:22 openstackgerrit Anthony PERARD proposed openstack/nova master: Add exclusion list for tempest for a libvirt+xen job https://review.openstack.org/449687
16:43:10 openstackgerrit dane-fichter proposed openstack/nova master: Use cursive for signature verification https://review.openstack.org/351232
16:53:29 openstackgerrit Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677
16:55:56 oomichi andreaf: about https://review.openstack.org/#/c/448851/ , cinder v2 api is deprecated in this Pike cycle and we don't have enough v3 clients.
16:56:19 oomichi andreaf: the above patch itself seems good for me
16:56:29 openstackgerrit Maxim Nestratov proposed openstack/nova master: libvirt: Virtuozzo containers config drive support https://review.openstack.org/449818
16:56:42 oomichi andreaf: oops, sorry here is wrong channel.. should use qa channel
16:56:51 abhi89 Hi All.. does anybody know which option was introduced in openstack when they deprecared 'verbose' option (used for INFO logging) form /etc/nova.conf
16:57:31 abhi89 'verbose' option in /etc/nova.conf in default section, was deprecated from mitaka
16:57:46 jaypipes ralonsoh: https://review.openstack.org/448282 fixed per your request.
16:58:42 ralonsoh jaypipes: I'll review again the patch
16:58:58 jaypipes ralonsoh: cheers
17:00:06 jaypipes johnthetubaguy, dansmith: would be great to get your feedback on https://review.openstack.org/#/c/435954/.
17:02:56 openstackgerrit Béla Vancsics proposed openstack/nova master: Transform instance.trigger_crash_dump notification https://review.openstack.org/396225
17:02:57 openstackgerrit Béla Vancsics proposed openstack/nova master: Ensure instance is in active state after notification test https://review.openstack.org/448655

Earlier   Later