Monday, 2019-05-13

*** dsneddon has joined #openstack-ironic00:45
*** dsneddon has quit IRC00:49
*** altlogbot_0 has quit IRC00:50
*** oyrogerg has joined #openstack-ironic00:52
*** altlogbot_0 has joined #openstack-ironic00:54
*** altlogbot_0 has quit IRC00:54
*** altlogbot_2 has joined #openstack-ironic00:55
*** ileixe has quit IRC01:03
*** dsneddon has joined #openstack-ironic01:46
*** dsneddon has quit IRC01:50
*** ijw has quit IRC01:57
*** ijw has joined #openstack-ironic02:10
*** v12aml has quit IRC02:10
*** ijw has quit IRC02:18
*** dsneddon has joined #openstack-ironic02:46
*** dsneddon has quit IRC02:51
*** ijw has joined #openstack-ironic03:00
*** whoami-rajat has joined #openstack-ironic03:05
*** stendulker has joined #openstack-ironic03:37
*** dsneddon has joined #openstack-ironic03:47
*** dsneddon has quit IRC03:52
*** itlinux has joined #openstack-ironic04:10
*** stendulker has quit IRC04:20
*** dsneddon has joined #openstack-ironic04:48
*** ijw has quit IRC04:49
*** dsneddon has quit IRC04:53
*** e0ne has joined #openstack-ironic05:16
*** ijw has joined #openstack-ironic05:26
*** ijw has quit IRC05:33
*** dsneddon has joined #openstack-ironic05:49
*** dsneddon has quit IRC05:54
*** ijw has joined #openstack-ironic06:01
*** mkrai has joined #openstack-ironic06:02
*** rachit7 has joined #openstack-ironic06:03
*** ijw has quit IRC06:05
*** rachit7 has quit IRC06:09
*** rachit7 has joined #openstack-ironic06:10
*** pcaruana has joined #openstack-ironic06:16
rachit7Hey Guys..06:22
rachit7Anybody 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 question06:29
*** e0ne has quit IRC06:29
rachit7pcaruana: any idea?06:29
*** ijw has joined #openstack-ironic06:36
rachit7Anybody 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 question06:36
*** ijw has quit IRC06:40
*** dsneddon has joined #openstack-ironic06:50
*** dsneddon has quit IRC06:54
arne_wiebalckgood morning, ironic!06:57
rachit7arne_wiebalck: Good Morning06:59
arne_wiebalckrachit o/06:59
rachit7arne_wiebalck: I have one question06:59
arne_wiebalckrachit7: o/06:59
arne_wiebalckrachit7: tell me06:59
rachit7how 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 question06:59
rachit7arne_wiebalck: can you help me on this07:00
arne_wiebalckwhat do you mean by locked ?07:00
*** dsneddon has joined #openstack-ironic07:00
rachit7arne_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
rachit7arne_wiebalck: it keeps giving me log that node is still in clean_wait state07:02
arne_wiebalckis maintenance true for this host?07:02
rachit7arne_wiebalck: no its in False state and before performing clean_steps on particular node,i put the node in available state07:03
arne_wiebalckrachit7: 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 IRC07:06
rachit7arne_wiebalck: report back means it should call resume cleaning?07:07
arne_wiebalckrachit7: 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 node07:09
arne_wiebalckrachit7: so, I'd check the node to see what the IPA is doing07:09
arne_wiebalckrachit7: hence also my first question: if the node is in maintenance, the IPA will stall07:10
arne_wiebalckrachit7: and the conductor will wait07:10
rachit7arne_wiebalck: so how can i check log for IPA?07:12
arne_wiebalckrachit7: best way is to log into the node07:13
arne_wiebalckrachit7: if you can07:13
* arne_wiebalck has a meeting now, back in 30 mins or so07:14
rachit7arne_wiebalck: okay07:15
rachit7arne_wiebalck: thanks07:15
*** tesseract has joined #openstack-ironic07:17
*** rpittau|afk is now known as rpittau07:18
rpittaugood morning ironic! o/07:18
*** iurygregory has joined #openstack-ironic07:26
iurygregorygood morning07:26
rpittauhi iurygregory :)07:26
iurygregoryo/07:28
*** tssurya has joined #openstack-ironic07:31
*** itlinux has quit IRC07:35
mkraiHi Ironicers!07:43
iurygregorymorning mkrai07:47
mkraimorning iurygregory :)07:47
*** lucasagomes has joined #openstack-ironic07:54
*** ijw has joined #openstack-ironic07:57
arne_wiebalckmkrai rpittau iurygregory: good morning!07:58
iurygregoryarne_wiebalck, o/07:58
rpittauhi arne_wiebalck :)07:59
mkraiHi, 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 IRC08:01
mkraiI see that the node gets DHCP server IP correct and tftp too, but it connection timeout on contact host for files08:02
*** dsneddon has quit IRC08:03
*** dteselkin has joined #openstack-ironic08:04
rpittaumkrai: that sounds like firewall issues to me08:11
mkrairpittau: let me check08:12
*** dsneddon has joined #openstack-ironic08:15
*** dougsz has joined #openstack-ironic08:18
*** dsneddon has quit IRC08:20
mkrairpittau: firewall is disabled08:21
*** rachit7 has quit IRC08:24
*** e0ne has joined #openstack-ironic08:32
*** derekh has joined #openstack-ironic08:48
openstackgerritArne Wiebalck proposed openstack/ironic-python-agent master: Software RAID: Create/delete configurations  https://review.opendev.org/63468108:55
openstackgerritArne Wiebalck proposed openstack/ironic-python-agent master: Software RAID: Add IPA deploy support  https://review.opendev.org/63939008:55
*** rachit7 has joined #openstack-ironic09:01
*** oyrogerg has quit IRC09:04
dtantsurmorning ironic!09:10
dtantsurmkrai: 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
mkraimorning dtantsur09:11
mkraidtantsur: Ok, is there an RFE already?09:11
dtantsurmkrai: not from me at least. you can try looking for it in storyboard.09:12
mkraidtantsur: ok sure, if not I will create one.09:12
dtantsurthanks!09:12
dtantsurmkrai: if you need to check some existing services, keystone and ironic-inspector use flask09:12
dtantsurI'm open for review/help with this topic09:13
mkraidtantsur: sure, thanks!09:14
dtantsurrpittau: 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-ironic09:16
rpittaudtantsur: 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 jobs09:17
dtantsurrpittau: I'd go the first path, are there any problems with it?09:18
rpittaufor example, this https://review.opendev.org/657353 + this https://review.opendev.org/65346509:18
patchbotpatch 657353 - ironic (stable/rocky) - Disable metadata_csum when creating ext4 filesystems - 1 patch set09:18
patchbotpatch 653465 - ironic-python-agent (stable/rocky) - Switch to Debian Stretch for building CoreOS images - 1 patch set09:18
rpittaudtantsur: the "only" issue is that the changes are inter-dependent, to be able to pass we need both09:18
rpittaudtantsur: apologies, the real one for metadata_csum is this https://review.opendev.org/65739409:20
patchbotpatch 657394 - ironic-python-agent - Do not use metadata checksums with ext4 - 1 patch set09:20
dtantsurI think the ironic ones are also worth approving (doing it now)09:21
*** dsneddon has quit IRC09:21
rpittauyeah09:21
dtantsurrpittau: 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
patchbotpatch 657394 - ironic-python-agent - Do not use metadata checksums with ext4 - 1 patch set09:21
rpittaudtantsur: we don't have coreos jobs anymore in stein or master :/09:22
dtantsurrpittau: there is one non-voting job, and it seems to pass..09:22
rpittauI think we have only one non-voting09:22
dtantsurrpittau: anyway, could you propose backports of https://review.opendev.org/#/c/657394/ to check that it actually fixes the stable jobs?09:23
patchbotpatch 657394 - ironic-python-agent - Do not use metadata checksums with ext4 - 1 patch set09:23
rpittaudtantsur: can you please point me to the change where the coreos job is passing ?09:23
rpittaudtantsur: will do09:23
dtantsurrpittau: https://review.opendev.org/#/c/656275/ has a run from May 5th09:24
patchbotpatch 656275 - ironic-python-agent - Add more channel number for detecting BMC IP address (MERGED) - 6 patch sets09:24
dtantsurthx!09:24
dtantsurdo I get it right that only IPA stable is broken atm? The whiteboard makes it look like everything is RED09:24
rpittaudtantsur: 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/65650709:24
patchbotpatch 656507 - ironic-python-agent (stable/rocky) - [WIP|DNM] testing coreos direct legacy job - 7 patch sets09:24
*** mgoddard has joined #openstack-ironic09:25
dtantsurrpittau: right. then we have to squash the backports :(09:25
rpittaudtantsur: yes, only IPA stable <= rocky is broken09:25
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent stable/stein: Do not use metadata checksums with ext4  https://review.opendev.org/65877009:27
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent stable/rocky: Do not use metadata checksums with ext4  https://review.opendev.org/65877109:27
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent stable/queens: Do not use metadata checksums with ext4  https://review.opendev.org/65877209:27
openstackgerritRiccardo Pittau proposed openstack/ironic-python-agent stable/pike: Do not use metadata checksums with ext4  https://review.opendev.org/65877309:28
*** dsneddon has joined #openstack-ironic09:30
openstackgerritMerged openstack/ironic master: Update Python 3 test runtimes for Train  https://review.opendev.org/65793509:44
mkraidtantsur: Can you please review the allocation update API patches whenever you can?09:54
mkraidtantsur: https://review.opendev.org/#/c/653491/09:54
patchbotpatch 653491 - ironic - Add API to allow update allocation name and extra ... - 6 patch sets09:54
*** ijw has joined #openstack-ironic10:01
*** ijw has quit IRC10:05
dtantsurmkrai: I'm just back from 2 weeks of absence, will get to everything in its turn :)10:13
mkraidtantsur: yes sure, whenever you've time. No hurry :)10:13
*** dsneddon has quit IRC10:33
openstackgerritMerged openstack/bifrost master: Rename review.openstack.org to review.opendev.org  https://review.opendev.org/65855110:48
*** ijw has joined #openstack-ironic10:58
hjensasI'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
patchbotpatch 576873 - networking-baremetal - Enable ironic notifications - 19 patch sets10:59
patchbotpatch 658787 - neutron - Notify ironic on port status changes - 1 patch set10:59
*** ijw has quit IRC11:02
vdrokhjensas: I can do it if you wish11:03
hjensasvdrok: thanks!11:03
openstackgerritMerged openstack/ironic stable/rocky: Disable metadata_csum when creating ext4 filesystems  https://review.opendev.org/65735311:21
openstackgerritMerged openstack/ironic stable/queens: Disable metadata_csum when creating ext4 filesystems  https://review.opendev.org/65735511:21
openstackgerritMerged openstack/ironic stable/pike: Disable metadata_csum when creating ext4 filesystems  https://review.opendev.org/65735611:23
*** ijw has joined #openstack-ironic11:34
*** ijw has quit IRC11:38
*** jcoufal has joined #openstack-ironic11:56
*** dsneddon has joined #openstack-ironic12:00
*** rh-jelabarre has joined #openstack-ironic12:00
*** lennyb has quit IRC12:12
*** seba has joined #openstack-ironic12:51
*** pcaruana|afk| has joined #openstack-ironic12:55
*** pcaruana has quit IRC12:57
jrollmorning everyone12:58
*** dirk has quit IRC12:58
*** bfournie has quit IRC12:58
dtantsurmorning jroll12:58
*** aprice has quit IRC12:59
*** rpittau has quit IRC12:59
*** aprice has joined #openstack-ironic12:59
*** dmellado_ has joined #openstack-ironic12:59
*** vdrok has quit IRC12:59
*** vdrok has joined #openstack-ironic13:00
*** persia has quit IRC13:00
*** dmellado has quit IRC13:00
*** dmellado_ is now known as dmellado13:00
*** persia has joined #openstack-ironic13:00
*** dirk has joined #openstack-ironic13:00
*** rpittau has joined #openstack-ironic13:02
*** Goneri has joined #openstack-ironic13:02
*** dsneddon has quit IRC13:03
*** rloo has joined #openstack-ironic13:22
dtantsurjroll: do you remember why https://review.opendev.org/#/c/563722/ wasn't backported further?13:39
patchbotpatch 563722 - nova - ironic: add instance_uuid before any other spawn a... (MERGED) - 11 patch sets13:39
jrolldtantsur: dunno, I only put it up as a "we should fix it this way" and then TheJulia took over13:41
dtantsurk13:43
dtantsuroh, it's adding new driver calls13:43
jrollcertainly possible to deal with in the stable branches though13:44
jrolloh maybe not13:44
jroller, yeah, they have a base implementation, I would think that would be ok13:44
*** priteau has joined #openstack-ironic13:53
*** ijw has joined #openstack-ironic14:03
*** gmann has joined #openstack-ironic14:06
*** gmann is now known as gmann_pto14:07
*** ijw has quit IRC14:09
TheJuliajroll: dtantsur: I totally remember nothing at the moment :(14:09
*** ijw has joined #openstack-ironic14:10
dtantsurmorning TheJulia, how are you?14:10
TheJuliaFighting RH summit plague and pondering escaping the airport lounge to go find real breakfast this morning before I board a flight for vacation14:11
TheJuliaEmotionally all over the place as well, but that is kind of to be expected.14:11
dtantsurTheJulia: ++ good idea. yeah..14:11
dtantsurAre you going to chair the meeting or should I?14:12
TheJuliadtantsur: 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
dtantsursure, I'll take care of things here14:14
TheJulia<314:14
* dtantsur gets back to his expense reports :(14:14
TheJuliaUgh, I started mine in Boston, and yeeaaahh.... it will need to wait until I get back next week.14:14
openstackgerritDoug Hellmann proposed openstack/ironic-inspector master: clarify that extra hardware data does not always go to swift  https://review.opendev.org/65882614:16
*** ijw has quit IRC14:19
*** ijw has joined #openstack-ironic14:20
openstackgerritDoug Hellmann proposed openstack/ironic-python-agent master: report the URL when heartbeats fail  https://review.opendev.org/65883114:26
openstackgerritDoug Hellmann proposed openstack/ironic-python-agent master: show inspection callback url in error messages  https://review.opendev.org/65883314:32
openstackgerritmelissaml proposed openstack/ironic-specs master: Rename review.openstack.org to review.opendev.org  https://review.opendev.org/65855614:34
*** cdearborn has joined #openstack-ironic14:43
*** rachit7 has quit IRC14:45
*** ijw has quit IRC14:49
*** kaifeng has joined #openstack-ironic14:51
*** ijw has joined #openstack-ironic14:51
*** stendulker has joined #openstack-ironic14:54
*** pcaruana|afk| has quit IRC14:55
openstackgerritDoug Hellmann proposed openstack/ironic-inspector master: clarify that extra hardware data does not always go to swift  https://review.opendev.org/65882614:56
*** dsneddon has joined #openstack-ironic15:00
*** beekneemech is now known as bnemec15:00
*** ijw has quit IRC15:00
*** ijw has joined #openstack-ironic15:01
* etingof is wondering if we meet today...?15:01
etingofdtantsur, might be pulled too much into his reports ;)15:02
*** dsneddon has quit IRC15:05
NobodyCamGood Morning Ironic'ers15:09
NobodyCamHppy Monday15:09
arne_wiebalckNobodyCam: o/15:09
*** stendulker has quit IRC15:09
NobodyCamo/ Good morning arne_wiebalck15:09
rpittauhey NobodyCam :)15:09
NobodyCamgood morning rpittau :)15:10
dtantsurugh, yeah, sorry15:10
dtantsur#startmeeting ironic15:10
openstackMeeting 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
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:10
*** openstack changes topic to " (Meeting topic: ironic)"15:10
openstackThe meeting name has been set to 'ironic'15:10
etingof\o/15:10
dtantsurhi all, sorry for the late start (first day after 2 weeks of absence, sigh)15:10
bdoddo/15:10
kaifengo/15:10
rpittaumeeting! o/15:10
hjensaso/15:10
arne_wiebalcko/15:10
jrollmorning :)15:11
dtantsurour agenda is pretty light:15:11
dtantsur#link https://wiki.openstack.org/wiki/Meetings/Ironic15:11
rlooo/15:11
dtantsur#topic Announcements / Reminder15: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 Denver15:11
dtantsurSubjectively, a lot is going on around ironic15:12
arne_wiebalckthanks for the notes dtantsur, that's useful15:12
rloothx dtantsur, for your notes on the summit/ptg!15:12
rpittauthank 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, 201915:12
dtantsur#link https://www.openstack.org/summit/shanghai-201915:13
*** ijw has quit IRC15:13
dtantsurif you've dreamt of going to China, this is your chance :) registration is open already15:13
rpittauwell I guess I can't miss 3 summits in a row :P15:14
dtantsurheh15:14
dtantsurI have mixed feelings about the next one :)15:14
dtantsuranyway15:14
dtantsuranything else to announce?15:14
dtantsurhmm, maybe this one:15:14
dtantsur#link https://www.openstack.org/bare-metal/ The official bare metal program15:15
cdearborno/ sorry i'm late!15:15
dtantsuralso you'll notice that we have Baremetal SIG business as part of our agenda15:15
dtantsuranything else?15:15
mgoddardo/15:15
mjtureko/15:15
mkraio/15:15
rajiniro/15:16
dtantsurNo action items from the last week, and I doubt we have anything new on the whiteboard (or do we?)15:16
dtantsurI guess TheJulia will propose the priorities for Train officially next week15:17
bahao/15:17
dtantsurthose who like spoilers, check https://etherpad.openstack.org/p/DEN-train-ironic-ptg after line 63915:17
dtantsur* 53915:17
dtantsur#topic Deciding on priorities for the coming week15:18
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:18
dtantsurwhat do you think about the list as it is now?15:20
TheJuliaI 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
TheJuliaRe priorities, next Monday most likely :(15:21
dtantsurreasonable up-to-date, I just added the mdns thingy15:21
rpittauI did update some stuff based on merged things last week15:21
etingoflet's review this as well please? -- https://review.opendev.org/65568515:21
patchbotpatch 655685 - ironic-specs - Add spec for indicator management - 6 patch sets15:21
TheJuliadtantsur: perfect :)15:21
dtantsuretingof: fair enough, added15:22
dtantsurany other comments?15:22
rlooI 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
patchbotpatch 656799 - ironic-specs - Add support for node retirement - 4 patch sets15:22
*** ddurst- is now known as ddurst15:22
arne_wiebalckrloo: it's on the agenda :)15:22
dtantsurrloo: yeah, we have this on the agenda, and I'd like it on priorities to highlight this discussion15:23
rlooarne_wiebalck: oh, is that what 'needs follow-up from the PTG' is about?15:23
dtantsurrloo: yes15:23
* etingof has 10-patch long chain against sushy-tools to review...15:23
* dtantsur needs time and spoons to type letters about that spec15:23
rlooarne_wiebalck, dtantsur: ok, thx for clarifying15:23
dtantsurarne_wiebalck: you can find some of the information in my summary15:23
arne_wiebalckdtantsur: ok15:23
dtantsuretingof: 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
etingofdtantsur, I've refactored them heavily since your last review15:24
dtantsurokay, I'll check again if my concerns still hold15:24
etingofthough the things you did not like still there ;)15:24
dtantsurrloo: exactly15:24
dtantsuretingof: well, here we go :) do you really want me to review them? ;)15:25
etingofdtantsur, absolutely!15:25
rloois cisco-ucs out?15:26
dtantsurTheJulia: ^^?15:26
etingofdtantsur, I hope that with my recent additions (chassis, indicators and vmedia boot emulation) the design I am proposing makes more sense...15:26
TheJuliaYeah, pretty much. :(15:26
rloook, i'm deleting that from vendor priorities then.15:27
dtantsuretingof: if you found a real manager object in libvirt - yes, otherwise I expect a similar comment15:27
dtantsuranyway, does the list look good?15:27
TheJuliaI did chat with ?ianw? In Denver and he stressed that we should do what is best for the project.15:28
dtantsursigh15:28
etingofdtantsur, it's the other way round - if you find a real manager and chassis objects in libvirt, then your design makes sense ;)15:28
TheJuliaYeah. :(15:28
TheJuliaAnyway, airplane mode time.15:29
dtantsur#topic Baremetal SIG15:29
*** openstack changes topic to "Baremetal SIG (Meeting topic: ironic)"15:29
dtantsurAnything for today?15:29
TheJuliaThe white paper could use some assistance with regards to use cases15:29
rloodtantsur: as an intro -- did anyone mention why we have baremetal sig in this meeting?15:30
dtantsurrloo: I did not, because I was not present when it was decided15:30
rpittauwhen was that decided ?15:30
TheJuliaTl;dr there needs to be a periodic reminder, so the meeting was the best time15:30
rloodtantsur: 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
TheJuliaDuring 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
TheJuliaChris 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
rloofair 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
TheJuliaI concur, I won’t be able to do it this week.15:33
rloo(since not everyone was at ptg/summit)15:33
TheJuliaAnyway, they just closed the boarding door, need to quite literally disconnect now :(15:33
arne_wiebalckI can ping Chris.15:34
TheJuliaHave a wonderful day everyone15:34
rlooBYE TheJulia!15:34
dtantsursafe flight TheJulia15:34
rpittauTheJulia: safe flight :)15:34
*** yolanda_ has quit IRC15:34
dtantsurarne_wiebalck: please! I think we can do it off-meeting.15:34
rloook, AI for arne_wiebalck to ping chris to ask chris to send email out etc.15:34
dtantsur++15:36
dtantsur#topic RFE review15:36
*** openstack changes topic to "RFE review (Meeting topic: ironic)"15:36
*** itlinux has joined #openstack-ironic15:36
dtantsur#link https://review.opendev.org/#/c/656799/ Support for node retirement / nodes in failure states15:36
patchbotpatch 656799 - ironic-specs - Add support for node retirement - 4 patch sets15:36
dtantsuras I said, I did not have time to write more detailed thoughts - sorry for that15:37
dtantsurbut something that did come up in the room is that people want a new state more than a new flag15:37
arne_wiebalckthis spec is basically a summary of a discussion I had with jroll and TheJulia15:37
*** yolanda_ has joined #openstack-ironic15:37
dtantsurI was initially quite against it, but then got more or less convinced15:37
dtantsuralthough this may be a difference between retiring nodes and "failing" them..15:38
arne_wiebalckhow do you  mark an active node for retirement when retired is a state?15:38
dtantsurwith a state transition that does not, however, tear it down15:38
arne_wiebalckso instances can be on nodes in state retired?15:39
dtantsurthat's what people wanted for nodes at fault: keep it intact, but mark it as very broken15:39
dtantsuryep15:39
arne_wiebalckok15:39
jrollhmm15:39
dtantsurwhich is going to confuse the heck out of nova, I suspect15:39
rpittauinteresting, it goes slightly against logic imho15:39
jrollI'd love to see the proposed state machine15:39
jroller, state diagram15:40
arne_wiebalckthe idea behind the flag was to be very close to 'maintenace', an attribute you assign to a node15:40
dtantsurthe thing about maintenance is that it can happen in literally any state15:40
arne_wiebalckand such a flag would not interfere with the state machine15:40
arne_wiebalckretired as well15:40
dtantsurdoes it make sense to retire a node in "enroll"? "cleaning"? "manageble"?15:40
rpittauwell in theory retirement too15:40
*** pramchan has joined #openstack-ironic15:40
arne_wiebalckdtantsur: I think so, why not?15:40
rlooi think more thinking is needed here. wondering if we want a 'retire' state, AND some flag eg 'next-step' or 'next-phase'15:41
rpittauI think it does15:41
dtantsurarne_wiebalck: if you mark an available node as retired, how do you prevent nova from scheduling to it?15:41
arne_wiebalckhow about using the same mechanism 'maintenance' does?15:42
arne_wiebalckthe same way15:42
dtantsurmaintenance has been a part of our API since the inception15:42
dtantsurso it's something everybody is used to checking15:42
dtantsurolder tooling would not take retired into account15:42
dtantsurI think this ^^ should be in the spec even if we keep the flag approach15:43
arne_wiebalckolder tooling like  ...?15:43
dtantsurnova, metalsmith, metal3 are the tools I'm aware of15:43
arne_wiebalckwe could also set maintenance in addition to retired15:43
dtantsurmaintenance has side effects like preventing cleaning from working15:43
rlooat what point is the node 'retired' ?15:44
arne_wiebalcktrue, I take that back :)15:44
*** pcaruana|afk| has joined #openstack-ironic15:44
rloowe want the node to-be-retired, but when is it 'retired'?15:44
arne_wiebalckrloo, it is not retired, it's more marked for retirement15:44
rlooi'm having trouble understanding the state of the node in this.15:44
rlooarne_wiebalck: from the spec, it seems like what you want is to indicate 'do not make this available'15:45
arne_wiebalckrloo: right15:45
arne_wiebalckI'd like to mark an 'active' node as on its way out15:45
rlooarne_wiebalck: and (based on problem description) you want to be able to search/list nodes15:45
arne_wiebalckrloo: correct15:46
dtantsurarne_wiebalck: but not only active nodes, all states?15:46
* arne_wiebalck thinks rloo is preparing a suggestion15:46
arne_wiebalckdtantsur: yes15:46
rlooso end user/nova invokes 'delete' on their instance.15:46
rlooi'm not preparing a solution, just thinking about the problem. sorry.15:46
arne_wiebalckdtantsur: also nodes in clean_failed may be retired15:47
arne_wiebalckrloo: :)15:47
rlooseems 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_wiebalckrloo: correct15:48
dtantsuryeah, 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
rloowondering 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_wiebalckmaintenance is very close15:48
arne_wiebalckbut has the cleaning issue15:48
rloowasn'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-ironic15:49
rloo(and firmware update isn't part of the cleaning)15:49
rlooie, they may want the node to go to mgt after cleaning, not available.15:49
rloodo 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_wiebalckrloo: we don't necessarily need to change the state of the node I think15:51
*** pramchan has quit IRC15:51
rlooarne_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-ironic15:52
rlooi am not against a 'retire*' state or something, just trying to understand/generalize.15:52
arne_wiebalckrloo: if it is easy extract the list of nodes in this state: yes15:53
arne_wiebalckrloo: sounds good :)15:53
dtantsurI wonder if this solution generalizes to the case discussed on the PTG15:53
arne_wiebalckwhat way discussed at the ptg?15:53
rloonow i'd like some more operator feedback (wrt explicit 'retire*' something). and dtantsur, yeah.15:53
arne_wiebalcks/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_wiebalcksounds very close15:54
arne_wiebalckrloo' suggestion seems to cover both, no?15:54
rpittaualthough faulty sounds much more close to maintenance15:54
rlooarne_wiebalck: i think so :)15:54
arne_wiebalckquestion still is whether this should be a state15:55
rloorpittau: 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
dtantsurarne_wiebalck: it depends on how you want the existing API consumers to behave wrt retirement15:56
rlooarne_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
dtantsura new state will force all tooling to handle it explicitly15:56
dtantsura new flag on a node will be ignored by all consumers we don't update explicitly15:56
rpittauI totally agree with the non-abuse of maintenance, but I see 2 new states here, if we go down that road15:57
arne_wiebalckdtantsur: for the consumers, I thought ironic gives the nodes rather than nova fetching the nodes, no?15:57
rloorpittau: i have nothing against new states -- as long as they make sense :)15:57
dtantsurrpittau: maintenance is ruled out because it will prevent cleaning from working15:57
dtantsurarne_wiebalck: nova fetches nodes with some filters15:58
arne_wiebalckdtantsur: ah, ok15:59
arne_wiebalckdtantsur: I thought ironic hides them from nova15:59
arne_wiebalckthe ones in maintenance15:59
dtantsurI think it's done on the nova side16:00
dtantsurarne_wiebalck: https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L886-L89416:00
arne_wiebalckok ... that makes things more complicated ... in any case :)16:00
dtantsurwe're a bit out of time, let's move the discussion back to the spec?16:00
dtantsurwe need to cover existing API consumers impact, allocation API impact, etc16:00
dtantsurI also left one comment about the API design16:01
dtantsurthanks all!16:01
arne_wiebalckok, please comment on the spec then, thanks!16:01
dtantsur#endmeeting16: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
openstackMeeting ended Mon May 13 16:01:30 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)16:01
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-05-13-15.10.html16:01
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-05-13-15.10.txt16:01
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-05-13-15.10.log.html16:01
*** lucasagomes has quit IRC16:01
*** kaifeng has quit IRC16:07
*** tesseract has quit IRC16:20
*** rachit7 has joined #openstack-ironic16:21
*** dtantsur is now known as dtantsur|afk16:23
dtantsur|afkc u16:23
rpittaugood night! o/16:25
*** rpittau is now known as rpittau|afk16:25
arne_wiebalckbye everyone o/16:28
*** whoami-rajat has quit IRC16:31
*** whoami-rajat has joined #openstack-ironic16:36
*** ijw has joined #openstack-ironic16:37
*** priteau has quit IRC16:38
*** e0ne has quit IRC16:42
*** mgoddard has quit IRC16:50
*** mgoddard has joined #openstack-ironic16:52
*** dougsz has quit IRC16:56
*** mgoddard has quit IRC16:57
*** dsneddon has joined #openstack-ironic17:01
*** dsneddon has quit IRC17:06
*** med_ has joined #openstack-ironic17:18
*** dsneddon has joined #openstack-ironic17:19
*** mgoddard has joined #openstack-ironic17:29
*** jroll has quit IRC17:34
*** jroll has joined #openstack-ironic17:35
*** derekh has quit IRC17:37
*** ijw has quit IRC17:46
*** ijw has joined #openstack-ironic17:47
*** itlinux has quit IRC17:50
*** itlinux has joined #openstack-ironic17:53
*** ijw has quit IRC17:55
*** ijw has joined #openstack-ironic17:55
*** ijw has quit IRC18:03
*** ijw has joined #openstack-ironic18:03
*** ijw has quit IRC18:05
*** ijw has joined #openstack-ironic18:06
openstackgerritIlya Etingof proposed openstack/sushy master: Make Manager->Action field optional  https://review.opendev.org/65888618:10
openstackgerritIlya Etingof proposed openstack/sushy master: Make Manager->Actions field optional  https://review.opendev.org/65888618:11
*** ijw has quit IRC18:13
*** ijw has joined #openstack-ironic18:14
openstackgerritIlya Etingof proposed openstack/ironic master: WIP: Add Redfish Virtual Media support  https://review.opendev.org/63845318:15
*** e0ne has joined #openstack-ironic18:17
*** ijw has quit IRC18:20
*** ijw has joined #openstack-ironic18:20
*** e0ne has quit IRC18:54
*** diga has quit IRC18:56
*** ijw has quit IRC19:00
*** ijw has joined #openstack-ironic19:01
*** ijw has quit IRC19:05
*** jcoufal has quit IRC19:05
*** ijw has joined #openstack-ironic19:05
*** ijw has quit IRC19:08
*** ijw has joined #openstack-ironic19:09
*** ijw has quit IRC19:12
*** ijw has joined #openstack-ironic19:13
*** ijw has quit IRC19:29
*** ijw has joined #openstack-ironic19:29
*** gmann_pto has quit IRC19:33
*** persia has quit IRC19:35
*** persia has joined #openstack-ironic19:36
*** ijw has quit IRC19:47
*** ijw has joined #openstack-ironic19:47
*** rachit7 has quit IRC19:49
*** rh-jelabarre has quit IRC19:50
*** rh-jelabarre has joined #openstack-ironic19:53
*** ijw has quit IRC19:54
*** ijw has joined #openstack-ironic19:55
*** ijw has quit IRC20:07
*** ijw has joined #openstack-ironic20:08
*** e0ne has joined #openstack-ironic20:14
*** jungleboyj has joined #openstack-ironic20:25
*** e0ne has quit IRC20:26
*** itlinux has quit IRC20:32
stevebakermorning20:36
*** tssurya has quit IRC20:38
*** pcaruana|afk| has quit IRC20:46
*** derekh has joined #openstack-ironic20:59
*** openstackgerrit has quit IRC21:09
*** jtomasek has quit IRC21:17
*** ijw has quit IRC21:29
*** Goneri has quit IRC21:38
*** whoami-rajat has quit IRC21:41
*** ijw has joined #openstack-ironic21:51
*** ijw has quit IRC22:06
*** ijw has joined #openstack-ironic22:06
*** ijw has quit IRC22:07
*** ijw has joined #openstack-ironic22:09
*** ijw has quit IRC22:11
*** ijw has joined #openstack-ironic22:11
*** jcoufal has joined #openstack-ironic22:22
*** jcoufal has quit IRC22:26
*** ijw has quit IRC22:27
*** ijw has joined #openstack-ironic22:27
*** ijw has quit IRC22:27
*** ijw has joined #openstack-ironic22:28
*** ijw has quit IRC22:28
*** ijw has joined #openstack-ironic22:29
*** ijw has quit IRC22:31
*** ijw has joined #openstack-ironic22:32
*** ijw has quit IRC22:33
*** openstackgerrit has joined #openstack-ironic22:34
openstackgerritDoug Hellmann proposed openstack/ironic-python-agent master: show inspection callback url in error messages  https://review.opendev.org/65883322:34
*** ijw has joined #openstack-ironic22:34
*** ijw has quit IRC22:35
*** ijw has joined #openstack-ironic22:35
*** ijw has quit IRC22:36
*** ijw has joined #openstack-ironic22:36
*** ijw has quit IRC22:36
*** ijw has joined #openstack-ironic22:37
*** ijw has quit IRC22:39
*** ijw has joined #openstack-ironic22:40
*** ijw has quit IRC22:40
*** ijw has joined #openstack-ironic22:41
*** ijw has quit IRC22:53
*** ijw has joined #openstack-ironic22:54
*** ijw has quit IRC22:55
*** ijw has joined #openstack-ironic22:56
*** jcoufal has joined #openstack-ironic23:00
*** ijw has quit IRC23:00
*** ijw has joined #openstack-ironic23:00
*** pramchan has quit IRC23:01
*** jcoufal has quit IRC23:01
*** rcernin has joined #openstack-ironic23:04
*** ijw has quit IRC23:06
*** rcernin has quit IRC23:07
*** rcernin has joined #openstack-ironic23:07
*** jcoufal has joined #openstack-ironic23:27
*** derekh has quit IRC23:30
*** jcoufal has quit IRC23:35
*** cdearborn has quit IRC23:53

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!