Wednesday, 2021-12-01

*** pmannidi|AFK is now known as pmannidi01:02
opendevreviewSteve Baker proposed openstack/ironic master: Add missing mode setting on pxe created directories  https://review.opendev.org/c/openstack/ironic/+/80929403:47
opendevreviewSteve Baker proposed openstack/ironic master: Move place_loaders_for_boot to boot driver __init__  https://review.opendev.org/c/openstack/ironic/+/81972903:47
opendevreviewSteve Baker proposed openstack/ironic master: Write master grub config on startup  https://review.opendev.org/c/openstack/ironic/+/81558003:47
opendevreviewSteve Baker proposed openstack/ironic master: Capture [pxe]loader_file_paths for distros  https://review.opendev.org/c/openstack/ironic/+/81539203:47
hgy_Hey, Ironic!05:28
rpittaugood morning ironic! o/07:24
arne_wiebalckGood morning hgy rpittau and Ironic!07:38
rpittauhey arne_wiebalck :)07:38
hgyhey arne_wiebalck :)07:49
hgyhey rpittau07:50
hgy:)07:50
rpittauhey hgy :)07:50
opendevreviewRiccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on CentOS Stream 9  https://review.opendev.org/c/openstack/bifrost/+/81905807:51
jandershey hgy rpittau arne_wiebalck and Ironic o/07:55
rpittauhey janders :)07:55
hgyhey janders :)07:56
dtantsurmorning ironic09:09
rpittaugood morning dtantsur :)09:09
dtantsurstendulker: hi, could you maybe merge https://review.opendev.org/c/x/proliantutils/+/819787 or find someone who could approve? it's blocking the ironic CI.09:11
stendulkersure. will get it done09:12
dtantsurthx!09:12
opendevreviewArne Wiebalck proposed openstack/ironic-python-agent master: Add SMART self test to disk burn-in  https://review.opendev.org/c/openstack/ironic-python-agent/+/81950509:16
rpittaudtantsur: selinux claimed another victim09:21
rpittauDec  1 08:12:58 localhost audit[9647]: AVC avc:  denied  { watch } for  pid=9647 comm="dnsmasq" path="/etc/dnsmasq.d/bifrost.dhcp-hosts.d" dev="xvda1" ino=5299315 scontext=system_u:system_r:dnsmasq_t:s0 tcontext=unconfined_u:object_r:dnsmasq_etc_t:s0 tclass=dir permissive=009:21
arne_wiebalckhey janders and dtantsur o/09:23
iurygregorygood morning Ironic o/09:24
rpittauhey iurygregory :)09:24
iurygregoryrpittau, o/09:24
opendevreviewArne Wiebalck proposed openstack/ironic-python-agent master: Add SMART self test to disk burn-in  https://review.opendev.org/c/openstack/ironic-python-agent/+/81950509:29
rpittauI have the feeling there was a regression in dnsmasq selinux policy in centos stream 909:30
iurygregory^ not the first thing I was expecting to read :D09:32
rpittauoh I posted the error before you joined :)09:32
rpittauI'm glad to post it again09:32
iurygregoryhahaha09:32
iurygregorypls do :D09:32
iurygregoryI still need a lot of coffee to wake up09:33
rpittauI collected the entire entry, so there you go https://paste.openstack.org/show/811367/09:33
rpittauoh yeah, coffee sounds like a great idea09:33
iurygregoryspecially when you are awake at 6:34 am09:34
iurygregorystill better than 4am...09:35
rpittaujetlag is not cool :/09:35
iurygregoryyup =(09:35
dtantsurrpittau: this directory is created by us, so we're probably not setting the right context on it10:39
rpittauyeah, even though I'm not seeing that happening on centos 810:41
dtantsurprobably stricter policies for 9?10:43
dtantsurin any case, it should be easy to fix once you figure out the correct context10:43
dtantsurstendulker, Nisha, thank you for merging the proliantutils fix. Would it be possible to issue a release with it?10:46
stendulkerdtantsur: yes, it would be done soon10:46
dtantsurthanks again!10:47
iurygregoryquick question, anyone already working on the release for the bugfix branch?10:49
rpittauiurygregory: I think we want to wait next week ?10:50
iurygregorymaybe? 10:52
rpittaudtantsur: the context seems correct so I think the policy is actually missing the "watch" directive for dnsmasq_etc_t10:54
iurygregoryI will update the Yoga Themes to reflect that10:56
dtantsuriurygregory: I think the themes already say that? or did I forget already?10:57
iurygregorydtantsur, they do (I was planning the release for the bugfix this week)10:57
iurygregorybut after talking with rpittau it makes sense to do next week =)10:58
opendevreviewDmitry Tantsur proposed openstack/ironic master: Refactor common configuration bits from service commands  https://review.opendev.org/c/openstack/ironic/+/82001711:21
opendevreviewDmitry Tantsur proposed openstack/ironic master: [WIP] All-in-one Ironic service with a local RPC bus  https://review.opendev.org/c/openstack/ironic/+/81962011:23
opendevreviewDmitry Tantsur proposed openstack/ironic master: DNM try to repair RPC  https://review.opendev.org/c/openstack/ironic/+/81990311:25
opendevreviewRiccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on CentOS Stream 9  https://review.opendev.org/c/openstack/bifrost/+/81905811:29
opendevreviewRiccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on CentOS Stream 9  https://review.opendev.org/c/openstack/bifrost/+/81905811:30
dmelladodtantsur:  o/11:39
dtantsurhey dmellado, long time no see :)11:39
dmelladoa long time indeed!11:39
dmelladobut I still had glowing-bear around xD11:39
dtantsur(we just chatted privately, and I've realized that Bifrost docs could use more love)11:39
dtantsur\o/11:39
janderssee you tomorrow Ironic o/11:53
stendulker_dtantsur: proliantutils release is done. New version is 2.12.212:22
dtantsurawesome :)12:22
opendevreviewRiccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on CentOS Stream 9  https://review.opendev.org/c/openstack/bifrost/+/81905813:17
TheJuliagood morning13:34
dtantsurmorning TheJulia 13:40
* TheJulia wonders why so many people suddenly want to do software raid13:45
dtantsuryeah, interesting13:55
dtantsurgood that we're ready (arne_wiebalck++)13:55
dtantsuralso PSA: if you patch has failed ironic-tox-unit-with-driver-libs because of iLO issues, it should be safe to recheck now.13:55
arne_wiebalckTheJulia: dtantsur: \o/ , you have clients who ask for it?13:58
iurygregorygood morning TheJulia =)14:03
iurygregoryarne_wiebalck, now we have some clients that are interested =)14:05
arne_wiebalckiurygregory: nice, good to see it is useful!14:06
dtantsurhttps://review.opendev.org/c/openstack/ironic/+/819620 fails all devstack jobs because the RPC stopped working14:12
dtantsurI can find no logical reason why. The new features are not used anywhere. Could someone stare with me?14:13
* TheJulia gets out the onion peeler14:15
TheJuliadtantsur: refresh the change, I left a comment14:21
dtantsurTheJulia: I don't think that's it - see the response14:22
dtantsurI actually have a testing patch that removes the "if", it fails the same: https://review.opendev.org/c/openstack/ironic/+/81990314:23
dtantsurfunnily, JSON RPC keeps working, as shown by the functional tests14:24
dtantsur(this is probably why I cannot reproduce the failure on bifrost)14:24
TheJuliaahh yeah14:25
dtantsureven unit tests pass as they are14:27
opendevreviewRiccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on CentOS Stream 9  https://review.opendev.org/c/openstack/bifrost/+/81905814:29
opendevreviewArne Wiebalck proposed openstack/ironic-python-agent master: Burn-in: Add options for named log files  https://review.opendev.org/c/openstack/ironic-python-agent/+/82003214:30
dtantsurTheJulia: I guess I'll start splitting up this patch further14:31
opendevreviewDmitry Tantsur proposed openstack/ironic master: [WIP] All-in-one Ironic service with a local RPC bus  https://review.opendev.org/c/openstack/ironic/+/81962014:43
TheJuliahmm, yeah, that doesn't seem to make any sense14:44
dtantsur:(14:44
dtantsurI will create 3 patches out of this one to see which one is breaking14:45
TheJuliayeah14:45
TheJuliamakes a lot of sense to do that14:45
TheJuliaunfortunately14:45
opendevreviewDmitry Tantsur proposed openstack/ironic master: DNM try to repair RPC  https://review.opendev.org/c/openstack/ironic/+/81990314:47
opendevreviewDmitry Tantsur proposed openstack/ironic master: [WIP] Use a local call instead of RPC for the combined service  https://review.opendev.org/c/openstack/ironic/+/82003514:47
opendevreviewDmitry Tantsur proposed openstack/ironic master: [WIP] Add rpc_transport == none  https://review.opendev.org/c/openstack/ironic/+/82003614:48
dtantsurokay, now wait14:48
TheJuliawait()15:05
TheJulianow to figure out what was going on with https://review.opendev.org/c/openstack/python-ironicclient/+/81992915:06
iurygregorydtantsur, rpittau re brifrost in arm, the idea would include having a job running bifrost in arm nodes right?15:24
opendevreviewJulia Kreger proposed openstack/python-ironicclient master: Add node history support  https://review.opendev.org/c/openstack/python-ironicclient/+/81992915:25
rpittauiurygregory: at least, yes15:30
iurygregoryrpittau, ack15:30
opendevreviewArne Wiebalck proposed openstack/ironic-python-agent master: Burn-in: Add options for named log files  https://review.opendev.org/c/openstack/ironic-python-agent/+/82003215:38
dtantsurTheJulia: https://review.opendev.org/c/openstack/ironic/+/819620/ passes, https://review.opendev.org/c/openstack/ironic/+/820035/ does not. okay, it's a data point.15:56
TheJuliahmmmm15:57
dtantsurwhich makes _prepare_call a suspect15:58
dtantsurexcept that.. I don't see anything wrong with it, nor does the CI expose any clue15:59
TheJuliadtantsur: did it still look like it actually put the message on the rpc bus?16:03
dtantsurwell, at least the timeout happened somewhere in oslo.messaging code16:03
dtantsurrabbitmq logs are less than helpful, but they do show connections from ironic stuff16:04
TheJuliaokay16:04
TheJuliahow would this impact the reading side... 16:04
TheJuliathis is bizzar16:04
dtantsuryeah16:06
dtantsurneed to go, see you tomorrow o/16:11
arne_wiebalckOps question on deprecation of iscsi and upgrades: do we have recipe of what operators should to move their active instances from iscsi to direct, for instance, as the API does not allow this?16:42
arne_wiebalck*should do16:42
*** sshnaidm is now known as sshnaidm|afk16:46
rpittaugood night! o/16:59
TheJuliaarne_wiebalck: a recipe... I thought you could chagne the interface on a deployed node17:01
jingvarHi!17:05
jingvarI have issues with manual installation Bifrost on Debian (Buster/ bullseye )17:07
jingvarthere are a lot of unmet dependencies17:08
jingvarlike libc6-dev : Depends: libc6 (= 2.28-10)17:08
jingvaris it ok?17:08
arne_wiebalckTheJulia: I don't think you can.17:18
arne_wiebalckTheJulia: Hmm, you can move it to maintenance, and then set it :)17:20
arne_wiebalckTheJulia: I guess that works.17:20
arne_wiebalckTheJulia: Yeah, it does :)17:21
TheJuliatjat was wjat O was tjomlomg17:27
TheJuliajingvar: Well, not exactly expected but makes sense since some of the python modules it installs requires compilation17:27
jingvarIt unxpected for me. I thought Debian is not Stream :)17:29
*** sshnaidm|afk is now known as sshnaidm17:29
*** gmann is now known as gmann_afk17:34
arne_wiebalckTheJulia: Are you aware of anything that needs to be changed in addition when the deploy-interface is changed? Asking for a friend who is thinking about doing the move from iscsi to direct via a DB update :-)17:38
*** gmann_afk is now known as gmann18:53
stevebaker[m]morning ironic!20:00
TheJuliastevebaker[m]: good morning!20:02
TheJuliaarne_wiebalck: well, have you confirmed direct works in your enviornemnt, meaning that your http download setting is set and/or swift is present ?20:03
TheJuliaarne_wiebalck: if so, yeah, just a db update should work :\20:03
arne_wiebalckTheJulia: Oh, yes: we have a couple of thousand nodes with direct already. 20:06
arne_wiebalckTheJulia: We were just thinking it is time now to move the rest, before we move to the next upgrade.20:07
arne_wiebalckTheJulia: Ok, thanks!20:08
TheJuliaarne_wiebalck: oh good, you had me *super* worried there for a minute20:24
TheJuliaor five ;)20:24
arne_wiebalckTheJulia: heh, sorry :)20:39
opendevreviewJulia Kreger proposed openstack/sushy master: Raise an AccessError with SessionService init  https://review.opendev.org/c/openstack/sushy/+/82007622:10
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: nuke the redfish cache on errors  https://review.opendev.org/c/openstack/ironic/+/82007722:21
opendevreviewJulia Kreger proposed openstack/sushy master: Raise an AccessError with SessionService init  https://review.opendev.org/c/openstack/sushy/+/82007622:22
TheJuliastevebaker[m]: ^^ I... think that would do it22:22
TheJuliaI tried to make the story fairly detailed, I guess we shall see what CI comes back with22:22
NobodyCamso we're currently running IPA on a CentOs8 based image. is the suggestion to move to stream?  22:26
TheJuliaNobodyCam: in the grand scheme of things, yeah22:54
TheJuliaNobodyCam: one thing to keep in mind you may see a break with packages as they transition to newer versions, but it has stabilized for the most part22:55
TheJuliajust keep a backup copy of the last known good thing22:55
TheJuliabut I don't think anything has happened recently22:55
NobodyCam++ good to know22:55
NobodyCamThank you TheJulia !!!22:55
TheJulialike, the last thing we hit was a slight change in grub as it jumped from 2.02 to 2.0422:56
stevebaker[m]ok, I've refreshed my nova deleting/cleaning change https://review.opendev.org/c/openstack/nova/+/81372923:17
stevebaker[m]arne_wiebalck: ^^23:17
*** pmannidi is now known as pmannidi|Interview23:59

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