Wednesday, 2018-10-17

*** rloo has quit IRC00:13
*** mcarden has joined #openstack-ironic00:14
*** gyee has quit IRC00:17
*** bfournie has joined #openstack-ironic00:37
*** hoangcx has quit IRC01:07
*** hshiina has joined #openstack-ironic01:35
*** MattMan_ has quit IRC01:45
*** MattMan_ has joined #openstack-ironic01:46
*** Chaserjim has quit IRC01:48
*** Chaserjim has joined #openstack-ironic01:49
*** Chaserjim has quit IRC01:54
*** Chaserjim has joined #openstack-ironic02:20
*** lucasagomes has quit IRC04:03
*** hshiina has quit IRC04:04
*** hshiina has joined #openstack-ironic04:04
*** Chaserjim has quit IRC04:27
*** skazi has quit IRC04:44
*** pcaruana has quit IRC04:46
*** e0ne has joined #openstack-ironic05:05
openstackgerritAnkit Kumar proposed openstack/ironic-specs master: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers  https://review.openstack.org/61123805:25
*** bdodd has quit IRC05:25
openstackgerritparesh sao proposed openstack/sushy master: Requests session keyword arguments for sushy connector  https://review.openstack.org/60780905:26
*** bdodd has joined #openstack-ironic05:29
*** hjensas has quit IRC06:12
iurygregorygood morning06:18
openstackgerritOlivier Bourdon proposed openstack/bifrost master: [DNM] Testing CI paramaters  https://review.openstack.org/60748006:31
*** pcaruana has joined #openstack-ironic06:34
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Begin change over to use python3  https://review.openstack.org/58832506:37
*** gabys has quit IRC06:41
*** moshele has joined #openstack-ironic06:43
*** e0ne has quit IRC06:55
*** gabys has joined #openstack-ironic06:57
*** adrianc has joined #openstack-ironic07:02
*** Chaserjim has joined #openstack-ironic07:03
*** gabys has quit IRC07:06
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Fix CI for CentOS  https://review.openstack.org/60748007:08
*** Chaserjim has quit IRC07:08
*** rcernin has quit IRC07:09
*** Chaserjim has joined #openstack-ironic07:10
*** hshiina_ has joined #openstack-ironic07:13
*** hjensas has joined #openstack-ironic07:14
*** Chaserjim has quit IRC07:15
*** hshiina has quit IRC07:17
etingofiurygregory, o/07:29
iurygregoryetingof, morning =)07:29
*** sthussey has quit IRC07:45
*** mmethot is now known as mmethot|away07:49
*** dougsz has joined #openstack-ironic08:05
*** Chaserjim has joined #openstack-ironic08:14
*** e0ne has joined #openstack-ironic08:15
*** Chaserjim has quit IRC08:19
openstackgerritDigambar proposed openstack/ironic master: Fix CPU count returned by introspection in Ironic iDRAC driver  https://review.openstack.org/60998208:23
*** tssurya has joined #openstack-ironic08:26
derekhTheJulia: will do08:45
*** lucasagomes has joined #openstack-ironic08:48
*** hshiina_ has quit IRC08:56
openstackgerritAnkit Kumar proposed openstack/ironic-specs master: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers  https://review.openstack.org/61123809:02
*** dtantsur|afk is now known as dtantsur09:12
olivierbmorning everyone09:13
iurygregorymorning =)09:13
olivierbI have a question about https://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-ironic-install/defaults/main.yml#L9:L1309:13
olivierbwhy are the *_source_install true in some cases and false in others ? just trying to understand differences of handling for all components09:14
dtantsurmorning ironic09:14
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Allow streaming raw partition images  https://review.openstack.org/60570709:16
iurygregorymorning dtantsur o/09:17
openstackgerritDmitry Tantsur proposed openstack/ironic master: [DNM] Allow streaming raw partition images  https://review.openstack.org/61001309:17
iurygregoryolivierb, may this will give you an idea i think https://github.com/openstack/bifrost/blob/0232b6b4e507963c035c989a412869903bbe4276/releasenotes/notes/openstacksdk-addition-efec3878ef988295.yaml09:19
iurygregoryits a way to let configurable so you can install from pypi thats what i understood (not 100% sure)09:20
iurygregorys/may/maybe09:21
olivierbiurygregory thanks I now understand this point but wonderning for instance why *client_source_install are false compared to ironicinspector_source_install true for instance09:23
olivierbs/for instance// ;-)09:23
sambetts|afkwe typically always install the clients from released versions, even in CI, however the services were never published to pypi so we always had to install them from source until recently09:24
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Begin change over to use python3  https://review.openstack.org/58832509:24
iurygregorysambetts|afk, tks!  olivierb see his answer =)09:27
openstackgerritAnkit Kumar proposed openstack/ironic-specs master: Add 'inspect_ports' as a 'driver_info' parameter to iLO drivers  https://review.openstack.org/61123809:33
*** Emine has joined #openstack-ironic09:40
openstackgerritNguyen Van Trung proposed openstack/ironic master: [WIP] Support firmware upgrade for irmc driver  https://review.openstack.org/59999209:43
olivierbsambetts|afk ok this makes sense, however if you look at http://logs.openstack.org/25/588325/35/check/bifrost-integration-tinyipa-ubuntu-xenial/c90fe2b/job-output.txt.gz (Python3) this may be an issue, currently testing this09:44
sambetts|afkolivierb: the cannot uninstall packages from the system is the main reason for switching to using virtualenvs for everything I think09:46
olivierbsambetts|afk got it09:47
olivierbthx09:47
sambetts|afkI don't think switching between packaged version and source version will make a difference on that, because either way the dependecies are the same, and if they conflict with the system install packages then pip will explode trying to install it09:48
sambetts|afkuninstall it09:48
sambetts|afkso unless you "apt-get uninstall pyyaml" first then it makes pip sad09:49
*** adrianc has quit IRC09:53
openstackgerritMarkos Chandras (hwoarang) proposed openstack/bifrost master: playbooks: Convert venv_python_path to proper role  https://review.openstack.org/61089810:05
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Fix typo  https://review.openstack.org/61128210:07
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Begin change over to use python3  https://review.openstack.org/58832510:08
*** adrianc has joined #openstack-ironic10:09
*** Emine has quit IRC10:10
*** Emine has joined #openstack-ironic10:13
*** Emine has quit IRC10:30
*** e0ne has quit IRC10:45
*** Emine has joined #openstack-ironic10:50
*** Chaserjim has joined #openstack-ironic10:52
*** Chaserjim has quit IRC10:57
jrollmorninnnnng11:05
iurygregorymorning jroll o/11:06
*** moshele has quit IRC11:22
*** bfournie has quit IRC11:22
*** pcaruana has quit IRC11:28
*** moshele has joined #openstack-ironic11:29
dtantsurmorning jroll11:43
*** e0ne has joined #openstack-ironic11:46
*** dtantsur is now known as dtantsur|brb11:48
*** trown|outtypewww is now known as trown11:56
*** bfournie has joined #openstack-ironic11:57
iurygregorydtantsur|brb, tks for the +A ;) do you want a patch to change what you said? =)11:59
*** mmethot|away is now known as mmethot12:03
*** moshele has quit IRC12:04
openstackgerritJim Rollenhagen proposed openstack/ironic-specs master: No IPA to conductor communication  https://review.openstack.org/21220612:05
*** moshele has joined #openstack-ironic12:05
jrollthis becomes more work on each revision12:05
*** rh-jelabarre has joined #openstack-ironic12:06
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Fix CI for CentOS  https://review.openstack.org/60748012:19
iurygregoryjroll, do you have a minute for a newbie question?12:23
jrolliurygregory: you can always ask, I may not have the answer :)12:23
*** pcaruana has joined #openstack-ironic12:24
iurygregoryjroll, i was reviewing a patch, if a new attribute is added to a class i should keep the alphabetical order right? like https://review.openstack.org/#/c/547356/27/ironic/drivers/base.py12:25
patchbotpatch 547356 - ironic - Implement basic interfaces for GraphicalConsole In... - 27 patch sets12:25
iurygregoryit should go after deploy the new attribute i think12:26
jrolliurygregory: in general, it doesn't matter. since we have it alphabetical there, I'd prefer to keep it that way. but it's a bit of a nit, I guess12:26
iurygregoryjroll, tks =)12:27
*** moshele has quit IRC12:49
*** moshele has joined #openstack-ironic12:51
*** Chaserjim has joined #openstack-ironic12:53
*** dtantsur|brb is now known as dtantsur12:55
*** Chaserjim has quit IRC12:57
*** dnuka has joined #openstack-ironic12:59
*** hjensas has quit IRC13:06
openstackgerritDmitry Tantsur proposed openstack/ironic master: Allow streaming raw partition images  https://review.openstack.org/61001313:08
dtantsurthis seems to work now ^^^13:08
iurygregory\o/13:12
*** sthussey has joined #openstack-ironic13:15
*** Chaserjim has joined #openstack-ironic13:27
TheJuliamgoddard: would you please press the restore button on https://review.openstack.org/#/c/600411/ ?13:30
patchbotpatch 600411 - governance - Add tenks under Ironic governance (ABANDONED) - 1 patch set13:30
TheJuliajroll: sorry :(13:31
*** Chaserjim has quit IRC13:32
TheJuliaCan I twist anyone's arm for an ipa review? https://review.openstack.org/#/c/592639/13:37
patchbotpatch 592639 - ironic-python-agent - Fix multi-device behavior - 5 patch sets13:37
openstackgerritMerged openstack/ironic-specs master: Add virtual Bare Metal Clusters spec  https://review.openstack.org/57958313:37
*** dims has quit IRC13:40
dtantsurfolks, you can make the word a bit better place (at least for the other side of API) by reviewing https://review.openstack.org/596671 :)13:40
patchbotpatch 596671 - ironic - Add version discovery information to the /v1 endpoint - 1 patch set13:40
dtantsur* world13:40
*** gabys has joined #openstack-ironic13:43
TheJuliaI'm surprised I've not reviewed this, I thought I had13:44
*** munimeha1 has joined #openstack-ironic13:47
*** mgoddard has quit IRC13:49
*** mjturek has joined #openstack-ironic13:54
*** mgoddard has joined #openstack-ironic13:54
*** gabys has quit IRC14:00
*** gabys has joined #openstack-ironic14:01
*** dims has joined #openstack-ironic14:03
*** gabys has quit IRC14:06
*** dims has quit IRC14:16
*** rloo has joined #openstack-ironic14:17
openstackgerritIlya Etingof proposed openstack/sushy master: Untie `Connection` object from authenticator object on `.close()`  https://review.openstack.org/61109514:19
*** dims has joined #openstack-ironic14:19
*** Chaserjim has joined #openstack-ironic14:20
openstackgerritMerged openstack/ironic master: Remove Vagrant  https://review.openstack.org/61096414:22
iurygregoryyay 1st patch \o/14:23
TheJulia:)14:23
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Remove deprecate authentication options for accessing ironic  https://review.openstack.org/58704114:26
dtantsurcongrats iurygregory :)14:27
iurygregorytks 1st of many \o/14:28
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector master: Remove deprecated authentication options for accessing ironic  https://review.openstack.org/58704114:31
*** openstackgerrit has quit IRC14:36
*** logan_ has joined #openstack-ironic14:40
*** hamzy has quit IRC14:41
*** bdodd_ has joined #openstack-ironic14:47
*** rloo has quit IRC14:49
*** munimeha1 has quit IRC14:49
*** sthussey has quit IRC14:49
*** bdodd has quit IRC14:49
*** devananda has quit IRC14:49
*** vabada has quit IRC14:49
*** logan- has quit IRC14:49
*** logan_ is now known as logan-14:49
*** Chaserjim has quit IRC14:54
*** dnuka has quit IRC14:54
*** bdodd_ is now known as bdodd14:57
*** openstackgerrit has joined #openstack-ironic15:10
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Fix typos  https://review.openstack.org/61128215:10
openstackgerritOlivier Bourdon proposed openstack/bifrost master: Begin change over to use python3  https://review.openstack.org/58832515:11
*** hamzy has joined #openstack-ironic15:12
openstackgerritChristopher Dearborn proposed openstack/ironic-specs master: Add firmware update spec for Dell EMC hardware  https://review.openstack.org/60213215:15
NobodyCamGood Morning Ironic'ers15:15
NobodyCamHappy Hump Day15:15
dtantsurhey hey NobodyCam15:15
TheJulia\o/ hump day15:16
NobodyCam:) Good Morning dtantsur and TheJulia15:16
olivierbgood afternoon everyone ;-)15:17
dtantsurg'afternoon olivierb15:17
*** rloo has joined #openstack-ironic15:18
*** rloo has quit IRC15:19
*** rloo has joined #openstack-ironic15:20
*** Chaserjim has joined #openstack-ironic15:21
bdoddGood morning everyone15:22
dtantsurmorning bdodd15:22
bdodddtantsur o/15:22
bdoddRegarding the Redfish BIOS implementation, I’m having some trouble with my periodic task to monitor the success or failure of setting the BIOS attributes and could use some input.15:23
bdoddIn the apply_configuration() clean step, after I have set the BIOS attributes on the system, I issue a reboot to the system, set 'cleaning_reboot' in driver_internal_info and return states.CLEANWAIT.15:24
bdoddWhen my period task fires, I locate the target task and check if the BIOS settings were made in the server.15:24
bdoddIf I find that the BIOS settings were not applied, I can log the error and set the last_error.15:24
bdoddSo far, so good15:25
bdoddBut typically the state of the node has already gone through the cleaning and clean wait states and is already back in manageable state.15:25
bdoddSo I can’t (as far as I can tell) move the state to clean failed.15:26
bdoddI was expecting or wanting the state to be held in cleaning or clean wait until the periodic task could verify the BIOS settings and be able to issue a ‘fail’ event to move it to clean failed.15:26
*** moshele has quit IRC15:27
*** rloo has quit IRC15:27
openstackgerritMerged openstack/sushy master: Fix crashing `BasicAuth` context manager  https://review.openstack.org/61110715:28
bdoddNot sure if I'm taking the right approach or completely off in the weeds. Any insights appreciated. :-)15:29
dtantsuretingof: ^^^15:29
olivierbdtantsur glad you liked the green CI15:29
TheJuliabdodd: weeds can grow quickly :(  I'm on a meeting right now so I'll try to read/grok what you wrote after my meeting15:30
bdoddSure, thanks TheJulia15:30
openstackgerritJim Rollenhagen proposed openstack/ironic-specs master: No IPA to conductor communication  https://review.openstack.org/21220615:39
*** vdrok has quit IRC15:41
*** vdrok has joined #openstack-ironic15:42
*** gabys has joined #openstack-ironic15:51
*** gabys has quit IRC15:56
*** gabys has joined #openstack-ironic16:01
*** hamzy has quit IRC16:01
*** hamzy has joined #openstack-ironic16:01
openstackgerritMerged openstack/ironic-python-agent stable/rocky: Raise udevadm settle timeout from 5 to 20 seconds  https://review.openstack.org/61014816:02
openstackgerritMerged openstack/ironic-python-agent master: Add missing dependencies to CoreOS IPA docker image  https://review.openstack.org/61093316:02
*** gabys has quit IRC16:06
TheJuliabdodd: so, cleaning reboot should occur, task should hang in CLEANWAIT until the next heartbeat occurs16:06
TheJuliabdodd: I guess at that point the steps on the bios interface would need a sanity check/validate step in that case to verify. I know there is the collect current bios configuration as well16:07
openstackgerritPierre Riteau proposed openstack/ironic-python-agent stable/rocky: Add missing dependencies to CoreOS IPA docker image  https://review.openstack.org/61137716:07
*** priteau has joined #openstack-ironic16:08
TheJuliathe collection step after cleaning could fail it if it still has the requested data, I guess. Or I think you can issue a cleaning reboot and the step not yet be completed16:09
openstackgerritPierre Riteau proposed openstack/ironic-python-agent stable/queens: Add missing dependencies to CoreOS IPA docker image  https://review.openstack.org/61137916:12
*** tssurya has quit IRC16:14
bdoddTheJulia - Ah, so perhaps I should be making my checks in the cache_bios_settings() call rather than in a periodic task. Is that along the lines of what you are thinking?16:16
TheJuliaI feel like the step can issue the reboot with the flag step, and upon recollection with the next heartbeat in, then it should run16:16
TheJuliabdodd: two different ways to do it I think16:16
TheJuliabut I don't think a periodic task is the best way because as you've indicated it never really catches the failure16:16
TheJuliaI guess I need to look at the code to think through this too, is the patch up to date?16:17
bdoddTheJulia No, I haven't updated the patch with this code yet. Was hoping to get it working better first.16:18
* TheJulia opens up the cleaning code16:19
*** gabys has joined #openstack-ironic16:19
openstackgerritIlya Etingof proposed openstack/sushy master: Untie `Connection` from authenticator on `.close()`  https://review.openstack.org/61109516:19
*** sthussey_ has joined #openstack-ironic16:20
*** munimeha1 has joined #openstack-ironic16:24
*** sthussey has joined #openstack-ironic16:24
*** vabada has joined #openstack-ironic16:24
*** sthussey has quit IRC16:24
*** sthussey_ is now known as sthussey16:24
*** gabys has quit IRC16:25
*** rloo has joined #openstack-ironic16:26
*** moshele has joined #openstack-ironic16:26
*** dtantsur is now known as dtantsur|afk16:27
dtantsur|afksee you tomorrow16:27
TheJuliabdodd: two choices I think. One we update the conductor cleaning logic to have a second flag "reboot in progress as part of step" (although that gets complex with already complex logic... we might need it anyone one day) or you do validation in the cache step and can fail the cleaning there. The downside of the latter idea is someone would need to manually invoke it along side setting the configuration16:28
TheJuliagoodnight dtantsur|afk16:28
TheJuliaactually16:29
TheJuliaset driver internal info 'skip_current_clean_step' to False16:29
TheJuliait should re-call the same step16:29
TheJuliaso you could keep a driveR_internal_info flag to know if your in-progress or not16:30
TheJuliaand validate if you are post-apply, then just make sure we nuke the driver_internal_info flag you created before exiting the second call of apply_configuration16:31
bdoddTheJulia - just to clarify - at what point should I set 'skip_current_clean_step'?16:31
TheJuliabdodd: when you set cleaning_reboot16:31
*** rloo has quit IRC16:32
TheJuliaapply_configuration should get called again upon the next heartbeat, and you can do validation then16:32
*** jesusaur has quit IRC16:32
TheJuliabdodd: so you set the two driver_internal_info flags, tell the hardware to reboot, and I guess you can just let the method exit such that we are still waiting for cleaning16:33
bdoddSo the first apply_config would doing the BIOS settings and reboot and the second would check the result, right?16:34
TheJuliayup16:35
bdoddTheJulia - Thanks! I think I get the idea. I'll try that out.16:36
TheJuliathe changes just need to be saved to the node object (task.node.save())16:36
bdoddRight16:36
TheJuliaawesome16:37
*** gabys has joined #openstack-ironic16:37
*** jesusaur has joined #openstack-ironic16:37
*** moshele has quit IRC16:38
*** gabys has quit IRC16:43
*** e0ne has quit IRC16:46
*** gabys has joined #openstack-ironic16:54
*** dougsz has quit IRC16:58
*** gabys has quit IRC16:59
*** derekh has quit IRC17:00
*** rloo has joined #openstack-ironic17:06
*** hamzy has quit IRC17:07
*** hamzy has joined #openstack-ironic17:08
*** rloo has quit IRC17:12
*** hamzy has quit IRC17:12
*** hamzy has joined #openstack-ironic17:12
*** rloo has joined #openstack-ironic17:12
*** rloo has quit IRC17:14
*** rloo has joined #openstack-ironic17:14
*** gabys has joined #openstack-ironic17:15
*** rloo has quit IRC17:17
*** rloo has joined #openstack-ironic17:18
*** gabys has quit IRC17:19
*** adrianc has quit IRC17:20
*** hjensas has joined #openstack-ironic17:24
*** trown is now known as trown|lunch17:28
*** gabys has joined #openstack-ironic17:50
*** gabys has quit IRC17:55
openstackgerritChristopher Dearborn proposed openstack/ironic-specs master: Add firmware update spec for Dell EMC hardware  https://review.openstack.org/60213218:02
*** mgoddard has quit IRC18:11
*** gabys has joined #openstack-ironic18:14
*** gabys has quit IRC18:18
TheJuliahjensas: fyi, I'm going to tag you on a review re ipv6/boot-from-url support18:22
*** skazi has joined #openstack-ironic18:31
*** e0ne has joined #openstack-ironic18:35
*** moshele has joined #openstack-ironic18:37
*** mbeierl has quit IRC18:38
*** trown|lunch is now known as trown18:44
*** e0ne has quit IRC18:48
*** e0ne has joined #openstack-ironic18:49
openstackgerritJim Rollenhagen proposed openstack/ironic master: Add api-ref for conductor group  https://review.openstack.org/61141518:52
jrollpeople that love docs, this is for you ^18:52
*** e0ne has quit IRC18:55
*** mbeierl has joined #openstack-ironic19:05
*** jesusaur has quit IRC19:08
openstackgerritJim Rollenhagen proposed openstack/ironic master: Add api-ref for conductor group  https://review.openstack.org/61141519:08
*** gabys has joined #openstack-ironic19:10
*** jesusaur has joined #openstack-ironic19:12
*** gabys has quit IRC19:14
*** moshele has quit IRC19:15
TheJulia<319:19
*** hamzy has quit IRC19:21
*** hamzy has joined #openstack-ironic19:21
*** e0ne has joined #openstack-ironic19:22
mjturekTheJulia: been looking through your ipxe boot interface patch. I admittedly haven't checked out the entire series so I might be missing some context, but is passing around ipxe_enabled as an arg intended as a temporary measure? Once [pxe]ipxe_enabled is removed you should be able to infer ipxe_enabled from just the task object. Or are you trying to avoid the weirdness you mention in is_ipxe_enabled?19:26
mjtureksorry for the wall of text19:26
*** hamzy has quit IRC19:30
*** jtomasek has quit IRC19:38
*** e0ne has quit IRC19:42
*** moshele has joined #openstack-ironic19:49
TheJuliamjturek: no worries19:50
TheJuliait really seems dtantsur wants to avoid inferring from the task object, so passing in as an argument seems to be our cleanest delination since we also end up then doing less calls back to an is_ipxe_enabled method19:51
mjturekokay - that's a fair reason!19:51
TheJuliayeah, I thought about the cost of object comparisons too, and he does have a point that it is more costly19:52
TheJuliaso... *shrug*19:52
mjturekagreed, it just seemed so tempting lol19:52
TheJuliaoh I know :)19:55
TheJuliakind of why I did it :)19:55
*** moshele has quit IRC20:02
*** hamzy has joined #openstack-ironic20:17
*** gabys has joined #openstack-ironic20:43
*** trown is now known as trown|outtypewww21:07
*** rcernin has joined #openstack-ironic21:18
*** olivierb has quit IRC21:30
*** rloo has quit IRC21:32
*** gabys has quit IRC21:35
*** gabys has joined #openstack-ironic21:35
*** rloo has joined #openstack-ironic21:38
*** gabys has quit IRC21:40
*** priteau has quit IRC21:50
*** bnemec has quit IRC22:01
*** bfournie has quit IRC22:02
*** munimeha1 has quit IRC22:20
*** Chaserjim has quit IRC22:52
*** Chaserjim has joined #openstack-ironic22:53
openstackgerritJulia Kreger proposed openstack/ironic master: Fix IPv6 and add initial boot-from-url  https://review.openstack.org/61071522:57
*** Chaserjim has quit IRC22:58
*** Chaserjim has joined #openstack-ironic22:59
*** Chaserjim has quit IRC23:03
*** rloo has quit IRC23:10
*** gabys has joined #openstack-ironic23:43

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