*** dtantsur|afk has quit IRC | 01:08 | |
*** dtantsur has joined #openstack-ironic | 01:08 | |
*** anton has quit IRC | 01:09 | |
*** anton has joined #openstack-ironic | 01:09 | |
*** krtaylor has quit IRC | 01:32 | |
*** akhilaki has joined #openstack-ironic | 01:54 | |
*** d0ugal has quit IRC | 02:49 | |
*** cjloader has joined #openstack-ironic | 02:49 | |
*** d0ugal has joined #openstack-ironic | 02:50 | |
*** namnh has joined #openstack-ironic | 02:55 | |
*** namnh has quit IRC | 03:02 | |
*** mvpnitesh has joined #openstack-ironic | 03:40 | |
*** akhilaki has quit IRC | 03:47 | |
*** Nisha_Agarwal has joined #openstack-ironic | 04:00 | |
*** namnh has joined #openstack-ironic | 04:36 | |
*** namnh has quit IRC | 04:36 | |
*** namnh has joined #openstack-ironic | 04:37 | |
*** links has joined #openstack-ironic | 04:39 | |
*** namnh has quit IRC | 04:41 | |
*** namnh has joined #openstack-ironic | 04:52 | |
*** namnh has quit IRC | 04:56 | |
*** rbartal has joined #openstack-ironic | 05:12 | |
*** jaganathan has joined #openstack-ironic | 05:23 | |
*** jaganathan has quit IRC | 05:23 | |
*** jaganathan has joined #openstack-ironic | 05:24 | |
*** namnh has joined #openstack-ironic | 05:33 | |
*** namnh has quit IRC | 05:38 | |
*** masber has joined #openstack-ironic | 05:41 | |
*** marios has joined #openstack-ironic | 05:46 | |
*** d0ugal has quit IRC | 05:48 | |
*** masuberu has joined #openstack-ironic | 05:51 | |
*** d0ugal has joined #openstack-ironic | 05:54 | |
*** masber has quit IRC | 05:55 | |
*** masuberu has quit IRC | 06: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 dependencies | 06:33 |
---|---|---|
patchbot | patch 563894 - bifrost - Add support for NTP servers in dnsmasq configuration | 06:33 |
patchbot | patch 564794 - requirements - update constraint for reno to new release 2.9.1 (MERGED) | 06:33 |
*** namnh has joined #openstack-ironic | 06:35 | |
*** namnh has quit IRC | 06:41 | |
*** namnh has joined #openstack-ironic | 06:57 | |
*** namnh has quit IRC | 07:02 | |
*** Nisha_Agarwal has quit IRC | 07:06 | |
*** Guest1988 has joined #openstack-ironic | 07:10 | |
*** Guest1988 has quit IRC | 07:17 | |
*** namnh has joined #openstack-ironic | 07:17 | |
*** namnh has quit IRC | 07:21 | |
*** Nisha_Agarwal has joined #openstack-ironic | 07:27 | |
olivierb- | however after relaunching the tests and looking more closely at the log, I can see | 07: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.1 | 07: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 please | 07:35 |
*** hoonetorg has quit IRC | 07:41 | |
*** lucas-afk is now known as lucasagomes | 07:54 | |
*** hoonetorg has joined #openstack-ironic | 07:55 | |
*** e0ne has joined #openstack-ironic | 07:57 | |
*** namnh has joined #openstack-ironic | 08:05 | |
*** Nisha_Agarwal has quit IRC | 08:08 | |
*** namnh has quit IRC | 08:09 | |
*** Nisha_Agarwal has joined #openstack-ironic | 08:10 | |
*** livelace has joined #openstack-ironic | 08:16 | |
*** cjloader has quit IRC | 08:23 | |
*** Nisha_Agarwal has quit IRC | 08:27 | |
*** derekh has joined #openstack-ironic | 08:29 | |
*** namnh has joined #openstack-ironic | 08:35 | |
*** namnh has quit IRC | 08:39 | |
*** nmathew has joined #openstack-ironic | 08:41 | |
*** pmannidi has quit IRC | 08:52 | |
*** namnh has joined #openstack-ironic | 08:56 | |
*** rbartal has quit IRC | 08:58 | |
vdrok | olivierb-: 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 master | 08:59 |
patchbot | patch 564763 - reno - include the branch name in anchors to make them mo... (MERGED) | 08:59 |
*** namnh has quit IRC | 09:01 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: WIP: Add Node BIOS support - REST API https://review.openstack.org/512579 | 09:08 |
*** namnh has joined #openstack-ironic | 09:10 | |
*** namnh has quit IRC | 09:15 | |
*** racedo has joined #openstack-ironic | 09:18 | |
*** MattMan has quit IRC | 09:18 | |
*** MattMan has joined #openstack-ironic | 09:18 | |
*** namnh has joined #openstack-ironic | 09:25 | |
*** nmathew has quit IRC | 09:29 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Switch to the fake-hardware hardware type for API tests https://review.openstack.org/564519 | 09:36 |
*** yolanda_ is now known as yolanda | 09:39 | |
yolanda | hi 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 |
patchbot | patch 512579 - ironic - WIP: Add Node BIOS support - REST API | 09:40 |
sambetts | olivierb-: I've worked out the issue behind that warning, /me will push a fix | 09:56 |
openstackgerrit | Sam Betts proposed openstack/bifrost master: Prevent reno rendering current release notes twice https://review.openstack.org/565180 | 10:03 |
sambetts | olivierb-: ^^^ | 10:03 |
*** livelace-link has quit IRC | 10:09 | |
*** livelace-link has joined #openstack-ironic | 10:12 | |
openstackgerrit | Merged openstack/ironic master: Remove the Keystone API V2.0 endpoint registration https://review.openstack.org/564999 | 10:26 |
olivierb- | sambetts many thanks | 10:30 |
*** namnh has quit IRC | 10:44 | |
*** jcoufal has joined #openstack-ironic | 11:00 | |
*** lucasagomes is now known as lucas-hungry | 11:04 | |
*** namnh has joined #openstack-ironic | 11:05 | |
*** namnh has quit IRC | 11:09 | |
dtantsur | morning/afternoon ironic | 11:15 |
dtantsur | yolanda: hi, these are going to be clean steps IIRC | 11:16 |
dtantsur | so they should just be marked as such | 11:17 |
*** dprince has joined #openstack-ironic | 11:18 | |
yolanda | dtantsur, is there some code i can take as example on how to set up clean steps from rest api? | 11:20 |
dtantsur | yolanda: no explicit set up needed | 11:21 |
*** rbartal has joined #openstack-ironic | 11:21 | |
yolanda | dtantsur, i'm talking about http://git.openstack.org/cgit/openstack/ironic-specs/tree/specs/approved/generic-bios-config.rst#n131 | 11:24 |
yolanda | see 2 api calls: factory_reset and apply_configuration | 11:24 |
dtantsur | s/api calls/clean steps/, yes | 11:24 |
yolanda | no need to expose those on rest api? | 11:25 |
dtantsur | yolanda: yep, clean steps are exposed implicitly | 11:25 |
yolanda | ah ok, i was confused about that, thinking that i had to expose on the bios api | 11:25 |
*** mvpnitesh has quit IRC | 11:25 | |
yolanda | thx | 11:26 |
*** namnh has joined #openstack-ironic | 11:26 | |
*** namnh has quit IRC | 11:30 | |
*** rbartal has quit IRC | 11:32 | |
*** parlos has joined #openstack-ironic | 11:33 | |
openstackgerrit | Harald Jensås proposed openstack/ironic-inspector master: Call shutdown() on SIGTERM - controlled teardown https://review.openstack.org/563335 | 11:33 |
*** rbudden has joined #openstack-ironic | 11:36 | |
dtantsur | yolanda: it's confusing indeed... we may eventually end up with Clean Steps v2 to clean up the mess :) | 11:38 |
*** namnh has joined #openstack-ironic | 11:42 | |
*** parlos_ has joined #openstack-ironic | 11:43 | |
*** parlos has quit IRC | 11:44 | |
*** rbartal has joined #openstack-ironic | 11:47 | |
*** parlos has joined #openstack-ironic | 11:55 | |
*** parlos_ has quit IRC | 11:55 | |
*** lucas-hungry is now known as lucasagomes | 11:57 | |
*** parlos_ has joined #openstack-ironic | 11:57 | |
*** parlos has quit IRC | 11:59 | |
*** parlos has joined #openstack-ironic | 12:01 | |
*** parlos_ has quit IRC | 12:03 | |
jroll | morning | 12:15 |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add Node BIOS support - REST API https://review.openstack.org/512579 | 12:16 |
dtantsur | morning jroll | 12:17 |
yolanda | dtantsur, i pushed the rest api change. It will need a lot of review | 12:17 |
dtantsur | sure :) | 12:17 |
*** trown|outtypewww is now known as trown | 12:18 | |
*** namnh has quit IRC | 12:22 | |
*** kzaitsev_pi has quit IRC | 12:23 | |
*** zshi has quit IRC | 12:23 | |
*** namnh has joined #openstack-ironic | 12:24 | |
*** zshi has joined #openstack-ironic | 12:25 | |
jroll | heh, just noticed the base driver supported attribute is false, so the classic oneview drivers were never marked supported | 12:26 |
sambetts | :/ | 12:26 |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Deprecate Oneview https://review.openstack.org/564587 | 12:28 |
jroll | ^ that marks the interfaces unsupported as well, should be ready to land now | 12:28 |
*** rkharya has joined #openstack-ironic | 12:28 | |
*** namnh has quit IRC | 12:28 | |
*** kzaitsev_pi has joined #openstack-ironic | 12:30 | |
dtantsur | jroll: well, right, the classic drivers are deprecated, all of them | 12:31 |
dtantsur | I'm not sure we have "supported" notion for interfaces btw | 12:31 |
jroll | dtantsur: ah, that's true. and we do have it: https://github.com/openstack/ironic/blob/master/ironic/drivers/base.py#L52 | 12:31 |
jroll | er | 12:32 |
jroll | https://github.com/openstack/ironic/blob/master/ironic/drivers/base.py#L200 | 12:32 |
dtantsur | jroll: TIL :D | 12:35 |
*** rkharya has quit IRC | 12:38 | |
*** kzaitsev_pi has quit IRC | 12:39 | |
jroll | ha | 12:39 |
*** kzaitsev_pi has joined #openstack-ironic | 12:40 | |
* dtantsur dreams of stopping working on the driver composition one day | 12:40 | |
TheJulia | good morning everyone | 12:49 |
dtantsur | morning TheJulia | 12:50 |
sambetts | o/ | 12:50 |
dtantsur | hmm, https://review.openstack.org/#/c/536928/ merged on Feb 1st, meaning that I can remove the classic drivers tomorrow :D | 12:50 |
patchbot | patch 536928 - ironic - Deprecate classic drivers (MERGED) | 12:50 |
dtantsur | TheJulia: should I send The Very Last Warning to the ML re ^^^? | 12:51 |
jroll | ++ | 12:52 |
dtantsur | we need https://review.openstack.org/564519 before we can switch though. but the drivers will hopefully be gone by Rocky-2 | 12:54 |
patchbot | patch 564519 - ironic - Switch to the fake-hardware hardware type for API ... | 12:54 |
*** rloo has joined #openstack-ironic | 12:56 | |
* dtantsur done | 12:58 | |
* jroll will take a look | 12:58 | |
rloo | Happy. Monday morning ironic'ers, jroll, dtantsur | 13:00 |
dtantsur | hey rloo | 13:01 |
jroll | \o | 13:01 |
dtantsur | jroll: right now I need some opinion on https://review.openstack.org/#/c/564560/ to move forward | 13:02 |
patchbot | patch 564560 - ironic - Move API (functional) tests to separate jobs | 13:02 |
jroll | dtantsur: ah, ok, on my list now | 13:02 |
rloo | dtantsur, 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 |
patchbot | patch 562314 - ironic (stable/queens) - Remove vifs upon teardown | 13:09 |
dtantsur | I'm +1 to asking ops | 13:13 |
* dtantsur does not have time to write it though | 13:13 | |
TheJulia | dtantsur: Yeah, might as well send one last warning to the mailing list | 13:20 |
TheJulia | rloo: 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 IRC | 13:22 | |
rloo | TheJulia: 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 |
rloo | dtantsur: wrt that backport patch, you could approve it, it doesn't require me. I am just uncomfortable as-is. | 13:23 |
TheJulia | rloo: Okay, I can send that email later today | 13:24 |
rloo | thx TheJulia! | 13:25 |
*** superdan is now known as dansmith | 13:29 | |
*** jroll has quit IRC | 13:36 | |
dtantsur | rloo: I'll wait for you to become comfortable :) | 13:36 |
rloo | dtantsur: okay :) | 13:36 |
rloo | dtantsur: btw, wrt https://review.openstack.org/#/c/564517/, is it tested that fake-hardware works? | 13:36 |
patchbot | patch 564517 - ironic-tempest-plugin - Allow using the fake-hardware hardware type for AP... | 13:36 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Do not run functional (API) tests in the CI https://review.openstack.org/565222 | 13:37 |
dtantsur | rloo: I have a patch for that, but I'm holding it for now: https://review.openstack.org/564519 | 13:37 |
patchbot | patch 564519 - ironic - Switch to the fake-hardware hardware type for API ... | 13:37 |
*** lbragstad has joined #openstack-ironic | 13:37 | |
*** rh-jelabarre has quit IRC | 13:38 | |
*** jroll has joined #openstack-ironic | 13:38 | |
rloo | dtantsur: ah, looks good then, thx. | 13:38 |
openstackgerrit | Merged openstack/bifrost master: Prevent reno rendering current release notes twice https://review.openstack.org/565180 | 13:41 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Do not run functional (API) tests in the CI https://review.openstack.org/565223 | 13:41 |
*** dprince has quit IRC | 13:42 | |
* TheJulia makes coffee | 13:45 | |
*** rh-jelabarre has joined #openstack-ironic | 13:46 | |
openstackgerrit | yolanda.robla proposed openstack/bifrost master: Add support for NTP servers in dnsmasq configuration https://review.openstack.org/563894 | 13:47 |
*** r-daneel has joined #openstack-ironic | 13:47 | |
*** livelace has quit IRC | 14:02 | |
rloo | dtantsur: 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 |
patchbot | patch 555610 - ironic - Implementation of inspect wait state (MERGED) | 14:03 |
dtantsur | rloo: well, the way inspect interface is implemented now, it's not needed, because it's either sync or async, not mixed | 14:04 |
dtantsur | we can update the state machine if we ever need it, right? | 14:04 |
rloo | dtantsur: 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-ironic | 14:05 | |
dtantsur | yeah, I'm fine either way | 14:06 |
rloo | dtantsur: ok, i'll just comment in the patch. thx :) | 14:06 |
*** namnh has joined #openstack-ironic | 14:09 | |
dtantsur | rloo: I only care about this change at all because of http://specs.openstack.org/openstack/ironic-specs/specs/not-implemented/inspection-abort.html | 14:10 |
rloo | dtantsur: ah, yes. would be good to have that. | 14:10 |
*** namnh has quit IRC | 14:13 | |
rloo | TheJulia: 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 |
patchbot | patch 557755 - ironic-python-agent - Pass empty string to hdparm --security-unlock in G... | 14:14 |
*** ElCoyote_ has joined #openstack-ironic | 14:22 | |
*** rbartal has quit IRC | 14:24 | |
*** e0ne has quit IRC | 14:26 | |
*** rajinir has joined #openstack-ironic | 14:28 | |
rloo | dtantsur: 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 |
patchbot | patch 550464 - ironic-inspector - Change the default discovery driver to fake-hardware | 14:29 |
* dtantsur looks | 14:30 | |
dtantsur | rloo: yep, good suggestions | 14:30 |
rloo | dtantsur: ok, will update/approve in a few min. | 14:30 |
dtantsur | thanks! | 14:30 |
*** hemna_ has joined #openstack-ironic | 14:31 | |
*** baha has joined #openstack-ironic | 14:35 | |
openstackgerrit | Ruby Loo proposed openstack/ironic-inspector master: Change the default discovery driver to fake-hardware https://review.openstack.org/550464 | 14:36 |
TheJulia | okay, yay , things like vpn work agian | 14:45 |
dtantsur | heh | 14:47 |
rloo | TheJulia: good! :) | 14:47 |
dtantsur | TheJulia: are we ready to land https://review.openstack.org/#/c/564587/ ? | 14:47 |
TheJulia | rloo: 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 needlessly | 14:47 |
rloo | 13 minutes til meeting, update whiteboard please! | 14:47 |
patchbot | patch 564587 - ironic - Deprecate Oneview | 14:47 |
rloo | TheJulia: so you're good with the code as-is? (except the unit test)? | 14:48 |
rloo | TheJulia: i'm not actually sure I understand why NULL worked before... but clearly it did. for some users anyway... | 14:49 |
TheJulia | yeah | 14:49 |
TheJulia | maybe jroll mgiht have memories in regards to wiping disks? | 14:49 |
TheJulia | just for sanity's sake | 14:49 |
*** parlos has quit IRC | 14:49 | |
TheJulia | rloo: w/r/t oneview, I think were good to land the patch, I can follow-up with doc updates later | 14:49 |
dtantsur | http://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-ironic | 14:50 | |
rloo | TheJulia: wrt oneview, ok. i think i was waiting for CI but can approve. | 14:50 |
* dtantsur pressed the button already | 14:50 | |
*** namnh has joined #openstack-ironic | 14:50 | |
rloo | dtantsur: those tests pass w/o your change? | 14:52 |
*** jiapei_ has joined #openstack-ironic | 14:53 | |
rloo | TheJulia: the bifrost/pip10 thing was fixed last week, right? (still in whiteboard) | 14:53 |
TheJulia | rloo: yeah | 14:53 |
*** cjloader has joined #openstack-ironic | 14:54 | |
jroll | TheJulia: I don't think I've ever run that code path, I seem to remember that being post-onmetal-working | 14:54 |
TheJulia | hmmm | 14:54 |
TheJulia | must have been jayf then | 14:55 |
*** arxcruz|pto has joined #openstack-ironic | 14:55 | |
*** namnh has quit IRC | 14:55 | |
dtantsur | rloo: dunno, haven't seen the IPA CI in a while | 14:55 |
jroll | well, he worked on the same thing | 14:55 |
jroll | :P | 14:55 |
dtantsur | but I only change the tempest regex.... | 14:55 |
JayF | TheJulia: it wasn't me </shaggy> What is it, exactly, that I didn't do though? :P | 14:56 |
jroll | TheJulia: maybe we did, dunno. would never be able to remember the specific arguments we passed :) | 14:56 |
jroll | JayF: https://review.openstack.org/#/c/557755/4/ironic_python_agent/hardware.py | 14:56 |
patchbot | patch 557755 - ironic-python-agent - Pass empty string to hdparm --security-unlock in G... | 14:56 |
rloo | dtantsur: sigh. although i just reviewed an IPA patch https://review.openstack.org/#/c/557755/. Oh, apr 24 was last date. | 14:57 |
patchbot | patch 557755 - ironic-python-agent - Pass empty string to hdparm --security-unlock in G... | 14:57 |
JayF | we absolutely literally had to pass in 'NULL' | 14:57 |
JayF | for those machines | 14:57 |
dtantsur | hmm, should we try both then? | 14:57 |
* rloo is amazed at JayF memory | 14:57 | |
rloo | +1 to trying both then. | 14:57 |
JayF | but the firmwares on the SATADOMs we used in that generation were absolutely godawful | 14:57 |
JayF | rloo: when you spend multiple days trying to figure out the magic words, it's easy to remember them | 14:58 |
TheJulia | I'm also +1 to trying both in that case. | 14:58 |
JayF | jroll: figuring out ^ that magic was the first thing clif did (with me) on the team | 14:58 |
jroll | ah | 14:58 |
jroll | thanks JayF :) | 14:58 |
JayF | jroll: $20 says it was a language-barrier problem with us requesting a "null password" from the group that made those | 14:59 |
jroll | heh | 14:59 |
jroll | could be | 14:59 |
JayF | rloo: TheJulia: That's the flavor for that, BTW ^ -- that security unlock string is a /password/ you have to know to unlock the disk | 14:59 |
TheJulia | #startmeeting ironic | 15:00 |
openstack | Meeting 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 |
JayF | so it doesn't surprise me emptystring works on a lot of machines | 15:00 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
*** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
openstack | The meeting name has been set to 'ironic' | 15:00 |
TheJulia | JayF: I knew that, but I thought there was some magic code in hdparm for certian cases | 15:00 |
TheJulia | anyway! | 15:00 |
rloo | thx JayF (want to join our meeting?) | 15:00 |
TheJulia | Meeting time! | 15:00 |
rloo | o/ | 15:00 |
arxcruz|pto | o/ | 15:00 |
*** beekneemech is now known as bnemec | 15:00 | |
* dtantsur lurks | 15:00 | |
* TheJulia hands out cokies | 15:00 | |
TheJulia | cokies | 15:00 |
jiapei_ | o/ | 15:00 |
vdrok | щ. | 15:00 |
vdrok | woops, o/ | 15:00 |
jroll | \o | 15:00 |
TheJulia | okay, o key... why you no like cookies :( | 15:00 |
* jroll runs to grab water | 15:01 | |
TheJulia | A great comedic start to the morning! | 15:01 |
dtantsur | vdrok: щ to you as well :D | 15:01 |
JayF | o/ (the goodbye kind!) | 15:01 |
TheJulia | Our agenda can be found at https://wiki.openstack.org/wiki/Meetings/Ironic | 15:01 |
dtantsur | hey hey JayF :) | 15:01 |
TheJulia | o/ JayF | 15:01 |
vdrok | :) | 15:01 |
jiapei_ | Good evening :) | 15:01 |
TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic | 15:01 |
TheJulia | Anyway, getting started! | 15:01 |
TheJulia | #topic Announcements/Reminders | 15:02 |
*** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)" | 15:02 | |
TheJulia | We are a few weeks out from the Summit, I think we should cancel the meeting, but we can vote on that next week | 15:03 |
TheJulia | aside from that, I have nothing else to announce or remind us of this week. | 15:03 |
rloo | TheJulia: cancel the meeting the week of the summit, right? | 15:03 |
*** persia has quit IRC | 15:03 | |
TheJulia | rloo: yes | 15:04 |
rloo | TheJulia: +1 to cancel :) | 15:04 |
*** namnh has joined #openstack-ironic | 15:04 | |
*** jaganathan has quit IRC | 15:04 | |
TheJulia | Does anyone else have anything to annouce? | 15:04 |
TheJulia | I guess not, and we can move on | 15:05 |
TheJulia | #topic Reviewing action items from last week | 15: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.html | 15:07 |
TheJulia | #link https://storyboard.openstack.org/#!/story/2001884 | 15:07 |
TheJulia | It looks like dmitry posted one patch. dtantsur will there be a second patch? | 15:07 |
dtantsur | yep, but it feels less urgent | 15:08 |
TheJulia | Okay, thanks! | 15:08 |
TheJulia | #topic Review Subteam Status Reports | 15:09 |
*** openstack changes topic to "Review Subteam Status Reports (Meeting topic: ironic)" | 15:09 | |
openstackgerrit | Merged openstack/bifrost master: Add support for NTP servers in dnsmasq configuration https://review.openstack.org/563894 | 15:09 |
TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:09 |
*** rh-jelabarre has quit IRC | 15:10 | |
TheJulia | Looks like we at least landed one of the bios interface patches last week | 15:11 |
rloo | SIGHUP work is done too :) | 15:12 |
dtantsur | \o/ | 15:12 |
jroll | nice! | 15:12 |
TheJulia | dtantsur: any remaining hardware type work on the horizon regarding the patches to delete classic drivers? | 15:12 |
TheJulia | rloo: by chance has that been updated in storyboard yet? | 15:13 |
dtantsur | yep | 15:13 |
dtantsur | https://review.openstack.org/#/q/topic:api-jobs is needed to move forward with s/fake/fake-hardware/ in the CI | 15:13 |
rloo | TheJulia: i don't know; it is on my list to check/do after this meeting :) | 15:13 |
TheJulia | dtantsur: any this week that we need to have the ironic community attention on? | 15:13 |
TheJulia | rloo: ack, let me know if you need me to do anything then | 15:13 |
dtantsur | these patches, then https://review.openstack.org/564519 | 15:13 |
patchbot | patch 564519 - ironic - Switch to the fake-hardware hardware type for API ... | 15:13 |
rloo | TheJulia: will do, thx! | 15:13 |
TheJulia | dtantsur: any expectation on unblocking the last patch this week? | 15:15 |
dtantsur | TheJulia: it's blocked by that three patches, essentially | 15:15 |
dtantsur | then I'll double-check that I don't break any CI, and unblock it | 15:15 |
dtantsur | so yes, totally possible | 15:15 |
TheJulia | okay, then I'll just leave the link to the three patches for now | 15:16 |
dtantsur | k | 15:16 |
TheJulia | I 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 issues | 15:17 |
* TheJulia sighs that the cats have realized how to open the window blinds.... | 15:18 | |
TheJulia | Looking through things on the whiteboard, I think we should go ahead and remove oneview references, thoughts? | 15:19 |
jroll | +1 | 15:19 |
rloo | TheJulia: +1 (i was going to ask, thx) | 15:19 |
TheJulia | #action TheJulia to scrub the whiteboard for oneview items this week | 15:20 |
TheJulia | I'll go ahead and put a doc note up as well | 15:20 |
rloo | TheJulia: might be useful to mention here, why we're doing this | 15:20 |
TheJulia | #action TheJulia to put up deprecation doc note in the oneview docs. | 15:20 |
TheJulia | I'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 |
TheJulia | I'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 |
TheJulia | Since 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 |
TheJulia | Any questions? | 15:23 |
*** dprince has joined #openstack-ironic | 15:24 | |
* TheJulia hears crickets as a service | 15:24 | |
TheJulia | so with that, lets move to priorites for the week, then discusison. | 15:24 |
TheJulia | Everyone good with that? | 15:25 |
dtantsur | yep | 15:25 |
TheJulia | #topic Priorities for the coming week | 15:25 |
*** openstack changes topic to "Priorities for the coming week (Meeting topic: ironic)" | 15:25 | |
TheJulia | Starting around line 102 | 15:26 |
TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:26 |
dtantsur | LGTM :) | 15:26 |
TheJulia | not much different from last week. I removed the merged patches, added a link to make dtantsur's life easier | 15:27 |
jroll | ditto | 15:27 |
* dtantsur likes making dtantsur's life easier :D | 15:27 | |
TheJulia | heh | 15:27 |
TheJulia | Anyway, moving on# | 15:28 |
TheJulia | err | 15:28 |
TheJulia | ! | 15:28 |
TheJulia | #topic Discussion | 15:28 |
vdrok | oh btw all the prerequisites for https://review.openstack.org/359061 are merged | 15:28 |
*** openstack changes topic to "Discussion (Meeting topic: ironic)" | 15:28 | |
patchbot | patch 359061 - python-ironicclient - Switch to none auth for standalone mode | 15:28 |
dtantsur | vdrok: oh, quite a long read :) I'll try to get to it | 15:28 |
arxcruz|pto | Hi guys, I'm from tripleo-ci team, and we would like to add a tripleo job on ironic project | 15:28 |
vdrok | thx! | 15:28 |
rloo | vdrok: thx! (I need to get a dev env set up to test that...) | 15:29 |
arxcruz|pto | initially the job will take 2 hours to finish, installing tripleo and running tempest on undercloud | 15:29 |
arxcruz|pto | non voting obviously, but hopefully with time, get it in voting | 15:29 |
arxcruz|pto | we are working hard to decrease the time of the job as well | 15:29 |
TheJulia | arxcruz|pto: why is it that this job is required? | 15:29 |
vdrok | rloo: there also is a patch on top running functional tests with it :) https://review.openstack.org/508330 | 15:29 |
patchbot | patch 508330 - python-ironicclient - WIP allow to use none auth in functional tests | 15:29 |
arxcruz|pto | TheJulia: not required, but would help us to catch problems in tripleo faster since tripleo uses ironic a lot | 15:30 |
rloo | vdrok: thx. will try to look soon, it has been on my list for awhile... | 15:30 |
dtantsur | TheJulia: one benefit for us is testing local boot | 15:30 |
arxcruz|pto | sometimes you don't see the problem running on devstack | 15:30 |
*** cjloader has quit IRC | 15:30 | |
TheJulia | We 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 more | 15:30 |
rloo | 2 hours is a long time. what is the longest time for our existing tests? | 15:30 |
rloo | (existing voting tests) | 15:31 |
dtantsur | rloo: I suspect grenade jobs are comparable | 15:31 |
TheJulia | rloo: multinode grenade is like 1 hour 45 minutes on a good day | 15:31 |
jroll | dtantsur: don't we test local boot already? | 15:31 |
*** cjloader has joined #openstack-ironic | 15:31 | |
dtantsur | jroll: IIRC no :( | 15:31 |
arxcruz|pto | as i said, we are working to reduce to 1.5 hours | 15:31 |
dtantsur | we were blocked by cirros not supporting grub | 15:31 |
TheJulia | rloo: 2hr 20-minute-ish on a moderate day | 15:31 |
jroll | wut | 15:31 |
jroll | :( | 15:31 |
arxcruz|pto | it will install the undercloud and run ironic tempest tests | 15:31 |
TheJulia | arxcruz|pto: is that even really required for a test executing reporting to ironic? | 15:32 |
*** rloo has quit IRC | 15:32 | |
TheJulia | since all of our other tests run tempest based quites | 15:32 |
TheJulia | squites | 15:32 |
TheJulia | suites | 15:32 |
* TheJulia should just give up on typing today | 15:32 | |
arxcruz|pto | TheJulia: yeah, but, not in tripleo environment | 15:32 |
dtantsur | TheJulia: typing is overrated | 15:32 |
*** rloo has joined #openstack-ironic | 15:32 | |
TheJulia | dtantsur: indeed | 15:32 |
dtantsur | arxcruz|pto: I think the question is how it benefits ironic, not tripleo :) | 15:33 |
arxcruz|pto | we see several problems that pass without nobody see on devstack, when we change the environment to tripleo, things start to pop up everywhere | 15:33 |
arxcruz|pto | dtantsur: less bugs to fix later :D | 15:33 |
dtantsur | I like being tested by something resembling production, to be honest | 15:33 |
jroll | to be clear, is this tripleo job in the infra queue or some third party ci queue? | 15:33 |
TheJulia | arxcruz|pto: Has Ironic had recent changes that have broken TripleO? | 15:33 |
arxcruz|pto | TheJulia: from time to time, I don't recally anything right now, we usually try to fix in our side | 15:34 |
arxcruz|pto | but we want to increase the coverage | 15:34 |
arxcruz|pto | jroll: infra queue | 15:34 |
jroll | ok | 15:34 |
dtantsur | arxcruz|pto: but there are also RDO 3rdparty jobs, right? | 15:34 |
arxcruz|pto | dtantsur: yes | 15:34 |
dtantsur | adding these could be easier, no? | 15:34 |
dtantsur | I mean, they don't block the gate if they take too long | 15:35 |
arxcruz|pto | well, I'm opened to options :) | 15:35 |
jroll | I 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 too | 15:35 |
TheJulia | I really like the idea of a third party job personally | 15:35 |
arxcruz|pto | if you guys agree, we can add a job, non voting, if it's not working, we can move on with other options | 15:35 |
*** cjloader has quit IRC | 15:35 | |
dtantsur | arxcruz|pto: we can even create a featureset using ironic both in under and over cloud | 15:35 |
dtantsur | double ironic testing \o/ :D | 15:35 |
arxcruz|pto | dtantsur: let's start simple :D lol | 15:35 |
TheJulia | arxcruz|pto: move on with other jobs? sounds like there is more to this plan? | 15:35 |
arxcruz|pto | TheJulia: sorry, what I meant was move on with other options, like move the job to thirdy party | 15:36 |
arxcruz|pto | third* | 15:36 |
rloo | my 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 |
dtantsur | arxcruz|pto: it is OVB, right? | 15:36 |
rloo | arxcruz|pto: would it make sense to put it in RDO 3rdparty jobs first? | 15:37 |
arxcruz|pto | dtantsur: nope, multinode | 15:37 |
dtantsur | arxcruz|pto: ehmm, wait please | 15:37 |
arxcruz|pto | rloo: i can check if that's possible | 15:37 |
dtantsur | last time I checked tripleo multinode CI did not deploy ironic instances | 15:37 |
arxcruz|pto | weshay: can it be ovb on rdocloud ? | 15:37 |
arxcruz|pto | dtantsur: we are working on a featureset for that | 15:37 |
dtantsur | how do you run tempest if you don't have virtual ironic nodes? | 15:37 |
dtantsur | ah, okay | 15:37 |
jroll | rloo: infra generally does not worry about resources if the job has value | 15:37 |
dtantsur | my concern is only about the possibility :) if you can use virtualbmc/whatever on a multinode tripleo job - fine | 15:38 |
arxcruz|pto | dtantsur: it will be a whole new featureset | 15:38 |
weshay | arxcruz|pto, we can add jobs to rdocloud especially for ironic | 15:38 |
weshay | ya | 15:38 |
rloo | jroll: oh, ok. | 15:38 |
weshay | that is only 3rd party though | 15:38 |
arxcruz|pto | so, my boss said yes, so up to you guys :D | 15:38 |
arxcruz|pto | dtantsur: we can work on a job that run ironic in a way that benefit both teams :) | 15:39 |
TheJulia | So 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 |
weshay | I say especially for ironic as ovb lends itself better to ironic testing | 15:40 |
dtantsur | TheJulia: precisely | 15:40 |
jroll | TheJulia: +1 | 15: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|pto | so, ovb job, on third party, non voting for now, running ironic and testing ironic tempest plugins sounds good to everyone ? | 15:41 |
rloo | arxcruz|pto: i think that's a good start. | 15:41 |
dtantsur | ++ great plan | 15:41 |
arxcruz|pto | weshay: ^ | 15:41 |
TheJulia | Sounds good, voting would need to be another discussion, but we can have that later once we have statistics | 15:41 |
weshay | +1 | 15:42 |
weshay | who owns the ai? | 15:42 |
jroll | ++ | 15:42 |
TheJulia | ai? | 15:42 |
jroll | ai? | 15:42 |
dtantsur | ai? | 15:42 |
weshay | lolz | 15:42 |
jroll | :P | 15:42 |
weshay | action item | 15:42 |
weshay | sorry | 15:42 |
weshay | pointy head here | 15:42 |
dtantsur | arxcruz|pto of course :D | 15:42 |
rloo | arxcruz|pto i think | 15:42 |
jroll | yep | 15:42 |
arxcruz|pto | TheJulia: sure, just to have it running on ironic, will help us to catch problems earlier | 15:42 |
arxcruz|pto | yes, me | 15:42 |
arxcruz|pto | I'll work this week on it | 15:43 |
dtantsur | cool, thanks arxcruz|pto | 15:43 |
arxcruz|pto | dtantsur: thanks too :) | 15:43 |
dtantsur | (and why aren't you on pto??) | 15:43 |
jroll | for a moment I had hopes we had built artificial intelligence to review changes | 15:43 |
TheJulia | #action arxcruz|pto to begin working on tripleo ci jobs using ironic reporting to ironic. | 15:43 |
dtantsur | jroll: it's me, no? | 15:43 |
arxcruz|pto | dtantsur: I forgot this meeting :) | 15:43 |
vdrok | need 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 |
patchbot | patch 517921 - nova - ironic: Report resources as reserved when needed | 15:43 |
arxcruz|pto | i am supposed to be on pto | 15:43 |
jroll | dtantsur: lol, true. we need moar AI though! | 15:43 |
rloo | ciao vdrok | 15:44 |
TheJulia | code review AI as long as it +2's even with a hundred nitpicks. | 15:44 |
*** marios has quit IRC | 15:44 | |
jroll | vdrok: thanks! | 15:45 |
TheJulia | So our other discussion item was regarding the backport of https://review.openstack.org/#/c/562314/ | 15:45 |
patchbot | patch 562314 - ironic (stable/queens) - Remove vifs upon teardown | 15:45 |
dtantsur | arxcruz|pto: go then :) brewdog is calling! | 15:45 |
TheJulia | That 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 visibility | 15:45 |
rloo | Thank you TheJulia! | 15:46 |
arxcruz|pto | dtantsur: been there too many times this month already, edeka had a promotion of belgian beers ;) | 15:46 |
TheJulia | Moving to open discussion if there are no objections | 15:46 |
dtantsur | arxcruz|pto: oh :) | 15:46 |
rloo | There 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 |
jroll | so with that backport - what use case depends on the VIFs remaining attached? non-nova I guess? | 15:47 |
*** akhilaki has joined #openstack-ironic | 15:47 | |
dtantsur | jroll: I cannot imagine, but it's an API behavior change | 15:47 |
TheJulia | jroll: manual ironic + neutron outside of nova | 15:47 |
dtantsur | so, in theory, we're breaking people | 15:47 |
jroll | TheJulia: yeah, okay | 15:48 |
jiapei_ | +1 | 15:48 |
jroll | this is very much a bug for nova users, used to hit similar all the time at rackspace | 15:48 |
jroll | leads to undeployable nodes | 15:48 |
TheJulia | Yup | 15:49 |
jroll | I'm okay with backporting it, with an email to the ops list | 15:49 |
TheJulia | If anyone wonders wehre my brain has been for the last two weeks, its been hunting bugs along these lines | 15:49 |
dtantsur | jroll: my position as well | 15:49 |
TheJulia | and the intertwined chaos they create | 15:49 |
TheJulia | #topic Open Discussion | 15:50 |
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:50 | |
TheJulia | So what else do we have to discuss? | 15:50 |
jroll | I would suggest adding a config flag, but I don't believe people will see that and not see the email/release notes | 15:50 |
jiapei_ | TheJulia: I uploaded a new patch https://review.openstack.org/#/c/561405/ | 15:50 |
patchbot | patch 561405 - ironic - Fix XClarity parameters discrepancy | 15:50 |
jiapei_ | Could you have a review of it? :) | 15:50 |
jroll | so not worth it | 15:50 |
TheJulia | With 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 |
rloo | thanks TheJulia, for dealing with storyboard | 15:51 |
rloo | TheJulia: that reminds me, bug bash on Thursday | 15:51 |
TheJulia | jroll: 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 easier | 15:51 |
dtantsur | yep, detaching VIFs explicitly may be annoying | 15:52 |
jroll | TheJulia: makes sense | 15:52 |
TheJulia | #info Bug Bash Thursday! | 15:53 |
TheJulia | rloo: have call details been identified? | 15:53 |
*** srart has quit IRC | 15:53 | |
rloo | TheJulia: i looked earlier today and i think mike wants you to do a hangout. sec... | 15:54 |
jroll | TheJulia: last email was asking if we can use your bluejeans, so no :) | 15:54 |
*** srart has joined #openstack-ironic | 15:54 | |
TheJulia | :( | 15:54 |
* dtantsur can also provide bluejeans, if needed | 15:54 | |
rloo | yup. is that ok TheJulia? | 15:54 |
rloo | we only need one pair of bluejeans :) | 15:54 |
TheJulia | What was time time? | 15:54 |
jroll | 9am eastern | 15:54 |
jroll | probably best to use dtantsur's | 15:55 |
rloo | 1-3pm UTC | 15:55 |
openstackgerrit | Merged openstack/ironic stable/queens: Collect periodic tasks from all enabled hardware interfaces https://review.openstack.org/564201 | 15:55 |
dtantsur | so, using mine? | 15:55 |
TheJulia | it looks like I have a call right before | 15:56 |
TheJulia | so either of us can do it I think | 15:56 |
* dtantsur waits for the PTL's decision :) | 15:56 | |
TheJulia | dtantsur: 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 long | 15:56 |
dtantsur | TheJulia: okay, I'll do it | 15:57 |
TheJulia | thanks! | 15:57 |
*** baha has quit IRC | 15:57 | |
TheJulia | Anyway, I think we are out of things to discuss today | 15:57 |
TheJulia | And we have two minutes left! | 15:57 |
TheJulia | Anything else for this meeting? | 15:58 |
rloo | crickets | 15:58 |
TheJulia | Thanks everyone! | 15:58 |
jroll | TheJulia: thanks for hosting :) | 15:59 |
TheJulia | #endmeeting | 15: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 | |
openstack | Meeting ended Mon Apr 30 15:59:26 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:59 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-30-15.00.html | 15:59 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-30-15.00.txt | 15:59 |
openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-30-15.00.log.html | 15:59 |
*** cjloader has joined #openstack-ironic | 15:59 | |
dtantsur | TheJulia: https://bluejeans.com/309964257 will be the link | 16:01 |
TheJulia | dtantsur: awesome | 16:01 |
dtantsur | now I have to go, see you on Wednesday | 16:02 |
* jroll finds lunch | 16:02 | |
rloo | TheJulia: 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/2001545 | 16:02 |
*** cjloader has quit IRC | 16:02 | |
*** dtantsur is now known as dtantsur|afk | 16:02 | |
dtantsur|afk | o/ | 16:02 |
*** cjloader has joined #openstack-ironic | 16:02 | |
rloo | bye dtantsur|afk | 16:02 |
*** namnh has quit IRC | 16:04 | |
TheJulia | Oh right, may day | 16:05 |
*** namnh has joined #openstack-ironic | 16:06 | |
*** namnh_ has joined #openstack-ironic | 16:07 | |
*** lucasagomes is now known as lucas-hungry | 16:08 | |
*** ijw has joined #openstack-ironic | 16:09 | |
rloo | the whiteboard indicates that some releasenotes jobs are/were broken by new reno. I'm going to assume they have been fixed. | 16:10 |
jroll | rloo: yes | 16:10 |
rloo | thx for confirming jroll! | 16:11 |
*** namnh has quit IRC | 16:11 | |
*** trown is now known as trown|lunch | 16:11 | |
*** ijw has quit IRC | 16:12 | |
*** namnh_ has quit IRC | 16:22 | |
NobodyCam | Good Morning Ironic'ers | 16:24 |
NobodyCam | Happy Monday | 16:24 |
TheJulia | Good morning NobodyCam | 16:31 |
NobodyCam | :) | 16:32 |
*** itlinux has joined #openstack-ironic | 16:34 | |
*** athomas_ has quit IRC | 16:43 | |
*** jiapei has quit IRC | 16:48 | |
*** jiapei_ is now known as jiapei | 16:49 | |
*** athomas has joined #openstack-ironic | 16:51 | |
TheJulia | rloo: email sent to the mailing list | 17:00 |
TheJulia | well | 17:00 |
TheJulia | ops mailing list | 17:00 |
rloo | TheJulia: thx! | 17:00 |
*** derekh has quit IRC | 17:02 | |
*** athomas has quit IRC | 17:04 | |
* TheJulia makes some lunch and then gets back to the never ending to-do list | 17:05 | |
*** dprince has quit IRC | 17:25 | |
*** itlinux has quit IRC | 17:29 | |
openstackgerrit | Merged openstack/ironic master: Deprecate Oneview https://review.openstack.org/564587 | 17:30 |
*** jcoufal_ has joined #openstack-ironic | 17:31 | |
*** jcoufal has quit IRC | 17:34 | |
*** trown|lunch is now known as trown | 17:34 | |
* sambetts --> home | 17:37 | |
sambetts | night all | 17:37 |
*** sambetts is now known as sambetts|afk | 17:37 | |
*** fragatin_ has quit IRC | 17:41 | |
*** akhilaki has quit IRC | 17:51 | |
*** fragatina has joined #openstack-ironic | 17:59 | |
*** r-daneel has quit IRC | 18:06 | |
*** r-daneel has joined #openstack-ironic | 18:06 | |
TheJulia | is anyone able to build docs locally? | 18:16 |
*** itlinux has joined #openstack-ironic | 18:18 | |
jroll | I can try | 18:19 |
jroll | remember zuul doesn't use tox for that | 18:19 |
TheJulia | yeah... I'm some errors like "mock" is not found now | 18:20 |
TheJulia | which makes negative sense | 18:20 |
jroll | O_o | 18:21 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Add note to oneview docs re: derprecation https://review.openstack.org/565274 | 18:23 |
TheJulia | yeah, I just tossed my patch up after spending a half hour of scratching my head with the local job | 18:23 |
TheJulia | well, my local copy of it | 18:23 |
* jroll is running it now | 18:24 | |
*** livelace has quit IRC | 18:26 | |
jroll | TheJulia: looks like mine built okay from master, though failed on a few warnings | 18:35 |
TheJulia | hmm | 18:35 |
* jroll puts through tee quickly because tmux scrollback is being weird | 18:36 | |
jroll | oh I do see those errors while watching it | 18:37 |
*** cdearborn has joined #openstack-ironic | 18:37 | |
jroll | though I don't think they're causing a failure | 18:37 |
jroll | https://gist.github.com/jimrollenhagen/e83432f8c058422c671fe8467f8c680a | 18:39 |
jroll | (for anyone following along) | 18:39 |
jroll | oh, we don't install test-requirements.txt in that tox environment | 18:40 |
jroll | and for whatever reason, autodoc is trying to doc test modules | 18:40 |
*** e0ne has joined #openstack-ironic | 18:42 | |
*** itlinux has quit IRC | 19:07 | |
openstackgerrit | Merged openstack/ironic-inspector master: Change the default discovery driver to fake-hardware https://review.openstack.org/550464 | 19:09 |
*** livelace has joined #openstack-ironic | 19:12 | |
*** livelace has quit IRC | 19:18 | |
*** hemna_ has quit IRC | 19:22 | |
*** d0ugal has quit IRC | 19:45 | |
*** d0ugal has joined #openstack-ironic | 19:46 | |
*** fragatina has quit IRC | 19:51 | |
*** d0ugal_ has joined #openstack-ironic | 20:00 | |
*** d0ugal has quit IRC | 20:02 | |
*** itlinux has joined #openstack-ironic | 20:06 | |
*** itlinux has quit IRC | 20:17 | |
*** hemna_ has joined #openstack-ironic | 20:18 | |
*** d0ugal_ has quit IRC | 20:18 | |
*** itlinux has joined #openstack-ironic | 20:19 | |
*** d0ugal_ has joined #openstack-ironic | 20:19 | |
*** fragatina has joined #openstack-ironic | 20:19 | |
*** fragatina has quit IRC | 20:20 | |
*** fragatina has joined #openstack-ironic | 20:22 | |
*** d0ugal_ has quit IRC | 20:36 | |
*** itlinux has quit IRC | 20:36 | |
*** dprince has joined #openstack-ironic | 20:38 | |
*** d0ugal_ has joined #openstack-ironic | 20:43 | |
*** trown is now known as trown|outtypewww | 21:10 | |
*** jcoufal_ has quit IRC | 21:11 | |
*** dprince has quit IRC | 21:17 | |
*** d0ugal_ has quit IRC | 21:32 | |
*** d0ugal_ has joined #openstack-ironic | 21:35 | |
*** d0ugal_ has quit IRC | 21:38 | |
*** d0ugal_ has joined #openstack-ironic | 21:39 | |
*** rbudden has quit IRC | 21:39 | |
*** d0ugal_ has quit IRC | 21:57 | |
*** d0ugal_ has joined #openstack-ironic | 21:59 | |
*** cjloader has quit IRC | 22:03 | |
*** e0ne has quit IRC | 22:03 | |
*** rcernin has joined #openstack-ironic | 22:13 | |
*** cdearborn has quit IRC | 22:16 | |
*** rbudden has joined #openstack-ironic | 22:20 | |
*** rh-jelabarre has joined #openstack-ironic | 22:51 | |
*** lbragstad has quit IRC | 23:05 | |
*** rajinir has quit IRC | 23:08 | |
*** r-daneel has quit IRC | 23:09 | |
*** d0ugal__ has joined #openstack-ironic | 23:11 | |
*** d0ugal_ has quit IRC | 23:14 | |
*** pmannidi has joined #openstack-ironic | 23:15 | |
*** rbudden has quit IRC | 23:23 | |
*** d0ugal__ has quit IRC | 23:29 | |
*** jiapei has quit IRC | 23:32 | |
*** hemna_ has quit IRC | 23:48 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!