Wednesday, 2018-06-27

openstackgerritMerged openstack/ironic stable/pike: Fix error when deleting a non-existent port  https://review.openstack.org/57802600:01
*** alexpilo_ has joined #openstack-ironic00:01
*** alexpilotti has quit IRC00:04
*** mjturek has quit IRC00:07
*** alexpilotti has joined #openstack-ironic00:11
*** alexpilo_ has quit IRC00:15
*** yolanda_ has quit IRC00:16
*** alexpilo_ has joined #openstack-ironic00:21
*** alexpilotti has quit IRC00:24
*** cjloader has joined #openstack-ironic00:28
*** alexpilotti has joined #openstack-ironic00:31
*** cjloader has quit IRC00:33
*** alexpilo_ has quit IRC00:34
*** mjturek has joined #openstack-ironic00:36
*** hshiina has joined #openstack-ironic00:38
*** lifeless has quit IRC00:38
*** murrdoc has quit IRC00:39
*** bfournie has joined #openstack-ironic00:40
*** alexpilo_ has joined #openstack-ironic00:41
*** alexpilotti has quit IRC00:45
*** lifeless has joined #openstack-ironic00:45
*** mjturek has quit IRC00:50
*** alexpilotti has joined #openstack-ironic00:51
*** alexpilo_ has quit IRC00:54
*** alexpilo_ has joined #openstack-ironic01:01
*** alexpilotti has quit IRC01:05
openstackgerritMerged openstack/ironic-python-agent master: Switch from sourceforge to github for ipmitool  https://review.openstack.org/57805401:09
*** alexpilotti has joined #openstack-ironic01:11
*** phuongnh has joined #openstack-ironic01:11
*** alexpilo_ has quit IRC01:14
*** tiendc has joined #openstack-ironic01:15
*** alexpilo_ has joined #openstack-ironic01:21
*** alexpilotti has quit IRC01:24
*** cjloader has joined #openstack-ironic01:29
*** larsks has joined #openstack-ironic01:30
*** alexpilotti has joined #openstack-ironic01:31
*** stendulker has joined #openstack-ironic01:34
*** cjloader has quit IRC01:34
*** alexpilo_ has quit IRC01:35
*** alexpilo_ has joined #openstack-ironic01:41
*** alexpilotti has quit IRC01:44
*** zshi has quit IRC01:46
*** zshi has joined #openstack-ironic01:46
*** gyee has quit IRC01:51
*** alexpilotti has joined #openstack-ironic01:51
*** alexpilo_ has quit IRC01:55
*** alexpilo_ has joined #openstack-ironic02:01
*** alexpilotti has quit IRC02:04
*** alexpilotti has joined #openstack-ironic02:11
*** stendulker has quit IRC02:12
*** alexpilo_ has quit IRC02:14
*** alexpilo_ has joined #openstack-ironic02:21
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Refuse secure erase if ATA command does not work  https://review.openstack.org/57520602:22
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Provide knob to disable ata secure erase  https://review.openstack.org/57520702:22
*** alexpilotti has quit IRC02:25
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Refuse secure erase if ATA command does not work  https://review.openstack.org/57520602:26
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Provide knob to disable ata secure erase  https://review.openstack.org/57520702:26
*** rloo has quit IRC02:27
openstackgerritJulia Kreger proposed openstack/ironic-python-agent stable/queens: Switch from sourceforge to github for ipmitool  https://review.openstack.org/57824902:29
openstackgerritJulia Kreger proposed openstack/ironic-python-agent stable/pike: Switch from sourceforge to github for ipmitool  https://review.openstack.org/57825002:30
*** alexpilotti has joined #openstack-ironic02:31
*** alexpilo_ has quit IRC02:35
*** spartacloud has joined #openstack-ironic02:36
*** alexpilo_ has joined #openstack-ironic02:41
*** spartacloud has quit IRC02:42
openstackgerritJulia Kreger proposed openstack/ironic-python-agent stable/ocata: Switch from sourceforge to github for ipmitool  https://review.openstack.org/57825202:42
*** alexpilotti has quit IRC02:44
*** spartacloud has joined #openstack-ironic02:49
*** alexpilotti has joined #openstack-ironic02:51
*** alexpilo_ has quit IRC02:55
*** alexpilo_ has joined #openstack-ironic03:01
*** jaganathan has joined #openstack-ironic03:02
*** alexpilotti has quit IRC03:05
*** alexpilotti has joined #openstack-ironic03:11
*** alexpilo_ has quit IRC03:15
*** alexpilo_ has joined #openstack-ironic03:21
*** alexpilotti has quit IRC03:25
*** alexpilotti has joined #openstack-ironic03:31
*** alexpilo_ has quit IRC03:35
*** alexpilo_ has joined #openstack-ironic03:41
*** alexpilotti has quit IRC03:44
*** hamzy has joined #openstack-ironic03:47
*** alexpilotti has joined #openstack-ironic03:51
*** alexpilo_ has quit IRC03:55
*** alexpilo_ has joined #openstack-ironic04:01
*** alexpilotti has quit IRC04:04
*** alexpilotti has joined #openstack-ironic04:11
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Add security interface spec  https://review.openstack.org/57671804:11
*** alexpilo_ has quit IRC04:15
*** AlexeyAbashkin has joined #openstack-ironic04:20
*** alexpilo_ has joined #openstack-ironic04:21
*** alexpilotti has quit IRC04:25
*** cjloader has joined #openstack-ironic04:27
*** alexpilotti has joined #openstack-ironic04:31
*** cjloader has quit IRC04:31
*** alexpilo_ has quit IRC04:35
*** hwoarang has quit IRC04:40
*** alexpilo_ has joined #openstack-ironic04:41
*** alexpilotti has quit IRC04:45
*** AlexeyAbashkin has quit IRC04:49
*** alexpilotti has joined #openstack-ironic04:51
*** alexpilo_ has quit IRC04:55
openstackgerritMerged openstack/ironic master: Remove the OneView classic drivers  https://review.openstack.org/57803305:00
*** alexpilo_ has joined #openstack-ironic05:01
*** alexpilotti has quit IRC05:05
*** alexpilotti has joined #openstack-ironic05:11
*** alexpilo_ has quit IRC05:14
*** alexpilo_ has joined #openstack-ironic05:21
*** hwoarang has joined #openstack-ironic05:23
*** alexpilotti has quit IRC05:25
*** hshiina2 has joined #openstack-ironic05:29
*** alexpilotti has joined #openstack-ironic05:31
*** hshiina has quit IRC05:32
*** alexpilo_ has quit IRC05:35
*** e0ne has joined #openstack-ironic05:35
*** yangyape_ has quit IRC05:35
*** hshiina2 is now known as hshiina05:36
*** e0ne has quit IRC05:36
*** spartacloud has quit IRC05:38
*** mjura has joined #openstack-ironic05:39
*** alexpilo_ has joined #openstack-ironic05:41
*** alexpilotti has quit IRC05:45
*** yolanda_ has joined #openstack-ironic05:47
*** alexpilotti has joined #openstack-ironic05:51
*** alexpilo_ has quit IRC05:55
*** alexpilo_ has joined #openstack-ironic06:02
*** alexpilotti has quit IRC06:05
*** spartacloud has joined #openstack-ironic06:05
*** hwoarang has quit IRC06:06
*** yolanda__ has joined #openstack-ironic06:06
openstackgerritMerged openstack/ironic master: During cleaning, use current node.driver_internal_info  https://review.openstack.org/57749006:06
*** hwoarang has joined #openstack-ironic06:06
*** yolanda_ has quit IRC06:09
*** spartacloud has quit IRC06:10
*** alexpilotti has joined #openstack-ironic06:11
*** yolanda__ has quit IRC06:14
*** alexpilo_ has quit IRC06:15
*** threestrands has quit IRC06:19
*** alexpilo_ has joined #openstack-ironic06:21
*** alexpilotti has quit IRC06:25
*** namnh has joined #openstack-ironic06:28
*** dmellado has joined #openstack-ironic06:29
*** yolanda__ has joined #openstack-ironic06:30
*** alexpilotti has joined #openstack-ironic06:31
*** jcoufal has joined #openstack-ironic06:32
*** yangyapeng has joined #openstack-ironic06:33
*** AlexeyAbashkin has joined #openstack-ironic06:35
*** alexpilo_ has quit IRC06:35
*** jcoufal_ has joined #openstack-ironic06:36
*** AlexeyAbashkin has quit IRC06:37
*** jcoufal has quit IRC06:39
*** alexpilo_ has joined #openstack-ironic06:41
*** pcaruana has joined #openstack-ironic06:44
*** alexpilotti has quit IRC06:45
*** yolanda__ has quit IRC06:50
*** yangyapeng has quit IRC06:50
*** yangyapeng has joined #openstack-ironic06:51
*** yangyapeng has quit IRC06:51
*** yangyapeng has joined #openstack-ironic06:51
*** alexpilotti has joined #openstack-ironic06:51
*** yolanda__ has joined #openstack-ironic06:53
*** yolanda__ is now known as yolanda06:53
*** alexpilo_ has quit IRC06:55
*** alexpilo_ has joined #openstack-ironic07:01
*** hjensas|afk has quit IRC07:02
*** pcaruana has quit IRC07:02
*** alexpilotti has quit IRC07:05
*** tesseract has joined #openstack-ironic07:07
*** alexpilotti has joined #openstack-ironic07:11
*** alexpilo_ has quit IRC07:15
*** alexpilo_ has joined #openstack-ironic07:21
*** alexpilotti has quit IRC07:25
*** alexpilotti has joined #openstack-ironic07:32
*** alexpilo_ has quit IRC07:35
*** alexpilo_ has joined #openstack-ironic07:42
*** alexpilotti has quit IRC07:45
*** rcernin has quit IRC07:51
*** e0ne has joined #openstack-ironic07:51
*** alexpilotti has joined #openstack-ironic07:52
*** rcernin has joined #openstack-ironic07:52
*** alexpilo_ has quit IRC07:55
*** ajya has joined #openstack-ironic07:55
*** pcaruana has joined #openstack-ironic07:57
*** alexpilo_ has joined #openstack-ironic08:02
*** alexpilotti has quit IRC08:05
*** dougsz has joined #openstack-ironic08:11
*** alexpilotti has joined #openstack-ironic08:12
*** serlex has joined #openstack-ironic08:12
*** alexpilo_ has quit IRC08:15
*** milan_ has joined #openstack-ironic08:16
*** dougsz has quit IRC08:18
*** alexpilo_ has joined #openstack-ironic08:22
openstackgerritDmitry Tantsur proposed openstack/ironic master: Remove the ipmitool classic drivers  https://review.openstack.org/57809808:22
*** dtantsur|afk is now known as dtantsur08:22
dtantsurmorning ironic08:23
dtantsurwho is responsible for the IBM CI? it's still using agent_ipmitool and will be broken around today08:23
etingofgood morning ironic and dtantsur08:24
dtantsurrpioso|afk, rajinir: hey! do you think we can make the redfish job of your CI reporting on sushy?08:24
*** alexpilotti has quit IRC08:25
*** mrunge_ is now known as mrunge08:27
*** mgoddard has joined #openstack-ironic08:28
*** alexpilotti has joined #openstack-ironic08:32
*** jcoufal_ has quit IRC08:32
*** alexpilo_ has quit IRC08:35
openstackgerrityolanda.robla proposed openstack/ironic-tempest-plugin master: Add bios testing to plugin  https://review.openstack.org/56855108:38
*** alexpilo_ has joined #openstack-ironic08:42
*** alexpilotti has quit IRC08:45
*** e0ne has quit IRC08:46
*** rcernin has quit IRC08:47
*** mvpnitesh has joined #openstack-ironic08:49
*** derekh has joined #openstack-ironic08:52
*** alexpilotti has joined #openstack-ironic08:52
*** MrBerns has joined #openstack-ironic08:52
mvpniteshhi all, i've Openstack Ironic setup of Pike release. I want to install inspector in that setup, so i'm using https://docs.openstack.org/ironic-inspector/pike/install/index.html. I am not able to see rootwrap.conf and rootwrap.d folder inside "/etc/ironic-inspector/" after running "sudo ironic-inspector-rootwrap"08:54
*** d0ugal has quit IRC08:54
*** alexpilo_ has quit IRC08:55
openstackgerritIlya Etingof proposed openstack/ironic master: Add snmp driver auto discovery  https://review.openstack.org/38973608:57
*** mvpnitesh has quit IRC09:00
*** alexpilo_ has joined #openstack-ironic09:02
*** mvpnitesh has joined #openstack-ironic09:03
*** alexpilotti has quit IRC09:05
*** alexpilotti has joined #openstack-ironic09:12
*** ianychoi has quit IRC09:14
*** ianychoi has joined #openstack-ironic09:15
*** hjensas|afk has joined #openstack-ironic09:15
*** alexpilo_ has quit IRC09:15
*** alexpilotti has quit IRC09:20
*** alexpilotti has joined #openstack-ironic09:22
hshiinaregarding ironic-lib gate, I filed a bug: https://storyboard.openstack.org/#!/story/200272709:22
hshiinai haven't fully investigated today. i'll do it tomorrow. see you.09:22
*** hshiina is now known as hshiina|afk09:22
openstackgerritMark Goddard proposed openstack/networking-generic-switch master: Rework documentation layout  https://review.openstack.org/57777409:27
openstackgerritMark Goddard proposed openstack/networking-generic-switch master: Add documentation on configuration of synchronisation  https://review.openstack.org/57743009:27
openstackgerritMark Goddard proposed openstack/networking-generic-switch master: Add support for Dell PowerConnect switches  https://review.openstack.org/57338009:27
openstackgerritMark Goddard proposed openstack/networking-generic-switch master: Rework documentation layout  https://review.openstack.org/57777409:29
openstackgerritMark Goddard proposed openstack/networking-generic-switch master: Add documentation on configuration of synchronisation  https://review.openstack.org/57743009:29
openstackgerritMark Goddard proposed openstack/networking-generic-switch master: Add support for Dell PowerConnect switches  https://review.openstack.org/57338009:29
*** _milan_ has joined #openstack-ironic09:29
*** alexpilotti has quit IRC09:41
*** threestrands has joined #openstack-ironic09:45
*** threestrands has quit IRC09:45
*** threestrands has joined #openstack-ironic09:45
*** threestrands has quit IRC09:46
*** threestrands has joined #openstack-ironic09:47
*** yangyapeng has quit IRC09:58
*** yangyapeng has joined #openstack-ironic09:59
*** yangyapeng has quit IRC10:04
*** jcoufal has joined #openstack-ironic10:05
openstackgerritMerged openstack/ironic-python-agent stable/queens: Switch from sourceforge to github for ipmitool  https://review.openstack.org/57824910:18
openstackgerritMerged openstack/ironic-python-agent stable/pike: Switch from sourceforge to github for ipmitool  https://review.openstack.org/57825010:18
openstackgerritMerged openstack/ironic-python-agent stable/ocata: Switch from sourceforge to github for ipmitool  https://review.openstack.org/57825210:18
*** pcaruana has quit IRC10:31
openstackgerritDmitry Tantsur proposed openstack/ironic-lib master: Do not run API (functional) tests in the CI  https://review.openstack.org/57835510:37
dtantsurTheJulia, hshiina|afk, this should fix the ironic-lib CI hopefully ^^^10:38
*** namnh has quit IRC10:40
*** e0ne has joined #openstack-ironic10:41
*** jaganathan has quit IRC10:46
jroll\o10:47
dtantsurmorning jroll10:49
*** rbartal has quit IRC10:49
openstackgerritIlya Etingof proposed openstack/ironic master: Add snmp driver auto discovery  https://review.openstack.org/38973610:51
etingofjroll, good morning o/10:52
*** milan_ has quit IRC11:06
*** phuongnh has quit IRC11:10
*** rbartal has joined #openstack-ironic11:19
*** MattMan has quit IRC11:19
*** MattMan has joined #openstack-ironic11:20
openstackgerritMerged openstack/ironic master: Remove the iRMC classic drivers  https://review.openstack.org/57807411:24
*** niteshmvp has joined #openstack-ironic11:29
*** e0ne has quit IRC11:29
dtantsurthe last vendor driver ^^^ :)11:31
*** mvpnitesh has quit IRC11:32
MrBernsHi guys, Im currently running a bifrost standalone install with keystone enabled, and when I try to save some introspection data using ' openstack baremetal introspection data save' I get the below error11:37
MrBernsSwift failed to get object inspector_data-00000000-0000-0000-0000-000000000002 in container ironic-inspector. Error was: Object GET failed: http://localhost:8080/ironic-inspector/inspector_data-00000000-0000-0000-0000-000000000002 404 Not Found  [first 60 chars of response]11:37
MrBernsAny idea what could be causing this?11:37
openstackgerrityolanda.robla proposed openstack/ironic-tempest-plugin master: Add bios testing to plugin  https://review.openstack.org/56855111:38
*** e0ne has joined #openstack-ironic11:42
*** tiendc has quit IRC11:47
openstackgerritMerged openstack/ironic-python-agent master: Cleanup CI jobs for IPA  https://review.openstack.org/57787911:58
*** dtantsur is now known as dtantsur|brb12:06
openstackgerritDmitry Tantsur proposed openstack/ironic master: Remove the ipmitool classic drivers  https://review.openstack.org/57809812:07
*** jcoufal has quit IRC12:11
*** rh-jelabarre has joined #openstack-ironic12:11
*** dprince has joined #openstack-ironic12:17
vabadaHi ironic, I have a question regarding the baremetal_observer role. If I add it to my deployment, would I have to ensure other openstack services explicitly deny actions performed by this role? (Since I just want to (obviously) grant read access to the baremetal service...12:20
*** trown|outtypewww is now known as trown12:30
TheJuliaMrBerns: is there an /httboot/ironic-inspector folder?12:32
TheJuliaMrBerns: I suspect it didn't manage to post the object, or swift client now actually really reallly just won't post or get without it being swift.12:33
* TheJulia is really struggling to keep her eyes open this morning12:33
MrBernsThere is indeed a /httpboot/ironic-inspector folder12:33
TheJuliaMrBerns: any content?12:34
MrBernsinspector_data-243922ac-a918-4947-8b5f-b00c9423e109  inspector_data-243922ac-a918-4947-8b5f-b00c9423e109-UNPROCESSED12:34
*** zul has joined #openstack-ironic12:35
TheJuliaumm... interesting12:35
MrBernsIt seems that the first file has the inspection data were looking for12:35
TheJuliaI'm not sure how inspector is determining to post the data.. and I wonder if somewhere we have a missalignment12:40
*** zul has quit IRC12:44
*** KrisK19 has joined #openstack-ironic12:45
KrisK19hello, we are using ironic with ipa. Since we have new Dell 740 Servers with broadcom 25G cards we were unable to do manage/provide for a new added ironic host. At coreos 766.5.0 we were unable to find driver for card so we checked latest version 1745.7.0, 1745.3.1,1576.5.0, 1465.8.0, 1353.8.0 (crash with: system halted). We were testing https://stable.release.core-os.net/amd64-usr/ (and created images via make coreos) we tested as well12:50
KrisK19http://tarballs.openstack.org/ironic-python-agent/coreos/files/?C=M;O=D with the same result (we are talking about current and queens), any suggestions ?12:50
*** jcoufal has joined #openstack-ironic12:50
*** rloo has joined #openstack-ironic12:50
MrBernsTheJulia, it seems the introspection data is empty: http://paste.openstack.org/show/724397/. Does it look for this in /httpboot/ironic-inspector?12:59
*** zul has joined #openstack-ironic13:00
TheJuliaKrisK19: you could try building an IPA ramdisk using diskimage-builder13:02
KrisK19sure13:02
TheJuliaMrBerns: curious....13:03
TheJuliaKrisK19: another thing is that you might just need a kernel flag to netboot coreos... like "noapic" was once needed on some older dell poweredge gear13:04
*** _milan_ is now known as milan_13:04
TheJuliaKrisK19: you can update the pxe_append_params in ironic.conf and restart the conductor service for it to pickup new flags to append.. that is if you happen to know what might make coreos happy on your hardware.13:04
KrisK19TheJulia: I will add in config when server will boot13:05
KrisK19should be faster and te same effect ;)13:05
*** mjturek has joined #openstack-ironic13:08
TheJuliadtantsur|brb: I have a question for you on the last classic driver removal patch when you have a minute13:14
*** niteshmvp has quit IRC13:21
*** e0ne has quit IRC13:26
*** milan_ is now known as milan_|mtg13:27
*** milan_ has joined #openstack-ironic13:28
openstackgerritPierre Riteau proposed openstack/networking-generic-switch master: Fix typo in configuration help message  https://review.openstack.org/57838013:29
*** milan_|mtg has quit IRC13:31
*** dtantsur|brb is now known as dtantsur13:35
dtantsurmorning TheJulia, go ahead13:35
TheJuliawhat is morning?13:35
*** racedo has quit IRC13:35
TheJuliadtantsur: I posted it to the change, re the enabled_drives config option13:35
dtantsurTheJulia: I learned this word in the German textbook actually13:35
dtantsuraha13:35
dtantsurTheJulia: so, two options, I've been just contemplating them. we either remove it completely when we remove the classic driver mechanism. or we leave it for a cycle, but raise an error if it's non-empty13:36
TheJuliaI've pondered going back to sleep... It has been hard to keep my eyes open this morning13:36
dtantsuroh :(13:36
TheJuliadtantsur: I'd lean towards mechanism removal13:37
dtantsurTheJulia: my thought was to warn the operators that the removal is upon them13:37
dtantsurbut I'm fine either way13:37
TheJuliawell, error would be kind of a rough way to communicate that if they didn't read the release notes13:38
*** e0ne has joined #openstack-ironic13:38
TheJuliathen agian, they would be unlikely to have  afunctional deployment13:38
dtantsurTheJulia: if we remove the option, they'll end up with only hardware types13:38
dtantsurand I agree, it may or may not be what they want13:38
*** rbartal has quit IRC13:39
*** rbartal has joined #openstack-ironic13:40
*** priteau has joined #openstack-ironic13:41
*** racedo has joined #openstack-ironic13:44
TheJuliadtantsur: we have some operators that hang out here. An informal poll perhaps?14:01
*** hjensas|afk has quit IRC14:02
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Do not disable inspector periodic tasks if [inspector]enabled is False  https://review.openstack.org/57840014:04
dtantsurmaybe, but I'm a bit swamped today14:05
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Add security interface spec  https://review.openstack.org/57671814:08
openstackgerritMerged openstack/ironic master: Move parse_instance_info_capabilities() to common utils.py  https://review.openstack.org/57739814:08
openstackgerritMerged openstack/ironic-python-agent stable/pike: [LLDP] Skip NICs that say they are ready but are unreadable.  https://review.openstack.org/57036614:09
openstackgerritMerged openstack/ironic-python-agent stable/ocata: [LLDP] Skip NICs that say they are ready but are unreadable.  https://review.openstack.org/57036914:09
openstackgerritMerged openstack/ironic-lib master: Do not run API (functional) tests in the CI  https://review.openstack.org/57835514:09
dtantsurTheJulia: do we still need https://review.openstack.org/#/c/568828/ ?14:09
patchbotpatch 568828 - ironic-lib - Slightly increase job timeouts14:09
TheJuliadtantsur: I doubt it since you turned off the functional tests14:10
dtantsuryep, and that patch landed without rechecks14:10
TheJuliaexactly so we're now well under the thresholds, so I think I can abandon it. We do need to rework job names soon-ish, but thats not a big deal14:11
dtantsuryep14:11
dtantsurjob names are now quite confusing14:11
TheJuliayup :(14:11
dtantsurso, setting CI status to green? any other known issues?14:11
TheJuliamgagne: mgoddard: hogepodge: penick: random ops-centric poll: With the removal of classic drivers, do you feel that if one is configured in ironic.conf that Ironic should issue an error, or that ironic should just silently remove the deprecated configuration options and move on in life?14:12
TheJuliadtantsur: your change will need to be backported on ironic-lib stable branches, but as of right now, I think we're good14:12
dtantsurright, backporting14:13
openstackgerritDmitry Tantsur proposed openstack/ironic-lib stable/queens: Do not run API (functional) tests in the CI  https://review.openstack.org/57840514:13
jrollif it's not a pain to leave it for a cycle I don't see why not14:13
dtantsurTheJulia: actually, backporting them is optional, since the breaking configuration change only landed on master14:14
TheJuliajroll: I guess that is the middle ground and we can just issue a warning, dtantsur was mentioning error which  makes me think conductor doesn't start14:14
TheJuliadtantsur: oh! okay!14:14
jrollTheJulia: I mean error, I think14:14
dtantsuryeah. proposed the queens backport anyway, will make the CI a bit lighter14:14
jrollwe've been issuing warnings, no?14:15
dtantsuryep14:15
TheJuliaIt is already deprecated, and deprecated through log messages is... not ideal, I guess error makes sense to halt the conductor because the operator really needs to change the configuration14:15
mgoddardTheJulia: the normal behaviour would be to ignore options that have been removed, right? If we've been through a deprecation cycle then I see no reason why drivers would be different. You'll find out soon enough if your driver has been removed :)14:16
rlooi think error/halt ^^ Otherwise, even though we don't support it, they may think we still do.14:16
rlooit is on startup of conductor, right, so error'ing there should be fine.14:16
mgoddardOTOH I am more plugged into development than some operators, so I know it's coming14:16
* jroll == mgoddard14:17
* TheJulia blinks at the two mgoddard's14:17
TheJuliaoh no, that was true14:17
* TheJulia needs more coffee14:17
mgoddardjroll > mgoddard14:17
jrollnah :)14:17
mgoddard:p14:18
openstackgerritDmitry Tantsur proposed openstack/ironic master: Do not disable inspector periodic tasks if [inspector]enabled is False  https://review.openstack.org/57840014:19
*** KrisK19 has quit IRC14:19
dtantsurfound this weird bit ^^^ would appreciate someone double-checking my conclusions14:20
rloodtantsur: what can we use to disable that periodic task then? I mean, if someone isn't going to use inspector at all, there should be a way to disable that, no?14:22
dtantsurrloo: not enabling the 'inspect' interface14:22
dtantsurit is enabled/disabled indirectly by enabling/disabling the drivers and/or interfaces that use it14:23
rloodtantsur: is there code somewhere that doesn't invoke _periodic_check-result()? I thought periodic tasks would always get invoked if not disabled14:23
dtantsurrloo: they're only collected from enabled drivers and interfaces14:23
openstackgerritDmitry Tantsur proposed openstack/ironic master: Deprecate [inspector]enabled option  https://review.openstack.org/57841114:23
rloodtantsur: ah yes, this is in the inspector code, not the conductor.14:24
rloodtantsur: in that case, it makes sense.14:24
*** threestrands has quit IRC14:30
*** racedo has quit IRC14:30
NobodyCamGood Morning Ironic'ers14:33
NobodyCamand (ofc) Happy Hump Day!14:33
openstackgerritDmitry Tantsur proposed openstack/ironic stable/queens: Do not disable inspector periodic tasks if [inspector]enabled is False  https://review.openstack.org/57841914:33
dtantsurtrying backport ^^ to make sure it does not break old drivers14:34
dtantsurmorning NobodyCam14:34
NobodyCamMorning dtantsur .. thats an interesting title for a patch14:35
dtantsurI can imagine :)14:35
NobodyCam:) heheheh14:35
*** rh-jelabarre has quit IRC14:37
*** mjturek has quit IRC14:41
TheJuliais ti wednesday?14:44
* TheJulia has no idea what day it is14:44
* TheJulia might be over stating that feeling14:44
*** alexpilotti has joined #openstack-ironic14:45
*** markguz has joined #openstack-ironic14:46
*** alexpilotti has quit IRC14:49
*** hamzy has quit IRC14:49
*** hamzy has joined #openstack-ironic14:52
*** hamzy has quit IRC14:53
*** hamzy has joined #openstack-ironic14:54
*** mjturek has joined #openstack-ironic14:56
*** rbartal has quit IRC14:59
*** mjura has quit IRC15:01
*** stendulker has joined #openstack-ironic15:04
NobodyCamTheJulia: HEHEHE :p15:05
*** mjturek has quit IRC15:05
*** zul has quit IRC15:05
*** mjturek has joined #openstack-ironic15:05
*** rpioso|afk is now known as rpioso15:09
rpiosoGood morning :)15:09
dtantsurmorning rpioso. have you seen my question in the scrollback?15:10
rpiosodtantsur: Morning. Back scrolling ...15:11
NobodyCamMorning rpioso :)15:12
rpiosoNobodyCam: Happy Wednesday15:12
NobodyCam:)15:13
*** alexpilotti has joined #openstack-ironic15:13
rpiosodtantsur: Does a report CI job gate the merge of a change?15:16
rpiosos/report/reporting/15:17
dtantsurrpioso: not for 3rd party CI. but we'll at least know we're breaking it15:24
rpiosodtantsur: So they would be similar to what we have in ironic today, correct?15:25
dtantsurright15:25
*** jcoufal has quit IRC15:25
rpiosodtantsur: I'll check with rajinir and my team.15:26
dtantsurcool, thanks15:26
stendulkerGood morning dtantsur, NobodyCam , rpioso, TheJulia, rloo15:26
dtantsurprobably we need the same for dracclient15:26
dtantsurhey stendulker15:27
NobodyCamMorning stendulker :)15:27
rpiosodtantsur: np15:27
rpiosostendulker: o/15:27
stendulkerdtantsur, NobodyCam, rpioso : o/15:27
stendulkerrpioso: One query about drac implementation of RAID15:27
stendulkerrpioso: I understand drac RAID performs OOB configuration and needs reboot for it to complete it.15:28
rpiosostendulker: Please ask.15:28
rloohiya stendulker :)15:28
NobodyCamMorning rloo :)15:28
rloohi NobodyCam!15:28
stendulkerAnd it resumes the cleaning upon receiving the job completion post reboot.15:28
rpiosorloo: Good morning15:28
rloomorning rpioso:)15:29
stendulkerrpioso: Do you ensure that the agent is up and running before invoking resumption of cleaning?15:29
stendulkerrloo: o/15:30
stendulkerrpioso: If so, how do you ensure that?15:30
rpiosostendulker: The idrac h/w type reboots the server to apply the RAID configuration changes made by the RAID clean step. Those are performed by a configuration job. It then periodically monitors the status of the job. Once it sees that the job has successfully completed it resumes cleaning.15:31
rpiosostendulker: Bear in mind that the idrac's RAID clean steps are manual and OOB.15:31
*** baha has joined #openstack-ironic15:32
*** KrisK19 has joined #openstack-ironic15:32
stendulkerrpioso: Yes, I undestand that. To resume cleaning, the agent must be running on the bare metal post reboot. How is that ensured?15:32
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client master: Support passing manage_boot argument in Python API  https://review.openstack.org/33947915:33
rpiosostendulker: It calls notify_conductor_resume_clean() and considers its work done.15:33
*** mjturek has quit IRC15:33
stendulkerrpioso: notify_conductor_resume_clean() would try to call next cleaning step. But if the agent is not running, wont the cleaning fail?15:34
*** mjturek has joined #openstack-ironic15:34
*** yangyapeng has joined #openstack-ironic15:34
* rpioso is checking a workflow that uses both RAID clean steps back-to-back15:34
*** KrisK19_ has joined #openstack-ironic15:36
rpiosostendulker: That workflow performs them one at a time, instead of back-to-back. So, I don't know from experience whether cleaning would fail.15:36
*** KrisK19 has quit IRC15:37
*** KrisK19_ is now known as KrisK1915:37
stendulkerrpioso: I think it would fail, as in Ironic code we do not wait for agent to run as part of notify_conductor_resume_clean()15:37
rpiosostendulker: Does a cleaning with multiple manual, OOB clean steps require the agent to be running before proceeding onto the next step?15:37
rpiosostendulker: Kk15:38
stendulkerI think if the next step is inband, it matters.15:38
stendulkerThe manual steps could be mix of inband and OOB15:38
rpiosostendulker: Please bear in mind that I didn't develop the code we're discussing.15:38
*** yangyapeng has quit IRC15:38
rpiosostendulker: That's where I thought you might be going :)15:39
stendulkerrpioso: :)15:39
rpiosostendulker: Shouldn't the conductor do the right thing when notify_conductor_resume_clean() is called?15:39
rpiosostendulker: It doesn't seem to me to be a driver responsibility.15:40
stendulkerrpioso: It assumes agent is running.15:40
rpiosostendulker: Where "It" is the conductor?15:40
stendulkerrpioso: yes15:41
*** KrisK19 has quit IRC15:41
rpiosostendulker: Does it make the same assumption when it executes the first clean step in the sequence?15:41
rpiosostendulker: Or does it ensure that the agent is running before kicking that off?15:42
stendulkerrpioso: Yes. It waits for heartbeat from agent to execute first clean step15:42
rpiosostendulker: It seems to me that it should do the same thing for subsequent steps.15:42
*** gyee has joined #openstack-ironic15:42
stendulkernotify_conductor_resume_clean() do not follow that path15:43
stendulkerTheJulia, rloo, dtantsur: I need your opinion on the OOB RAID configuration which needs a reboot to make it effective15:43
rpiosostendulker: If memory serves, I believe the agent terminated early in the processing of the idrac's RAID clean step when it realized it had nothing to do. That is well before the manual, OOB RAID clean step did its thing.15:44
*** KrisK19 has joined #openstack-ironic15:44
stendulkerThere is a implementation for irmc RAID which takes same approach as idrac RAID wherein it reboots the baremetal and resumes cleaning without waiting for agent to come up.15:44
stendulkerhttps://review.openstack.org/#/c/512979/15:45
patchbotpatch 512979 - ironic - Support RAID configuration for BM via iRMC driver15:45
stendulkerThe HPE RAID also has similar requirement but it is trying to split that into 2 clean steps. One OOB to create RAID (this will call reboot) and other one inband to read RAID config15:46
stendulkerhttps://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ilo-oob-raid.html15:46
* TheJulia reads15:46
rpiosostendulker: I'm wondering if the agent shut itself down or if the RAID clean step caused it to terminate when it power cycled the server.15:46
stendulkerrpioso: It would get shutdown when server is power cycled.15:47
TheJuliastendulker: regarding ensured, the heartbeat mechanism is what drives the next cleaning step, so it becomes IPA initiated at that point15:47
rpiosostendulker: I believe it shut itself down.15:47
*** cjloader has joined #openstack-ironic15:48
stendulkerTheJulia: When driver calls notify_conductor_resume_clean() conductor would not wait for heartbeat15:48
TheJuliaO.O15:48
stendulkerTheJulia: https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/drac/raid.py#L889-L89215:48
rpiosostendulker: Perhaps it only shuts itself down if all of the clean steps are OOB.15:48
*** pbourke has quit IRC15:48
*** KrisK19 has quit IRC15:49
rpiosostendulker: That's been the case in my experience.15:49
*** pbourke has joined #openstack-ironic15:50
stendulkerTheJulia: Am I missing something here?15:51
TheJuliaso yeah, we shouldn't really be calling continue_node_clean on the conductor from with-in a driver15:52
stendulkerTheJulia: During normal cleaning, upon receiving heartbeat it would start cleaning15:53
stendulkerhttps://github.com/openstack/ironic/blob/master/ironic/drivers/modules/agent_base_vendor.py#L318-L33015:53
TheJuliaipa should be able to return to service, unless we're talking about manual cleaning and oob15:53
stendulkerTheJulia: Its OOB clean step -> Reboot -> Driver directly calls manager_utils.notify_conductor_resume_clean()15:54
rpiosoTheJulia: The idrac h/w type supports manual, OOB RAID cleaning.15:54
stendulkerTheJulia: All this in manual cleaning15:54
TheJuliaso in theory, there is not another clean step15:55
stendulkerTheJulia: I think this will cause issue if the step after OOB clean step is an inband clean step15:55
TheJuliaif osmeone is just setting the raid, then this is not an issue, but if someone is trying to assert multiple clean steps then it becomes problematic unless they are all oob15:55
TheJuliastendulker: I am in agreement15:55
TheJuliastill sifting through the code though15:56
stendulkerTheJulia: So what do you suggest? All 3 vendors (drac- implemented, irmc and hpe in review) have similar requirement15:56
stendulkerTheJulia: It looks fine today, but this  may hit us in future if we were to have another inband clean-step post OOB RAID step15:57
TheJuliawait a second...15:57
openstackgerritDmitry Tantsur proposed openstack/ironic master: [WIP] Remove support for creating and loading classic drivers  https://review.openstack.org/57845715:59
dtantsursuch a great feeling to work on ^^^ :)15:59
TheJuliastendulker: I believe we need to opportunisticly try to perform the next step. It should fail... or if it does, be greaceful about it, we just likely need to convey through to the _do_next_clean_step in manager.py that we can gracefully break there and that we need to remain in clean wait. presently any failure raising any exception goes to the cleaning_error_handler which wipes out cleaning step data preventing16:02
TheJuliaresumption.16:02
*** baha has quit IRC16:03
stendulkerTheJulia: So you suggest the immediate clean step after reboot fails and then we try to resume the cleaning?16:05
*** mjturek has quit IRC16:05
stendulkerby going into cleanwait and wait for heartbeat...16:06
TheJuliastendulker: make it tollerant of a failure, or explicitly catch failures that would be indicitive of connectivity failures, gracefully exit the method and allow cleaning to pickup on the next heartbeat16:06
TheJuliayes16:06
TheJuliathe actual next step should be updated I believe, and it hsould proceed, but I've not walked the heartbeat to cleaning code to confirm that is how it will execute16:06
stendulkerTheJulia: ok, that should work I suppose, we can re-attempt the clean step if there is a connection fails16:08
*** milan_ has quit IRC16:08
TheJuliastendulker: so I think this is something we need to fix sooner rather than later because as soon as bios interfaces are usable, people will try grouping oob steps16:09
TheJuliastendulker: can you create a story on storyboard for us to track this?16:10
stendulkerTheJulia: yes, I will try to push a patch for the same16:10
TheJuliastendulker: awesome, thanks!16:10
stendulkerTheJulia: ok, will create a story for this16:10
stendulkerTheJulia: Thank you  :)16:11
stendulkerrpioso: Thank you :)16:11
rpiososetndulker, TheJulia: Thank you16:11
rpiosos/setndulker/stendulker/16:11
rpiosostendulker: np16:12
*** e0ne has quit IRC16:13
rpiosostendulker, TheJulia: Do we believe multiple OOB clean steps would presently fail?16:13
TheJuliarpioso: I believe so16:13
TheJuliarpioso: that is based upon walking the code, althouht it might take a while or race with the periodic task triggering the next step16:14
rpiosoTheJulia: Got it16:14
dtantsurpas-ha: hey! are you still working on http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/ipxe-boot-interface.html ? Got somebody asking about it downstream.16:14
dtantsurTheJulia: do you known who to ping about the IBM CI? they're still using agent_ipmitool16:17
TheJuliadtantsur: mjturek typically is who I've pinged16:18
TheJuliahamzy might know ^^^16:18
dtantsuryeah, but he does not seem online16:18
*** mjturek has joined #openstack-ironic16:22
stendulkerLogging off now. Good day all :)16:22
*** stendulker has quit IRC16:23
*** baha has joined #openstack-ironic16:23
*** serlex has quit IRC16:23
*** racedo has joined #openstack-ironic16:27
openstackgerritNguyen Van Trung proposed openstack/ironic master: Support RAID configuration for BM via iRMC driver  https://review.openstack.org/51297916:29
*** tesseract has quit IRC16:34
openstackgerritIlya Etingof proposed openstack/ironic master: Caching of PDU autodiscovery  https://review.openstack.org/39923416:36
*** mgoddard has quit IRC16:44
*** MrBerns has quit IRC16:45
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Periodic task execution mode  https://review.openstack.org/57848016:46
TheJuliaThat ^^^ might be completely and totally half baked, but it seems to be a workable idea to scaling the conductor without fun things like processes stomping on each other16:47
TheJuliaand keeps a single process "in charge", but the tasks run in the background16:48
*** derekh has quit IRC16:51
*** spartacloud has joined #openstack-ironic16:52
*** spartacloud has quit IRC16:57
dtantsurcool16:58
* dtantsur needs fresh brain to properly read it16:59
TheJuliaSince the big scale issue seems to be the tasks and consumption of workers when external things execute17:00
dtantsurTheJulia: but JFYI futurist allows random jitter when running tasks IIRC17:00
TheJuliaThere is the whole bit of iscsi write out cases... but maybe we flip this around later and have workers do it since they will still be on the rpc bus17:00
TheJuliadtantsur: that was the word I was looking for!17:00
TheJuliajitter!17:00
dtantsur:)17:00
dtantsurokay, g'night everyone17:04
*** dtantsur is now known as dtantsur|afk17:04
*** trown is now known as trown|lunch17:16
*** mgoddard has joined #openstack-ironic17:21
*** yangyapeng has joined #openstack-ironic17:24
* TheJulia goes and finds coffee and lunch17:28
NobodyCammmm Coffee17:28
*** yangyapeng has quit IRC17:28
*** zul has joined #openstack-ironic17:33
*** racedo has quit IRC17:36
*** murrdoc has joined #openstack-ironic17:41
openstackgerritMerged openstack/networking-generic-switch master: Fix typo in configuration help message  https://review.openstack.org/57838017:43
*** dprince has quit IRC17:48
openstackgerritMerged openstack/ironic-inspector master: Add manage_boot parameter to introspection API  https://review.openstack.org/31680117:55
*** dprince has joined #openstack-ironic18:07
*** yolanda_ has joined #openstack-ironic18:11
*** trown|lunch is now known as trown18:14
*** yolanda has quit IRC18:14
*** e0ne has joined #openstack-ironic18:16
*** hjensas|afk has joined #openstack-ironic18:17
hamzydtantsur|afk, I can ask questions internally..18:21
hamzymmedvede still works on our CI, right?18:21
*** rh-jelabarre has joined #openstack-ironic18:22
mmedvedeyes18:25
mmedvededtantsur|afk: but yes, mjturek may know more about switching IBM pkvmci from agent_ipmitool18:27
openstackgerritSergii Golovatiuk proposed openstack/ironic master: Reboot server after 10 tries  https://review.openstack.org/57851218:28
*** e0ne has quit IRC18:32
mjturekdtantsur|afk: we need to switch to the equivalent hardware type, correct?18:33
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Add Boot from Ramdisk spec  https://review.openstack.org/57671718:37
TheJuliamjturek: yes18:41
mjturekTheJulia: alright I'll take a look at the jobs and get that moving18:41
openstackgerritChuck Short proposed openstack/ironic-python-agent master: Switch to using stestr  https://review.openstack.org/57852118:44
*** e0ne has joined #openstack-ironic18:44
*** mgoddard has quit IRC18:50
openstackgerritChuck Short proposed openstack/ironic-lib master: Switch to using stestr  https://review.openstack.org/57852218:50
openstackgerritChuck Short proposed openstack/ironic-inspector master: Switch to using stestr  https://review.openstack.org/57852418:52
*** markguz__ has joined #openstack-ironic18:58
*** murrdoc has quit IRC19:00
*** markguz has quit IRC19:01
*** mjturek has quit IRC19:15
*** baha has quit IRC19:15
*** athomas has quit IRC19:21
*** rh-jelabarre has quit IRC19:34
*** markguz__ has quit IRC19:38
*** markguz has joined #openstack-ironic19:39
*** markguz__ has joined #openstack-ironic19:43
*** markguz has quit IRC19:44
*** markguz__ has quit IRC19:45
*** markguz has joined #openstack-ironic19:46
*** markguz__ has joined #openstack-ironic19:48
TheJuliaolivierbourdon38: up for the challenge of hacking together some unit tests? :)19:48
*** markguz has quit IRC19:50
*** AlexeyAbashkin has joined #openstack-ironic20:02
*** dprince has quit IRC20:15
*** ajya has quit IRC20:17
*** zul has quit IRC20:27
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client master: Provide proper error message if interface name is invalid  https://review.openstack.org/57855220:32
*** e0ne has quit IRC20:42
TheJuliaSimple API client patch for a quick review https://review.openstack.org/#/c/316801/20:43
patchbotpatch 316801 - ironic-inspector - Add manage_boot parameter to introspection API (MERGED)20:43
TheJuliaerr20:43
TheJuliawrong patch20:43
*** yolanda_ has quit IRC20:43
TheJuliahttps://review.openstack.org/#/c/339479/20:44
patchbotpatch 339479 - python-ironic-inspector-client - Support passing manage_boot argument in Python API20:44
*** yolanda_ has joined #openstack-ironic20:44
*** AlexeyAbashkin has quit IRC20:45
*** MrBerns has joined #openstack-ironic20:47
openstackgerritAleksandra Bezborodova proposed openstack/sushy-tools master: Rewrite unit tests for emulator  https://review.openstack.org/56672720:48
*** yolanda has joined #openstack-ironic20:53
*** MrBerns has quit IRC20:54
*** yolanda_ has quit IRC20:55
*** trown is now known as trown|outtypewww20:55
*** yolanda_ has joined #openstack-ironic20:59
*** AlexeyAbashkin has joined #openstack-ironic21:01
*** yolanda has quit IRC21:02
*** jtomasek has quit IRC21:03
*** zul has joined #openstack-ironic21:05
*** priteau has quit IRC21:10
*** cjloader has quit IRC21:10
*** cjloader_ has joined #openstack-ironic21:10
*** zul has quit IRC21:12
*** AlexeyAbashkin has quit IRC21:12
*** cjloader_ has quit IRC21:15
openstackgerritMerged openstack/ironic master: Add snmp driver auto discovery  https://review.openstack.org/38973621:23
openstackgerritMerged openstack/ironic master: Add helper method for testing node fields  https://review.openstack.org/57785621:24
openstackgerritMerged openstack/ironic-lib stable/queens: Do not run API (functional) tests in the CI  https://review.openstack.org/57840521:24
*** jtomasek has joined #openstack-ironic21:36
*** rcernin has joined #openstack-ironic21:50
openstackgerritJulia Kreger proposed openstack/ironic-python-agent master: Enable Auto VM engine for CoreOS CI jobs  https://review.openstack.org/57856621:56
openstackgerritJulia Kreger proposed openstack/ironic master: ramdisk deploy driver  https://review.openstack.org/56894022:08
TheJuliaIf I can get another core to take a look at  https://review.openstack.org/57856622:10
patchbotpatch 578566 - ironic-python-agent - Enable Auto VM engine for CoreOS CI jobs22:10
TheJuliaI would greatly appreciate it22:10
TheJuliaand buy beer22:10
TheJulialots of beer22:10
*** cjloader has joined #openstack-ironic22:35
*** cjloader has quit IRC22:40
*** kzaitsev1pi has joined #openstack-ironic22:50
*** kzaitsev_pi has quit IRC22:52
*** pmannidi has joined #openstack-ironic22:53
*** zshi has quit IRC23:10
*** yolanda__ has joined #openstack-ironic23:11
*** yolanda_ has quit IRC23:15
*** ianychoi has quit IRC23:16
*** yolanda__ has quit IRC23:18
*** ianychoi has joined #openstack-ironic23:19
*** yolanda has joined #openstack-ironic23:20
*** yolanda_ has joined #openstack-ironic23:21
*** yolanda has quit IRC23:24
*** yolanda__ has joined #openstack-ironic23:25
openstackgerritBob Fournier proposed openstack/python-ironic-inspector-client master: Provide proper error message if interface name is invalid  https://review.openstack.org/57855223:27
*** yolanda_ has quit IRC23:29
*** yolanda has joined #openstack-ironic23:29
*** yolanda__ has quit IRC23:31
*** cjloader has joined #openstack-ironic23:36
*** cjloader has quit IRC23:40
*** mjturek has joined #openstack-ironic23:44
*** mjturek has quit IRC23:49
*** markguz__ has quit IRC23:58
*** markguz has joined #openstack-ironic23:59
*** alex_xu has quit IRC23:59

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