Wednesday, 2020-03-11

*** erbarr has quit IRC00:01
*** ianychoi has quit IRC00:07
*** rloo has quit IRC00:07
*** Goneri has quit IRC00:08
*** ianychoi has joined #openstack-ironic00:08
*** diga has joined #openstack-ironic00:10
*** igordc has joined #openstack-ironic00:12
*** gyee has quit IRC00:13
*** aedc_ has joined #openstack-ironic00:50
*** aedc has quit IRC00:52
*** iurygregory has quit IRC01:17
*** dmellado has quit IRC01:22
*** TxGirlGeek has quit IRC01:27
*** ianychoi has quit IRC01:30
*** ianychoi has joined #openstack-ironic01:32
*** dmellado has joined #openstack-ironic01:46
*** TxGirlGeek has joined #openstack-ironic01:56
*** ianychoi has quit IRC02:10
*** ianychoi has joined #openstack-ironic02:18
*** k_mouza has joined #openstack-ironic02:24
*** jdandrea has quit IRC02:28
*** k_mouza has quit IRC02:29
*** diurnalist has joined #openstack-ironic02:32
*** diurnalist has quit IRC02:52
*** mkrai has joined #openstack-ironic02:57
*** ricolin_ has joined #openstack-ironic03:01
*** TxGirlGeek has quit IRC03:03
*** igordc has quit IRC03:30
*** ianychoi has quit IRC03:35
*** ianychoi has joined #openstack-ironic03:37
*** ricolin_ has quit IRC03:38
*** diga has quit IRC03:47
*** tkaprol has joined #openstack-ironic03:53
*** tkaprol has quit IRC04:01
*** tzumainn has quit IRC04:22
*** ricolin has quit IRC04:44
*** diurnalist has joined #openstack-ironic04:49
*** diurnalist has quit IRC04:54
*** ianychoi has quit IRC05:00
*** ianychoi has joined #openstack-ironic05:03
*** ricolin has joined #openstack-ironic05:51
*** rcernin has quit IRC05:58
*** aedc_ has quit IRC06:01
*** ianychoi has quit IRC06:01
*** aedc has joined #openstack-ironic06:02
*** ianychoi has joined #openstack-ironic06:03
*** ianychoi has quit IRC06:11
*** ianychoi has joined #openstack-ironic06:13
*** ianychoi has quit IRC06:29
*** ianychoi has joined #openstack-ironic06:31
*** ianychoi has quit IRC06:39
*** ianychoi has joined #openstack-ironic06:41
*** ianychoi has quit IRC06:58
*** ianychoi has joined #openstack-ironic07:05
*** iurygregory has joined #openstack-ironic07:29
iurygregorygood morning Ironic07:29
arne_wiebalckGood morning iurygregory and ironic!07:37
* etingof has never encountered redfish-over-serial so far... is it like over a SLIP connection?07:42
etingofgood morning, ironic o/07:42
*** aedc has quit IRC07:47
*** tesseract has joined #openstack-ironic08:02
*** sziviani_ has joined #openstack-ironic08:10
*** tkajinam has quit IRC08:14
*** mkrai has quit IRC08:18
*** mkrai has joined #openstack-ironic08:25
*** k_mouza has joined #openstack-ironic08:25
*** k_mouza has quit IRC08:29
*** amoralej|off is now known as amoralej08:31
*** gkadam has joined #openstack-ironic08:34
*** gkadam has quit IRC08:34
*** sshnaidm has joined #openstack-ironic08:41
sshnaidmhi, all, I have a question - does "node provide" always include a cleaning?08:42
*** mgoddard has quit IRC08:49
*** mgoddard has joined #openstack-ironic08:50
iurygregorysshnaidm, you mean you need to run cleaning before runing openstack baremetal node provide or that the command will also execute cleaning for you?08:52
sshnaidmiurygregory, currently when I do "node provide" it also does cleaning08:52
sshnaidmiurygregory, I'm curious if it's mandatory step?08:53
iurygregorysshnaidm, probably you have auto cleaning enable I would say08:53
sshnaidmiurygregory, oh, I see08:53
iurygregoryhttps://docs.openstack.org/ironic/latest/admin/cleaning.html#automated-cleaning08:53
iurygregorysshnaidm, np ^^08:54
sshnaidmiurygregory, so it's only from ironic config, and can't be configured in client, right?08:54
*** chihhsin_ has quit IRC08:54
*** ntt has joined #openstack-ironic08:54
iurygregorysshnaidm, you mean to turn off cleaning?08:54
iurygregoryturn off automated cleaning*08:55
sshnaidmiurygregory, yeah08:55
sshnaidmwhen I do request with client for "node provide" or SDK08:55
*** chihhsin has joined #openstack-ironic08:55
iurygregoryyou can still have enable and change the configs08:56
*** dtantsur|afk is now known as dtantsur08:56
dtantsurmorning ironic08:56
iurygregoryyou can set priority for the cleaning steps I would say08:56
iurygregorymorning dtantsur08:56
sshnaidmiurygregory, I mean, when I have autoclean=true, and I do "node provide" - is there any option to override "autoclean" in a client?08:57
sshnaidmand not to clean in this case08:57
dtantsursshnaidm: we deliberately did not provide such08:57
iurygregorysshnaidm, I don't think so =(08:57
sshnaidmdtantsur, iurygregory ack, thanks08:57
dtantsursshnaidm: disabling cleaning requires careful thinking. it is known to cause issues08:57
dtantsurwe leave it up to an operator, not to an API user.08:57
*** rpittau|afk is now known as rpittau08:58
rpittaugood morning ironic! o/08:58
sshnaidmdtantsur, ack08:58
iurygregorythis is a miracle08:58
iurygregoryrpittau, you are late today hehe08:58
iurygregoryconsidering that dtantsur said morning first08:59
dtantsurheh08:59
dtantsurI come later when I have a German class in the morning (or when it's Monday)08:59
iurygregory++ for Monday08:59
iurygregoryfunny thing not even people from qa were able to figure out our problem with grenade in stable/stein .-.09:02
dtantsursigh09:06
dtantsurgrenade, tempest, devstack.. they all are ridiculous to my taste. once something goes sideways, you're lost in the darkness.09:07
dtantsurand all people who understood grenade have left already iirc09:07
dtantsur</rant>09:07
iurygregorydamm D:09:08
rpittau<rant> they are truly complex, I think the worse thing is that sometimes we struggle to make ci works even if real-life tests work </rant>09:08
*** lucasagomes has joined #openstack-ironic09:10
nttHi, I'd like to know if I can use ironic to install centos 7 iso on baremetal servers. Is there some guide for this? I don't know how to add centos7 image to ironic and then deploy on servers. The inspection part already works. Thank you09:12
iurygregoryyou can09:12
iurygregorylet me see if I can find the link09:12
iurygregoryyou can use diskimage builder to create your centos7 image09:13
*** belmoreira has joined #openstack-ironic09:13
iurygregorybut you can't install the newest package since  centos7 is deprecated in favor of centos 809:14
*** dougsz has joined #openstack-ironic09:15
iurygregoryhttps://docs.openstack.org/diskimage-builder/latest/09:15
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Use openstacksdk for ironic module  https://review.opendev.org/67217909:16
*** ociuhandu has joined #openstack-ironic09:16
dtantsuriurygregory: centos7 as a user image works fine, the problems are with IPA09:24
dtantsurntt: do you use nova or only ironic? do you use bifrost?09:24
iurygregorydtantsur, oh .-.09:24
*** k_mouza has joined #openstack-ironic09:24
dtantsurfolks, especially those not from Red Hat, this is the last call to review https://review.opendev.org/#/c/672780/09:24
patchbotpatch 672780 - ironic-specs - Update L3 based deployment spec - 36 patch sets09:24
dtantsurit will probably be approved by EOD09:25
nttdtantsur, I installed an undercloud but I can try other solutions in order to use ironic as a standalone service. Anyway, I don't know what bifrost is09:25
dtantsurntt: undercloud, okay. so you have glance and nova and everything09:25
dtantsurntt: bifrost is https://docs.openstack.org/bifrost/latest/09:25
dtantsurdepending on your case it may fit better or worse09:25
*** uzumaki has joined #openstack-ironic09:26
dtantsurntt: you can get information on glance images from https://docs.openstack.org/ironic/latest/install/configure-glance-images.html (and glance docs, of course)09:26
uzumakihey guys! quick question, what exactly is the difference between cleaning and deploying states? If both states have same interfaces attached to the node, support similar operations09:30
dtantsuruzumaki: cleaning process prepares a node for entering the cloud. wipes disks, prepares RAID (optionally), etc09:32
dtantsuruzumaki: https://docs.openstack.org/ironic/latest/admin/cleaning.html is an overview09:32
arne_wiebalckdtantsur: etingof: re the L3 spec, just to be sure, this is an alternative to the current L2 based deployments, not a replacement, right?09:33
dtantsuralternative, yes09:33
arne_wiebalckdtantsur: etingof: the other point I have is the one I made at the mid cycle about the scalability when per node images have to be created for many nodes09:34
dtantsuryeah. I think etingof was going to research using additional virtual media slots for configuration.09:34
dtantsurI'm not sure that it's a blocker here, given that the feature is opt-in.09:35
arne_wiebalckdtantsur: I agree, it's more that I would not want to be excluded as I like the feature! :)09:36
dtantsursure :)09:36
dtantsuretingof: this ^^ may justify a follow-up to the follow-up09:36
arne_wiebalckdtantsur: thanks!09:36
*** ociuhandu has quit IRC09:38
etingofI do not see any technical problems with putting config drive on virtual anything (floppy, usb), not sure it this needs any research09:38
dtantsurarne_wiebalck: this whole conversation is actually reflected in the midcycle notes that we still have to finish and send..09:38
dtantsuretingof: not research, but it may need additional ramdisk-side changes09:38
dtantsuraround the logic to find the configdrive09:38
uzumakidtantsur, what's a fake cleaning step?09:39
etingofhopefully, if Glean behaves in the same way as cloud-init i.e. searches all removable media with config-2 label, that should work automatically on ramdisk09:39
dtantsuruzumaki: you mean https://docs.openstack.org/ironic/latest/admin/cleaning.html#id8 ? it's a bad wording for clean steps coming from the 'fake' driver.09:39
* dtantsur wonders if we should exclude them09:40
uzumakiYeah that. I see. And what exactly is the purpose of the fake driver? testing?09:40
etingofwe kind of have virtual floppy support in ironic09:40
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Fix and add logs and info  https://review.opendev.org/70716209:40
etingofin that light, I do not expect major difficulties with pushing config-drive to virtual anything...09:41
dtantsuruzumaki: exactly09:42
openstackgerritDmitry Tantsur proposed openstack/ironic master: Skip clean steps from 'fake' interfaces in the documentation  https://review.opendev.org/71229709:42
dtantsur^^^09:42
uzumakidtantsur, ah! I see. Thanks!09:42
*** derekh has joined #openstack-ironic09:42
uzumakiAnd what's deploy steps? It's the same interfaces, but the things you do with them are different?09:42
dtantsuretingof: I mean, the ramdisk side09:43
dtantsuruzumaki: deploy steps are (in-progress) way of splitting and customizing the deploy process09:43
dtantsurthey're based on a similar mechanism, and the same steps can work as clean and deploy steps09:43
dtantsur(like with RAID)09:43
uzumakiI know that. That's what makes me think what's different in deploy steps, and if we have those, why do we need clean steps?09:44
dtantsuruzumaki: clean steps happen in advance and after tear down and can cover long-running processes09:44
dtantsurthey can also include things that don't change from deployment to deployment09:44
dtantsurdeploy steps should only include operations specific to each deployment09:44
uzumakiI see I see. Then it makes sense.09:45
etingofhmm, by saying Glean I meant ramdisk side. is there anything else on ramdisk that should be made aware of network_data.json location?09:45
dtantsuretingof: does glean currently support USB devices as a source?09:46
etingoflet me google it up09:46
dtantsurit's probably a small change, if any09:47
dtantsurjust let not forget about high-scale deployments like CERN :)09:47
dtantsurI think metal3 folks also voiced concerns about rebuilding ISO every time09:47
etingofyeah, we are operating on the assumption that Glean is easy to tackle anyway09:48
etingofthough my patch against it seems forgotten long ago09:49
*** khansa has joined #openstack-ironic09:49
etingofhere is what it says in README: "Specifically, it will mount the special block-device with label config-2 and parse the network_info.json and vendor_info.json files within."09:49
*** ociuhandu has joined #openstack-ironic09:50
etingofin the code it searches block device by label -- https://opendev.org/opendev/glean/src/branch/master/glean/init/glean.sh#L3709:52
sshnaidmI'm trying to "provide" nodes by name and it seems not working: conn.baremetal.set_node_provision_state("baremetal-1003-2-44884-0", "provide", wait=True)09:57
sshnaidmonly by UUIDs09:57
sshnaidmis it as designed..?09:57
sshnaidmnode cleaning call works both with uuid and names09:57
dtantsuretingof: lovely, then any devices should work09:58
etingofthat's what I assume09:58
dtantsursshnaidm: mmm, it may be a limitation of microversion handling. lemme check the sdk.09:58
dtantsuryeah, need fixing09:59
sshnaidmdtantsur, ok10:01
*** khansa has quit IRC10:01
*** alexmcleod has joined #openstack-ironic10:02
*** mkrai_ has joined #openstack-ironic10:05
*** mkrai has quit IRC10:08
openstackgerritRiccardo Pittau proposed openstack/bifrost master: [WIP] Fix dhcp job  https://review.opendev.org/71230410:10
*** Blinkiz has quit IRC10:15
*** Blinkiz has joined #openstack-ironic10:16
-openstackstatus- NOTICE: The mail server for lists.openstack.org is currently not handling emails. The infra team will investigate and fix during US morning.10:26
openstackgerritMark Goddard proposed openstack/bifrost master: Fix Ubuntu keystone deployment  https://review.opendev.org/71211210:27
openstackgerritDmitry Tantsur proposed openstack/ironic master: Skip clean steps from 'fake' interfaces in the documentation  https://review.opendev.org/71229710:39
*** rh-jelabarre has joined #openstack-ironic10:50
openstackgerritMark Goddard proposed openstack/tenks master: Pin virtualbmc to <2 when using Python 2  https://review.opendev.org/71233910:51
*** Lucas_Gray has joined #openstack-ironic10:52
openstackgerritMark Goddard proposed openstack/tenks master: CI: Switch to kayobe-overcloud-centos8  https://review.opendev.org/71213611:01
*** priteau has joined #openstack-ironic11:02
*** Lucas_Gray has quit IRC11:03
*** hjensas has quit IRC11:06
rpittauyay encoders "stderr": "/usr/bin/env: ‘python’: No such file or directory\n"11:07
*** Lucas_Gray has joined #openstack-ironic11:08
dtantsurlol11:08
etingofpython is priceless11:12
etingofor does it bring a lot of Euros? not sure11:12
openstackgerritIlya Etingof proposed openstack/sushy-cli master: Add `--insecure` and `--tls-certificates` options  https://review.opendev.org/71205811:12
iurygregoryit brings pain XD11:12
*** Lucas_Gray has quit IRC11:13
dtantsurmgoddard: hi! did you have a chance to request an ironic-lib release?11:13
openstackgerritMark Goddard proposed openstack/bifrost master: Fix Ubuntu keystone deployment  https://review.opendev.org/71211211:14
mgoddarddtantsur: no, but will do so now11:14
dtantsurthanks11:14
*** Lucas_Gray has joined #openstack-ironic11:15
openstackgerritRiccardo Pittau proposed openstack/bifrost master: [WIP] Fix dhcp job  https://review.opendev.org/71230411:15
*** rh-jelabarre has quit IRC11:18
*** hjensas has joined #openstack-ironic11:22
*** ianychoi has quit IRC11:24
openstackgerritMerged openstack/sushy-tools master: Always log via Flask application  https://review.opendev.org/69363911:25
*** diurnalist has joined #openstack-ironic11:26
*** diurnalist has quit IRC11:31
*** Wryhder has joined #openstack-ironic11:33
*** Lucas_Gray has quit IRC11:34
*** aedc has joined #openstack-ironic11:34
*** Wryhder is now known as Lucas_Gray11:34
*** uzumaki has quit IRC11:35
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Switch to stestr  https://review.opendev.org/71234411:42
*** rpittau is now known as rpittau|bbl11:58
*** Goneri has joined #openstack-ironic11:58
* rpittau|bbl going on an adventure11:59
*** ntt has quit IRC11:59
*** aedc has quit IRC11:59
iurygregorygood luck rpittau|bbl12:04
*** sshnaidm is now known as sshnaidm|afk12:10
*** ociuhandu has quit IRC12:15
openstackgerritMark Goddard proposed openstack/tenks master: Pin virtualbmc to <2 when using Python 2  https://review.opendev.org/71233912:21
*** ociuhandu has joined #openstack-ironic12:27
*** rh-jelabarre has joined #openstack-ironic12:28
*** ociuhandu has quit IRC12:29
*** ociuhandu has joined #openstack-ironic12:30
*** belmoreira has quit IRC12:31
*** ociuhandu_ has joined #openstack-ironic12:32
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: [WIP] Split ironic-standalone  https://review.opendev.org/71204212:33
*** ociuhandu has quit IRC12:35
*** mkrai_ has quit IRC12:42
*** Lucas_Gray has quit IRC12:47
*** Lucas_Gray has joined #openstack-ironic12:48
*** aedc has joined #openstack-ironic12:52
*** Lucas_Gray has quit IRC12:54
*** rloo has joined #openstack-ironic12:55
*** jdandrea has joined #openstack-ironic13:00
*** Lucas_Gray has joined #openstack-ironic13:02
openstackgerritIlya Etingof proposed openstack/sushy-cli master: Add `--insecure` and `--tls-certificates` options  https://review.opendev.org/71205813:05
*** aedc has quit IRC13:14
*** ociuhandu_ has quit IRC13:15
openstackgerritMark Goddard proposed openstack/bifrost master: Fix Ubuntu keystone deployment  https://review.opendev.org/71211213:16
*** ociuhandu has joined #openstack-ironic13:17
*** irclogbot_1 has quit IRC13:22
*** irclogbot_2 has joined #openstack-ironic13:23
openstackgerritMark Goddard proposed openstack/tenks master: Pin virtualbmc to <2 when using Python 2  https://review.opendev.org/71233913:24
*** Lucas_Gray has quit IRC13:30
*** Lucas_Gray has joined #openstack-ironic13:32
*** tzumainn has joined #openstack-ironic13:35
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: In-band deploy steps  https://review.opendev.org/69876813:35
openstackgerritDmitry Tantsur proposed openstack/ironic master: WIP: Decompose core deploy step  https://review.opendev.org/69830913:35
*** amoralej is now known as amoralej|lunch13:40
*** ociuhandu has quit IRC13:41
*** ociuhandu has joined #openstack-ironic13:43
*** rh-jelabarre has quit IRC13:43
*** rh-jelabarre has joined #openstack-ironic13:44
*** khansa has joined #openstack-ironic13:48
*** ociuhandu has quit IRC13:48
*** zigo has quit IRC13:49
openstackgerritIlya Etingof proposed openstack/sushy-cli master: Add `--insecure` and `--tls-certificates` options  https://review.opendev.org/71205814:02
openstackgerritIlya Etingof proposed openstack/sushy-cli master: Add `--insecure` and `--tls-certificates` options  https://review.opendev.org/71205814:12
openstackgerritIlya Etingof proposed openstack/sushy-cli master: Add non-default Redfish service mount location  https://review.opendev.org/71244714:12
*** amoralej|lunch is now known as amoralej14:14
*** TxGirlGeek has joined #openstack-ironic14:17
TheJuliagood mroning14:18
iurygregorygood morning TheJulia14:19
dtantsurTheJulia: morning!14:20
dtantsurhow do you feel today?14:20
openstackgerritMark Goddard proposed openstack/tenks master: Pin virtualbmc to <2 when using Python 2  https://review.opendev.org/71233914:22
*** ociuhandu has joined #openstack-ironic14:22
*** ociuhandu has quit IRC14:27
TheJuliaGreetings dtantsur, fairly human this morning14:32
dtantsurgreat :)14:32
*** ociuhandu has joined #openstack-ironic14:37
*** sshnaidm|afk is now known as sshnaidm14:37
openstackgerritMark Goddard proposed openstack/bifrost master: Fix Ubuntu keystone deployment  https://review.opendev.org/71211214:41
*** uzumaki has joined #openstack-ironic14:42
iurygregoryTheJulia, I was talking with some folks about modules in python that are in compliance with FIPS, they told me would be better avoid random and use https://docs.python.org/3/library/secrets.html#module-secrets wdyt? (Asking because this would apply to your agent token patches )14:42
TheJuliaseems odd to avoid the rng14:46
TheJuliaon a plus side, they have a single method to do it14:47
TheJuliatoken_urlsafe(128) would do the job14:47
*** sziviani_ has quit IRC14:47
iurygregoryyeah14:48
dtantsurmmm, yeah, it's handy14:48
dtantsurnote that it will be longer than 12814:49
dtantsurTheJulia: maybe it's the reason to drop the requirement on 128 symbols on the IPA side?14:49
*** diurnalist has joined #openstack-ironic14:50
*** mkrai_ has joined #openstack-ironic14:50
dtantsurthe default length on my laptop seems to be 43 symbols from token_urlsafe14:50
iurygregorydtantsur, same for me14:52
iurygregoryand well if we need more than 128 we can have it, the downside is that is a new requirement (for the greater good)14:53
*** diurnalist has quit IRC14:54
TheJuliayeah, I guess, I just wanted something suitably long to be super hard for someone to just or type out from seeing a screen14:56
* dtantsur cannot quickly remember 43 random letters14:56
TheJulia>32 chars work for everyone?14:56
dtantsuryeah14:57
TheJuliaI'll update the ipa side in a minute, need to also jump on a call. I would <3 a couple of reviews quickly though14:57
dtantsura good thing: if 32 is ever deemed not enough, we'll get more with a python update, no actions on our side14:57
dtantsurreviews is something I can do today (although I'm planning on a short walk in the near future)14:58
TheJuliaheh14:58
iurygregoryDo you want me to put as comment in the patches? =)14:58
*** Lucas_Gray has quit IRC14:59
*** Wryhder has joined #openstack-ironic14:59
*** Wryhder is now known as Lucas_Gray15:00
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Agent token support  https://review.opendev.org/69261415:02
TheJuliadone15:02
TheJuliaiurygregory: which patches?15:02
iurygregoryin ironic for the token15:02
TheJuliaiurygregory: just propose a new patch, the first patch already merged15:03
*** bdodd has joined #openstack-ironic15:03
iurygregoryTheJulia, I will do15:03
*** Lucas_Gray has quit IRC15:04
*** Lucas_Gray has joined #openstack-ironic15:08
openstackgerritVerification of a change to openstack/bifrost failed: Make centos8 job vote again  https://review.opendev.org/71141615:11
openstackgerritVerification of a change to openstack/bifrost failed: Fix and add logs and info  https://review.opendev.org/70716215:11
*** diurnalist has joined #openstack-ironic15:19
openstackgerritMerged openstack/ironic master: Follow up to console port allocation  https://review.opendev.org/71180215:29
iurygregoryError: Failed to download metadata for repo 'AppStream' -.-' CI you are better than this we know...15:33
TheJulia:(15:33
*** gyee has joined #openstack-ironic15:47
*** zigo has joined #openstack-ironic15:52
*** dtantsur is now known as dtantsur|brb15:53
*** TxGirlGeek has quit IRC15:55
*** TxGirlGeek has joined #openstack-ironic15:55
openstackgerritJulia Kreger proposed openstack/ironic master: Make image_checksum optional if other checksum is present  https://review.opendev.org/71181615:55
openstackgerritMerged openstack/ironic stable/train: Make reservation checks caseless  https://review.opendev.org/71206415:59
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Log the agent version  https://review.opendev.org/71199515:59
openstackgerritMark Goddard proposed openstack/bifrost stable/train: Remove ip kernel command line argument  https://review.opendev.org/71249116:18
openstackgerritMark Goddard proposed openstack/bifrost stable/stein: Remove ip kernel command line argument  https://review.opendev.org/71249216:18
arne_wiebalckWe're planning to move our setup to conductor groups. The plan is to a) add a conductor group A setting to all conductors, b) add the group A to all nodes, c) reconfigure our single n-compute to use this group A, d) add new nova-c's with groups B, C, .. e) partition the nodes in the park to B, C, ... does this sound ok?16:19
openstackgerritMark Goddard proposed openstack/bifrost stable/rocky: Remove ip kernel command line argument  https://review.opendev.org/71249316:20
arne_wiebalckThe main point is to not confuse nova-c which may happen if they run alongside nova-c's which do not have a group configured.16:20
*** belmoreira has joined #openstack-ironic16:21
*** mkrai_ has quit IRC16:22
*** ricolin has quit IRC16:25
TheJuliaiurygregory: you may want to take a look at https://review.opendev.org/#/c/696156/7 Possible another path for us to see if we can hunt down a uefi bootloader, but I think we need to wait until after it merges. And it may not be possible, but it is a thought16:25
patchbotpatch 696156 - ironic-python-agent - Software RAID: Add UEFI support - 7 patch sets16:25
TheJuliaarne_wiebalck: I think that should work16:28
iurygregoryTheJulia, I was looking =)16:28
arne_wiebalckTheJulia: we tried this with a few nodes (<10), next stop is 5000 nodes, so I thought we better check :)16:29
arne_wiebalckTheJulia: at least if there is sth obviously wrong16:30
*** khansa has quit IRC16:32
*** jcoufal has joined #openstack-ironic16:37
openstackgerritMark Goddard proposed openstack/tenks master: Pin virtualbmc to <2 when using Python 2  https://review.opendev.org/71233916:38
*** belmoreira has quit IRC16:39
*** rpittau|bbl is now known as rpittau16:43
openstackgerritMerged openstack/bifrost master: Make centos8 job vote again  https://review.opendev.org/71141616:45
rpittau\o/16:47
iurygregoryI have powers on recheck hehehe16:47
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Switch to stestr  https://review.opendev.org/71234416:52
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Switch to stestr  https://review.opendev.org/71234416:52
uzumakiGuys, quick question. When configuring BIOS with clean steps, the settings parameter we provide, it is a list of k/v pairs which are strings. Are the names of these configs standardized somehow?16:54
openstackgerritMark Goddard proposed openstack/tenks master: Bump ansible version constraints to >=2.7,<2.10  https://review.opendev.org/71010016:55
openstackgerritMark Goddard proposed openstack/tenks master: CentOS 8 support  https://review.opendev.org/69588116:55
uzumakiI mean, is it okay to expose a drac-specific param name like "ProcVirtualization" to the settings list in clean steps? Or does it need to be masked behind another generic name?16:56
openstackgerritIlya Etingof proposed openstack/sushy-cli master: Add `--insecure` and `--tls-certificates` options  https://review.opendev.org/71205816:58
*** ricolin has joined #openstack-ironic17:01
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Fix dhcp job  https://review.opendev.org/71230417:04
*** lucasagomes has quit IRC17:04
openstackgerritIlya Etingof proposed openstack/sushy-cli master: Add non-default Redfish service mount location  https://review.opendev.org/71244717:05
openstackgerritIlya Etingof proposed openstack/sushy-cli master: Add manager vmedia command  https://review.opendev.org/70889017:05
openstackgerritMark Goddard proposed openstack/tenks master: CentOS 8 support  https://review.opendev.org/69588117:05
*** rh-jelabarre has quit IRC17:05
openstackgerritIlya Etingof proposed openstack/sushy-cli master: Add system BIOS management command  https://review.opendev.org/71154117:05
openstackgerritMerged openstack/tenks master: Pin virtualbmc to <2 when using Python 2  https://review.opendev.org/71233917:07
*** ociuhandu has quit IRC17:07
arne_wiebalckbye everyone o/17:07
*** rh-jelabarre has joined #openstack-ironic17:08
iurygregoryuzumaki, I think it's fine see https://docs.openstack.org/ironic/latest/admin/bios.html#apply-bios-configuration17:09
rpittaupatch to have dib as default image for tests is green! :O https://review.opendev.org/71027817:09
patchbotpatch 710278 - ironic - Default IRONIC_RAMDISK_TYPE to dib - 16 patch sets17:09
iurygregorybye arne_wiebalck17:09
uzumakiiurygregory, Yeah I was looking at just that. I think exposing the config names is okay. Thanks!17:10
iurygregoryrpittau, "greenish"17:10
iurygregoryXD17:10
openstackgerritGhanshyam Mann proposed openstack/ironic stable/train: DNM: testing grenade job  https://review.opendev.org/71251017:12
openstackgerritMark Goddard proposed openstack/tenks master: CI: Switch to kayobe-overcloud-centos8  https://review.opendev.org/71213617:12
TheJuliauzumaki: can confirm, it is fine. We've long recognized that each vendor has separate preferred names. and some just don't even expose some of the settings  at all :(17:13
uzumakiI see. I'm also wondering, should I expose all the vendor methods as clean steps or not? For instance, with dracwsman, I have 5 passthru methods, right? But I can hide them behind the standard BIOSInterface abstract methods of apply_configuration, factory_reset and cache_bios_settings17:15
uzumakiI think exposing set_bios_config/commit_bios_config etc will make things weird17:16
openstackgerritRiccardo Pittau proposed openstack/ironic master: Default IRONIC_RAMDISK_TYPE to dib  https://review.opendev.org/71027817:20
TheJuliauzumaki: separate clean steps would be preferred to replace all vendor passthrough functionality17:21
uzumakiThat's what I'm thinking as well. So, for instance, the get_bios_config/set_bios_config/commit_bios_config methods can all be 'hidden' behind the apply_configuration clean step17:23
uzumakiBut does it make sense to expose 'abandon_bios_config' as a clean step?17:23
TheJuliaI suspect it does if those explicit methods are still available and someone is kind of outside the high level get/set abstraction17:24
TheJuliai.e. if they have a different workflow that they need to follow, I guess it would work17:24
uzumakiYeah, that makes me want to expose abandon_bios_config as a clean step which will allow them to revoke any pending uncommitted changes through a clean step call17:24
TheJuliasounds reasonable17:25
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Adapt ironic inspector jobs to ironic dib-based jobs  https://review.opendev.org/71251617:25
uzumakiGreat! clean step it is17:25
TheJulianot every fleet/action will fit into the high level get/set model, we just figured that model is the easiest for most to implement something into17:25
TheJuliaI may do abandon as a separate patch, fwiw17:26
TheJuliaunless you see a need as part of the overall process17:26
TheJuliaimplementor's call :)17:26
uzumakiI see. Well, since I'm doing it, might as well do the whole thing right now, than worry about it later17:26
rpiosouzumaki: Please consider making it separate.17:27
uzumakiyou mean making "abandon_bios_config" a separate clean step?17:27
rpiosouzumaki: y17:27
uzumakiI'm asking, what do you mean by 'making it separate'?17:28
rpiosouzumaki: A separate clean step and separate proposed change.17:28
uzumakiOh! Okay. Sure, no problem.17:29
uzumakijust trying to cover all corners right now for completeness sake, we can prioritize things once I have the whole thing in front of me17:30
rpiosouzumaki: I expect making it separate would simplify things for the reviewers, because the the fundamental BIOS implementation is a parity thing. The abandon add-on could be a child patch which is submitted at the same time.17:30
uzumakioh I see. Makes sense.17:31
rpiosoOf course, the child one would depend on the main BIOS interface implementation, so it could not merge until the main one does.17:31
TheJuliauzumaki: fwiw, I'm thinking the same way as rpioso, my caveat was more along the lines of "if you need it as part of the workflow to prevent some sort of failure case"17:31
openstackgerritRiccardo Pittau proposed openstack/ironic master: Default IRONIC_RAMDISK_TYPE to dib  https://review.opendev.org/71027817:32
uzumakiOh, I don't believe I need it in my workflow, like I said, was doing it to cover all corners. I can make a note of it, and implement it in a later change. Totally okay.17:32
rpiosouzumaki: And I agree with TheJulia :-)17:32
uzumakiAnd I agree with the both of you :D17:32
*** Lucas_Gray has quit IRC17:33
TheJulia\o/ consensus17:33
rpiosouzumaki: Note well that later could be less than a millisecond :-)17:33
rpiosoJust line up the changes and enter "git review".17:33
uzumakihaha, sure thing :D So, what I have so far, is the implementation plan for DracWSManBIOS interface which will go into the bios.py of drac module. Where else does code need to go to enable this? Update the driver to add the name of this newly implemented interface?17:35
*** erbarr has joined #openstack-ironic17:36
openstackgerritMerged openstack/tenks master: Bump ansible version constraints to >=2.7,<2.10  https://review.opendev.org/71010017:36
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: Adapt ironic inspector jobs to ironic dib-based jobs  https://review.opendev.org/71251617:38
*** bnemec has quit IRC17:38
rpiosouzumaki: Create the idrac-wsman entrypoint in setup.cfg, update the driver, class IDRACHardware, to declare support for the new BIOS interface implementation, implement unit tests to provide code execution coverage and exercise relevant workflows, write a release note, and update the iDRAC hardware type's documentation.17:39
*** bnemec has joined #openstack-ironic17:39
uzumakithat is so helpful, thanks!17:41
rpiosouzumaki: You're most welcome. And thank you!17:42
*** tzumainn has left #openstack-ironic17:44
*** tzumainn has joined #openstack-ironic17:44
*** dougsz has quit IRC17:49
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Fix dhcp job  https://review.opendev.org/71230417:50
*** k_mouza has quit IRC17:59
openstackgerritMerged openstack/bifrost stable/train: Remove ip kernel command line argument  https://review.opendev.org/71249118:03
*** priteau has quit IRC18:03
*** sshnaidm is now known as sshnaidm|afk18:04
*** derekh has quit IRC18:07
openstackgerritMerged openstack/tenks master: CentOS 8 support  https://review.opendev.org/69588118:07
*** dsneddon has quit IRC18:15
openstackgerritMerged openstack/bifrost stable/stein: Remove ip kernel command line argument  https://review.opendev.org/71249218:17
openstackgerritMerged openstack/bifrost stable/rocky: Remove ip kernel command line argument  https://review.opendev.org/71249318:17
openstackgerritMerged openstack/bifrost master: Fix and add logs and info  https://review.opendev.org/70716218:17
*** uzumaki has quit IRC18:24
*** dtantsur|brb is now known as dtantsur18:24
*** amoralej is now known as amoralej|off18:33
*** tesseract has quit IRC18:43
openstackgerritMerged openstack/ironic-specs master: Update L3 based deployment spec  https://review.opendev.org/67278018:59
*** uzumaki has joined #openstack-ironic19:09
uzumakirpioso, when you say create the idrac-wsman entrypoint, you mean appending the line "idrac = ironic.drivers.modules.drac.bios:DracWSManBIOS" here https://github.com/openstack/ironic/blob/master/setup.cfg#L6019:10
uzumakii mean *idrac-wsman = stuff*19:11
* rpioso is in a meeting :-(. Will be back in a bit.19:12
uzumakioh, sure sure, take your time19:12
openstackgerritMerged openstack/ironic stable/stein: Disable grenade jobs from Stein  https://review.opendev.org/71200219:14
openstackgerritMerged openstack/ironic master: Skip clean steps from 'fake' interfaces in the documentation  https://review.opendev.org/71229719:14
rpiosouzumaki: Yes, that's what I mean.19:16
uzumakiGreat!19:16
*** ociuhandu has joined #openstack-ironic19:17
*** ociuhandu has quit IRC19:21
*** sziviani_ has joined #openstack-ironic19:22
uzumakifor testing, I need to implement particular execution workflows for the IDRACHardware in test_drac.py and DracWSMANBIOS in test_bios.py. Is there anything else I need to be aware of? from testing perspective?19:34
uzumakirpioso,19:34
uzumakiI'll spend some time with the testing base.py to understand the whole thing, just checking if I got all the concerns in one place19:35
rpiosouzumaki: You may find some ideas in the BIOS interface spec and generic Redfish BIOS interface implementation's unit tests. That's where I would look for things beyond code execution coverage unit testing.19:36
uzumakisounds good. Great. I'll defer the documentation update discussion till later, let's get the patch going first. Thanks for the help. I'm signing off now. Will pick it up tomorrow.19:38
rpiosouzumaki: Cool! Hope you have a great nite.19:39
uzumakithanks, have good day! :)19:39
*** uzumaki has quit IRC19:39
rpittaugood night! o/19:40
*** rpittau is now known as rpittau|afk19:40
*** dtantsur is now known as dtantsur|afk19:40
dtantsur|afko/19:40
openstackgerritMerged openstack/ironic-python-agent master: Log the agent version  https://review.opendev.org/71199519:45
openstackgerritMerged openstack/ironic-python-agent master: Add NTP time sync  https://review.opendev.org/70791119:51
*** ociuhandu has joined #openstack-ironic19:52
*** ociuhandu has quit IRC19:56
*** ociuhandu has joined #openstack-ironic19:56
TheJulia\o/19:57
JayFTheJulia: so for https://review.opendev.org/707911 ... that's going to break if someone has HW clock in local time. While I know that's a ... ridiculous configuration ... it is common. Do we just trust users to configure the timezone correctly on the ramdisk?20:02
patchbotpatch 707911 - ironic-python-agent - Add NTP time sync (MERGED) - 7 patch sets20:02
JayFat a minimum I wonder if we want to update the release note to warn about that particular dragon20:03
*** sziviani_ has quit IRC20:05
*** ociuhandu has quit IRC20:06
*** ociuhandu has joined #openstack-ironic20:09
*** ociuhandu has quit IRC20:14
openstackgerritMerged openstack/ironic master: Finalize removal of ipxe_enabled option  https://review.opendev.org/69835220:17
*** sziviani_ has joined #openstack-ironic20:21
openstackgerritJulia Kreger proposed openstack/ironic master: WIP ipv6 jobs  https://review.opendev.org/61316920:23
TheJuliaJayF: good point, I think a release note is likely the route to go right now20:24
*** jcoufal has quit IRC20:39
openstackgerritMark Goddard proposed openstack/bifrost master: Fix Ubuntu keystone deployment  https://review.opendev.org/71211220:42
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Amending ntp time sync release note  https://review.opendev.org/71255220:48
openstackgerritJulia Kreger proposed openstack/ironic master: Add indicators REST API endpoints  https://review.opendev.org/65178520:58
*** bdodd has quit IRC21:07
*** ijw has quit IRC21:18
*** ijw has joined #openstack-ironic21:21
*** ijw_ has joined #openstack-ironic21:24
*** ijw has quit IRC21:25
*** rcernin has joined #openstack-ironic21:28
*** TxGirlGeek has quit IRC21:44
openstackgerritJulia Kreger proposed openstack/ironic master: WIP ipv6 jobs  https://review.opendev.org/61316921:51
*** ociuhandu has joined #openstack-ironic22:04
*** TxGirlGeek has joined #openstack-ironic22:13
TheJuliamgoddard: I left a comment on the bifrost keystone fix. I suspect the first command is failing, I had to add a retry to the very first command which was os_project22:15
*** alexmcleod has quit IRC22:16
*** tkaprol has joined #openstack-ironic22:17
*** jdandrea has quit IRC22:22
*** tkaprol has quit IRC22:22
*** diurnalist has quit IRC22:23
*** dustinc has quit IRC22:31
*** portdirect has quit IRC22:31
*** andrein has quit IRC22:31
*** portdirect has joined #openstack-ironic22:34
*** andrein has joined #openstack-ironic22:36
*** guilhermesp has quit IRC22:36
*** jungleboyj has quit IRC22:36
*** phrobb has quit IRC22:36
*** mnaser has quit IRC22:36
*** gmann has quit IRC22:36
*** donnyd has quit IRC22:36
*** erbarr has quit IRC22:36
*** mnaser has joined #openstack-ironic22:39
*** jungleboyj has joined #openstack-ironic22:40
*** dustinc has joined #openstack-ironic22:40
*** phrobb has joined #openstack-ironic22:40
*** gmann has joined #openstack-ironic22:40
*** guilhermesp has joined #openstack-ironic22:41
*** ociuhandu has quit IRC22:42
*** k_mouza has joined #openstack-ironic22:43
*** ociuhandu has joined #openstack-ironic22:43
*** donnyd has joined #openstack-ironic22:44
*** erbarr has joined #openstack-ironic22:45
*** johnhaan has quit IRC22:46
*** rajinir has quit IRC22:46
*** arne_wiebalck has quit IRC22:46
*** irclogbot_2 has quit IRC22:47
*** k_mouza has quit IRC22:47
*** irclogbot_1 has joined #openstack-ironic22:48
*** ociuhandu has quit IRC22:48
*** openstackstatus has quit IRC22:48
*** johnhaan has joined #openstack-ironic22:49
*** seongsoocho has quit IRC22:50
*** mnasiadka has quit IRC22:50
*** rajinir has joined #openstack-ironic22:50
*** diurnalist has joined #openstack-ironic22:50
*** arne_wiebalck has joined #openstack-ironic22:51
*** rh-jelabarre has quit IRC22:53
openstackgerritJulia Kreger proposed openstack/ironic master: Hash the rescue_password  https://review.opendev.org/69141322:53
*** seongsoocho has joined #openstack-ironic22:54
*** mnasiadka has joined #openstack-ironic22:54
*** ildikov has quit IRC22:56
*** tonyb[m] has quit IRC22:56
*** rpioso has quit IRC22:56
*** ildikov has joined #openstack-ironic22:59
*** tonyb[m] has joined #openstack-ironic23:01
*** tkajinam has joined #openstack-ironic23:01
*** rpioso has joined #openstack-ironic23:03
*** ianychoi has joined #openstack-ironic23:06
*** Goneri has quit IRC23:15
*** ijw_ has quit IRC23:36
*** diurnalist has quit IRC23:37
*** sziviani_ has quit IRC23:46
*** ijw has joined #openstack-ironic23:54
*** ijw has quit IRC23:59

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!