Tuesday, 2019-04-09

*** gyee has quit IRC00:09
*** andrein has quit IRC00:13
*** ijw has quit IRC00:14
*** cdearborn has quit IRC00:29
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Remove deprecated store_data_location  https://review.openstack.org/65083300:50
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Remove deprecated ssl options  https://review.openstack.org/65082400:53
*** tiendc has joined #openstack-ironic01:20
*** sthussey has quit IRC01:51
openstackgerritKaifeng Wang proposed openstack/ironic-python-agent master: Fix docs job failure due to malformated docstring  https://review.openstack.org/65105802:29
*** whoami-rajat has joined #openstack-ironic02:37
*** diablo_rojo has quit IRC02:37
*** baha has joined #openstack-ironic02:51
*** baha has quit IRC02:51
*** dsneddon has quit IRC03:28
*** dsneddon has joined #openstack-ironic03:53
*** dsneddon has quit IRC03:58
*** dsneddon has joined #openstack-ironic04:30
*** hwoarang_ has quit IRC04:37
*** hwoarang has joined #openstack-ironic04:42
*** e0ne has joined #openstack-ironic05:25
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Remove deprecated ssl options  https://review.openstack.org/65082405:27
*** e0ne has quit IRC05:29
*** mkrai has joined #openstack-ironic05:30
*** andrein has joined #openstack-ironic05:30
*** dsneddon has quit IRC05:34
*** andrein has quit IRC05:41
*** mkrai has quit IRC05:57
*** dsneddon has joined #openstack-ironic06:06
*** dsneddon has quit IRC06:11
*** dsneddon has joined #openstack-ironic06:14
*** dsneddon has quit IRC06:19
*** pcaruana has joined #openstack-ironic06:30
arne_wiebalckgood morning, ironic!06:37
*** dsneddon has joined #openstack-ironic06:46
*** dsneddon has quit IRC06:51
*** hwoarang has quit IRC06:57
*** hwoarang has joined #openstack-ironic07:04
*** rpittau|afk is now known as rpittau07:12
rpittaugood morning ironic! o/07:12
*** SpamapS has quit IRC07:15
*** SpamapS has joined #openstack-ironic07:17
*** dsneddon has joined #openstack-ironic07:17
*** dsneddon has quit IRC07:21
*** dsneddon has joined #openstack-ironic07:24
*** tssurya has joined #openstack-ironic07:28
*** dsneddon has quit IRC07:28
*** andrein has joined #openstack-ironic07:44
*** xXraphXx has joined #openstack-ironic07:52
*** dtantsur|afk is now known as dtantsur08:00
dtantsurmorning ironic08:00
*** dsneddon has joined #openstack-ironic08:00
*** rcernin has quit IRC08:01
*** dsneddon has quit IRC08:06
*** dougsz has joined #openstack-ironic08:08
*** sziviani has joined #openstack-ironic08:09
*** e0ne has joined #openstack-ironic08:13
mgoddardmorning dtantsur rpittau arne_wiebalck08:14
arne_wiebalckmgoddard: o/08:14
rpittauhi mgoddard :)08:14
*** priteau has joined #openstack-ironic08:15
*** andrein has quit IRC08:17
iurygregorymorning arne_wiebalck rpittau dtantsur mgoddard and all other Ironicer's o/08:18
rpittauhey iurygregory :)08:18
arne_wiebalckgood morning iurygregory o/08:19
*** lucasagomes has joined #openstack-ironic08:27
openstackgerritRiccardo Pittau proposed openstack/bifrost master: Improve iptables test info collection  https://review.openstack.org/64994608:30
*** luzC has quit IRC08:31
*** luzC has joined #openstack-ironic08:31
*** e0ne has quit IRC08:37
*** derekh has joined #openstack-ironic08:40
*** dsneddon has joined #openstack-ironic08:42
*** andrein has joined #openstack-ironic08:46
*** dsneddon has quit IRC08:47
*** e0ne has joined #openstack-ironic08:47
openstackgerritMerged openstack/ironic-python-agent master: Fix docs job failure due to malformated docstring  https://review.openstack.org/65105808:58
openstackgerritMerged openstack/bifrost master: Pin proliantutils version  https://review.openstack.org/64819309:08
*** dsneddon has joined #openstack-ironic09:13
openstackgerritRiccardo Pittau proposed openstack/bifrost master: [WIP/DNM] Testing ubuntu bionic job  https://review.openstack.org/64931409:15
*** andrein has quit IRC09:15
*** dsneddon has quit IRC09:19
*** e0ne has quit IRC09:25
openstackgerritMerged openstack/sushy-tools master: Include all libvirt domains in Systems  https://review.openstack.org/64870009:28
*** livelace has joined #openstack-ironic09:33
*** S4ren has joined #openstack-ironic09:35
etingofthese sushy-tools patches that are not in merge conflict state... once merged we get redfish jobs actually functional for the basic stuff like boot mode and boot device setting \o/09:39
iurygregory\o/09:39
*** e0ne has joined #openstack-ironic09:44
*** Lucas_Gray has joined #openstack-ironic09:45
*** yolanda_ has quit IRC09:47
*** andrein has joined #openstack-ironic09:51
*** Lucas_Gray has quit IRC09:53
*** Lucas_Gray has joined #openstack-ironic09:54
*** diga has joined #openstack-ironic09:55
digaGood Morning Ironic09:56
*** mkrai has joined #openstack-ironic09:58
openstackgerritRiccardo Pittau proposed openstack/sushy master: Expand Drive schema  https://review.openstack.org/65117910:01
rpittauhey diga :)10:07
digarpittau:  :)10:24
openstackgerritRiccardo Pittau proposed openstack/sushy master: Expand Drive schema  https://review.openstack.org/65117910:25
*** e0ne has quit IRC10:37
*** e0ne has joined #openstack-ironic10:44
*** livelace has quit IRC10:56
jrollmorning11:03
iurygregorymorning jroll11:06
*** mkrai has quit IRC11:13
dtantsurmorning jroll11:26
*** macintoshme has quit IRC11:27
*** macintoshme has joined #openstack-ironic11:28
*** e0ne has quit IRC11:30
openstackgerritIlya Etingof proposed openstack/virtualbmc master: Respect $VIRTUAL_BMC for non-default config path  https://review.openstack.org/65120211:36
*** andrein has quit IRC11:55
*** dsneddon has joined #openstack-ironic11:56
*** dsneddon has quit IRC12:01
*** dsneddon has joined #openstack-ironic12:02
openstackgerritIlya Etingof proposed openstack/ironic master: Add systemd unit for sushy emulator in devstack  https://review.openstack.org/64482212:04
openstackgerritIlya Etingof proposed openstack/ironic master: Add UEFI firmware to Redfish emulator config  https://review.openstack.org/64838212:05
openstackgerritIlya Etingof proposed openstack/ironic master: Run IPMI, SNMP and Redfish BMC emulators as stack  https://review.openstack.org/64849612:06
openstackgerritIlya Etingof proposed openstack/ironic master: Improve VirtualBMC use in Devstack  https://review.openstack.org/64851012:07
*** dsneddon has quit IRC12:07
openstackgerritIlya Etingof proposed openstack/ironic master: Move devstack emulators configs under /etc/ironic  https://review.openstack.org/65121212:09
*** dsneddon has joined #openstack-ironic12:12
*** dsneddon has quit IRC12:16
*** andrein has joined #openstack-ironic12:18
*** livelace has joined #openstack-ironic12:20
*** tiendc has quit IRC12:22
openstackgerritraphael.glon proposed openstack/ironic master: Ansible module: fix partition_configdrive.sh file  https://review.openstack.org/64966312:30
*** livelace has quit IRC12:32
*** priteau has quit IRC12:42
*** openstackgerrit has quit IRC12:44
arne_wiebalckstendulker, you around?12:44
dtantsurTheJulia, rpittau, FYI: https://review.openstack.org/#/c/651222/12:44
patchbotpatch 651222 - api-sig - Add guideline on DNS-based service discovery - 1 patch set12:45
*** dtantsur is now known as dtantsur|brb12:46
*** Goneri has joined #openstack-ironic12:50
*** e0ne has joined #openstack-ironic12:54
*** jroll has quit IRC12:55
*** baha has joined #openstack-ironic12:57
*** sthussey has joined #openstack-ironic12:58
*** jroll has joined #openstack-ironic12:59
*** mbeierl has joined #openstack-ironic13:02
*** mjturek has joined #openstack-ironic13:07
*** jcoufal has joined #openstack-ironic13:10
*** bfournie has joined #openstack-ironic13:11
*** andrein has quit IRC13:17
rpiosoGood morning, ironicers13:18
*** rloo has joined #openstack-ironic13:20
*** priteau has joined #openstack-ironic13:27
iurygregorymorning rpioso13:34
*** mjturek has quit IRC13:35
*** mjturek has joined #openstack-ironic13:36
rpiosoiurygregory: :-)13:37
*** andrein has joined #openstack-ironic13:47
*** baha has quit IRC13:53
*** e0ne has quit IRC13:53
*** Lucas_Gray has quit IRC13:57
*** diga has quit IRC13:58
*** Lucas_Gray has joined #openstack-ironic14:01
TheJuliadtantsur|brb: seems to just mirror the rfc which works for me14:04
*** priteau has quit IRC14:04
TheJuliaso ipa could look for baremetal-introspection._tcp.openstack.local and baremetal._tcp... and if found could possibly use those. It could also check the local dns name I guess. which would be a nice value add for general operators14:05
*** baha has joined #openstack-ironic14:05
rpittauoh wow my connection was stuck for like an hour.....14:14
rpittauanyway14:14
rpittaudtantsur|brb: except for the comment I left, it looks good14:15
*** dtantsur|brb is now known as dtantsur14:30
dtantsurmorning TheJulia, rpioso14:31
rpiosodtantsur: Howdy o/14:31
TheJuliao/14:31
rpiosoTheJulia: Hey14:31
*** e0ne has joined #openstack-ironic14:31
* rpioso wonders if we're going to merge the idrac branch destabilization fixes today14:32
TheJuliaWell, I have meetings and then I need to run into Palm Desert to get a new power supply for my desktop :(14:33
TheJuliaafter which reviewing will be easy :(14:33
rpiosoSounds like a nice, warm place to pick up a power supply ;-)14:34
TheJuliaIndeed. Likely not as loud as my UPS this morning screaming when the PSU died14:34
TheJuliahopefully the UPS is still good14:34
* etingof is wondering how does UPS know that PSU is going south...14:36
TheJuliaI think it failed closed...14:36
TheJuliaI think, I've not looked for the ohm meter yet14:36
etingofif it fails closed, the thing might heat and blow up eventually14:37
*** openstackgerrit has joined #openstack-ironic14:38
openstackgerritMark Goddard proposed openstack/bifrost master: Don't fix /etc/hosts if Rabbit is disabled  https://review.openstack.org/65125714:38
TheJuliait sounded like something popped, and the UPS went into a "nope nope nope not going to do this" mode14:38
etingof(to release the blue smoke essence on which everything operates)14:38
etingofso what exactly UPS senses? too high load?14:38
* rpioso feels etingof may have had too much experience with failed PSUs14:39
TheJuliaI'm guessing arch or ground fault14:39
TheJuliabut the screen says general fault14:39
* etingof is an electrical engineer by trade14:40
etingofI wonder if UPS vendor docs has any explanation on the general fault...?14:40
etingofbut if you plug some other load into the UPS it does not scream?14:41
TheJuliathe docs say "you blew a fuse" but it powers up and without being plugged in14:42
*** livelace has joined #openstack-ironic14:42
etingofah, so UPS does not work under any load?14:42
TheJuliadoesn't work with my desktop plugged in now14:43
TheJuliaand I can't power it up... so thinking the PSU blew14:43
etingofmay be plug PSU directly into the wall socket?14:43
etingofit may blow up if it's shorted14:43
etingofalternatively, may be plug something simple like a desktop lamp into the UPS to see if it feels good then...?14:44
openstackgerritJerry Wang proposed openstack/ironic master: Add DHCP server part to make the document more detail  https://review.openstack.org/64603014:46
rpittauI wouldn't plug the PSU to the wall socket if it's shorted :/14:46
etingofthat's the smoke test14:48
rpittauit's literally a smoke test xD14:49
*** jistr is now known as jistr|call14:51
etingoflet's call it build verification test14:51
*** ijw has joined #openstack-ironic14:51
TheJuliaetingof: already tried, nothing :\14:54
TheJuliaand a low load 120v item does work on the UPS \o/14:54
etingofno smoke? that's not shortage then14:55
TheJuliawell, the smoke may have been in the PSU14:56
TheJuliaIt occurs to me that this would be the second PSU that has died in it...14:56
* TheJulia wonders if she needs to get rid of the spinning iron oxide14:56
etingofthe fuse in the PSU is frequently accessible if one has a screwdriver...14:57
rpiosoTheJulia: Thank you! :)15:03
TheJuliaetingof: true!15:03
TheJuliaso it seems grenade is busted15:03
*** ijw has quit IRC15:05
*** ijw has joined #openstack-ironic15:05
TheJulialooks like OVS is failing to start up phy-brbm-infra15:08
*** ijw has quit IRC15:09
*** ijw has joined #openstack-ironic15:09
NobodyCamGood Morning Ironic'ers !15:13
dtantsurmorning NobodyCam15:15
NobodyCam:)15:15
rpittauhey NobodyCam :)15:16
NobodyCamhowdy dtantsur and rpittau :) how goes the battle15:16
dtantsurlearning mDNS and DNS-SD, having fun with it :) and you?15:16
NobodyCamhacking on Magnum :) (and Horizon)15:17
rpittaufighting with redfish schema :)15:17
*** mkrai has joined #openstack-ironic15:18
TheJulia\o/15:21
NobodyCamgood morning TheJulia :)15:22
*** e0ne has quit IRC15:23
*** Namrata has joined #openstack-ironic15:26
*** e0ne has joined #openstack-ironic15:27
TheJulialooks like we have to backport I81b8cc348900837b876216065d7a2bf4a3665b8b to stable/rocky15:29
NamrataHi Ironicers! I have two physical machines. One for devstack ironic setup and other for provisioning. My question is will the devstack guide will work for physical baremetal machine provisioning as well?15:29
TheJuliaNamrata: You mean to be provisioned?15:30
etingofmkrai, o/ how does it go?15:30
NamrataTheJulia: Yes15:30
TheJuliaNamrata: And your following ironic's devstack docs?15:30
NamrataTheJulia: Yes15:31
TheJuliaSo you should be good then15:31
NamrataTheJulia: Thank you :)15:32
*** iurygregory_ has joined #openstack-ironic15:32
*** iurygregory has quit IRC15:35
*** gyee has joined #openstack-ironic15:35
openstackgerritJulia Kreger proposed openstack/ironic stable/rocky: Replace use of Q_USE_PROVIDERNET_FOR_PUBLIC  https://review.openstack.org/65127815:36
*** iurygregory_ is now known as iurygregory15:38
openstackgerritJulia Kreger proposed openstack/ironic master: Test commit for grenade  https://review.openstack.org/65127915:38
TheJulialets see if ^^^ fixes the gate15:38
iurygregoryTheJulia, if you have some time https://review.openstack.org/#/c/647774/ =) this will be necessary before I can backport the change to have python3 and python2 jobs for python-ironicclient in stable/stein15:38
patchbotpatch 647774 - python-ironicclient (stable/stein) - Move to zuulv3 - 2 patch sets15:38
TheJuliaiurygregory: can you post the backport anyway?15:45
iurygregorysure15:45
*** ijw has quit IRC15:50
*** ijw has joined #openstack-ironic15:51
rpittauTheJulia: if you have 12.7 seconds -> https://review.openstack.org/64769315:51
patchbotpatch 647693 - networking-generic-switch (stable/rocky) - Fix pep8 test - 1 patch set15:51
TheJulia12.7 seconds + the the context switch tax?15:51
rpittauwell that is too just too variable :/15:52
TheJuliaheh15:52
*** Namrata has quit IRC15:55
openstackgerritRiccardo Pittau proposed openstack/sushy master: Adding Thermal resource schema  https://review.openstack.org/65128415:55
*** andrein has quit IRC15:56
*** andrein has joined #openstack-ironic15:57
TheJuliahttp://logs.openstack.org/22/644822/6/check/ironic-grenade-dsvm-multinode-multitenant/4d1807b/logs/grenade.sh.txt.gz#_2019-04-09_12_35_03_102 <-- any ideas folks?15:57
*** jistr|call is now known as jistr16:00
openstackgerritMichael Turek proposed openstack/ironic master: WIP: Do not install syslinux when not deploying to an x86_64 node  https://review.openstack.org/65128516:01
*** diablo_rojo has joined #openstack-ironic16:01
rpittauTheJulia: is that broken everywhere? :/16:02
*** mjturek has quit IRC16:05
*** livelace has quit IRC16:06
*** mkrai has quit IRC16:07
*** e0ne has quit IRC16:08
*** Lucas_Gray has quit IRC16:08
rpittauTheJulia: that's really weird, shouldn't that be brbm ? http://logs.openstack.org/22/644822/6/check/ironic-grenade-dsvm/6c42ed0/logs/grenade.sh.txt.gz#_2019-04-09_13_18_04_69516:12
*** priteau has joined #openstack-ironic16:13
rpittauah no, that's the second port16:13
TheJuliano, multinode16:15
TheJuliawe create that someplace16:15
TheJuliaand I guess we're somehow not invoking that, I just don't remember where we actually create that off hand16:15
TheJuliarpittau: broke on stein->master jobs16:16
TheJuliawhich makes no sense16:16
rpittaufantastic16:16
TheJuliaAnyway, I'm going to step away shortly and go get a power supply16:16
rpittaugood luck!16:17
TheJulianeed more screens so I can hunt stuff down :(16:18
TheJuliaMy _guess_ is that somewhere we've got conditional logic around branch around networking and it is breaking16:21
TheJuliaanyway, bbl16:21
*** lucasagomes has quit IRC16:22
*** dsneddon has joined #openstack-ironic16:22
openstackgerritRiccardo Pittau proposed openstack/ironic master: [DNM|WIP] testing grenade multinode multitenant  https://review.openstack.org/65131916:25
*** dsneddon has quit IRC16:28
*** dsneddon has joined #openstack-ironic16:29
*** ijw has quit IRC16:43
dtantsurTheJulia, rpittau, I think the mDNS RFE is ready for early review: https://storyboard.openstack.org/#!/story/200539316:44
dtantsurand anyone who is curious ^^16:45
TheJuliak, will look when I get back from obtaining a PSU16:45
dtantsurcool, yeah. I'll check your comments tomorrow16:45
*** mjturek has joined #openstack-ironic16:51
mjturekrpittau wrt https://storyboard.openstack.org/#!/story/2005373 is this what you're suggesting? https://review.openstack.org/#/c/651285/16:55
patchbotpatch 651285 - ironic - WIP: Do not install syslinux when not deploying to... - 1 patch set16:55
rpittaumjturek: I had a quick look, yes, something like that16:55
rpittaumjturek: I guess you need the same for rpms16:58
mgoddardanyone able to look at https://review.openstack.org/651257?16:58
patchbotpatch 651257 - bifrost - Don't fix /etc/hosts if Rabbit is disabled - 1 patch set16:58
mgoddardhas 1x +216:58
rpittaumgoddard: I don't know, I like rabbits :P16:58
mgoddardrpittau: too late for that now :)16:59
rpittauI know :(16:59
*** dougsz has quit IRC17:00
*** derekh has quit IRC17:00
mgoddardthanks rpittau :)17:00
openstackgerritArne Wiebalck proposed openstack/ironic master: Do not tear down node upon cleaning failure  https://review.openstack.org/65094217:01
rpittaumgoddard: The needs of the many outweigh the needs of the few, or the one17:02
etingofrabbits just shift the shape turning into the wererabbits17:02
etingofmeaning still rpc17:02
*** dtantsur is now known as dtantsur|afk17:03
dtantsur|afksee you17:03
arne_wiebalckbye, everyone o/17:03
* rpittau waiting for the grenade to explode before leaving17:04
* arne_wiebalck ducks17:04
mjturekrpittau: cool I will polish it up then!17:04
*** S4ren has quit IRC17:04
*** priteau has quit IRC17:04
*** andrein has quit IRC17:05
*** dsneddon has quit IRC17:17
*** ijw has joined #openstack-ironic17:28
*** livelace has joined #openstack-ironic17:32
*** gmann is now known as gmann_afk17:40
openstackgerritMichael Turek proposed openstack/ironic master: WIP: Do not install syslinux when not deploying to an x86_64 node  https://review.openstack.org/65128517:46
*** dsneddon has joined #openstack-ironic17:51
rpittauhttps://www.irccloud.com/pastebin/5l46R0fU/18:11
rpittaurmrrrr18:11
rpittauTheJulia: the first vxlan config is working -> http://logs.openstack.org/19/651319/1/check/ironic-grenade-dsvm-multinode-multitenant/b81d27f/logs/grenade.sh.txt.gz#_2019-04-09_17_08_36_26118:12
rpittauTheJulia: the second vxlan config is failing -> http://logs.openstack.org/19/651319/1/check/ironic-grenade-dsvm-multinode-multitenant/b81d27f/logs/grenade.sh.txt.gz#_2019-04-09_17_20_45_54018:13
*** ijw has quit IRC18:13
*** e0ne has joined #openstack-ironic18:16
openstackgerritRiccardo Pittau proposed openstack/ironic stable/stein: [DNM|WIP] testing grenade multinode multitenant  https://review.openstack.org/65134518:19
*** jcoufal has quit IRC18:19
rpittauTheJulia: I think the bridge was renamed but devstack is still creating br-infra18:20
*** gmann_afk is now known as gmann18:21
rpittauI spawned a test job for stable/stein, let's see how it goes18:21
rpittaugood night! o/18:21
*** rpittau is now known as rpittau|afk18:21
*** e0ne has quit IRC18:23
*** andrein has joined #openstack-ironic18:26
diablo_rojoTheJulia, from our convo yesterday and the screen shot you provided, is it correct to say 'Deployment templates for standalone Ironic users that request allocations of bare metal nodes can now submit configuration data as opposed to pre-formed configuration drives.'18:28
diablo_rojoJust want to be sure I am getting it 100% correct18:28
*** e0ne has joined #openstack-ironic18:33
TheJuliadiablo_rojo: no, configuration drives are a completely separate thing. deploy templates allow operators to invoke steps as part of their deployment process18:33
TheJuliaSo if the configuration and drivers support raid, the it allows any ironic user to use a trait about a node to trigger ironic to execute a different process18:34
TheJuliawhich can be user define18:34
TheJuliad18:34
TheJuliaconfiguration drives hold data like ssh keys18:34
TheJuliaI wouldn't really focus on standalone, the interface is the same if there is a trait matched18:35
*** andrein has quit IRC18:36
*** andrein has joined #openstack-ironic18:37
*** betherly has joined #openstack-ironic18:38
*** dsneddon has quit IRC18:40
rpittau|afkwhile cooking I realized that what I wrote before doesn't correspond to what I observed18:40
*** dsneddon has joined #openstack-ironic18:40
diablo_rojoTheJulia, so saying that in a caption-y way for the screenshot would be like.. um..18:40
* diablo_rojo thinks18:40
rpittau|afkthe problem is that we changed the name of the bridge but we didn't reflect that in the devstack/lib/ironic18:41
rpittau|afkanyway, back to cooking18:41
TheJuliarpittau|afk: if you've got links, I can carry it forward18:43
*** betherly has quit IRC18:43
diablo_rojo'Deployment templates now allow operators to, depending on node traits and configuration, trigger ironic to execute different processes'18:44
diablo_rojo?18:44
rpittau|afkTheJulia: https://review.openstack.org/#/c/651345/18:45
patchbotpatch 651345 - ironic (stable/stein) - [DNM|WIP] testing grenade multinode multitenant - 1 patch set18:45
rpittau|afkTheJulia: https://review.openstack.org/#/c/651319/18:46
patchbotpatch 651319 - ironic - [DNM|WIP] testing grenade multinode multitenant - 1 patch set18:46
TheJulias/trigger/to trigger/   s/processes/steps during deployment./18:48
TheJuliamaybe condense the first two parts down completely. ?18:48
etingofshould one bump web api microversion if /node/management/indicators endpoint is added?18:49
TheJuliaetingof: yes18:49
*** diablo_rojo_phon has joined #openstack-ironic18:49
etingofno node object changes though18:49
etingofalright18:49
TheJuliacontrols if the endpoint is still acessile18:49
TheJuliaetingof: will we have any way of storing the last known state to the db?18:49
TheJuliaspecifically thinking the baremetal operator polling through trying to decide if it has to turn lights on18:50
etingofTheJulia, that's on on the table atm18:50
TheJuliak18:50
etingofdo we need to?18:50
etingofcaching the light18:50
TheJuliaI think dhellmann's thoughts for the baremetal operator would have made it semi-required so that the operator can decide if the light needs to be changed... and we _really_ don't want to go query light status every time... or do we?18:51
TheJuliaI guess we _could_ as long as the operator is not too aggressive with the polling... :\   At the same time, the operator shouldn't try to turn off a blinking chassis that is blinking independently because of an internal alarm18:51
TheJuliathat would be concelaing a possible issue18:52
dhellmannwe cache the power state of the host right? why treat this any differently?18:52
etingofhmmm, if we fully own the LED, may be caching makes sense18:52
dhellmanndoesn't ironic check the power state periodically?18:52
* TheJulia now begins the great quandry of where did the electrician's screw driver get off to....18:52
etingofdhellmann, oh, it does18:53
etingofdhellmann, every 60 secs18:53
dhellmannso I would just include the LED status refresh in with the other regular updates, and hold that value in the database so an API user can query it18:53
dhellmannthen at the point where the light is toggled, check it's state and don't throw an error if the state matches the desired state18:54
dhellmannso if the cache says "off" but is out of date, and ironic gets the instruction to turn the light "on" that is not an error18:54
dhellmannmake sense?18:55
etingofdhellmann, this may get complicated if we consider other LEDs such as drives and chassis18:55
*** aprice has joined #openstack-ironic18:55
dhellmannjust save them all, right?18:55
etingofdhellmann, any single node may not own the whole chassis18:55
dhellmannlike in a blade system?18:56
etingofdhellmann, consider we have two nodes managed by ironic and having individual records in the DB18:56
etingofdhellmann, so these two example nodes reside within a single chassis with a single LED18:56
dhellmannwhat type of hardware would that be?18:56
etingofdhellmann, well, that's how redfish perceives the world I think18:57
etingofdhellmann, blade servers for instance, could be?18:57
dhellmannok, that's the answer I was looking for18:57
TheJuliahold up18:58
dhellmannhow does ironic tell which components other than LEDs a blade within a blade server owns?18:58
TheJuliaLets not focus on blade connotations and chassies18:58
TheJuliain ironic, we configure the IPMI bridge settings for blades, so chassis is the blade18:58
TheJuliafor a single node machine, it is that single machine chassis18:59
dhellmannok18:59
*** betherly has joined #openstack-ironic18:59
TheJuliawe can't represent or consider odd edge cases, and the lowest end hardware I know of still has distinct LEDs per sled18:59
* TheJulia is just having one of those days and wishes the PSU didn't die this morning19:00
* etingof has a feeling that many-nodes-in-a-chassis is the new norm with redfish19:00
dhellmannis it possible for us to proceed for now as though we don't care about that case and then to solve it later?19:01
TheJuliayes, and we should keep the same mapping. We only know of the node, so if we turn on the "chassis" indicator, it means the indicator for that node19:01
openstackgerritMerged openstack/bifrost master: Don't fix /etc/hosts if Rabbit is disabled  https://review.openstack.org/65125719:01
dhellmannI mean, presumably the cached state would be read correctly for all of the different nodes that see the led, right?19:02
*** ijw has joined #openstack-ironic19:02
dhellmannTheJulia : +119:02
TheJuliawe're conflating the issue19:02
* TheJulia realizes she will need to re-wire the area behind the desk and holds off on replacing the PSU until later :(19:03
etingofI do not understand how do we synchronize cached LED states across nodes...19:03
etingoffree hanging PSU may have better cooling19:04
*** betherly has quit IRC19:04
TheJuliaetingof: we would have to have another worker poll19:09
etingofif we want to cache drive LEDs as well, that would require calling redfish BMC once per drive19:09
etingofbecause drive LED is concealed inside the drive json doc19:09
TheJuliawhy not just loop at a node level and then let the node management interface collect all related LED data19:09
TheJuliaso a separate periodic runs, get the general indicator state for ipmi, next node runs with redfish because it has that hardware type and management interface... and then that interface tries to walk it all19:10
* etingof is fearful of accidental BMC murder19:10
TheJuliaetingof: All we can do is give it a try and make it a selectable option19:11
TheJulia"It was etingof with the drive indicator LED status check in the data center..."19:11
* etingof is changing his git ID19:11
TheJulialol19:12
etingofthe other thing: these LEDs, aren't the supposed to light up by themselves as well?19:13
TheJuliaIndependent control exists _and_ systems will change the LED if there is a failure like... power supply unplugged19:13
etingofaren't they designed to serve as a failure indicator as well as a unit identification aid for humans?19:13
TheJuliaIndeed!19:13
TheJuliaOn a plus side, if we know one is blinking, we can be fairly sure a human did it or there is a failure19:14
TheJuliaUnless your older HP gear and can actually push a button19:14
* TheJulia misses that gear19:14
etingofif they tend to light up independently of ironic, would ironic have reasonably consistent cache?19:14
TheJuliaironic should try to poll but not try to assert19:14
etingofwell, unless we read them really frequently19:14
TheJuliaonly an ironic user should attempt to request the LED state to change19:15
etingofyes19:15
*** e0ne has quit IRC19:15
etingofright, so no synchronous BMC calls for indicators?19:15
etingoffrom REST API19:15
*** whoami-rajat has quit IRC19:17
etingofi.e. we gonna have periodic job filling up indicator info in node objects, then REST calls read node objects19:17
*** betherly has joined #openstack-ironic19:20
*** betherly has quit IRC19:25
TheJuliaetingof: yeah, no synch calls19:26
etingofso the web UI user has no way to get the true state of the indicator unless waited for cca 60 secs for periodic job to sync up...?19:27
TheJuliado we really want to enable the same path with get_boot_device?19:27
etingofthat's my feeling19:27
etingofbecause LEDs are supposed to be interactive, delays may render them confusing19:28
TheJuliaThat is a super valid point19:28
TheJuliawell, we can always cache and have a worker update cache too19:28
TheJuliasimplest is emulating get_boot_device and then iterating from there19:29
TheJuliathe nice thing is, we can actually cache this data if we want19:29
TheJuliaboot device, not really19:29
etingofwe can cache LEDs *if* we have a way to update it at once by user query19:29
* etingof has a vision of a confused user stating at the LEDs in his web UI and at the LEDs of his server19:31
etingofah, s/stating/staring/19:31
*** tssurya has quit IRC19:39
TheJuliafew do it and the BMC uis can already confuse that :(19:40
*** betherly has joined #openstack-ironic19:41
*** e0ne has joined #openstack-ironic19:44
*** betherly has quit IRC19:45
openstackgerritJulia Kreger proposed openstack/ironic master: [DNM|WIP] testing grenade multinode multitenant  https://review.openstack.org/65131919:51
openstackgerritJulia Kreger proposed openstack/ironic master: Temporarily mark grenade multinode as non-voting  https://review.openstack.org/65136619:58
openstackgerritJulia Kreger proposed openstack/ironic master: Revert "Temporarily mark grenade multinode as non-voting"  https://review.openstack.org/65136719:58
TheJuliaboom19:58
*** betherly has joined #openstack-ironic20:01
TheJuliajroll: rloo: If either of you are around, it would be appreciated if you single core +2+A https://review.openstack.org/#/c/651366/ so we can unblock the gate for now20:05
patchbotpatch 651366 - ironic - Temporarily mark grenade multinode as non-voting - 1 patch set20:05
rlooTheJulia: looking...20:06
*** betherly has quit IRC20:06
TheJuliathanks20:06
rloo+A'd20:07
TheJuliamuchas gracias20:08
*** baha has quit IRC20:09
jrollwell that was an easy ping to respond to :P20:11
jrollthanks rloo20:11
rloo:)20:13
*** mjturek has quit IRC20:18
*** betherly has joined #openstack-ironic20:22
*** Goneri has quit IRC20:24
*** betherly has quit IRC20:26
*** pcaruana has quit IRC20:31
*** ijw has quit IRC20:33
*** pcaruana has joined #openstack-ironic20:33
*** pcaruana has quit IRC20:36
*** pcaruana has joined #openstack-ironic20:39
*** pcaruana has quit IRC20:47
*** betherly has joined #openstack-ironic21:03
*** betherly has quit IRC21:08
*** SpamapS has quit IRC21:10
*** SpamapS has joined #openstack-ironic21:11
rpiosojroll: I'm reviewing a series of unmerged changes. Their commits have parent-child relationships. A parent was changed. Will Gerrit require its child(ren) to be rebased to the parent's most recent commit before it will merge it/them? In the Gerrit web UI, the child's "Parent(s)" SHA has a yellow dot next to it. When the mouse is hovered over that dot, it states, "Not current - rebase possible".21:14
rpiosojroll: I believe they must be rebased, but I'd like to make sure. It's been a while.21:16
* rpioso will brb21:19
*** bfournie has quit IRC21:23
*** Goneri has joined #openstack-ironic21:27
*** e0ne has quit IRC21:28
TheJuliarpioso: it will try to apply them, but if there is a merge conflict they turn red.  Each association can be older and out of date if the entire sereis is not periodically rebased and gives you that "Not current" view21:39
*** betherly has joined #openstack-ironic21:44
*** betherly has quit IRC21:49
rpiosoTheJulia: Assuming no rebasing and no merge conflicts, will they merge in their original ancestry order?21:53
TheJuliaYes, if there are no conflicts and gerrit doesn't get to oterribly confused21:54
rpiosoTheJulia: Cool. Thank you :)21:54
TheJuliathere is some weird case where it thinks there may be a merge conflict where it won't though, but that is... fairly rare21:55
openstackgerritBill Dodd proposed openstack/sushy master: Update model to support ApplyTime annotations  https://review.openstack.org/64365621:58
*** ijw has joined #openstack-ironic22:00
*** andrein has quit IRC22:10
*** rcernin has joined #openstack-ironic22:28
*** bfournie has joined #openstack-ironic22:48
*** betherly has joined #openstack-ironic22:57
*** betherly has quit IRC23:02
*** rloo has quit IRC23:02
openstackgerritRichard G. Pioso proposed openstack/ironic master: Place upper bound on python-dracclient version  https://review.openstack.org/65140823:08
*** livelace has quit IRC23:14
TheJuliadevstack fix for stable/rocky https://review.openstack.org/#/c/65127823:17
patchbotpatch 651278 - ironic (stable/rocky) - Replace use of Q_USE_PROVIDERNET_FOR_PUBLIC - 1 patch set23:17
*** betherly has joined #openstack-ironic23:18
openstackgerritJulia Kreger proposed openstack/networking-generic-switch stable/rocky: Fix pep8 test  https://review.openstack.org/64769323:21
*** betherly has quit IRC23:22
*** betherly has joined #openstack-ironic23:38
*** betherly has quit IRC23:44
*** hwoarang has quit IRC23:54
*** hwoarang has joined #openstack-ironic23:58
*** betherly has joined #openstack-ironic23:59

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