*** pmannidi|AFK is now known as pmannidi | 01:02 | |
opendevreview | Steve Baker proposed openstack/ironic master: Add missing mode setting on pxe created directories https://review.opendev.org/c/openstack/ironic/+/809294 | 03:47 |
---|---|---|
opendevreview | Steve Baker proposed openstack/ironic master: Move place_loaders_for_boot to boot driver __init__ https://review.opendev.org/c/openstack/ironic/+/819729 | 03:47 |
opendevreview | Steve Baker proposed openstack/ironic master: Write master grub config on startup https://review.opendev.org/c/openstack/ironic/+/815580 | 03:47 |
opendevreview | Steve Baker proposed openstack/ironic master: Capture [pxe]loader_file_paths for distros https://review.opendev.org/c/openstack/ironic/+/815392 | 03:47 |
hgy_ | Hey, Ironic! | 05:28 |
rpittau | good morning ironic! o/ | 07:24 |
arne_wiebalck | Good morning hgy rpittau and Ironic! | 07:38 |
rpittau | hey arne_wiebalck :) | 07:38 |
hgy | hey arne_wiebalck :) | 07:49 |
hgy | hey rpittau | 07:50 |
hgy | :) | 07:50 |
rpittau | hey hgy :) | 07:50 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on CentOS Stream 9 https://review.opendev.org/c/openstack/bifrost/+/819058 | 07:51 |
janders | hey hgy rpittau arne_wiebalck and Ironic o/ | 07:55 |
rpittau | hey janders :) | 07:55 |
hgy | hey janders :) | 07:56 |
dtantsur | morning ironic | 09:09 |
rpittau | good morning dtantsur :) | 09:09 |
dtantsur | stendulker: 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 |
stendulker | sure. will get it done | 09:12 |
dtantsur | thx! | 09:12 |
opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent master: Add SMART self test to disk burn-in https://review.opendev.org/c/openstack/ironic-python-agent/+/819505 | 09:16 |
rpittau | dtantsur: selinux claimed another victim | 09:21 |
rpittau | Dec 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=0 | 09:21 |
arne_wiebalck | hey janders and dtantsur o/ | 09:23 |
iurygregory | good morning Ironic o/ | 09:24 |
rpittau | hey iurygregory :) | 09:24 |
iurygregory | rpittau, o/ | 09:24 |
opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent master: Add SMART self test to disk burn-in https://review.opendev.org/c/openstack/ironic-python-agent/+/819505 | 09:29 |
rpittau | I have the feeling there was a regression in dnsmasq selinux policy in centos stream 9 | 09:30 |
iurygregory | ^ not the first thing I was expecting to read :D | 09:32 |
rpittau | oh I posted the error before you joined :) | 09:32 |
rpittau | I'm glad to post it again | 09:32 |
iurygregory | hahaha | 09:32 |
iurygregory | pls do :D | 09:32 |
iurygregory | I still need a lot of coffee to wake up | 09:33 |
rpittau | I collected the entire entry, so there you go https://paste.openstack.org/show/811367/ | 09:33 |
rpittau | oh yeah, coffee sounds like a great idea | 09:33 |
iurygregory | specially when you are awake at 6:34 am | 09:34 |
iurygregory | still better than 4am... | 09:35 |
rpittau | jetlag is not cool :/ | 09:35 |
iurygregory | yup =( | 09:35 |
dtantsur | rpittau: this directory is created by us, so we're probably not setting the right context on it | 10:39 |
rpittau | yeah, even though I'm not seeing that happening on centos 8 | 10:41 |
dtantsur | probably stricter policies for 9? | 10:43 |
dtantsur | in any case, it should be easy to fix once you figure out the correct context | 10:43 |
dtantsur | stendulker, Nisha, thank you for merging the proliantutils fix. Would it be possible to issue a release with it? | 10:46 |
stendulker | dtantsur: yes, it would be done soon | 10:46 |
dtantsur | thanks again! | 10:47 |
iurygregory | quick question, anyone already working on the release for the bugfix branch? | 10:49 |
rpittau | iurygregory: I think we want to wait next week ? | 10:50 |
iurygregory | maybe? | 10:52 |
rpittau | dtantsur: the context seems correct so I think the policy is actually missing the "watch" directive for dnsmasq_etc_t | 10:54 |
iurygregory | I will update the Yoga Themes to reflect that | 10:56 |
dtantsur | iurygregory: I think the themes already say that? or did I forget already? | 10:57 |
iurygregory | dtantsur, they do (I was planning the release for the bugfix this week) | 10:57 |
iurygregory | but after talking with rpittau it makes sense to do next week =) | 10:58 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: Refactor common configuration bits from service commands https://review.opendev.org/c/openstack/ironic/+/820017 | 11:21 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] All-in-one Ironic service with a local RPC bus https://review.opendev.org/c/openstack/ironic/+/819620 | 11:23 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: DNM try to repair RPC https://review.opendev.org/c/openstack/ironic/+/819903 | 11:25 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on CentOS Stream 9 https://review.opendev.org/c/openstack/bifrost/+/819058 | 11:29 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on CentOS Stream 9 https://review.opendev.org/c/openstack/bifrost/+/819058 | 11:30 |
dmellado | dtantsur: o/ | 11:39 |
dtantsur | hey dmellado, long time no see :) | 11:39 |
dmellado | a long time indeed! | 11:39 |
dmellado | but I still had glowing-bear around xD | 11:39 |
dtantsur | (we just chatted privately, and I've realized that Bifrost docs could use more love) | 11:39 |
dtantsur | \o/ | 11:39 |
janders | see you tomorrow Ironic o/ | 11:53 |
stendulker_ | dtantsur: proliantutils release is done. New version is 2.12.2 | 12:22 |
dtantsur | awesome :) | 12:22 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on CentOS Stream 9 https://review.opendev.org/c/openstack/bifrost/+/819058 | 13:17 |
TheJulia | good morning | 13:34 |
dtantsur | morning TheJulia | 13:40 |
* TheJulia wonders why so many people suddenly want to do software raid | 13:45 | |
dtantsur | yeah, interesting | 13:55 |
dtantsur | good that we're ready (arne_wiebalck++) | 13:55 |
dtantsur | also 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_wiebalck | TheJulia: dtantsur: \o/ , you have clients who ask for it? | 13:58 |
iurygregory | good morning TheJulia =) | 14:03 |
iurygregory | arne_wiebalck, now we have some clients that are interested =) | 14:05 |
arne_wiebalck | iurygregory: nice, good to see it is useful! | 14:06 |
dtantsur | https://review.opendev.org/c/openstack/ironic/+/819620 fails all devstack jobs because the RPC stopped working | 14:12 |
dtantsur | I 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 peeler | 14:15 | |
TheJulia | dtantsur: refresh the change, I left a comment | 14:21 |
dtantsur | TheJulia: I don't think that's it - see the response | 14:22 |
dtantsur | I actually have a testing patch that removes the "if", it fails the same: https://review.opendev.org/c/openstack/ironic/+/819903 | 14:23 |
dtantsur | funnily, JSON RPC keeps working, as shown by the functional tests | 14:24 |
dtantsur | (this is probably why I cannot reproduce the failure on bifrost) | 14:24 |
TheJulia | ahh yeah | 14:25 |
dtantsur | even unit tests pass as they are | 14:27 |
opendevreview | Riccardo Pittau proposed openstack/bifrost master: [WIP] Run bifrost on CentOS Stream 9 https://review.opendev.org/c/openstack/bifrost/+/819058 | 14:29 |
opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent master: Burn-in: Add options for named log files https://review.opendev.org/c/openstack/ironic-python-agent/+/820032 | 14:30 |
dtantsur | TheJulia: I guess I'll start splitting up this patch further | 14:31 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] All-in-one Ironic service with a local RPC bus https://review.opendev.org/c/openstack/ironic/+/819620 | 14:43 |
TheJulia | hmm, yeah, that doesn't seem to make any sense | 14:44 |
dtantsur | :( | 14:44 |
dtantsur | I will create 3 patches out of this one to see which one is breaking | 14:45 |
TheJulia | yeah | 14:45 |
TheJulia | makes a lot of sense to do that | 14:45 |
TheJulia | unfortunately | 14:45 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: DNM try to repair RPC https://review.opendev.org/c/openstack/ironic/+/819903 | 14:47 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Use a local call instead of RPC for the combined service https://review.opendev.org/c/openstack/ironic/+/820035 | 14:47 |
opendevreview | Dmitry Tantsur proposed openstack/ironic master: [WIP] Add rpc_transport == none https://review.opendev.org/c/openstack/ironic/+/820036 | 14:48 |
dtantsur | okay, now wait | 14:48 |
TheJulia | wait() | 15:05 |
TheJulia | now to figure out what was going on with https://review.opendev.org/c/openstack/python-ironicclient/+/819929 | 15:06 |
iurygregory | dtantsur, rpittau re brifrost in arm, the idea would include having a job running bifrost in arm nodes right? | 15:24 |
opendevreview | Julia Kreger proposed openstack/python-ironicclient master: Add node history support https://review.opendev.org/c/openstack/python-ironicclient/+/819929 | 15:25 |
rpittau | iurygregory: at least, yes | 15:30 |
iurygregory | rpittau, ack | 15:30 |
opendevreview | Arne Wiebalck proposed openstack/ironic-python-agent master: Burn-in: Add options for named log files https://review.opendev.org/c/openstack/ironic-python-agent/+/820032 | 15:38 |
dtantsur | TheJulia: 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 |
TheJulia | hmmmm | 15:57 |
dtantsur | which makes _prepare_call a suspect | 15:58 |
dtantsur | except that.. I don't see anything wrong with it, nor does the CI expose any clue | 15:59 |
TheJulia | dtantsur: did it still look like it actually put the message on the rpc bus? | 16:03 |
dtantsur | well, at least the timeout happened somewhere in oslo.messaging code | 16:03 |
dtantsur | rabbitmq logs are less than helpful, but they do show connections from ironic stuff | 16:04 |
TheJulia | okay | 16:04 |
TheJulia | how would this impact the reading side... | 16:04 |
TheJulia | this is bizzar | 16:04 |
dtantsur | yeah | 16:06 |
dtantsur | need to go, see you tomorrow o/ | 16:11 |
arne_wiebalck | Ops 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 do | 16:42 |
*** sshnaidm is now known as sshnaidm|afk | 16:46 | |
rpittau | good night! o/ | 16:59 |
TheJulia | arne_wiebalck: a recipe... I thought you could chagne the interface on a deployed node | 17:01 |
jingvar | Hi! | 17:05 |
jingvar | I have issues with manual installation Bifrost on Debian (Buster/ bullseye ) | 17:07 |
jingvar | there are a lot of unmet dependencies | 17:08 |
jingvar | like libc6-dev : Depends: libc6 (= 2.28-10) | 17:08 |
jingvar | is it ok? | 17:08 |
arne_wiebalck | TheJulia: I don't think you can. | 17:18 |
arne_wiebalck | TheJulia: Hmm, you can move it to maintenance, and then set it :) | 17:20 |
arne_wiebalck | TheJulia: I guess that works. | 17:20 |
arne_wiebalck | TheJulia: Yeah, it does :) | 17:21 |
TheJulia | tjat was wjat O was tjomlomg | 17:27 |
TheJulia | jingvar: Well, not exactly expected but makes sense since some of the python modules it installs requires compilation | 17:27 |
jingvar | It unxpected for me. I thought Debian is not Stream :) | 17:29 |
*** sshnaidm|afk is now known as sshnaidm | 17:29 | |
*** gmann is now known as gmann_afk | 17:34 | |
arne_wiebalck | TheJulia: 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 gmann | 18:53 | |
stevebaker[m] | morning ironic! | 20:00 |
TheJulia | stevebaker[m]: good morning! | 20:02 |
TheJulia | arne_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 |
TheJulia | arne_wiebalck: if so, yeah, just a db update should work :\ | 20:03 |
arne_wiebalck | TheJulia: Oh, yes: we have a couple of thousand nodes with direct already. | 20:06 |
arne_wiebalck | TheJulia: We were just thinking it is time now to move the rest, before we move to the next upgrade. | 20:07 |
arne_wiebalck | TheJulia: Ok, thanks! | 20:08 |
TheJulia | arne_wiebalck: oh good, you had me *super* worried there for a minute | 20:24 |
TheJulia | or five ;) | 20:24 |
arne_wiebalck | TheJulia: heh, sorry :) | 20:39 |
opendevreview | Julia Kreger proposed openstack/sushy master: Raise an AccessError with SessionService init https://review.opendev.org/c/openstack/sushy/+/820076 | 22:10 |
opendevreview | Julia Kreger proposed openstack/ironic master: WIP: nuke the redfish cache on errors https://review.opendev.org/c/openstack/ironic/+/820077 | 22:21 |
opendevreview | Julia Kreger proposed openstack/sushy master: Raise an AccessError with SessionService init https://review.opendev.org/c/openstack/sushy/+/820076 | 22:22 |
TheJulia | stevebaker[m]: ^^ I... think that would do it | 22:22 |
TheJulia | I tried to make the story fairly detailed, I guess we shall see what CI comes back with | 22:22 |
NobodyCam | so we're currently running IPA on a CentOs8 based image. is the suggestion to move to stream? | 22:26 |
TheJulia | NobodyCam: in the grand scheme of things, yeah | 22:54 |
TheJulia | NobodyCam: 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 part | 22:55 |
TheJulia | just keep a backup copy of the last known good thing | 22:55 |
TheJulia | but I don't think anything has happened recently | 22:55 |
NobodyCam | ++ good to know | 22:55 |
NobodyCam | Thank you TheJulia !!! | 22:55 |
TheJulia | like, the last thing we hit was a slight change in grub as it jumped from 2.02 to 2.04 | 22:56 |
stevebaker[m] | ok, I've refreshed my nova deleting/cleaning change https://review.opendev.org/c/openstack/nova/+/813729 | 23:17 |
stevebaker[m] | arne_wiebalck: ^^ | 23:17 |
*** pmannidi is now known as pmannidi|Interview | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!