| *** rcernin has quit IRC | 00:17 | |
| *** rcernin has joined #openstack-ironic | 00:17 | |
| *** dsneddon has joined #openstack-ironic | 00:26 | |
| *** ociuhandu has joined #openstack-ironic | 01:25 | |
| *** dsneddon has quit IRC | 01:30 | |
| *** ociuhandu has quit IRC | 01:32 | |
| *** dsneddon has joined #openstack-ironic | 02:00 | |
| *** ociuhandu has joined #openstack-ironic | 02:02 | |
| *** ociuhandu has quit IRC | 02:06 | |
| *** ociuhandu has joined #openstack-ironic | 02:19 | |
| *** ociuhandu has quit IRC | 02:24 | |
| openstackgerrit | Merged openstack/ironic-prometheus-exporter master: Drop python 2.7 support and testing https://review.opendev.org/695739 | 02:25 |
|---|---|---|
| *** chenhaw has joined #openstack-ironic | 02:30 | |
| *** ociuhandu has joined #openstack-ironic | 03:33 | |
| *** cp has quit IRC | 03:35 | |
| *** ociuhandu has quit IRC | 03:40 | |
| *** vesper11 has quit IRC | 03:45 | |
| *** vesper11 has joined #openstack-ironic | 03:47 | |
| *** ricolin has joined #openstack-ironic | 03:54 | |
| *** ociuhandu has joined #openstack-ironic | 04:14 | |
| *** cp has joined #openstack-ironic | 04:20 | |
| *** ociuhandu has quit IRC | 04:22 | |
| *** ociuhandu has joined #openstack-ironic | 04:22 | |
| *** ociuhandu has quit IRC | 04:23 | |
| *** ociuhandu has joined #openstack-ironic | 04:25 | |
| *** ociuhandu has quit IRC | 04:34 | |
| *** ociuhandu has joined #openstack-ironic | 04:36 | |
| *** ociuhandu has quit IRC | 04:41 | |
| *** ociuhandu has joined #openstack-ironic | 04:54 | |
| *** rachit7 has joined #openstack-ironic | 04:59 | |
| *** ociuhandu has quit IRC | 04:59 | |
| *** mkrai has joined #openstack-ironic | 05:09 | |
| *** ociuhandu has joined #openstack-ironic | 05:30 | |
| *** dsneddon has quit IRC | 05:32 | |
| *** dsneddon has joined #openstack-ironic | 05:34 | |
| *** ociuhandu has quit IRC | 05:35 | |
| *** dsneddon has quit IRC | 05:40 | |
| *** dsneddon has joined #openstack-ironic | 05:45 | |
| *** dsneddon has quit IRC | 05:50 | |
| *** jistr has quit IRC | 05:51 | |
| *** jistr has joined #openstack-ironic | 05:56 | |
| *** jistr has quit IRC | 06:06 | |
| *** jistr has joined #openstack-ironic | 06:06 | |
| *** cp has quit IRC | 06:21 | |
| *** dsneddon has joined #openstack-ironic | 06:26 | |
| *** jtomasek has joined #openstack-ironic | 06:33 | |
| *** cp has joined #openstack-ironic | 06:34 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic-python-agent master: Rename job to mention python3 https://review.opendev.org/695841 | 06:36 |
| *** cp has quit IRC | 06:42 | |
| *** cp- has joined #openstack-ironic | 06:49 | |
| *** rcernin has quit IRC | 07:09 | |
| *** dtantsur|afk is now known as dtantsur | 07:17 | |
| dtantsur | morning ironic | 07:17 |
| arne_wiebalck | Good morninig dtantsur and ironic! | 07:24 |
| chenhaw | hi all, am looking for guides to setup Ironic on ubuntu 18.04, anyone has any good resources for reference? | 07:31 |
| *** dsneddon has quit IRC | 07:35 | |
| openstackgerrit | Merged openstack/networking-baremetal master: Fix region option name in documentation https://review.opendev.org/695660 | 07:36 |
| arne_wiebalck | Hi chenhaw, I'd start from here: https://docs.openstack.org/ironic/latest/install/install-ubuntu.html | 07:36 |
| *** ociuhandu has joined #openstack-ironic | 07:38 | |
| *** kaifeng|afk is now known as kaifeng | 07:40 | |
| kaifeng | morning dantsur arne_wiebalck o/ | 07:40 |
| arne_wiebalck | Good morning, kaifeng! o/ | 07:41 |
| *** ociuhandu has quit IRC | 07:44 | |
| openstackgerrit | Madhuri Kumari proposed openstack/ironic-python-agent master: Rename job to mention python3 https://review.opendev.org/695841 | 07:51 |
| *** dsneddon has joined #openstack-ironic | 08:00 | |
| *** dsneddon has quit IRC | 08:05 | |
| *** dsneddon has joined #openstack-ironic | 08:12 | |
| *** tesseract has joined #openstack-ironic | 08:16 | |
| *** dsneddon has quit IRC | 08:16 | |
| *** dsneddon has joined #openstack-ironic | 08:22 | |
| *** dsneddon has quit IRC | 08:27 | |
| *** mkrai has quit IRC | 08:31 | |
| *** rpittau|afk is now known as rpittau | 08:33 | |
| rpittau | good morning ironic! o/ | 08:33 |
| *** ociuhandu has joined #openstack-ironic | 08:35 | |
| *** ociuhandu has quit IRC | 08:40 | |
| kaifeng | hi rpittau o/ | 08:44 |
| rpittau | hey kaifeng :) | 08:45 |
| *** ociuhandu has joined #openstack-ironic | 08:50 | |
| *** lucasagomes has joined #openstack-ironic | 08:55 | |
| *** dougsz has joined #openstack-ironic | 09:05 | |
| openstackgerrit | raphael.glon proposed openstack/ironic master: Feature: finalize_rescue, send sha512 hashed password, not clear ones https://review.opendev.org/695649 | 09:06 |
| openstackgerrit | Mark Goddard proposed openstack/tenks master: CI: Use python3 in deploy-teardown jobs https://review.opendev.org/695738 | 09:09 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Replace WSME and Pecan with Werkzeug https://review.opendev.org/695041 | 09:09 |
| *** ricolin has quit IRC | 09:16 | |
| dtantsur | kaifeng, hey, could you check if https://review.opendev.org/#/c/681261/ and https://review.opendev.org/#/c/688720/ can be approved? both have 2x +2 | 09:19 |
| patchbot | patch 681261 - ironic - Foundation for boot/network management for in-band... - 14 patch sets | 09:19 |
| patchbot | patch 688720 - ironic - Wire in in-band inspection for PXE boot and neutro... - 16 patch sets | 09:20 |
| dtantsur | or rpittau if you have some time ^^ | 09:20 |
| kaifeng | dtantsur: np | 09:20 |
| *** Lucas_Gray has joined #openstack-ironic | 09:21 | |
| *** dsneddon has joined #openstack-ironic | 09:31 | |
| *** alexmcleod has joined #openstack-ironic | 09:32 | |
| *** mkrai has joined #openstack-ironic | 09:32 | |
| *** dsneddon has quit IRC | 09:37 | |
| *** tssurya has joined #openstack-ironic | 09:37 | |
| *** ociuhandu has quit IRC | 09:40 | |
| *** iurygregory has joined #openstack-ironic | 09:40 | |
| iurygregory | good morning o/ | 09:40 |
| rpittau | hey iurygregory :) | 09:42 |
| iurygregory | rpittau, o/ | 09:42 |
| kaifeng | iurygregory o/ | 09:43 |
| iurygregory | kaifeng, o/ | 09:43 |
| *** derekh has joined #openstack-ironic | 09:43 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-inspector-specs master: Drop python 2.7 support and testing https://review.opendev.org/695731 | 09:46 |
| *** kaifeng is now known as kaifeng|afk | 09:47 | |
| *** chihhsin has quit IRC | 09:48 | |
| *** ociuhandu has joined #openstack-ironic | 09:49 | |
| *** ociuhandu has quit IRC | 09:54 | |
| *** jistr is now known as jistr|afk | 09:58 | |
| * iurygregory is wondering is this will bring trouble to our projects that are testing centos7 with devstack https://review.opendev.org/#/c/694967/ for e.g metalsmith | 09:58 | |
| patchbot | patch 694967 - devstack - Drop centos7 platform job (MERGED) - 1 patch set | 09:58 |
| * iurygregory nvm its only the job that was dropped | 09:59 | |
| *** mkrai has quit IRC | 10:02 | |
| *** dsneddon has joined #openstack-ironic | 10:05 | |
| dtantsur | iurygregory: metalsmith is not doing centos with devstack. it has a job with a centos-based user image. | 10:13 |
| iurygregory | dtantsur, yeah I checked XD | 10:13 |
| iurygregory | when you have time check my comments on the patch o/ | 10:13 |
| iurygregory | oh you checked =D | 10:14 |
| * iurygregory needs more coffee | 10:14 | |
| dtantsur | :) | 10:16 |
| dtantsur | that's Monday | 10:16 |
| *** sshnaidm|off is now known as sshnaidm | 10:18 | |
| iurygregory | make sense XD | 10:19 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/metalsmith master: Drop python 2.7 support and testing https://review.opendev.org/695727 | 10:20 |
| *** rachit7 has quit IRC | 10:21 | |
| *** jistr|afk is now known as jistr | 10:22 | |
| openstackgerrit | Merged openstack/sushy-tools master: Drop python 2.7 support and testing https://review.opendev.org/695799 | 10:22 |
| openstackgerrit | Merged openstack/ironic-python-agent-builder master: Drop python 2.7 support and testing https://review.opendev.org/695797 | 10:25 |
| *** ociuhandu has joined #openstack-ironic | 10:31 | |
| *** ociuhandu has quit IRC | 10:36 | |
| *** chihhsin has joined #openstack-ironic | 10:41 | |
| *** iurygregory has quit IRC | 10:43 | |
| openstackgerrit | Mark Goddard proposed openstack/tenks master: CI: Use python3 in deploy-teardown jobs https://review.opendev.org/695738 | 10:46 |
| openstackgerrit | Mark Goddard proposed openstack/tenks master: WIP: CentOS 8 support https://review.opendev.org/695881 | 10:58 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Replace WSME and Pecan with Werkzeug https://review.opendev.org/695041 | 11:05 |
| *** chenhaw has quit IRC | 11:07 | |
| *** dsneddon has quit IRC | 11:10 | |
| *** dsneddon has joined #openstack-ironic | 11:12 | |
| *** pcaruana has joined #openstack-ironic | 11:14 | |
| *** dsneddon has quit IRC | 11:17 | |
| *** Lucas_Gray has quit IRC | 11:19 | |
| *** ociuhandu has joined #openstack-ironic | 11:23 | |
| *** openstack has joined #openstack-ironic | 11:53 | |
| *** ChanServ sets mode: +o openstack | 11:53 | |
| *** mkrai has quit IRC | 11:54 | |
| *** ociuhandu has joined #openstack-ironic | 11:54 | |
| *** jtomasek has joined #openstack-ironic | 11:55 | |
| *** iurygregory has joined #openstack-ironic | 12:05 | |
| *** mkrai has joined #openstack-ironic | 12:05 | |
| *** mkrai has quit IRC | 12:14 | |
| *** mkrai has joined #openstack-ironic | 12:21 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/metalsmith master: Drop python 2.7 support and testing https://review.opendev.org/695727 | 12:33 |
| openstackgerrit | raphael.glon proposed openstack/ironic master: Feature: finalize_rescue, send sha512 hashed password, not clear ones https://review.opendev.org/695649 | 12:37 |
| *** Lucas_Gray has joined #openstack-ironic | 12:44 | |
| *** rh-jelabarre has joined #openstack-ironic | 12:52 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic-specs master: Update L3 based deployment spec https://review.opendev.org/672780 | 12:53 |
| *** rachit7 has joined #openstack-ironic | 12:55 | |
| *** dsneddon has quit IRC | 12:56 | |
| *** bfournie has quit IRC | 12:59 | |
| *** dsneddon has joined #openstack-ironic | 13:04 | |
| *** goldyfruit has quit IRC | 13:11 | |
| *** dsneddon has quit IRC | 13:11 | |
| *** goldyfruit has joined #openstack-ironic | 13:14 | |
| *** goldyfruit has quit IRC | 13:19 | |
| *** dsneddon has joined #openstack-ironic | 13:22 | |
| *** dsneddon has quit IRC | 13:27 | |
| *** dsneddon has joined #openstack-ironic | 13:32 | |
| *** dsneddon has quit IRC | 13:45 | |
| TheJulia | good morning | 13:48 |
| *** bfournie has joined #openstack-ironic | 13:49 | |
| iurygregory | good morning TheJulia | 13:49 |
| *** dsneddon has joined #openstack-ironic | 13:51 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Rename job to mention python3 https://review.opendev.org/695841 | 13:54 |
| openstackgerrit | Merged openstack/ironic master: Remove unused migration tests https://review.opendev.org/695612 | 13:54 |
| *** dsneddon has quit IRC | 13:55 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [DNM] test ovmf manual upgrade https://review.opendev.org/692309 | 13:56 |
| *** dsneddon has joined #openstack-ironic | 14:01 | |
| *** dsneddon has quit IRC | 14:06 | |
| openstackgerrit | Merged openstack/tenks master: CI: Use python3 in deploy-teardown jobs https://review.opendev.org/695738 | 14:08 |
| iurygregory | mgoddard, hey o/ | 14:09 |
| mgoddard | hi iurygregory | 14:09 |
| iurygregory | locally I'm able to run tox -e py3 without problems when I removed mock from test-requirements | 14:09 |
| iurygregory | in a virtual env | 14:10 |
| iurygregory | do you mind if I update the tenks patch removing the requirement? | 14:10 |
| mgoddard | iurygregory: maybe mock is installed due to another dep/ | 14:11 |
| mgoddard | I just created a py3 venv and could not import mock | 14:12 |
| iurygregory | let me check if is installed in the venv | 14:12 |
| *** dsneddon has joined #openstack-ironic | 14:12 | |
| iurygregory | mgoddard, http://paste.openstack.org/show/786672/ I don't see mock | 14:13 |
| rpittau | in py3 is part of unittest | 14:13 |
| iurygregory | so I would say we are fine to remove | 14:14 |
| mgoddard | yeah, but in my test (py35) I could import unittest.mock but not import mock | 14:14 |
| iurygregory | tox -e py35 ? | 14:14 |
| mgoddard | just a python interpreter | 14:15 |
| iurygregory | I do think we will be supporting python 3.6+ | 14:15 |
| rpittau | aren't we removing py35 ? | 14:15 |
| mgoddard | yes, but I only have 35 on my laptop :) | 14:15 |
| iurygregory | ouch | 14:15 |
| rpittau | lol | 14:15 |
| iurygregory | "The minimum version of Python now supported | 14:16 |
| iurygregory | by tenks is Python 3.6." =) | 14:16 |
| rpittau | this is the inverse of "it works on my laptop" :D | 14:16 |
| mgoddard | or not in this case :) | 14:16 |
| iurygregory | rpittau, lol | 14:16 |
| mgoddard | oh yeah, misread | 14:16 |
| *** dsneddon has quit IRC | 14:16 | |
| iurygregory | I'm running with Python 3.7.5 | 14:17 |
| iurygregory | on Fedora31 | 14:17 |
| mgoddard | python3 -m venv test && test/bin/python -c 'import mock' | 14:17 |
| rachit7 | Hi rpittau | 14:21 |
| rpittau | so anyway, mgoddard you're right we need to import unittest.mock, but we don't import mock anywhere in tenks directly | 14:21 |
| rpittau | I mean, we do, but we import it as fallback | 14:21 |
| rpittau | mgoddard: e.g. https://opendev.org/openstack/tenks/src/branch/master/tests/test_tenks_update_state.py#L26 | 14:22 |
| *** dsneddon has joined #openstack-ironic | 14:22 | |
| rpittau | hi rachit7 | 14:22 |
| rachit7 | rpittau : Can you add this patch in your review list https://review.opendev.org/#/c/671038/ ? if possible? | 14:22 |
| patchbot | patch 671038 - ironic - DRAC: Drives conversion from raid to jbod - 21 patch sets | 14:22 |
| mgoddard | oh neat, Will made it work properly :) | 14:23 |
| rpittau | mgoddard: eheh yeah | 14:23 |
| mgoddard | iurygregory: feel free to remove then | 14:23 |
| iurygregory | mgoddard, ack | 14:23 |
| rpittau | rachit7: sure thing :) | 14:23 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/tenks master: Drop python 2.7 support and testing https://review.opendev.org/695701 | 14:24 |
| rachit7 | That will be great help rpittau ..thanks again :) | 14:24 |
| rpittau | rachit7: np | 14:24 |
| mgoddard | iurygregory: not sure if you saw, but I fixed the deploy tests to use py3 where possible: https://review.opendev.org/695738 | 14:25 |
| patchbot | patch 695738 - tenks - CI: Use python3 in deploy-teardown jobs (MERGED) - 6 patch sets | 14:25 |
| iurygregory | mgoddard, awesome! | 14:25 |
| iurygregory | I forgot to pull the last master but it's ok since zuul is smart =) | 14:26 |
| *** dsneddon has quit IRC | 14:26 | |
| *** Lucas_Gray has quit IRC | 14:27 | |
| *** kaifeng has joined #openstack-ironic | 14:29 | |
| *** dsneddon has joined #openstack-ironic | 14:32 | |
| *** dsneddon has quit IRC | 14:36 | |
| *** goldyfruit has joined #openstack-ironic | 14:38 | |
| *** tzumainn has joined #openstack-ironic | 14:38 | |
| *** dsneddon has joined #openstack-ironic | 14:39 | |
| *** baha has quit IRC | 14:41 | |
| *** root has joined #openstack-ironic | 14:43 | |
| *** root is now known as Guest18001 | 14:44 | |
| *** Guest18001 is now known as jerrywang | 14:44 | |
| TheJulia | Has CI been generally working this past week, just the typical race condition/network booting failures that we sometimes see? | 14:44 |
| iurygregory | CI seems to be in good mood for most of the time I would say | 14:47 |
| *** dsneddon has quit IRC | 14:47 | |
| *** stendulker has joined #openstack-ironic | 14:48 | |
| *** rloo has joined #openstack-ironic | 14:50 | |
| *** goldyfruit has quit IRC | 14:53 | |
| *** cdearborn has joined #openstack-ironic | 14:54 | |
| *** goldyfruit has joined #openstack-ironic | 14:55 | |
| openstackgerrit | Merged openstack/ironic master: docs: update the local development quickstart to use JSON RPC https://review.opendev.org/695887 | 14:58 |
| *** goldyfruit_ has joined #openstack-ironic | 14:59 | |
| TheJulia | okay, because I went through some reviews on friday and I just couldn't tell from the patch status if we were just having an unhappy time or a few sporatic failures had hit all at once | 14:59 |
| TheJulia | well, all at once is more like "for the things being looked at" | 14:59 |
| *** aedc_ has joined #openstack-ironic | 14:59 | |
| TheJulia | #startmeeting ironic | 15:00 |
| openstack | Meeting started Mon Nov 25 15:00:32 2019 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| TheJulia | o/ | 15:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| etingof | o/ | 15:00 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| iurygregory | o/ | 15:00 |
| mkrai | o/ | 15:00 |
| TheJulia | I guess it is meeting time! | 15:00 |
| rpioso | o/ | 15:00 |
| stendulker | o/ | 15:00 |
| rpittau | o/ | 15:00 |
| kaifeng | o/ | 15:00 |
| arne_wiebalck | o/ | 15:00 |
| cdearborn | o/ | 15:00 |
| mgoddard | \o | 15:01 |
| *** goldyfruit has quit IRC | 15:01 | |
| TheJulia | We have a couple items on our agenda today. Please quickly review the agenda, I did do some clean-up and if I accidently removed an item that was not carried over from last week, please let me know | 15:01 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:01 |
| *** baha has joined #openstack-ironic | 15:01 | |
| jerrywang | o/ | 15:01 |
| *** xXraphXx has quit IRC | 15:02 | |
| *** zaneb has joined #openstack-ironic | 15:02 | |
| TheJulia | #topic Announcements / Reminders | 15:02 |
| *** openstack changes topic to "Announcements / Reminders (Meeting topic: ironic)" | 15:02 | |
| iurygregory | didn't we discussed about the Mid Cycle and the Ops Meetup in London ? | 15:02 |
| * dtantsur is lurking | 15:02 | |
| *** aedc has quit IRC | 15:02 | |
| TheJulia | iurygregory: coming back to it, hopefully people have talked inside their orgs a little bit to get a feeling if what has been proposed works | 15:02 |
| *** xXraphXx has joined #openstack-ironic | 15:03 | |
| iurygregory | TheJulia, make sense ++ | 15:03 |
| iurygregory | =) | 15:03 |
| TheJulia | I don't think I have any announcements this week. | 15:03 |
| iurygregory | maybe say we started dropping Py2.7? | 15:03 |
| mkrai | iurygregory, ++ | 15:03 |
| TheJulia | #info Support for Python 2.7 testing started being dropped last week. If you see any patches, please review them. | 15:03 |
| TheJulia | Additionally, There are a few specs in ironic-specs that could use some eyes from the community | 15:04 |
| dtantsur | yep, and the actual compatibility will be gone as well | 15:04 |
| dtantsur | now right away, but soon probably | 15:04 |
| *** aedc_ has quit IRC | 15:04 | |
| TheJulia | dtantsur: will break regardless very quickly | 15:04 |
| *** aedc has joined #openstack-ironic | 15:05 | |
| iurygregory | dtantsur, i think after we drop on all projects | 15:05 |
| * etingof just updated his L3 spec and seeks for feedback | 15:05 | |
| TheJulia | Does anyone have anything they would like to remind us of or announcements to raise? | 15:05 |
| iurygregory | (maybe on Phase2) | 15:05 |
| TheJulia | I'd like to cut a release very soon fwiw, so it could make sense to get all of the python2.7 related stuff taken care of before then. Anyway that doesn't really matter at the moment. | 15:06 |
| *** rloo has quit IRC | 15:06 | |
| TheJulia | Okay, well I guess we're good to move along | 15:06 |
| TheJulia | Next would be action item review, I don't think we had any last week. Checking | 15:07 |
| dtantsur | iurygregory: sooner, I'm afraid | 15:07 |
| TheJulia | No action items | 15:07 |
| iurygregory | dtantsur, I'm ok i was just wondering the effect on the clients =) | 15:07 |
| TheJulia | #topic Review subteam status reports | 15:07 |
| *** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:07 | |
| TheJulia | The subteam list needs to be updated based on priorities, which needs to be merged, just giving some additional time for any last minute feedback. | 15:08 |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:08 |
| TheJulia | Line 260 | 15:08 |
| TheJulia | #link https://review.opendev.org/#/c/694704/ | 15:09 |
| patchbot | patch 694704 - ironic-specs - Ussuri project priorities - 5 patch sets | 15:09 |
| *** zaneb has quit IRC | 15:09 | |
| TheJulia | dtantsur: could you add your IPA patch to the replacing WSME list? | 15:09 |
| *** zaneb has joined #openstack-ironic | 15:10 | |
| *** aedc has quit IRC | 15:10 | |
| *** ociuhandu has quit IRC | 15:10 | |
| * TheJulia adds software raid5/6 to the raid topic | 15:10 | |
| dtantsur | done | 15:10 |
| * dtantsur is in a zombie mode, sorry | 15:11 | |
| TheJulia | zombie status is acceptable :) | 15:11 |
| iurygregory | "monday mode" | 15:11 |
| TheJulia | dtantsur: I saw someplace you mentioned somewhere that zeroconf had been updated with ipv6 support completed. Does that necessitate any ironic changes or should it just be ensure that it gets into the available window with constraints? | 15:12 |
| dtantsur | yep, there is an ironic-lib patch | 15:12 |
| TheJulia | awesome | 15:12 |
| dtantsur | it's on the list | 15:13 |
| TheJulia | awesome, thanks | 15:13 |
| TheJulia | I guess we'll need to get that merged and released very soon? | 15:13 |
| *** dsneddon has joined #openstack-ironic | 15:13 | |
| TheJulia | dtantsur: thanks for putting that on the proposed chagnes for review list | 15:13 |
| arne_wiebalck | the retirement spec has been updated with the discussion from the PTG ... if anyone has time / is interested | 15:14 |
| TheJulia | arne_wiebalck: thanks! | 15:15 |
| rpittau | dtantsur TheJulia the problem with that patch as I see it is that it breaks compatibility with python 2, should we wait for all the other py2.7 dropping patches to merge first ? at least related to whatever uses ironic-lib | 15:15 |
| TheJulia | I think we're good, shall we proceed to priorities for the coming week | 15:15 |
| TheJulia | rpittau: Likely, we're going to have to have major version revs all around I think... | 15:15 |
| rpittau | yeah | 15:16 |
| TheJulia | So we can release newest ironic-lib and I guess block it from constraints for a few days while the rest of the py2.7 stuff merges | 15:16 |
| rpittau | ack | 15:17 |
| TheJulia | #topic Deciding on priorities for the coming week | 15:17 |
| *** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:17 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:17 |
| *** zbitter has joined #openstack-ironic | 15:18 | |
| TheJulia | Line 163 | 15:18 |
| * TheJulia wonders if zane needs some IRC attachment glue :( | 15:18 | |
| *** dsneddon has quit IRC | 15:18 | |
| TheJulia | the two proposed look reasonable to me. I kind of want to propose the raid 5/6 patch I put up, but it failed standalone tests and I don't know why yet | 15:18 |
| dtantsur | TheJulia, rpittau, we can just wait with ironic-lib release until the consumers are ready | 15:19 |
| TheJulia | it feels like a chicken/egg sort of thing to me | 15:20 |
| TheJulia | but that is also true | 15:20 |
| *** zaneb has quit IRC | 15:20 | |
| iurygregory | I've added a link for the patches that are open for Drop Py27 | 15:20 |
| TheJulia | dtantsur: would it make sense to put a reno into ironic and block it on ironic-lib release for zeroconf ipv6 changes? | 15:21 |
| TheJulia | iurygregory: I saw, thanks! | 15:21 |
| iurygregory | sorry for the long link =) | 15:21 |
| TheJulia | No worries! | 15:21 |
| TheJulia | Does anyone have anything else to add? | 15:21 |
| *** dsneddon has joined #openstack-ironic | 15:22 | |
| TheJulia | I guess the list looks good for me so if we're good we can proceed | 15:22 |
| rpittau | let's | 15:23 |
| TheJulia | Good, I was starting to look at how I could get crickets delivered :) | 15:24 |
| iurygregory | ++ to move | 15:24 |
| TheJulia | Onward! | 15:24 |
| TheJulia | #topic Discussion | 15:24 |
| *** openstack changes topic to "Discussion (Meeting topic: ironic)" | 15:24 | |
| TheJulia | First item, a returning item from last week is having a mid-cycle. I believe arne_wiebalck wants to firm the dates up so he can book space | 15:24 |
| rpittau | crickets over IP ? (sorry) | 15:24 |
| TheJulia | rpittau: hmm. Cricket over IP over Avian Carrier? | 15:25 |
| arne_wiebalck | it'd be good to decide if we want to have a mid-cycle here at CERN. | 15:25 |
| rpittau | yes! | 15:25 |
| * iurygregory can't confirm I couldn't talk to my manager but the dates are good I would say =) | 15:25 | |
| iurygregory | arne_wiebalck, we want ofc XD | 15:26 |
| * etingof is in the same boat with iurygregory | 15:26 | |
| TheJulia | I think it is a good idea, I've gotten some "seems like it should be reasonable" feedback from my management, but only discussion in passing really | 15:26 |
| TheJulia | I'm sure that can firm up. | 15:26 |
| *** dsneddon has quit IRC | 15:26 | |
| arne_wiebalck | do we need a minimum attendance before we say it's on? | 15:27 |
| *** zaneb has joined #openstack-ironic | 15:27 | |
| arne_wiebalck | how about we give it another week? | 15:28 |
| TheJulia | Hmm, devopsdays Geneva is Febuary 24th-25th | 15:28 |
| arne_wiebalck | uh, didn't know ... | 15:28 |
| rpittau | oh interesting | 15:28 |
| *** zbitter has quit IRC | 15:29 | |
| *** aedc has joined #openstack-ironic | 15:29 | |
| * iurygregory would say change to 26 27 the mid-cycle | 15:29 | |
| TheJulia | arne_wiebalck: could be additional reasoning. Anyway I think if you reserve the space, and if you have a sign-up or attendance tracking thing that we could use instead of an etherpad? | 15:30 |
| rpioso | Will remote participation be possible? | 15:30 |
| arne_wiebalck | https://indico.cern.ch/event/863986/ | 15:30 |
| arne_wiebalck | rpioso: yes | 15:30 |
| TheJulia | iurygregory: that may be logical or not. I suspect we should look at hotel capacity/access | 15:30 |
| * TheJulia adds mental note to check that in the next day or so | 15:30 | |
| iurygregory | TheJulia, agree =) | 15:31 |
| TheJulia | Anyway, seems like we should proceed onward to the next discussion topic | 15:31 |
| * arne_wiebalck will check the room for 26/27 | 15:31 | |
| TheJulia | iurygregory the topic is yours | 15:31 |
| iurygregory | ok o/ | 15:31 |
| iurygregory | IPA using UEFI + Secure Boot https://storyboard.openstack.org/#!/story/2006847 | 15:32 |
| *** dsneddon has joined #openstack-ironic | 15:32 | |
| iurygregory | I've talked with many people, and we have different ideas on how we should solve this, and I was thinking that the topic also has many different use cases and it would be worth a discussion about it | 15:33 |
| iurygregory | TheJulia, if I'm correct we can't use mokutil because we run under chroot? | 15:33 |
| iurygregory | that would be the reason we can't ensure after we reboot? | 15:34 |
| TheJulia | iurygregory: no, because IPA's state may not be the state desired for the running instance | 15:34 |
| *** rachit7 has quit IRC | 15:35 | |
| TheJulia | so we may be network booting legacy bios mode or uefi mode (depending on hardware and configuration) and then trying to use that transient state to infer the desired final operating mode of the instance | 15:35 |
| iurygregory | if the operator wants to use secure boot this is something he would set on configuration that would be available for the ipa? | 15:37 |
| *** zbitter has joined #openstack-ironic | 15:37 | |
| *** dsneddon has quit IRC | 15:37 | |
| TheJulia | In theory, but it would take some work for the operator. They wouldn't really be able to use iPXE without getting signed binaries | 15:37 |
| TheJulia | which is doable, but takes some work | 15:37 |
| TheJulia | I guess there is just no guarentee between the two states so use of current state to determine post reboot state is not really an option. Which is why I've been thinking we can only look at data on disk | 15:38 |
| TheJulia | Does anyone have any thoughts on this? | 15:39 |
| iurygregory | also while talking with some people I got the feedback that call gru2-install on non-SB case it's not ok since this could prevent SB to be enable later.. | 15:39 |
| TheJulia | indeed | 15:39 |
| *** zaneb has quit IRC | 15:39 | |
| TheJulia | iurygregory: it feels like a chart with options and constraints needs to be created to visually map this out | 15:41 |
| TheJulia | Maybe something in https://ethercalc.openstack.org/ and use that to discuss options? | 15:41 |
| TheJulia | so it is slightly more visual? | 15:41 |
| iurygregory | make sense to me | 15:42 |
| TheJulia | Okay, then seems like maybe we can return to this item next week or hopefully settle it this week :) | 15:42 |
| iurygregory | yeah =) | 15:42 |
| TheJulia | Are we good to move on? | 15:42 |
| iurygregory | ++ | 15:42 |
| TheJulia | Okay! | 15:42 |
| TheJulia | #topic Baremetal SIG | 15:43 |
| *** dsneddon has joined #openstack-ironic | 15:43 | |
| *** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)" | 15:43 | |
| TheJulia | arne_wiebalck: Any updates? I've not heard anything since my last discussion with the foundation about trying to encourage use case contribution to the whitepaper | 15:43 |
| arne_wiebalck | I haven't heard from the foundation or any potential authors ... have you? | 15:43 |
| TheJulia | But I know some people were likely at kubecon last week, and we're unlikely to see any action in the states since many have two days off this week. | 15:43 |
| arne_wiebalck | Ah :) | 15:43 |
| arne_wiebalck | Too early to chase ? | 15:44 |
| TheJulia | perhaps | 15:44 |
| arne_wiebalck | ok | 15:44 |
| TheJulia | I'm happy to send a follow-up email this week though | 15:44 |
| rpioso | Any word on the possibility of remote participation? | 15:44 |
| arne_wiebalck | sounds good | 15:44 |
| TheJulia | rpioso: what do you mean? | 15:44 |
| rpioso | TheJulia: In the ops meetup. | 15:45 |
| * iurygregory I was thinking it was for the mid-cycle | 15:45 | |
| TheJulia | rpioso: I have no idea, it may be worthwhile to ask those folks | 15:45 |
| * rpioso is his manager's messenger :-) | 15:45 | |
| TheJulia | rpioso: do you have the link for their planning/discussion etherpad? | 15:45 |
| arne_wiebalck | https://etherpad.openstack.org/p/LON-2020-OPS-TOPICS | 15:45 |
| TheJulia | arne_wiebalck: thanks! | 15:45 |
| rpioso | TheJulia: I do now :-) | 15:46 |
| rpioso | arne_wiebalck: Thank you! | 15:46 |
| TheJulia | Awesome, I guess we're good to proceed then | 15:46 |
| TheJulia | #action TheJulia to follow-up with foundation folks regarding use case follow-up | 15:46 |
| TheJulia | Just so I hopefully don't forget | 15:46 |
| *** zaneb has joined #openstack-ironic | 15:46 | |
| *** Goneri has joined #openstack-ironic | 15:47 | |
| TheJulia | #topic RFE Review | 15:47 |
| *** openstack changes topic to "RFE Review (Meeting topic: ironic)" | 15:47 | |
| TheJulia | We have two proposed RFEs to review | 15:47 |
| TheJulia | The first is to support compressed images | 15:48 |
| TheJulia | #link https://storyboard.openstack.org/#!/story/2006936 | 15:48 |
| *** dsneddon has quit IRC | 15:48 | |
| *** zbitter has quit IRC | 15:48 | |
| * kaifeng thought we already have | 15:48 | |
| TheJulia | We have some if done in the qcow2 file... | 15:48 |
| kaifeng | I think the same is true for compressed | 15:48 |
| TheJulia | I'm not sure, but there is a possibility that some of it could be orphaned code or not in the path that is being used | 15:49 |
| kaifeng | there is no difference to convert a compressed/uncompressed image to raw if memory serves. | 15:49 |
| TheJulia | I think qemu-convert can read stdin... I think | 15:50 |
| etingof | why can't we keep the images as-is, but compress them just for transmission (HTTP)? | 15:50 |
| TheJulia | Well... people sometimes don't support that and the issue is it is files on a webserver that have already been compressed | 15:51 |
| rpittau | it would add the compress time to the transmission time in that case | 15:51 |
| TheJulia | That may or may not support passing the stream arguments. | 15:51 |
| kaifeng | if i understand the story correct, it means compressed user-image | 15:51 |
| TheJulia | yes | 15:51 |
| etingof | ramdisk can't be that large ;) | 15:51 |
| kaifeng | i believe we just need to turn off stream raw | 15:52 |
| TheJulia | I've heard of 2+ GB IPA ramdisks | 15:52 |
| kaifeng | and it will work | 15:52 |
| TheJulia | hmm, good point | 15:52 |
| TheJulia | We need more information | 15:52 |
| TheJulia | I've asked shardy to update the RFE with some more contextual information | 15:52 |
| *** dsneddon has joined #openstack-ironic | 15:53 | |
| TheJulia | Next RFE! | 15:54 |
| TheJulia | #link https://storyboard.openstack.org/#!/story/2006910 | 15:54 |
| TheJulia | A one-shot deployment API | 15:54 |
| TheJulia | It seems reasonable to me, and while it is not a spec and it is an IPA change, it would be a virtual endpoint, so I guess I'm good with it and seeing where it ends up | 15:56 |
| * dtantsur is back, sorry | 15:57 | |
| *** zbitter has joined #openstack-ironic | 15:57 | |
| TheJulia | dtantsur: welcome back | 15:57 |
| iurygregory | the RFE description is almost a spec =) | 15:57 |
| dtantsur | I like detailed RFEs ;) | 15:57 |
| TheJulia | Indeed :) | 15:57 |
| dtantsur | the deployment API has been discussed.. many times. this work is loosely based on sambetts' ideas. | 15:57 |
| *** ociuhandu has joined #openstack-ironic | 15:58 | |
| TheJulia | I'm good with it, any objections or support for it? | 15:58 |
| iurygregory | +1 | 15:59 |
| * etingof is confused by "These resources will be purely virtual: they won't be backed by database objects." followed by "Deployment objects" description | 15:59 | |
| mgoddard | will we switch the nova virt driver to use it? | 15:59 |
| dtantsur | etingof: these are objects in the API, but there is no database objects behind them | 15:59 |
| * kaifeng needs more reading, but definitely no objection for new ideas | 15:59 | |
| dtantsur | mgoddard: there are a few words about it there :) | 15:59 |
| *** zaneb has quit IRC | 16:00 | |
| dtantsur | essentially, nova may benefit from a multi-step approach | 16:00 |
| *** zbitter is now known as zaneb | 16:00 | |
| mgoddard | dtantsur: skimmed for them :) | 16:00 |
| TheJulia | dtantsur: single step you mean? | 16:00 |
| dtantsur | TheJulia: well, I think nova is a special consumers who may actually use a multi-step approach | 16:00 |
| dtantsur | e.g. I think VIF attachment is separate | 16:00 |
| etingof | will it play well with HA? what happens if conductor dies along the way? | 16:01 |
| TheJulia | dtantsur: it is... I guess yeah it may be best to stay multi all along | 16:01 |
| TheJulia | etingof: deployment dies | 16:01 |
| dtantsur | etingof: there is no state in the conductor, it's all taken from the nodes | 16:01 |
| dtantsur | it's essentially what metalsmith implements on the client side, but moved to ironic-api | 16:01 |
| TheJulia | etingof: or to be more precise, the deployment is moved to deploy failed | 16:01 |
| TheJulia | if it is not completed. | 16:01 |
| dtantsur | right, the same way it's done now | 16:02 |
| TheJulia | yup | 16:02 |
| * dtantsur grabs whiskey while waiting for objections | 16:03 | |
| etingof | could one deploy twice? | 16:03 |
| *** ociuhandu has quit IRC | 16:03 | |
| TheJulia | Anyway, we're past our ending time. If there are no objections, I think that is the end of our meeting | 16:03 |
| etingof | if it's all virtual, I suspect no state | 16:03 |
| mgoddard | dtantsur: objection! pass the whiskey along | 16:03 |
| dtantsur | :D | 16:03 |
| dtantsur | etingof: how do you imagine it? | 16:03 |
| TheJulia | mgoddard: Very wise :) | 16:03 |
| TheJulia | I guess you guys can discuss | 16:03 |
| etingof | double curl | 16:03 |
| TheJulia | Well if there is nothing else, thanks everyone! | 16:04 |
| dtantsur | etingof: same is right now: the 2nd request gets CONFLICT | 16:04 |
| dtantsur | thanks TheJulia | 16:04 |
| etingof | no racing? | 16:04 |
| TheJulia | task.node.save() occurs when the lock is raised | 16:04 |
| TheJulia | the second connection would read the db and see the lock | 16:04 |
| dtantsur | etingof: it's build around nodes internally | 16:04 |
| dtantsur | so usual node locks (and our beloved Node locked error) apply | 16:04 |
| TheJulia | #endmeeting | 16:04 |
| *** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list." | 16:04 | |
| openstack | Meeting ended Mon Nov 25 16:04:56 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:04 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-11-25-15.00.html | 16:05 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-11-25-15.00.txt | 16:05 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-11-25-15.00.log.html | 16:05 |
| dtantsur | (allocations were funnier, they're not attached to a node initially) | 16:05 |
| kaifeng | yeah, i am feeling something like allocation | 16:06 |
| dtantsur | but a deployment is always attached to a node | 16:06 |
| *** zbitter has joined #openstack-ironic | 16:06 | |
| dtantsur | so we'll use node locks | 16:06 |
| *** stendulker has quit IRC | 16:07 | |
| kaifeng | POST /v1/deployments | 16:07 |
| kaifeng | how do we choose the target node here? | 16:07 |
| dtantsur | it's inside body as "node" | 16:07 |
| dtantsur | (same as for ports, port groups, etc) | 16:08 |
| *** zaneb has quit IRC | 16:09 | |
| *** jerrywang has quit IRC | 16:09 | |
| etingof | how do I get automatically created UUID of the node? | 16:09 |
| *** mkrai has quit IRC | 16:09 | |
| kaifeng | hmm, got it | 16:10 |
| kaifeng | do we have a start state for the deployment? | 16:10 |
| dtantsur | kaifeng: "deploying", this is where node start their jorney | 16:10 |
| dtantsur | etingof: sorry, I don't quite get what you're asking. the user story is "As a user of ironic I want to provision a node with a given image" | 16:11 |
| dtantsur | "a node" is known to the user and is passed via API | 16:11 |
| dtantsur | am I missing something? | 16:11 |
| kaifeng | oh, i mean start with "available" right? | 16:11 |
| etingof | I am not sure I understand this "f uuid is not provided, it's auto-generated." | 16:12 |
| kaifeng | i think it represents instance uuid | 16:12 |
| dtantsur | yep, kaifeng is right | 16:13 |
| kaifeng | for two use cases: uuid/no-uuid -> nova/without nova | 16:13 |
| dtantsur | yep | 16:13 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [DNM] test ovmf manual upgrade https://review.opendev.org/692309 | 16:14 |
| dtantsur | etingof: so, deployment.uuid == node.instance_uuid != node.uuid | 16:14 |
| dtantsur | is this what confuses you? | 16:14 |
| etingof | yeah | 16:14 |
| *** openstackstatus has joined #openstack-ironic | 16:14 | |
| *** ChanServ sets mode: +v openstackstatus | 16:14 | |
| rpittau | well to be honest it might be confusing in general | 16:15 |
| dtantsur | I can imagine :) | 16:15 |
| * etingof thinks that openstack can be confusing at times | 16:15 | |
| dtantsur | the instance_uuid thing comes from nova essentially. but we have to account for it in all new work. | 16:15 |
| dtantsur | this is why we have allocation_uuid that matches instance_uuid, etc | 16:15 |
| rpioso | TheJulia: Have you had any further thoughts re: soft power off failure we discussed on Friday? | 16:15 |
| dtantsur | and now here | 16:15 |
| etingof | do I always absolutely have to have instance_uuid associated with the node? | 16:16 |
| etingof | can I just openstack baremetal node provide ? | 16:16 |
| rpittau | I guess during and after deployment, yes | 16:16 |
| *** zaneb has joined #openstack-ironic | 16:16 | |
| *** zaneb has quit IRC | 16:16 | |
| etingof | right, I see | 16:16 |
| dtantsur | etingof: no, you do not | 16:17 |
| *** zbitter has quit IRC | 16:17 | |
| dtantsur | but this is our way of locking a node | 16:17 |
| dtantsur | if you don't do it, there may be races indeed | 16:18 |
| dtantsur | (yes, it's not the best way to implement locks. but see above re nova) | 16:18 |
| dtantsur | so you don't have to, but you should | 16:18 |
| etingof | makes sense | 16:18 |
| etingof | may be worth noting this tribal wisdom somewhere | 16:18 |
| dtantsur | good call. some developer docs? | 16:19 |
| dtantsur | "why ironic API is so weird" | 16:19 |
| dtantsur | "because NOVA!" | 16:19 |
| rpittau | FAQ section ? | 16:19 |
| etingof | may be near instance_uuid description? | 16:19 |
| kaifeng | i remember there was a patch in nova side that update instance_uuid before deployment starts | 16:20 |
| kaifeng | which is dedicated for this purpose | 16:20 |
| kaifeng | i mean, lock | 16:20 |
| dtantsur | correct | 16:22 |
| etingof | pardon my racing paranoia, but is it so that once POST /v1/deployments returns, the node, its ports and everything related is firmly locked? | 16:25 |
| etingof | no concurrent messing would pass unnoticed | 16:25 |
| dtantsur | correct | 16:26 |
| etingof | does this hold "failure to validate the image won't leave VIFs on the node" if ironic dies right upon validation failure? | 16:31 |
| dtantsur | etingof: we don't have THIS level of locking right now anywhere in ironic, but it paves the way to implementing it. | 16:32 |
| dtantsur | we use locking, but not transactions - to be clear | 16:33 |
| dtantsur | the deployment API will try its best to clean up on failures | 16:33 |
| dtantsur | for your case, clean up happens on conductor restart | 16:33 |
| dtantsur | (it's already the case) | 16:33 |
| *** goldyfruit___ has joined #openstack-ironic | 16:33 | |
| etingof | awesome | 16:34 |
| *** goldyfruit_ has quit IRC | 16:36 | |
| *** akotobi has joined #openstack-ironic | 16:38 | |
| *** akotobi has left #openstack-ironic | 16:39 | |
| rpittau | good night! o/ | 16:40 |
| *** rpittau is now known as rpittau|afk | 16:40 | |
| *** dtantsur is now known as dtantsur|afk | 16:44 | |
| dtantsur|afk | o/ | 16:44 |
| TheJulia | guys, I'm going to go lay down and hope my back chills out and stops hurting so I can sit in a chair | 16:45 |
| dtantsur|afk | TheJulia: I hope you get better soon | 16:46 |
| rpioso | TheJulia: Hope you soon feel better. | 16:46 |
| TheJulia | thanks | 16:46 |
| * TheJulia goes and tries to do yoga first | 16:46 | |
| TheJulia | tries | 16:46 |
| sshnaidm | TheJulia, dtantsur|afk hi, I have a general question, is there any interest to have Ironic ansible modules? Like I saw it now in ansible codebase, existing ones don't even cover 10% of Ironic functionality and look kinda abandoned. | 16:51 |
| etingof | take some rest and get well, TheJulia! | 16:51 |
| sshnaidm | maybe just there are no use cases for them and it's not really needed..? | 16:52 |
| *** kaifeng has quit IRC | 16:52 | |
| *** goldyfruit_ has joined #openstack-ironic | 16:55 | |
| *** iurygregory has quit IRC | 16:56 | |
| *** goldyfruit___ has quit IRC | 16:57 | |
| arne_wiebalck | xXraphXx: if you have a moment, I'd appreciate your input on https://review.opendev.org/#/c/686580/ and https://review.opendev.org/#/c/686585/ ... these encapsulate the "identify the rootfs" functionality from your patch https://review.opendev.org/#/c/674879 | 16:59 |
| patchbot | patch 686580 - ironic - Software RAID: Identify the root fs via its UUID f... - 4 patch sets | 16:59 |
| patchbot | patch 686585 - ironic-python-agent - Software RAID: Use UUID to find root fs - 5 patch sets | 16:59 |
| patchbot | patch 674879 - ironic-python-agent - image extension, install_bootloader improvements - 10 patch sets | 16:59 |
| *** tssurya has quit IRC | 17:00 | |
| arne_wiebalck | It's basically an implementation of my last comment on your patch. | 17:00 |
| *** dsneddon has quit IRC | 17:03 | |
| *** lucasagomes has quit IRC | 17:08 | |
| *** dsneddon has joined #openstack-ironic | 17:09 | |
| xXraphXx | arne_wiebalck, hi -> https://review.opendev.org/#/c/686580/ looks good to me, for the second one question: | 17:11 |
| patchbot | patch 686580 - ironic - Software RAID: Identify the root fs via its UUID f... - 4 patch sets | 17:11 |
| xXraphXx | the efi partition/ppc part is still assumed to be on the same device as the roofs. Is it wanted ? Or what you meant is that once your patches are merged we rebase https://review.opendev.org/#/c/674879 on top of them ? | 17:14 |
| patchbot | patch 674879 - ironic-python-agent - image extension, install_bootloader improvements - 10 patch sets | 17:14 |
| *** dsneddon has quit IRC | 17:14 | |
| arne_wiebalck | xXraphXx: thanks for checking! | 17:15 |
| arne_wiebalck | xXraphXx: I haven't looked into the efi partition yet. | 17:16 |
| arne_wiebalck | xXraphXx: This was mostly to split off the hardcoded partition issue into a separate patch. | 17:16 |
| arne_wiebalck | xXraphXx: So, yes, 674879 could now be rebased on top. | 17:17 |
| *** igordc has joined #openstack-ironic | 17:18 | |
| arne_wiebalck | xXraphXx: I have a UEFI node for testing atm, so I could give UEFI w/ s/w RAID a try ... if you have some time to rebase? | 17:18 |
| *** dsneddon has joined #openstack-ironic | 17:19 | |
| * arne_wiebalck has to go ... | 17:20 | |
| xXraphXx | ok so +1 for both your pull requests :) | 17:20 |
| xXraphXx | I'll try to rebase mine on top | 17:20 |
| arne_wiebalck | he he, thanks | 17:20 |
| arne_wiebalck | xXraphXx: cool, once that is done, I see if I can test ... is there anything in addition needed, or should this just work? | 17:21 |
| xXraphXx | no there is still something that will miss :( : | 17:22 |
| xXraphXx | the efi part needs to be built outside the softraid, unless I'm wrong nothing currently builds it, that was actually the point of this pr at the time: | 17:23 |
| xXraphXx | https://review.opendev.org/675672 | 17:23 |
| patchbot | patch 675672 - ironic-python-agent - Additional command to build efi partitions - 2 patch sets | 17:23 |
| xXraphXx | * built or relocated (if for example the efi part was already in the whole_disk_image) | 17:25 |
| arne_wiebalck | hmm, I see: built for partition image, relocate for whole disk image? | 17:27 |
| xXraphXx | yep | 17:27 |
| arne_wiebalck | ok, so these two patches are needed | 17:27 |
| arne_wiebalck | I figure the 2nd patch needs some more work? | 17:29 |
| *** FlorianFa has quit IRC | 17:29 | |
| *** dsneddon has quit IRC | 17:30 | |
| openstackgerrit | Merged openstack/tenks master: Drop python 2.7 support and testing https://review.opendev.org/695701 | 17:32 |
| xXraphXx | yep -> for sure there currently misses one dep in the ipa builder + the extension might not just be fit to the agent_base_vendor, so probably not the way to go. | 17:32 |
| xXraphXx | oh and forgot the gpt for softraid is still not merged -> needed too: sth like https://review.opendev.org/#/c/674803/ | 17:35 |
| patchbot | patch 674803 - ironic-python-agent - Software raid: default partition table type change - 14 patch sets | 17:35 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Fix reading/setting libvirt domain boot device https://review.opendev.org/686183 | 17:42 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Fix incorrect parameter description in docstrings https://review.opendev.org/686567 | 17:43 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Fix defaults to storage and drive resources https://review.opendev.org/687539 | 17:43 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Fix emulator socket binding options https://review.opendev.org/684685 | 17:43 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Fix `BootSourceOverrideMode` enumeration https://review.opendev.org/688458 | 17:44 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Always log via Flask application https://review.opendev.org/693639 | 17:44 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Fix libvirt CD image insertion https://review.opendev.org/693561 | 17:44 |
| *** aedc_ has joined #openstack-ironic | 17:55 | |
| *** tesseract has quit IRC | 17:57 | |
| *** aedc has quit IRC | 17:58 | |
| *** aedc_ has quit IRC | 18:00 | |
| openstackgerrit | Merged openstack/ironic-python-agent stable/train: Bump up ipa-ip-lookup-attempts to 6 https://review.opendev.org/695546 | 18:03 |
| *** derekh has quit IRC | 18:05 | |
| *** akotobi has joined #openstack-ironic | 18:10 | |
| *** akotobi has left #openstack-ironic | 18:10 | |
| openstackgerrit | Merged openstack/metalsmith master: Drop python 2.7 support and testing https://review.opendev.org/695727 | 18:14 |
| *** jtomasek has quit IRC | 18:19 | |
| *** rloo has joined #openstack-ironic | 18:27 | |
| *** dsneddon has joined #openstack-ironic | 18:29 | |
| *** rloo has quit IRC | 18:31 | |
| *** rloo has joined #openstack-ironic | 18:31 | |
| *** brandor5 has joined #openstack-ironic | 18:35 | |
| *** dougsz has quit IRC | 18:36 | |
| openstackgerrit | Merged openstack/ironic master: Mock out the correct greenthread sleep method https://review.opendev.org/695208 | 18:37 |
| brandor5 | hello everyone: I'm looking for a way to build baremetal images that wouldn't be hard to incorporate into a CI/CD pipeline that's not diskimage-builder, anyone? | 18:37 |
| *** igordc has quit IRC | 18:39 | |
| *** dsneddon has quit IRC | 18:44 | |
| *** ociuhandu has joined #openstack-ironic | 18:45 | |
| goldyfruit_ | Guys, do you know if Redfish driver is fully working with iDrac 7? | 18:47 |
| goldyfruit_ | During the cleaning process I got an error from Refish: ironic.common.exception.RedfishError: Redfish exception occurred. Error: Redfish reboot failed for node 0937ddd3-7a62-45da-8224-2115b1c26173. Error: The parameter "value" value "force restart" is invalid. Valid values are: {'graceful restart', 'nmi', 'on', 'push power button', 'force off', 'graceful shutdown'} | 18:48 |
| goldyfruit_ | I have the latest firmware available for my iDrac | 18:48 |
| *** dsneddon has joined #openstack-ironic | 18:49 | |
| goldyfruit_ | Maybe Redfish should have a fallback to "graceful restart" for example if "force restart" is not available | 18:50 |
| *** ociuhandu has quit IRC | 18:51 | |
| goldyfruit_ | I mean Redfish integration into Ironic | 18:52 |
| *** ociuhandu has joined #openstack-ironic | 18:52 | |
| rpioso | goldyfruit_: It works well with iDRAC 9 (14G). The idrac HW type supports that. | 18:54 |
| rpioso | Presently, it supports only that generation. | 18:55 |
| goldyfruit_ | rpioso, so we should not use Redfish with iDRAC <9 ? | 18:56 |
| *** hwoarang_ has quit IRC | 18:57 | |
| *** brandor5 has left #openstack-ironic | 18:58 | |
| *** hwoarang has joined #openstack-ironic | 18:59 | |
| goldyfruit_ | By changing the code to GracefulRestart I'm able to continue | 19:02 |
| goldyfruit_ | 2019-11-25 19:01:09.829 6 DEBUG sushy.connector [req-77176711-72b4-4a91-bae3-fb543c08256f - - - - -] HTTP request: POST https://192.168.1.111/redfish/v1/Systems/System.Embedded.1/Actions/ComputerSystem.Reset; headers: {'OData-Version': '4.0'}; body: {'ResetType': 'GracefulRestart'}; session arguments: {}; _op /usr/local/lib/python3.7/dist-packages/sushy/connector.py:89 | 19:02 |
| *** alexmcleod has quit IRC | 19:08 | |
| rpioso | goldyfruit_: \o/ | 19:18 |
| rpioso | Contributions which expand support are welcome :-) | 19:18 |
| goldyfruit_ | This hack for sure will never pass | 19:19 |
| goldyfruit_ | Now i'm facing this an issue with this change: https://github.com/openstack/ironic/commit/931c12598296dad676b2d64b66fef6e95ad3939b | 19:19 |
| rpioso | goldyfruit_: I'm not familiar with that change. | 19:21 |
| goldyfruit_ | Pretty new | 19:21 |
| goldyfruit_ | This error from API log: https://paste.api-zulu.com/ejuzefoqib.go | 19:23 |
| *** cdearborn has quit IRC | 19:23 | |
| goldyfruit_ | My agent_url on the node is "agent_url": "http://10.0.0.201:9999", | 19:24 |
| goldyfruit_ | This is not the same IP :/ | 19:24 |
| *** ociuhandu has quit IRC | 19:32 | |
| *** ociuhandu has joined #openstack-ironic | 19:33 | |
| *** ociuhandu has quit IRC | 19:38 | |
| *** Goneri has quit IRC | 19:39 | |
| *** igordc has joined #openstack-ironic | 19:55 | |
| *** Goneri has joined #openstack-ironic | 20:10 | |
| *** mrda has quit IRC | 20:38 | |
| *** mrda has joined #openstack-ironic | 20:40 | |
| *** igordc has quit IRC | 20:47 | |
| *** ociuhandu has joined #openstack-ironic | 20:50 | |
| *** igordc has joined #openstack-ironic | 20:53 | |
| *** ociuhandu has quit IRC | 21:11 | |
| *** ociuhandu has joined #openstack-ironic | 21:12 | |
| goldyfruit_ | Yeah really seems that Redfish and iDRAC 7 doesn't work really well with Sushy | 21:16 |
| *** ociuhandu has quit IRC | 21:17 | |
| goldyfruit_ | I can see the PATCH successfully applied to the BIOS configuration but nothing changed into the BIOS | 21:17 |
| goldyfruit_ | But using this script from Dell https://github.com/dell/iDRAC-Redfish-Scripting/blob/master/Redfish%20Python/BiosSetAttributeREDFISH.py works for me | 21:18 |
| goldyfruit_ | The main difference is the Jobs part | 21:18 |
| rpioso | goldyfruit_: To apply BIOS settings, either one of two things must be done -- a) Create an OEM configuration job or b) include a @Redfish.SettingsApplyTime annotation in the PATCH. iDRAC 7 supports the former, but doesn't presently support the latter. | 21:19 |
| rpioso | goldyfruit_: iDRAC 9 supports both. | 21:21 |
| goldyfruit_ | rpioso, oh no!!! | 21:21 |
| rpioso | goldyfruit_: A Redfish BIOS interface implementation is not presently supported for any iDRAC, because there's no code which does one of those two things. | 21:21 |
| goldyfruit_ | We mostly have iDRAC 7 servers :/ | 21:22 |
| rpioso | goldyfruit_: It's an opportunity, if you're interested. | 21:23 |
| goldyfruit_ | It's to complex for my little dev skills | 21:23 |
| rpioso | goldyfruit_: The generic redfish BIOS interface implementation was designed to be adapted to vendor implementation differences and there is a nascent sushy extension for iDRAC. | 21:24 |
| rpioso | goldyfruit_: If you're curious, see https://opendev.org/openstack/ironic/src/commit/b7dc8091980b4456f33a9cb8678e21a167099c29/ironic/drivers/modules/redfish/bios.py#L214 and https://opendev.org/openstack/ironic/src/commit/b7dc8091980b4456f33a9cb8678e21a167099c29/ironic/drivers/modules/redfish/bios.py#L200 in the generic redfish BIOS interface impl and https://github.com/etingof/sushy-oem-idrac | 21:27 |
| goldyfruit_ | rpioso, thanks for the info, I'll have a look into Sushy to see how it works | 21:27 |
| rpioso | goldyfruit_: That extension is available on PyPi -- https://pypi.org/project/sushy-oem-idrac/ | 21:29 |
| rpioso | goldyfruit_: yw | 21:29 |
| *** rcernin has joined #openstack-ironic | 21:31 | |
| goldyfruit_ | rpioso, so if I understand, we need to add (for iDRAC 7) the job creation because the SettingsApplyTime doesn't exist which is the one that handles the job creation (for iDRAC > 7) ? | 21:33 |
| rpioso | goldfruit_: You are absolutely correct :-) | 21:34 |
| goldyfruit_ | rpioso, cool | 21:34 |
| rpioso | goldyfruit_: ^^^ Ooops. And presently, it's for iDRAC == 9 | 21:34 |
| goldyfruit_ | I'm curious how complex it is for Dell to had this missing endpoints/parameters | 21:35 |
| goldyfruit_ | Via a firmware update | 21:35 |
| rpioso | goldyfruit_: Behind the scenes, the iDRAC creates a job when that annotation is provided. | 21:35 |
| * rpioso is also curious about many things :-) | 21:36 | |
| goldyfruit_ | ^^ | 21:36 |
| rpioso | goldyfruit_: idrac HW type support of Redfish was recently added during Train, so ... | 21:37 |
| *** pcaruana has quit IRC | 21:38 | |
| rpioso | goldyfruit_: Again, contributions are most welcome. | 21:38 |
| goldyfruit_ | rpioso, I understand not worries :) | 21:38 |
| goldyfruit_ | I'm trying to contribute at my level when I can | 21:38 |
| rpioso | goldyfruit_: That's especially true of earlier generations. | 21:39 |
| rpioso | goldyfruit_: Thank you! | 21:39 |
| *** tzumainn has quit IRC | 21:45 | |
| *** bfournie has quit IRC | 22:04 | |
| *** ociuhandu has joined #openstack-ironic | 23:30 | |
| *** ociuhandu has quit IRC | 23:35 | |
| *** bfournie has joined #openstack-ironic | 23:35 | |
| openstackgerrit | Merged openstack/ironic master: Add `instance_info/kernel_append_params` to `redfish` https://review.opendev.org/687092 | 23:39 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!