| janders | excellent | 00:06 |
|---|---|---|
| janders | what instrument do you play? | 00:06 |
| janders | wineries -> Martinborough? | 00:06 |
| *** diurnalist has joined #openstack-ironic | 00:51 | |
| *** diurnalist has quit IRC | 00:59 | |
| stevebaker | janders: electric guitar mainly. We spent a day in Martinborough with friends tooling around on bicycles | 01:04 |
| janders | awesome! :) | 01:05 |
| janders | (both the guitar part and the Martinborough part - love their Pinot and the scenic drive there) | 01:05 |
| janders | I need to chase up some chores, will be off most of the afternoon and back in the evening | 01:06 |
| *** janders has quit IRC | 01:06 | |
| *** janders has joined #openstack-ironic | 01:11 | |
| stevebaker | janders: all good, thanks | 01:14 |
| *** rh-jelabarre has quit IRC | 01:21 | |
| *** yaawang has quit IRC | 02:05 | |
| *** yaawang has joined #openstack-ironic | 02:06 | |
| *** yaawang has quit IRC | 02:10 | |
| *** yaawang has joined #openstack-ironic | 02:11 | |
| *** yaawang has quit IRC | 02:15 | |
| *** yaawang has joined #openstack-ironic | 02:16 | |
| *** tzumainn has quit IRC | 02:29 | |
| *** Yumeng has joined #openstack-ironic | 02:32 | |
| *** markmcclain has quit IRC | 03:31 | |
| *** mkrai has joined #openstack-ironic | 03:31 | |
| *** markmcclain has joined #openstack-ironic | 03:35 | |
| *** gyee has quit IRC | 03:50 | |
| *** rcernin has quit IRC | 04:03 | |
| *** mkrai has quit IRC | 04:23 | |
| *** rcernin has joined #openstack-ironic | 04:24 | |
| *** mkrai has joined #openstack-ironic | 04:26 | |
| openstackgerrit | paresh sao proposed openstack/ironic master: [WIP] OOB one button secure erase for iLO5 based HPE Proliant servers. https://review.opendev.org/743868 | 05:08 |
| *** ociuhandu has joined #openstack-ironic | 05:15 | |
| *** ociuhandu has quit IRC | 05:20 | |
| *** alexmcleod has quit IRC | 05:46 | |
| *** alexmcleod has joined #openstack-ironic | 05:46 | |
| *** mkrai has quit IRC | 05:59 | |
| *** mkrai_ has joined #openstack-ironic | 05:59 | |
| *** Qianbiao has joined #openstack-ironic | 06:40 | |
| *** ociuhandu has joined #openstack-ironic | 06:41 | |
| *** yaawang has quit IRC | 06:44 | |
| *** yaawang has joined #openstack-ironic | 06:44 | |
| arne_wiebalck | Good morning, ironic! | 06:51 |
| *** rcernin has quit IRC | 06:54 | |
| *** diurnalist has joined #openstack-ironic | 06:56 | |
| *** ntt__ has joined #openstack-ironic | 06:58 | |
| *** diurnalist has quit IRC | 07:00 | |
| *** rcernin has joined #openstack-ironic | 07:20 | |
| *** rcernin has quit IRC | 07:25 | |
| iurygregory | good morning arne_wiebalck and Ironic! | 07:27 |
| *** Zempashi has joined #openstack-ironic | 07:27 | |
| *** dtantsur|afk is now known as dtantsur | 07:30 | |
| dtantsur | morning ironic | 07:30 |
| *** ociuhandu has quit IRC | 07:33 | |
| dtantsur | a very easy patch to review: https://review.opendev.org/#/c/744887/ | 07:35 |
| dtantsur | saved me some grey hair when debugging sushy-tools | 07:35 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Replace the static Manager driver with a new fake one https://review.opendev.org/744862 | 07:43 |
| *** dougsz has joined #openstack-ironic | 07:44 | |
| arne_wiebalck | hey iurygregory and dtantsur o/ | 07:44 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/ussuri: Wipe agent token and URL on rescue and unrescue https://review.opendev.org/745083 | 07:45 |
| rpittau | good morning ironic! o/ | 07:45 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic stable/ussuri: Change UEFI PXE job to use tinyipa https://review.opendev.org/744979 | 07:49 |
| rpittau | whoops didnt see there was a backport already :/ | 07:49 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Clean up bindep.txt https://review.opendev.org/744930 | 07:50 |
| *** Lucas_Gray has joined #openstack-ironic | 07:53 | |
| *** ociuhandu has joined #openstack-ironic | 07:54 | |
| *** Yumeng has quit IRC | 07:57 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Get rid of Resources cache, store resources on the Application https://review.opendev.org/744911 | 07:58 |
| *** lucasagomes has joined #openstack-ironic | 08:16 | |
| *** belmoreira has joined #openstack-ironic | 08:25 | |
| *** priteau has joined #openstack-ironic | 08:29 | |
| *** derekh has joined #openstack-ironic | 08:30 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Replace the static Manager driver with a new fake one https://review.opendev.org/744862 | 08:42 |
| openstackgerrit | Merged openstack/ironic-lib master: Add logging to the HTTP basic auth code https://review.opendev.org/744899 | 08:46 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Get rid of Resources cache, store resources on the Application https://review.opendev.org/744911 | 08:47 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Flatten the emulator resources https://review.opendev.org/745092 | 08:48 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Clean up bindep.txt https://review.opendev.org/744930 | 08:51 |
| *** iurygregory has quit IRC | 08:51 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Dynamic roles: consolidate auth parameters in one place https://review.opendev.org/743169 | 08:54 |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Support HTTP basic auth and switch bifrost-cli to it https://review.opendev.org/741964 | 08:55 |
| *** iurygregory has joined #openstack-ironic | 09:02 | |
| *** k_mouza has joined #openstack-ironic | 09:10 | |
| openstackgerrit | Merged openstack/sushy-tools master: Log API errors for easier debugging https://review.opendev.org/744887 | 09:18 |
| *** Qianbiao has quit IRC | 09:23 | |
| *** Qianbiao has joined #openstack-ironic | 09:23 | |
| *** bdodd has quit IRC | 09:26 | |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: Change to "collections" where needed https://review.opendev.org/743186 | 09:29 |
| *** bdodd has joined #openstack-ironic | 09:30 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Use property plus abstractmethod for abstractproperty https://review.opendev.org/744945 | 09:33 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Use property plus abstractmethod for abstractproperty https://review.opendev.org/744945 | 09:34 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Replace the static Manager driver with a new fake one https://review.opendev.org/744862 | 09:42 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Get rid of Resources cache, store resources on the Application https://review.opendev.org/744911 | 09:45 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Flatten the emulator resources https://review.opendev.org/745092 | 09:46 |
| ajya | FYI, I submitted the deploy step issue to write down current findings - https://storyboard.openstack.org/#!/story/2007987 | 09:53 |
| *** rcernin has joined #openstack-ironic | 09:55 | |
| *** rcernin has quit IRC | 09:59 | |
| *** k_mouza has quit IRC | 10:06 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Clean up bindep.txt https://review.opendev.org/744930 | 10:09 |
| *** alexmcleod has quit IRC | 10:09 | |
| *** alexmcleod has joined #openstack-ironic | 10:10 | |
| openstackgerrit | Riccardo Pittau proposed openstack/bifrost master: Change to "collections" where needed https://review.opendev.org/743186 | 10:16 |
| *** k_mouza has joined #openstack-ironic | 10:18 | |
| *** janders has quit IRC | 10:30 | |
| *** tkajinam has quit IRC | 10:33 | |
| *** janders has joined #openstack-ironic | 10:35 | |
| openstackgerrit | Merged openstack/sushy-tools master: Use property plus abstractmethod for abstractproperty https://review.opendev.org/744943 | 10:46 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Replace the static Manager driver with a new fake one https://review.opendev.org/744862 | 10:50 |
| dtantsur | folks, please review ^^^ it's a bit long, but fixes an important issue when used with >1 nodes | 10:50 |
| dtantsur | rpittau, iurygregory ^^^ | 10:50 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Get rid of Resources cache, store resources on the Application https://review.opendev.org/744911 | 10:51 |
| openstackgerrit | Dmitry Tantsur proposed openstack/sushy-tools master: Flatten the emulator resources https://review.opendev.org/745092 | 10:52 |
| iurygregory | dtantsur, ack | 10:57 |
| *** ociuhandu has quit IRC | 11:00 | |
| *** ociuhandu has joined #openstack-ironic | 11:01 | |
| *** ociuhandu has quit IRC | 11:06 | |
| *** janders has quit IRC | 11:20 | |
| *** Lucas_Gray has quit IRC | 11:23 | |
| openstackgerrit | Merged openstack/ironic master: Fix iscsi deploy steps priorities https://review.opendev.org/744866 | 11:24 |
| *** mkrai_ has quit IRC | 11:25 | |
| *** mkrai_ has joined #openstack-ironic | 11:25 | |
| *** sshnaidm is now known as sshnaidm|afk | 11:33 | |
| *** ociuhandu has joined #openstack-ironic | 11:38 | |
| *** Lucas_Gray has joined #openstack-ironic | 11:39 | |
| *** mkrai_ has quit IRC | 11:44 | |
| *** ociuhandu has quit IRC | 11:44 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Auto extend the timeout for RAX hosts https://review.opendev.org/741853 | 11:46 |
| *** JamesBenson has quit IRC | 11:48 | |
| *** Qianbiao has quit IRC | 11:49 | |
| *** sshnaidm|afk is now known as sshnaidm | 11:49 | |
| *** k_mouza has quit IRC | 11:50 | |
| dtantsur | rpittau: no use rechecking the suse job, it's hard broken | 11:59 |
| *** tzumainn has joined #openstack-ironic | 12:00 | |
| rpittau | dtantsur: oh ok, didn't know, thought it was a transient or cache error as the package is in the repo | 12:00 |
| dtantsur | it's been broken for days, I've sent a request for help on the ML | 12:01 |
| rpittau | ah saw the mail now | 12:01 |
| *** janders has joined #openstack-ironic | 12:01 | |
| *** k_mouza has joined #openstack-ironic | 12:03 | |
| rpittau | interesting though, trying in an isolated environment with opensuse 15.1 and the pkg is correctly installed | 12:03 |
| rpittau | oh well | 12:03 |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: [DNM] test ci https://review.opendev.org/744888 | 12:04 |
| *** rcernin has joined #openstack-ironic | 12:08 | |
| *** rh-jelabarre has joined #openstack-ironic | 12:08 | |
| *** sdanni has joined #openstack-ironic | 12:17 | |
| *** ociuhandu has joined #openstack-ironic | 12:28 | |
| *** bdodd has quit IRC | 12:44 | |
| *** mkrai has joined #openstack-ironic | 12:46 | |
| *** bdodd has joined #openstack-ironic | 12:48 | |
| *** diurnalist has joined #openstack-ironic | 12:56 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: agent_client: support custom TLS certificates https://review.opendev.org/744731 | 13:00 |
| *** diurnalist has quit IRC | 13:01 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Support changing the protocol part of callback_url to https https://review.opendev.org/745142 | 13:15 |
| iurygregory | in 45min we will have the ironic grenade multinode meeting =) | 13:16 |
| *** mkrai has quit IRC | 13:21 | |
| TheJulia | good morning | 13:24 |
| dtantsur | morning TheJulia | 13:27 |
| dtantsur | on, ironic in the Red Hat Research paper: https://research.redhat.com/wp-content/uploads/2020/02/RHRQ-v2.2-digital.pdf (page 16) | 13:27 |
| *** Qianbiao has joined #openstack-ironic | 13:29 | |
| TheJulia | mentioned in two articles nice | 13:30 |
| TheJulia | So shiny! | 13:30 |
| dtantsur | tzumainn++ | 13:32 |
| iurygregory | good morning TheJulia =) | 13:32 |
| tzumainn | ... I kinda forgot I wrote it | 13:33 |
| iurygregory | wow nice dtantsur ! | 13:33 |
| dtantsur | tzumainn: lol | 13:33 |
| iurygregory | thanks for sharing | 13:33 |
| TheJulia | tzumainn: thanks! | 13:33 |
| * TheJulia caffinates | 13:36 | |
| TheJulia | well, at least stuff merged to master last night | 13:39 |
| *** rcernin has quit IRC | 13:40 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/rocky: Pin Ironic Tempest Plugin https://review.opendev.org/744773 | 13:43 |
| TheJulia | doh | 13:43 |
| iurygregory | it was green no? | 13:46 |
| iurygregory | .-. | 13:46 |
| TheJulia | nope | 13:47 |
| TheJulia | python-pip tries to re-install and things go *boom* | 13:47 |
| iurygregory | oh =( | 13:48 |
| iurygregory | oh stein was green | 13:48 |
| iurygregory | since you update to put the multinode non-voting | 13:48 |
| TheJulia | I also disabled the coreos jobs, since it is completely OEL at this point. They too were failing | 13:51 |
| TheJulia | EOL | 13:51 |
| iurygregory | yeah | 13:52 |
| iurygregory | makes sense | 13:52 |
| dtantsur | coreos jobs omg | 13:53 |
| dtantsur | maybe we just close everything older than train? :) | 13:53 |
| iurygregory | do we care about the "experimental" jobs in ipa? | 13:53 |
| dtantsur | I don't think so | 13:53 |
| iurygregory | I don't even think it would make that sense since we test equivalent in CI | 13:53 |
| TheJulia | well, anything older than train can basically take the appraoch of "we dont' REALLY need to keep the jobs working" but they are super useful for nasty bug fixes | 13:53 |
| TheJulia | that _should_ be backported | 13:54 |
| TheJulia | but yeah, the coreos jobs can just go away | 13:54 |
| TheJulia | https://www.ebay.com/itm/New-Shop-Made-Boring-Not-A-Flamethrower-GRAY-FREE-Priority-Shipping/184390419122 would do nicely | 13:55 |
| iurygregory | WOW | 13:56 |
| * TheJulia wonders if this morning's blend of coffee is enough or too much caffeine | 13:56 | |
| iurygregory | you will need caffeine | 13:58 |
| iurygregory | since we will start talking about grenade.. | 13:58 |
| TheJulia | bring it! | 13:58 |
| *** mkrai has joined #openstack-ironic | 13:58 | |
| dtantsur | TheJulia: "May not ship to Germany" :( | 14:00 |
| iurygregory | stendulker is not around but he answered the doodle | 14:00 |
| dtantsur | a bit late for him, I suspect | 14:00 |
| iurygregory | for anyone how wants to join https://bluejeans.com/imelofer | 14:02 |
| * dtantsur wonders if his open patches list will shrink substantially during his PTO | 14:03 | |
| *** mbuil has quit IRC | 14:04 | |
| *** mbuil has joined #openstack-ironic | 14:04 | |
| *** mkrai has quit IRC | 14:11 | |
| *** mkrai has joined #openstack-ironic | 14:12 | |
| brtknr | hey folks, why do some of my nodes fail to rebuild because "Operation was aborted due to conductor take over" | 14:31 |
| dtantsur | brtknr: was a conductor restarted during the process? | 14:33 |
| brtknr | ironic-conductor? | 14:33 |
| dtantsur | yeah | 14:33 |
| dtantsur | this happens when a new conductor takes over nodes from a dead conductor | 14:33 |
| brtknr | nope not as far as i can tell | 14:34 |
| brtknr | last time one of the contollers was restarted was 30 hrs ago | 14:35 |
| brtknr | this happened now | 14:35 |
| dtantsur | weird. you need to find the corresponding record in the conductor logs (one of them) and see what exactly caused it | 14:37 |
| brtknr | dtantsur: 2020-08-04 15:01:40.209 6 ERROR ironic.conductor.task_manager [req-23427b64-1ada-48b8-af60-7e281c5b83bf - - - - -] Node 6e256e2c-3174-4269-aa01-4ed5d17e2e2d moved to provision state "deploy failed" from state "deploying"; target provision state is "active" | 14:40 |
| brtknr | this ERROR precedes the takeover warning | 14:40 |
| brtknr | dtantsur: 2020-08-04 15:01:39.756 6 WARNING ironic.conductor.manager [-] Forcibly removed reservation of conductor cpu-e-1041 on node 2fe009ea-5457-4f5e-8afd-b357abff21ce as that conductor w | 14:42 |
| brtknr | ent offline | 14:42 |
| brtknr | I also see this | 14:42 |
| brtknr | which is probably whats causing the issue | 14:44 |
| brtknr | thanks, I'll keep digging | 14:44 |
| *** diurnalist has joined #openstack-ironic | 14:57 | |
| *** Qianbiao has quit IRC | 15:01 | |
| *** belmoreira has quit IRC | 15:06 | |
| TheJulia | dtantsur: when are you heading out on pto? | 15:14 |
| TheJulia | Tomorrow, just next week? | 15:15 |
| dtantsur | TheJulia: next week, still here tomorrow | 15:16 |
| TheJulia | awesome, hopefully I can get you to take a ntoehr quick look at the TypeError avoidance/fix retry patch | 15:17 |
| TheJulia | another | 15:17 |
| dtantsur | yep | 15:17 |
| dtantsur | I also hope to get at least the critical part of the sushy-tools patch series merged | 15:18 |
| TheJulia | fyi, regarding the forum: https://wiki.openstack.org/wiki/Forum#Committee_Timelines <-- call for brainstorming coming :) | 15:18 |
| TheJulia | dtantsur: I reviewed last night and everything looked fairly good to me | 15:18 |
| TheJulia | well, reviewed yesterday afternoon before I gave an internal talk | 15:18 |
| dtantsur | thx, I had to fix one thing that I spotted later | 15:18 |
| dtantsur | in https://review.opendev.org/744862 | 15:18 |
| TheJulia | dtantsur: was that one a wip yesterday? | 15:19 |
| dtantsur | yep | 15:19 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/networking-generic-switch master: Generate key on PEM format https://review.opendev.org/739186 | 15:20 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/networking-generic-switch stable/ussuri: Generate key on PEM format https://review.opendev.org/745169 | 15:21 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Change UEFI PXE job to use tinyipa https://review.opendev.org/744979 | 15:21 |
| TheJulia | dtantsur: that explains it, reviewing now | 15:21 |
| TheJulia | GAAAAAAAAAHHHHH | 15:24 |
| rpittau | :( | 15:24 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Native zuulv3 grenade multinode multitenant https://review.opendev.org/705030 | 15:26 |
| iurygregory | fingers crossed | 15:26 |
| * iurygregory facepalm - I added an empty line to a file lol | 15:31 | |
| *** ntt__ has quit IRC | 15:37 | |
| *** mkrai has quit IRC | 15:44 | |
| *** penick has joined #openstack-ironic | 15:54 | |
| *** antotala has joined #openstack-ironic | 15:56 | |
| *** lucasagomes has quit IRC | 16:00 | |
| *** penick has quit IRC | 16:03 | |
| *** penick has joined #openstack-ironic | 16:04 | |
| *** iurygregory has quit IRC | 16:10 | |
| *** iurygregory has joined #openstack-ironic | 16:11 | |
| openstackgerrit | Riccardo Pittau proposed openstack/ironic master: Enforce autospec in test_cinder https://review.opendev.org/745178 | 16:18 |
| *** dougsz has quit IRC | 16:29 | |
| *** Lucas_Gray has quit IRC | 16:30 | |
| *** antotala has quit IRC | 16:34 | |
| *** patchbot has joined #openstack-ironic | 16:34 | |
| *** k_mouza has quit IRC | 16:45 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Pin ironic-tempest-plugin https://review.opendev.org/744547 | 16:48 |
| *** gyee has joined #openstack-ironic | 16:51 | |
| *** JamesBenson has joined #openstack-ironic | 16:56 | |
| *** diurnalist has quit IRC | 16:57 | |
| *** derekh has quit IRC | 17:01 | |
| *** dtantsur is now known as dtantsur|afk | 17:03 | |
| dtantsur|afk | o/ | 17:03 |
| openstackgerrit | Merged openstack/ironic stable/rocky: Pin ironic-tempest-plugin https://review.opendev.org/744567 | 17:05 |
| stbenjam | Has anyone ever seen "Error: could not inspect: could not inspect node, node is currently 'inspect failed', last error was 'Failed to inspect hardware. Reason: unable to start inspection: HttpException: 500: Server Error for url: http://172.22.0.2:5050/v1/introspection/c0028cc5-7008-45c5-a0f7-f2c8067b7c38?manage_boot=False, Internal server error (TypeError): 'dict' object is not callable" | 17:17 |
| *** penick has quit IRC | 17:19 | |
| *** k_mouza has joined #openstack-ironic | 17:21 | |
| openstackgerrit | Merged openstack/virtualbmc master: Set min version of tox to 3.2.1 https://review.opendev.org/741192 | 17:25 |
| *** k_mouza has quit IRC | 17:26 | |
| *** priteau has quit IRC | 17:27 | |
| *** priteau has joined #openstack-ironic | 17:27 | |
| *** priteau has quit IRC | 17:32 | |
| *** patchbot has quit IRC | 18:01 | |
| *** patchbot has joined #openstack-ironic | 18:01 | |
| *** priteau has joined #openstack-ironic | 18:07 | |
| *** priteau has quit IRC | 18:11 | |
| TheJulia | stbenjam: nope :( | 18:15 |
| TheJulia | stbenjam: anything in the inspector log? | 18:16 |
| *** k_mouza has joined #openstack-ironic | 18:19 | |
| stbenjam | TheJulia: Don't have them :-( Never saw it locally. We only have 1 example of it so far, and no logs got captured. It was just interesting to me since we've never seen any error like that | 18:19 |
| TheJulia | stbenjam: I kind of wonder if the client ended up sending an invalid message body | 18:20 |
| TheJulia | That could explain the error at least | 18:20 |
| *** k_mouza has quit IRC | 18:23 | |
| *** k_mouza has joined #openstack-ironic | 18:29 | |
| *** gmann is now known as gmann_afk | 18:35 | |
| openstackgerrit | Verification of a change to openstack/ironic failed: Auto extend the timeout for RAX hosts https://review.opendev.org/741853 | 18:47 |
| *** sshnaidm is now known as sshnaidm|afk | 18:47 | |
| *** k_mouza has quit IRC | 18:51 | |
| dking | I'm wanting to add a switches sys-name from LLDP to local_link_connection during inspection as the chassis ID (currently being stored in switch_id) is only a MAC address and less helpful than a hostname for the switch. However, I see that the only allowed fields in local_link_connection are "switch_id" and "port_id", and an optional "switch_info". I'd like to keep the switch_id as is, since I'm sure a lot relies on it already a | 18:51 |
| dking | nd it's a mandatory TLV. I also don't like "switch_info" as that probably already has its place and is vendor specific. Could a new entry be added, or should inspector update one of the other fields? | 18:51 |
| *** k_mouza has joined #openstack-ironic | 18:58 | |
| *** diurnalist has joined #openstack-ironic | 18:59 | |
| dking | TheJulia: ^ Any thoughts on that? | 19:02 |
| *** k_mouza has quit IRC | 19:08 | |
| bfournie | dking: agree that switch_id should not be touched, I think its reasonable to set switch_info to the sys-name received via lldp, see how its set by user here https://docs.openstack.org/ironic/train/admin/multitenancy.html. Could also add a a configurable setting to prevent lldp plugin from overwriting (default=don't overwrite) | 19:13 |
| dking | bfournie: Thank you. It looks like the current rule already doesn't create a patch if the item already exists, so it should be fine adding it there. It's just that other folks may have introspection rules later that check for that field to be empty before adding in their custom info. But if that works best for everybody else, I'm fine with it. | 19:21 |
| dking | And I meant, it looks to make sure it's not overwriting here if CONF.processing.overwrite_existing is set. | 19:23 |
| dking | s/is set/is not set/ | 19:32 |
| *** Zempashi has quit IRC | 20:07 | |
| *** Zempashi has joined #openstack-ironic | 20:09 | |
| openstackgerrit | Merged openstack/ironic stable/ussuri: Change UEFI PXE job to use tinyipa https://review.opendev.org/744979 | 20:16 |
| openstackgerrit | Merged openstack/ironic-tempest-plugin master: Enforce check on flake8 E129 test https://review.opendev.org/744514 | 20:16 |
| openstackgerrit | Verification of a change to openstack/ironic failed: Pin ironic-tempest-plugin https://review.opendev.org/744547 | 20:16 |
| TheJulia | oh yay! | 20:16 |
| openstackgerrit | Julia Kreger proposed openstack/ironic bugfix/15.1: Wipe agent token and URL on rescue and unrescue https://review.opendev.org/745222 | 20:28 |
| iurygregory | wut py36 failure | 20:55 |
| iurygregory | ty sqlalchemy | 20:56 |
| *** Lucas_Gray has joined #openstack-ironic | 21:01 | |
| stevebaker | morning | 21:07 |
| iurygregory | morning stevebaker | 21:08 |
| *** k_mouza has joined #openstack-ironic | 21:09 | |
| *** k_mouza has quit IRC | 21:14 | |
| *** rcernin has joined #openstack-ironic | 21:21 | |
| stevebaker | hey I've got an appointment in town, I'll be back in a couple of hours | 21:23 |
| *** Lucas_Gray has quit IRC | 21:27 | |
| *** rcernin has quit IRC | 21:27 | |
| *** sdanni has quit IRC | 21:31 | |
| *** rcernin has joined #openstack-ironic | 21:34 | |
| *** Lucas_Gray has joined #openstack-ironic | 21:34 | |
| openstackgerrit | Merged openstack/sushy-tools master: Replace the static Manager driver with a new fake one https://review.opendev.org/744862 | 21:40 |
| *** rcernin has quit IRC | 21:42 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-python-agent stable/queens: Pin Ironic Tempest Plugin https://review.opendev.org/744969 | 21:45 |
| openstackgerrit | Julia Kreger proposed openstack/ironic stable/queens: Pin ironic-tempest-plugin https://review.opendev.org/745236 | 21:58 |
| TheJulia | can't believe I forgot about that one | 22:00 |
| TheJulia | iurygregory: heh | 22:00 |
| iurygregory | TheJulia, it happens =) | 22:00 |
| *** ociuhandu_ has joined #openstack-ironic | 22:03 | |
| *** ociuhandu has quit IRC | 22:04 | |
| janders | g'day Ironic o/ | 22:06 |
| iurygregory | morning janders o/ | 22:06 |
| clarkb | we think we've landed a fix for the suse repo problems (not sure we fully understand why the fix was necessary but testing shows it is happier) | 22:07 |
| clarkb | not sure what changes ya'll might want to recheck with taht in place but I think you can do so no | 22:07 |
| clarkb | (bifrost things iirc) | 22:07 |
| iurygregory | clarkb, tks! | 22:08 |
| * janders is reading Red Hat research publication | 22:08 | |
| janders | very cool! | 22:08 |
| janders | I should probably chat to the authors :) we're passionate about similar things | 22:10 |
| *** rcernin has joined #openstack-ironic | 22:11 | |
| *** hoonetorg has quit IRC | 22:15 | |
| *** rcernin has quit IRC | 22:17 | |
| TheJulia | clarkb: thanks | 22:34 |
| TheJulia | tzumainn: janders, janders: tzumainn | 22:34 |
| TheJulia | enjoy! | 22:35 |
| *** gmann_afk is now known as gmann | 22:35 | |
| TheJulia | i'm going to call it a day folks, I managed to actually spend some time on reviewing things once again \o/ | 22:36 |
| TheJulia | The priorities for the week list has been updated, most everything is up to date. I did pull approval for a python-ironicclient patch... the failure looks legitimate :( | 22:39 |
| iurygregory | aww .-. | 22:41 |
| TheJulia | c'est la vie | 22:41 |
| openstackgerrit | Merged openstack/bifrost master: Clean up bindep.txt https://review.opendev.org/744930 | 22:42 |
| janders | thanks TheJulia :) | 22:45 |
| janders | tzumainn great work on TSI. I've done some work on multi-tenant baremetal-openstack with SDN InfiniBand - my motivation was creating foundation for software-defined supercomputing | 22:47 |
| *** rcernin has joined #openstack-ironic | 22:47 | |
| janders | I very much like the multi-tenancy improvements you describe in https://research.redhat.com/wp-content/uploads/2020/02/RHRQ-v2.2-digital.pdf and related presos - great work! | 22:48 |
| *** rh-jelabarre has quit IRC | 22:49 | |
| *** rh-jelabarre has joined #openstack-ironic | 22:50 | |
| janders | s/TSI/ESI | 22:52 |
| *** rcernin has quit IRC | 22:52 | |
| openstackgerrit | Merged openstack/bifrost master: Add developer mode to bifrost https://review.opendev.org/744233 | 22:53 |
| *** tkajinam has joined #openstack-ironic | 22:55 | |
| *** Lucas_Gray has quit IRC | 22:56 | |
| *** JamesBenson has quit IRC | 23:01 | |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic-tempest-plugin master: Update basic_ops https://review.opendev.org/741445 | 23:13 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Add tempest default_boot_option and altflavor https://review.opendev.org/741886 | 23:14 |
| *** diurnalist has quit IRC | 23:17 | |
| *** rcernin has joined #openstack-ironic | 23:20 | |
| *** Zempashi has joined #openstack-ironic | 23:39 | |
| openstackgerrit | Merged openstack/ironic stable/ussuri: Remove locks before RPC bus is started https://review.opendev.org/743808 | 23:52 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!