Friday, 2021-03-26

*** larsks has quit IRC00:00
*** larsks has joined #openstack-ironic00:01
iurygregorystevebaker, hey you still around for some questions? =)00:06
iurygregorymorning janders o/00:14
jandershey iurygregory o/00:14
iurygregoryzer0c00l, I've never used IRONIC_BRNACH normally I would add the refs to the patch in the plugin line..00:16
iurygregoryalso I would change IRONIC_VM_SPECS_RAM to 307200:17
iurygregorysince I think you are testing with DIB and not moderateipa (old tinyipa lol)00:17
iurygregoryalso looking at a job that runs with direct+ipmi+dib I've found some differences in you local.conf, maybe you can test in a fresh env using the one from the CI?00:21
iurygregoryhttps://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_00d/770526/4/check/ironic-tempest-ipa-wholedisk-bios-ipmi-direct-dib/00d125b/controller/logs/local_conf.txt00:22
iurygregoryyou need to remove HOST_IP, SERVICE_HOST00:22
iurygregorychange ERROR_ON_CLONE to "False"00:22
*** tosky has quit IRC00:26
stevebakeriurygregory: hey yeah00:31
iurygregoryyou idea to save the cipher suite that works makes sense to me, but we don't have access to the node in _exec_ipmitool00:33
iurygregoryyour*00:33
iurygregorywe only receive driver_info, command, check_exit_code=None, kill_on_timeout=False00:34
iurygregoryI think we need to call node.save() to override00:36
stevebakeriurygregory: yeah, every caller to _exec_ipmitool has the task though, so task or node would need to be added as an argument00:43
iurygregoryyeah...00:43
iurygregoryif task/node was present it would make things easier hehe00:45
stevebakeromg I just discovered gerrit diff view has blame annotations!01:05
*** tzumainn has quit IRC01:07
jandersstevebaker handy! I never tried clicking that button, it's really cool01:07
jandersthanks!01:08
* stevebaker mashes the b key01:08
iurygregorystevebaker, WOOT?!01:09
iurygregoryholy moly!01:09
stevebakeranother way for me to discover that the person who wrote that crappy code was me01:10
iurygregoryLOL01:10
iurygregoryI'm still trying to process the comment for the new function XD01:11
iurygregorythe idea to user driver_info.get('cipher_suite') for the actual_cs would make sense, but populate with choose_cipher_suite in case is None I don't think it makes sense .-.01:13
iurygregorywe should only go for choose_cipher_suite in case we see the exception I would say .-.01:14
*** k_mouza has joined #openstack-ironic01:15
*** k_mouza has quit IRC01:20
stevebakeriurygregory: ok, I'll add a comment that my suggestion doesn't need to be actioned, other reviewers can add their opinions01:31
iurygregorystevebaker, I'm doing some tests with the idea to look in the list for the '-C' and add the next_cs to use01:32
iurygregoryfor example change the if that adds the '-C' and actual_cs to if not '-C' in cmd_args and change_cs for example =)01:33
stevebakercool01:33
iurygregorygoing to see how it goes and if things work I will push a new patch set in a few01:34
*** paras33__ has joined #openstack-ironic01:37
openstackgerritvinay50muddu proposed openstack/ironic master: Support clean step to add certificates to iLO  https://review.opendev.org/c/openstack/ironic/+/78313301:44
openstackgerritliuyuanfeng proposed openstack/ironic master: when we use smart card, we set boot_device to bios to wait agent to do something.But after agent finishes,we need to set boot_device to disk. If we do not do so,when ironic finishes deploy, node will boot to bois.  https://review.opendev.org/c/openstack/ironic/+/78313401:44
*** jamesdenton has quit IRC01:49
*** jamesden_ has joined #openstack-ironic01:49
iurygregorystevebaker, it worked \o/ I'm moving the code from L16-L30 to a new function http://paste.openstack.org/show/803937/ , I will try to improve the logic a bit more (specially on L26 to be sure things won't go side away)02:02
iurygregoryat least I think the if is a bit more clear now =)02:02
iurygregorywdyt?02:02
iurygregorythe new function would need to return cs_problem and the new args updated02:05
*** cp- has joined #openstack-ironic02:06
stevebakeriurygregory: yes, and I assume lines 2-4 wouldn't be needed anymore?02:06
iurygregoryI haven't test without, let me give a try02:07
iurygregoryyeah =) my bad ^^02:09
iurygregorythanks for the feedback o/02:10
*** larsks has quit IRC02:28
*** larsks has joined #openstack-ironic02:29
*** rcernin has quit IRC02:46
*** rcernin has joined #openstack-ironic02:52
openstackgerritRichard G. Pioso proposed x/sushy-oem-idrac master: Add reset iDRAC method  https://review.opendev.org/c/x/sushy-oem-idrac/+/78225403:02
*** gudrutis has joined #openstack-ironic03:06
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic master: Automaticaly set cipher suite  https://review.opendev.org/c/openstack/ironic/+/77052603:08
openstackgerritRichard G. Pioso proposed x/sushy-oem-idrac master: Add reset iDRAC method  https://review.opendev.org/c/x/sushy-oem-idrac/+/78225403:13
*** k_mouza has joined #openstack-ironic03:16
*** k_mouza has quit IRC03:20
*** early has quit IRC03:31
jandersstevebaker thanks heaps for your comments i/r/t https://review.opendev.org/c/openstack/ironic/+/744117 - good pickup03:32
jandersI will try to submit a revised change later today03:32
jandersI missed the sort bit03:33
openstackgerritliuzhuangzhuang proposed openstack/ironic master: update grub2 file name  https://review.opendev.org/c/openstack/ironic/+/78048103:38
*** mkrai has joined #openstack-ironic03:43
*** early has joined #openstack-ironic03:54
*** tkajinam has quit IRC04:01
*** tkajinam has joined #openstack-ironic04:01
*** mkrai has quit IRC04:39
*** mkrai has joined #openstack-ironic04:40
*** k_mouza has joined #openstack-ironic04:46
*** k_mouza has quit IRC04:50
*** mkrai has quit IRC05:23
*** mkrai_ has joined #openstack-ironic05:23
*** abhishekyadav has joined #openstack-ironic05:52
*** zzzeek has joined #openstack-ironic06:12
*** zzzeek has quit IRC06:16
*** zzzeek has joined #openstack-ironic06:19
*** zzzeek has quit IRC06:26
*** zzzeek has joined #openstack-ironic06:28
*** arne_wiebalck has joined #openstack-ironic06:36
arne_wiebalckGood morning, ironic!06:36
*** ftarasenko has joined #openstack-ironic07:05
*** rpittau|afk has quit IRC07:05
*** vdrok has quit IRC07:05
*** rpittau|afk has joined #openstack-ironic07:06
*** NobodyCam has quit IRC07:06
*** vdrok has joined #openstack-ironic07:06
*** NobodyCam has joined #openstack-ironic07:06
*** ricolin has joined #openstack-ironic07:09
*** rcernin has quit IRC07:11
*** ricolin has quit IRC07:14
*** ricolin has joined #openstack-ironic07:17
*** zzzeek has quit IRC07:25
*** rcernin has joined #openstack-ironic07:38
*** zzzeek has joined #openstack-ironic07:40
*** rcernin has quit IRC07:42
*** zzzeek has quit IRC07:44
openstackgerritYogesh proposed openstack/ironic master: Add idrac HW type IPMI interface support  https://review.opendev.org/c/openstack/ironic/+/77186207:53
*** rcernin has joined #openstack-ironic07:55
*** zzzeek has joined #openstack-ironic07:58
*** rcernin has quit IRC07:59
*** zzzeek has quit IRC08:04
*** rpittau|afk is now known as rpittau08:09
rpittaugood morning ironic! o/08:09
rpittauhappy friday!08:09
*** rcernin has joined #openstack-ironic08:13
*** rcernin has quit IRC08:17
*** lourot has quit IRC08:18
*** lourot has joined #openstack-ironic08:18
*** ajya has joined #openstack-ironic08:31
openstackgerritsonali bhausaheb borkar proposed openstack/ironic master: DRAC : idrac-redfish inspect updates pxe port  https://review.opendev.org/c/openstack/ironic/+/69332808:37
*** zzzeek has joined #openstack-ironic08:42
*** zzzeek has quit IRC08:47
*** abhishekyadav has quit IRC08:53
openstackgerritAija Jauntēva proposed x/sushy-oem-idrac master: Add reset iDRAC method  https://review.opendev.org/c/x/sushy-oem-idrac/+/78225408:53
*** moshiur has joined #openstack-ironic09:01
*** lucasagomes has joined #openstack-ironic09:01
*** rcernin has joined #openstack-ironic09:01
openstackgerritDhuldev Valekar proposed openstack/ironic master: DRAC: Added redfish management clean steps  https://review.opendev.org/c/openstack/ironic/+/72159309:03
*** derekh has joined #openstack-ironic09:04
moshiurrpittau, dtantsur|afk : Is it possible to remove the Depends-on tag from the following PR:https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/778726 and put use the Depends-on tag in other PR in Diskimage-builder PR. Then we can merge the PR from ironic-python-agent-builder.09:04
rpittaumoshiur: well, you can try, you just need to remove it from the commit message09:05
*** rcernin has quit IRC09:06
moshiurrpittau: thanks I will do it and move the depends on tag to other PR.09:08
rpittaumoshiur: the CI will run again, it might fail wihtout the DIB changes09:08
*** zzzeek has joined #openstack-ironic09:10
*** zzzeek has quit IRC09:13
*** zzzeek has joined #openstack-ironic09:15
*** dtantsur|afk is now known as dtantsur09:17
dtantsurmoshiur: no09:17
dtantsuryou change requires that change. unless the opensuse job passes anyway, we will not merge your change09:17
dtantsurasked differently: what's the point of merging your change if you won't be able to build an opensuse image anyway?09:18
openstackgerritMoshiur Rahman proposed openstack/ironic-python-agent-builder master: Fix: IPA image building with OpenSuse.  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/77872609:18
moshiurdtantsur: sorry didn't noticed your message.09:20
moshiurdtantsur: I am adding it again09:21
dtantsurthanks!09:22
dtantsurI understand your frustration btw09:22
dtantsurmoshiur: I think what you need in your DIB change is to expand the commit message with an explanation for every part of it09:23
dtantsurmaybe with example errors09:23
openstackgerritMoshiur Rahman proposed openstack/ironic-python-agent-builder master: Fix: IPA image building with OpenSuse.  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/77872609:23
rpittaumoshiur: apologies, dtantsur is right, the correct dependency is ipa-builder -> dib and not viceversa09:25
moshiurthanks dtantsur.09:25
moshiurrpittau: no problem I reverse the changes09:25
rpittaumaybe opening a bug in launchpad would help ?09:26
dtantsuragainst DIB? yeah, possibly09:26
rpittaumoshiur: maybe linking a launchpad bug to your DIB change can help https://launchpad.net/diskimage-builder09:27
dtantsurmake sure to paste the actual error there09:27
moshiurrpittau, dtantsur: ok I will do that.  all the code reviews are gone form my PR, would you please add the reviews again.09:28
rpittaumoshiur: I re-added mine already to ipa-builder09:29
dtantsurI will check again over the course of the day09:29
*** tosky has joined #openstack-ironic09:30
moshiurTheJulia: would you please add your review and workflow again?09:31
moshiurThanks dtantsur, rpittau.09:31
openstackgerritAija Jauntēva proposed openstack/ironic master: Add configuration mold storage  https://review.opendev.org/c/openstack/ironic/+/76992809:40
*** hoonetorg has joined #openstack-ironic09:49
*** rcernin has joined #openstack-ironic09:53
*** paras333 has joined #openstack-ironic09:53
*** paras33__ has quit IRC09:54
*** abhishekyadav has joined #openstack-ironic09:56
*** k_mouza has joined #openstack-ironic09:59
*** rcernin has quit IRC10:00
dtantsurSPUC time?10:00
dtantsurjanders, rpittau, arne_wiebalck, ajya ^^10:01
abhishekyadavTheJulia: Good Morning! Regarding your comment on the story https://storyboard.openstack.org/#!/story/2008723, we got below spec (from Richard) which was available, is this the spec you were referring  to: https://opendev.org/openstack/ironic-specs/src/branch/master/specs/approved/drac-firmware-update-spec.rst . Please let us know if any more10:03
abhishekyadavdetails needs to be added here. For "how" and "what" changes, will add as we progress with this story.10:03
dtantsurabhishekyadav: I think her request is to make the RFE a bit more detailed on how exactly the feature will work10:04
dtantsuroptions added, swift vs local server, etc10:04
arne_wiebalckdtantsur: sorry, in another meeting10:04
abhishekyadavdtantsur: okay, will do that. Thanks10:05
*** rcernin has joined #openstack-ironic10:05
*** rcernin has quit IRC10:10
*** rcernin has joined #openstack-ironic10:27
*** rcernin has quit IRC10:32
openstackgerritDmitry Tantsur proposed openstack/ironic master: Support deploy vendor data in addition to network data  https://review.opendev.org/c/openstack/ironic/+/78254810:32
*** k_mouza has quit IRC10:39
*** k_mouza has joined #openstack-ironic10:39
*** rcernin has joined #openstack-ironic10:42
*** k_mouza_ has joined #openstack-ironic10:43
*** k_mouza has quit IRC10:44
*** ociuhandu has joined #openstack-ironic10:47
*** rcernin has quit IRC10:52
*** rcernin has joined #openstack-ironic11:00
*** ociuhandu has quit IRC11:02
*** rcernin has quit IRC11:05
openstackgerritRiccardo Pittau proposed openstack/ironic master: Update basic local.conf for quickstart  https://review.opendev.org/c/openstack/ironic/+/78327011:14
*** rcernin has joined #openstack-ironic11:17
*** rcernin has quit IRC11:22
*** ociuhandu has joined #openstack-ironic11:32
jandersdtantsur rpittau arne_wiebalck ajya sorry I wasn't able to make it to the SPUC today11:36
jandersdid I miss rpittau's Sargegnan wine run-through? :)11:37
dtantsurno, we just chatted11:37
rpittaujanders: no, I left it for the next time :)11:38
jandersawesome thanks rpittau :)11:38
rpittauor next timeS, not sure how long it will take :D11:39
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Allow disabling integrated dhcp from bifrost cli  https://review.opendev.org/c/openstack/bifrost/+/78270011:41
*** ociuhandu has quit IRC11:41
*** mkrai_ has quit IRC11:42
*** mkrai_ has joined #openstack-ironic11:42
jandersrpittau we can make it a series :)11:43
rpittaujanders: ahah that could work!11:43
*** rcernin has joined #openstack-ironic11:50
*** rcernin has quit IRC11:54
*** ociuhandu has joined #openstack-ironic11:56
iurygregorygood morning Ironic o/11:58
rpittauhey iurygregory :)11:58
jandershey iurygregory o/11:59
jandersHappy Friday11:59
iurygregoryHappy Friday \o/12:01
*** mkrai_ has quit IRC12:01
*** ociuhandu has quit IRC12:10
*** k_mouza_ has quit IRC12:13
derekhiurygregory: you were asking for details about my vmedia problem https://storyboard.openstack.org/#!/story/200876312:19
iurygregoryderekh, thanks!12:19
*** ociuhandu has joined #openstack-ironic12:22
jandershave a great weekend Ironic o/12:25
janderssee you next week12:25
dtantsuryou too!12:25
jandersnext week = back to daylight savings time - and Easter Break coming, too \o/12:25
jandersthank you dtantsur :)12:25
derekho/12:25
*** tzumainn has joined #openstack-ironic12:30
*** ociuhandu has quit IRC12:31
*** ociuhandu has joined #openstack-ironic12:45
*** ociuhandu has quit IRC12:50
*** ociuhandu has joined #openstack-ironic12:52
*** ociuhandu has quit IRC12:57
*** rcernin has joined #openstack-ironic12:57
*** rcernin has quit IRC12:58
*** rcernin has joined #openstack-ironic12:58
*** ociuhandu has joined #openstack-ironic13:00
*** k_mouza has joined #openstack-ironic13:00
TheJuliagood morning everyone13:09
*** paras333 has quit IRC13:10
*** ociuhandu has quit IRC13:10
openstackgerritMerged openstack/ironic master: redfish-virtual-media: allow USB devices instead of floppies  https://review.opendev.org/c/openstack/ironic/+/78302013:10
iurygregorygood morning TheJulia =)13:10
TheJuliaabhishekyadav: you don't need to do a spec for a driver behavior parity item of work, just make your RFE a bit more verbose.13:10
openstackgerritMerged openstack/ironic master: redfish boot_interfaces, ipmitool -> pxe  https://review.opendev.org/c/openstack/ironic/+/78303613:10
*** rloo has joined #openstack-ironic13:29
dtantsurmorning TheJulia13:32
openstackgerritMerged openstack/ironic-python-agent-builder master: Fix tinyipa virtual media indicator flag  https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/78311013:34
bdoddGood morning, Ironic.13:35
bdoddI'm retiring as of today. It's been a pleasure working with all you very talented and dedicated folks on the Ironic team. I learned a lot. All the best in your future endeavors!13:37
dtantsurbdodd: cool luck! thank you for hard work.13:38
rpittauthanks bdodd! Good luck :)13:38
bdodddtantsur and rpittau Thanks!13:39
*** ociuhandu has joined #openstack-ironic13:40
*** k_mouza has quit IRC13:47
*** k_mouza_ has joined #openstack-ironic13:47
rpittauclosing a bit early today, have a great weekend everyone! o/13:50
*** rpittau is now known as rpittau|afk13:50
TheJuliabdodd: congratulations! Enjoy retirement!13:55
TheJuliawow, I totally mispelled that13:55
TheJuliaeh, computer says no13:55
TheJuliajust doesn't look right :\13:55
TheJuliayay for learning disorders13:55
TheJulias/disorder/disability13:55
bdoddThanks TheJulia I will! :)13:55
*** ociuhandu has quit IRC13:58
moshiurrpittau|afk, dtantsur: opened a bug report in DIB with errorlogs, https://bugs.launchpad.net/diskimage-builder/+bug/192151014:02
openstackLaunchpad bug 1921510 in diskimage-builder "IPA image building for openSuse is failing in release check." [Undecided,New]14:02
dtantsurgreat14:02
*** ociuhandu has joined #openstack-ironic14:10
*** derekh has quit IRC14:20
*** ociuhandu has quit IRC14:21
*** ociuhandu has joined #openstack-ironic14:33
openstackgerritDmitry Tantsur proposed openstack/ironic master: Allow using per-site network_data schema  https://review.opendev.org/c/openstack/ironic/+/78331214:33
dtantsurI cannot believe I'm seriously proposing this ^^ but here we go14:33
*** ociuhandu has quit IRC14:38
*** moshiur has quit IRC14:40
*** ociuhandu has joined #openstack-ironic14:42
*** ociuhandu has quit IRC14:44
*** ociuhandu has joined #openstack-ironic14:44
*** derekh has joined #openstack-ironic14:59
*** abhishekyadav has quit IRC15:00
*** ociuhandu has quit IRC15:04
*** rcernin has quit IRC15:06
openstackgerritMerged x/sushy-oem-idrac master: Add reset iDRAC method  https://review.opendev.org/c/x/sushy-oem-idrac/+/78225415:06
*** mjturek has joined #openstack-ironic15:13
*** paras333 has joined #openstack-ironic15:18
*** paras333 has quit IRC15:18
dtantsurTheJulia: at some point in time we should discuss SPUC times given the DST15:39
dtantsuri.e. when do we meet today? the morning SPUC happened in European time15:40
TheJuliaYeah, I think it might be a good thing, but  maybe next week. I won't be around for today's SPUC due to an errand15:41
dtantsurah, okay15:41
*** ociuhandu has joined #openstack-ironic15:43
*** jamesden_ has quit IRC15:50
*** jamesdenton has joined #openstack-ironic15:51
*** ociuhandu has quit IRC16:08
*** ociuhandu has joined #openstack-ironic16:39
*** gyee has joined #openstack-ironic16:40
*** ociuhandu_ has joined #openstack-ironic16:44
*** ociuhandu_ has quit IRC16:44
*** ociuhandu has quit IRC16:48
*** lucasagomes has quit IRC16:58
openstackgerritAija Jauntēva proposed openstack/ironic master: DRAC: Added redfish management clean steps  https://review.opendev.org/c/openstack/ironic/+/72159316:58
JayFspuc?17:01
dtantsurJayF: TheJulia is out, I'm a bit busy with testing The Patch17:05
JayFThe Patch(tm)17:05
dtantsuryeah17:06
*** iurygregory has quit IRC17:12
*** iurygregory has joined #openstack-ironic17:13
JayFzer0c00l and I are having a devstack-fixin-party in the spuc17:13
JayFif anyone else wants to drop in :)17:13
iurygregoryI will join in a few17:14
iurygregoryjust finished my lunch =)17:14
JayFdtantsur: do many folks devstack on rhel 8? Or do you all mostly use ubuntu17:17
JayFwondering if zer0c00l is having more headaches than usual b/c he's on rhel and gates are ubuntu17:17
dtantsurmost of us don't use devstack :)17:17
dtantsurI've been using bifrost exclusively for many months17:17
dtantsurbut yes, I'd expect devstack on RHEL to be problematic17:18
dtantsurpreviously one of us would fix it from time to time17:18
JayFIs how to do that documented? like with getting VMs and stuff?17:18
JayFor are you testing against real metal17:18
dtantsuryes, yes, rarely17:18
dtantsurgit clone https://opendev.org/openstack/bifrost && cd bifrost && ./bifrost-cli testenv && ./bifrost-cli install --testenv17:19
dtantsur:)17:19
dtantsurI put quite some effort in victoria to make this 4 command sequence possible17:20
JayFwe are going to do that in a separate vm simultanously17:20
JayFrace it with the stack.sh lol17:20
dtantsurheh17:20
dtantsurokay, headache is getting worse, I'm afraid I won't finish all testing today17:28
dtantsurhave a great weekend folks17:28
*** dtantsur is now known as dtantsur|afk17:28
*** yoctozepto has quit IRC17:41
*** yoctozepto has joined #openstack-ironic17:51
*** k_mouza_ has quit IRC17:58
*** jamesdenton has quit IRC18:09
*** jamesdenton has joined #openstack-ironic18:10
openstackgerritOpenStack Release Bot proposed openstack/networking-generic-switch stable/wallaby: Update .gitreview for stable/wallaby  https://review.opendev.org/c/openstack/networking-generic-switch/+/78342218:33
openstackgerritOpenStack Release Bot proposed openstack/networking-generic-switch stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby  https://review.opendev.org/c/openstack/networking-generic-switch/+/78342518:34
openstackgerritOpenStack Release Bot proposed openstack/networking-baremetal stable/wallaby: Update .gitreview for stable/wallaby  https://review.opendev.org/c/openstack/networking-baremetal/+/78342718:34
openstackgerritOpenStack Release Bot proposed openstack/networking-generic-switch master: Update master for stable/wallaby  https://review.opendev.org/c/openstack/networking-generic-switch/+/78342818:34
openstackgerritOpenStack Release Bot proposed openstack/networking-baremetal stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby  https://review.opendev.org/c/openstack/networking-baremetal/+/78343018:34
openstackgerritOpenStack Release Bot proposed openstack/networking-generic-switch master: Add Python3 xena unit tests  https://review.opendev.org/c/openstack/networking-generic-switch/+/78343118:34
openstackgerritOpenStack Release Bot proposed openstack/networking-baremetal master: Update master for stable/wallaby  https://review.opendev.org/c/openstack/networking-baremetal/+/78343218:35
openstackgerritOpenStack Release Bot proposed openstack/networking-baremetal master: Add Python3 xena unit tests  https://review.opendev.org/c/openstack/networking-baremetal/+/78343418:35
arne_wiebalckbye everyone, see you next week o/18:46
iurygregorybye arne_wiebalck19:00
*** rcernin has joined #openstack-ironic19:03
*** rcernin has quit IRC19:07
*** tkajinam_ has joined #openstack-ironic19:40
*** tkajinam has quit IRC19:41
rpiosobdodd: Congratulations! It was a pleasure working with and learning from you. I very much appreciate the substantial contributions you made to enhancing ironic's Redfish support. Hope you enjoy retirement. All the best :)20:24
rpiosoI could use some semver assistance. I'm preparing to cut a new release of sushy-oem-idrac based on https://opendev.org/x/sushy-oem-idrac/commit/83b7eb0de79c10ca045adf2154cba3f310ef5900. The current version is 2.0.0. Is the change to sushy_oem_idrac/constants.py -- https://opendev.org/x/sushy-oem-idrac/src/branch/master/sushy_oem_idrac/constants.py -- considered a breaking change? They are not published by the20:31
rpiosopackage's __init__.py. AFAIA, they are only used internally, but ...20:31
*** k_mouza has joined #openstack-ironic20:33
rpiosoiurygregory, TheJulia: ^^^20:34
* iurygregory reads20:37
*** k_mouza has quit IRC20:37
iurygregoryrpioso, this commit https://opendev.org/x/sushy-oem-idrac/commit/ae279799fdd453ca060ee1e1cf56ecc41c8ca0bd ?20:38
rpiosoiurygregory: Let's move this to #openstack-release :)20:41
bdoddrpioso Thank you! And thanks for all your help, expertise, and encouragement.21:27
*** derekh has quit IRC21:30
*** jamesdenton has quit IRC21:36
*** jamesden_ has joined #openstack-ironic21:36
*** ayoung has joined #openstack-ironic21:38
rpiosobdodd: You're most welcome.21:38
* rpioso hopes bdodd will hang out here a bit on Monday, so others who may have missed his announcement will have a chance to wish him all the best.21:41
* bdodd will try :)21:41
*** rloo has quit IRC22:37
openstackgerritJulia Kreger proposed openstack/ironic stable/train: Fixes the issue that instance bond port can't get IP address  https://review.opendev.org/c/openstack/ironic/+/78254323:27
*** gyee has quit IRC23:31
*** gyee has joined #openstack-ironic23:32
*** tosky has quit IRC23:39

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