*** dsneddon has joined #openstack-ironic | 00:45 | |
*** dsneddon has quit IRC | 00:49 | |
*** altlogbot_0 has quit IRC | 00:50 | |
*** oyrogerg has joined #openstack-ironic | 00:52 | |
*** altlogbot_0 has joined #openstack-ironic | 00:54 | |
*** altlogbot_0 has quit IRC | 00:54 | |
*** altlogbot_2 has joined #openstack-ironic | 00:55 | |
*** ileixe has quit IRC | 01:03 | |
*** dsneddon has joined #openstack-ironic | 01:46 | |
*** dsneddon has quit IRC | 01:50 | |
*** ijw has quit IRC | 01:57 | |
*** ijw has joined #openstack-ironic | 02:10 | |
*** v12aml has quit IRC | 02:10 | |
*** ijw has quit IRC | 02:18 | |
*** dsneddon has joined #openstack-ironic | 02:46 | |
*** dsneddon has quit IRC | 02:51 | |
*** ijw has joined #openstack-ironic | 03:00 | |
*** whoami-rajat has joined #openstack-ironic | 03:05 | |
*** stendulker has joined #openstack-ironic | 03:37 | |
*** dsneddon has joined #openstack-ironic | 03:47 | |
*** dsneddon has quit IRC | 03:52 | |
*** itlinux has joined #openstack-ironic | 04:10 | |
*** stendulker has quit IRC | 04:20 | |
*** dsneddon has joined #openstack-ironic | 04:48 | |
*** ijw has quit IRC | 04:49 | |
*** dsneddon has quit IRC | 04:53 | |
*** e0ne has joined #openstack-ironic | 05:16 | |
*** ijw has joined #openstack-ironic | 05:26 | |
*** ijw has quit IRC | 05:33 | |
*** dsneddon has joined #openstack-ironic | 05:49 | |
*** dsneddon has quit IRC | 05:54 | |
*** ijw has joined #openstack-ironic | 06:01 | |
*** mkrai has joined #openstack-ironic | 06:02 | |
*** rachit7 has joined #openstack-ironic | 06:03 | |
*** ijw has quit IRC | 06:05 | |
*** rachit7 has quit IRC | 06:09 | |
*** rachit7 has joined #openstack-ironic | 06:10 | |
*** pcaruana has joined #openstack-ironic | 06:16 | |
rachit7 | Hey Guys.. | 06:22 |
---|---|---|
rachit7 | Anybody know how to get out node from locked state to perform cleaning steps in ironic???..i am just getting hands on ironic so sorry for asking noob question | 06:29 |
*** e0ne has quit IRC | 06:29 | |
rachit7 | pcaruana: any idea? | 06:29 |
*** ijw has joined #openstack-ironic | 06:36 | |
rachit7 | Anybody know how to get out node from locked state to perform cleaning steps in ironic???..i am just getting hands on ironic so sorry for asking noob question | 06:36 |
*** ijw has quit IRC | 06:40 | |
*** dsneddon has joined #openstack-ironic | 06:50 | |
*** dsneddon has quit IRC | 06:54 | |
arne_wiebalck | good morning, ironic! | 06:57 |
rachit7 | arne_wiebalck: Good Morning | 06:59 |
arne_wiebalck | rachit o/ | 06:59 |
rachit7 | arne_wiebalck: I have one question | 06:59 |
arne_wiebalck | rachit7: o/ | 06:59 |
arne_wiebalck | rachit7: tell me | 06:59 |
rachit7 | how to get out node from locked state to perform cleaning steps in ironic???..i am just getting hands on ironic so sorry for asking noob question | 06:59 |
rachit7 | arne_wiebalck: can you help me on this | 07:00 |
arne_wiebalck | what do you mean by locked ? | 07:00 |
*** dsneddon has joined #openstack-ironic | 07:00 | |
rachit7 | arne_wiebalck: ironic conductor locked the node and does not release the lock and due to that i am not able to perform clean_steps (for delete_configuration) | 07:01 |
rachit7 | arne_wiebalck: it keeps giving me log that node is still in clean_wait state | 07:02 |
arne_wiebalck | is maintenance true for this host? | 07:02 |
rachit7 | arne_wiebalck: no its in False state and before performing clean_steps on particular node,i put the node in available state | 07:03 |
arne_wiebalck | rachit7: clean_wait means the conductor is waiting for the node to report back on cleaning progress ... so what does the node do (on the console)? | 07:05 |
*** rcernin has quit IRC | 07:06 | |
rachit7 | arne_wiebalck: report back means it should call resume cleaning? | 07:07 |
arne_wiebalck | rachit7: report back means the control is with the IPA on the node atm, and the conductor won't move until it hears back from the node | 07:09 |
arne_wiebalck | rachit7: so, I'd check the node to see what the IPA is doing | 07:09 |
arne_wiebalck | rachit7: hence also my first question: if the node is in maintenance, the IPA will stall | 07:10 |
arne_wiebalck | rachit7: and the conductor will wait | 07:10 |
rachit7 | arne_wiebalck: so how can i check log for IPA? | 07:12 |
arne_wiebalck | rachit7: best way is to log into the node | 07:13 |
arne_wiebalck | rachit7: if you can | 07:13 |
* arne_wiebalck has a meeting now, back in 30 mins or so | 07:14 | |
rachit7 | arne_wiebalck: okay | 07:15 |
rachit7 | arne_wiebalck: thanks | 07:15 |
*** tesseract has joined #openstack-ironic | 07:17 | |
*** rpittau|afk is now known as rpittau | 07:18 | |
rpittau | good morning ironic! o/ | 07:18 |
*** iurygregory has joined #openstack-ironic | 07:26 | |
iurygregory | good morning | 07:26 |
rpittau | hi iurygregory :) | 07:26 |
iurygregory | o/ | 07:28 |
*** tssurya has joined #openstack-ironic | 07:31 | |
*** itlinux has quit IRC | 07:35 | |
mkrai | Hi Ironicers! | 07:43 |
iurygregory | morning mkrai | 07:47 |
mkrai | morning iurygregory :) | 07:47 |
*** lucasagomes has joined #openstack-ironic | 07:54 | |
*** ijw has joined #openstack-ironic | 07:57 | |
arne_wiebalck | mkrai rpittau iurygregory: good morning! | 07:58 |
iurygregory | arne_wiebalck, o/ | 07:58 |
rpittau | hi arne_wiebalck :) | 07:59 |
mkrai | Hi, if I am not able to ping the host IP from the neutron dhcp network namespace, what should I check? | 08:01 |
*** ijw has quit IRC | 08:01 | |
mkrai | I see that the node gets DHCP server IP correct and tftp too, but it connection timeout on contact host for files | 08:02 |
*** dsneddon has quit IRC | 08:03 | |
*** dteselkin has joined #openstack-ironic | 08:04 | |
rpittau | mkrai: that sounds like firewall issues to me | 08:11 |
mkrai | rpittau: let me check | 08:12 |
*** dsneddon has joined #openstack-ironic | 08:15 | |
*** dougsz has joined #openstack-ironic | 08:18 | |
*** dsneddon has quit IRC | 08:20 | |
mkrai | rpittau: firewall is disabled | 08:21 |
*** rachit7 has quit IRC | 08:24 | |
*** e0ne has joined #openstack-ironic | 08:32 | |
*** derekh has joined #openstack-ironic | 08:48 | |
openstackgerrit | Arne Wiebalck proposed openstack/ironic-python-agent master: Software RAID: Create/delete configurations https://review.opendev.org/634681 | 08:55 |
openstackgerrit | Arne Wiebalck proposed openstack/ironic-python-agent master: Software RAID: Add IPA deploy support https://review.opendev.org/639390 | 08:55 |
*** rachit7 has joined #openstack-ironic | 09:01 | |
*** oyrogerg has quit IRC | 09:04 | |
dtantsur | morning ironic! | 09:10 |
dtantsur | mkrai: hi! re wsme->flask, I don't think it needs a spec since there should not be user-visible changes. An RFE is needed though. | 09:10 |
mkrai | morning dtantsur | 09:11 |
mkrai | dtantsur: Ok, is there an RFE already? | 09:11 |
dtantsur | mkrai: not from me at least. you can try looking for it in storyboard. | 09:12 |
mkrai | dtantsur: ok sure, if not I will create one. | 09:12 |
dtantsur | thanks! | 09:12 |
dtantsur | mkrai: if you need to check some existing services, keystone and ironic-inspector use flask | 09:12 |
dtantsur | I'm open for review/help with this topic | 09:13 |
mkrai | dtantsur: sure, thanks! | 09:14 |
dtantsur | rpittau: hi! do you know if the gate status on the whiteboard is up-to-date? if yes, how can I help? | 09:15 |
*** dsneddon has joined #openstack-ironic | 09:16 | |
rpittau | dtantsur: that should be up-to-date, we should add the change to remove the metadata_csum option to the Debian stretch one, or disable coreos jobs temporary in stable branches, merge stretch + metadata_csum, then re-enable coreos jobs | 09:17 |
dtantsur | rpittau: I'd go the first path, are there any problems with it? | 09:18 |
rpittau | for example, this https://review.opendev.org/657353 + this https://review.opendev.org/653465 | 09:18 |
patchbot | patch 657353 - ironic (stable/rocky) - Disable metadata_csum when creating ext4 filesystems - 1 patch set | 09:18 |
patchbot | patch 653465 - ironic-python-agent (stable/rocky) - Switch to Debian Stretch for building CoreOS images - 1 patch set | 09:18 |
rpittau | dtantsur: the "only" issue is that the changes are inter-dependent, to be able to pass we need both | 09:18 |
rpittau | dtantsur: apologies, the real one for metadata_csum is this https://review.opendev.org/657394 | 09:20 |
patchbot | patch 657394 - ironic-python-agent - Do not use metadata checksums with ext4 - 1 patch set | 09:20 |
dtantsur | I think the ironic ones are also worth approving (doing it now) | 09:21 |
*** dsneddon has quit IRC | 09:21 | |
rpittau | yeah | 09:21 |
dtantsur | rpittau: re https://review.opendev.org/#/c/657394/ do you know why master and stein pass without it? is it because of newer cirros or ...? | 09:21 |
patchbot | patch 657394 - ironic-python-agent - Do not use metadata checksums with ext4 - 1 patch set | 09:21 |
rpittau | dtantsur: we don't have coreos jobs anymore in stein or master :/ | 09:22 |
dtantsur | rpittau: there is one non-voting job, and it seems to pass.. | 09:22 |
rpittau | I think we have only one non-voting | 09:22 |
dtantsur | rpittau: anyway, could you propose backports of https://review.opendev.org/#/c/657394/ to check that it actually fixes the stable jobs? | 09:23 |
patchbot | patch 657394 - ironic-python-agent - Do not use metadata checksums with ext4 - 1 patch set | 09:23 |
rpittau | dtantsur: can you please point me to the change where the coreos job is passing ? | 09:23 |
rpittau | dtantsur: will do | 09:23 |
dtantsur | rpittau: https://review.opendev.org/#/c/656275/ has a run from May 5th | 09:24 |
patchbot | patch 656275 - ironic-python-agent - Add more channel number for detecting BMC IP address (MERGED) - 6 patch sets | 09:24 |
dtantsur | thx! | 09:24 |
dtantsur | do I get it right that only IPA stable is broken atm? The whiteboard makes it look like everything is RED | 09:24 |
rpittau | dtantsur: the problem is that it will not pass without debian stretch, also please this test that verifies the job is fixed https://review.opendev.org/656507 | 09:24 |
patchbot | patch 656507 - ironic-python-agent (stable/rocky) - [WIP|DNM] testing coreos direct legacy job - 7 patch sets | 09:24 |
*** mgoddard has joined #openstack-ironic | 09:25 | |
dtantsur | rpittau: right. then we have to squash the backports :( | 09:25 |
rpittau | dtantsur: yes, only IPA stable <= rocky is broken | 09:25 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent stable/stein: Do not use metadata checksums with ext4 https://review.opendev.org/658770 | 09:27 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent stable/rocky: Do not use metadata checksums with ext4 https://review.opendev.org/658771 | 09:27 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent stable/queens: Do not use metadata checksums with ext4 https://review.opendev.org/658772 | 09:27 |
openstackgerrit | Riccardo Pittau proposed openstack/ironic-python-agent stable/pike: Do not use metadata checksums with ext4 https://review.opendev.org/658773 | 09:28 |
*** dsneddon has joined #openstack-ironic | 09:30 | |
openstackgerrit | Merged openstack/ironic master: Update Python 3 test runtimes for Train https://review.opendev.org/657935 | 09:44 |
mkrai | dtantsur: Can you please review the allocation update API patches whenever you can? | 09:54 |
mkrai | dtantsur: https://review.opendev.org/#/c/653491/ | 09:54 |
patchbot | patch 653491 - ironic - Add API to allow update allocation name and extra ... - 6 patch sets | 09:54 |
*** ijw has joined #openstack-ironic | 10:01 | |
*** ijw has quit IRC | 10:05 | |
dtantsur | mkrai: I'm just back from 2 weeks of absence, will get to everything in its turn :) | 10:13 |
mkrai | dtantsur: yes sure, whenever you've time. No hurry :) | 10:13 |
*** dsneddon has quit IRC | 10:33 | |
openstackgerrit | Merged openstack/bifrost master: Rename review.openstack.org to review.opendev.org https://review.opendev.org/658551 | 10:48 |
*** ijw has joined #openstack-ironic | 10:58 | |
hjensas | I'd like to abandon, https://review.opendev.org/576873 it's being replaced by https://review.opendev.org/658787. I don't have the permissions, as it's initially vdrok's change. | 10:59 |
patchbot | patch 576873 - networking-baremetal - Enable ironic notifications - 19 patch sets | 10:59 |
patchbot | patch 658787 - neutron - Notify ironic on port status changes - 1 patch set | 10:59 |
*** ijw has quit IRC | 11:02 | |
vdrok | hjensas: I can do it if you wish | 11:03 |
hjensas | vdrok: thanks! | 11:03 |
openstackgerrit | Merged openstack/ironic stable/rocky: Disable metadata_csum when creating ext4 filesystems https://review.opendev.org/657353 | 11:21 |
openstackgerrit | Merged openstack/ironic stable/queens: Disable metadata_csum when creating ext4 filesystems https://review.opendev.org/657355 | 11:21 |
openstackgerrit | Merged openstack/ironic stable/pike: Disable metadata_csum when creating ext4 filesystems https://review.opendev.org/657356 | 11:23 |
*** ijw has joined #openstack-ironic | 11:34 | |
*** ijw has quit IRC | 11:38 | |
*** jcoufal has joined #openstack-ironic | 11:56 | |
*** dsneddon has joined #openstack-ironic | 12:00 | |
*** rh-jelabarre has joined #openstack-ironic | 12:00 | |
*** lennyb has quit IRC | 12:12 | |
*** seba has joined #openstack-ironic | 12:51 | |
*** pcaruana|afk| has joined #openstack-ironic | 12:55 | |
*** pcaruana has quit IRC | 12:57 | |
jroll | morning everyone | 12:58 |
*** dirk has quit IRC | 12:58 | |
*** bfournie has quit IRC | 12:58 | |
dtantsur | morning jroll | 12:58 |
*** aprice has quit IRC | 12:59 | |
*** rpittau has quit IRC | 12:59 | |
*** aprice has joined #openstack-ironic | 12:59 | |
*** dmellado_ has joined #openstack-ironic | 12:59 | |
*** vdrok has quit IRC | 12:59 | |
*** vdrok has joined #openstack-ironic | 13:00 | |
*** persia has quit IRC | 13:00 | |
*** dmellado has quit IRC | 13:00 | |
*** dmellado_ is now known as dmellado | 13:00 | |
*** persia has joined #openstack-ironic | 13:00 | |
*** dirk has joined #openstack-ironic | 13:00 | |
*** rpittau has joined #openstack-ironic | 13:02 | |
*** Goneri has joined #openstack-ironic | 13:02 | |
*** dsneddon has quit IRC | 13:03 | |
*** rloo has joined #openstack-ironic | 13:22 | |
dtantsur | jroll: do you remember why https://review.opendev.org/#/c/563722/ wasn't backported further? | 13:39 |
patchbot | patch 563722 - nova - ironic: add instance_uuid before any other spawn a... (MERGED) - 11 patch sets | 13:39 |
jroll | dtantsur: dunno, I only put it up as a "we should fix it this way" and then TheJulia took over | 13:41 |
dtantsur | k | 13:43 |
dtantsur | oh, it's adding new driver calls | 13:43 |
jroll | certainly possible to deal with in the stable branches though | 13:44 |
jroll | oh maybe not | 13:44 |
jroll | er, yeah, they have a base implementation, I would think that would be ok | 13:44 |
*** priteau has joined #openstack-ironic | 13:53 | |
*** ijw has joined #openstack-ironic | 14:03 | |
*** gmann has joined #openstack-ironic | 14:06 | |
*** gmann is now known as gmann_pto | 14:07 | |
*** ijw has quit IRC | 14:09 | |
TheJulia | jroll: dtantsur: I totally remember nothing at the moment :( | 14:09 |
*** ijw has joined #openstack-ironic | 14:10 | |
dtantsur | morning TheJulia, how are you? | 14:10 |
TheJulia | Fighting RH summit plague and pondering escaping the airport lounge to go find real breakfast this morning before I board a flight for vacation | 14:11 |
TheJulia | Emotionally all over the place as well, but that is kind of to be expected. | 14:11 |
dtantsur | TheJulia: ++ good idea. yeah.. | 14:11 |
dtantsur | Are you going to chair the meeting or should I? | 14:12 |
TheJulia | dtantsur: please chair it and stand in for me this week. Next week I should be around on monday without any issues. I will need to run some errands during the day as well next week before I fly home to deal with my mom’s estate on Wednesday. I’ll be on next Wednesday in-flight wifi permitting. | 14:13 |
dtantsur | sure, I'll take care of things here | 14:14 |
TheJulia | <3 | 14:14 |
* dtantsur gets back to his expense reports :( | 14:14 | |
TheJulia | Ugh, I started mine in Boston, and yeeaaahh.... it will need to wait until I get back next week. | 14:14 |
openstackgerrit | Doug Hellmann proposed openstack/ironic-inspector master: clarify that extra hardware data does not always go to swift https://review.opendev.org/658826 | 14:16 |
*** ijw has quit IRC | 14:19 | |
*** ijw has joined #openstack-ironic | 14:20 | |
openstackgerrit | Doug Hellmann proposed openstack/ironic-python-agent master: report the URL when heartbeats fail https://review.opendev.org/658831 | 14:26 |
openstackgerrit | Doug Hellmann proposed openstack/ironic-python-agent master: show inspection callback url in error messages https://review.opendev.org/658833 | 14:32 |
openstackgerrit | melissaml proposed openstack/ironic-specs master: Rename review.openstack.org to review.opendev.org https://review.opendev.org/658556 | 14:34 |
*** cdearborn has joined #openstack-ironic | 14:43 | |
*** rachit7 has quit IRC | 14:45 | |
*** ijw has quit IRC | 14:49 | |
*** kaifeng has joined #openstack-ironic | 14:51 | |
*** ijw has joined #openstack-ironic | 14:51 | |
*** stendulker has joined #openstack-ironic | 14:54 | |
*** pcaruana|afk| has quit IRC | 14:55 | |
openstackgerrit | Doug Hellmann proposed openstack/ironic-inspector master: clarify that extra hardware data does not always go to swift https://review.opendev.org/658826 | 14:56 |
*** dsneddon has joined #openstack-ironic | 15:00 | |
*** beekneemech is now known as bnemec | 15:00 | |
*** ijw has quit IRC | 15:00 | |
*** ijw has joined #openstack-ironic | 15:01 | |
* etingof is wondering if we meet today...? | 15:01 | |
etingof | dtantsur, might be pulled too much into his reports ;) | 15:02 |
*** dsneddon has quit IRC | 15:05 | |
NobodyCam | Good Morning Ironic'ers | 15:09 |
NobodyCam | Hppy Monday | 15:09 |
arne_wiebalck | NobodyCam: o/ | 15:09 |
*** stendulker has quit IRC | 15:09 | |
NobodyCam | o/ Good morning arne_wiebalck | 15:09 |
rpittau | hey NobodyCam :) | 15:09 |
NobodyCam | good morning rpittau :) | 15:10 |
dtantsur | ugh, yeah, sorry | 15:10 |
dtantsur | #startmeeting ironic | 15:10 |
openstack | Meeting started Mon May 13 15:10:29 2019 UTC and is due to finish in 60 minutes. The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:10 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:10 |
*** openstack changes topic to " (Meeting topic: ironic)" | 15:10 | |
openstack | The meeting name has been set to 'ironic' | 15:10 |
etingof | \o/ | 15:10 |
dtantsur | hi all, sorry for the late start (first day after 2 weeks of absence, sigh) | 15:10 |
bdodd | o/ | 15:10 |
kaifeng | o/ | 15:10 |
rpittau | meeting! o/ | 15:10 |
hjensas | o/ | 15:10 |
arne_wiebalck | o/ | 15:10 |
jroll | morning :) | 15:11 |
dtantsur | our agenda is pretty light: | 15:11 |
dtantsur | #link https://wiki.openstack.org/wiki/Meetings/Ironic | 15:11 |
rloo | o/ | 15:11 |
dtantsur | #topic Announcements / Reminder | 15:11 |
*** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)" | 15:11 | |
dtantsur | #info The Summit is over, thanks all :) | 15:11 |
dtantsur | #link https://dtantsur.github.io/posts/ironic-denver-2019/ dtantsur's notes from the Summit + PTG in Denver | 15:11 |
dtantsur | Subjectively, a lot is going on around ironic | 15:12 |
arne_wiebalck | thanks for the notes dtantsur, that's useful | 15:12 |
rloo | thx dtantsur, for your notes on the summit/ptg! | 15:12 |
rpittau | thank you for the notes dtantsur :) | 15:12 |
dtantsur | #info The next Summit + PTG will be in Shanghai, November 4 - 6, and the PTG will be November 6 - 8, 2019 | 15:12 |
dtantsur | #link https://www.openstack.org/summit/shanghai-2019 | 15:13 |
*** ijw has quit IRC | 15:13 | |
dtantsur | if you've dreamt of going to China, this is your chance :) registration is open already | 15:13 |
rpittau | well I guess I can't miss 3 summits in a row :P | 15:14 |
dtantsur | heh | 15:14 |
dtantsur | I have mixed feelings about the next one :) | 15:14 |
dtantsur | anyway | 15:14 |
dtantsur | anything else to announce? | 15:14 |
dtantsur | hmm, maybe this one: | 15:14 |
dtantsur | #link https://www.openstack.org/bare-metal/ The official bare metal program | 15:15 |
cdearborn | o/ sorry i'm late! | 15:15 |
dtantsur | also you'll notice that we have Baremetal SIG business as part of our agenda | 15:15 |
dtantsur | anything else? | 15:15 |
mgoddard | o/ | 15:15 |
mjturek | o/ | 15:15 |
mkrai | o/ | 15:15 |
rajinir | o/ | 15:16 |
dtantsur | No action items from the last week, and I doubt we have anything new on the whiteboard (or do we?) | 15:16 |
dtantsur | I guess TheJulia will propose the priorities for Train officially next week | 15:17 |
baha | o/ | 15:17 |
dtantsur | those who like spoilers, check https://etherpad.openstack.org/p/DEN-train-ironic-ptg after line 639 | 15:17 |
dtantsur | * 539 | 15:17 |
dtantsur | #topic Deciding on priorities for the coming week | 15:18 |
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:18 | |
dtantsur | what do you think about the list as it is now? | 15:20 |
TheJulia | I didn't have time to update it, so I think it is okay to just carry it forward if it is up to date. | 15:21 |
TheJulia | Re priorities, next Monday most likely :( | 15:21 |
dtantsur | reasonable up-to-date, I just added the mdns thingy | 15:21 |
rpittau | I did update some stuff based on merged things last week | 15:21 |
etingof | let's review this as well please? -- https://review.opendev.org/655685 | 15:21 |
patchbot | patch 655685 - ironic-specs - Add spec for indicator management - 6 patch sets | 15:21 |
TheJulia | dtantsur: perfect :) | 15:21 |
dtantsur | etingof: fair enough, added | 15:22 |
dtantsur | any other comments? | 15:22 |
rloo | I haven't read the retirement spec (https://review.opendev.org/#/c/656799/) but we did discuss that at ptg, wrt beth's ask for other states? so wondering if more work is required in that spec. | 15:22 |
patchbot | patch 656799 - ironic-specs - Add support for node retirement - 4 patch sets | 15:22 |
*** ddurst- is now known as ddurst | 15:22 | |
arne_wiebalck | rloo: it's on the agenda :) | 15:22 |
dtantsur | rloo: yeah, we have this on the agenda, and I'd like it on priorities to highlight this discussion | 15:23 |
rloo | arne_wiebalck: oh, is that what 'needs follow-up from the PTG' is about? | 15:23 |
dtantsur | rloo: yes | 15:23 |
* etingof has 10-patch long chain against sushy-tools to review... | 15:23 | |
* dtantsur needs time and spoons to type letters about that spec | 15:23 | |
rloo | arne_wiebalck, dtantsur: ok, thx for clarifying | 15:23 |
dtantsur | arne_wiebalck: you can find some of the information in my summary | 15:23 |
arne_wiebalck | dtantsur: ok | 15:23 |
dtantsur | etingof: I'm still -1 on half of them, but ignoring in case somebody cares less.. | 15:23 |
* rloo thinks there are a lot of priorities for this week but i guess we had them all in previous weeks. | 15:24 | |
etingof | dtantsur, I've refactored them heavily since your last review | 15:24 |
dtantsur | okay, I'll check again if my concerns still hold | 15:24 |
etingof | though the things you did not like still there ;) | 15:24 |
dtantsur | rloo: exactly | 15:24 |
dtantsur | etingof: well, here we go :) do you really want me to review them? ;) | 15:25 |
etingof | dtantsur, absolutely! | 15:25 |
rloo | is cisco-ucs out? | 15:26 |
dtantsur | TheJulia: ^^? | 15:26 |
etingof | dtantsur, I hope that with my recent additions (chassis, indicators and vmedia boot emulation) the design I am proposing makes more sense... | 15:26 |
TheJulia | Yeah, pretty much. :( | 15:26 |
rloo | ok, i'm deleting that from vendor priorities then. | 15:27 |
dtantsur | etingof: if you found a real manager object in libvirt - yes, otherwise I expect a similar comment | 15:27 |
dtantsur | anyway, does the list look good? | 15:27 |
TheJulia | I did chat with ?ianw? In Denver and he stressed that we should do what is best for the project. | 15:28 |
dtantsur | sigh | 15:28 |
etingof | dtantsur, it's the other way round - if you find a real manager and chassis objects in libvirt, then your design makes sense ;) | 15:28 |
TheJulia | Yeah. :( | 15:28 |
TheJulia | Anyway, airplane mode time. | 15:29 |
dtantsur | #topic Baremetal SIG | 15:29 |
*** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)" | 15:29 | |
dtantsur | Anything for today? | 15:29 |
TheJulia | The white paper could use some assistance with regards to use cases | 15:29 |
rloo | dtantsur: as an intro -- did anyone mention why we have baremetal sig in this meeting? | 15:30 |
dtantsur | rloo: I did not, because I was not present when it was decided | 15:30 |
rpittau | when was that decided ? | 15:30 |
TheJulia | Tl;dr there needs to be a periodic reminder, so the meeting was the best time | 15:30 |
rloo | dtantsur: i can't recall now, if there was email/announcement about this. if there wasn't, there should be, or mention it here? | 15:31 |
TheJulia | During one of the baremetal sessions there was consensus that it would be good to at least raise as part of this meeting given overlaps. | 15:31 |
TheJulia | Chris hodge... appears to be offline at the moment, but this is largely for human wrangling, so if there are no items then we can skip past. | 15:32 |
rloo | fair enough, and that is fine with me. could we have an action item for someone (chris?) to send out email to both groups, about this being in the ironic meeting? | 15:33 |
rloo | (unless folks disagree with it) | 15:33 |
TheJulia | I concur, I won’t be able to do it this week. | 15:33 |
rloo | (since not everyone was at ptg/summit) | 15:33 |
TheJulia | Anyway, they just closed the boarding door, need to quite literally disconnect now :( | 15:33 |
arne_wiebalck | I can ping Chris. | 15:34 |
TheJulia | Have a wonderful day everyone | 15:34 |
rloo | BYE TheJulia! | 15:34 |
dtantsur | safe flight TheJulia | 15:34 |
rpittau | TheJulia: safe flight :) | 15:34 |
*** yolanda_ has quit IRC | 15:34 | |
dtantsur | arne_wiebalck: please! I think we can do it off-meeting. | 15:34 |
rloo | ok, AI for arne_wiebalck to ping chris to ask chris to send email out etc. | 15:34 |
dtantsur | ++ | 15:36 |
dtantsur | #topic RFE review | 15:36 |
*** openstack changes topic to "RFE review (Meeting topic: ironic)" | 15:36 | |
*** itlinux has joined #openstack-ironic | 15:36 | |
dtantsur | #link https://review.opendev.org/#/c/656799/ Support for node retirement / nodes in failure states | 15:36 |
patchbot | patch 656799 - ironic-specs - Add support for node retirement - 4 patch sets | 15:36 |
dtantsur | as I said, I did not have time to write more detailed thoughts - sorry for that | 15:37 |
dtantsur | but something that did come up in the room is that people want a new state more than a new flag | 15:37 |
arne_wiebalck | this spec is basically a summary of a discussion I had with jroll and TheJulia | 15:37 |
*** yolanda_ has joined #openstack-ironic | 15:37 | |
dtantsur | I was initially quite against it, but then got more or less convinced | 15:37 |
dtantsur | although this may be a difference between retiring nodes and "failing" them.. | 15:38 |
arne_wiebalck | how do you mark an active node for retirement when retired is a state? | 15:38 |
dtantsur | with a state transition that does not, however, tear it down | 15:38 |
arne_wiebalck | so instances can be on nodes in state retired? | 15:39 |
dtantsur | that's what people wanted for nodes at fault: keep it intact, but mark it as very broken | 15:39 |
dtantsur | yep | 15:39 |
arne_wiebalck | ok | 15:39 |
jroll | hmm | 15:39 |
dtantsur | which is going to confuse the heck out of nova, I suspect | 15:39 |
rpittau | interesting, it goes slightly against logic imho | 15:39 |
jroll | I'd love to see the proposed state machine | 15:39 |
jroll | er, state diagram | 15:40 |
arne_wiebalck | the idea behind the flag was to be very close to 'maintenace', an attribute you assign to a node | 15:40 |
dtantsur | the thing about maintenance is that it can happen in literally any state | 15:40 |
arne_wiebalck | and such a flag would not interfere with the state machine | 15:40 |
arne_wiebalck | retired as well | 15:40 |
dtantsur | does it make sense to retire a node in "enroll"? "cleaning"? "manageble"? | 15:40 |
rpittau | well in theory retirement too | 15:40 |
*** pramchan has joined #openstack-ironic | 15:40 | |
arne_wiebalck | dtantsur: I think so, why not? | 15:40 |
rloo | i think more thinking is needed here. wondering if we want a 'retire' state, AND some flag eg 'next-step' or 'next-phase' | 15:41 |
rpittau | I think it does | 15:41 |
dtantsur | arne_wiebalck: if you mark an available node as retired, how do you prevent nova from scheduling to it? | 15:41 |
arne_wiebalck | how about using the same mechanism 'maintenance' does? | 15:42 |
arne_wiebalck | the same way | 15:42 |
dtantsur | maintenance has been a part of our API since the inception | 15:42 |
dtantsur | so it's something everybody is used to checking | 15:42 |
dtantsur | older tooling would not take retired into account | 15:42 |
dtantsur | I think this ^^ should be in the spec even if we keep the flag approach | 15:43 |
arne_wiebalck | older tooling like ...? | 15:43 |
dtantsur | nova, metalsmith, metal3 are the tools I'm aware of | 15:43 |
arne_wiebalck | we could also set maintenance in addition to retired | 15:43 |
dtantsur | maintenance has side effects like preventing cleaning from working | 15:43 |
rloo | at what point is the node 'retired' ? | 15:44 |
arne_wiebalck | true, I take that back :) | 15:44 |
*** pcaruana|afk| has joined #openstack-ironic | 15:44 | |
rloo | we want the node to-be-retired, but when is it 'retired'? | 15:44 |
arne_wiebalck | rloo, it is not retired, it's more marked for retirement | 15:44 |
rloo | i'm having trouble understanding the state of the node in this. | 15:44 |
rloo | arne_wiebalck: from the spec, it seems like what you want is to indicate 'do not make this available' | 15:45 |
arne_wiebalck | rloo: right | 15:45 |
arne_wiebalck | I'd like to mark an 'active' node as on its way out | 15:45 |
rloo | arne_wiebalck: and (based on problem description) you want to be able to search/list nodes | 15:45 |
arne_wiebalck | rloo: correct | 15:46 |
dtantsur | arne_wiebalck: but not only active nodes, all states? | 15:46 |
* arne_wiebalck thinks rloo is preparing a suggestion | 15:46 | |
arne_wiebalck | dtantsur: yes | 15:46 |
rloo | so end user/nova invokes 'delete' on their instance. | 15:46 |
rloo | i'm not preparing a solution, just thinking about the problem. sorry. | 15:46 |
arne_wiebalck | dtantsur: also nodes in clean_failed may be retired | 15:47 |
arne_wiebalck | rloo: :) | 15:47 |
rloo | seems like one could want to 'retire' a node in any state. in some states, the operator could just do an 'openstack node delete', right, no need to sayd 'openstack node retire ...'. | 15:47 |
arne_wiebalck | rloo: correct | 15:48 |
dtantsur | yeah, this ^^ is something I'm not quite sure about. I understand we want to avoid unprovisioning an active node right away, but why keep nodes in available? | 15:48 |
rloo | wondering if it is sufficient to put in a mechanism to say 'do not make available'. but if we did that, it would not necessarily mean it is for retirement. | 15:48 |
arne_wiebalck | maintenance is very close | 15:48 |
arne_wiebalck | but has the cleaning issue | 15:48 |
rloo | wasn't there a case where someone might want to do firmware update after an instance is removed, so they don't want the cleaned node to go to available right away? | 15:49 |
*** diga has joined #openstack-ironic | 15:49 | |
rloo | (and firmware update isn't part of the cleaning) | 15:49 |
rloo | ie, they may want the node to go to mgt after cleaning, not available. | 15:49 |
rloo | do we need a 'retirement' state, or do we want a mechanism to move instance-deleted nodes from cleaning to some non-avail state like manage? | 15:50 |
arne_wiebalck | rloo: we don't necessarily need to change the state of the node I think | 15:51 |
*** pramchan has quit IRC | 15:51 | |
rloo | arne_wiebalck: so for you, you're good if you could indicate 'when deleting instance, do cleaning and go to manage (instead of available)', and then indicate via eg extra specs or something 'this is due for retirement'? | 15:52 |
*** pramchan has joined #openstack-ironic | 15:52 | |
rloo | i am not against a 'retire*' state or something, just trying to understand/generalize. | 15:52 |
arne_wiebalck | rloo: if it is easy extract the list of nodes in this state: yes | 15:53 |
arne_wiebalck | rloo: sounds good :) | 15:53 |
dtantsur | I wonder if this solution generalizes to the case discussed on the PTG | 15:53 |
arne_wiebalck | what way discussed at the ptg? | 15:53 |
rloo | now i'd like some more operator feedback (wrt explicit 'retire*' something). and dtantsur, yeah. | 15:53 |
arne_wiebalck | s/way/was/ | 15:53 |
dtantsur | "We detected that this node is broken; do not kill it right away, but mark as faulty" | 15:53 |
arne_wiebalck | sounds very close | 15:54 |
arne_wiebalck | rloo' suggestion seems to cover both, no? | 15:54 |
rpittau | although faulty sounds much more close to maintenance | 15:54 |
rloo | arne_wiebalck: i think so :) | 15:54 |
arne_wiebalck | question still is whether this should be a state | 15:55 |
rloo | rpittau: true. i think the idea was that 'faulty' was not quite the same as 'maintenance' and we shouldn't overload/use 'maintenance' for non-maintenance things. of course, we haven't really defined what 'maintenance' really means :) | 15:55 |
dtantsur | arne_wiebalck: it depends on how you want the existing API consumers to behave wrt retirement | 15:56 |
rloo | arne_wiebalck: right, that's what i wanted to hear from you. do you need/want these nodes-to-be-retired, to end up in some new state? | 15:56 |
dtantsur | (not only on that, but it's a big part of the question) | 15:56 |
dtantsur | a new state will force all tooling to handle it explicitly | 15:56 |
dtantsur | a new flag on a node will be ignored by all consumers we don't update explicitly | 15:56 |
rpittau | I totally agree with the non-abuse of maintenance, but I see 2 new states here, if we go down that road | 15:57 |
arne_wiebalck | dtantsur: for the consumers, I thought ironic gives the nodes rather than nova fetching the nodes, no? | 15:57 |
rloo | rpittau: i have nothing against new states -- as long as they make sense :) | 15:57 |
dtantsur | rpittau: maintenance is ruled out because it will prevent cleaning from working | 15:57 |
dtantsur | arne_wiebalck: nova fetches nodes with some filters | 15:58 |
arne_wiebalck | dtantsur: ah, ok | 15:59 |
arne_wiebalck | dtantsur: I thought ironic hides them from nova | 15:59 |
arne_wiebalck | the ones in maintenance | 15:59 |
dtantsur | I think it's done on the nova side | 16:00 |
dtantsur | arne_wiebalck: https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L886-L894 | 16:00 |
arne_wiebalck | ok ... that makes things more complicated ... in any case :) | 16:00 |
dtantsur | we're a bit out of time, let's move the discussion back to the spec? | 16:00 |
dtantsur | we need to cover existing API consumers impact, allocation API impact, etc | 16:00 |
dtantsur | I also left one comment about the API design | 16:01 |
dtantsur | thanks all! | 16:01 |
arne_wiebalck | ok, please comment on the spec then, thanks! | 16:01 |
dtantsur | #endmeeting | 16:01 |
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-dev mailing list." | 16:01 | |
openstack | Meeting ended Mon May 13 16:01:30 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-05-13-15.10.html | 16:01 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-05-13-15.10.txt | 16:01 |
openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-05-13-15.10.log.html | 16:01 |
*** lucasagomes has quit IRC | 16:01 | |
*** kaifeng has quit IRC | 16:07 | |
*** tesseract has quit IRC | 16:20 | |
*** rachit7 has joined #openstack-ironic | 16:21 | |
*** dtantsur is now known as dtantsur|afk | 16:23 | |
dtantsur|afk | c u | 16:23 |
rpittau | good night! o/ | 16:25 |
*** rpittau is now known as rpittau|afk | 16:25 | |
arne_wiebalck | bye everyone o/ | 16:28 |
*** whoami-rajat has quit IRC | 16:31 | |
*** whoami-rajat has joined #openstack-ironic | 16:36 | |
*** ijw has joined #openstack-ironic | 16:37 | |
*** priteau has quit IRC | 16:38 | |
*** e0ne has quit IRC | 16:42 | |
*** mgoddard has quit IRC | 16:50 | |
*** mgoddard has joined #openstack-ironic | 16:52 | |
*** dougsz has quit IRC | 16:56 | |
*** mgoddard has quit IRC | 16:57 | |
*** dsneddon has joined #openstack-ironic | 17:01 | |
*** dsneddon has quit IRC | 17:06 | |
*** med_ has joined #openstack-ironic | 17:18 | |
*** dsneddon has joined #openstack-ironic | 17:19 | |
*** mgoddard has joined #openstack-ironic | 17:29 | |
*** jroll has quit IRC | 17:34 | |
*** jroll has joined #openstack-ironic | 17:35 | |
*** derekh has quit IRC | 17:37 | |
*** ijw has quit IRC | 17:46 | |
*** ijw has joined #openstack-ironic | 17:47 | |
*** itlinux has quit IRC | 17:50 | |
*** itlinux has joined #openstack-ironic | 17:53 | |
*** ijw has quit IRC | 17:55 | |
*** ijw has joined #openstack-ironic | 17:55 | |
*** ijw has quit IRC | 18:03 | |
*** ijw has joined #openstack-ironic | 18:03 | |
*** ijw has quit IRC | 18:05 | |
*** ijw has joined #openstack-ironic | 18:06 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy master: Make Manager->Action field optional https://review.opendev.org/658886 | 18:10 |
openstackgerrit | Ilya Etingof proposed openstack/sushy master: Make Manager->Actions field optional https://review.opendev.org/658886 | 18:11 |
*** ijw has quit IRC | 18:13 | |
*** ijw has joined #openstack-ironic | 18:14 | |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: WIP: Add Redfish Virtual Media support https://review.opendev.org/638453 | 18:15 |
*** e0ne has joined #openstack-ironic | 18:17 | |
*** ijw has quit IRC | 18:20 | |
*** ijw has joined #openstack-ironic | 18:20 | |
*** e0ne has quit IRC | 18:54 | |
*** diga has quit IRC | 18:56 | |
*** ijw has quit IRC | 19:00 | |
*** ijw has joined #openstack-ironic | 19:01 | |
*** ijw has quit IRC | 19:05 | |
*** jcoufal has quit IRC | 19:05 | |
*** ijw has joined #openstack-ironic | 19:05 | |
*** ijw has quit IRC | 19:08 | |
*** ijw has joined #openstack-ironic | 19:09 | |
*** ijw has quit IRC | 19:12 | |
*** ijw has joined #openstack-ironic | 19:13 | |
*** ijw has quit IRC | 19:29 | |
*** ijw has joined #openstack-ironic | 19:29 | |
*** gmann_pto has quit IRC | 19:33 | |
*** persia has quit IRC | 19:35 | |
*** persia has joined #openstack-ironic | 19:36 | |
*** ijw has quit IRC | 19:47 | |
*** ijw has joined #openstack-ironic | 19:47 | |
*** rachit7 has quit IRC | 19:49 | |
*** rh-jelabarre has quit IRC | 19:50 | |
*** rh-jelabarre has joined #openstack-ironic | 19:53 | |
*** ijw has quit IRC | 19:54 | |
*** ijw has joined #openstack-ironic | 19:55 | |
*** ijw has quit IRC | 20:07 | |
*** ijw has joined #openstack-ironic | 20:08 | |
*** e0ne has joined #openstack-ironic | 20:14 | |
*** jungleboyj has joined #openstack-ironic | 20:25 | |
*** e0ne has quit IRC | 20:26 | |
*** itlinux has quit IRC | 20:32 | |
stevebaker | morning | 20:36 |
*** tssurya has quit IRC | 20:38 | |
*** pcaruana|afk| has quit IRC | 20:46 | |
*** derekh has joined #openstack-ironic | 20:59 | |
*** openstackgerrit has quit IRC | 21:09 | |
*** jtomasek has quit IRC | 21:17 | |
*** ijw has quit IRC | 21:29 | |
*** Goneri has quit IRC | 21:38 | |
*** whoami-rajat has quit IRC | 21:41 | |
*** ijw has joined #openstack-ironic | 21:51 | |
*** ijw has quit IRC | 22:06 | |
*** ijw has joined #openstack-ironic | 22:06 | |
*** ijw has quit IRC | 22:07 | |
*** ijw has joined #openstack-ironic | 22:09 | |
*** ijw has quit IRC | 22:11 | |
*** ijw has joined #openstack-ironic | 22:11 | |
*** jcoufal has joined #openstack-ironic | 22:22 | |
*** jcoufal has quit IRC | 22:26 | |
*** ijw has quit IRC | 22:27 | |
*** ijw has joined #openstack-ironic | 22:27 | |
*** ijw has quit IRC | 22:27 | |
*** ijw has joined #openstack-ironic | 22:28 | |
*** ijw has quit IRC | 22:28 | |
*** ijw has joined #openstack-ironic | 22:29 | |
*** ijw has quit IRC | 22:31 | |
*** ijw has joined #openstack-ironic | 22:32 | |
*** ijw has quit IRC | 22:33 | |
*** openstackgerrit has joined #openstack-ironic | 22:34 | |
openstackgerrit | Doug Hellmann proposed openstack/ironic-python-agent master: show inspection callback url in error messages https://review.opendev.org/658833 | 22:34 |
*** ijw has joined #openstack-ironic | 22:34 | |
*** ijw has quit IRC | 22:35 | |
*** ijw has joined #openstack-ironic | 22:35 | |
*** ijw has quit IRC | 22:36 | |
*** ijw has joined #openstack-ironic | 22:36 | |
*** ijw has quit IRC | 22:36 | |
*** ijw has joined #openstack-ironic | 22:37 | |
*** ijw has quit IRC | 22:39 | |
*** ijw has joined #openstack-ironic | 22:40 | |
*** ijw has quit IRC | 22:40 | |
*** ijw has joined #openstack-ironic | 22:41 | |
*** ijw has quit IRC | 22:53 | |
*** ijw has joined #openstack-ironic | 22:54 | |
*** ijw has quit IRC | 22:55 | |
*** ijw has joined #openstack-ironic | 22:56 | |
*** jcoufal has joined #openstack-ironic | 23:00 | |
*** ijw has quit IRC | 23:00 | |
*** ijw has joined #openstack-ironic | 23:00 | |
*** pramchan has quit IRC | 23:01 | |
*** jcoufal has quit IRC | 23:01 | |
*** rcernin has joined #openstack-ironic | 23:04 | |
*** ijw has quit IRC | 23:06 | |
*** rcernin has quit IRC | 23:07 | |
*** rcernin has joined #openstack-ironic | 23:07 | |
*** jcoufal has joined #openstack-ironic | 23:27 | |
*** derekh has quit IRC | 23:30 | |
*** jcoufal has quit IRC | 23:35 | |
*** cdearborn has quit IRC | 23:53 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!