Wednesday, 2021-08-04

opendevreviewSteve Baker proposed openstack/ironic master: Set postgress password encryption for FIPS compliance  https://review.opendev.org/c/openstack/ironic/+/80312801:18
opendevreviewSteve Baker proposed openstack/ironic master: DNM/WIP - Add FIPS jobs  https://review.opendev.org/c/openstack/ironic/+/79773901:18
opendevreviewJulia Kreger proposed openstack/ironic master: Implements node history: database  https://review.opendev.org/c/openstack/ironic/+/76800901:21
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Record Errors in history  https://review.opendev.org/c/openstack/ironic/+/80329201:21
TheJuliawheeeee no internet for a couple hours01:22
TheJuliadtantsur: NobodyCam: arne_wiebalck: I could use opinions on batch sizes for cleaning up node history records, wrt ^^^. The spec called for 300 entries by default and to batch a maximum of a thousand nodes per day. Operationally I guess that might be okay, but then again churn might be higher in some environments. I've not written tests around the db size of the work, but if they work it should keep things fairly clean.01:24
opendevreviewZhouHao proposed openstack/ironic master: update irmc document  https://review.opendev.org/c/openstack/ironic/+/80341202:06
opendevreviewZhouHao proposed openstack/ironic master: update irmc document  https://review.opendev.org/c/openstack/ironic/+/80341202:57
opendevreviewZhouHao proposed openstack/ironic master: update irmc document  https://review.opendev.org/c/openstack/ironic/+/80341203:30
opendevreviewcenne proposed openstack/ironic master: Add api endpoints for changing boot_mode and secure_boot state  https://review.opendev.org/c/openstack/ironic/+/80008404:46
*** pmannidi is now known as pmannidi|Lunch05:48
*** pmannidi|Lunch is now known as pmannidi06:46
iurygregorygood morning Ironic o/07:04
jandersgood morning iurygregory o/07:12
iurygregoryhey janders o/07:12
*** rpittau|afk is now known as rpittau07:13
rpittaugood morning ironic! o/07:14
iurygregorymorning rpittau o/07:14
rpittauhey iurygregory :)07:15
jandershey rpittau o/07:25
rpittauhey janders :)07:26
jandersdo you know if Lenovo servers may need a separate vMedia license?07:37
jandershitting this: https://paste.opendev.org/show/807875/07:37
jandersI can't see vMedia specific URLs while poking around with curl (and Lenovo doco references these)07:37
janderswill go poking around in the GUI now but if you have any pointers that would be awesome07:37
rpittauI would not exclude that or some kind of difference in the path, not sure though07:38
iurygregoryjanders, are you using the latest FW?07:38
jandersiurygregory: likely not - but having said that I never flashed a Lenovo machine07:39
iurygregoryor maybe they have the resource under a different name (I don't think this would be the case)07:39
rpittauwould be worth giving a try updating the FW07:39
jandersagreed07:39
jandersI will see how easy that is07:39
opendevreviewAija Jauntēva proposed x/sushy-oem-idrac master: Add Flake8 W503 ignore  https://review.opendev.org/c/x/sushy-oem-idrac/+/80342507:39
iurygregoryjanders, yeah, the only person I know that tested vmedia with Lenovo was timeu =)07:40
rpittauon a different note, I disabled the ovn services but they get installed anyway \o/07:41
iurygregory\o/07:41
iurygregoryyay!07:41
rpittauif anyone has a moment please check https://review.opendev.org/c/openstack/ironic/+/801350 thanks!07:46
iurygregoryrpittau, looking now07:47
opendevreviewRiccardo Pittau proposed openstack/networking-generic-switch master: Fix tempest based job  https://review.opendev.org/c/openstack/networking-generic-switch/+/80332007:53
cenneGood morning ironic 08:40
cennehey iurygregory,  janders,  rpittau 08:41
rpittauhey cenne :)08:41
opendevreviewRiccardo Pittau proposed x/sushy-oem-idrac master: Increase version of hacking and pycodestyle  https://review.opendev.org/c/x/sushy-oem-idrac/+/80342808:41
iurygregoryhey cenne o/08:42
opendevreviewMerged openstack/ironic stable/victoria: Use shim-signed on Ubuntu, shim is empty now  https://review.opendev.org/c/openstack/ironic/+/80333809:04
opendevreviewMerged openstack/ironic master: Add lower-constraints job to current development branch  https://review.opendev.org/c/openstack/ironic/+/80135009:12
opendevreviewRiccardo Pittau proposed openstack/ironic-python-agent master: Add lower-constraints job to current development branch  https://review.opendev.org/c/openstack/ironic-python-agent/+/80345309:37
opendevreviewRiccardo Pittau proposed openstack/networking-generic-switch master: Fix tempest based job  https://review.opendev.org/c/openstack/networking-generic-switch/+/80332010:06
opendevreviewMerged openstack/sushy stable/wallaby: Removing optional fields from insert_media payload  https://review.opendev.org/c/openstack/sushy/+/80319710:36
janders\o/10:41
jandersdtantsur https://review.opendev.org/c/openstack/ironic/+/803327 did fix things up for my patch indeed, thank you10:42
jandersnow - trying to test out WriteProtected on Lenovo, but I'm still yet to get vMedia working10:42
jandersBIOS upgrades done (but didn't help) figuring out IMM upgrade (which is the one that should matter more I guess)10:42
dtantsurmorning/afternoon ironic10:54
dtantsurTheJulia: I'd start with slow and conservative. 1000 nodes per day does surprise me: it seems too low10:56
jandershey dtantsur o/10:56
opendevreviewDmitry Tantsur proposed openstack/ironic bugfix/18.1: Use shim-signed on Ubuntu, shim is empty now  https://review.opendev.org/c/openstack/ironic/+/80346010:59
janderstimeu in order to be able to configure vMedia via RedFish on Lenovo, did you need a separate license?11:24
jandersI've got an X3550 and it refuses to cooperate vMedia-wise. Redfish paths that are supposed to be used for vMedia config are missing. I wonder if it may be a license problem...11:25
jandersjungleboyj I heard you may also have some insights into ^11:26
jandersthanks in advance!11:26
cennehttps://review.opendev.org/c/openstack/ironic/+/80008411:35
cenneHi ironic, if you are feeling like doing a review ^ please :) 11:36
jandersrpittau from the Lenovo repo it does seem there should be a VirtualMedia key under the manager: https://github.com/lenovo/python-redfish-lenovo/blob/master/examples/lenovo_mount_virtual_media.py#L10011:37
jandersand it's not there (which is what is breaking sushy)11:37
jandersso I am more leaning towards the missing license theory11:38
jandersthank you for pointing me to this repo, it's very useful11:38
iurygregoryI'm starting to have the feeling we need a license :D11:39
iurygregoryor maybe the model doesn't support vmedia at all? .-.11:40
jandersfunny thing is there is a GUI field for it11:40
jandersheck, I will try to configure it :D11:40
iurygregoryhehehehe11:40
iurygregoryHardware World <311:40
janders"it's called hardware, cause it's HARD..."11:41
jandersand compulsory reference to https://xkcd.com/927/11:41
opendevreviewcenne proposed openstack/ironic master: Add api endpoints for changing boot_mode and secure_boot state  https://review.opendev.org/c/openstack/ironic/+/80008411:42
iurygregoryafter reading this I went back to the time I was in the university hearing computer science jokes :D11:42
iurygregoryjanders, not sure if heard this before, "How to know the problem my computer has?" "If you are yelling at your computer the problem is software, if you are hitting your computer the problem is in the hardware"11:46
jandersLOL!11:46
jandersand what if it's both?11:46
jandersa hybrid one?11:46
iurygregoryyou are doing both at the same time LOL11:46
jandersw/r/t Lenovo vMedia via GUI - it reports success11:47
iurygregory\o/11:47
jandersso my suspicion is it does require a license for vMedia config over RedFish11:47
jandersTheJulia: I think I remember you talking about something like this - have you seen this before? (it's a Lenovo x3550 M5)11:48
*** pmannidi is now known as pmannidi|Gone12:14
*** pmannidi|Gone is now known as pmannidi|AFK12:14
rpittaujanders: glad it was helpful, in some way :)12:17
jungleboyjjanders:  I don't think that the x3650s support Redfish.  I ran into that problem recently in one of my labs.  The older IMMs only support IPMI.  You need to be on a newer system with xClarity Controller (XCC) for the BMC to get Redfish support.12:24
* dtantsur suspects janders has already left for a day12:30
jandersdtantsur: I probably should have, but still hanging around :)12:41
jandersthank you jungleboyj12:41
dtantsur:D12:41
jandersI see *some* RF support, but the vMedia URLs are missing12:41
janders(e.g. I was only able to get system's serial number to get BIOS/IMM upgrades via RF queries :) )12:42
jungleboyjjanders:  Interesting.13:13
TheJuliadtantsur: it would only be what it manages though, 1000 nodes that exceed 300 messages13:21
TheJuliaalso, good morning everyone13:22
dtantsurmorning TheJulia 13:22
dtantsurhonestly, I don't have a lot of opinions on numbers13:22
TheJuliaI just kind of feel like it might be too much13:22
TheJuliabut a larger operator may have feelings on their churn13:22
dtantsurwhat's your biggest worry, locking the table?13:23
TheJuliayeah, locking writes out13:23
jungleboyjSo, I talked to my guy who knows interacting with the BMCs well and he did confirm that RedFish on those systems was not the most fully functional thing.13:24
TheJuliabut the writes will retry13:24
TheJuliathe deletes won't based upon the code, which is fine, its about eventually clean up13:24
jungleboyjI can confirm it works well on our SRxxx line of systems.  :-)13:24
dtantsurTheJulia: they you need small batches, but it matters less how many of them13:24
dtantsurs/they/then/13:24
TheJuliayeah, it *should* be lots of small batches to begin with, it does it by node13:25
* TheJulia might just be over thinking it13:25
janderssee you tomorrow Ironic o/13:26
dtantsursee you janders 13:26
dtantsurTheJulia: so, I'd drop the requirement on the nodes per day. Instead, I'd use batches of, say, 100 per run of a periodic task (so with a delay between each 100).13:26
TheJuliahmm yeah13:27
TheJuliathe by node approach makes it super convenient because we can easily extract how many13:28
TheJuliawe *could* run a query to build a list of all of the ids and just limit that at execution time13:29
TheJuliajust requires an extra db query most likely13:29
dtantsurTheJulia: the delete question should return the number of rows affected13:29
dtantsurso in theory you can proceed to the next node if your 100 are not yet used up13:29
TheJuliayeah13:30
dtantsurwhich is nice if you have many nodes with not so many events for each13:31
dtantsur(maybe have a reasonable limit for nodes per iteration too)13:31
TheJuliathat hmm, I'd have to think about how to make the delete queries to keep the number of records under the threshold properly if I'm not pre-determining how many rows to delete for each node based on it's overage13:32
dtantsurTheJulia: I think we have more or less this code for online migrations13:33
TheJuliathat just uses a limit13:33
dtantsurare there any issues with a limit?13:33
TheJuliathe online migrations also works through everything, which is a thing I think we need to avoid because this table could become huge in some of the operators. My thought here was more tactically identify and delete where applicable13:34
dtantsurit may be faster to do it on sql level though13:35
TheJuliawhich is what I did13:35
TheJuliajust in two steps13:35
dtantsurI think we also do it in two steps for online migrations, I don't remember why13:35
TheJuliaI'd need to go look at the migration code again and have coffee and not have neck pain13:36
dtantsur:(13:36
dtantsurat least fix the coffee13:36
opendevreviewZhouHao proposed openstack/ironic master: update irmc document  https://review.opendev.org/c/openstack/ironic/+/80341213:55
TheJuliaAnd my router has died... Like sparks inside the case died14:15
rpittauoO14:16
dtantsurOo14:23
iurygregorywow =O14:33
JayFAt least you know it's bad when it puts up the magic smoke14:34
JayFas opposed to dropping every 64th packet or w/e14:35
iurygregorythis one is a bit more hard to debug :D ^14:35
opendevreviewMerged openstack/ironic bugfix/18.1: Use shim-signed on Ubuntu, shim is empty now  https://review.opendev.org/c/openstack/ironic/+/80346014:48
dtantsurI'm pondering a mode of deployment where all database access goes through a conductor. Thoughts?14:49
TheJuliaThat would be a huge bottleneck14:51
TheJuliaWe, at a minimum, need to support read slave connections because many more conductors would be needed14:52
TheJuliaWe also don't have the methods RPC remotes, but I *think* it is just a decorator14:52
dtantsurI'm interested in standalone case mostly14:53
dtantsurso, I don't think it should be the default mode of operation14:53
dtantsurbut it may be interesting in two cases (both coming from metal3 background - disclosure):14:53
dtantsur1) Ephemeral database. We'd like to use sqlite, but accessing one database from two processes is a recipe for problems (someone is trying it in production as we speak - fingers crossed for them)14:54
TheJuliaOkay, I do think it makes some sense as long as it is not overly complex to leverage. May just be a this or that logic thing that is needed14:54
dtantsur2) Very distributed cluster. Essentially make database local to the conductor group, while the control plane is stateless (from the ironic pov)14:54
TheJuliaThose both make lots of sense14:54
TheJuliaI still think we need reader connections :). For history read queries that would be perfect.14:55
dtantsurcould you elaborate on reader connections?14:55
TheJuliaSo oslo.db has wiring for a database replication slave target connection for reads but it is not exposed14:56
dtantsurthis seems somewhat orthogonal to my proposal?14:56
TheJuliaYes it is orthogonal14:57
dtantsurah, gotcha14:57
TheJuliaI was thinking it because you'd need more consuctors14:57
TheJuliaBut in metal3 or small scale standalone it shouldn't be an issue14:57
dtantsuryeah, we need read connections, I agree. I'm not sure we even update anything from the API layer any more.14:57
dtantsurmmm, yeah, we patch chassis and probably others14:58
dtantsurand allocations14:59
* dtantsur needs to stop planning future features and concentrate on the presentation he does tomorrow15:00
TheJulia++15:02
* TheJulia needs to check store inventory for routers15:03
dtantsurI've just realized that my perfect plan doesn't account for hash rings...15:25
NobodyCamGood Morning Ironic folks.... Happy hump day15:44
rpittauwell looks like I fixed at the 4th tentative https://review.opendev.org/c/openstack/networking-generic-switch/+/80332016:00
opendevreviewRiccardo Pittau proposed openstack/networking-generic-switch master: Increase version of hacking and pycodestyle  https://review.opendev.org/c/openstack/networking-generic-switch/+/80321316:00
dtantsurrpittau: nice job figuring it out!16:02
rpittauthanks :)16:02
* dtantsur is on slide 23, and the end is not near16:02
rpittaugood night! o/16:14
*** rpittau is now known as rpittau|afk16:14
dtantsuro/17:18
JayFo/17:22
*** ricolin_ is now known as ricolin18:02
opendevreviewEric Barrera proposed x/sushy-oem-idrac stable/wallaby: Enable coverage HTML output  https://review.opendev.org/c/x/sushy-oem-idrac/+/80344718:46
opendevreviewEric Barrera proposed x/sushy-oem-idrac stable/victoria: Enable coverage HTML output  https://review.opendev.org/c/x/sushy-oem-idrac/+/80344818:47
opendevreviewDanni Shi proposed openstack/ironic-python-agent master: Add an attestation extension  https://review.opendev.org/c/openstack/ironic-python-agent/+/80351019:37
NobodyCamcrazy question does anyone happen to have a check to ensure that a ironic node is able to be provisioned through nova.. 19:37
* TheJulia screams into the technology void19:37
NobodyCamLOL19:37
TheJuliaable to be provisioned? I mean, nova's classifer is is available and not in maintenance state19:37
TheJulianova does call validate before hand, but that just validates the information supplied, it typically becomes a gigo problem at that point19:38
TheJuliaunless there are other known things like if cleaning is disabled/bypassed and that always fails19:38
NobodyCamand placement record is really updated19:39
NobodyCaminventory has been recorded as available 19:39
opendevreviewBob Fournier proposed openstack/python-ironicclient master: Include BIOS registry fields in bios setting list command  https://review.opendev.org/c/openstack/python-ironicclient/+/80333219:44
TheJuliaI think your hitting upon a gap in the mechanics of nova and nova-compute as it relates to using baremetal20:15
stevebakermorning20:21
JayFNobodyCam: TheJulia: TBH, I see that a little like asking libvirt if it has VMs available to provision according to nova's criterion... we just don't have that data at the ironic layer20:23
TheJuliawell, we have no insight into nova's inner processes20:24
JayFI mean, just by design we don't know what nova has configured, for instance for flavors20:24
JayFwe just present resources and it's up to nova to categorize and filter them20:24
JayFI think it's a proper antifeature (e.g. a good separation to have) that you can't ask Ironic the question NobodyCam wants to ask it :|20:25
NobodyCam:)20:34
TheJuliaOkay. New mesh base station ordered20:35
TheJuliarouter... seeming to be okay20:35
TheJulianew router, total garbage20:35
TheJuliaHows your day?20:35
NobodyCamplacement-y heheheh20:35
JayFWhat brand did you get TheJulia? 20:37
TheJuliafor which part :) We keep a very complex network here :)20:38
JayFI've had orbi (like the 1st or 2nd gen one), and Eero is what I use now20:38
JayFfor AP / mesh wifi20:38
JayFI run my own router but still use an off-the-shelf mesh wifi for my APs20:38
TheJuliaWe have the first generation orbi20:38
TheJuliaSame20:38
JayFand you... like it?20:38
JayFmaybe just as an AP it's less terrible20:38
TheJuliaas just an AP, they rock20:38
JayFif I enabled v6 on that, when using it as router+ap, it just absolutely was terrible20:39
TheJuliayeah, found the same thing with the repalcement router, it couldn't grok v6 properly and wasn't discovering it on the ISP network20:39
JayFI have an embedded linux box as my router20:40
TheJuliaso back to the Linksys ?EA8500? with openwrt router using its antennas which have poor coverage/performance on the far side of the house20:40
JayFrunning gentoo linux :) 20:40
JayFit's fun to do a `ps aux` on your router and have it fit on one screen20:40
TheJulia++20:40
JayFit did take me hours and a ritual sacrifice to get DHCPv6 delegation working in systemd-network lol20:40
TheJulialol20:41
TheJuliastevebaker: o/20:41
stevebakertoday I will learn about postgresql authentication20:43
TheJuliaheh20:43
TheJuliaenjoy?20:43
stevebakerit'll just be running bits of test-setup.sh manually and figuring out why it isn't working https://review.opendev.org/c/openstack/ironic/+/80312820:50
stevebakerTheJulia: what do our jobs actually use postgres for?20:50
TheJuliastevebaker: unit tests and we have one integration test20:50
TheJuliawe're one of the *few* projects taht run it afaik20:50
TheJuliaso are the VMs even rebooting?20:50
opendevreviewSteve Baker proposed openstack/ironic master: Set postgress password encryption for FIPS compliance  https://review.opendev.org/c/openstack/ironic/+/80312820:52
opendevreviewSteve Baker proposed openstack/ironic master: DNM/WIP - Add FIPS jobs  https://review.opendev.org/c/openstack/ironic/+/79773920:52
stevebakerTheJulia: I haven't got to that yet, but I've touched base with ade in #tripleo. I see keystone have a working job which runs the enable-fips role, so it works for them20:53
TheJuliaweird...20:57
opendevreviewLeo McGann proposed openstack/ironic master: Keylime attestation interface  https://review.opendev.org/c/openstack/ironic/+/80351721:04
opendevreviewSteve Baker proposed openstack/ironic master: Set postgress password encryption for FIPS compliance  https://review.opendev.org/c/openstack/ironic/+/80312821:28
opendevreviewSteve Baker proposed openstack/ironic master: DNM/WIP - Add FIPS jobs  https://review.opendev.org/c/openstack/ironic/+/79773921:28
opendevreviewMerged openstack/networking-generic-switch master: Fix tempest based job  https://review.opendev.org/c/openstack/networking-generic-switch/+/80332022:53
stevebakerTheJulia: the reboot is no longer causing an issue for the enable-fips job, its now failing in install_apache_uwsgi23:02
TheJuliafreaky23:04
*** pmannidi|AFK is now known as pmannidi23:25
stevebakerI've done no commits to devstack for 6 years, and now 2. TWO23:46
opendevreviewSteve Baker proposed openstack/ironic master: Set postgresql password encryption for FIPS compliance  https://review.opendev.org/c/openstack/ironic/+/80312823:47
opendevreviewSteve Baker proposed openstack/ironic master: DNM/WIP - Add FIPS jobs  https://review.opendev.org/c/openstack/ironic/+/79773923:47
stevebakerTheJulia: the postgres change is happy now23:48

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!