Wednesday, 2019-04-10

« Tuesday, 2019-04-09 Index Thursday, 2019-04-11 »
*** betherly has quit IRC00:04
*** sthussey has quit IRC00:14
*** Goneri has quit IRC00:22
openstackgerritMerged openstack/networking-baremetal stable/queens: Replace openstack.org git:// URLs with https://  https://review.openstack.org/64655900:45
openstackgerritMerged openstack/networking-baremetal master: Replace openstack.org git:// URLs with https://  https://review.openstack.org/64655700:45
openstackgerritMerged openstack/ironic-python-agent stable/rocky: Replace openstack.org git:// URLs with https://  https://review.openstack.org/64641100:46
openstackgerritMerged openstack/networking-generic-switch stable/rocky: Replace openstack.org git:// URLs with https://  https://review.openstack.org/64658400:48
openstackgerritMerged openstack/bifrost stable/rocky: Replace openstack.org git:// URLs with https://  https://review.openstack.org/64606800:51
openstackgerritMerged openstack/bifrost stable/queens: Replace openstack.org git:// URLs with https://  https://review.openstack.org/64606700:51
openstackgerritMerged openstack/ironic stable/pike: Replace openstack.org git:// URLs with https://  https://review.openstack.org/64639300:51
openstackgerritMerged openstack/ironic stable/stein: doc: update ibmc driver support servers document  https://review.openstack.org/65017000:57
*** gyee has quit IRC01:03
*** betherly has joined #openstack-ironic01:11
*** whoami-rajat has joined #openstack-ironic01:12
*** tiendc has joined #openstack-ironic01:13
*** betherly has quit IRC01:16
*** openstackgerrit has quit IRC01:30
*** openstackgerrit has joined #openstack-ironic01:40
openstackgerritJulia Kreger proposed openstack/ironic master: Use the PUBLIC_BRIDGE for vxlan  https://review.openstack.org/65142001:40
openstackgerritJulia Kreger proposed openstack/ironic stable/stein: Use the PUBLIC_BRIDGE for vxlan  https://review.openstack.org/65142201:42
TheJuliarpittau|afk: ^^^^ those _should_ do it. The commit message should explain what the deal is.01:43
TheJuliastable/stein will need to merge before master is working of course01:43
*** betherly has joined #openstack-ironic01:53
*** betherly has quit IRC01:57
*** ijw has quit IRC02:06
openstackgerritMerged openstack/ironic master: Temporarily mark grenade multinode as non-voting  https://review.openstack.org/65136602:31
openstackgerritJulia Kreger proposed openstack/ironic master: Split ibmc power/reboot classes  https://review.openstack.org/65014702:35
*** dsneddon has quit IRC02:44
*** betherly has joined #openstack-ironic02:45
*** dsneddon has joined #openstack-ironic02:47
*** betherly has quit IRC02:50
*** hwoarang has quit IRC02:57
*** hwoarang has joined #openstack-ironic02:58
*** mmethot has quit IRC03:09
*** mmethot has joined #openstack-ironic03:09
*** rh-jelabarre has quit IRC03:44
openstackgerritMerged openstack/ironic-python-agent master: Discover IPv6 BMC address  https://review.openstack.org/64079503:49
*** betherly has joined #openstack-ironic04:18
*** betherly has quit IRC04:23
*** diablo_rojo has quit IRC04:29
*** openstackstatus has quit IRC04:35
*** openstackstatus has joined #openstack-ironic04:36
*** ChanServ sets mode: +v openstackstatus04:36
*** bfournie has quit IRC04:48
*** betherly has joined #openstack-ironic05:00
*** betherly has quit IRC05:04
*** pcaruana has joined #openstack-ironic05:06
*** dsneddon has quit IRC05:44
*** dsneddon has joined #openstack-ironic05:45
openstackgerritMerged openstack/ironic master: Add systemd unit for sushy emulator in devstack  https://review.openstack.org/64482205:45
*** mkrai has joined #openstack-ironic05:47
*** hwoarang has quit IRC06:10
*** hwoarang has joined #openstack-ironic06:15
*** dsneddon has quit IRC06:16
*** e0ne has joined #openstack-ironic06:28
*** ijw has joined #openstack-ironic06:29
*** hwoarang has quit IRC06:33
*** hwoarang has joined #openstack-ironic06:35
arne_wiebalckgood morning, ironic!06:35
*** dsneddon has joined #openstack-ironic06:41
etingofarne_wiebalck, o/06:45
openstackgerritIlya Etingof proposed openstack/ironic master: Move devstack emulators configs under /etc/ironic  https://review.openstack.org/65121206:46
arne_wiebalcketingof: good morning o/06:46
*** e0ne has quit IRC06:48
*** e0ne has joined #openstack-ironic06:57
*** rpittau|afk is now known as rpittau07:05
rpittaugood morning ironic! o/07:05
arne_wiebalckrpittau: o/07:14
rpittauhey arne_wiebalck :)07:15
rpittauTheJulia: that's exactly what I meant, thanks! :)07:16
*** tssurya has joined #openstack-ironic07:26
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP|DNM] testing uefi job with ubuntu bionic  https://review.openstack.org/64550707:31
openstackgerritRiccardo Pittau proposed openstack/ironic master: [WIP|DNM] testing uefi job with ubuntu bionic  https://review.openstack.org/64893807:32
*** e0ne has quit IRC07:33
*** gkadam has joined #openstack-ironic07:34
*** pcaruana has quit IRC07:34
*** gkadam has quit IRC07:34
*** pcaruana has joined #openstack-ironic07:35
*** etingof has quit IRC07:35
*** e0ne has joined #openstack-ironic07:40
rpittaudtantsur|afk: the mdns support rfe looks good, maybe just add a clarification on the fall back chain and default behavior of ipa-api-url and ipa-inspection-callback-url after this change07:45
*** e0ne has quit IRC07:48
*** e0ne has joined #openstack-ironic07:53
*** livelace has joined #openstack-ironic07:58
iurygregorymorning all07:59
iurygregoryanyone ever saw a problem when we dont cherry-pick the full patch and when we try to do cherry-pick the other files we get ! [remote rejected] HEAD -> refs/publish/stable/stein/py3 (change https://review.openstack.org/645146 closed) ?08:00
patchbotpatch 645146 - python-ironicclient (stable/stein) - Fix allocation tests (MERGED) - 1 patch set08:00
*** mbeierl has quit IRC08:02
*** priteau has joined #openstack-ironic08:05
*** mbeierl has joined #openstack-ironic08:05
*** dougsz has joined #openstack-ironic08:09
*** S4ren has joined #openstack-ironic08:21
rpittauhey iurygregory :)08:29
*** lucasagomes has joined #openstack-ironic08:30
*** derekh has joined #openstack-ironic08:35
openstackgerritMadhuri Kumari proposed openstack/ironic-specs master: Add support for Intel Speed Select in Ironic  https://review.openstack.org/65145908:37
openstackgerritMadhuri Kumari proposed openstack/ironic-specs master: Add support for Intel Speed Select in Ironic  https://review.openstack.org/65145908:45
*** e0ne has quit IRC08:47
*** dtantsur|afk is now known as dtantsur08:58
* dtantsur tries to open his eyes08:58
*** e0ne has joined #openstack-ironic08:58
*** livelace has quit IRC08:59
openstackgerritIury Gregory Melo Ferreira proposed openstack/python-ironicclient stable/stein: Run jobs under python2 and python3  https://review.openstack.org/65146608:59
dtantsurrpittau: what exactly would you like clarified?08:59
rpittaudtantsur: if mdns fails, what's next? also, what is the default behavior in case ipa-api-url and ipa-inspection-callback-url are not present?09:00
dtantsurrpittau: I think it's more or less clear from the text. if mdns fails, we have no other options. I explicitly mention that absent ipa-api-url means mdns. The only thing missing is explicitly mentioning that missing ipa-inspection-callback-url means inspection disabled.09:02
dtantsurI will add it09:02
rpittaudtantsur: I think the phrase about ipa-api-url defaulting to mdns if not present is a bit confusing, to me at least :/09:04
*** ijw has quit IRC09:05
*** e0ne has quit IRC09:06
dtantsurrpittau: do you mean, we shouldn't default to mdns?09:07
rpittaudtantsur: no, it's fine if the default is mdns, what I mean is that when I read the text, thatis not clear to me09:08
rpittauI think it's just a matter of punctuation :D09:08
dtantsurrpittau: better now?09:09
rpittaudtantsur: yes, thanks :)09:09
*** Lucas_Gray has joined #openstack-ironic09:25
*** Lucas_Gray has quit IRC09:31
*** diga has joined #openstack-ironic09:31
digaHello Ironic09:31
digadtantsur: Hi09:33
*** Lucas_Gray has joined #openstack-ironic09:34
digadtantsur: Requirement check for this patch is failing https://review.openstack.org/#/c/634903/09:34
patchbotpatch 634903 - ironic - Modify the iDRAC driver to use realtime RAID creation - 13 patch sets09:34
digaI updated my patch with required version09:35
dtantsurdiga: yeah, it's not related to your patch, I think09:35
digadtantsur: yes, because tox is successfully running locally09:35
digadtantsur: how to deal with this then ?09:36
dtantsurgimme a minute09:36
digaok09:36
openstackgerritDmitry Tantsur proposed openstack/ironic master: Uncap jsonschema in requirements  https://review.openstack.org/65148309:37
dtantsurdiga: probably this ^^^09:37
digadtantsur: okay :)09:38
digadtantsur: I will update the requirement.txt09:39
digadtantsur: should add dependency for your patch or update the requirement.txt09:39
*** livelace has joined #openstack-ironic09:41
dtantsurdiga: I think you should add a dependency or rebase on top of it09:41
digadtantsur: sure09:42
*** rcernin has quit IRC09:47
dtantsurhi mkrai! do you still plan on working on https://storyboard.openstack.org/#!/story/2005126 ? Let me know if you need any help with it.09:56
mkraidtantsur: Hi, yes. I will post the patches soon10:00
dtantsurgreat10:03
openstackgerritDigambar proposed openstack/ironic master: Modify the iDRAC driver to use realtime RAID creation  https://review.openstack.org/63490310:05
*** tiendc has quit IRC10:08
digadtantsur: Added dependency of "Uncap jsonschema" to my patch10:11
digaplease review10:13
*** tssurya has quit IRC10:31
*** mkrai has quit IRC10:43
*** macintoshme has quit IRC10:49
*** macintoshme has joined #openstack-ironic10:52
openstackgerritRiccardo Pittau proposed openstack/sushy master: Adding Thermal resource schema  https://review.openstack.org/65128410:58
*** etingof has joined #openstack-ironic11:07
*** tssurya has joined #openstack-ironic11:08
*** e0ne has joined #openstack-ironic11:12
*** e0ne has quit IRC11:15
dtantsuriurygregory: do you have an example of IPMI sensor data handy?11:16
iurygregorydtantsur, yep11:16
*** e0ne has joined #openstack-ironic11:19
*** diga has quit IRC11:19
*** e0ne has quit IRC11:20
*** e0ne has joined #openstack-ironic11:24
*** livelace has quit IRC11:24
*** macintoshme has quit IRC11:28
*** macintoshme has joined #openstack-ironic11:28
dtantsurmgoddard, rpittau, this is needed for the CI: https://review.openstack.org/#/c/651483/11:29
patchbotpatch 651483 - ironic - Uncap jsonschema in requirements - 1 patch set11:29
*** rh-jelabarre has joined #openstack-ironic11:42
*** bfournie has joined #openstack-ironic11:58
*** e0ne has quit IRC12:09
*** e0ne has joined #openstack-ironic12:12
*** priteau has quit IRC12:12
*** e0ne has quit IRC12:13
*** priteau has joined #openstack-ironic12:14
openstackgerritDigambar proposed openstack/ironic master: Modify the iDRAC driver to use realtime RAID creation  https://review.openstack.org/63490312:22
*** e0ne has joined #openstack-ironic12:28
openstackgerritMichael Turek proposed openstack/ironic master: WIP: Do not install syslinux when not deploying to an x86_64 node  https://review.openstack.org/65128512:31
openstackgerritRiccardo Pittau proposed openstack/ironic-tempest-plugin master: Temporary limiting sphinx to version < 2.0  https://review.openstack.org/65151912:42
*** Goneri has joined #openstack-ironic12:42
openstackgerritArkady Shtempler proposed openstack/ironic-tempest-plugin master: Single tenant tests: BM to BM and BM to VM  https://review.openstack.org/65083912:50
openstackgerritRiccardo Pittau proposed openstack/ironic-tempest-plugin master: Removing obsolete oslosphinx  https://review.openstack.org/65152212:51
*** mjturek has joined #openstack-ironic12:53
*** macintoshme has quit IRC12:54
openstackgerritRiccardo Pittau proposed openstack/ironic-tempest-plugin master: Removing obsolete oslosphinx  https://review.openstack.org/65152212:58
*** yolanda_ has joined #openstack-ironic12:58
openstackgerritArkady Shtempler proposed openstack/ironic-tempest-plugin master: Single tenant tests: BM to BM and BM to VM  https://review.openstack.org/65083913:00
*** irclogbot_1 has joined #openstack-ironic13:03
openstackgerritIlya Etingof proposed openstack/ironic master: Move devstack emulators configs under /etc/ironic  https://review.openstack.org/65121213:04
*** altlogbot_3 has joined #openstack-ironic13:07
dtantsuranyone not from Red Hat feels like some reading? https://storyboard.openstack.org/#!/story/200539313:20
*** e0ne has quit IRC13:20
mjturekdtantsur: I'll take a gander!13:22
dtantsurthx!13:22
* dtantsur google translates "gander" as "a male goose" and decides to not ask question :D13:23
*** e0ne has joined #openstack-ironic13:25
*** baha has joined #openstack-ironic13:26
mjturekhahaha13:30
mjturekyeah doesn't make much sense when you think about it13:30
openstackgerritRiccardo Pittau proposed openstack/ironic-tempest-plugin master: Removing obsolete oslosphinx  https://review.openstack.org/65152213:40
*** yolanda_ has quit IRC13:49
jrolldtantsur: does this depend on the bare metal being in the same L2 domain as the conductor? or is there other magic going on with mDNS?13:53
*** e0ne has quit IRC13:54
openstackgerritPradip Kadam proposed openstack/ironic master: added reset_idrac and known_good_state clean steps  https://review.openstack.org/65153813:55
dtantsurjroll: in theory mDNS traffic can be routed, I'm not sure it will in practice13:57
dtantsurI think it's a fair assumption13:57
*** e0ne has joined #openstack-ironic13:57
* dtantsur looks at IPv6 support stage and sighs13:59
openstackgerritMerged openstack/ironic stable/pike: Revert "Fix OOB introspection to use pxe_enabled flag in idrac driver"  https://review.openstack.org/65016414:00
jrolldtantsur: thanks. interesting idea14:01
jrolldtantsur: did you consider etcd/zookeeper/consul/etc?14:01
dtantsurall these options would work but are much heavier (and you need to pass credentials)14:01
TheJuliarpittau: by chance did you check the other multinode job?14:01
dtantsurmorning TheJulia14:01
dtantsuralthough with etcd we won't have IPv6 troubles..14:02
* TheJulia is somehow lost14:02
jrollheavier, yes, but I wonder if they provide benefits which outweigh the downsides14:02
dtantsurand also, how do we find etcd? :)14:03
rpittauTheJulia: only very quickly, it looks like the same or a similar issue14:03
TheJuliaThe one I'm looking at says the tests passed, but zuul reports failure14:03
rpittauTheJulia: this time with br-ex14:03
rpittauTheJulia: ovs-vsctl: no bridge named br-ex14:04
*** andrein has joined #openstack-ironic14:04
TheJuliahttp://logs.openstack.org/22/651422/1/check/ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode/44dec9a/ <-- no errors that I can see14:05
rpittauTheJulia: http://logs.openstack.org/22/651422/1/check/ironic-tempest-ipa-wholedisk-direct-tinyipa-multinode/44dec9a/compute1/logs/devstacklog.txt.gz#_2019-04-10_11_05_40_36914:05
TheJuliarpittau: so we need to pass in a PHYSICAL_BRIDGE var to that job I guess14:05
openstackgerritVarsha Verma proposed openstack/ironic master: Removes `hash_distribution_replicas` configuration option  https://review.openstack.org/65091214:06
TheJuliayeah, pass it into the subnode14:06
* TheJulia goes and pulls the patch down14:06
openstackgerritJulia Kreger proposed openstack/ironic stable/stein: Use the PUBLIC_BRIDGE for vxlan  https://review.openstack.org/65142214:08
* etingof is wondering, if rechecks will get the grenade job done, eventually?14:09
rpittauetingof: the multinode I don't think so14:10
openstackgerritJulia Kreger proposed openstack/ironic master: Use the PUBLIC_BRIDGE for vxlan  https://review.openstack.org/65142014:10
TheJuliaetingof: nah, it is basically misconfigured at this point with the switch for grenade to using stable/stein -> master14:13
* etingof goes on drinking coffee heavily14:13
TheJuliaSpeaking of multinodes, it seems there is no raising a failure of the subnode setup... and I don't think there is anyplace to logically hook into it14:13
rpittauyay14:14
TheJuliabut basically it gets the node count off because it only ends up calculating local nodes for resource tracking14:14
*** openstackgerrit has quit IRC14:14
TheJuliait is how I actually back tracked and started finding the issue last night14:14
*** w14161_1 has quit IRC14:17
dtantsurjroll: talk to our folks, and to discover etcd we will have to use mdns :)14:23
TheJuliadtantsur: I have a strong feeling that is https://review.openstack.org/#/q/I37bff3c4d930ada483690d0dea17cb9a64d51fd2 will make our grenade job happier. Hopefully in the mean time we can get the other patches that seek to stabilize minor things merged as well14:35
* TheJulia feels bad the gate was broken hard for like 4 days14:35
dtantsurTheJulia: thanks, bookmarking14:36
*** openstackgerrit has joined #openstack-ironic14:37
openstackgerritRiccardo Pittau proposed openstack/sushy master: [WIP] Adding Power resource schema  https://review.openstack.org/65156014:37
etingofthe gate is clogged with patches14:37
TheJuliadown side is I've had to defer a few things to get enough solid time to focus on the gate :(14:38
jrolldtantsur: right14:42
jrolldtantsur: now that you say that, it makes me wonder why we can't just use regular DNS, instead of an IP address?14:42
dtantsurjroll: at the very least it requires changing the DNS configuration. the benefit of mDNS is that it's "zero-config".14:43
dtantsuralso TheJulia had concerns about DNS cache poisoning14:43
*** sthussey has joined #openstack-ironic14:43
jrollerm, why/where does it require changing DNS configuration?14:43
*** iurygregory has quit IRC14:44
jrollare we assuming the endpoints don't have a DNS name?14:44
jrollif an attacker can poison the cache on a DNS server, they could probably just run an mDNS responder and do the same thing, right?14:46
*** iurygregory has joined #openstack-ironic14:46
dtantsurjroll: how do you know the DNS name to resolve?14:46
*** mjturek has quit IRC14:47
jrolldtantsur: yes, this would be hardcoded. but you can change the IP that it resolves to without changing DHCP config.14:47
dtantsurjroll: so, IPA would try resolving ironic.local or something like that using whatever DNS it got from DHCP?14:48
jrolldtantsur: sure, any hostname14:49
dtantsurjroll: then we'll need to configure the DNS server to resolve it to something deployment specific?14:50
dtantsurand we won't be able to just use the customer's site DNS?14:50
openstackgerritRiccardo Pittau proposed openstack/sushy master: Adding Thermal resource schema  https://review.openstack.org/65128414:50
*** e0ne has quit IRC14:50
jrolldtantsur: ok so this is solving "I want to ship the same ramdisk to multiple deployments without assuming anything about local DNS"?14:51
dtantsuryeah14:52
jrollI see14:54
*** e0ne has joined #openstack-ironic14:54
jrollas someone not using tenant network isolation, this scares me :)14:55
*** ianychoi has quit IRC14:56
etingofanyone on the network might be able to claim the address...14:56
dtantsurthis of course assumes a secure provisioning network14:58
dtantsurwhich is the default assumption with ironic anyway14:58
jrollyeah, I understand14:58
*** diablo_rojo has joined #openstack-ironic15:02
*** mjturek has joined #openstack-ironic15:04
openstackgerritMerged openstack/ironic stable/rocky: Revert "Fix OOB introspection to use pxe_enabled flag in idrac driver"  https://review.openstack.org/65011615:11
openstackgerritMerged openstack/ironic master: Uncap jsonschema in requirements  https://review.openstack.org/65148315:11
openstackgerritRiccardo Pittau proposed openstack/sushy master: [WIP] Adding Power resource schema  https://review.openstack.org/65156015:14
TheJuliaetingof: did you add redfish virtual media to the ptg planning etherpad?15:24
TheJulianevermind, I elaborated it somewhere15:25
*** gyee has joined #openstack-ironic15:26
*** mjturek has quit IRC15:26
etingofTheJulia, I did not15:28
TheJuliamgoddard: how long do yout hink talking about deployment steps is going to take?15:29
mgoddardTheJulia: I think it's a 40 minute slot15:29
mgoddardso I guess 30 minutes15:30
mgoddardto allow for questions etc15:30
TheJuliaarne_wiebalck: I'm not really expecting software raid next steps to take very long, is your perception different?15:31
TheJuliahjensas: Same question for Neutron Event processing, how long do you perceive that it will take?15:31
arne_wiebalckTheJulia: Your referring to the PTG etherpad?15:31
TheJuliaarne_wiebalck: yes, but I'm building a new one with a schedule15:31
TheJuliaor rough schedule so trying to figure out timing and spacing15:32
arne_wiebalckTheJulia: no, I agree15:32
arne_wiebalckTheJulia: I won't be there anyway.15:32
*** mjturek has joined #openstack-ironic15:32
TheJuliaoh :(15:32
arne_wiebalck:(15:32
TheJuliaarne_wiebalck: okay15:32
TheJuliaI guess it is a good feedback loop point, I'll go a head and raise it15:32
arne_wiebalckTheJulia: Just wanted to put it on the radar that this is sth for the next cycle15:32
TheJulia++15:33
arne_wiebalckTheJulia: And see if people agree/disagree.15:33
arne_wiebalckTheJulia: Did you see the other point I added?15:33
TheJuliaretirement?15:33
arne_wiebalckTheJulia: yes.15:33
arne_wiebalckTheJulia: We're retiring the first nodes from ironic, actually :)15:33
arne_wiebalckTheJulia: And I'm trying to figure out a sensible workflow.15:34
TheJuliaAhh!15:34
TheJulianeat!15:34
TheJuliais it alright if I add that context?15:34
arne_wiebalckOh, sure!15:34
arne_wiebalckI'm actually discussing this with our operations team.15:35
arne_wiebalckAnd doing this with current ironic states is possible, but seems a little clumsy.15:35
arne_wiebalckMaybe others have a suggestion how todo this ...?15:35
hjensasTheJulia: Hard to say, 2-3 weeks. What scares me most is moving the provider.update_dhcp() calls that are currently hidden in pxe and ipxe drivers to somwhere we can do them async.15:38
TheJulia2-3 weeks to discuss?15:38
TheJuliajroll: maybe you folks have some thoughts for arne_wiebalck?15:39
TheJuliaarne_wiebalck: I do <3 the idea of a retiring flag or something that ends that moves a node off to a different state when released15:39
TheJuliato start out, that really could just be a separate flag to move it off to a new state and that state could permit deleting the node...15:40
TheJuliaat least as a first pass15:40
arne_wiebalckTheJulia: sounds good!15:40
jrollafaik we just delete the node form ironic to retire it15:40
jrollwe only retire unused nodes15:40
jroll(note that we can tell people they need to release the node)15:40
arne_wiebalckjroll: what about cleaning?15:41
TheJuliadtantsur: I'm guessing 15 minutes for official ipa images?15:41
jrollwe probably have a script that does the 'nova delete' and 'ironic delete' in one shot15:41
arne_wiebalckjroll: and how do yo identify nodes you can delete?15:41
jrollarne_wiebalck: what about it?15:41
dtantsurTheJulia: sorry, what's the context?15:41
arne_wiebalckyou have the same cleaning when it goes back to available vs retirement?15:41
jrollyeah15:42
hjensasTheJulia: oh, sorry for implementing. Now, to discuss at PTG Maby 30-60 minutes? (Probably at the lower end)15:42
jrollarne_wiebalck: there may be some extra cleaning outside of ironic after retiring, not sure15:42
arne_wiebalckjroll: we have a lightweight cleaning when we want to reuse15:42
arne_wiebalckjroll: and do full shred before retirement, for instance15:42
jrollarne_wiebalck: I think identifying nodes to retire is all done outside of ironic, between our CMDB, datacenter folks, hardware folks, etc15:42
arne_wiebalckjroll: remove IPMI creds15:42
arne_wiebalckjroll: I see15:43
rpittaumaybe add a "shred" step to cleaning?15:43
jrollarne_wiebalck: yeah, that's all outside of ironic afaik. ironic doesn't have a physical disk shredder :)15:43
arne_wiebalckjroll: I see:)15:43
arne_wiebalckjroll: we also donate nodes, so they need to re-burnin15:44
arne_wiebalckjroll: but we need to keep them separate from our prod nodes15:44
jrollarne_wiebalck: yeah, I'm the wrong person to ask about that. we mostly destroy them afaik :)15:44
*** iurygregory has quit IRC15:46
arne_wiebalckjroll: ok :)15:46
arne_wiebalckjroll: making the logic someone else's issue is nice as an approach ;)15:46
arne_wiebalckjroll: this is basically what I proposed15:47
jrollheh15:47
jrollarne_wiebalck: more like we haven't automated that (yet?) :)15:47
arne_wiebalckjroll: :)15:48
arne_wiebalckjroll: we have the ops team keeping an eye on the life cycle, users owning instances, and us (openstack/ironic) in between as the infra provider15:49
arne_wiebalckjroll: ops colleagues trigger the retirement (with us)15:49
arne_wiebalckjroll: we inform users (who delete instances)15:50
arne_wiebalckjroll: we hand the nodes over to ops people (for dismantling)15:50
dtantsurTheJulia: /me has finally understood, yeah 15 mins should be fine15:50
arne_wiebalckjroll: for now, a convenient way to mark nodes as "on their way out" would be good already15:51
arne_wiebalckjroll: I think this is what TheJulia suggested ^15:51
rpittaugood night! o/15:53
*** rpittau is now known as rpittau|afk15:53
TheJuliaYeah, that is exactly what I was thinkign... A user will eventually release the node... hopefully15:54
TheJuliaAnd maybe we make it a timed field "retired_at"15:54
TheJuliaNone meaning, not on a path to retirement....15:54
TheJuliaor something15:54
arne_wiebalckWhich should also block any new instance?15:54
arne_wiebalckBut still allows for cleaning (unlike maintenance).15:55
* arne_wiebalck is getting greedy15:55
jrollarne_wiebalck: right, that all makes sense to me15:55
jrollI think in the database I'd like to just send it to the MANAGEABLE state15:55
jrollafter cleaning15:56
arne_wiebalckjroll: or before15:56
jroll(if we mark it "on the retirement path")15:56
arne_wiebalckjroll: but this makes them indistinguishable from new nodes on their way into production15:56
jrollhm, maybe cleaning is a config, I could see it both ways15:56
jrollarne_wiebalck: well, not if "retired_at" is set15:56
jrollmaybe if "retired_at" is set, we don't allow the provide verb15:57
arne_wiebalckjroll: yes, I thought you suggested instead of "retired_at"15:57
jrollgotcha, I do think we'd need a mark like that15:57
arne_wiebalckjroll: agred15:57
arne_wiebalckjroll: agreed15:57
arne_wiebalckso, node is active, but admin can set retired_at15:58
TheJulia\o/ consensus building15:58
jrollproposal for the command: openstack node set --has-enough-money-to-be-lazy15:58
arne_wiebalckwhere would that node go if the instance is deleted?15:58
TheJuliajroll: s/lazy/to-send-directly-to-ironic-developers/15:58
jroll:)15:58
jrollarne_wiebalck: my proposal is manageable, seems like the best state for an operator to decide to delete from ironic or that there was an error and it should go back to available15:59
arne_wiebalckjroll: totally agree15:59
arne_wiebalckjroll: so, we'd some logic that after cleaning a node with retired_at != None goes to manageable (which would exclude it from scheduling)16:00
TheJuliaI kind of concur, I do like the provide verb being blocked16:00
arne_wiebalckTheJulia: that as well16:00
arne_wiebalckTheJulia: yes16:00
arne_wiebalckTheJulia: jroll: shall I write that up in a story?16:01
TheJuliaarne_wiebalck: please! :)16:01
jroll++16:01
arne_wiebalckcool, thx TheJulia jroll !16:02
mgoddardTheJulia: sorry, had summit slides in my head not PTG. Although timings are probably similar. There are quite a few small things, could probably fill an hour if available or less if not16:03
TheJuliamgoddard: shooting for 30 minutes16:06
mgoddardTheJulia: sure16:06
*** Lucas_Gray has quit IRC16:06
*** lucasagomes has quit IRC16:11
TheJuliahjensas: on the ptg etherpad, you raised a number of questions regarding networking. how long do you think that will take to get through?16:12
*** dtantsur is now known as dtantsur|afk16:15
dtantsur|afkc u16:15
TheJuliao/16:15
*** derekh has quit IRC16:28
*** hjensas has quit IRC16:29
*** w14161_1 has joined #openstack-ironic16:29
NobodyCamGood Morning Ironic'ers16:30
NobodyCamHappy Hump Day16:30
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicator management harness to ManagementInterface  https://review.openstack.org/64967516:36
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicator management harness to ManagementInterface  https://review.openstack.org/64967516:40
TheJuliaetingof: fyi https://github.com/dell/iDRAC-Redfish-Scripting/issues/24#issuecomment-48156382316:45
TheJuliagood morning NobodyCam16:45
NobodyCam:)16:45
*** altlogbot_3 has quit IRC16:46
*** e0ne has quit IRC16:55
*** S4ren has quit IRC16:57
*** dougsz has quit IRC17:00
arne_wiebalckBye, everyone o/17:01
TheJulia\o/ I invented time travel by putting the wrong times down17:02
*** priteau has quit IRC17:31
TheJuliaOkay, schedule posted17:39
TheJulia\o/17:39
TheJuliaonly took two days beyond what I had hoped :(17:39
dustincThanks! I was just wondering about that.. :)17:39
TheJuliawoot17:40
*** hwoarang has quit IRC18:02
*** hwoarang has joined #openstack-ironic18:03
* TheJulia goes and gets lunch18:15
*** diablo_rojo has left #openstack-ironic18:17
*** Goneri has quit IRC18:24
*** ijw has joined #openstack-ironic18:25
*** priteau has joined #openstack-ironic18:32
*** priteau has quit IRC18:37
*** hjensas has joined #openstack-ironic18:40
*** andrein has quit IRC18:45
-openstackstatus- NOTICE: Restarting Gerrit on review.openstack.org to pick up new configuration for the replication plugin19:05
*** tssurya has quit IRC19:07
*** ijw has quit IRC19:16
*** Goneri has joined #openstack-ironic19:25
*** andrein has joined #openstack-ironic19:37
*** mjturek has quit IRC19:44
*** ijw has joined #openstack-ironic19:47
*** macintoshme has joined #openstack-ironic19:50
*** priteau has joined #openstack-ironic20:33
*** Goneri has quit IRC20:36
arne_wiebalckTheJulia: jroll: https://storyboard.openstack.org/#!/story/200542520:42
*** priteau has quit IRC20:48
*** Goneri has joined #openstack-ironic20:49
*** Lucas_Gray has joined #openstack-ironic20:51
TheJuliaarne_wiebalck: thanks21:01
*** bfournie has quit IRC21:08
*** ijw has quit IRC21:31
*** ijw has joined #openstack-ironic21:35
*** hjensas has quit IRC21:36
rpiosoTheJulia: Are all docs built from master?21:38
openstackgerritRichard G. Pioso proposed openstack/ironic stable/rocky: Place upper bound on python-dracclient version  https://review.openstack.org/65165721:48
TheJuliarpioso: they are branched21:55
TheJuliaand versioned as a result, which is how we end up with docs.o.o/ironic/latest and ironic/queens/21:55
rpiosoTheJulia: I'm seeing build breakage on master and stable/rocky.21:57
* rpioso just kicked off a fresh docs build on master to capture the error21:58
rpiosoTheJulia: http://paste.openstack.org/show/749148/22:00
rpiosoTheJulia: I recall it working yesterday evening, just before I submitted https://review.openstack.org/#/c/65140822:02
patchbotpatch 651408 - ironic - Place upper bound on python-dracclient version - 1 patch set22:02
TheJuliaI already have the fix up for rocky22:03
TheJuliarpioso: rm -r -f doc/source/contributor/api/*22:03
rpiosoTheJulia: Sweet!22:04
*** rcernin has joined #openstack-ironic22:06
*** baha has quit IRC22:09
rpiosoTheJulia: Thank you :)22:11
*** hjensas has joined #openstack-ironic22:23
*** betherly has joined #openstack-ironic22:32
*** Lucas_Gray has quit IRC22:35
*** betherly has quit IRC22:37
*** whoami-rajat has quit IRC22:51
*** hwoarang has quit IRC23:10
*** hwoarang has joined #openstack-ironic23:12
*** betherly has joined #openstack-ironic23:12
*** betherly has quit IRC23:17
*** ianychoi has joined #openstack-ironic23:46
*** rcernin has quit IRC23:52
« Tuesday, 2019-04-09 Index Thursday, 2019-04-11 »

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