Earlier  
Posted Nick Remark
#openstack-nova - 2017-08-10
22:40:48 mriedem which would create the custom resource class from the node we're tracking
22:41:00 mriedem the admin could pre-create the custom resource classes, sure
22:41:11 mriedem but nova is also trying to create them if they don't already exist in placement and we have inventory for them
22:41:36 mriedem i'm no baremetal expert though, so i do'nt know the order in which things need to happen here to auto-create the custom resource class
22:41:45 edleafe well, just trying to figure out how to fix this. Gotta run out in a few minutes
22:42:05 mriedem i don't know if there is something to fix on the nova side,
22:42:15 mriedem and i haven't dug into the devstack changes in the ironic WIP patch
22:42:30 mriedem so i might just throw this in rc1
22:42:30 edleafe they're selecting a flavor with the custom RC, but placement is barfing on that since it never got created.
22:42:33 mriedem and deal with any issues in rc2
22:42:35 edleafe yeah
22:42:47 edleafe i'm not sure that's the best response from placement
22:43:06 mriedem rather than just not return allocation candidates you mean?
22:43:13 edleafe I mean I understand preventing typos and stuff
22:43:37 mriedem it seems ok, you're asking for allocation_candidates filtering on something which doesn't exist
22:44:00 mriedem and you should probably make sure you can ask for something that's in GET /resource_classes
22:44:16 mriedem it's a client side error somewhere
22:44:34 edleafe ok, so I guess the 400 is correct
22:44:44 edleafe the client can determine how to deal with that
22:45:33 edleafe I gotta run - will read the scrollback in the morning
23:22:42 openstackgerrit Matt Riedemann proposed openstack/nova master: Remove "dhcp_options_for_instance" https://review.openstack.org/492707
23:23:32 mtreinish efried: ok try: https://review.openstack.org/#/c/492651/ again
23:23:48 mtreinish I had to start playing with config settings locally and I think devstack should match the config which worked for me locally
23:30:36 efried mtreinish ack.
23:31:33 efried mtreinish Just got restacked with mod_wsgi, though, so I may push it out a bit.
23:48:37 mtreinish efried: no worries, I don't think it works anyway. It's still failing the gate
23:48:49 mtreinish I'll have to play with it some more tomorrow
#openstack-nova - 2017-08-11
00:32:44 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: List/show all server migration types https://review.openstack.org/489029
00:33:00 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Enable cold migration with target host https://review.openstack.org/489031
00:33:47 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Update a URL https://review.openstack.org/489028
01:29:30 dims alex_xu : hope you feel better
01:29:54 dims soon :)
01:52:25 openstackgerrit jichenjc proposed openstack/nova master: Remove RamFilter and DiskFilter in default filter https://review.openstack.org/492765
02:29:51 openstackgerrit Merged openstack/nova master: Remove provider allocs in confirm/revert resize https://review.openstack.org/488510
04:31:15 openstackgerrit Michael Still proposed openstack/nova master: Avoid chowning console logs in libvirt https://review.openstack.org/472229
04:31:16 openstackgerrit Michael Still proposed openstack/nova master: Move execs of touch to privsep. https://review.openstack.org/489190
04:31:16 openstackgerrit Michael Still proposed openstack/nova master: First attempt at adding a privsep user to nova itself. https://review.openstack.org/459166
04:31:17 openstackgerrit Michael Still proposed openstack/nova master: Move execs of tee to privsep. https://review.openstack.org/489438
04:31:17 openstackgerrit Michael Still proposed openstack/nova master: Move libvirts dmcrypt support to privsep. https://review.openstack.org/490737
04:31:18 openstackgerrit Michael Still proposed openstack/nova master: Don't shell out to mkdir, use ensure_tree() https://review.openstack.org/492326
04:31:19 openstackgerrit Michael Still proposed openstack/nova master: Move libvirt usages of chown to privsep. https://review.openstack.org/471972
04:31:20 openstackgerrit Michael Still proposed openstack/nova master: Move ploop commands to privsep. https://review.openstack.org/492325
04:31:21 openstackgerrit Michael Still proposed openstack/nova master: Read from console ptys using privsep. https://review.openstack.org/489486
04:31:22 openstackgerrit Michael Still proposed openstack/nova master: Refactor libvirt.utils.execute() away. https://review.openstack.org/489816
07:06:49 openstackgerrit Merged openstack/nova master: Resource tracker compatibility with Ocata and Pike https://review.openstack.org/491012
07:50:51 openstackgerrit jichenjc proposed openstack/nova master: refactory some code on get_password https://review.openstack.org/449458
07:51:10 openstackgerrit Markus Zoeller (markus_z) proposed openstack/nova master: docs: Explain the flow of the "serial console" feature https://review.openstack.org/476188
08:24:49 openstackgerrit Ilya Popov proposed openstack/nova master: Tests: Add cleanup of 'instances' directory https://review.openstack.org/491589
09:27:21 openstackgerrit yfzhao proposed openstack/python-novaclient master: Fix wrong links https://review.openstack.org/492921
09:34:41 gibi good morning
09:39:41 bauzas gibi: good lunch :p
09:41:47 gibi bauzas: the day started earlier a bit but I was occupied with internal meetings
09:42:03 gibi bauzas: but yes, a lunch is about to happen too
09:42:42 gibi bauzas: did Matt cut rc1 last night?
09:43:12 bauzas gibi: no
09:43:23 bauzas gibi: https://review.openstack.org/#/c/492788/2
09:43:39 bauzas he was waiting for the merge IIRC
09:43:53 bauzas gibi: now, the main problem is about the Ironic job
09:43:57 bauzas dtantsur: around ?
09:44:25 bauzas dtantsur: context is https://review.openstack.org/#/c/487954/14
09:44:37 openstackgerrit Balazs Gibizer proposed openstack/nova master: replace chance with filter scheduler in func tests https://review.openstack.org/491529
09:44:52 bauzas dtantsur: that change is atm a RC1 blocker but I don't see the Ironic job providing the log saying that the flavor is modified
09:45:06 gibi bauzas: thanks for the info
09:45:33 bauzas dtantsur: if we cut RC1 without ^, that could possibly be an upgrade problem for Queens
09:45:55 bauzas so I would appreciate any ironic folks support now :)
09:46:06 bauzas TheJulia: around too ?
09:47:58 smcginnis bauzas: Do you need to hold that RC request?
09:49:34 smcginnis bauzas: Oh, nevermind. Now I see your comment there.
09:50:04 bauzas smcginnis: yeah, I'd like to discuss with matt about whether we cut RC1 without the change I provided in my comment
09:50:21 bauzas smcginnis: do you accept that ? is it possible to cut RC1 only on Friday?
09:50:30 bauzas I mean, it's also a branch
09:50:41 smcginnis bauzas: I think we're probably fine waiting until Monday if we have to.
09:50:49 smcginnis bauzas: You're just waiting on one patch, right?
09:51:05 bauzas smcginnis: yup, I just rebased the releases tag with the merge sha1
09:51:32 bauzas smcginnis: so if we just cut the tag without the above change, a +W would only be needed
09:51:51 bauzas the only blocker for us is https://review.openstack.org/#/c/487954/
09:52:18 bauzas other bugs would be treated in other RCs
09:52:34 bauzas HTH
09:52:36 smcginnis bauzas: Cool, I think we're fine waiting a little then.
09:55:27 dtantsur morning bauzas
09:55:52 dtantsur I'll start looking in it in a few minutes
10:00:23 openstackgerrit Balazs Gibizer proposed openstack/nova master: test server evacuation with placement https://review.openstack.org/492548
10:03:15 bauzas dtantsur: FWIW, I'm forging a DNM patch for adding more voerbosity
10:04:29 dtantsur cool
10:16:55 openstackgerrit Sylvain Bauza proposed openstack/nova master: Make compute log less verbose with allocs autocorrection https://review.openstack.org/492571
10:18:56 dtantsur bauzas: I don't really understand the CI failure there.. it seems unrelated to me
10:19:19 bauzas dtantsur: hence me logging more
10:19:27 bauzas dtantsur: I'm even not sure we run the thread
10:19:43 bauzas given it's in a conditional
10:20:55 dtantsur we should not, it only runs for nodes with resource_class != None
10:21:25 dtantsur bauzas: I can make https://review.openstack.org/#/c/491777/ depends-on this patch, to see how it behaves with resource_classes. wdyt?
10:22:14 bauzas dtantsur: sure
10:22:43 bauzas dtantsur: if that doesn't trample you waiting for devstack patch being merged
10:23:00 bauzas dtantsur: I mean, if you can wait for that devstack patch to be merged, that's fine to me
10:23:01 dtantsur we need to make sure it won't blow up after merging the both changes :)
10:23:06 bauzas yeah
10:23:32 bauzas anyway, just doing an urgent internal bug thingy and then I'm back to the DNM patch
10:23:43 dtantsur ack
10:27:44 dtantsur ok, both patches updated, waiting for the CI
10:41:06 openstackgerrit Dmitry Tantsur proposed openstack/nova master: Deprecate bare metal filters https://review.openstack.org/492563

Earlier   Later