| opendevreview | Verification of a change to openstack/ironic-python-agent-builder master failed: Preliminary work to support CentOS 9 Stream https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/806809 | 01:20 |
|---|---|---|
| opendevreview | Verification of a change to openstack/ironic-python-agent-builder master failed: Remove support for iSCSI deploy from the DIB builder https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/808214 | 01:20 |
| *** pmannidi is now known as pmannidi|Lunch | 03:33 | |
| *** pmannidi|Lunch is now known as pmannidi | 05:06 | |
| iurygregory | good morning Ironic o/ | 05:54 |
| stendulker_ | Good morning iurygregory | 05:59 |
| iurygregory | stendulker_, o/ | 06:01 |
| arne_wiebalck | Good morning iurygregory stendulker_ and Ironic! | 06:45 |
| stendulker_ | arne_wiebalck o/ | 06:46 |
| iurygregory | arne_wiebalck, o/ | 06:46 |
| opendevreview | Merged openstack/ironic master: Always update cache for HTTP images if Last Modified is unknown https://review.opendev.org/c/openstack/ironic/+/807863 | 07:19 |
| opendevreview | Merged openstack/ironic master: Enable parallel downloads and allow tuning concurrency https://review.opendev.org/c/openstack/ironic/+/807870 | 07:19 |
| opendevreview | Verification of a change to openstack/ironic bugfix/18.1 failed: Improve edge-case debugging for deployment and cleaning https://review.opendev.org/c/openstack/ironic/+/807262 | 07:19 |
| *** pmannidi is now known as pmannidi|AFK | 07:27 | |
| zigo | Hi there! I'm looking into a way, with HPE machines (ie: ProLiant DL385 Gen10 Plus) to set the "Boot controller order" (ie: boot on the "Embedded SATA Controller #1" instead of the "Storage Slot 12: HPE SMART Array") in a programatic way (ie: redfish or something else on the shell). Since Ironic deals with hardware, I wondered if any of you could point me to the right direction ... | 07:42 |
| zigo | Any clue?!? | 07:42 |
| opendevreview | Merged openstack/ironic stable/wallaby: Improve edge-case debugging for deployment and cleaning https://review.opendev.org/c/openstack/ironic/+/807263 | 08:38 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Fix idrac-wsman set_power_state to wait on HW https://review.opendev.org/c/openstack/ironic/+/808098 | 09:09 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/808694 | 09:24 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/808696 | 09:24 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter master: Update master for stable/xena https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/808698 | 09:24 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-prometheus-exporter master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/808700 | 09:24 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/ironic-ui/+/808702 | 09:24 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/ironic-ui/+/808703 | 09:24 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui master: Update master for stable/xena https://review.opendev.org/c/openstack/ironic-ui/+/808704 | 09:24 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-ui master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/ironic-ui/+/808705 | 09:24 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Fix clear foreign config in idrac-redfish RAID https://review.opendev.org/c/openstack/ironic/+/804848 | 09:51 |
| opendevreview | Merged openstack/ironic-prometheus-exporter stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/808694 | 09:54 |
| opendevreview | Merged openstack/ironic-prometheus-exporter stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/808696 | 09:54 |
| opendevreview | Merged openstack/ironic-ui stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/ironic-ui/+/808702 | 10:13 |
| opendevreview | Merged openstack/ironic-ui stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/ironic-ui/+/808703 | 10:16 |
| *** hjensas is now known as hjensas|afk | 12:45 | |
| TheJulia | good morning | 13:14 |
| TheJulia | zigo: programatically, externally... ehh... There *is* exposure on the rest api of the uefi nvram but I'm not sure it is settable via the bmc, the pattern that I know works is doing so from with-in a running OS with efibootmgr | 13:16 |
| TheJulia | Also, good morning everyone | 13:16 |
| arne_wiebalck | Hey TheJulia o/ | 13:18 |
| TheJulia | good morning | 13:20 |
| iurygregory | good morning TheJulia | 13:27 |
| zigo | TheJulia: Thanks. I've found the HP tool called ilorest, but it's packaging is HORRIBLE. | 13:27 |
| zigo | TheJulia: They use pyinstaller, which bundles even the Python interpreter !!! | 13:27 |
| TheJulia | zigo: ugh | 13:27 |
| zigo | I'm trying to make sense of it, but it's kind of not easy ... | 13:27 |
| zigo | From https://pyinstaller.readthedocs.io/en/stable/operating-mode.html# : | 13:28 |
| zigo | "PyInstaller reads a Python script written by you. It analyzes your code to discover every other module and library your script needs in order to execute. Then it collects copies of all those files – including the active Python interpreter! – and puts them with your script in a single folder, or optionally in a single executable file." | 13:28 |
| zigo | :/ | 13:28 |
| TheJulia | One person's feature is another person's bug :( | 13:29 |
| TheJulia | zigo search for "Change UEFI boot order example" at https://hewlettpackard.github.io/ilo-rest-api-docs/ilo5/#computersystem-v1_4_0-computersystem | 13:33 |
| zigo | Thanks a thousands time. | 13:34 |
| TheJulia | Above that it describes decoding the settings | 13:34 |
| zigo | I've been spending days and weeks (literally) searching for a solution... | 13:34 |
| TheJulia | zigo: I had a feeling it was burried under ComputerSystem in their redfish API. | 13:34 |
| TheJulia | I <3 their docs on their API implementation | 13:35 |
| TheJulia | not every vendor unfortunately provides those settings as a human readable string | 13:36 |
| TheJulia | and the whole custom field they have :( | 13:36 |
| TheJulia | c'est la vie | 13:36 |
| *** tosky is now known as Guest7131 | 13:40 | |
| *** tosky_ is now known as tosky | 13:40 | |
| zigo | I just found out: | 13:51 |
| zigo | curl -X PATCH /redfish/v1/systems/1/bios/oem/hpe/boot/settings/ -d '{"PersistentBootConfigOrder": ["HD.EmbSATA.3.1", "HD.EmbSATA.4.1", "HD.Slot.12.2", "HD.Slot.12.3", "HD.Slot.12.4", "HD.Slot.12.5", "HD.Slot.12.6", "HD.Slot.12.7", "HD.Slot.12.8", "HD.Slot.12.9", "HD.Slot.12.10", "HD.Slot.12.11", "HD.Slot.12.12", "HD.Slot.12.13", "Generic.USB.1.1", "OCP.Slot.10.1.Httpv4", "OCP.Slot.10.1.IPv4", "OCP.Slot.10.1.Httpv6", "OCP.Slot.10.1.IPv6"]}' | 13:51 |
| zigo | :) | 13:51 |
| TheJulia | I don't think OCP was in that list | 14:03 |
| TheJulia | zigo: I guess that is the embedded controller, it is mapping a disk alias over them it looks like based upon slot/sub-instance | 14:06 |
| iurygregory | OCP slot LOL | 14:28 |
| zigo | It's not working indeed ... | 14:35 |
| opendevreview | Julia Kreger proposed openstack/ironic master: API endpoints to get node history https://review.opendev.org/c/openstack/ironic/+/803855 | 14:42 |
| TheJulia | zigo: hmmmm There was something with patching bios settings directly | 14:44 |
| TheJulia | so the patch example also has patching /redfish/v1/Systems/1/BIOS/Boot/Settings/ instead of the oem path. | 14:45 |
| TheJulia | hpe would propose stuff on oem paths, then move towards standardizing it | 14:45 |
| *** JasonF is now known as JayF | 14:47 | |
| iurygregory | #startmeeting ironic | 15:00 |
| opendevmeet | Meeting started Mon Sep 13 15:00:06 2021 UTC and is due to finish in 60 minutes. The chair is iurygregory. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'ironic' | 15:00 |
| iurygregory | o/ | 15:00 |
| TheJulia | o/ | 15:00 |
| arne_wiebalck | o/ | 15:00 |
| sam_z | o/ | 15:00 |
| ajya | o/ | 15:00 |
| iurygregory | Hello everyone, welcome to our weekly meeting | 15:01 |
| rloo | o/ | 15:01 |
| rpioso | o/ | 15:01 |
| stendulker | o/ | 15:01 |
| iurygregory | seems like we have enough people so let's get started :D | 15:01 |
| iurygregory | #topic Announcements/Reminders | 15:01 |
| iurygregory | #info This week is the R-3 week - we have RC1 for all projects with release-model cycle-with-rc under the ironic | 15:01 |
| erbarr | o/ | 15:02 |
| iurygregory | #info Yoga PTG is in 5 weeks! Don't forget to register and add topics in our etherpad! | 15:02 |
| iurygregory | #link https://www.eventbrite.com/e/project-teams-gathering-october-2021-tickets-161235669227 | 15:02 |
| iurygregory | #link https://etherpad.opendev.org/p/ironic-yoga-ptg | 15:02 |
| iurygregory | Does anyone have something to add to our Announcements/Reminders? =) | 15:03 |
| TheJulia | iurygregory: I guess we need to try and release end of this week or at the latest sometime next week. | 15:04 |
| TheJulia | Since we typically trail a little after RC1 release as often we devstack changes which can break us at times. | 15:05 |
| iurygregory | TheJulia, agree, I would say that depending on the project we can wait till next week | 15:05 |
| TheJulia | sooner obviously better, of course | 15:06 |
| iurygregory | yeah for most of the projects I think we are in a good shape, ipa-builder we will be able to release this week, we need to take a look at ironic and inspector | 15:06 |
| iurygregory | moving on | 15:07 |
| iurygregory | #topic Review action items from previous meeting | 15:08 |
| iurygregory | skipping since we don't have any Action Item | 15:08 |
| iurygregory | #topic Review subteam status reports | 15:08 |
| iurygregory | starting on L60 | 15:08 |
| TheJulia | url? :) | 15:08 |
| iurygregory | #link https://etherpad.opendev.org/p/IronicWhiteBoard | 15:09 |
| iurygregory | sorry =) | 15:09 |
| TheJulia | no worries | 15:09 |
| TheJulia | I'm going to note db performance done | 15:09 |
| iurygregory | makes sense to me | 15:10 |
| TheJulia | I've noticed our periodics are becoming less... efficent, and Dmitry and I chatted about this some last week but it will be a longer term better solution | 15:10 |
| iurygregory | do we have some data that confirms that is becoming less efficient? (just wondering) =) | 15:11 |
| TheJulia | knowledge of hundreds of needless extra db queries | 15:11 |
| TheJulia | based upon the pattern | 15:11 |
| iurygregory | got it | 15:11 |
| TheJulia | For example, if we request driver_internal_info from nodes_iter, and then pull a task before making a decision based on d_i_info, then we've triggered more db queries needlessly | 15:12 |
| TheJulia | when d_i_i had the answer all along | 15:12 |
| TheJulia | I may look around and try to fix those this week | 15:12 |
| * TheJulia notes node history | 15:13 | |
| iurygregory | got it, yeah if we have the information handy we don't need to go to the db... | 15:13 |
| TheJulia | Updated the node event history stuffs | 15:15 |
| TheJulia | with links! | 15:15 |
| iurygregory | links++\ | 15:15 |
| arne_wiebalck | TheJulia: I will replace my name with yours for this item :) | 15:15 |
| TheJulia | arne_wiebalck: okay :) | 15:16 |
| iurygregory | or just add TheJulia =) | 15:16 |
| TheJulia | arne_wiebalck: please feel free to review. I got nobodycam to skim the periodic logic and he +1'ed it based upon his use case and also incorporated his feedback | 15:16 |
| arne_wiebalck | TheJulia: ok, I will! | 15:17 |
| iurygregory | are we good to proceed? | 15:17 |
| TheJulia | yes | 15:17 |
| iurygregory | #topic Deciding on priorities for the coming week | 15:18 |
| iurygregory | #link https://review.opendev.org/q/status:open+hashtag:ironic-week-prio | 15:18 |
| TheJulia | I'd like to get reviews on the events patches this week, it would be a good item to get out there and people can at least use it as soon as the client is out early next cycle | 15:18 |
| ajya | Can these be added to priorities - https://review.opendev.org/c/openstack/ironic/+/808098 https://review.opendev.org/c/openstack/ironic/+/805431 https://review.opendev.org/c/openstack/ironic/+/805428 | 15:18 |
| ajya | given that release is close and 2 of them have been open for a while | 15:19 |
| iurygregory | ajya, ++ make sense to add them I would say | 15:19 |
| iurygregory | TheJulia, I didn't see any patches for events (maybe I missed) | 15:20 |
| iurygregory | but I does sound like something we should try to include | 15:20 |
| TheJulia | iurygregory: node history | 15:20 |
| iurygregory | oh ok, related to node history! | 15:21 |
| TheJulia | adding | 15:21 |
| TheJulia | added | 15:21 |
| iurygregory | ajya, I think we probably want to backport https://review.opendev.org/c/openstack/ironic/+/805428 right? | 15:22 |
| iurygregory | can you add a releasenote? | 15:22 |
| TheJulia | https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/808198 would also be good to land this week | 15:22 |
| ajya | iurygregory: yes to backport and releasenote | 15:23 |
| iurygregory | TheJulia, ++ I will take a look | 15:23 |
| TheJulia | hmmm https://review.opendev.org/c/openstack/networking-generic-switch/+/763794 is tagged as a wip | 15:25 |
| * TheJulia wonders if it is not really | 15:25 | |
| TheJulia | of course, it has also missed the release | 15:25 |
| iurygregory | TheJulia, well we can hold a bit the ngs release (but we need to do this week) | 15:25 |
| * iurygregory wondering why the patch is marked as WIP... | 15:26 | |
| TheJulia | oh yeah | 15:26 |
| TheJulia | it hasn't released yet | 15:26 |
| TheJulia | that we need to go ahead and get sorted this week | 15:26 |
| iurygregory | yup, we can still hold ngs, networking-baremetal | 15:26 |
| TheJulia | yup | 15:27 |
| TheJulia | it has only had very minor changes since the last release | 15:27 |
| TheJulia | being networking-baremetal | 15:27 |
| iurygregory | exactly | 15:28 |
| iurygregory | later today I will take a look at the non-tagged in patches in our projects and see if there is something we want to include | 15:29 |
| TheJulia | ++ | 15:29 |
| iurygregory | should we move on? | 15:29 |
| TheJulia | yes | 15:29 |
| iurygregory | #topic Discussion | 15:30 |
| iurygregory | I have a topic, but I forgot to add to our agenda | 15:30 |
| TheJulia | Okay | 15:30 |
| iurygregory | I was talking with the release team since we need the RC for some of our deliverables | 15:31 |
| TheJulia | should be the neutron tied ones | 15:31 |
| iurygregory | we have projects like IPE and ironic-ui (that doesn't get a lot of changes) and we are using cycle-with-rc | 15:31 |
| TheJulia | Because the release team wants to cut a new release regardless of if it changes. | 15:32 |
| TheJulia | which... artificially increases the version | 15:32 |
| TheJulia | sadly | 15:32 |
| TheJulia | What are they proposing? | 15:32 |
| iurygregory | yes, but they will cut with the major bump, I don't think it makes sense... | 15:32 |
| TheJulia | Yeah... That has been a point of contention in the past. | 15:33 |
| iurygregory | yup, I noticed that they asked for the change to RC | 15:34 |
| iurygregory | https://opendev.org/openstack/releases/commit/7ac3b50659a225846831fec978ecc89041ee459e | 15:34 |
| iurygregory | for the end user perspective if they check the version they will think we added a bunch of new things that required a Major bump.. | 15:34 |
| TheJulia | OpenStack's All or Nothing model is *largely* driven to help drive market forces and new versions are part of that, but at the same time they are supposed to also pull in the last release or the next appropriate release. | 15:34 |
| TheJulia | however, that has changed such where they require a release to be pulled in during a cycle | 15:35 |
| TheJulia | iurygregory: exactly | 15:35 |
| TheJulia | iurygregory: and even if we had a release note to the contrary, the perception has been set. | 15:35 |
| iurygregory | yup! | 15:35 |
| iurygregory | oh we added this small feature, there is no breaking change in our API (major bump \o/) | 15:36 |
| TheJulia | yup | 15:37 |
| TheJulia | As I said, it is really driven to try and play the market with "new version" | 15:37 |
| iurygregory | so I think we probably need to try to evaluate our projects to cycle-with-intermediary, or maybe see with the release team if we can have a new model (not sure) | 15:37 |
| opendevreview | Verification of a change to openstack/ironic-inspector master failed: Update parameters for authtoken middleware https://review.opendev.org/c/openstack/ironic-inspector/+/805443 | 15:37 |
| TheJulia | iurygregory: It could be good to discuss | 15:37 |
| TheJulia | iurygregory: perhaps a ptg session with the release team. I'm sure largely they are driven by the "this is the way it has been documented", and that can be changed if they are willing | 15:38 |
| iurygregory | TheJulia, yeah I was thinking about that | 15:38 |
| iurygregory | I will talk with Herve about this | 15:39 |
| TheJulia | Okay, let me know if there is any way I can help | 15:39 |
| iurygregory | #action iurygregory to talk with Herve about changing the release of some ironic projects | 15:40 |
| iurygregory | TheJulia, sure! tks | 15:40 |
| iurygregory | that's it from my side | 15:40 |
| TheJulia | onward? | 15:40 |
| iurygregory | #topic Baremetal SIG | 15:40 |
| iurygregory | #link https://etherpad.opendev.org/p/bare-metal-sig | 15:40 |
| iurygregory | arne_wiebalck, do you have something for the Baremetal SIG ? | 15:41 |
| arne_wiebalck | Lively ops/feedback discussion last week. | 15:41 |
| arne_wiebalck | There were suggestions to have a 2nd one as we did not really finish. | 15:41 |
| TheJulia | second one would be good | 15:41 |
| arne_wiebalck | Maybe it is a good idea to have another one right before the PTG? | 15:41 |
| TheJulia | I know two firms that just had stuff come up and couldn't make it | 15:42 |
| TheJulia | arne_wiebalck: ++ | 15:42 |
| iurygregory | ++ | 15:42 |
| arne_wiebalck | oh, ok! | 15:42 |
| arne_wiebalck | How do we find a slot? | 15:42 |
| arne_wiebalck | Also for TheJulia to join :) | 15:42 |
| arne_wiebalck | TheJulia: you have a permanent clash with the slot? | 15:42 |
| TheJulia | arne_wiebalck: yes, OSF board meetings | 15:42 |
| TheJulia | err | 15:43 |
| TheJulia | OIF | 15:43 |
| arne_wiebalck | TheJulia: they picked our slot? :-D | 15:43 |
| iurygregory | I also know I friend that couldn't join because it was holiday in Brazil | 15:43 |
| TheJulia | arne_wiebalck: it has always been their slot, just... not every month and sometimes shifted time zone wise. This year threw a wrench into things as we were not able to do in-person | 15:43 |
| TheJulia | I think maybe a little later would help, at least from some of the feedback from west-coast US operators was that it was just really early for most of them | 15:44 |
| arne_wiebalck | later, but same day of the week? | 15:44 |
| TheJulia | could be good | 15:44 |
| arne_wiebalck | would 2 hrs later be good enough? | 15:45 |
| TheJulia | yes | 15:45 |
| arne_wiebalck | that is still acceptable in Europe, I think | 15:45 |
| arne_wiebalck | well, some of Europe at least | 15:45 |
| TheJulia | time zones are hard :( | 15:45 |
| arne_wiebalck | ok, let's try, I will schedule one and we pick up the usual short tutorials after, soounds ok? | 15:45 |
| iurygregory | ++ | 15:46 |
| iurygregory | arne_wiebalck, thanks! | 15:46 |
| arne_wiebalck | cool, that's it from the SIG I believe | 15:46 |
| iurygregory | #topic RFE review | 15:47 |
| iurygregory | no topics so skipping | 15:47 |
| iurygregory | #topic Open discussion | 15:47 |
| iurygregory | Anyone have anything for our open discussion? | 15:48 |
| TheJulia | I've got nothing | 15:49 |
| TheJulia | arne_wiebalck: that sounds good to me | 15:49 |
| sam_z | i will be submitting a spec for review soon (once i can figure out how to do so) | 15:49 |
| TheJulia | I expect we should review jam attestation tomorrow | 15:49 |
| sam_z | it's for the redfish power control intermediary | 15:50 |
| iurygregory | sam_z, feel free to ask here and we can try to help you =) | 15:50 |
| TheJulia | sam_z: awesome | 15:50 |
| sam_z | will do! | 15:50 |
| iurygregory | TheJulia, ++ for review jam tomorrow | 15:50 |
| TheJulia | I can drive the jams | 15:50 |
| iurygregory | I can join without problems I think | 15:51 |
| iurygregory | so lets go for our last topic | 15:51 |
| iurygregory | #topic Who is going to run the next meeting? | 15:51 |
| iurygregory | Do we have any volunteers? | 15:51 |
| iurygregory | ok, I will run the meeting next week =) | 15:53 |
| iurygregory | Thanks everyone! | 15:53 |
| iurygregory | #endmeeting | 15:53 |
| opendevmeet | Meeting ended Mon Sep 13 15:53:27 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:53 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-09-13-15.00.html | 15:53 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-09-13-15.00.txt | 15:53 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/ironic/2021/ironic.2021-09-13-15.00.log.html | 15:53 |
| ajya | TheJulia: left a comment about the locking thing. I can move it up otherwise, however, I also need to check for interface, but I have only driver name, not object before getting the task. | 15:55 |
| ajya | Can this be fixed when new pattern is established? | 15:55 |
| opendevreview | Merged openstack/ironic-ui master: Update master for stable/xena https://review.opendev.org/c/openstack/ironic-ui/+/808704 | 15:55 |
| opendevreview | Aija Jauntēva proposed openstack/ironic master: Fix iDRAC import configuration missing task handling https://review.opendev.org/c/openstack/ironic/+/805428 | 15:57 |
| ajya | iurygregory: added the release note ^ | 15:57 |
| opendevreview | Merged openstack/bifrost master: CI: copy bifrost logs to the log directory https://review.opendev.org/c/openstack/bifrost/+/806155 | 16:01 |
| opendevreview | Merged openstack/bifrost master: CI: collect keystone information https://review.opendev.org/c/openstack/bifrost/+/806156 | 16:05 |
| TheJulia | ajya: you can check for interface if the field matches, but it doesn't need to be the first thing done | 16:06 |
| TheJulia | ajya: checking for interface first with a task is just a huge performance hit to the db | 16:06 |
| * TheJulia opens the current state of master to do some periodic cleanups | 16:09 | |
| opendevreview | Merged openstack/ironic-ui master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/ironic-ui/+/808705 | 16:09 |
| NobodyCam | Good Morning OpenStack Folks!... happy ... ummm.... Monday | 16:12 |
| opendevreview | Merged openstack/python-ironicclient master: Update master for stable/xena https://review.opendev.org/c/openstack/python-ironicclient/+/808294 | 16:13 |
| TheJulia | good morning NobodyCam | 16:15 |
| TheJulia | umm.. Monday. | 16:15 |
| NobodyCam | heheheh o/ | 16:15 |
| arne_wiebalck | Good morning NobodyCam, happy Monday ;) | 16:26 |
| arne_wiebalck | Bye everyone, see you tmrw o/ | 16:26 |
| NobodyCam | Good morning and good evening arne_wiebalck | 16:26 |
| NobodyCam | Hehehe | 16:27 |
| TheJulia | ajya: patch inbound, just running unit tests now | 16:34 |
| TheJulia | err, syntax errors | 16:36 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Preliminary work to support CentOS 9 Stream https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/806809 | 16:39 |
| * TheJulia does the math on just how many db queries could occur on an idel deployment | 17:00 | |
| opendevreview | Samuel Zuk proposed openstack/ironic-specs master: Power control intermediary spec proposal https://review.opendev.org/c/openstack/ironic-specs/+/808804 | 17:01 |
| sam_z | aye there we go | 17:01 |
| sam_z | had a last bit of trouble figuring out the link in the specs/not-implemented directory needed to be both soft and relative | 17:02 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Fix driver task pattern to reduce periodic db load https://review.opendev.org/c/openstack/ironic/+/808806 | 17:09 |
| TheJulia | arne_wiebalck: ^^^ fyi | 17:09 |
| TheJulia | sam_z: it happens :) I'll try to take a look today | 17:09 |
| TheJulia | ajya: rpioso: https://review.opendev.org/c/openstack/ironic/+/808806 <-- please read the commit message first for as much context as possible | 17:11 |
| opendevreview | Merged openstack/python-ironicclient master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/python-ironicclient/+/808299 | 17:24 |
| sam_z | i'm having a hard time figuring out why this zuul build failed, does anyone have any insights as to why this happened? https://zuul.opendev.org/t/openstack/build/51c81bf24053496589ab37bb6a169831 | 17:42 |
| TheJulia | I guess sphinx got updated and it just *really* doesn't like our docs | 17:53 |
| TheJulia | pdf build blows up wonderfully | 17:53 |
| sam_z | so it's a CI problem? | 18:02 |
| TheJulia | not sure | 18:02 |
| TheJulia | its throwing a ton of errors | 18:02 |
| * TheJulia tries locally | 18:03 | |
| * TheJulia needs to give the desktop some coffee | 18:05 | |
| TheJulia | installing latex... | 18:08 |
| sam_z | > this operation will consume 5846MB of disk space | 18:09 |
| TheJulia | I just got a "1.1 GB free" warning | 18:10 |
| sam_z | latex needs to relax lol | 18:10 |
| TheJulia | heh, looks like systemd is holding open some file handlers :\ | 18:12 |
| TheJulia | so works locally | 18:19 |
| TheJulia | trying with your patch | 18:20 |
| TheJulia | so, with your patch on top it hangs | 18:23 |
| TheJulia | expecting input 8| | 18:23 |
| sam_z | huh. | 18:24 |
| TheJulia | hmmm | 18:25 |
| TheJulia | oh | 18:26 |
| * TheJulia tries tweeking the formatting | 18:28 | |
| sam_z | one thing i do remember doing was changing how the .rst was indented like halfway thru writing the spec | 18:28 |
| TheJulia | sam_z: does the rendered output look correct? | 18:30 |
| sam_z | it looked fine when i ran "tox" with no arguments | 18:31 |
| sam_z | i.e. it built the html docs fine, but i never tried building the pdf docs | 18:31 |
| TheJulia | few do | 18:34 |
| sam_z | is that something i should do going forward? | 18:36 |
| TheJulia | eh, normally this stuff doesn't nr break like this | 18:44 |
| TheJulia | Underfull \hbox (badness 10000) in paragraph at lines 7637--7641 <-- closest I've found so far to maybe a cause | 18:45 |
| TheJulia | hmm | 18:47 |
| * TheJulia raises an eyebrow, nukes /doc/build and re-builds | 18:47 | |
| TheJulia | it is something to do with the tables | 19:00 |
| * TheJulia re-attempts the build | 19:01 | |
| TheJulia | sam_z: [*]_ seems to be your problem | 19:12 |
| sam_z | that must have been where the square brackets were!! | 19:13 |
| sam_z | https://www.overleaf.com/learn/latex/Errors/Missing_number,_treated_as_zero i saw this | 19:14 |
| sam_z | seems to be working a lot better after removing those footnotes | 19:16 |
| sam_z | and yep there we go, they built just fine after removing the footnotes from the table... | 19:16 |
| TheJulia | maybe explicitly number them? | 19:17 |
| sam_z | will try that | 19:18 |
| sam_z | that seems to have worked! | 19:27 |
| sam_z | bizarre stuff. | 19:27 |
| opendevreview | Samuel Zuk proposed openstack/ironic-specs master: Power control intermediary spec proposal https://review.opendev.org/c/openstack/ironic-specs/+/808818 | 19:28 |
| sam_z | oop | 19:29 |
| sam_z | so i did not realize that it was git-review that added the Change-Id when i ran git commit --amend so i commented it out... | 19:33 |
| opendevreview | Samuel Zuk proposed openstack/ironic-specs master: Power control intermediary spec proposal https://review.opendev.org/c/openstack/ironic-specs/+/808804 | 19:50 |
| sam_z | ok there we go | 19:50 |
| TheJulia | heh | 20:01 |
| TheJulia | I *suspect* the text is getting passed through and mis-interpretted, tbh | 20:01 |
| sam_z | it could possibly be that the symbols getting auto-inserted are causing some sort of problem? when i went thru the log i saw a bunch of missing character errors for things such as three periods getting turned into an elipsis and straight quotes getting turned into left/right quotes | 20:11 |
| TheJulia | sam_z: yeah, I was seeing some of that. underscore is supposed to be a link but I suspect it only works with rst translation to html | 20:18 |
| TheJulia | sam_z: reviewing now | 20:20 |
| TheJulia | intruiging | 20:27 |
| opendevreview | Steve Baker proposed openstack/bifrost master: WIP support grub network boot https://review.opendev.org/c/openstack/bifrost/+/807220 | 21:17 |
| * TheJulia offers gerrit coffee | 21:17 | |
| TheJulia | sam_z: overall, pretty good, posted some feedback | 21:28 |
| sam_z | TheJulia: will check it out, thank u! | 21:28 |
| sam_z | i have to run for today, but i will take a look at your comments and respond and/or update the plans accordingly tomorrow! | 21:33 |
| *** pmannidi|AFK is now known as pmannidi | 23:18 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!