Thursday, 2021-08-05

*** pmannidi is now known as pmannidi|brb00:04
opendevreviewJulia Kreger proposed openstack/ironic master: Fix upgrade logic to allow for bundled changes  https://review.opendev.org/c/openstack/ironic/+/80353200:15
TheJuliastevebaker: awesome00:16
opendevreviewJulia Kreger proposed openstack/ironic master: Implements node history: database  https://review.opendev.org/c/openstack/ironic/+/76800900:18
TheJuliaso... I noticed ^^^ change was failing grenade and I was like "uhh, oh.... the upgrade check doesn't handle that gracefully....00:18
TheJulia"00:18
NobodyCamwow my devstack IPA seeming hanging on trying to get a IPMI address for the VM its attempting to deploy00:26
TheJuliaNobodyCam: o'rly01:01
TheJuliaNobodyCam: arm by chance?01:02
*** pmannidi|brb is now known as pmannidi01:06
opendevreviewSteve Baker proposed openstack/ironic master: DNM/WIP - Add FIPS jobs  https://review.opendev.org/c/openstack/ironic/+/79773902:26
TheJuliastevebaker: https://review.opendev.org/c/openstack/puppet-ironic/+/803537 02:39
TheJuliamaybe... dunno.02:39
* stevebaker looks02:47
iurygregorygood morning Ironic o/06:37
* cenne impressed and excited seeing mentors geek out about routers / home network06:45
cennegoof morning iurygregory!06:45
cenne*good :p06:45
iurygregoryhey cenne o/07:04
cennehey iurygregory 07:20
opendevreviewcenne proposed openstack/ironic master: Add api endpoints for changing boot_mode and secure_boot state  https://review.opendev.org/c/openstack/ironic/+/80008407:24
*** rpittau|afk is now known as rpittau07:47
rpittaugood morning ironic! o/07:47
*** pmannidi is now known as pmannidi|brb07:49
dtantsurmorning ironic07:54
iurygregorymorning rpittau dtantsur o/07:58
rpittauhey iurygregory :)07:58
janders_good morning iurygregory cenne rpittau dtantsur and Ironic o/08:05
rpittauhey janders_ :)08:05
iurygregoryhey janders_ o/08:06
cennehey janders_08:06
iurygregoryI'm wondering if is correct that ironic reports 500 Internal server error when trying to remove a subscription that doesn't exist via vendor passthru... dtantsur rpittau any thoughts? =) from an API point of view it should be a 404, since RedfishError is subclass of DriverOperationError / IronicException the default code is 500 hehe09:02
rpittauthat's a good point, strictly speaking the link is broken because of a the missing path to the subscription, so it does sound like it could be a 404 instead09:07
iurygregoryyeah, if we try to delete a subscription that doesn't exist in the BMC it reports 40409:11
iurygregoryI will push a fix for this =)09:12
dtantsur++09:21
opendevreviewAija Jauntēva proposed x/sushy-oem-idrac master: Add RAID mode conversion methods  https://review.opendev.org/c/x/sushy-oem-idrac/+/79950609:47
opendevreviewAija Jauntēva proposed x/sushy-oem-idrac master: Add OEM Task extension  https://review.opendev.org/c/x/sushy-oem-idrac/+/80026909:55
opendevreviewAija Jauntēva proposed x/sushy-oem-idrac master: Add RAID clear foreign config method  https://review.opendev.org/c/x/sushy-oem-idrac/+/80007109:59
* dtantsur wonders where stevebaker's interest in postgres is coming from10:03
opendevreviewRiccardo Pittau proposed openstack/ironic-inspector master: Add lower-constraints job to current development branch  https://review.opendev.org/c/openstack/ironic-inspector/+/80357010:21
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic master: Fix delete subscription  https://review.opendev.org/c/openstack/ironic/+/80358611:06
opendevreviewMerged x/sushy-oem-idrac master: Add Flake8 W503 ignore  https://review.opendev.org/c/x/sushy-oem-idrac/+/80342512:16
janders_see you tomorrow Ironic o/12:21
rpittaubye janders_ 12:21
opendevreviewDmitry Tantsur proposed openstack/sushy-tools master: Support HTTP basic auth  https://review.opendev.org/c/openstack/sushy-tools/+/80359612:42
dtantsurfinally got to this ^^12:42
iurygregoryyay I've found another bug in the subscriptions hehe12:44
iurygregoryforgot to ignore context protocol in the validation part if they are None hehe12:44
opendevreviewDmitry Tantsur proposed openstack/sushy-tools master: Use correct status code with werkzeug exceptions  https://review.opendev.org/c/openstack/sushy-tools/+/80359812:53
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic master: Fix subscription vendor passthru  https://review.opendev.org/c/openstack/ironic/+/80358613:16
* TheJulia looks at her work laptop doing funky framebbufer things13:22
opendevreviewRiccardo Pittau proposed openstack/sushy master: Add lower-constraints job to current development branch  https://review.opendev.org/c/openstack/sushy/+/80360113:25
dtantsurmorning TheJulia 13:39
opendevreviewAija Jauntēva proposed x/sushy-oem-idrac master: Update set_virtual_boot_device for no manager  https://review.opendev.org/c/x/sushy-oem-idrac/+/79208714:16
opendevreviewDmitry Tantsur proposed openstack/ironic master: Retry stdlib ssl.SSLError  https://review.opendev.org/c/openstack/ironic/+/80360414:18
TheJuliagood morning14:19
dtantsurthe SSL issue ^^ is something we've seen in the wild14:25
dtantsurprobably "thanks" to eventlet14:25
TheJuliaoh joy14:34
opendevreviewJulia Kreger proposed openstack/ironic master: Implements node history: database  https://review.opendev.org/c/openstack/ironic/+/76800914:39
cenneTheJulia: If you can review this https://review.opendev.org/c/openstack/ironic/+/800084 it'd be great. I made the changes you suggested last time. + added a policy per yesterdays discussion. 14:41
opendevreviewJulia Kreger proposed openstack/ironic master: Fix upgrade logic to allow for bundled changes  https://review.opendev.org/c/openstack/ironic/+/80353214:42
TheJuliacenne: I'll try to look today14:42
opendevreviewJulia Kreger proposed openstack/ironic master: Implements node history: database  https://review.opendev.org/c/openstack/ironic/+/76800914:42
cenneThanks. ! :)14:42
opendevreviewJulia Kreger proposed openstack/ironic-tempest-plugin master: remove ebtables fix env var from wallaby stable job  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/79967914:54
opendevreviewRiccardo Pittau proposed openstack/ironic-lib master: Add lower-constraints job to current development branch  https://review.opendev.org/c/openstack/ironic-lib/+/80362815:50
NobodyCamLOL; it was a issue with my local environment. With slack and outlook open the vm we just to slow.. (host system was memory swapping) ;p15:55
NobodyCamand Good Morning Ironic'ers15:55
rpittauslack and outlook? why torturing a poor vm?15:56
rpittaumorning NobodyCam :)15:56
TheJuliaYeah, that does seem quite mean to the VM15:57
NobodyCamLOL; Good Morning rpittau ... slack and outlook on host stealing all the vm's cpu cycles15:57
NobodyCamhehehe15:58
NobodyCamGood Morning TheJulia 15:58
TheJuliaom nom nom cpu cycles15:58
opendevreviewBob Fournier proposed openstack/ironic master: [WIP] For particular vendors set the boot device prior to installing image  https://review.opendev.org/c/openstack/ironic/+/80238916:04
* dtantsur has been talking for 80 minutes and is now exhausted16:22
TheJuliaugh16:25
*** sshnaidm is now known as sshnaidm|afk16:29
rpittaugood night! o/16:41
*** rpittau is now known as rpittau|afk16:41
NobodyCamnight rpittau|afk 17:12
dtantsursee you o/17:16
-opendevstatus- NOTICE: The Gerrit service on review.opendev.org is going down for a quick restart to adjust its database connection configuration, and should return to service momentarily20:03
stevebakergood morning20:19
janders_good morning stevebaker and Ironic o/20:52
stevebakerjanders_: hey!20:53
bfourniegood morning janders_ and stevebaker20:59
stevebakerbfournie: hey Bob!20:59
janders_hey bfournie! :)21:00

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