Monday, 2019-09-23

*** Lucas_Gray has quit IRC00:11
*** dsneddon has joined #openstack-ironic00:47
*** dsneddon has quit IRC00:52
*** mkowalski_ has joined #openstack-ironic01:28
*** mkowalski__ has quit IRC01:30
*** dsneddon has joined #openstack-ironic02:48
*** dsneddon has quit IRC02:53
*** mkrai has joined #openstack-ironic03:19
*** ricolin has joined #openstack-ironic03:20
*** dhellmann has quit IRC04:25
*** dhellmann has joined #openstack-ironic04:30
*** mkowalski_ has quit IRC04:35
*** mkowalski_ has joined #openstack-ironic04:35
*** mkowalski_ has quit IRC04:41
*** monica_pardhi has joined #openstack-ironic04:44
*** dsneddon has joined #openstack-ironic04:49
*** dsneddon has quit IRC04:54
*** whoami-rajat has joined #openstack-ironic05:11
*** iurygregory has joined #openstack-ironic05:12
iurygregorygood morning Ironic05:12
rpiosoGood morning iurygregory05:13
iurygregoryrpioso, o/05:13
*** rachit7 has joined #openstack-ironic05:14
openstackgerritSteve Baker proposed openstack/ironic master: Allow vendor_data to be included in a configdrive dict  https://review.opendev.org/68384705:22
openstackgerritRichard G. Pioso proposed openstack/ironic master: Add Redfish vmedia boot interface to idrac HW type  https://review.opendev.org/67249805:41
*** yolanda has quit IRC05:42
*** yolanda has joined #openstack-ironic05:49
openstackgerritRichard G. Pioso proposed openstack/ironic master: Add Redfish vmedia boot interface to idrac HW type  https://review.opendev.org/67249806:04
rpiosoetingof: Please take a look at ^^^06:05
*** rpioso is now known as rpioso|afk06:05
openstackgerritRotanChen proposed openstack/ironic-inspector master: Remove unused import statement.  https://review.opendev.org/68233006:05
*** yolanda has quit IRC06:22
*** TxGirlGeek has joined #openstack-ironic06:23
arne_wiebalckGood morning, ironic!06:38
iurygregorymorning arne_wiebalck o/06:38
arne_wiebalckhey iurygregory o/06:39
*** rpittau|afk is now known as rpittau06:40
rpittaugood morning ironic! o/06:40
iurygregorymorning rpittau o/06:41
rpittauhey iurygregory :)06:41
arne_wiebalckhey rpittau o/06:41
rpittauhi arne_wiebalck :)06:41
*** TxGirlGeek has quit IRC06:42
*** dsneddon has joined #openstack-ironic06:50
*** dsneddon has quit IRC06:55
arne_wiebalckWhen adding new controllers, I noticed that the conductor affinity is updated on most but not all nodes, some nodes keep the affinity to deleted controllers (and I update them manually) ... can someone remind me why for some node the affinity is not re-calculated?06:57
*** tesseract has joined #openstack-ironic07:00
*** rcernin has quit IRC07:04
*** alexmcleod has joined #openstack-ironic07:13
*** kiseok7 has quit IRC07:18
*** monica_pardhi has quit IRC07:22
mkraigood morning ironic!07:31
mkraiarne_wiebalck, rpittau iurygregory hi o/07:31
*** e0ne has joined #openstack-ironic07:34
mkraiI was seeing conductor do_node_clean code and found that there is two power,network validate call. Is it intended?07:34
mkraihttps://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1177-L1178 and https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1316-L131707:35
*** e0ne has quit IRC07:35
*** pcaruana has joined #openstack-ironic07:41
rpittaumkrai: have you seen this comment? https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L117107:44
mkrairpittau, Yes but the second validate in _do_node_clean is also immediate. I mean no actual processing07:46
*** jtomasek has quit IRC07:47
*** jtomasek has joined #openstack-ironic07:48
rpittaumkrai: sorry, I'm not sure I follow, the power and network validate are done as first steps, before starting anu processing07:48
etingofrpioso|afk, o/ reviewed, LGTM07:48
rpittauany*07:48
etingofrpioso|afk, I accidentally posted my older comments to bdodd's review as well. you can just ignore those..07:49
mkrairpittau, https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1289-L1311, is it actual processing?07:50
rpittaumkrai: in terms of clean steps, no, that is preparation to cleaning.07:53
mkrairpittau, Ok thanks07:54
rpittaumkrai: not even preparation actually, we're just deciding if we're going to clean or not a node07:55
mkrairpittau, Yes and it isn't a real processing07:56
mkraiJust some condition check07:56
rpittaumkrai: correct07:56
mkraiThat's why the validation seemed redundant to me.07:56
mkraiDoes it makes sense?07:56
*** toabctl has joined #openstack-ironic08:02
rpittaumkrai: I understand why it looks redundant, it is indeed, although I think it's necessary, something could change betweem method calls08:02
rpittauok, probably not network :)08:03
*** yolanda has joined #openstack-ironic08:09
*** e0ne has joined #openstack-ironic08:18
*** iurygregory has quit IRC08:19
*** gregoryo has joined #openstack-ironic08:21
openstackgerritpengyuesheng proposed openstack/python-ironic-inspector-client master: Update the constraints url  https://review.opendev.org/68388208:44
*** rcernin has joined #openstack-ironic08:45
*** iurygregory has joined #openstack-ironic08:45
*** derekh has joined #openstack-ironic08:49
*** yolanda has quit IRC08:50
*** dsneddon has joined #openstack-ironic08:51
*** gregoryo has quit IRC08:55
*** dsneddon has quit IRC08:55
*** ociuhandu has joined #openstack-ironic09:05
*** dtantsur|afk is now known as dtantsur09:07
dtantsurmorning ironic09:07
dtantsurmgoddard: hey, sorry, I may have to skip the 2nd part (even 2/3) of the meeting today. could you take it over once I leave?09:08
openstackgerritDmitry Tantsur proposed openstack/ironic stable/stein: devstack: wait for conductor to start and register itself  https://review.opendev.org/68390009:09
*** mkowalski_ has joined #openstack-ironic09:09
openstackgerritDmitry Tantsur proposed openstack/ironic stable/rocky: devstack: wait for conductor to start and register itself  https://review.opendev.org/68390109:09
openstackgerritDmitry Tantsur proposed openstack/ironic stable/queens: devstack: wait for conductor to start and register itself  https://review.opendev.org/68390209:10
iurygregorymorning dtantsur09:12
mkrairpittau, Sorry I was away, what could change?09:15
mkraimorning dtantsur09:15
*** bobmel has joined #openstack-ironic09:24
*** bobmel has quit IRC09:25
*** Lucas_Gray has joined #openstack-ironic09:30
mgoddarddtantsur: sure, no problem. If you give me your IRC credentials I could impersonate you, and maybe no one will ever know :p (except meeting quality will quickly drop)09:40
*** dsneddon has joined #openstack-ironic09:40
dtantsur:D09:40
*** Lucas_Gray has quit IRC09:46
*** Lucas_Gray has joined #openstack-ironic09:48
*** ricolin has quit IRC09:48
*** ociuhandu has quit IRC09:59
*** ociuhandu has joined #openstack-ironic10:00
dtantsurxXraphXx: hey, around re https://review.opendev.org/#/c/683421/ ?10:04
patchbotpatch 683421 - ironic-python-agent - Revert "Software raid: mbr/gpt partition table alt... - 3 patch sets10:04
dtantsurI managed to figure out from IRC scrollback why we are doing it. But to have an understandable git history, we need to provide a reasoning in the commit message.10:05
dtantsurif you could update the commit message, I could approve right away10:05
*** bobmel has joined #openstack-ironic10:05
xXraphXxdtantsur, hi, sure doing10:05
dtantsurthx!10:05
*** ociuhandu has quit IRC10:06
*** rachit7 has quit IRC10:18
openstackgerritraphael.glon proposed openstack/ironic-python-agent master: Revert "Software raid: mbr/gpt partition table alternative"  https://review.opendev.org/68342110:21
xXraphXxdtantsur, done, would the commit message be ok ?10:22
openstackgerritraphael.glon proposed openstack/ironic-python-agent master: Revert "Software raid: mbr/gpt partition table alternative"  https://review.opendev.org/68342110:26
*** rachit7 has joined #openstack-ironic10:39
*** ociuhandu has joined #openstack-ironic10:41
*** kaifeng has joined #openstack-ironic10:42
kaifengmorning ironic o/10:44
*** ociuhandu has quit IRC10:46
*** Lucas_Gray has quit IRC10:46
*** Lucas_Gray has joined #openstack-ironic10:48
*** ociuhandu has joined #openstack-ironic10:50
*** ociuhandu has quit IRC10:54
dtantsurxXraphXx: done (sorry, was on lunch)10:57
dtantsurmorning (evening?) kaifeng :)10:57
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Explicitly tear down software RAID after testing it  https://review.opendev.org/68392111:08
dtantsurthis ^^ may help with the standalone job11:08
openstackgerritMerged openstack/ironic master: Don't resume deployment or cleaning on heartbeat when polling  https://review.opendev.org/68127211:09
kaifengdtantsur: yep11:17
*** dsneddon has quit IRC11:17
kaifengdtantsur: there is a fix may help as well https://review.opendev.org/68227011:17
patchbotpatch 682270 - ironic-tempest-plugin - Fix an issue when nodes are not available in stand... - 1 patch set11:17
*** ociuhandu has joined #openstack-ironic11:19
*** ociuhandu has quit IRC11:23
*** lucasagomes has joined #openstack-ironic11:31
*** rh-jelabarre has joined #openstack-ironic11:31
dtantsurkaifeng: yep, +211:32
kaifengdtantsur: just a look on your patch, i don't think the state transition works this way, I wonder why the delete_node is set to False?11:34
* kaifeng insert add/take/whatever into above words11:35
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Explicitly tear down software RAID after testing it  https://review.opendev.org/68392111:37
dtantsurkaifeng: delete_node is set to False to prevent tearDown from failing (it will try unprovision the already unprovisioned node)11:38
*** bobmel has quit IRC11:40
kaifengso sometimes terminate_node is only used for vif cleanup right?11:42
openstackgerritMerged openstack/ironic master: Add iDRAC RAID deploy steps  https://review.opendev.org/64173111:43
kaifengoh and I left a suggestion on the patch of conductor startup checking, i think we could use the /v1/conductors for the same purpose11:45
*** rh-jelabarre has quit IRC11:45
*** rh-jelabarre has joined #openstack-ironic11:45
kaifengfor a multinode environment, checking driver list is not sufficient11:46
*** dsneddon has joined #openstack-ironic11:47
*** rcernin has quit IRC11:49
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: DIB: do not use /tmp for a virtualenv in the source build  https://review.opendev.org/68392911:50
dtantsurkaifeng: yep, we could. I can try following up with it, but I may not have time in the near future.11:51
kaifengdtantsur: np, anytime :)11:52
kaifengi gotta go now, see you later o/11:53
*** kaifeng has quit IRC11:53
*** bfournie has quit IRC11:57
*** ociuhandu has joined #openstack-ironic11:57
*** goldyfruit_ has quit IRC11:57
*** ociuhandu has quit IRC11:58
*** ociuhandu has joined #openstack-ironic11:59
*** iurygregory has quit IRC12:01
TheJuliadtantsur: did you see my note regarding the metalsmith job only ever using network booting?12:17
TheJuliaWhich likely explains some of the confusion.12:17
*** dsneddon has quit IRC12:22
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Fix TypeError when endpoint cannot be detected from a session  https://review.opendev.org/68394412:27
*** rpioso|afk is now known as rpioso12:28
dtantsurTheJulia: the centos job should (at least used to) use local booting12:28
dtantsurI think it falls back to networking booting because we don't configure the boot loader12:29
*** rloo has joined #openstack-ironic12:29
dtantsurTheJulia: re bifrost failure: looking. the above ironicclient patch is one of the parts of it.12:29
dtantsurbut I suspect ironicclient 3.0.0 breaks no-auth :(12:30
rpiosoetingof: Thank you! I'll revise the commit message and release note to mention the introduction of the new dependency.12:30
*** rcernin has joined #openstack-ironic12:33
dtantsuroh wow, SessionClient is SUCH A MESS12:33
*** dsneddon has joined #openstack-ironic12:33
etingofrpioso, what should we do with our new drivers-requirements dependency that apparently fails a few CI jobs?12:36
etingofrpioso, comment it out or release sushy-oem-idrac on pypi (if that helps at all?)12:37
*** dsneddon has quit IRC12:38
rpiosoetingof: Good question. The plan is to add that repo to opendev's 'x' namespace. That will make it available on PyPI. However, that'll take some time.12:41
rpiosoetingof: Could it be commented out or left out and later added? Or would that not comply with stable branch policy?12:42
etingofdtantsur, might know ^12:44
mbuilguys, I am trying ironic stand-alone and suddenly, when executing "openstack baremetal node list" I get "Missing value auth-url required for auth plugin password". Is it not enough with export OS_TOKEN=fake-token and export OS_URL=http://localhost:6385/? Any hint? thanks!12:44
dtantsuretingof, rpioso, we cannot merge code that is not usable by end users, if that's your question12:45
dtantsurif there is a dependency, it has to be installable12:45
etingofrpioso, perhaps you've researched already what needs to be done to move the library to `x`...?12:46
*** bfournie has joined #openstack-ironic12:47
rpiosoetingof: I have.12:48
* rpioso digs up the link to the document.12:49
*** jcoufal has joined #openstack-ironic12:50
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicator management to redfish hw type  https://review.opendev.org/65274012:51
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Fix TypeError when using endpoint_override with SessionClient  https://review.opendev.org/68394412:51
dtantsurTheJulia: this ^^ may be the fix12:51
*** iurygregory has joined #openstack-ironic12:51
dtantsuretingof: some parts of https://docs.openstack.org/infra/manual/creators.html12:51
openstackgerritIlya Etingof proposed openstack/ironic master: Add indicators REST API endpoints  https://review.opendev.org/65178512:51
mbuilfound the solution ==> https://github.com/openstack/bifrost/commit/6df83e0bc56d519bb6d37190228e0977d89da6a312:54
rpiosoetingof: https://docs.openstack.org/infra/manual/creators.html12:55
dtantsurmbuil: right, OS_TOKEN was removed from openstackclient12:55
mbuildtantsur: I'll update https://docs.openstack.org/ironic/latest/install/standalone.html12:57
dtantsurthanks!12:57
openstackgerritMerged openstack/ironic master: Allow retrying PXE boot if it takes too long  https://review.opendev.org/68312712:58
openstackgerritRachit Kapadia proposed openstack/ironic master: DRAC: Fix a bug for delete_config with multiple controllers  https://review.opendev.org/68066012:59
*** ricolin has joined #openstack-ironic12:59
*** e0ne_ has joined #openstack-ironic12:59
*** goldyfruit_ has joined #openstack-ironic13:01
*** e0ne has quit IRC13:01
etingofrpioso, on the bright side, sushy-oem-idrac is structured similarly to sushy i.e. it should hopefully be easy to adopt as an OS project...13:01
etingofrpioso, if you are short of cycles, I can probable pursue this13:02
rpiosodtantsur, etingof: Could it be completed in time for Train?13:03
dtantsurrpioso: feature work is essentially out of question for Train13:04
rpiosoetingof: I'll pursue it and of course keep you looped in.13:06
rpiosoetingof: I'd like to explore your review and code comments about needing to reboot the server to set the boot device. It appears to describe two reboots. In my experience, only one reboot has been required. During the next reboot, the setting is applied and utilized. Would later today work?13:08
*** ociuhandu has quit IRC13:10
*** ociuhandu has joined #openstack-ironic13:10
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Remove the requirement on either endpoint_override or os_ironic_api_version  https://review.opendev.org/68395213:10
etingofrpioso, are you saying that it is possible to submit two iDRAC jobs in a row (set-boot-device and set-boot-mode) without a reboot in-between?13:11
*** e0ne has joined #openstack-ironic13:12
*** e0ne_ has quit IRC13:12
openstackgerritManuel Buil proposed openstack/ironic master: Update env. variables in the documentation  https://review.opendev.org/68395313:13
rpiosoetingof: Let's start simple by keeping boot mode out of the discussion.13:14
*** dsneddon has joined #openstack-ironic13:14
rpiosoetingof: But later today :-)13:14
rpiosoetingof: If that works for you.13:14
etingofrpioso, sure!13:15
*** ociuhandu has quit IRC13:17
*** mkrai has quit IRC13:18
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Make the dependency on oslo.config explicit  https://review.opendev.org/68395513:18
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Make the dependency on oslo.config explicit  https://review.opendev.org/68395513:20
*** mkrai has joined #openstack-ironic13:21
*** beekneemech is now known as bnemec13:24
*** sthussey has joined #openstack-ironic13:26
jrollmorning ironic13:27
dtantsurmorning jroll13:27
*** ociuhandu has joined #openstack-ironic13:27
rpittauhey jroll :)13:28
*** goldyfruit___ has joined #openstack-ironic13:30
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Fix TypeError when using endpoint_override with SessionClient  https://review.opendev.org/68394413:31
*** ociuhandu has quit IRC13:33
*** goldyfruit_ has quit IRC13:33
*** whoami-rajat has quit IRC13:33
arne_wiebalckjroll: o/13:36
*** yolanda has joined #openstack-ironic13:44
*** rcernin has quit IRC13:45
*** diga has joined #openstack-ironic13:45
*** dsneddon has quit IRC13:46
*** bdodd has joined #openstack-ironic13:50
*** yolanda has quit IRC13:52
*** dsneddon has joined #openstack-ironic13:54
*** yolanda has joined #openstack-ironic13:56
*** dsneddon has quit IRC14:00
*** jcoufal_ has joined #openstack-ironic14:00
*** ociuhandu has joined #openstack-ironic14:00
openstackgerritDmitry Tantsur proposed openstack/ironic master: Document PXE retries  https://review.opendev.org/68396714:02
*** yolanda has quit IRC14:03
*** jcoufal has quit IRC14:04
*** tzumainn has joined #openstack-ironic14:04
*** ociuhandu has quit IRC14:04
mbuilguys, our ironic-conductor logs show this when we try to do the inspection: "ConnectFailure: Unable to establish connection to http://localhost:5050: HTTPConnectionPool(host='localhost', port=5050): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7ff1b52a7c90>: Failed to establish a new connection: [Errno 111] ECONNREFUSED',))". I can see that a process should be14:05
mbuillistening on port 5050 but that port is not open. Any hint what might be the problem?14:05
openstackgerritRiccardo Pittau proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection  https://review.opendev.org/68397114:12
*** dsneddon has joined #openstack-ironic14:16
openstackgerritGaĆ«tan Trellu proposed openstack/python-ironicclient master: Fix usage of --owner filter during node list  https://review.opendev.org/68266614:17
dtantsurmbuil: maybe it's not listening on localhost?14:18
*** yolanda has joined #openstack-ironic14:19
*** dsneddon has quit IRC14:21
*** iurygregory has quit IRC14:23
*** jtwill98 has joined #openstack-ironic14:29
*** TxGirlGeek has joined #openstack-ironic14:35
mbuildtantsur: by reading the conf. it seems the inspector should be listening to port 5050. Should I see through 'ps' any process called inspector? Or what is the name of the process? Maybe I can try to start it manually and check if there are any problems14:37
dtantsurmbuil: usually, ironic-inspector.14:37
mbuildtantsur: thanks. That throws "ERROR oslo.messaging._drivers.impl_rabbit [-] Connection failed: [Errno 111] ECONNREFUSED (retrying in 2.0 seconds): error: [Errno 111] ECONNREFUSED"14:38
rpittaumbuil: that's a problem with rabbitmq (or any other message broker you're using)14:40
rpittaumbuil: you should check that rabbitmq is running and the rabbit conf is correct in your setup14:41
mbuilrpittau: is rabbitmq also used in bifrost with standalone ironic?14:42
rpittaumbuil: it recently switched to json-rpc by default14:44
openstackgerritMerged openstack/ironic-python-agent-builder master: DIB: rename files and paths to match the element name  https://review.opendev.org/68264714:48
openstackgerritMerged openstack/ironic-python-agent-builder master: Correct variables in the POST job image_name -> image_type  https://review.opendev.org/68289914:48
*** S4ren has joined #openstack-ironic14:49
*** ociuhandu has joined #openstack-ironic14:51
mbuilrpittau: I have rpc_transport = json-rpc in my ironic.conf. Do you have the command to start ironic-inspector handy? I'd like to try14:51
*** kaifeng has joined #openstack-ironic14:52
openstackgerritMark Goddard proposed openstack/ironic master: Release notes cleanup for 13.0.0 (mk2)  https://review.opendev.org/68398314:53
mbuilrpittau: I still get the rabbitmq error when executing "ironic-inspector --config-file /etc/ironic/ironic.conf" even if ironic.conf has "rpc_transport = json-rpc", any idea? Maybe I need another parameter there?14:54
*** yolanda has quit IRC14:54
*** dsneddon has joined #openstack-ironic14:55
*** nick_kar has joined #openstack-ironic14:56
*** ociuhandu has quit IRC14:57
*** mkowalski_ has quit IRC14:58
kaifengmbuil: it seems you are feeding inspector service with ironic configuration file14:58
rpittaumbuil: out of curiosity, which version of ironic-inspector are you using?14:58
rpittauand yes, also that kaifeng :D14:58
kaifenghmm, i never tried that, what the purpose behind this?14:59
dtantsur#startmeeting ironic15:00
openstackMeeting started Mon Sep 23 15:00:02 2019 UTC and is due to finish in 60 minutes.  The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.15:00
*** openstack changes topic to " (Meeting topic: ironic)"15:00
openstackThe meeting name has been set to 'ironic'15:00
dtantsurhey all, meeting time!15:00
etingofo/15:00
rpittauo/15:00
rlooo/15:00
kaifengo/15:00
bdoddo/15:00
dtantsurI'll have to run quite soon, will pass my chair rights to mgoddard then15:00
dtantsur#chair mgoddard15:00
openstackCurrent chairs: dtantsur mgoddard15:00
jroll\o15:00
dtantsurthe empty agenda is located at:15:00
dtantsur#link https://wiki.openstack.org/wiki/Meetings/Ironic15:00
mgoddardlo15:00
rpiosoo/15:00
dtantsur#topic Announcements / Reminder15:01
*** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)"15:01
dtantsur#info This week we must have stable/train for all our branched project15:01
*** cdearborn has joined #openstack-ironic15:01
dtantsur#link https://releases.openstack.org/train/schedule.html15:01
cdearborno/15:01
arne_wiebalcko/15:01
dtantsur#info We're in feature freeze, please hold on approving features until the branches are created15:01
rpittauit's release-3 week15:02
openstackgerritMark Goddard proposed openstack/ironic master: Update release mappings for Train  https://review.opendev.org/68398415:02
dtantsurgood call ^^^15:02
dtantsuranything else to announce?15:02
mgoddardsemi-plug, semi-useful info, I wrote a blog about bios, raid, scheduling, deploy steps and deploy templates: http://www.stackhpc.com/bespoke-bare-metal.html15:02
mgoddardbased on the talk I gave in denver15:03
mgoddardnot really an announcement, but hey :)15:03
dtantsur#link http://www.stackhpc.com/bespoke-bare-metal.html15:03
dtantsurwhy not :)15:03
arne_wiebalcknice!15:03
dtantsurwell, kind of an announcement: our CI feels better, please try not to break it :)15:04
dtantsurany ideas for further improvements are very welcome15:04
*** ociuhandu has joined #openstack-ironic15:04
dtantsuranything else?15:04
*** whoami-rajat has joined #openstack-ironic15:04
*** ociuhandu has quit IRC15:04
rloodtantsur: we may get to it, but if we don't. before you take off. please let us know what you think needs to land before we cut a train rc1.15:05
dtantsurI'm not aware of anything that MUST land15:05
rlooalso, I don't know if we did this, but I want to congratulate (and THANK!) TheJulia for being Ussuri PTL.15:05
*** ociuhandu has joined #openstack-ironic15:05
rpittauand when we are cutting train RC1 :)15:05
*** stendulker has joined #openstack-ironic15:05
mgoddardrloo: you could #thank15:05
*** pcaruana has quit IRC15:05
dtantsurhowever, I have a few fixes for ironicclient, that seem quite urgent15:05
*** w14161_1 has joined #openstack-ironic15:06
dtantsurrpittau: for us RC1 == stable/train15:06
jrollrloo: +100015:06
dtantsurwe don't do proper RCs, so the last release before branching is an RC15:06
rloodtantsur: i suspect rpittau wants to know what day we'll be cutting that.15:06
dtantsurThursday the latest15:06
mkraio/ sorry I am late15:06
rpittaurloo: yeah15:06
rpittaudtantsur: thanks15:06
*** dsneddon has quit IRC15:06
dtantsurdoing it later may cause us troubles if grenade gets branched15:06
dtantsuractually, I'm doing to check tomorrow if we can release or not15:07
rloo#thank TheJulia for volunteering to be the PTL for ussuri15:07
dtantsurso consider it happening at any point15:07
dtantsurunless somebody brings up anything blocking15:07
dtantsur#topic Review subteam status reports (capped at ten minutes)15:07
*** openstack changes topic to "Review subteam status reports (capped at ten minutes) (Meeting topic: ironic)"15:07
dtantsur#link https://etherpad.openstack.org/p/IronicWhiteBoard line 26615:07
dtantsurI don't expect a lot of changes, but there's been some progress around IPA-builder and RAID15:07
*** ociuhandu has quit IRC15:08
openstackgerritMerged openstack/ironic stable/queens: devstack: wait for conductor to start and register itself  https://review.opendev.org/68390215:08
openstackgerritMerged openstack/ironic-inspector master: Docs for ironic inspector with tooz  https://review.opendev.org/67454215:08
openstackgerritMerged openstack/ironic-python-agent master: Revert "Software raid: mbr/gpt partition table alternative"  https://review.opendev.org/68342115:08
*** ociuhandu has joined #openstack-ironic15:08
rlooooo. reduced rfe bugs by 5 :)15:09
rloowell,not bugs. stories i guess.15:09
dtantsuryep :)15:09
mkraihttps://review.opendev.org/#/c/665835/ do we need this one for train?15:09
patchbotpatch 665835 - ironic - Block port deletions where vif is present - 3 patch sets15:09
*** w14161_1 is now known as jrwang15:10
dtantsurI'm not sure how I feel about a behavior change after feature freeze15:10
dtantsuris it critical for anything mkrai?15:10
rloowrt Software RAID, L361. The 2 patches. I think we're punting those to U.15:11
dtantsurseems so15:12
arne_wiebalckrloo: yes15:12
mkraidtantsur, Not urgent if team thinks so.15:12
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent-builder master: DIB: do not use /tmp for a virtualenv in the source build  https://review.opendev.org/68392915:12
*** ociuhandu has quit IRC15:13
rlooam going to delete the "power state callbacks to Nova" section (L369ish)15:13
dtantsuryep15:13
dtantsurare we still going to land https://review.opendev.org/#/c/651785/ in train?15:13
patchbotpatch 651785 - ironic - Add indicators REST API endpoints - 16 patch sets15:13
arne_wiebalckshall we remove all the "old" s/w RAID lines 350-363 ?15:13
dtantsurmgoddard: this ^^ may affect release_mappings15:14
dtantsurarne_wiebalck: likely15:14
mgoddarddtantsur: I didn't think we were aiming for landing indicators?15:14
*** TxGirlGeek has quit IRC15:14
dtantsurmgoddard: it's probably a bit too late, yes15:14
mgoddardhasn't had a +2 yet15:14
rlooYay, we now have pdf docs!15:15
dtantsurwe have landed https://review.opendev.org/#/c/649675/, but it's not user-visible15:15
patchbotpatch 649675 - ironic - Add indicator management harness to ManagementInte... (MERGED) - 13 patch sets15:15
dtantsurlet's hold on the API until we release then15:15
dtantsuranything else on the statuses?15:16
mgoddardI guess we'd need the redfish implementation patch too to make the API useful?15:16
dtantsurmgoddard: they're ordered correctly now, the API one goes last15:16
mgoddardright15:17
kaifengrloo: it doesn't look nice for some projects :)15:17
rlookaifeng: well... as long as the info is there, that's better than nothing. now people can pretty-ify them :D15:17
dtantsurkaifeng: PDF docs? well, now people may start improving them15:17
kaifengi expect there would some common tooling for the purpose15:18
rlooand many thanks kaifeng, for working on the pdfs!15:18
dtantsurkaifeng++15:18
rpiosoThank you for reviewing the idrac changes. Two of them have merged \o/15:18
dtantsurnp15:19
dtantsurare we ready to move on?15:19
kaifengnp :)15:19
mgoddardkaifeng++, especially since I shamelessly stole the patch for kolla :D15:19
rloo++ movin'15:19
dtantsurheh15:19
dtantsur#topic Deciding on priorities for the coming week15:19
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)"15:19
*** ociuhandu has joined #openstack-ironic15:19
kaifengmgoddard: the root source is from the doc team :)15:20
mgoddardkaifeng: we're all thieves when it comes to it15:20
*** ociuhandu has quit IRC15:21
*** ociuhandu has joined #openstack-ironic15:22
rlooimitation is the sincerest form of flattery15:22
* kaifeng quickly learns15:23
rloorpioso: are there still idrac changes that need to land in train?15:23
rloorpioso: *need* :)15:23
dtantsurokay, I've cleaned up the list. is there anything that has to be there?15:23
rpiosorloo: Glad you asked.15:24
dtantsurnote that "High Priority" is before branching, "Weekly Priorities" go after15:24
rpiosoWe would truly like to land https://review.opendev.org/#/c/67249815:24
patchbotpatch 672498 - ironic - Add Redfish vmedia boot interface to idrac HW type - 20 patch sets15:24
rpiosoIt's an idrac HW type change.15:24
dtantsurI think this ^^ goes into after-train bucket, especially since it adds a dependency15:25
dtantsurwith this said, I have to go. I'm leaving finishing the list in mgoddard's capable hands :)15:25
mgoddardok. dtantsur's gone, chuck whatever you like into train :p15:25
dtantsurhey!!15:25
mgoddard(kidding)15:26
arne_wiebalck:-D15:26
dtantsur:D15:26
*** dtantsur is now known as dtantsur|afk15:26
* rpioso waves farewell to dtantsur15:26
dtantsur|afko/15:26
rpioso;-)15:26
mgoddardI think it's the right call rpioso - we'd like to cut the release tomorrow ideally, so I think anything without at least one +2 at this point needs a solid argument behind it15:26
rpiosomgoddard: There's high demand for it in Train.15:27
rpiosomgoddard: It highly leverages an already merged change.15:28
rloorpioso: i haven't reviewed it; CI isn't happy and no +2. I guess if you can find two cores willing to review and approve before whenever the branch/rc1 is done...15:29
rpittaurpioso: there are some comments that need still to be addressed15:29
rloorpioso: i don't know that it is a high priority though.15:29
*** rachit7 has quit IRC15:29
rpiosoetingof and I are feverishly working on addressing the remaining comments.15:30
mgoddardthe addition of sushy-oem-idrac as a dependency could be risky15:30
rloorpioso: well, it IS a high priority from your point of view15:30
rpiosorloo: Understood and understood.15:30
mgoddardI think rloo has the answer here15:30
mgoddardtreat it as an FFE, and find two cores to sponsor15:31
rpittauhas sushy-oem-idrac actually been released yet ?15:31
mgoddardunless the rest of the team thinks it is too risky15:31
rpiosoWhatever the community could reasonably do would be much appreciated.15:31
rpiosoWe're willing to submit an FFE.15:32
mgoddardI don't see any releases on https://github.com/etingof/sushy-oem-idrac15:32
rpiosomgoddard: It's being published on PyPI as I keyboard.15:32
etingofmgoddard need to reload his browser ;)15:32
mgoddardnope15:33
rloothat seems risky then. releasing a package just before this feature needs to land. not enough testing/feedback :-(15:33
mgoddardok15:33
mgoddardI see i now15:33
mgoddardit is a driver-req, so optional15:33
rpiosoThe package is outside of OpenStack/OpenDev15:33
rpiosoOut-of-tree15:34
rloook, let's take it to the mailing list then :)15:34
rpioso+115:34
mgoddardrpioso: can you raise an FFE on openstack-discuss for this?15:35
rpiosomgoddard: Absolutely!15:35
mgoddardthanks15:35
rpiosoThank you!15:35
mgoddardok, priorities15:36
mgoddardare we missing anything?15:36
rloono. i don't think so.15:37
rpittauall the priorities from last week where addressed, in a way or another15:37
rloobut it just occurred to me. didn't julia do something wrt shanghai forum or... something?15:37
mgoddardshe proposed two sessions AFAIK15:37
rpittaurloo: she proposed some sessions for the forum15:37
rlooah, yeah. that. worth an announcement.15:38
mgoddardoperator feedback and one other about a specific feature request15:38
mgoddardanyone have a link?15:38
*** yolanda has joined #openstack-ironic15:40
mgoddardsnapshotting, that was it15:40
* rpioso doesn't see it on https://wiki.openstack.org/wiki/Forum/Shanghai2019, yet.15:40
mgoddard#info Two forum sessions proposed for Shanghai summit - Operator Feedback and snapshots requirements gathering15:40
mgoddardhttp://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2019-09-19.log.html#t2019-09-19T18:37:1415:40
rpittauyeah I don't think the schedule is up-to-date yet15:41
mgoddardok, no new RFEs to review15:41
mgoddard#topic Open Discussion15:41
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:41
mgoddardHas anyone got anything to discuss today?15:42
TheJuliaI have nothing but I need to go to airplane mode now15:42
rpittauI should have told before when dtantsur|afk was also here, but if there15:42
tzumainnhi! if anyone has time to take a look at this spec - https://review.opendev.org/#/c/681066/ - that would be great!15:42
patchbotpatch 681066 - ironic-specs - Expose node owner information to oslo.policy checks - 6 patch sets15:42
mgoddardhave a nice flight TheJulia15:42
TheJuliaThanks15:42
rpittauwhoops15:42
rpittauhave a safe flight TheJulia15:43
TheJuliatzumainn: I'll try this week15:43
rpiosoSafe and uneventful travels, TheJulia.15:43
tzumainnTheJulia, thanks, no worries if oyu can't - I know this is a very busy time!15:43
kaifenghave a safe travel TheJulia15:44
rpittauanything that needs to be done for the releases, I would be glad to help :)15:44
*** dsneddon has joined #openstack-ironic15:45
*** mkowalski_ has joined #openstack-ironic15:45
mgoddardrpittau: thanks. I'm going through the TODO list: https://docs.openstack.org/ironic/latest/contributor/releasing.html15:45
mgoddardreviews appreciated15:45
rpittaumgoddard: oh handy :)15:45
rpittaumgoddard: I left a comment on the release-mappings update patch15:47
mgoddardtzumainn: I'll add your spec to the list in our priorities15:47
tzumainnmgoddard, thanks!15:47
mgoddardshould give it a bit more visibility15:47
mgoddardrpittau: thanks, I'll respond15:48
mgoddardI hear crickets15:49
*** mkowalski__ has joined #openstack-ironic15:49
mgoddardthanks everyone15:49
rpiosomgoddard: Thank you15:49
kaifengthanks mgoddard o/15:49
rpittaumgoddard: thanks!15:49
mgoddardwith a little luck we'll have train behind us for the next meeting15:49
mgoddardonto Ussuri!15:50
rloo+++15:50
mgoddard#endmeeting15:50
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list."15:50
rpittaumgoddard: hopefully not literally a train :D15:50
openstackMeeting ended Mon Sep 23 15:50:08 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:50
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-09-23-15.00.html15:50
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-09-23-15.00.txt15:50
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-09-23-15.00.log.html15:50
rpiosorpittau: lol15:50
mgoddardrpittau: these jokes will be not nearly as easy with Ussuri15:50
rpittaumgoddard: Ussuri is a river, so they could be even worse :D15:51
mgoddarduh oh15:51
rpittauanyhow, good night! o/15:52
*** mkowalski_ has quit IRC15:52
*** rpittau is now known as rpittau|afk15:52
*** TxGirlGeek has joined #openstack-ironic15:52
rpiosobdodd: I asked a question on https://review.opendev.org/#/c/672498/ I'm hoping you or mraineri could help answer. It's at https://review.opendev.org/#/c/672498/19/ironic/drivers/modules/drac/boot.py@10615:52
patchbotpatch 672498 - ironic - Add Redfish vmedia boot interface to idrac HW type - 20 patch sets15:52
patchbotpatch 672498 - ironic - Add Redfish vmedia boot interface to idrac HW type - 20 patch sets15:52
* kaifeng heard an old saying that you can't jump into the same Ussuri twice.15:52
bdoddrpioso I'll take a look. I think mraineri is on leave for the rest of the month.15:56
*** ociuhandu has quit IRC15:56
*** ociuhandu has joined #openstack-ironic15:57
rpiosobdodd: Thank you so much! There's another, related comment several lines after that one. Ack on mraineri's leave.15:57
*** kaifeng has quit IRC15:58
*** ociuhandu has quit IRC16:01
*** ociuhandu has joined #openstack-ironic16:02
*** Lucas_Gray has quit IRC16:04
*** stendulker has quit IRC16:04
openstackgerritGaĆ«tan Trellu proposed openstack/python-ironicclient master: Fix usage of --owner filter during node list  https://review.opendev.org/68266616:05
*** pcaruana has joined #openstack-ironic16:06
*** Lucas_Gray has joined #openstack-ironic16:06
*** ociuhandu has quit IRC16:06
*** alexmcleod has quit IRC16:07
openstackgerritMerged openstack/python-ironicclient master: Make the dependency on oslo.config explicit  https://review.opendev.org/68395516:09
*** ociuhandu has joined #openstack-ironic16:10
*** mkrai has quit IRC16:12
*** e0ne has quit IRC16:16
openstackgerritMark Goddard proposed openstack/ironic-inspector master: Clean up release notes for 10.0.0  https://review.opendev.org/68399216:18
*** etingof has quit IRC16:23
*** Lucas_Gray has quit IRC16:25
*** Lucas_Gray has joined #openstack-ironic16:29
*** problem_v has joined #openstack-ironic16:41
*** dtruong has joined #openstack-ironic16:41
*** pcaruana has quit IRC16:46
*** Goneri has joined #openstack-ironic16:54
openstackgerritMerged openstack/python-ironicclient master: Fix TypeError when using endpoint_override with SessionClient  https://review.opendev.org/68394417:05
*** whoami-rajat has quit IRC17:14
*** etingof has joined #openstack-ironic17:16
*** derekh has quit IRC17:24
*** ociuhandu has quit IRC17:27
*** ociuhandu has joined #openstack-ironic17:28
*** ociuhandu has quit IRC17:28
*** ociuhandu has joined #openstack-ironic17:28
*** dsneddon has quit IRC17:38
*** S4ren has quit IRC17:38
*** e0ne has joined #openstack-ironic17:42
*** jtomasek has quit IRC17:42
*** yaawang has quit IRC17:57
*** yaawang has joined #openstack-ironic17:59
*** e0ne has quit IRC18:01
*** ricolin has quit IRC18:02
*** igordc has joined #openstack-ironic18:06
*** goldyfruit___ has quit IRC18:11
*** dsneddon has joined #openstack-ironic18:12
*** Lucas_Gray has quit IRC18:16
*** Goneri has quit IRC18:18
*** dsneddon has quit IRC18:18
*** ociuhandu has quit IRC18:19
*** dsneddon has joined #openstack-ironic18:24
*** diga has quit IRC18:28
*** jtomasek has joined #openstack-ironic18:36
*** goldyfruit___ has joined #openstack-ironic18:41
*** e0ne has joined #openstack-ironic18:43
*** e0ne has quit IRC18:45
*** igordc has quit IRC18:51
*** e0ne has joined #openstack-ironic18:51
*** goldyfruit___ has quit IRC19:01
*** lucasagomes has quit IRC19:06
*** lucasagomes has joined #openstack-ironic19:06
*** lucasagomes has quit IRC19:14
*** mgoddard has quit IRC19:17
*** mgoddard has joined #openstack-ironic19:20
*** goldyfruit has joined #openstack-ironic19:23
*** jcoufal_ has quit IRC19:30
*** jcoufal has joined #openstack-ironic19:31
*** jtwill98 has quit IRC19:35
*** pcaruana has joined #openstack-ironic19:43
*** e0ne has quit IRC19:46
*** derekh has joined #openstack-ironic20:01
*** derekh has quit IRC20:01
*** pcaruana has quit IRC20:12
*** Lucas_Gray has joined #openstack-ironic20:31
*** jtomasek has quit IRC20:43
TheJuliaUssuri jokes... oh my20:57
TheJuliaI guess we should start early!20:57
TheJuliadtantsur|afk: thanks for the ironicclient fix21:02
TheJuliadtantsur|afk: re: centos job, we need something like https://review.opendev.org/#/c/683262/ to actually cause the image to be part of the equation21:03
patchbotpatch 683262 - ironic - WIP: if wholedisk set boot dev to disk - 2 patch sets21:03
TheJuliadtantsur|afk: I'm goin gto remove the wip flag, since scarily, it passes. I really don't have time to whip up unit test on it though.21:03
TheJuliaActually, I'll create a bug and all that stuff21:03
*** jcoufal has quit IRC21:04
rpiosoTheJulia: During today's meeting, we discussed landing https://review.opendev.org/#/c/672498/ in Train. We agreed I would send an FFE to the ML today. Is that agreeable with you?21:07
patchbotpatch 672498 - ironic - Add Redfish vmedia boot interface to idrac HW type - 20 patch sets21:07
TheJuliaironic-cores: https://review.opendev.org/#/c/679332/ would be good to review. It basically hardens the code path on IPA such that we properly tear-down the iscsi connection. Ultimately this likely will help reduce some of the more odd race conditions.21:10
patchbotpatch 679332 - ironic-python-agent - Attempt tgtd session detach - 4 patch sets21:10
TheJuliarpioso: It is agreeable with me21:10
rpiosoThank you! The FFE will be sent out over the next few hours.21:11
TheJuliadtantsur|afk: oh wow. it is not really deterministic21:12
TheJuliaoh...21:13
TheJuliahttps://7f88627495221102c59d-fa02bede2ecdc79d66ce6b922331a4c1.ssl.cf2.rackcdn.com/679332/4/check/metalsmith-integration-ipa-src/1a382a3/controller/logs/ and https://openstack.fortnebula.com:13808/v1/AUTH_e8fd161dc34c421a979a9e6421f823e9/zuul_opendev_logs_919/641731/24/check/metalsmith-integration-glance-localboot-centos7 seem to be doing the right thing.21:14
TheJuliaI've +2'ed https://review.opendev.org/#/c/681838/4 based upon what is basically a glance. arne_wiebalck if you would do the honors of a quick review, I think that would cover it.21:18
patchbotpatch 681838 - ironic-python-agent - Delete_configuration, consider removed raid member... - 4 patch sets21:18
* TheJulia gives up writing story because can't find a easily reproduced error, but decides to remove the wip flag anyway21:18
*** tesseract has quit IRC21:23
openstackgerritJulia Kreger proposed openstack/ironic master: if wholedisk set boot dev to disk  https://review.opendev.org/68326221:26
TheJuliadtantsur|afk: ^^^ has passed the gate twice, 3rd time is a charm right?21:27
*** Wryhder has joined #openstack-ironic21:27
*** igordc has joined #openstack-ironic21:27
*** Lucas_Gray has quit IRC21:27
*** Wryhder is now known as Lucas_Gray21:28
*** bdodd has quit IRC21:28
*** goldyfruit has quit IRC21:32
*** bfournie has quit IRC21:33
*** rcernin has joined #openstack-ironic21:38
*** rcernin has quit IRC21:40
*** rcernin has joined #openstack-ironic21:40
*** cdearborn has quit IRC21:41
*** goldyfruit has joined #openstack-ironic21:43
openstackgerritJulia Kreger proposed openstack/ironic master: Fix get_boot_option logic  https://review.opendev.org/67913221:46
openstackgerritMerged openstack/ironic master: Document PXE retries  https://review.opendev.org/68396722:03
openstackgerritMerged openstack/ironic master: Release notes cleanup for 13.0.0 (mk2)  https://review.opendev.org/68398322:11
openstackgerritMerged openstack/ironic master: Update env. variables in the documentation  https://review.opendev.org/68395322:13
*** bfournie has joined #openstack-ironic22:33
*** jrwang has quit IRC22:40
*** mkowalski__ has quit IRC22:46
*** mkowalski__ has joined #openstack-ironic22:47
*** rh-jelabarre has quit IRC23:06
*** TxGirlGeek has quit IRC23:06
*** Lucas_Gray has quit IRC23:26
*** sthussey has quit IRC23:45

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