Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-01-18
14:46:47 alex_xu jaypipes: efried after this series finish https://review.openstack.org/#/q/topic:bp/nested-resource-providers+(status:open+OR+status:merged), we still only support single request group, right? we still need some SQL to support multiple request groups?
14:46:59 jaypipes alex_xu: yes sir
14:48:04 efried alex_xu I'm not sure we'll need extra sql per se; I'm hoping to be able to mix and match the existing helper methods including the ones jaypipes is working on now.
14:48:15 alex_xu jaypipes: ok...so I guess granular request isn't target to this release?
14:48:22 jaypipes alex_xu: nope, it never was.
14:48:36 efried uh
14:48:39 efried Yes it was.
14:48:42 efried Approved blueprint.
14:48:53 efried All the foundational code is merged.
14:48:55 alex_xu jaypipes: ok...so we didn't have API patch to expose trait in this release...
14:49:11 jaypipes alex_xu: cdent has a patch proposed for that
14:49:17 jaypipes alex_xu: one sec, grabbing link
14:50:03 jaypipes alex_xu: https://review.openstack.org/#/c/534886/
14:50:06 efried The last pieces of granular enablement are here: https://review.openstack.org/#/c/517757/ (placement side); and https://review.openstack.org/#/c/515811/ (scheduler side)
14:50:29 efried With ^, cdent's patch won't be necessary.
14:50:33 jaypipes alex_xu: you want to take over cdent's patch?
14:50:59 alex_xu jaypipes: yea, if cdent is ok for that
14:51:09 efried Hello? Can you hear me?
14:51:15 jaypipes sorry efried
14:51:27 alex_xu efried: yea, I hear you, sorry about that :)
14:52:02 jaypipes ack, efried's patches are better/further along
14:52:30 efried alex_xu If you've got the bandwidth and inclination to help, I would much rather have you work on https://review.openstack.org/#/c/517757/
14:52:43 jaypipes efried: the issue is that we really want the traits handling to be done even if granular request groups isn't merged in queens
14:52:53 efried jaypipes Yeah, I get that.
14:53:10 efried It would just be a shame to duplicate the effort.
14:53:29 alex_xu efried: it is a little hard to finish that, i guess it still need some more magic sql
14:53:30 efried And the granular stuf is [ ] this close.
14:54:40 efried alex_xu It shouldn't. The idea there is to combine the existing helpers. We have one helper that targets an *individual* provider - that would be used for each granular request. Then we have another helper (the one jaypipes is working on) that targets trees - that would be used for the non-granular request.
14:55:08 efried alex_xu Then when we have the results from all of those, we just have to combine them in permutations such that each result uses the same non-sharing tree.
14:55:44 efried There may be a way to do it "more efficiently" with some magic sql, but we can optimize that laaaaater.
14:58:43 alex_xu efried: I can take a look at that, but we just left one week, I still thought it isn't easy thing
14:59:25 efried alex_xu Nothing is easy :)
15:17:02 openstackgerrit Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
15:17:04 sean-k-mooney spotz: did you get your question about booting a vm on a subnet answered
15:18:03 sean-k-mooney spotz: today you have to first create the neutron port on the subnet and then add --nic port-id=<uuid of port you created>
15:18:58 sean-k-mooney spotz: nova only supports creating porst for you automatically when specifying a network. anything more advanced and you have to bring your own ports
15:32:34 jaypipes efried: k, back from breakfasting.
15:32:53 efried jaypipes Okay, made it through your comments. Would like to refine on a couple of points before moving forward with deltas.
15:33:38 jaypipes efried: k. hangout?
15:33:48 efried okay.
15:35:38 efried jaypipes Are you setting that up or should I?
15:36:28 amorin hello everybody
15:36:44 amorin I'd like to cherry-pick this commit again:
15:36:46 amorin https://review.openstack.org/#/c/508640/
15:36:48 jaypipes efried: I will, just warming coffeee
15:36:50 amorin how can we reopen it?
15:37:56 efried amorin You should still be able to cherry-pick it even though it's abandoned, nah?
15:38:53 jaypipes efried: https://hangouts.google.com/call/Kkf0T_Tilgyrg2Hctaz8AAEE
15:39:12 amorin efried: both gerrit web interface and git review throw me this error:
15:39:19 amorin ! [remote rejected] HEAD -> refs/publish/stable/ocata/bug/1662626-cp-ocata (change https://review.openstack.org/508640 closed)
15:39:33 efried Okay, a core ought to be able to un-abandon it for you.
15:40:08 amorin efried: ok, who should I ask?
15:40:29 ildikov amorin: efried: this is the original commit: https://review.openstack.org/#/c/430400/
15:41:25 amorin true
15:41:30 ildikov amorin: efried: that should be ok for cherry-picking
15:41:36 amorin ildikov: this is the one I am trying to cherry-pick
15:41:48 amorin Cannot create new patch set of change 508640 because it is abandoned
15:41:51 openstackgerrit Merged openstack/nova master: placement: _get_trees_matching_all_resources() https://review.openstack.org/531512
15:42:01 openstackgerrit Merged openstack/nova master: Fix openstackdocstheme options for api-ref https://review.openstack.org/534712
15:42:02 amorin from web page
15:42:10 ildikov amorin: what I've just linked in that's merged
15:42:33 ildikov amorin: Change 430400
15:43:12 ildikov amorin: you're trying with an already cherry-picked patch which landed on Ocata and got abandoned after Matt's comment that it should've been backported to Pike first
15:43:27 amorin yup
15:43:36 amorin but it's already in pike actually
15:43:52 amorin I think it was merged while pike was the master branch
15:44:52 ildikov I didn't check that
15:45:10 ildikov are you trying to get it landed on Ocata?
15:45:22 amorin yes, both Ocata and newton if possible
15:45:22 gibi amorin, ildikov: yeah it seems that it is in stable/pike according to gerrit
15:45:37 ildikov gibi: yeah, I've just checked
15:45:37 amorin reopen this https://review.openstack.org/#/c/508640/
15:45:42 amorin and this
15:45:44 amorin https://review.openstack.org/#/c/508641/
15:45:58 ildikov amorin: I think Newton is EOL now, but gibi should be smarter
15:46:03 amorin because the related but : https://bugs.launchpad.net/nova/+bug/1662626
15:46:04 openstack Launchpad bug 1662626 in nova (Ubuntu Xenial) "live-migrate left in migrating as domain not found" [Medium,Triaged]
15:46:14 ildikov amorin: in both cases you only need to get the patch re-opened
15:46:15 amorin is supposed to be fixed
15:46:25 ildikov amorin: no more cherry-picking is needed
15:46:40 amorin ildikov: ok, so I need some core to reopen it right?
15:46:47 ildikov amorin: https://review.openstack.org/#/q/b706155888d740841d745c52aae543cf82fab0bc
15:46:58 ildikov amorin: yes, correct
15:47:13 ildikov amorin: gibi has the power for instance :)
15:47:15 gibi amorin: it seems you need a stable core to reopen the abandoned patches
15:47:21 gibi ildikov: it seems not :)
15:47:32 ildikov gibi: sadness :(
15:47:37 amorin :(
15:47:58 gibi I only have such power on master branch
15:48:38 gibi amorin: I think this group is the nova stable cores https://review.openstack.org/#/admin/groups/540,members
15:48:51 amorin thanks
15:50:25 amorin is there any official way to contact them without too much bothering?
15:51:59 spotz sean-k-mooney: I actually was able to do it and by just giving it an IP in the subnet range so I let the student know that method
15:52:25 spotz sean-k-mooney: plus specifiying the network
15:52:54 gibi bauzas: could you help amorin reopening these abandoned stable backports? https://review.openstack.org/#/q/I23ed9819061bfa436b12180110666c5b8c3e0f70
15:53:03 gibi bauzas: I have no stable power
15:54:00 gibi dansmith, tonyb ^^
15:55:29 ameeda what about this log ? undercloud.bootpc > gateway.bootps: BOOTP/DHCP, Request from 00:a7:b7:5c:d0:dd (oui Unknown), length 30
15:55:33 lyarwood gibi: the stable/ocata change is open, newton is EOL so I'd rather not open it up
15:55:35 bauzas gibi: sure, doing it

Earlier   Later