Monday, 2018-04-30

*** dtantsur|afk has quit IRC01:08
*** dtantsur has joined #openstack-ironic01:08
*** anton has quit IRC01:09
*** anton has joined #openstack-ironic01:09
*** krtaylor has quit IRC01:32
*** akhilaki has joined #openstack-ironic01:54
*** d0ugal has quit IRC02:49
*** cjloader has joined #openstack-ironic02:49
*** d0ugal has joined #openstack-ironic02:50
*** namnh has joined #openstack-ironic02:55
*** namnh has quit IRC03:02
*** mvpnitesh has joined #openstack-ironic03:40
*** akhilaki has quit IRC03:47
*** Nisha_Agarwal has joined #openstack-ironic04:00
*** namnh has joined #openstack-ironic04:36
*** namnh has quit IRC04:36
*** namnh has joined #openstack-ironic04:37
*** links has joined #openstack-ironic04:39
*** namnh has quit IRC04:41
*** namnh has joined #openstack-ironic04:52
*** namnh has quit IRC04:56
*** rbartal has joined #openstack-ironic05:12
*** jaganathan has joined #openstack-ironic05:23
*** jaganathan has quit IRC05:23
*** jaganathan has joined #openstack-ironic05:24
*** namnh has joined #openstack-ironic05:33
*** namnh has quit IRC05:38
*** masber has joined #openstack-ironic05:41
*** marios has joined #openstack-ironic05:46
*** d0ugal has quit IRC05:48
*** masuberu has joined #openstack-ironic05:51
*** d0ugal has joined #openstack-ironic05:54
*** masber has quit IRC05:55
*** masuberu has quit IRC06:02
olivierb-morning everyone, concerning https://review.openstack.org/#/c/563894/ which is still in error after reno update hase been merged via https://review.openstack.org/564794 are there anything to do in bifrost to have this passed (like changing reno===2.7.0 in lowercontraints.txt) ? Sorry not being very familiar with inter-projects dependencies06:33
patchbotpatch 563894 - bifrost - Add support for NTP servers in dnsmasq configuration06:33
patchbotpatch 564794 - requirements - update constraint for reno to new release 2.9.1 (MERGED)06:33
*** namnh has joined #openstack-ironic06:35
*** namnh has quit IRC06:41
*** namnh has joined #openstack-ironic06:57
*** namnh has quit IRC07:02
*** Nisha_Agarwal has quit IRC07:06
*** Guest1988 has joined #openstack-ironic07:10
*** Guest1988 has quit IRC07:17
*** namnh has joined #openstack-ironic07:17
*** namnh has quit IRC07:21
*** Nisha_Agarwal has joined #openstack-ironic07:27
olivierb-however after relaunching the tests and looking more closely at the log, I can see07:34
olivierb-2018-04-30 07:02:16.091993 | ubuntu-xenial | HEAD is now at b8eb3f6 update constraint for reno to new release 2.9.107:34
olivierb-so the failure still seems to be there but not related to this fix :-( Could someone please have a look and tell what's wrong please07:35
*** hoonetorg has quit IRC07:41
*** lucas-afk is now known as lucasagomes07:54
*** hoonetorg has joined #openstack-ironic07:55
*** e0ne has joined #openstack-ironic07:57
*** namnh has joined #openstack-ironic08:05
*** Nisha_Agarwal has quit IRC08:08
*** namnh has quit IRC08:09
*** Nisha_Agarwal has joined #openstack-ironic08:10
*** livelace has joined #openstack-ironic08:16
*** cjloader has quit IRC08:23
*** Nisha_Agarwal has quit IRC08:27
*** derekh has joined #openstack-ironic08:29
*** namnh has joined #openstack-ironic08:35
*** namnh has quit IRC08:39
*** nmathew has joined #openstack-ironic08:41
*** pmannidi has quit IRC08:52
*** namnh has joined #openstack-ironic08:56
*** rbartal has quit IRC08:58
vdrokolivierb-: i'm not sure what's going on there, but it seems this patch https://review.openstack.org/564763 was intended for stable branches, not for master08:59
patchbotpatch 564763 - reno - include the branch name in anchors to make them mo... (MERGED)08:59
*** namnh has quit IRC09:01
openstackgerrityolanda.robla proposed openstack/ironic master: WIP: Add Node BIOS support - REST API  https://review.openstack.org/51257909:08
*** namnh has joined #openstack-ironic09:10
*** namnh has quit IRC09:15
*** racedo has joined #openstack-ironic09:18
*** MattMan has quit IRC09:18
*** MattMan has joined #openstack-ironic09:18
*** namnh has joined #openstack-ironic09:25
*** nmathew has quit IRC09:29
openstackgerritDmitry Tantsur proposed openstack/ironic master: Switch to the fake-hardware hardware type for API tests  https://review.openstack.org/56451909:36
*** yolanda_ is now known as yolanda09:39
yolandahi dtantsur , i have a question about https://review.openstack.org/#/c/512579/ . Specifically about the factory_reset and apply_configuration methods for the REST API... i'm unsure on how that can be implemented. Should we have matching methods on RPC API?09:40
patchbotpatch 512579 - ironic - WIP: Add Node BIOS support - REST API09:40
sambettsolivierb-: I've worked out the issue behind that warning, /me will push a fix09:56
openstackgerritSam Betts proposed openstack/bifrost master: Prevent reno rendering current release notes twice  https://review.openstack.org/56518010:03
sambettsolivierb-: ^^^10:03
*** livelace-link has quit IRC10:09
*** livelace-link has joined #openstack-ironic10:12
openstackgerritMerged openstack/ironic master: Remove the Keystone API V2.0 endpoint registration  https://review.openstack.org/56499910:26
olivierb-sambetts many thanks10:30
*** namnh has quit IRC10:44
*** jcoufal has joined #openstack-ironic11:00
*** lucasagomes is now known as lucas-hungry11:04
*** namnh has joined #openstack-ironic11:05
*** namnh has quit IRC11:09
dtantsurmorning/afternoon ironic11:15
dtantsuryolanda: hi, these are going to be clean steps IIRC11:16
dtantsurso they should just be marked as such11:17
*** dprince has joined #openstack-ironic11:18
yolandadtantsur, is there some code i can take as example on how to set up clean steps from rest api?11:20
dtantsuryolanda: no explicit set up needed11:21
*** rbartal has joined #openstack-ironic11:21
yolandadtantsur, i'm talking about http://git.openstack.org/cgit/openstack/ironic-specs/tree/specs/approved/generic-bios-config.rst#n13111:24
yolandasee 2 api calls: factory_reset and apply_configuration11:24
dtantsurs/api calls/clean steps/, yes11:24
yolandano need to expose those on rest api?11:25
dtantsuryolanda: yep, clean steps are exposed implicitly11:25
yolandaah ok, i was confused about that, thinking that i had to expose on the bios api11:25
*** mvpnitesh has quit IRC11:25
yolandathx11:26
*** namnh has joined #openstack-ironic11:26
*** namnh has quit IRC11:30
*** rbartal has quit IRC11:32
*** parlos has joined #openstack-ironic11:33
openstackgerritHarald JensÃ¥s proposed openstack/ironic-inspector master: Call shutdown() on SIGTERM - controlled teardown  https://review.openstack.org/56333511:33
*** rbudden has joined #openstack-ironic11:36
dtantsuryolanda: it's confusing indeed... we may eventually end up with Clean Steps v2 to clean up the mess :)11:38
*** namnh has joined #openstack-ironic11:42
*** parlos_ has joined #openstack-ironic11:43
*** parlos has quit IRC11:44
*** rbartal has joined #openstack-ironic11:47
*** parlos has joined #openstack-ironic11:55
*** parlos_ has quit IRC11:55
*** lucas-hungry is now known as lucasagomes11:57
*** parlos_ has joined #openstack-ironic11:57
*** parlos has quit IRC11:59
*** parlos has joined #openstack-ironic12:01
*** parlos_ has quit IRC12:03
jrollmorning12:15
openstackgerrityolanda.robla proposed openstack/ironic master: Add Node BIOS support - REST API  https://review.openstack.org/51257912:16
dtantsurmorning jroll12:17
yolandadtantsur,  i pushed the rest api change. It will need a lot of review12:17
dtantsursure :)12:17
*** trown|outtypewww is now known as trown12:18
*** namnh has quit IRC12:22
*** kzaitsev_pi has quit IRC12:23
*** zshi has quit IRC12:23
*** namnh has joined #openstack-ironic12:24
*** zshi has joined #openstack-ironic12:25
jrollheh, just noticed the base driver supported attribute is false, so the classic oneview drivers were never marked supported12:26
sambetts:/12:26
openstackgerritJim Rollenhagen proposed openstack/ironic master: Deprecate Oneview  https://review.openstack.org/56458712:28
jroll^ that marks the interfaces unsupported as well, should be ready to land now12:28
*** rkharya has joined #openstack-ironic12:28
*** namnh has quit IRC12:28
*** kzaitsev_pi has joined #openstack-ironic12:30
dtantsurjroll: well, right, the classic drivers are deprecated, all of them12:31
dtantsurI'm not sure we have "supported" notion for interfaces btw12:31
jrolldtantsur: ah, that's true. and we do have it: https://github.com/openstack/ironic/blob/master/ironic/drivers/base.py#L5212:31
jroller12:32
jrollhttps://github.com/openstack/ironic/blob/master/ironic/drivers/base.py#L20012:32
dtantsurjroll: TIL :D12:35
*** rkharya has quit IRC12:38
*** kzaitsev_pi has quit IRC12:39
jrollha12:39
*** kzaitsev_pi has joined #openstack-ironic12:40
* dtantsur dreams of stopping working on the driver composition one day12:40
TheJuliagood morning everyone12:49
dtantsurmorning TheJulia12:50
sambettso/12:50
dtantsurhmm, https://review.openstack.org/#/c/536928/ merged on Feb 1st, meaning that I can remove the classic drivers tomorrow :D12:50
patchbotpatch 536928 - ironic - Deprecate classic drivers (MERGED)12:50
dtantsurTheJulia: should I send The Very Last Warning to the ML re ^^^?12:51
jroll++12:52
dtantsurwe need https://review.openstack.org/564519 before we can switch though. but the drivers will hopefully be gone by Rocky-212:54
patchbotpatch 564519 - ironic - Switch to the fake-hardware hardware type for API ...12:54
*** rloo has joined #openstack-ironic12:56
* dtantsur done12:58
* jroll will take a look12:58
rlooHappy. Monday morning ironic'ers, jroll, dtantsur13:00
dtantsurhey rloo13:01
jroll\o13:01
dtantsurjroll: right now I need some opinion on https://review.openstack.org/#/c/564560/ to move forward13:02
patchbotpatch 564560 - ironic - Move API (functional) tests to separate jobs13:02
jrolldtantsur: ah, ok, on my list now13:02
rloodtantsur, TheJulia: wrt https://review.openstack.org/#/c/562314/. What about sending email out to dev-list/operators, to see if anyone uses that behavior, and mention that we want to backport this? I feel like it is a good thing to backport, but I am worried about 'silently' backporting it...13:09
patchbotpatch 562314 - ironic (stable/queens) - Remove vifs upon teardown13:09
dtantsurI'm +1 to asking ops13:13
* dtantsur does not have time to write it though13:13
TheJuliadtantsur: Yeah, might as well send one last warning to the mailing list13:20
TheJuliarloo: I'm don't think we are silently backporting it, but we have operators that work closely to the community that would see it. Sadly those are likely a chunk of chose operators that are on that list.13:21
*** links has quit IRC13:22
rlooTheJulia: well, by 'silent', we're backporting and releasing, and assuming folks will read the release notes. Given that there's an upgrade impact to this, I feel like we ought to advertise/mention it beyond just the release notes.13:22
rloodtantsur: wrt that backport patch, you could approve it, it doesn't require me. I am just uncomfortable as-is.13:23
TheJuliarloo: Okay, I can send that email later today13:24
rloothx TheJulia!13:25
*** superdan is now known as dansmith13:29
*** jroll has quit IRC13:36
dtantsurrloo: I'll wait for you to become comfortable :)13:36
rloodtantsur: okay :)13:36
rloodtantsur: btw, wrt https://review.openstack.org/#/c/564517/, is it tested that fake-hardware works?13:36
patchbotpatch 564517 - ironic-tempest-plugin - Allow using the fake-hardware hardware type for AP...13:36
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Do not run functional (API) tests in the CI  https://review.openstack.org/56522213:37
dtantsurrloo: I have a patch for that, but I'm holding it for now: https://review.openstack.org/56451913:37
patchbotpatch 564519 - ironic - Switch to the fake-hardware hardware type for API ...13:37
*** lbragstad has joined #openstack-ironic13:37
*** rh-jelabarre has quit IRC13:38
*** jroll has joined #openstack-ironic13:38
rloodtantsur: ah, looks good then, thx.13:38
openstackgerritMerged openstack/bifrost master: Prevent reno rendering current release notes twice  https://review.openstack.org/56518013:41
openstackgerritDmitry Tantsur proposed openstack/python-ironicclient master: Do not run functional (API) tests in the CI  https://review.openstack.org/56522313:41
*** dprince has quit IRC13:42
* TheJulia makes coffee13:45
*** rh-jelabarre has joined #openstack-ironic13:46
openstackgerrityolanda.robla proposed openstack/bifrost master: Add support for NTP servers in dnsmasq configuration  https://review.openstack.org/56389413:47
*** r-daneel has joined #openstack-ironic13:47
*** livelace has quit IRC14:02
rloodtantsur: i just got around to looking at https://review.openstack.org/#/c/555610/. For all of our other X-WAIT states, they transition to X-ING (some very briefly in this state) before transitioning to the target state. Wondering if it makes sense to do so with INSPECTWAIT too (you know, 'consistency')14:03
patchbotpatch 555610 - ironic - Implementation of inspect wait state (MERGED)14:03
dtantsurrloo: well, the way inspect interface is implemented now, it's not needed, because it's either sync or async, not mixed14:04
dtantsurwe can update the state machine if we ever need it, right?14:04
rloodtantsur: yeah, i think we can update if we need it. i doubt (hope not anyway) that we'll need it. for now, more of a consistency thing, i haven't thought beyond that as to why/whether we'll need it.14:05
*** livelace has joined #openstack-ironic14:05
dtantsuryeah, I'm fine either way14:06
rloodtantsur: ok, i'll just comment in the patch. thx :)14:06
*** namnh has joined #openstack-ironic14:09
dtantsurrloo: I only care about this change at all because of http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/inspection-abort.html14:10
rloodtantsur: ah, yes. would be good to have that.14:10
*** namnh has quit IRC14:13
rlooTheJulia: given your concern with https://review.openstack.org/#/c/557755/ (wrt existing ipa where NULL worked), would it make sense to try with NULL if '' failed?14:14
patchbotpatch 557755 - ironic-python-agent - Pass empty string to hdparm --security-unlock in G...14:14
*** ElCoyote_ has joined #openstack-ironic14:22
*** rbartal has quit IRC14:24
*** e0ne has quit IRC14:26
*** rajinir has joined #openstack-ironic14:28
rloodtantsur: if you're good with my comments for https://review.openstack.org/#/c/550464/, i can update and approve. if you're not good, i can approve :)14:29
patchbotpatch 550464 - ironic-inspector - Change the default discovery driver to fake-hardware14:29
* dtantsur looks14:30
dtantsurrloo: yep, good suggestions14:30
rloodtantsur: ok, will update/approve in a few min.14:30
dtantsurthanks!14:30
*** hemna_ has joined #openstack-ironic14:31
*** baha has joined #openstack-ironic14:35
openstackgerritRuby Loo proposed openstack/ironic-inspector master: Change the default discovery driver to fake-hardware  https://review.openstack.org/55046414:36
TheJuliaokay, yay , things like vpn work agian14:45
dtantsurheh14:47
rlooTheJulia: good! :)14:47
dtantsurTheJulia: are we ready to land https://review.openstack.org/#/c/564587/ ?14:47
TheJuliarloo: so there is some magic under the hood of hdparm for NULL translation to equate to nothing, but... I don't know, I guess I worry needlessly14:47
rloo13 minutes til meeting, update whiteboard please!14:47
patchbotpatch 564587 - ironic - Deprecate Oneview14:47
rlooTheJulia: so you're good with the code as-is? (except the unit test)?14:48
rlooTheJulia: i'm not actually sure I understand why NULL worked before... but clearly it did. for some users anyway...14:49
TheJuliayeah14:49
TheJuliamaybe jroll mgiht have memories in regards to wiping disks?14:49
TheJuliajust for sanity's sake14:49
*** parlos has quit IRC14:49
TheJuliarloo: w/r/t oneview, I think were good to land the patch, I can follow-up with doc updates later14:49
dtantsurhttp://logs.openstack.org/22/565222/1/check/ipa-tempest-dsvm-partition-bios-agent_ipmitool-coreos-src/2dac742/logs/ironic-bm-logs/node-0_console.txt.gz WUT??14:50
*** jiapei has joined #openstack-ironic14:50
rlooTheJulia: wrt oneview, ok. i think i was waiting for CI but can approve.14:50
* dtantsur pressed the button already14:50
*** namnh has joined #openstack-ironic14:50
rloodtantsur: those tests pass w/o your change?14:52
*** jiapei_ has joined #openstack-ironic14:53
rlooTheJulia: the bifrost/pip10 thing was fixed last week, right? (still in whiteboard)14:53
TheJuliarloo: yeah14:53
*** cjloader has joined #openstack-ironic14:54
jrollTheJulia: I don't think I've ever run that code path, I seem to remember that being post-onmetal-working14:54
TheJuliahmmm14:54
TheJuliamust have been jayf then14:55
*** arxcruz|pto has joined #openstack-ironic14:55
*** namnh has quit IRC14:55
dtantsurrloo: dunno, haven't seen the IPA CI in a while14:55
jrollwell, he worked on the same thing14:55
jroll:P14:55
dtantsurbut I only change the tempest regex....14:55
JayFTheJulia: it wasn't me </shaggy> What is it, exactly, that I didn't do though? :P14:56
jrollTheJulia: maybe we did, dunno. would never be able to remember the specific arguments we passed :)14:56
jrollJayF: https://review.openstack.org/#/c/557755/4/ironic_python_agent/hardware.py14:56
patchbotpatch 557755 - ironic-python-agent - Pass empty string to hdparm --security-unlock in G...14:56
rloodtantsur: sigh. although i just reviewed an IPA patch https://review.openstack.org/#/c/557755/. Oh, apr 24 was last date.14:57
patchbotpatch 557755 - ironic-python-agent - Pass empty string to hdparm --security-unlock in G...14:57
JayFwe absolutely literally had to pass in 'NULL'14:57
JayFfor those machines14:57
dtantsurhmm, should we try both then?14:57
* rloo is amazed at JayF memory14:57
rloo+1 to trying both then.14:57
JayFbut the firmwares on the SATADOMs we used in that generation were absolutely godawful14:57
JayFrloo: when you spend multiple days trying to figure out the magic words, it's easy to remember them14:58
TheJuliaI'm also +1 to trying both in that case.14:58
JayFjroll: figuring out ^ that magic was the first thing clif did (with me) on the team14:58
jrollah14:58
jrollthanks JayF :)14:58
JayFjroll: $20 says it was a language-barrier problem with us requesting a "null password" from the group that made those14:59
jrollheh14:59
jrollcould be14:59
JayFrloo: TheJulia: That's the flavor for that, BTW ^ -- that security unlock string is a /password/ you have to know to unlock the  disk14:59
TheJulia#startmeeting ironic15:00
openstackMeeting started Mon Apr 30 15:00:10 2018 UTC and is due to finish in 60 minutes.  The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot.15:00
JayFso it doesn't surprise me emptystring works on a lot of machines15: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
TheJuliaJayF: I knew that, but I thought there was some magic code in hdparm for certian cases15:00
TheJuliaanyway!15:00
rloothx JayF (want to join our meeting?)15:00
TheJuliaMeeting time!15:00
rlooo/15:00
arxcruz|ptoo/15:00
*** beekneemech is now known as bnemec15:00
* dtantsur lurks15:00
* TheJulia hands out cokies15:00
TheJuliacokies15:00
jiapei_o/15:00
vdrokщ.15:00
vdrokwoops, o/15:00
jroll\o15:00
TheJuliaokay, o key... why you no like cookies :(15:00
* jroll runs to grab water15:01
TheJuliaA great comedic start to the morning!15:01
dtantsurvdrok: щ to you as well :D15:01
JayFo/ (the goodbye kind!)15:01
TheJuliaOur agenda can be found at https://wiki.openstack.org/wiki/Meetings/Ironic15:01
dtantsurhey hey JayF :)15:01
TheJuliao/ JayF15:01
vdrok:)15:01
jiapei_Good evening :)15:01
TheJulia#link https://wiki.openstack.org/wiki/Meetings/Ironic15:01
TheJuliaAnyway, getting started!15:01
TheJulia#topic Announcements/Reminders15:02
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)"15:02
TheJuliaWe are a few weeks out from the Summit, I think we should cancel the meeting, but we can vote on that next week15:03
TheJuliaaside from that, I have nothing else to announce or remind us of this week.15:03
rlooTheJulia: cancel the meeting the week of the summit, right?15:03
*** persia has quit IRC15:03
TheJuliarloo: yes15:04
rlooTheJulia: +1 to cancel :)15:04
*** namnh has joined #openstack-ironic15:04
*** jaganathan has quit IRC15:04
TheJuliaDoes anyone else have anything to annouce?15:04
TheJuliaI guess not, and we can move on15:05
TheJulia#topic Reviewing action items from last week15:06
*** openstack changes topic to "Reviewing action items from last week (Meeting topic: ironic)"15:06
TheJulia#link http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-23-15.00.log.html15:07
TheJulia#link https://storyboard.openstack.org/#!/story/200188415:07
TheJuliaIt looks like dmitry posted one patch. dtantsur will there be a second patch?15:07
dtantsuryep, but it feels less urgent15:08
TheJuliaOkay, thanks!15:08
TheJulia#topic Review Subteam Status Reports15:09
*** openstack changes topic to "Review Subteam Status Reports (Meeting topic: ironic)"15:09
openstackgerritMerged openstack/bifrost master: Add support for NTP servers in dnsmasq configuration  https://review.openstack.org/56389415:09
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:09
*** rh-jelabarre has quit IRC15:10
TheJuliaLooks like we at least landed one of the bios interface patches last week15:11
rlooSIGHUP work is done too :)15:12
dtantsur\o/15:12
jrollnice!15:12
TheJuliadtantsur: any remaining hardware type work on the horizon regarding the patches to delete classic drivers?15:12
TheJuliarloo: by chance has that been updated in storyboard yet?15:13
dtantsuryep15:13
dtantsurhttps://review.openstack.org/#/q/topic:api-jobs is needed to move forward with s/fake/fake-hardware/ in the CI15:13
rlooTheJulia: i don't know; it is on my list to check/do after this meeting :)15:13
TheJuliadtantsur: any this week that we need to have the ironic community attention on?15:13
TheJuliarloo: ack, let me know if you need me to do anything then15:13
dtantsurthese patches, then https://review.openstack.org/56451915:13
patchbotpatch 564519 - ironic - Switch to the fake-hardware hardware type for API ...15:13
rlooTheJulia: will do, thx!15:13
TheJuliadtantsur: any expectation on unblocking the last patch this week?15:15
dtantsurTheJulia: it's blocked by that three patches, essentially15:15
dtantsurthen I'll double-check that I don't break any CI, and unblock it15:15
dtantsurso yes, totally possible15:15
TheJuliaokay, then I'll just leave the link to the three patches for now15:16
dtantsurk15:16
TheJuliaI made some minor updates to the priorities for this week list. Sadly I didn't have much time last week since I was hunting nova/heat issues15:17
* TheJulia sighs that the cats have realized how to open the window blinds....15:18
TheJuliaLooking through things on the whiteboard, I think we should go ahead and remove oneview references, thoughts?15:19
jroll+115:19
rlooTheJulia: +1 (i was going to ask, thx)15:19
TheJulia#action TheJulia to scrub the whiteboard for oneview items this week15:20
TheJuliaI'll go ahead and put a doc note up as well15:20
rlooTheJulia: might be useful to mention here, why we're doing this15:20
TheJulia#action TheJulia to put up deprecation doc note in the oneview docs.15:20
TheJuliaI'm sure as a number of you remember when contributors from UFCG, about two weeks before the PTG, entered the channel and said that HPE had cancelled their further efforts and had requested that the hardware be shipped back to HPE.15:21
TheJuliaI've repeatedly engaged HPE and haven't gotten any traction, nor has there been any public action indicating that they may be working to re-establish CI for oneview.15:22
TheJuliaSince there is no third party CI, it is impossible for us to support the driver and ensure it is in working condition, hence why we're deprecating and will eventually remove it from the source code tree if third party CI is never re-established.15:23
TheJuliaAny questions?15:23
*** dprince has joined #openstack-ironic15:24
* TheJulia hears crickets as a service15:24
TheJuliaso with that, lets move to priorites for the week, then discusison.15:24
TheJuliaEveryone good with that?15:25
dtantsuryep15:25
TheJulia#topic Priorities for the coming week15:25
*** openstack changes topic to "Priorities for the coming week (Meeting topic: ironic)"15:25
TheJuliaStarting around line 10215:26
TheJulia#link https://etherpad.openstack.org/p/IronicWhiteBoard15:26
dtantsurLGTM :)15:26
TheJulianot much different from last week. I removed the merged patches, added a link to make dtantsur's life easier15:27
jrollditto15:27
* dtantsur likes making dtantsur's life easier :D15:27
TheJuliaheh15:27
TheJuliaAnyway, moving on#15:28
TheJuliaerr15:28
TheJulia!15:28
TheJulia#topic Discussion15:28
vdrokoh btw all the prerequisites for https://review.openstack.org/359061 are merged15:28
*** openstack changes topic to "Discussion (Meeting topic: ironic)"15:28
patchbotpatch 359061 - python-ironicclient - Switch to none auth for standalone mode15:28
dtantsurvdrok: oh, quite a long read :) I'll try to get to it15:28
arxcruz|ptoHi guys, I'm from tripleo-ci team, and we would like to add a tripleo job on ironic project15:28
vdrokthx!15:28
rloovdrok: thx! (I need to get a dev env set up to test that...)15:29
arxcruz|ptoinitially the job will take 2 hours to finish, installing tripleo and running tempest on undercloud15:29
arxcruz|ptonon voting obviously, but hopefully with time, get it in voting15:29
arxcruz|ptowe are working hard to decrease the time of the job as well15:29
TheJuliaarxcruz|pto: why is it that this job is required?15:29
vdrokrloo: there also is a patch on top running functional tests with it :) https://review.openstack.org/50833015:29
patchbotpatch 508330 - python-ironicclient - WIP allow to use none auth in functional tests15:29
arxcruz|ptoTheJulia: not required, but would help us to catch problems in tripleo faster since tripleo uses ironic a lot15:30
rloovdrok: thx. will try to look soon, it has been on my list for awhile...15:30
dtantsurTheJulia: one benefit for us is testing local boot15:30
arxcruz|ptosometimes you don't see the problem running on devstack15:30
*** cjloader has quit IRC15:30
TheJuliaWe already have a few two hour jobs which, I guess I'm worried about about adding another one and slowing down our CI runs even more15:30
rloo2 hours is a long time. what is the longest time for our existing tests?15:30
rloo(existing voting tests)15:31
dtantsurrloo: I suspect grenade jobs are comparable15:31
TheJuliarloo: multinode grenade is like 1 hour 45 minutes on a good day15:31
jrolldtantsur: don't we test local boot already?15:31
*** cjloader has joined #openstack-ironic15:31
dtantsurjroll: IIRC no :(15:31
arxcruz|ptoas i said, we are working to reduce to 1.5 hours15:31
dtantsurwe were blocked by cirros not supporting grub15:31
TheJuliarloo: 2hr 20-minute-ish on a moderate day15:31
jrollwut15:31
jroll:(15:31
arxcruz|ptoit will install the undercloud and run ironic tempest tests15:31
TheJuliaarxcruz|pto: is that even really required for a test executing reporting to ironic?15:32
*** rloo has quit IRC15:32
TheJuliasince all of our other tests run tempest based quites15:32
TheJuliasquites15:32
TheJuliasuites15:32
* TheJulia should just give up on typing today15:32
arxcruz|ptoTheJulia: yeah, but, not in tripleo environment15:32
dtantsurTheJulia: typing is overrated15:32
*** rloo has joined #openstack-ironic15:32
TheJuliadtantsur: indeed15:32
dtantsurarxcruz|pto: I think the question is how it benefits ironic, not tripleo :)15:33
arxcruz|ptowe see several problems that pass without nobody see on devstack, when we change the environment to tripleo, things start to pop up everywhere15:33
arxcruz|ptodtantsur: less bugs to fix later :D15:33
dtantsurI like being tested by something resembling production, to be honest15:33
jrollto be clear, is this tripleo job in the infra queue or some third party ci queue?15:33
TheJuliaarxcruz|pto: Has Ironic had recent changes that have broken TripleO?15:33
arxcruz|ptoTheJulia: from time to time, I don't recally anything right now, we usually try to fix in our side15:34
arxcruz|ptobut we want to increase the coverage15:34
arxcruz|ptojroll: infra queue15:34
jrollok15:34
dtantsurarxcruz|pto: but there are also RDO 3rdparty jobs, right?15:34
arxcruz|ptodtantsur: yes15:34
dtantsuradding these could be easier, no?15:34
dtantsurI mean, they don't block the gate if they take too long15:35
arxcruz|ptowell, I'm opened to options :)15:35
jrollI don't have a problem with this, I see it as helping other openstack projects. third party would be nice to not increase results time, but I'm fine with infra queue too15:35
TheJuliaI really like the idea of a third party job personally15:35
arxcruz|ptoif you guys agree, we can add a job, non voting, if it's not working, we can move on with other options15:35
*** cjloader has quit IRC15:35
dtantsurarxcruz|pto: we can even create a featureset using ironic both in under and over cloud15:35
dtantsurdouble ironic testing \o/ :D15:35
arxcruz|ptodtantsur: let's start simple :D lol15:35
TheJuliaarxcruz|pto: move on with other jobs? sounds like there is more to this plan?15:35
arxcruz|ptoTheJulia: sorry, what I meant was move on with other options, like move the job to thirdy party15:36
arxcruz|ptothird*15:36
rloomy only concern with it, even as non-voting, is if it delays us getting the CI results, but since it is faster than grenade multi*, it should be OK. what about resources used for this testing?15:36
dtantsurarxcruz|pto: it is OVB, right?15:36
rlooarxcruz|pto: would it make sense to put it in RDO 3rdparty jobs first?15:37
arxcruz|ptodtantsur: nope, multinode15:37
dtantsurarxcruz|pto: ehmm, wait please15:37
arxcruz|ptorloo: i can check if that's possible15:37
dtantsurlast time I checked tripleo multinode CI did not deploy ironic instances15:37
arxcruz|ptoweshay: can it be ovb on rdocloud ?15:37
arxcruz|ptodtantsur: we are working on a featureset for that15:37
dtantsurhow do you run tempest if you don't have virtual ironic nodes?15:37
dtantsurah, okay15:37
jrollrloo: infra generally does not worry about resources if the job has value15:37
dtantsurmy concern is only about the possibility :) if you can use virtualbmc/whatever on a multinode tripleo job - fine15:38
arxcruz|ptodtantsur: it will be a whole new featureset15:38
weshayarxcruz|pto, we can add jobs to rdocloud especially for ironic15:38
weshayya15:38
rloojroll: oh, ok.15:38
weshaythat is only 3rd party though15:38
arxcruz|ptoso, my boss said yes, so up to you guys :D15:38
arxcruz|ptodtantsur: we can work on a job that run ironic in a way that benefit both teams :)15:39
TheJuliaSo to summarize, it seems like we're not opposed to the idea. That we would prefer third party style, but that is also not a hard requirement. Does that summarize contributors feelings well enough?15:40
weshayI say especially for ironic as ovb lends itself better to ironic testing15:40
dtantsurTheJulia: precisely15:40
jrollTheJulia: +115:40
rloo+1 (and as long as it doesn't take as long as our grenade jobs, so we aren't waiting for that test to finish)15:40
arxcruz|ptoso, ovb job, on third party, non voting for now, running ironic and testing ironic tempest plugins sounds good to everyone ?15:41
rlooarxcruz|pto: i think that's a good start.15:41
dtantsur++ great plan15:41
arxcruz|ptoweshay: ^15:41
TheJuliaSounds good, voting would need to be another discussion, but we can have that later once we have statistics15:41
weshay+115:42
weshaywho owns the ai?15:42
jroll++15:42
TheJuliaai?15:42
jrollai?15:42
dtantsurai?15:42
weshaylolz15:42
jroll:P15:42
weshayaction item15:42
weshaysorry15:42
weshaypointy head here15:42
dtantsurarxcruz|pto of course :D15:42
rlooarxcruz|pto i think15:42
jrollyep15:42
arxcruz|ptoTheJulia: sure, just to have it running on ironic, will help us to catch problems earlier15:42
arxcruz|ptoyes, me15:42
arxcruz|ptoI'll work this week on it15:43
dtantsurcool, thanks arxcruz|pto15:43
arxcruz|ptodtantsur: thanks too :)15:43
dtantsur(and why aren't you on pto??)15:43
jrollfor a moment I had hopes we had built artificial intelligence to review changes15:43
TheJulia#action arxcruz|pto to begin working on tripleo ci jobs using ironic reporting to ironic.15:43
dtantsurjroll: it's me, no?15:43
arxcruz|ptodtantsur: I forgot this meeting :)15:43
vdrokneed to run now, could someone also take a look at the following diff https://review.openstack.org/#/c/517921/3..4, it is about reporting resources as reserved vs deleting inventory during cleaning, maybe I am doing this wrong :)15:43
patchbotpatch 517921 - nova - ironic: Report resources as reserved when needed15:43
arxcruz|ptoi am supposed to be on pto15:43
jrolldtantsur: lol, true. we need moar AI though!15:43
rloociao vdrok15:44
TheJuliacode review AI as long as it +2's even with a hundred nitpicks.15:44
*** marios has quit IRC15:44
jrollvdrok: thanks!15:45
TheJuliaSo our other discussion item was regarding the backport of https://review.openstack.org/#/c/562314/15:45
patchbotpatch 562314 - ironic (stable/queens) - Remove vifs upon teardown15:45
dtantsurarxcruz|pto: go then :) brewdog is calling!15:45
TheJuliaThat was discussed a little earlier this morning between rloo, dtantsur, and myself. I'm presently drafting an email to the ops mailing list to run it past them/provide visibility15:45
rlooThank you TheJulia!15:46
arxcruz|ptodtantsur: been there too many times this month already, edeka had a promotion of belgian beers ;)15:46
TheJuliaMoving to open discussion if there are no objections15:46
dtantsurarxcruz|pto: oh :)15:46
rlooThere isn't anything wrong with that patch, except that it might break users (see upgrade note). Hence, my hesitation to land w/o more discussion/notice.15:46
jrollso with that backport - what use case depends on the VIFs remaining attached? non-nova I guess?15:47
*** akhilaki has joined #openstack-ironic15:47
dtantsurjroll: I cannot imagine, but it's an API behavior change15:47
TheJuliajroll: manual ironic + neutron outside of nova15:47
dtantsurso, in theory, we're breaking people15:47
jrollTheJulia: yeah, okay15:48
jiapei_+115:48
jrollthis is very much a bug for nova users, used to hit similar all the time at rackspace15:48
jrollleads to undeployable nodes15:48
TheJuliaYup15:49
jrollI'm okay with backporting it, with an email to the ops list15:49
TheJuliaIf anyone wonders wehre my brain has been for the last two weeks, its been hunting bugs along these lines15:49
dtantsurjroll: my position as well15:49
TheJuliaand the intertwined chaos they create15:49
TheJulia#topic Open Discussion15:50
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)"15:50
TheJuliaSo what else do we have to discuss?15:50
jrollI would suggest adding a config flag, but I don't believe people will see that and not see the email/release notes15:50
jiapei_TheJulia: I uploaded a new patch https://review.openstack.org/#/c/561405/15:50
patchbotpatch 561405 - ironic - Fix XClarity parameters discrepancy15:50
jiapei_Could you have a review of it? :)15:50
jrollso not worth it15:50
TheJuliaWith regards to storyboard, the new RFE's queue is starting to look sane, the untriaged queue is still crazy but I've been working on it.15:50
rloothanks TheJulia, for dealing with storyboard15:51
rlooTheJulia: that reminds me, bug bash on Thursday15:51
TheJuliajroll: yeah, I think the population using that scenario is extremely small, it is very rare for us to get people who are doing neutron integration with standalone seeking help so we do know they do exist :(  Then again, they might be happy since it might make their lives easier15:51
dtantsuryep, detaching VIFs explicitly may be annoying15:52
jrollTheJulia: makes sense15:52
TheJulia#info Bug Bash Thursday!15:53
TheJuliarloo: have call details been identified?15:53
*** srart has quit IRC15:53
rlooTheJulia: i looked earlier today and i think mike wants you to do a hangout. sec...15:54
jrollTheJulia: last email was asking if we can use your bluejeans, so no :)15:54
*** srart has joined #openstack-ironic15:54
TheJulia:(15:54
* dtantsur can also provide bluejeans, if needed15:54
rlooyup. is that ok TheJulia?15:54
rloowe only need one pair of bluejeans :)15:54
TheJuliaWhat was time time?15:54
jroll9am eastern15:54
jrollprobably best to use dtantsur's15:55
rloo1-3pm UTC15:55
openstackgerritMerged openstack/ironic stable/queens: Collect periodic tasks from all enabled hardware interfaces  https://review.openstack.org/56420115:55
dtantsurso, using mine?15:55
TheJuliait looks like I have a call right before15:56
TheJuliaso either of us can do it I think15:56
* dtantsur waits for the PTL's decision :)15:56
TheJuliadtantsur: if you want to host it, that would give me the ability to hang on the other call preceeding it a little longer if it runs long15:56
dtantsurTheJulia: okay, I'll do it15:57
TheJuliathanks!15:57
*** baha has quit IRC15:57
TheJuliaAnyway, I think we are out of things to discuss today15:57
TheJuliaAnd we have two minutes left!15:57
TheJuliaAnything else for this meeting?15:58
rloocrickets15:58
TheJuliaThanks everyone!15:58
jrollTheJulia: thanks for hosting :)15:59
TheJulia#endmeeting15:59
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic"15:59
openstackMeeting ended Mon Apr 30 15:59:26 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)15:59
openstackMinutes:        http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-30-15.00.html15:59
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-30-15.00.txt15:59
openstackLog:            http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-30-15.00.log.html15:59
*** cjloader has joined #openstack-ironic15:59
dtantsurTheJulia: https://bluejeans.com/309964257 will be the link16:01
TheJuliadtantsur: awesome16:01
dtantsurnow I have to go, see you on Wednesday16:02
* jroll finds lunch16:02
rlooTheJulia: wrt SIGHUP, I updated the story (and mentioned in our whiteboard; I will delete this item from our whiteboard later this week): https://storyboard.openstack.org/#!/story/200154516:02
*** cjloader has quit IRC16:02
*** dtantsur is now known as dtantsur|afk16:02
dtantsur|afko/16:02
*** cjloader has joined #openstack-ironic16:02
rloobye dtantsur|afk16:02
*** namnh has quit IRC16:04
TheJuliaOh right, may day16:05
*** namnh has joined #openstack-ironic16:06
*** namnh_ has joined #openstack-ironic16:07
*** lucasagomes is now known as lucas-hungry16:08
*** ijw has joined #openstack-ironic16:09
rloothe whiteboard indicates that some releasenotes jobs are/were broken by new reno. I'm going to assume they have been fixed.16:10
jrollrloo: yes16:10
rloothx for confirming jroll!16:11
*** namnh has quit IRC16:11
*** trown is now known as trown|lunch16:11
*** ijw has quit IRC16:12
*** namnh_ has quit IRC16:22
NobodyCamGood Morning Ironic'ers16:24
NobodyCamHappy Monday16:24
TheJuliaGood morning NobodyCam16:31
NobodyCam:)16:32
*** itlinux has joined #openstack-ironic16:34
*** athomas_ has quit IRC16:43
*** jiapei has quit IRC16:48
*** jiapei_ is now known as jiapei16:49
*** athomas has joined #openstack-ironic16:51
TheJuliarloo: email sent to the mailing list17:00
TheJuliawell17:00
TheJuliaops mailing list17:00
rlooTheJulia: thx!17:00
*** derekh has quit IRC17:02
*** athomas has quit IRC17:04
* TheJulia makes some lunch and then gets back to the never ending to-do list17:05
*** dprince has quit IRC17:25
*** itlinux has quit IRC17:29
openstackgerritMerged openstack/ironic master: Deprecate Oneview  https://review.openstack.org/56458717:30
*** jcoufal_ has joined #openstack-ironic17:31
*** jcoufal has quit IRC17:34
*** trown|lunch is now known as trown17:34
* sambetts --> home17:37
sambettsnight all17:37
*** sambetts is now known as sambetts|afk17:37
*** fragatin_ has quit IRC17:41
*** akhilaki has quit IRC17:51
*** fragatina has joined #openstack-ironic17:59
*** r-daneel has quit IRC18:06
*** r-daneel has joined #openstack-ironic18:06
TheJuliais anyone able to build docs locally?18:16
*** itlinux has joined #openstack-ironic18:18
jrollI can try18:19
jrollremember zuul doesn't use tox for that18:19
TheJuliayeah... I'm some errors like "mock" is not found now18:20
TheJuliawhich makes negative sense18:20
jrollO_o18:21
openstackgerritJulia Kreger proposed openstack/ironic master: Add note to oneview docs re: derprecation  https://review.openstack.org/56527418:23
TheJuliayeah, I just tossed my patch up after spending a half hour of scratching my head with the local job18:23
TheJuliawell, my local copy of it18:23
* jroll is running it now18:24
*** livelace has quit IRC18:26
jrollTheJulia: looks like mine built okay from master, though failed on a few warnings18:35
TheJuliahmm18:35
* jroll puts through tee quickly because tmux scrollback is being weird18:36
jrolloh I do see those errors while watching it18:37
*** cdearborn has joined #openstack-ironic18:37
jrollthough I don't think they're causing a failure18:37
jrollhttps://gist.github.com/jimrollenhagen/e83432f8c058422c671fe8467f8c680a18:39
jroll(for anyone following along)18:39
jrolloh, we don't install test-requirements.txt in that tox environment18:40
jrolland for whatever reason, autodoc is trying to doc test modules18:40
*** e0ne has joined #openstack-ironic18:42
*** itlinux has quit IRC19:07
openstackgerritMerged openstack/ironic-inspector master: Change the default discovery driver to fake-hardware  https://review.openstack.org/55046419:09
*** livelace has joined #openstack-ironic19:12
*** livelace has quit IRC19:18
*** hemna_ has quit IRC19:22
*** d0ugal has quit IRC19:45
*** d0ugal has joined #openstack-ironic19:46
*** fragatina has quit IRC19:51
*** d0ugal_ has joined #openstack-ironic20:00
*** d0ugal has quit IRC20:02
*** itlinux has joined #openstack-ironic20:06
*** itlinux has quit IRC20:17
*** hemna_ has joined #openstack-ironic20:18
*** d0ugal_ has quit IRC20:18
*** itlinux has joined #openstack-ironic20:19
*** d0ugal_ has joined #openstack-ironic20:19
*** fragatina has joined #openstack-ironic20:19
*** fragatina has quit IRC20:20
*** fragatina has joined #openstack-ironic20:22
*** d0ugal_ has quit IRC20:36
*** itlinux has quit IRC20:36
*** dprince has joined #openstack-ironic20:38
*** d0ugal_ has joined #openstack-ironic20:43
*** trown is now known as trown|outtypewww21:10
*** jcoufal_ has quit IRC21:11
*** dprince has quit IRC21:17
*** d0ugal_ has quit IRC21:32
*** d0ugal_ has joined #openstack-ironic21:35
*** d0ugal_ has quit IRC21:38
*** d0ugal_ has joined #openstack-ironic21:39
*** rbudden has quit IRC21:39
*** d0ugal_ has quit IRC21:57
*** d0ugal_ has joined #openstack-ironic21:59
*** cjloader has quit IRC22:03
*** e0ne has quit IRC22:03
*** rcernin has joined #openstack-ironic22:13
*** cdearborn has quit IRC22:16
*** rbudden has joined #openstack-ironic22:20
*** rh-jelabarre has joined #openstack-ironic22:51
*** lbragstad has quit IRC23:05
*** rajinir has quit IRC23:08
*** r-daneel has quit IRC23:09
*** d0ugal__ has joined #openstack-ironic23:11
*** d0ugal_ has quit IRC23:14
*** pmannidi has joined #openstack-ironic23:15
*** rbudden has quit IRC23:23
*** d0ugal__ has quit IRC23:29
*** jiapei has quit IRC23:32
*** hemna_ has quit IRC23:48

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