| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-10 | |||
| 21:56:55 | mriedem | would be nice if someone from ironic could say, i tested this manually and it's fine | |
| 21:57:00 | mriedem | edleafe: ^ did anyone test manually? | |
| 21:57:12 | edleafe | mriedem: not that I know of | |
| 21:58:38 | mriedem | ok i'll ask in -ironic but i think most of that team is gone for the day | |
| 21:58:42 | mriedem | bbibab | |
| 21:58:44 | mriedem | *bbiab even | |
| 22:01:02 | openstackgerrit | Merged openstack/nova master: Improve stable-api doc with current API state https://review.openstack.org/489926 | |
| 22:18:09 | bauzas | folks, see you tomorrow | |
| 22:18:27 | bauzas | will look at the branch if any | |
| 22:27:37 | edleafe | So it looks like when Ironic sets the resource_class for a node, it doesn't do anything to create that in Placement | |
| 22:28:00 | edleafe | which would explain http://logs.openstack.org/68/476968/12/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial/02053cf/logs/screen-n-sch.txt.gz#_Aug_09_20_35_41_533621 | |
| 22:33:51 | edleafe | mriedem: should I add code to ensure that the custom resource class for a node exists in that migration? I don't see anything in ironic where it is created | |
| 22:33:58 | kevinbenton | mriedem: kernel panic in VM http://logs.openstack.org/10/488510/33/gate/gate-tempest-dsvm-neutron-full-ubuntu-xenial/8b65cd3/logs/testr_results.html.gz | |
| 22:34:04 | kevinbenton | mriedem: how often does that happen? | |
| 22:36:39 | mriedem | edleafe: no, not in your change. it's used in the scheduler. the custom resource class is created in placement via the periodic updates in the RT | |
| 22:36:44 | mriedem | via the get_inventory() method to the ironic driver | |
| 22:37:57 | edleafe | mriedem: so that's not being run before the test failure above | |
| 22:38:59 | mriedem | edleafe: it's run on start of the compute service | |
| 22:39:03 | mriedem | and in the update_available_resource periodic, | |
| 22:39:09 | mriedem | the problem is we're not reporting any inventory for the node | |
| 22:39:27 | mriedem | so yeah, maybe the problem is a chicken and egg issue, idk | |
| 22:39:39 | mriedem | does the inventory not show up until we have a node that we're tracking with an instance? | |
| 22:39:49 | mriedem | and we don't have the instance w/o the custom resource class that the node is using | |
| 22:40:32 | edleafe | the error isn't that there is no inventory; it's that there is no such resource class | |
| 22:40:33 | mriedem | that interaction is a black box to me right now | |
| 22:40:42 | mriedem | edleafe: because we didn't PUT any inventory | |
| 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 | |