| * NobodyCam only has 300 Mb :) | 00:00 | |
| *** jerrygb has quit IRC | 00:00 | |
| JayF | 150mbit | 00:01 |
|---|---|---|
| NobodyCam | :) | 00:01 |
| JayF | Just got wifi problems figured out though. Apparently there's very little preference in most OSes for 5ghz over 2.4ghz | 00:01 |
| JayF | so diff ssid for 5ghz, connect devices to that, magically my wifi is always fast | 00:02 |
| *** jerrygb has joined #openstack-ironic | 00:02 | |
| * JayF lives in a big apt building with lots of contention in 2.4 | 00:02 | |
| NobodyCam | ahh :) | 00:02 |
| JayF | probably moving soonish though. You're in WA state, right? | 00:03 |
| NobodyCam | Not at this time of year, | 00:03 |
| NobodyCam | we're down in Palm Springs for winter | 00:03 |
| *** jamielennox|away is now known as jamielennox | 00:05 | |
| JayF | NobodyCam: someone set off the alarm | 00:06 |
| JayF | NobodyCam: tests broken on master :( netifaces | 00:06 |
| JayF | going to gist my science | 00:06 |
| JayF | every test failing with ImportError: No module named netifaces | 00:06 |
| JayF | pages and pages of scrollback /o\ | 00:06 |
| NobodyCam | JayF: yep | 00:07 |
| NobodyCam | :) | 00:07 |
| *** s-moriya has joined #openstack-ironic | 00:08 | |
| NobodyCam | JayF: you prob lso have a few ImportError: cannot import name collection | 00:08 |
| JayF | I just checked out HEAD^1 and trying tests | 00:09 |
| JayF | failing similarly, going HEAD^2 | 00:09 |
| JayF | rebuilidng my tox env everytime | 00:10 |
| JayF | looks almost like oslo_utils might have a dep it's not properly declaring? | 00:10 |
| JayF | did oslo_utils release recently? | 00:10 |
| devananda | krotscheck: if you want to be compatible with many versions of the server, then yes | 00:11 |
| *** aginwala has quit IRC | 00:11 | |
| * JayF seeing if oslo.utils master is passing tests | 00:13 | |
| devananda | tox -epy27 -r ==> I, too, get that error | 00:14 |
| JayF | jroll: around? | 00:14 |
| JayF | jroll: [upstream pagerduty alert] | 00:14 |
| JayF | lol | 00:14 |
| *** praneshp has quit IRC | 00:14 | |
| JayF | NobodyCam: devananda: tox is installing netifaces for oslo.utils tests, but not Ironic tests | 00:15 |
| devananda | JayF: looks like oslo.utils 3.1.0 was tagged 5 days ago | 00:16 |
| JayF | [testenv] | 00:16 |
| JayF | deps = -r{toxinidir}/test-requirements.txt | 00:16 |
| JayF | .[netifaces] | 00:16 |
| JayF | is in tox.ini for oslo.utils | 00:16 |
| JayF | so they're explicitly calling out netifaces as needed for testing w/o putting it in requirements | 00:16 |
| devananda | uhhh | 00:16 |
| devananda | dhellmann: ^ any thoughts ? | 00:16 |
| *** aginwala has joined #openstack-ironic | 00:17 | |
| *** jerrygb has quit IRC | 00:18 | |
| *** mbound_ has quit IRC | 00:18 | |
| *** jerrygb has joined #openstack-ironic | 00:18 | |
| *** alexpilotti has joined #openstack-ironic | 00:19 | |
| devananda | JayF: that import line in oslo.utils has been there for 11 months | 00:20 |
| devananda | JayF: also, my existing venv has netifaces==0.10.4 in it | 00:21 |
| devananda | *my existing ironic venv | 00:21 |
| devananda | something caused it to be installed, and now there is not something causing it to be installed | 00:21 |
| JayF | hmm | 00:21 |
| devananda | oslo.utils 3.0.0 has netifaces in requirements.txt | 00:22 |
| devananda | 3.1.0 does not .... /me continues investigating | 00:22 |
| *** jerrygb has quit IRC | 00:23 | |
| *** alexpilotti has quit IRC | 00:24 | |
| JayF | ccecc5fc4e92844616252689fb78510a578aee60 removed it | 00:24 |
| devananda | JayF: https://github.com/openstack/oslo.utils/commit/2abbe1c3b19490f570c29502cd56ec92daf4ca45 | 00:24 |
| JayF | with the commit message | 00:24 |
| devananda | heh | 00:24 |
| JayF | JINX | 00:24 |
| JayF | so that's a backwards compat breaking change | 00:25 |
| JayF | and should be reverted in 3.1.1? | 00:25 |
| devananda | https://bugs.launchpad.net/oslo.utils/+bug/1523706 | 00:25 |
| openstack | Launchpad bug 1523706 in oslo.utils "No module named netifaces" [Undecided,New] | 00:25 |
| devananda | looks like this hits several projects | 00:25 |
| NobodyCam | :0 | 00:26 |
| devananda | http://lists.openstack.org/pipermail/openstack-dev/2015-December/081660.html | 00:26 |
| JayF | devananda: is the right thing to add Ironic as impacted to that bug? | 00:26 |
| devananda | JayF: not really | 00:27 |
| JayF | devananda: or is that only if Ironic has action to take to fix the bug? | 00:27 |
| JayF | gotcha | 00:27 |
| devananda | JayF: yes ^ that | 00:27 |
| devananda | in an ideal world, we should have a way to track that | 00:28 |
| JayF | Sure, I just wanted to know the right thing to do :) | 00:28 |
| devananda | but LP actually has a bug where, if too many projects are tagged on the bug, no one can close the bug | 00:28 |
| JayF | no opposition with either way being right | 00:28 |
| devananda | it's great | 00:28 |
| JayF | I wasted 2h of my life on that | 00:28 |
| JayF | trying to cleanup bugs | 00:28 |
| devananda | heh | 00:28 |
| JayF | before someone in -infra clued me in | 00:29 |
| JayF | way back in the day | 00:29 |
| jroll | JayF: devananda thanks for finding that | 00:32 |
| *** betherly has quit IRC | 00:32 | |
| *** patrickeast has quit IRC | 00:32 | |
| devananda | jroll: I pinged fungi in -infra just now and added it to our whiteboard | 00:32 |
| jroll | yay, thanks | 00:33 |
| *** mtanino has quit IRC | 00:33 | |
| devananda | either we do nothing and wait for oslo to release a fix, or I think we could try landing a patch to our test-requirements | 00:33 |
| * jroll assumes but verifies there is a fix in queue for oslo | 00:33 | |
| jroll | oh, we could | 00:33 |
| jroll | presumably if we import it, we use it in more than tests | 00:33 |
| devananda | jroll: we dont use it at all | 00:34 |
| devananda | only oslo.utils needs it | 00:34 |
| devananda | but doesn't list it as a dep in this release. it's TOTALLY their bug, afaict | 00:34 |
| jroll | if we import oslo.utils though... | 00:34 |
| jroll | actually | 00:34 |
| devananda | we never directly use netifaces | 00:34 |
| jroll | no, but we import a thing that uses it | 00:35 |
| jroll | presumably outside of tests | 00:35 |
| devananda | if oslo.utils is using it -- it should really be listing it as a requirement itself | 00:35 |
| jroll | so it isn't a test dep | 00:35 |
| devananda | right. it's a dep-dep | 00:35 |
| *** Paradoxx has joined #openstack-ironic | 00:37 | |
| *** Paradoxx has quit IRC | 00:37 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic: Depend on oslo.utils[netifaces] https://review.openstack.org/254452 | 00:37 |
| jroll | devananda: ^ | 00:37 |
| * jroll tests | 00:37 | |
| JayF | what is that syntax? whoa! | 00:38 |
| devananda | testing | 00:38 |
| jroll | JayF: "extras" https://pythonhosted.org/setuptools/setuptools.html#declaring-extras-optional-features-with-their-own-dependencies | 00:38 |
| jroll | pip syntax is package[extra] | 00:38 |
| jroll | see also the tox.ini change in that oslo.utils commit | 00:38 |
| *** aginwala has quit IRC | 00:39 | |
| *** absubram has joined #openstack-ironic | 00:40 | |
| *** Paradoxx has joined #openstack-ironic | 00:40 | |
| devananda | jroll: totally works. +2'ing ahead of the gate | 00:40 |
| *** Paradoxx has quit IRC | 00:40 | |
| *** KBT has joined #openstack-ironic | 00:41 | |
| jroll | woo | 00:41 |
| jroll | and yes, we use it outside of tests btw | 00:41 |
| jlvillal | Is anyone working on fixing the iLO CI job? | 00:41 |
| *** absubram_ has joined #openstack-ironic | 00:41 | |
| *** yonglihe has quit IRC | 00:41 | |
| jroll | jlvillal: hopefully, but feel free to ignore for now | 00:42 |
| NobodyCam | jroll: JayF: devananda: Awesome :) | 00:42 |
| jlvillal | jroll, Okay. I wasn't sure if there was someone to ping about the job. | 00:42 |
| devananda | gotta run - dinner plans at 5, so... | 00:43 |
| jroll | jlvillal: in theory ramesh. but, ya know. | 00:43 |
| *** ChuckC has quit IRC | 00:44 | |
| *** ijw has joined #openstack-ironic | 00:44 | |
| NobodyCam | :p | 00:44 |
| jlvillal | jroll, Yeah I don't think it is his responsibility anymore :) | 00:44 |
| *** absubram has quit IRC | 00:44 | |
| *** absubram_ is now known as absubram | 00:44 | |
| * jlvillal will learn to be patient... | 00:44 | |
| NobodyCam | we'll see if we can find someone to look in to it | 00:45 |
| *** Nisha has quit IRC | 00:46 | |
| jlvillal | NobodyCam, Thanks. You rock! :) | 00:46 |
| NobodyCam | :) | 00:47 |
| jroll | devananda: oh, https://review.openstack.org/#/c/254379/ | 00:47 |
| * jroll abandons | 00:50 | |
| *** ijw has quit IRC | 00:51 | |
| *** ijw has joined #openstack-ironic | 00:53 | |
| *** aginwala has joined #openstack-ironic | 00:53 | |
| *** ionutbalutoiu has quit IRC | 00:55 | |
| *** aginwala has quit IRC | 00:55 | |
| *** Sukhdev has joined #openstack-ironic | 00:56 | |
| *** Sukhdev has quit IRC | 00:57 | |
| rloo | jroll: qq, wrt 'pip syntax is package[extra]' Does it install the package + extra, or only the extra from that package? | 00:57 |
| *** ijw has quit IRC | 00:57 | |
| jroll | rloo: both, afaik. tests seem to support that | 00:58 |
| jroll | rloo: I abandoned that change, though jfyi | 00:58 |
| rloo | jroll: yeah, i saw you abandoned. i was just curious. | 00:59 |
| openstackgerrit | Arun S A G proposed openstack/ironic-python-agent: Use latest CoreOS stable when building https://review.openstack.org/191505 | 00:59 |
| JayF | I'm glad to have seen that. No idea until today it was a thing at all | 00:59 |
| jroll | right on | 00:59 |
| rloo | JayF: yeah, learn something new every day on this job :) | 00:59 |
| jroll | devananda: mind reviewing https://etherpad.openstack.org/p/ironic-4.3.0-announce ? | 01:00 |
| jroll | both 4.2.2 and 4.3.0 there | 01:00 |
| JayF | jroll: should we include instructions for operators to re-clean fleet or something similar? | 01:03 |
| JayF | jroll: I know if we had this deployed to production, we'd push all AVAILABLE nodes back through cleaning | 01:03 |
| jroll | JayF: dunno, I would think that should have been in the OSSA if we did it | 01:04 |
| JayF | jroll: just a thought | 01:04 |
| jroll | yeah totes | 01:04 |
| *** thanhnt-z has joined #openstack-ironic | 01:05 | |
| *** rajathagasthya has quit IRC | 01:07 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Canary test commit **DO NOT MERGE** https://review.openstack.org/253639 | 01:09 |
| *** thanhnt-z has quit IRC | 01:09 | |
| naohirot | rloo: good evening, still there? | 01:14 |
| rloo | naohirot: hi | 01:14 |
| naohirot | rloo: Hi :) | 01:15 |
| zer0c00l | ping JayF | 01:15 |
| zer0c00l | JayF: About your comment in https://review.openstack.org/#/c/191505/17 | 01:15 |
| JayF | zer0c00l: already replied | 01:15 |
| zer0c00l | The sysrq-triggers don't work anymore | 01:15 |
| zer0c00l | okay | 01:15 |
| JayF | zer0c00l: nspawn keeps making more security things :( | 01:15 |
| JayF | zer0c00l: I have two patches in already undoing unconfigurable security stuff in nspawn | 01:15 |
| naohirot | rloo: I'd like to ask if my change is ok or not before submitting an update regarding | 01:15 |
| JayF | zer0c00l: So I propose we move the IPA ramdisk to use chroot, like I do downstream now | 01:16 |
| rloo | naohirot: what's the change? | 01:16 |
| jroll | JayF: I so want to just kill the coreos ramdisk, but it is so much easier to customize :/ | 01:16 |
| naohirot | rloo: https://review.openstack.org/#/c/253498/ | 01:16 |
| JayF | jroll: why kill it? Make it run in a chroot and it removes most headaches caused by containers | 01:16 |
| rloo | naohirot: what's the change? | 01:16 |
| JayF | jroll: just like we did downstream | 01:16 |
| naohirot | rloo: 'unknown value' come from the line 53. | 01:16 |
| jroll | JayF: because two different ways of solving the same problem, requiring changes to sometimes happen in two places | 01:17 |
| naohirot | rloo: should I change it to 'type of value is unknown'? | 01:17 |
| naohirot | rloo: Is that your suggestion? | 01:17 |
| JayF | jroll: TBH I'm not sure what's different in the technology today vs when we wanted to do a coreos ramdisk | 01:17 |
| rloo | naohirot: that was one of my suggestions, yes. what do you think? | 01:17 |
| JayF | jroll: it still stands that some deployers will want a more fully featured ramdisk, and for that, is the coreos option | 01:18 |
| *** penick has quit IRC | 01:18 | |
| rloo | naohirot: or just make it an empty string, and don't print it out. | 01:18 |
| JayF | jroll: just because a DIB element for IPA exists today doesn't mean the technical reasons we wanted IPA to have a more fully-featured ramdisk are now invalid | 01:18 |
| naohirot | rloo: Okay, now I got it. But Tan said that other project uses 'unknown value'. | 01:18 |
| *** mbound_ has joined #openstack-ironic | 01:19 | |
| rloo | naohirot: yeah, i saw his comment. but w/o knowing that other projects do that, if you just read the .sample, what would you think? | 01:19 |
| jroll | JayF: sure, but it seems like most people tend to use DIB for whatever reason, and we're doing extra effort to support both | 01:19 |
| rloo | naohirot: that was my suggestion. i can be overruled. | 01:19 |
| naohirot | rloo: I thought that if there is a precedent, it would be better to follow it. | 01:19 |
| JayF | jroll: well, at least Rackspace and presumedly Yahoo use teh coreos ramdisk, right? | 01:19 |
| zer0c00l | JayF: Yahoo doesn't use coreos | 01:20 |
| JayF | or maybe not, /me thought zer0c00l was yahoo | 01:20 |
| *** betherly has joined #openstack-ironic | 01:20 | |
| zer0c00l | no IPA at all | 01:20 |
| rloo | naohirot: yeah, we tend to follow if there is a precedent/consistent that makes some sense. i just want to buck the system in this case but again, i could be overruled. | 01:20 |
| JayF | zer0c00l: where are you from and you like my ramdisk? hehe | 01:20 |
| zer0c00l | we use anaconda with hacks now, planning to move to IPA | 01:20 |
| jroll | JayF: sure, I wonder if we can make the DIB ramdisk as good as the coreos one | 01:20 |
| naohirot | rloo: I think I agree with you, because it's type, but not value. | 01:20 |
| zer0c00l | yeash, i am from Yahoo | 01:20 |
| JayF | jroll: DIB ramdisks start out by ripping out init and replacing it with a bash script. Unless that's changed I think that's a design flaw/philosophy that isn't going to be able to be made "better" | 01:21 |
| *** ijw has joined #openstack-ironic | 01:21 | |
| zer0c00l | IMO the ramdisk itself is huge | 01:22 |
| naohirot | rloo: so I'm going to change L.53 to 'type of value is unknown'. | 01:22 |
| zer0c00l | It needs to be separated into a two level thing like anaconda | 01:22 |
| rloo | naohirot: sure, that's fine with me. | 01:22 |
| JayF | zer0c00l: in our env it's a tradeoff, right? We don't mind shipping around a bigger ramdisk if it means we have ssh access and some debugging stuff | 01:22 |
| zer0c00l | Anaconda does this, it has a smaller ramdisk | 01:22 |
| zer0c00l | and a bigger stage2 image which actually has the installer | 01:22 |
| zer0c00l | Called the squashfs.img | 01:22 |
| JayF | zer0c00l: we've talked about doing that, because it would mean we don't ship ramdisks anymore | 01:23 |
| zer0c00l | ramdisk boots and downloads the squashfs mounts it and switch root | 01:23 |
| *** coolsvap has quit IRC | 01:23 | |
| JayF | zer0c00l: you can use cloud-config-url= in coreos to have it pull down, remotely, a cloud-config.yml | 01:23 |
| naohirot | rloo: thank! I'll submit the update. | 01:23 |
| *** zhenguo has joined #openstack-ironic | 01:23 | |
| JayF | zer0c00l: and you could bootstrap that into downloading and extracting an IPA container for nspawn or chrooting | 01:23 |
| rloo | naohirot: yw | 01:23 |
| jroll | I still want to play with tinyipa one day | 01:23 |
| zer0c00l | JayF: Agree! | 01:23 |
| JayF | zer0c00l: this was on jroll's pet project list for a while | 01:23 |
| JayF | zer0c00l: the reason I haven't done it is simple: it's a cleaner fail with the bigger ramdisk. If I can't pull down the huge ramdisk, iPXE will bail out and reboot the machine so it can repxe and succeed | 01:24 |
| *** mbound_ has quit IRC | 01:24 | |
| JayF | zer0c00l: and I've never been sufficiently motivated to write something that robust to load a "stage two" for the IPA ramdisk | 01:24 |
| zer0c00l | :) | 01:25 |
| zer0c00l | So JayF , you will be sending the downstream chroot stuff soon? | 01:25 |
| zer0c00l | I am interested | 01:25 |
| JayF | zer0c00l: if you want it, then I have no problem upstreaming it | 01:25 |
| *** marios has quit IRC | 01:25 | |
| *** coolsvap has joined #openstack-ironic | 01:25 | |
| JayF | aweeks and I wrote all that together a few weeks back, I can upstream it no problem tomorrow | 01:25 |
| JayF | Just wasn't something I thought folks would want upstream :) | 01:26 |
| zer0c00l | JayF: The reboot stuff is kind of complex i agree! | 01:26 |
| JayF | zer0c00l: yeah and apologies for not actively reviewing that as well. That's a patch I should've been following closesly | 01:26 |
| *** marios has joined #openstack-ironic | 01:26 | |
| JayF | zer0c00l: and ironically enough - - we switched to chroot because /sys/ was r/o as well :) (apparently writing to /sys/ is done by mdadm, so if you want to build raids in the agent it's pretty wellrequired) | 01:26 |
| *** links has joined #openstack-ironic | 01:27 | |
| zer0c00l | JayF: Raid is a priority for us at Yahoo as well | 01:27 |
| zer0c00l | Would be nice to see that code upstream :) | 01:28 |
| JayF | zer0c00l: then sounds like we're preempting a bug. I'll toss that up first thing tomorrow | 01:28 |
| zer0c00l | Thanks :) | 01:28 |
| *** Sukhdev has joined #openstack-ironic | 01:28 | |
| JayF | np | 01:28 |
| * JayF heading out for the day o/ | 01:28 | |
| zer0c00l | later | 01:28 |
| *** rajathagasthya has joined #openstack-ironic | 01:28 | |
| *** garthb has quit IRC | 01:31 | |
| jroll | devananda: ignore that review request, I've sent the announcements now | 01:34 |
| *** baoli has joined #openstack-ironic | 01:36 | |
| *** smoriya is now known as smoriya_afk | 01:38 | |
| *** patrickeast has joined #openstack-ironic | 01:39 | |
| *** aginwala has joined #openstack-ironic | 01:39 | |
| lintan | morning Ironic, devananda,jroll,rloo,zer0c00l, JayF | 01:41 |
| jroll | \o lintan :) | 01:41 |
| rloo | hey, morning lintan | 01:41 |
| zer0c00l | Hey Morning lintan | 01:42 |
| *** smoriya has joined #openstack-ironic | 01:43 | |
| *** r-daneel has quit IRC | 01:43 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add oslo_config.Opt support in Ironic config generator https://review.openstack.org/253498 | 01:46 |
| *** s-moriya has quit IRC | 01:47 | |
| *** KBT has quit IRC | 01:47 | |
| Fdaisuke | morning, jroll, | 01:49 |
| jroll | hi Fdaisuke | 01:49 |
| Fdaisuke | jroll: I sincerely apologize for to ask you for the review.. I had overlooked that talk. | 01:49 |
| jroll | it's ok. | 01:49 |
| Fdaisuke | jroll: thank you! I'll be careful next time. | 01:49 |
| jroll | Fdaisuke: no, it's fine to ask for reviews sometimes, I just didn't understand the question | 01:50 |
| *** rajathagasthya has quit IRC | 01:52 | |
| jlvillal | naohirot, Just as an FYI. And not sure if it was required or not. But it helps if the patch is not rebased between revisions. As it makes it easier to compare revisions of a patch. | 01:53 |
| jlvillal | naohirot, Of course many times it is required to rebase the patch :) | 01:53 |
| naohirot | jlvillal: hi! | 01:53 |
| jlvillal | naohirot, I was looking at: https://review.openstack.org/#/c/253498/ | 01:54 |
| jlvillal | And trying to compare revision 1 vs 3. | 01:54 |
| jlvillal | naohirot, Not sure if I am making any sense... | 01:54 |
| *** chenke has joined #openstack-ironic | 01:54 | |
| naohirot | jlvillal: Oh, are you saying that rebase is not necessary in some case? | 01:55 |
| jlvillal | naohirot, In most cases. | 01:56 |
| naohirot | jlvillal: really? | 01:56 |
| jlvillal | naohirot, Yes :) | 01:56 |
| naohirot | jlvillal: I'll follow that way next time, I just didn't know that. Thanks for letting me know :) | 01:57 |
| chenke | Does ironic support assign 2 ip address if PC have 2 network card while deploy image? | 01:57 |
| jlvillal | naohirot, I try to only rebase if I need to. Usually Jenkins will report if the patch is in merge conflict. | 01:57 |
| jlvillal | naohirot, Thanks for doing that. It will make comparing revisions of a patch much easier. | 01:58 |
| naohirot | jlvillal: I see, that makes sense. | 01:58 |
| *** praneshp has joined #openstack-ironic | 02:05 | |
| *** Sukhdev has quit IRC | 02:11 | |
| *** KBT has joined #openstack-ironic | 02:11 | |
| *** KBT has quit IRC | 02:12 | |
| *** jerrygb has joined #openstack-ironic | 02:14 | |
| *** yonglihe has joined #openstack-ironic | 02:15 | |
| *** jerrygb has quit IRC | 02:16 | |
| *** rajathagasthya has joined #openstack-ironic | 02:17 | |
| *** alexpilotti has joined #openstack-ironic | 02:21 | |
| *** alexpilotti has quit IRC | 02:25 | |
| *** rajathagasthya has quit IRC | 02:27 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Update python-scciclient version number https://review.openstack.org/253358 | 02:36 |
| jroll | naohirot: hey, if you add scciclient to this list, it will update requirements.txt for you: https://github.com/openstack/requirements/blob/master/projects.txt | 02:45 |
| *** deray has joined #openstack-ironic | 02:45 | |
| jroll | naohirot: then you don't need to do this manually: https://review.openstack.org/#/c/248997/ | 02:45 |
| naohirot | jroll: Hi | 02:45 |
| jroll | naohirot: oh, now I see it's already done, but the patches were not merged: https://review.openstack.org/#/c/240484 | 02:47 |
| naohirot | jroll: I see. | 02:48 |
| *** changzhi has joined #openstack-ironic | 02:49 | |
| naohirot | jroll: I didn't notice that. I supposed to get the global requirement update patch. Actually I have been got it, and I unintentionally ignored :) | 02:50 |
| changzhi | hi, all. I create a node and run "ironic node-validate xx" happens an error. It says "Cannot validate PXE bootloader. Some parameters were missing in node's driver_info. Missing are: ['deploy_ramdisk', 'deploy_kernel']" | 02:50 |
| jroll | naohirot: it's easy to do :) | 02:51 |
| changzhi | how do i fill these params? | 02:51 |
| changzhi | deploy_ramdisk and deploy_kernel? | 02:51 |
| naohirot | jroll: Yeah, I watch the bot carefully :) | 02:52 |
| jroll | changzhi: ironic node-update $NODE_UUID add driver_info/deploy_kernel=$DEPLOY_VMLINUZ_UUID driver_info/deploy_ramdisk=$DEPLOY_INITRD_UUID | 02:52 |
| jroll | changzhi: this is in the deployment guide :) | 02:52 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Conductor handles manual cleaning https://review.openstack.org/247285 | 02:52 |
| changzhi | jroll: i know that. but how should I get deploy_ramdisk and deploy_kernel? :-) | 02:53 |
| jroll | changzhi: http://docs.openstack.org/developer/ironic/deploy/install-guide.html#image-requirements | 02:54 |
| jroll | changzhi: see Add the deploy images to the Image service | 02:54 |
| jroll | they get uploaded to glance | 02:54 |
| changzhi | jroll: thank you :-) | 02:54 |
| jroll | sure | 02:54 |
| *** baoli has quit IRC | 02:55 | |
| *** Marga_ has quit IRC | 02:57 | |
| changzhi | jroll: after run "disk-image-create ubuntu baremetal dhcp-all-interfaces grub2 -o my-image" | 03:01 |
| changzhi | jroll: where is the "my-image" ? | 03:01 |
| jroll | changzhi: in your current directory, I believe | 03:03 |
| *** aginwala has quit IRC | 03:03 | |
| TheJulia | yea/win 29 | 03:05 |
| TheJulia | doh | 03:05 |
| *** aginwala has joined #openstack-ironic | 03:07 | |
| *** aginwala has quit IRC | 03:07 | |
| changzhi | jroll: no ... the directory is empty :-( | 03:08 |
| jroll | changzhi: did the command succeed? | 03:08 |
| changzhi | jroll: yes. the command output a lot logs in console | 03:08 |
| Haomeng | changzhi: hi, changzhi | 03:08 |
| changzhi | haomeng: hi | 03:08 |
| jroll | changzhi: and no errors? that's odd | 03:09 |
| Haomeng | changzhi: can you help to check the last lines of the output, if there is no output, should be return with error | 03:09 |
| changzhi | the last output: Setting up grub2-common (2.02~beta2-9ubuntu1.4) ... | 03:09 |
| changzhi | Setting up grub-efi-amd64 (2.02~beta2-9ubuntu1.4) ... | 03:09 |
| changzhi | Setting up shim-signed (1.9+0.8-0ubuntu2) ... | 03:09 |
| changzhi | Processing triggers for libc-bin (2.19-0ubuntu6.6) ... | 03:09 |
| Haomeng | changzhi: we can share the log via http://paste.openstack.org/ | 03:10 |
| Haomeng | changzhi: for irc, it is not easy to post multi-lines and to read | 03:10 |
| changzhi | haomeng: ok. wait a moment | 03:10 |
| Haomeng | changzhi: ok, and run 'echo $?' to check return code, should not be 0 I think | 03:10 |
| *** aginwala has joined #openstack-ironic | 03:11 | |
| chenke | Haomeng: how are u? | 03:15 |
| Haomeng | chenke: good, and you? :) | 03:15 |
| TheJulia | changzhi: try the vm element instead of the grub element | 03:16 |
| TheJulia | the grub2 element | 03:16 |
| TheJulia | grub2 tries to install EFI, which in ubuntu causes it to fail because the package install asumes that it must install locally instead of just unpackage the files. | 03:17 |
| changzhi | http://paste.openstack.org/show/481112/ | 03:17 |
| Haomeng | changzhi: ok | 03:17 |
| chenke | Haomeng: fine too. i am doing a test to check whether ironic port support binding 2 MACaddress, thus we can init 2 NIC while deploy. do you meet this kind of requirement? | 03:19 |
| TheJulia | changzhi: checkout https://review.openstack.org/219612 | 03:19 |
| Haomeng | changzhi: so this is last line "Processing triggers for libc-bin (2.19-0ubuntu6.6) ..." ? | 03:19 |
| changzhi | haomeng: yes | 03:19 |
| changzhi | [stack@localhost images]$ echo $? | 03:20 |
| changzhi | 1 | 03:20 |
| *** ijw has quit IRC | 03:20 | |
| changzhi | :-( | 03:20 |
| Haomeng | changzhi: yes, it breaks | 03:20 |
| Haomeng | changzhi: libc-bin should be c runtime lib I think | 03:21 |
| changzhi | haomeng: dpkg: error processing package grub-efi-amd64-signed (--configure): | 03:21 |
| TheJulia | changzhi: https://review.openstack.org/219612 | 03:21 |
| TheJulia | it should work around the issue | 03:21 |
| Haomeng | TheJulia: cool:) | 03:21 |
| Haomeng | changzhi: can you apply the patch and try again | 03:22 |
| changzhi | thejulia: thx, i will try | 03:22 |
| Haomeng | changzhi: yes, more error can be found in syslog | 03:22 |
| *** alexpilotti has joined #openstack-ironic | 03:23 | |
| *** deray has quit IRC | 03:23 | |
| *** shuquan has joined #openstack-ironic | 03:23 | |
| *** alexpilotti has quit IRC | 03:27 | |
| *** shuquan has quit IRC | 03:28 | |
| rloo | hey folks, please don't bother doing rechecks. the gate is broken. see the etherpad. | 03:30 |
| *** shuquan has joined #openstack-ironic | 03:31 | |
| changzhi | TheJulia: you are great!! the patch goes well! | 03:31 |
| jroll | better yet, don't ever do a recheck without checking why the job failed! | 03:31 |
| jroll | rloo: go "home" ! :) | 03:32 |
| TheJulia | changzhi: glad to hear! | 03:32 |
| rloo | jroll: am going. i was just checking my email one last time and noticed the rechecks :) | 03:32 |
| changzhi | TheJulia: after I run "disk-image-create ubuntu vm dhcp-all-interfaces grub2 -o my-image". It generates my-image.d my-image.qcow2 | 03:32 |
| rloo | g'night all :D | 03:32 |
| changzhi | TheJulia: does it right? | 03:32 |
| *** rloo has quit IRC | 03:32 | |
| TheJulia | changzhi: yes, my-image.qcow2 should be a bootable disk image | 03:33 |
| changzhi | TheJulia: but it doesn't generate my-image.vmlinuz and my-image.initrd | 03:33 |
| changzhi | Thejulia: only my-image.qcow2 | 03:34 |
| TheJulia | changzhi: try removing vm, i think | 03:34 |
| jroll | TheJulia: oh, this is for deploy ramdisks | 03:34 |
| TheJulia | well, if your going to use it as a deploy ramdisk, you need to add ironic-agent to the command line, remove grub2 and vm | 03:35 |
| *** jerrygb has joined #openstack-ironic | 03:35 | |
| changzhi | disk-image-create ubuntu dhcp-all-interfaces grub2 -o my-image? this one? | 03:35 |
| * jroll totally didn't look at the cli that was being used | 03:35 | |
| changzhi | TheJulia: ? | 03:35 |
| TheJulia | so disk-image-create -o my-deploy-image ubuntu dhcp-all-interfaces ironic-agent | 03:35 |
| jroll | changzhi: did you read http://docs.openstack.org/developer/ironic/deploy/install-guide.html#building-or-downloading-a-deploy-ramdisk-image | 03:36 |
| TheJulia | That should help | 03:36 |
| *** deray has joined #openstack-ironic | 03:40 | |
| openstackgerrit | Kan proposed openstack/python-ironicclient: [WIP]Format the valid fields for list commands https://review.openstack.org/245541 | 03:45 |
| *** baoli has joined #openstack-ironic | 04:01 | |
| *** alexpilotti has joined #openstack-ironic | 04:01 | |
| *** alexpilotti has quit IRC | 04:06 | |
| *** Nisha has joined #openstack-ironic | 04:08 | |
| *** aginwala has quit IRC | 04:13 | |
| *** aginwala has joined #openstack-ironic | 04:16 | |
| *** aginwala has quit IRC | 04:20 | |
| *** shuquan has quit IRC | 04:25 | |
| *** shuquan has joined #openstack-ironic | 04:26 | |
| *** changzhi has quit IRC | 04:32 | |
| *** baoli has quit IRC | 04:37 | |
| *** boris-42_ has quit IRC | 04:43 | |
| *** kozhukalov_ has joined #openstack-ironic | 04:45 | |
| *** shuquan has quit IRC | 04:50 | |
| *** deray has quit IRC | 04:51 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Support Linux-IO in addition to tgtd https://review.openstack.org/234861 | 04:52 |
| openstackgerrit | Kan proposed openstack/python-ironicclient: Format the valid fields for list commands https://review.openstack.org/245541 | 04:55 |
| *** deray has joined #openstack-ironic | 04:59 | |
| *** kozhukalov_ has quit IRC | 05:01 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic: Add oslo_config.Opt support in Ironic config generator https://review.openstack.org/253498 | 05:03 |
| openstackgerrit | Merged openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/254472 | 05:06 |
| *** kalpase has joined #openstack-ironic | 05:10 | |
| openstackgerrit | Merged openstack/python-ironicclient: Correct node-port-list help info https://review.openstack.org/246729 | 05:15 |
| *** amotoki has joined #openstack-ironic | 05:20 | |
| *** Marga_ has joined #openstack-ironic | 05:20 | |
| *** Marga_ has quit IRC | 05:25 | |
| *** Sukhdev has joined #openstack-ironic | 05:26 | |
| *** shuquan has joined #openstack-ironic | 05:30 | |
| *** Marga_ has joined #openstack-ironic | 05:32 | |
| *** Marga_ has quit IRC | 05:32 | |
| *** Marga_ has joined #openstack-ironic | 05:33 | |
| *** absubram has quit IRC | 05:34 | |
| *** alexpilotti has joined #openstack-ironic | 05:37 | |
| *** hoangcx has joined #openstack-ironic | 05:38 | |
| *** aginwala has joined #openstack-ironic | 05:41 | |
| *** alexpilotti has quit IRC | 05:44 | |
| *** shuquan has quit IRC | 05:46 | |
| *** shuquan has joined #openstack-ironic | 05:47 | |
| *** amotoki has quit IRC | 05:48 | |
| *** kozhukalov_ has joined #openstack-ironic | 05:49 | |
| *** jerrygb has quit IRC | 05:50 | |
| *** dims has quit IRC | 05:57 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Test doc gate(Not merge) https://review.openstack.org/254553 | 05:58 |
| *** alexpilotti has joined #openstack-ironic | 06:01 | |
| *** pradipta has joined #openstack-ironic | 06:02 | |
| *** alexpilotti has quit IRC | 06:05 | |
| *** kozhukalov_ has quit IRC | 06:06 | |
| *** ionutbalutoiu has joined #openstack-ironic | 06:09 | |
| *** shuquan has quit IRC | 06:15 | |
| *** Sukhdev has quit IRC | 06:20 | |
| *** dims has joined #openstack-ironic | 06:20 | |
| *** Sukhdev has joined #openstack-ironic | 06:20 | |
| *** alexpilotti has joined #openstack-ironic | 06:22 | |
| *** Sukhdev has quit IRC | 06:25 | |
| *** alexpilotti has quit IRC | 06:26 | |
| *** ionutbalutoiu has quit IRC | 06:27 | |
| *** ionutbalutoiu has joined #openstack-ironic | 06:28 | |
| *** amotoki has joined #openstack-ironic | 06:30 | |
| openstackgerrit | Merged openstack/ironic: Add Code Contribution Guide for Ironic https://review.openstack.org/248647 | 06:38 |
| *** shuquan has joined #openstack-ironic | 06:38 | |
| *** e0ne has joined #openstack-ironic | 06:38 | |
| *** aginwala has quit IRC | 06:43 | |
| *** e0ne has quit IRC | 06:45 | |
| *** aginwala has joined #openstack-ironic | 06:46 | |
| openstackgerrit | Jinxing Fang proposed openstack/ironic: Remove the qpid message driver in config file https://review.openstack.org/254576 | 06:47 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/254471 | 06:47 |
| *** ChubYann has quit IRC | 06:47 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/254495 | 06:47 |
| *** smoriya has quit IRC | 06:49 | |
| *** e0ne has joined #openstack-ironic | 06:49 | |
| *** e0ne has quit IRC | 06:52 | |
| *** aginwala has quit IRC | 06:52 | |
| *** ionutbalutoiu has quit IRC | 06:58 | |
| *** ionutbalutoiu has joined #openstack-ironic | 06:58 | |
| *** alexpilotti has joined #openstack-ironic | 07:00 | |
| *** e0ne has joined #openstack-ironic | 07:02 | |
| *** alexpilotti has quit IRC | 07:05 | |
| *** dims has quit IRC | 07:11 | |
| *** sambetts-afk has quit IRC | 07:13 | |
| *** ionutbalutoiu has quit IRC | 07:17 | |
| *** ionutbalutoiu has joined #openstack-ironic | 07:17 | |
| *** pas-ha has joined #openstack-ironic | 07:18 | |
| pas-ha | morning all | 07:18 |
| *** sambetts has joined #openstack-ironic | 07:19 | |
| *** e0ne has quit IRC | 07:20 | |
| *** e0ne has joined #openstack-ironic | 07:21 | |
| *** alexpilotti has joined #openstack-ironic | 07:21 | |
| *** shuquan has quit IRC | 07:22 | |
| *** aginwala has joined #openstack-ironic | 07:25 | |
| *** alexpilotti has quit IRC | 07:25 | |
| *** shuquan has joined #openstack-ironic | 07:25 | |
| *** ukalifon1 has joined #openstack-ironic | 07:26 | |
| *** ionutbalutoiu has quit IRC | 07:29 | |
| *** dims has joined #openstack-ironic | 07:31 | |
| *** dims has quit IRC | 07:52 | |
| *** shuquan has quit IRC | 07:53 | |
| *** shuquan has joined #openstack-ironic | 07:54 | |
| *** e0ne has quit IRC | 07:56 | |
| *** chenke_ has joined #openstack-ironic | 07:59 | |
| *** chenke has quit IRC | 07:59 | |
| *** alexpilotti has joined #openstack-ironic | 08:00 | |
| *** alexpilotti has quit IRC | 08:05 | |
| *** jerrygb has joined #openstack-ironic | 08:06 | |
| *** jerrygb has quit IRC | 08:12 | |
| *** deray has quit IRC | 08:20 | |
| *** athomas has joined #openstack-ironic | 08:21 | |
| *** alexpilotti has joined #openstack-ironic | 08:21 | |
| *** ukalifon1 has quit IRC | 08:22 | |
| *** alexpilotti has quit IRC | 08:26 | |
| *** MattMan has quit IRC | 08:26 | |
| chenke_ | Haomeng: are u avaiable? | 08:27 |
| Haomeng | chenke_: yes | 08:27 |
| *** ionutbalutoiu has joined #openstack-ironic | 08:27 | |
| chenke_ | Haomeng: does ironic support assign ip whole deploy? | 08:27 |
| Haomeng | chenke_: that depends on image and dhcp settings I think | 08:28 |
| chenke_ | Haomeng: i mean fixed ip assign to the destination baremetal server | 08:28 |
| chenke_ | Haomeng: could you tell more details about that? | 08:29 |
| Haomeng | chenke_: for image, it is better that we create it with dhcp elements - dhcp-all-interfaces | 08:29 |
| Haomeng | chenke_: did you run dib with 'dhcp-all-interfaces' elements | 08:29 |
| chenke_ | Haomeng: yes. i did. now the baremetal sever boot up with ramdom ip, is it possible to give it a fix ip to baremetal server? | 08:31 |
| *** MattMan has joined #openstack-ironic | 08:32 | |
| chenke_ | Haomeng: eg, i can give a fixed ip to kvm instance, can we have it in ironic? | 08:35 |
| Haomeng | chenke_: yes, maybe we can do via confdrive, but I did not test it | 08:36 |
| Haomeng | chenke_: and we can debug if the baremetal dhcp request udp packages are sent to neutron dhcp or not | 08:37 |
| *** dtantsur|afk is now known as dtantsur | 08:37 | |
| *** yonglihe has quit IRC | 08:41 | |
| *** alexpilotti has joined #openstack-ironic | 08:42 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Migrate to using keystoneauth Sessions https://review.openstack.org/236982 | 08:42 |
| *** ifarkas has joined #openstack-ironic | 08:43 | |
| Haomeng | chenke_: run tcpdump | 08:45 |
| *** alexpilotti has quit IRC | 08:47 | |
| *** aginwala has quit IRC | 08:49 | |
| *** deray has joined #openstack-ironic | 08:50 | |
| *** aginwala has joined #openstack-ironic | 08:53 | |
| *** links has quit IRC | 08:55 | |
| *** ukalifon has joined #openstack-ironic | 08:58 | |
| aarefiev | morning folks! | 08:58 |
| chenke_ | Haomeng: i got it. tku! | 08:59 |
| Haomeng | chenke_: ok | 08:59 |
| *** alexpilotti has joined #openstack-ironic | 09:00 | |
| *** aginwala has quit IRC | 09:01 | |
| *** aginwala has joined #openstack-ironic | 09:01 | |
| *** e0ne has joined #openstack-ironic | 09:03 | |
| *** alexpilotti has quit IRC | 09:05 | |
| *** links has joined #openstack-ironic | 09:09 | |
| *** aginwala_ has joined #openstack-ironic | 09:10 | |
| *** mbound_ has joined #openstack-ironic | 09:10 | |
| dtantsur | aarefiev, o/ | 09:12 |
| *** ndipanov has quit IRC | 09:12 | |
| *** ndipanov has joined #openstack-ironic | 09:13 | |
| *** shakamunyi has joined #openstack-ironic | 09:14 | |
| *** aginwala has quit IRC | 09:14 | |
| *** aginwala_ has quit IRC | 09:15 | |
| *** _degorenko|afk is now known as degorenko | 09:15 | |
| *** shuquan has quit IRC | 09:20 | |
| *** apevec has joined #openstack-ironic | 09:22 | |
| *** shakamunyi has quit IRC | 09:23 | |
| *** purp has quit IRC | 09:23 | |
| *** shuquan has joined #openstack-ironic | 09:23 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Add Mitaka release notes page https://review.openstack.org/254336 | 09:24 |
| *** jcoufal has joined #openstack-ironic | 09:25 | |
| *** pradipta has quit IRC | 09:26 | |
| *** purp has joined #openstack-ironic | 09:26 | |
| *** kozhukalov_ has joined #openstack-ironic | 09:27 | |
| *** changzhi has joined #openstack-ironic | 09:28 | |
| changzhi | hi, did anyone met the problem when boot a vm from nova? the error is "No valid host was found. There are not enough hosts available" | 09:32 |
| changzhi | detail info at: http://paste.openstack.org/show/481126/ | 09:32 |
| dtantsur | changzhi, hi, please see http://docs.openstack.org/developer/ironic/deploy/troubleshooting.html#nova-returns-no-valid-host-was-found-error | 09:32 |
| *** electrofelix has joined #openstack-ironic | 09:33 | |
| changzhi | dtantsur: thx you :-) | 09:33 |
| *** cemason has joined #openstack-ironic | 09:34 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic: Remove wsgi reset_pool_size_to_default test https://review.openstack.org/254637 | 09:38 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic: Remove wsgi reset_pool_size_to_default test https://review.openstack.org/254637 | 09:39 |
| *** mkovacik has joined #openstack-ironic | 09:41 | |
| *** mkovacik has quit IRC | 09:50 | |
| *** athomas has quit IRC | 09:51 | |
| *** athomas has joined #openstack-ironic | 09:58 | |
| *** alexpilotti has joined #openstack-ironic | 10:01 | |
| *** zhenguo has quit IRC | 10:03 | |
| *** alexpilotti has quit IRC | 10:06 | |
| *** dtantsur is now known as dtantsur|brb | 10:08 | |
| *** mgould has joined #openstack-ironic | 10:13 | |
| mgould | morning Ironic! | 10:13 |
| *** jistr has joined #openstack-ironic | 10:14 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Clarify what changes need a release note https://review.openstack.org/254659 | 10:17 |
| vdrok | morning mgould | 10:17 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Clarify what changes need a release note https://review.openstack.org/254659 | 10:19 |
| yuriyz | morning Ironic | 10:26 |
| *** changzhi has quit IRC | 10:28 | |
| *** hoangcx has quit IRC | 10:28 | |
| *** mgould has quit IRC | 10:32 | |
| *** mgould has joined #openstack-ironic | 10:36 | |
| *** links has quit IRC | 10:37 | |
| *** shuquan has quit IRC | 10:51 | |
| *** Nisha has quit IRC | 10:53 | |
| *** links has joined #openstack-ironic | 10:58 | |
| *** masterbound has joined #openstack-ironic | 11:02 | |
| *** openstackgerrit has quit IRC | 11:02 | |
| *** openstackgerrit has joined #openstack-ironic | 11:03 | |
| *** lucas-dinner is now known as lucasagomes | 11:04 | |
| *** mbound_ has quit IRC | 11:05 | |
| *** amotoki has quit IRC | 11:07 | |
| *** mgoddard has joined #openstack-ironic | 11:07 | |
| *** deray has quit IRC | 11:10 | |
| *** mkovacik has joined #openstack-ironic | 11:15 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-specs: Implement libvirt-based driver for virtual HW https://review.openstack.org/254421 | 11:18 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic-specs: Implement libvirt-based driver for virtual HW https://review.openstack.org/254421 | 11:24 |
| *** masterbound has quit IRC | 11:29 | |
| *** Haomeng has quit IRC | 11:29 | |
| *** mbound_ has joined #openstack-ironic | 11:29 | |
| *** Haomeng has joined #openstack-ironic | 11:30 | |
| *** mgould has quit IRC | 11:31 | |
| *** dtantsur|brb is now known as dtantsur | 11:31 | |
| openstackgerrit | Syed Ismail Faizan Barmawer proposed openstack/ironic: Refactor disk partitioner code from ironic and use ironic-lib. https://review.openstack.org/184443 | 11:38 |
| *** Haomeng has quit IRC | 11:43 | |
| *** Haomeng has joined #openstack-ironic | 11:44 | |
| *** moorryan has quit IRC | 11:44 | |
| *** mgould has joined #openstack-ironic | 11:44 | |
| * mgould returns from losing Internet access | 11:45 | |
| mgould | if someone answered my questions above while I was gone, could you please /msg me the answer? | 11:45 |
| *** mikal has quit IRC | 11:46 | |
| *** mikal has joined #openstack-ironic | 11:47 | |
| dtantsur | mgould, morning! sorry, I don't see your questions, maybe they got lost as well | 11:47 |
| *** Haomeng has quit IRC | 11:47 | |
| dtantsur | morning vdrok and yuriyz too | 11:47 |
| mgould | dtantsur, morning! | 11:47 |
| *** Haomeng has joined #openstack-ironic | 11:47 | |
| mgould | I was asking about https://review.openstack.org/#/c/254421/ | 11:48 |
| mgould | does it only affect a driver that we currently use only for virtual machines? | 11:49 |
| mgould | or does libvirt handle physical hardware as well? | 11:49 |
| dtantsur | mgould, it doesn't, so it gives me some doubts | 11:49 |
| dtantsur | I'm not 100% convinced we need one more virtual driver (unless we aim for replacing all existent) | 11:49 |
| sambetts | I thought our ssh driver already talked to libvirt? | 11:50 |
| sambetts | o/ morning everyone | 11:50 |
| mgould | sambetts, morning! | 11:50 |
| dtantsur | morning sambetts | 11:51 |
| dtantsur | sambetts, it used ssh + virsh, folks are proposing directly using python API IIUC | 11:51 |
| sambetts | ooh, yeah then if we do that we should be replacing the ssh driver | 11:51 |
| sambetts | we don't need two virtual hardware drivers surly? | 11:51 |
| mgould | sambetts, Pavlo suggests replacing the ssh driver with libvirt lower down | 11:52 |
| mgould | possibly with a wrapper to maintain backwards compatibility | 11:52 |
| dtantsur | I'm not against replacing it, if it supports the same feature set | 11:52 |
| dtantsur | but I don't see the "replace" bit in the spec | 11:53 |
| dtantsur | (I didn't read it carefully though) | 11:53 |
| sambetts | a libvirt driver would certainly sound less confusing than an ssh driver :-P | 11:53 |
| *** ionutbalutoiu has quit IRC | 11:53 | |
| mgould | dtantsur, see under "Work Items" and "Testing" | 11:53 |
| *** ionutbalutoiu has joined #openstack-ironic | 11:53 | |
| mgould | sambetts, I like the idea of not maintaining a bash/python mixture | 11:53 |
| dtantsur | gotcha. I think this "possible future" is the only approach we can discuss | 11:54 |
| mgould | but I'm not yet familiar with the existing code, so I don't know how big a problem it is | 11:54 |
| mgould | dtantsur, you think we should replace the existing ssh driver or leave it alone, but not maintain ssh and libvirt drivers in parallel? | 11:54 |
| dtantsur | mgould, yes | 11:55 |
| *** thrash|g0ne is now known as thrash | 11:55 | |
| mgould | makes sense! | 11:56 |
| *** alexpilotti has joined #openstack-ironic | 12:02 | |
| *** e0ne has quit IRC | 12:03 | |
| *** masterbound has joined #openstack-ironic | 12:05 | |
| *** alexpilotti has quit IRC | 12:06 | |
| *** alexpilotti has joined #openstack-ironic | 12:06 | |
| *** mbound_ has quit IRC | 12:08 | |
| *** jerrygb has joined #openstack-ironic | 12:09 | |
| *** e0ne has joined #openstack-ironic | 12:11 | |
| *** jerrygb has quit IRC | 12:13 | |
| *** apevec has left #openstack-ironic | 12:17 | |
| *** mgould has quit IRC | 12:22 | |
| openstackgerrit | Merged openstack/ironic: Remove wsgi reset_pool_size_to_default test https://review.openstack.org/254637 | 12:28 |
| *** sinval has joined #openstack-ironic | 12:33 | |
| sinval | morning Ironic | 12:33 |
| dtantsur | morning sinval | 12:34 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/254471 | 12:35 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/254494 | 12:36 |
| *** mgould has joined #openstack-ironic | 12:36 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Add hardware properties API to management interface https://review.openstack.org/209612 | 12:36 |
| *** links has quit IRC | 12:36 | |
| *** shuquan has joined #openstack-ironic | 12:36 | |
| *** shuquan has quit IRC | 12:41 | |
| *** samueldmq has joined #openstack-ironic | 12:44 | |
| samueldmq | betherly: thiagop, sinval, afaranha and liliars are some folks who can help you to understand a bit more about oneviewclient :) | 12:48 |
| sinval | hey samueldmq o/ | 12:48 |
| samueldmq | sinval: hey | 12:48 |
| betherly | Thanks samueldmq | 12:49 |
| *** Haomeng has quit IRC | 12:49 | |
| samueldmq | sinval: betherly works with me and she found the client you guys developed very interesting | 12:49 |
| samueldmq | betherly: yw | 12:49 |
| *** Haomeng has joined #openstack-ironic | 12:49 | |
| sinval | samueldmq, betherly oh, how can I help you? | 12:49 |
| betherly | sinval: interested in what you did with the oneview client! Looks awesome | 12:50 |
| betherly | I'm working on the ironic horizon plugin now | 12:50 |
| sinval | betherly: are you serious? hahaha, thanks | 12:51 |
| sinval | betherly: oh, cool, you're working on ironic-webclient am I right? | 12:51 |
| betherly | sinval: nope the ironic-ui although it's connected to the webclient so both really | 12:52 |
| betherly | The webclient is standalone | 12:52 |
| *** mgoddard has quit IRC | 12:52 | |
| betherly | The plugin is for horizon | 12:52 |
| sinval | betherly, oh, I see | 12:54 |
| *** mgould has quit IRC | 12:58 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Follow up patch to correct code-contribute-guide https://review.openstack.org/254739 | 13:00 |
| *** ionutbalutoiu has quit IRC | 13:01 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Properly report preprocessing errors to a user https://review.openstack.org/254740 | 13:02 |
| *** ionutbalutoiu has joined #openstack-ironic | 13:02 | |
| dtantsur | jroll, backport of the pyudev fix to stable/liberty: https://review.openstack.org/#/c/254612/ | 13:04 |
| sinval | betherly: so, do you know about HP OneView? and that we have an Ironic driver for it? | 13:04 |
| sinval | betherly: the oneviewclient is basically a library that the Ironic oneview family of driver use to work with OneView | 13:06 |
| sinval | betherly: if the driver needs to get information about Server Hardwares, Server Profiles or any other resource of OneView, it will call the oneviewclient and the oneviewclient take care of interfacing with REST and things like that | 13:07 |
| *** dprince has joined #openstack-ironic | 13:07 | |
| sinval | betherly: is there something more specific that I can clarify? | 13:08 |
| *** ipukha has joined #openstack-ironic | 13:08 | |
| betherly | sinval: yes i have been looking at it! thanks for the extra information thats really helpful! | 13:11 |
| dtantsur | lucasagomes, could you review https://review.openstack.org/#/c/247514/ please? fixes collecting logs for IPA inspection | 13:12 |
| sinval | betherly: the oneviewclient has a bunch of workflow regarding complicated tasks, like cloning a Server Profile and applying the clone to a Server Hardware, things more related to the driver itself, it really make the driver code more readable and simple | 13:13 |
| lucasagomes | dtantsur, sure! Will do | 13:13 |
| sinval | betherly: sorry for not having a documentation for the oneviewclient, is part of our future working, I'm glad that you liked it, if you have some ideas for improvements please let me know | 13:14 |
| sinval | betherly: and if you need some help feel free to ping me :) | 13:15 |
| *** amotoki has joined #openstack-ironic | 13:15 | |
| *** dims has joined #openstack-ironic | 13:16 | |
| betherly | sinval: thanks i definitely will do that! | 13:17 |
| betherly | sinval: appreciate you getting in touch with me about it! | 13:18 |
| sinval | betherly: my pleasure | 13:19 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Properly report preprocessing errors to a user https://review.openstack.org/254740 | 13:20 |
| *** baoli has joined #openstack-ironic | 13:22 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Add power and thermal limits support https://review.openstack.org/251957 | 13:23 |
| lucasagomes | dtantsur, reviewed https://review.openstack.org/#/c/247514 | 13:24 |
| lucasagomes | dtantsur, lemme know if you agree with it, other than that it LGTM | 13:24 |
| dtantsur | lucasagomes, ack, will fix in a few | 13:25 |
| *** baoli_ has joined #openstack-ironic | 13:25 | |
| lucasagomes | cool | 13:25 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Properly report preprocessing errors to a user https://review.openstack.org/254740 | 13:25 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Intel Node Manager policies support https://review.openstack.org/214657 | 13:27 |
| *** baoli has quit IRC | 13:28 | |
| *** baoli_ has quit IRC | 13:31 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Fix "logs" inspection collector when logs contain non-ascii symbols https://review.openstack.org/247514 | 13:32 |
| dtantsur | lucasagomes, ^^ | 13:32 |
| *** baoli has joined #openstack-ironic | 13:32 | |
| lucasagomes | dtantsur, thanks | 13:34 |
| *** baoli has quit IRC | 13:37 | |
| *** baoli has joined #openstack-ironic | 13:37 | |
| *** mgoddard has joined #openstack-ironic | 13:38 | |
| *** baoli has quit IRC | 13:42 | |
| *** jerrygb has joined #openstack-ironic | 13:44 | |
| jroll | dtantsur: +2 | 13:47 |
| jroll | morning everyone | 13:47 |
| lucasagomes | jroll, hi there | 13:48 |
| lucasagomes | jroll, question, I backported a patch to stable/liberty to fix the iPXE stuff | 13:48 |
| lucasagomes | jroll, but we need two patches merged at the same time to avoid breakage | 13:49 |
| lucasagomes | I squashed them, is it acceptable? | 13:49 |
| lucasagomes | jroll, https://review.openstack.org/#/c/254759/ | 13:49 |
| jroll | lucasagomes: why squash? the process is generally don't squash unless you need to | 13:49 |
| lucasagomes | I found this old thread here http://lists.openstack.org/pipermail/openstack-stable-maint/2014-August/002541.html | 13:49 |
| lucasagomes | seems it's fine | 13:49 |
| lucasagomes | jroll, because fixing one bug introduced another | 13:50 |
| jroll | oh, the first one had a bug yeah | 13:50 |
| jroll | ok | 13:50 |
| jroll | totally cool | 13:50 |
| lucasagomes | ack thanks | 13:50 |
| jroll | thank YOU | 13:50 |
| *** Nisha has joined #openstack-ironic | 13:54 | |
| *** ionutbalutoiu has quit IRC | 13:55 | |
| *** naohirot has quit IRC | 13:56 | |
| *** kalpase has left #openstack-ironic | 13:59 | |
| *** rloo has joined #openstack-ironic | 14:01 | |
| *** mgoddard_ has joined #openstack-ironic | 14:02 | |
| *** mgoddard has quit IRC | 14:02 | |
| *** derekh has joined #openstack-ironic | 14:02 | |
| *** mgoddard has joined #openstack-ironic | 14:06 | |
| *** mgoddard_ has quit IRC | 14:06 | |
| jroll | lucasagomes: +2 on that btw | 14:07 |
| lucasagomes | jroll, cheers man! | 14:07 |
| *** ionutbalutoiu has joined #openstack-ironic | 14:08 | |
| rloo | good morning everyone. | 14:09 |
| rloo | Is the gate still broken? The etherpad sez it is but I think it is fixed? | 14:09 |
| lucasagomes | rloo, hi there, morning | 14:10 |
| lucasagomes | I think gate is fine, /me looks | 14:10 |
| rloo | hi lucasagomes | 14:11 |
| sinval | morning rloo and jroll | 14:11 |
| rloo | hi sinval, jroll | 14:12 |
| *** [1]cdearborn has joined #openstack-ironic | 14:21 | |
| rloo | i know how this got in: http://docs.openstack.org/developer/ironic/deploy/troubleshooting.html#nova-returns-no-valid-host-was-found-error | 14:23 |
| rloo | BUT anyone know how a similar thing got in http://docs.openstack.org/admin-guide-cloud/baremetal.html? | 14:24 |
| jroll | rloo: https://github.com/openstack/openstack-manuals/commit/23bd29a2821b1e2adfc07cd7fff8fdd7911425b9 | 14:30 |
| rloo | thx jroll. | 14:30 |
| rloo | it makes no sense to have two different versions of the same text. | 14:31 |
| jroll | yeah, totally agree | 14:31 |
| jroll | we need to do something where that guide can pull from our docs or something | 14:32 |
| rloo | jroll: I'm going to remove it from our dev; put a link to the official stuff. | 14:32 |
| jroll | rloo: sure, that seems like a good short term fix | 14:33 |
| rloo | jroll: speaking of docs, lucas raised an interesting question: https://review.openstack.org/#/c/247701/6/doc/source/deploy/cleaning.rst | 14:33 |
| rloo | jroll: we have versioned dev docs, do we want to continue mentioning things/differences between releaes | 14:33 |
| rloo | jroll: and if we do, only stable releases or our semvers too? | 14:33 |
| jroll | rloo: yeah, it's a good question, I'd prefer not to continue doing that | 14:34 |
| NobodyCam | Good morning jroll rloo lucasagomes dtantsur sambetts jlvillal vdrok sinval betherly JayF and the rest of the Ironicers | 14:35 |
| rloo | jroll: that's what i thought. but if we want our install guide to be part of openstack stuff -- it doesn't look like their install guide is versioned. | 14:35 |
| rloo | morning NobodyCam | 14:35 |
| lucasagomes | NobodyCam, morning! | 14:35 |
| sinval | morning NobodyCam | 14:35 |
| NobodyCam | :) | 14:35 |
| betherly | morning NobodyCam :) how are you? | 14:35 |
| rloo | jroll: but looking through the unversioned install guide, i don't know how/if they deal with changes between releases. | 14:35 |
| NobodyCam | in much less pain this morning :) | 14:35 |
| NobodyCam | thankk you for asking | 14:36 |
| jroll | rloo: oh, idk then. | 14:36 |
| jroll | rloo: the official docs are versioned: http://docs.openstack.org/kilo/ | 14:37 |
| jroll | just not the admin guide, I guess? | 14:37 |
| rloo | jroll: oh. they are versioned. i was looking at the 'master' (unversioned). | 14:38 |
| rloo | jroll: so yeah, i think we are good then. | 14:38 |
| rloo | back in a few. i need to get caffeine... | 14:39 |
| jroll | cool | 14:39 |
| *** lucasagomes is now known as lucas-hungry | 14:42 | |
| NobodyCam | mmmmm coffee | 14:42 |
| vdrok | morning NobodyCam sinval jroll rloo betherly ! | 14:43 |
| *** kkoski has joined #openstack-ironic | 14:43 | |
| NobodyCam | :) | 14:43 |
| sinval | hey vdrok :) | 14:43 |
| rloo | hi vdrok! | 14:45 |
| NobodyCam | w00t tests work again :) | 14:47 |
| NobodyCam | :p | 14:47 |
| rloo | lucas-hungry etc: gate looks good; i updated the etherpad! | 14:47 |
| rloo | NobodyCam: :) | 14:47 |
| *** kkoski has quit IRC | 14:49 | |
| *** dims_ has joined #openstack-ironic | 14:52 | |
| *** derekh is now known as derekh_afk | 14:53 | |
| *** dims has quit IRC | 14:55 | |
| *** baoli has joined #openstack-ironic | 14:58 | |
| TheJulia | Good morning | 14:58 |
| *** yonglihe has joined #openstack-ironic | 15:00 | |
| *** masterbound has quit IRC | 15:00 | |
| *** mgoddard has quit IRC | 15:00 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Support to wait for node deployment to complete https://review.openstack.org/254245 | 15:01 |
| *** longnv has quit IRC | 15:03 | |
| *** mgoddard has joined #openstack-ironic | 15:03 | |
| jroll | hey folks, let's get this done this week, maybe even today :) https://review.openstack.org/#/c/241294/ | 15:08 |
| *** baoli has quit IRC | 15:08 | |
| *** jerrygb has quit IRC | 15:09 | |
| *** mbound_ has joined #openstack-ironic | 15:10 | |
| *** jerrygb has joined #openstack-ironic | 15:10 | |
| rloo | jroll: will look at it today | 15:10 |
| NobodyCam | morning TheJulia | 15:12 |
| *** jerrygb has quit IRC | 15:15 | |
| *** lucas-hungry is now known as lucasagomes | 15:17 | |
| *** sinval has quit IRC | 15:19 | |
| dtantsur | morning jroll, NobodyCam, rloo and everyone | 15:22 |
| NobodyCam | :) | 15:22 |
| openstackgerrit | Merged openstack/bifrost: Properly specify dns in network_metadata https://review.openstack.org/253976 | 15:23 |
| *** jerrygb has joined #openstack-ironic | 15:24 | |
| rloo | hi dtantsur! | 15:24 |
| *** Nisha has quit IRC | 15:26 | |
| *** Nisha has joined #openstack-ironic | 15:27 | |
| *** romcheg1 has quit IRC | 15:28 | |
| *** romcheg1 has joined #openstack-ironic | 15:30 | |
| *** kkoski has joined #openstack-ironic | 15:30 | |
| *** Nisha has quit IRC | 15:31 | |
| *** Nisha has joined #openstack-ironic | 15:32 | |
| *** sinval has joined #openstack-ironic | 15:32 | |
| rloo | jroll: low hanging doc fruit for someone: https://bugs.launchpad.net/ironic/+bug/1523962 | 15:32 |
| openstack | Launchpad bug 1523962 in Ironic "documentation doesn't need to mention differences due to releases" [Low,New] | 15:32 |
| *** mtanino has joined #openstack-ironic | 15:33 | |
| jroll | yum, doc fruit | 15:33 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix boolean conditionals for Ansible 2.0 with git repo copies https://review.openstack.org/253639 | 15:34 |
| rloo | jroll: yeah, low hanging and sweet, not on-the-ground-rotting ;) | 15:35 |
| jroll | hehe | 15:35 |
| *** mgould has joined #openstack-ironic | 15:42 | |
| *** jistr is now known as jistr|mtg | 15:43 | |
| *** mgould has quit IRC | 15:46 | |
| *** ionutbalutoiu has quit IRC | 15:47 | |
| lucasagomes | rloo, oh that would be good | 15:50 |
| lucasagomes | that's related to that comment I added to ur patch today? | 15:50 |
| rloo | lucasagomes: about the versions in our docs? | 15:51 |
| lucasagomes | yeah | 15:51 |
| rloo | lucasagomes: yup! | 15:51 |
| lucasagomes | rloo, nice! | 15:51 |
| rloo | lucasagomes: i was going to make the change but i figure it should be a good one for someone else, and i should focus on reviews ;) | 15:51 |
| *** derekh_afk has quit IRC | 15:51 | |
| lucasagomes | heh fair enough | 15:52 |
| openstackgerrit | Merged openstack/ironic-lib: Updated from global requirements https://review.openstack.org/254494 | 15:52 |
| *** rkharya has joined #openstack-ironic | 15:56 | |
| krotscheck | BadCub: What's the magic word on that hardware lab request? | 15:57 |
| *** garthb has joined #openstack-ironic | 15:58 | |
| *** mgould has joined #openstack-ironic | 15:59 | |
| *** kkoski has quit IRC | 16:01 | |
| sinval | ping jlvillal and vdrok | 16:02 |
| *** kkoski has joined #openstack-ironic | 16:03 | |
| vdrok | sinval, pong | 16:03 |
| *** mbound_ has quit IRC | 16:04 | |
| sinval | jlvillal, vdrok: regarding your comments about the self.update(http_method=http_method) call (https://review.openstack.org/#/c/252150/3/ironicclient/common/base.py) | 16:04 |
| sinval | jlvillal, vdrok: that call should be for the child's update method which has the signature with http_method | 16:05 |
| BadCub | krotscheck: I have not started working on that because *release planning*. I have it on my agenda to start working on later this week. | 16:05 |
| krotscheck | BadCub: Alrightey! Will ping back after the shutdown, since I'm on vaca next week | 16:05 |
| sinval | jlvillal, vdrok: but, I realized that refatoring vendor_passthru is a bit more complicated than other methods, it needs more specific test cases to keep the behaviour during the refactoring and I prefer dealing with it in another patch, so the current will only handle create, update, delete and get methods | 16:05 |
| sinval | jlvillal, vdrok: what do you guys think? | 16:06 |
| BadCub | krotscheck: will do. Will also keep bethany in the loop | 16:06 |
| *** mbound_ has joined #openstack-ironic | 16:06 | |
| krotscheck | BadCub: Thanks! Also, I think she doesn't like that version of her name. | 16:06 |
| * krotscheck pings betherly to confirm? | 16:07 | |
| BadCub | oh? what is the new version? | 16:07 |
| BadCub | ugh, I see. Bad typo on my part | 16:07 |
| vdrok | sinval, hmmm, that is not clear at all at the first glance :) I'd vote for different naming to avoid confusion, like _update in base and update in children as it was before | 16:07 |
| BadCub | sorry betherly *grimace* | 16:07 |
| sinval | vdrok: I do agree with that, and dtantsur suggested that we should do it and I will do it | 16:08 |
| jlvillal | sinval, vdrok +1 on using _update | 16:09 |
| jlvillal | In the base class | 16:09 |
| vdrok | sinval, I'm ok with that, just add a todo or something :) | 16:09 |
| sinval | vdrok, jlvillal: will do, thank you | 16:09 |
| vdrok | thanks sinval :) | 16:10 |
| jlvillal | sinval, and the other functions too :) Not just _update | 16:10 |
| sinval | vdrok, jlvillal: I'll send another update to that patch regarding _update, _get, _create and _delete | 16:10 |
| jlvillal | sinval, Thanks | 16:10 |
| sinval | my pleasure | 16:11 |
| betherly | krotscheck BadCub thanks! | 16:17 |
| betherly | krotscheck: confirmed! bethany isnt my favourite but to be fair i brought it on myself choosing betherly as my IRC nick. no offence taken. oh...and if i did i would probably be appalling at this job ;) | 16:18 |
| BadCub | betherly: lol | 16:19 |
| *** baoli has joined #openstack-ironic | 16:23 | |
| *** rkharya has quit IRC | 16:28 | |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/254471 | 16:34 |
| openstackgerrit | Merged openstack/ironic: Clarify what changes need a release note https://review.openstack.org/254659 | 16:34 |
| *** Nisha has quit IRC | 16:37 | |
| *** Nisha has joined #openstack-ironic | 16:38 | |
| betherly | jroll: I have added governance and project setup for ironic-ui where the plugin will live. It will require the PTL of ironic to +1 to confirm you are happy with the setup. Thanks :) | 16:41 |
| betherly | jroll: https://review.openstack.org/#/c/254845/ | 16:41 |
| betherly | jroll: https://review.openstack.org/#/c/254853/ | 16:42 |
| *** sinval is now known as sinval-afk | 16:42 | |
| jroll | betherly: this is the horizon plugin? | 16:42 |
| betherly | jroll: thats right :) | 16:42 |
| jroll | k | 16:42 |
| jroll | thanks, will look | 16:42 |
| betherly | thanks! | 16:43 |
| *** boris-42_ has joined #openstack-ironic | 16:43 | |
| *** cemason has quit IRC | 16:46 | |
| *** openstackgerrit has quit IRC | 16:47 | |
| *** openstackgerrit has joined #openstack-ironic | 16:47 | |
| *** garthb_ has joined #openstack-ironic | 16:51 | |
| *** garthb has quit IRC | 16:51 | |
| *** degorenko is now known as _degorenko|afk | 16:53 | |
| jroll | rloo: wdyt about my responses to your comments on https://review.openstack.org/#/c/241294/ | 16:54 |
| rloo | jroll: sec, looking... | 16:54 |
| jroll | if you're good with that, I can re-spin unless krtaylor has a moment | 16:55 |
| *** ukalifon has quit IRC | 16:55 | |
| *** mgoddard has quit IRC | 16:59 | |
| rloo | jroll: commented. not good with the first bullet of yours. | 17:00 |
| rloo | jroll: oh, wait, the second bullet. i've already forgotten what i commented! | 17:00 |
| rloo | jroll: well, just read, cuz i don't know what i said. | 17:01 |
| *** vishwanathj has joined #openstack-ironic | 17:01 | |
| *** praneshp has quit IRC | 17:03 | |
| *** alex_xu has quit IRC | 17:04 | |
| jroll | rloo: words are hard, check what I've got now | 17:05 |
| *** mkovacik has quit IRC | 17:05 | |
| rloo | jroll: great! | 17:07 |
| jroll | rloo: cool, I'll respin it | 17:08 |
| jroll | sick of seeing that spec in the queue :) | 17:08 |
| rloo | jroll: thx. | 17:08 |
| *** sinval-afk is now known as sinval | 17:09 | |
| *** alex_xu has joined #openstack-ironic | 17:10 | |
| *** amotoki has quit IRC | 17:14 | |
| *** Nisha has quit IRC | 17:16 | |
| *** Nisha has joined #openstack-ironic | 17:17 | |
| *** mgould has quit IRC | 17:18 | |
| *** jistr|mtg is now known as jistr | 17:21 | |
| dtantsur | leaving for today, see you tomorrow | 17:22 |
| NobodyCam | night dtantsur | 17:22 |
| sambetts | o/ dtantsur | 17:22 |
| *** dtantsur is now known as dtantsur|afk | 17:22 | |
| lucasagomes | dtantsur|afk, see ya later | 17:24 |
| *** yarkot has joined #openstack-ironic | 17:25 | |
| *** ifarkas has quit IRC | 17:25 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Ironic third party CI spec https://review.openstack.org/241294 | 17:29 |
| jroll | rloo: found a bunch more stuff to change, mostly formatting nits and such :/ | 17:29 |
| jroll | but there you have it | 17:29 |
| * jroll waits for render | 17:29 | |
| * sambetts calls it a day | 17:35 | |
| sambetts | cya everyone o/ | 17:36 |
| *** sambetts is now known as sambetts-afk | 17:36 | |
| NobodyCam | night sambetts-afk | 17:36 |
| sambetts-afk | night NobodyCam o/ | 17:36 |
| *** aginwala has joined #openstack-ironic | 17:36 | |
| lucasagomes | sambetts-afk, night | 17:37 |
| *** garthb_ has quit IRC | 17:38 | |
| *** mbound_ has quit IRC | 17:39 | |
| *** aginwala has quit IRC | 17:42 | |
| lucasagomes | jroll, rloo remember me please, re 3rd party CI. A driver with no vendor behind it will be removed from tree? | 17:43 |
| lucasagomes | wol, iboot, etc... | 17:43 |
| jroll | lucasagomes: if it doesn't get third party CI, yes | 17:44 |
| jroll | for example, I'm working on finding someone to CI ipmitool | 17:44 |
| lucasagomes | jroll, right, I don't think we are actually able to CI wol, nobody will want to do that (I guess) | 17:45 |
| jroll | lucasagomes: for those in particular, I think the ironic team could provide the repository and such | 17:45 |
| jroll | yeah | 17:45 |
| *** aginwala has joined #openstack-ironic | 17:45 | |
| lucasagomes | I mean I use it in my machines and I can guarnatee it's working but I can't CI it | 17:45 |
| jroll | so for wol, I would propose a repo openstack/ironic-driver-wol that people can pip install | 17:45 |
| lucasagomes | jroll, right, yeah an extra repo seems fine | 17:45 |
| jroll | cool | 17:46 |
| lucasagomes | yeah I'm not super happy about it, but I can live with it | 17:46 |
| jroll | I mean... it's only one extra step | 17:46 |
| jroll | and you don't have to repeat it just to check an ironic bug fix | 17:47 |
| jroll | AND - it will help us make sure our driver layer is backwards compat happy, if devs are using out of tree drivers | 17:47 |
| *** cemason has joined #openstack-ironic | 17:47 | |
| jroll | which is actually a nice perk | 17:47 |
| lucasagomes | jroll, yeah OK with that | 17:48 |
| lucasagomes | I may start already setting up a pip repo for that and all | 17:49 |
| *** e0ne has quit IRC | 17:50 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add tests for testing node-delete command https://review.openstack.org/251458 | 17:51 |
| *** cemason has quit IRC | 17:52 | |
| *** ionutbalutoiu has joined #openstack-ironic | 17:54 | |
| *** yarkot has quit IRC | 17:58 | |
| lucasagomes | jroll, devananda I think the stable branches are broken http://logs.openstack.org/59/254759/1/check/gate-ironic-python27/9e4a526/console.html | 17:59 |
| lucasagomes | we need to backport https://review.openstack.org/#/c/213601/ | 17:59 |
| * lucasagomes do it | 17:59 | |
| *** Haomeng|2 has joined #openstack-ironic | 18:00 | |
| *** vishwanathj has quit IRC | 18:00 | |
| jroll | lucasagomes: I don't see why that patch would fix it? | 18:01 |
| *** Haomeng has quit IRC | 18:01 | |
| jroll | looks like an oslo.messaging break to me | 18:01 |
| lucasagomes | jroll, I will test it, but see the bug linked to it | 18:02 |
| * lucasagomes checks | 18:02 | |
| lucasagomes | oh messeging hmmm | 18:02 |
| jroll | 2015-12-08 14:07:48.352 | File "ironic/common/rpc.py", line 78, in <module> | 18:02 |
| jroll | 2015-12-08 14:07:48.352 | class RequestContextSerializer(messaging.RequestContextSerializer): | 18:02 |
| jroll | yeah | 18:02 |
| *** Nisha has quit IRC | 18:03 | |
| lucasagomes | yeah lemme look | 18:03 |
| jroll | wtf | 18:04 |
| jroll | https://github.com/openstack/oslo.messaging/commit/6cd1dcebc0801dc16db5f64c81baf1fe17165c88 | 18:04 |
| jroll | they removed it, but reverted | 18:04 |
| jroll | and now it's failing? | 18:04 |
| dims_ | jroll : oslo.messaging was using debtcollector to decorate the RequestContextSerializer as deprecated and ran into this debt collector bug (https://bugs.launchpad.net/debtcollector/+bug/1520397) so i requested this on ironic master (https://review.openstack.org/#/c/250571/) | 18:05 |
| openstack | Launchpad bug 1520397 in debtcollector "Problem with abstract classes" [Undecided,New] | 18:05 |
| dims_ | and that was merged on master | 18:05 |
| jroll | dims_: oh! thank you | 18:05 |
| lucasagomes | dims_, thanks! | 18:05 |
| * jroll backports | 18:05 | |
| *** jistr has quit IRC | 18:05 | |
| lucasagomes | yeah we problably need to backport that | 18:05 |
| lucasagomes | jroll, you on it? Or want me to do it? | 18:05 |
| jroll | https://review.openstack.org/#/c/254890/ | 18:05 |
| dims_ | i'd recommend backporting https://review.openstack.org/#/c/250571/1/ironic/common/rpc.py,cm to stable and both master and stable will not break from now on as you have your own :) | 18:06 |
| lucasagomes | jroll, cool | 18:06 |
| dims_ | and will work with both older and newer oslo.messaging to boot | 18:06 |
| lucasagomes | let's wait for jenkins | 18:06 |
| *** kozhukalov_ has quit IRC | 18:06 | |
| jroll | yep | 18:06 |
| jroll | thanks again dims_ :) | 18:06 |
| dims_ | yw jroll | 18:06 |
| lucasagomes | dims_, ta much! | 18:07 |
| *** ipukha has quit IRC | 18:09 | |
| jroll | lucasagomes: wanna rebase your ipxe backport on that? | 18:09 |
| lucasagomes | jroll, yeah I will do it | 18:09 |
| jroll | thanks | 18:09 |
| JayF | Do IPA bugs still go in Ironic in LP? | 18:10 |
| JayF | Or did we give it its own bugtracker when we started releasing it? | 18:10 |
| lucasagomes | JayF, I use https://bugs.launchpad.net/ironic-python-agent | 18:12 |
| JayF | 10-4 thanks | 18:12 |
| * JayF filing a bug about /proc/sys and /sys being r/o so he can resolve it with the chroot patch he's about to push up for zer0c00l | 18:13 | |
| * jroll wonders what bug is coming | 18:13 | |
| jroll | oh word | 18:13 |
| lucasagomes | JayF, no more containers? | 18:13 |
| JayF | it'll work like a container | 18:14 |
| JayF | but be implemented by chroot | 18:14 |
| lucasagomes | JayF, cause if we are using chroot, what's the advatange we get? Inst it better to just use the base OS to run IPA ? | 18:14 |
| JayF | lucasagomes: TBH, I don't see it as a huge change | 18:14 |
| *** derekh has joined #openstack-ironic | 18:14 | |
| JayF | lucasagomes: it's just a matter of how we encapsulate the IPA running environment | 18:15 |
| JayF | the change is relatively minor, I think you'll udnerstand better when you see it | 18:15 |
| * jroll brb | 18:15 | |
| lucasagomes | JayF, fair enough... because for some stuff I still see using the baseos > chroot | 18:15 |
| lucasagomes | JayF, in case we need to modprobe some kernel modules in IPA for example | 18:15 |
| lucasagomes | for IPMI or linux-io | 18:15 |
| JayF | lucasagomes: I do that in my container-based IPA today | 18:15 |
| *** derekh has quit IRC | 18:16 | |
| JayF | lucasagomes: you just have to tell nspawn to bugger off... which is what I'm doing for permanent with moving to chroot :) | 18:16 |
| lucasagomes | JayF, you modprobe it in the coreos image right? | 18:16 |
| JayF | when we flash bios | 18:16 |
| lucasagomes | or from within the container? | 18:16 |
| JayF | we load the bios mnfr's module from inside the container | 18:16 |
| lucasagomes | oh interesting | 18:16 |
| JayF | although downstream the last two agents we've pushed to prod have been chroot, for other reasons | 18:16 |
| JayF | yeah :) I've even got patches into nspawn to make more of the security bits options | 18:17 |
| lucasagomes | JayF, RAID? | 18:17 |
| JayF | *optional | 18:17 |
| JayF | lucasagomes: that's related to the bug I'm filing and resolving. | 18:17 |
| lucasagomes | things writing in /sys ? | 18:17 |
| JayF | lucasagomes: bingo. those are the cases I'm filing a bug for and pulling out to chroot | 18:17 |
| JayF | lucasagomes: and I can confirm both work in a chroot | 18:17 |
| lucasagomes | linux-io do write /sys/targets, that's a problem in a containized env | 18:17 |
| lucasagomes | JayF, thanks, yeah I think we can test in chroot | 18:18 |
| lucasagomes | see if that solves for us too | 18:18 |
| JayF | Yep :D | 18:18 |
| JayF | almost certainly will | 18:18 |
| JayF | there's no security in a chroot | 18:18 |
| JayF | https://bugs.launchpad.net/ironic-python-agent/+bug/1524033 | 18:18 |
| openstack | Launchpad bug 1524033 in ironic-python-agent "/sys and /proc/sys r/o from inside IPA Ramdisk" [Undecided,New] | 18:18 |
| lucasagomes | JayF, right, yeah you just "mount -o bind /sys chroot/sys" correct ? | 18:19 |
| JayF | well it looks more systemd than that | 18:19 |
| JayF | but yeah | 18:19 |
| lucasagomes | fair enough | 18:19 |
| JayF | give me literally 2m to finish writing the commit message and I'll push it up :) | 18:19 |
| lucasagomes | yeah it should work with linux-io | 18:19 |
| JayF | oh I'm going to need to reno too | 18:19 |
| lucasagomes | dtantsur|afk, ^ you may be interested on that | 18:20 |
| lucasagomes | JayF, thanks | 18:20 |
| *** cemason has joined #openstack-ironic | 18:21 | |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Run IPA in chroot instead of container in CoreOS https://review.openstack.org/254896 | 18:22 |
| JayF | ^ I couldn't find an IPA release notes file, if one exists and I need to update it please -1 me :) | 18:22 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic: Handle deprecated opts' group correctly https://review.openstack.org/254897 | 18:24 |
| jroll | JayF: reno isn't done for IPA yet | 18:24 |
| jroll | almost forgot :/ | 18:24 |
| *** cemason has quit IRC | 18:24 | |
| lucasagomes | [lucasagomes@lucasagomes ironic-python-agent]$ grep -R reno * | 18:25 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Run IPA in chroot instead of container in CoreOS https://review.openstack.org/254896 | 18:25 |
| lucasagomes | yeah, seems it's not there yet | 18:25 |
| jroll | on my todo list | 18:25 |
| *** shakamunyi has joined #openstack-ironic | 18:25 | |
| jroll | unless JayF wants to do it for me :D | 18:26 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add port option support for ipmitool https://review.openstack.org/254899 | 18:26 |
| *** mgoddard has joined #openstack-ironic | 18:26 | |
| JayF | jroll: /me returns to doing $important_downstream_things | 18:27 |
| jroll | :| | 18:27 |
| jroll | "if a patch doesn't have a release note, does it really exist?" | 18:27 |
| jroll | :P | 18:27 |
| JayF | Lay it on me, how much of a PITA is the migration to reno? | 18:27 |
| JayF | lol | 18:27 |
| jroll | not much | 18:28 |
| jroll | the backfill was the most annoying part | 18:28 |
| vdrok | have a good night everyone! | 18:28 |
| jroll | \o vdrok | 18:28 |
| sinval | night vdrok o/ | 18:29 |
| JayF | Yeah that's sorta what I was afraid the final step would be | 18:29 |
| JayF | "read git log since the beginning of time and write notes" | 18:29 |
| *** dims_ has quit IRC | 18:30 | |
| *** ijw has joined #openstack-ironic | 18:30 | |
| jroll | JayF: I mean, I could do that part :P | 18:31 |
| jroll | I did that for ironic, read every commit message for mitaka so far | 18:31 |
| openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Run IPA in chroot instead of container in CoreOS https://review.openstack.org/254896 | 18:31 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add port option support for ipmitool https://review.openstack.org/254899 | 18:32 |
| *** ijw_ has joined #openstack-ironic | 18:32 | |
| *** ijw has quit IRC | 18:32 | |
| *** jerrygb has quit IRC | 18:39 | |
| *** mkovacik has joined #openstack-ironic | 18:39 | |
| *** e0ne has joined #openstack-ironic | 18:39 | |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/python-ironicclient: Refactoring and removing duplicate code of "base.Manager" heirs without changing the API https://review.openstack.org/252150 | 18:39 |
| *** jerrygb has joined #openstack-ironic | 18:46 | |
| *** aginwala has quit IRC | 18:47 | |
| *** david-ly_ has joined #openstack-ironic | 18:50 | |
| *** david-ly_ is now known as david-lyle_ | 18:52 | |
| *** pas-ha has quit IRC | 18:53 | |
| *** david-lyle has quit IRC | 18:54 | |
| *** athomas has quit IRC | 18:57 | |
| *** penick has joined #openstack-ironic | 18:58 | |
| *** penick_ has joined #openstack-ironic | 19:00 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Support to wait for node deployment to complete https://review.openstack.org/254245 | 19:00 |
| *** openstackgerrit has quit IRC | 19:02 | |
| *** openstackgerrit has joined #openstack-ironic | 19:03 | |
| *** electrofelix has quit IRC | 19:04 | |
| *** penick has quit IRC | 19:04 | |
| *** penick_ is now known as penick | 19:04 | |
| *** cdearborn has joined #openstack-ironic | 19:09 | |
| *** Sukhdev has joined #openstack-ironic | 19:09 | |
| *** ChubYann has joined #openstack-ironic | 19:12 | |
| *** [2]cdearborn has joined #openstack-ironic | 19:12 | |
| *** kozhukalov_ has joined #openstack-ironic | 19:13 | |
| *** david-lyle_ is now known as david-lyle | 19:14 | |
| *** aginwala has joined #openstack-ironic | 19:15 | |
| *** [1]cdearborn has quit IRC | 19:21 | |
| *** praneshp has joined #openstack-ironic | 19:23 | |
| JayF | dtantsur|afk: need a overview of that change to chroot? I want more than just me to understand it :) | 19:23 |
| *** cdearborn has quit IRC | 19:27 | |
| *** [2]cdearborn is now known as cdearborn | 19:27 | |
| zer0c00l | JayF: Thanks. Will take a look at the patch. | 19:27 |
| *** hblixt has joined #openstack-ironic | 19:28 | |
| *** aginwala has quit IRC | 19:29 | |
| *** aginwala has joined #openstack-ironic | 19:30 | |
| devananda | TheJulia: regarding this line -- "{{dib_env_vars | combine({'DIB_RELEASE':dib_os_release}) }}" -- where is combine() coming from? | 19:35 |
| devananda | TheJulia: it's failing in my local env, using ansible 1.9 | 19:35 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add port option support for ipmitool https://review.openstack.org/254899 | 19:37 |
| *** harlowja has quit IRC | 19:38 | |
| *** harlowja has joined #openstack-ironic | 19:38 | |
| lucasagomes | JayF, thought about asking in the review but since you are online, do we need to use that "\2xd" for the systemd mount files? Can't we just use "-" (dash) ? | 19:39 |
| JayF | lucasagomes: For $systemd_reasons, yes | 19:39 |
| JayF | lucasagomes: - is translated to / for purposes of mount units | 19:39 |
| JayF | lucasagomes: so you have to escape it | 19:39 |
| lucasagomes | *sigh* | 19:40 |
| lucasagomes | JayF, thanks. Yeah because it looks quiet ugly | 19:40 |
| JayF | yeah. I thought about moving the container to /opt/ipa instead of /opt/ironic-python-agent ... but that's a more breaking change | 19:40 |
| JayF | even if it is ugly :x | 19:40 |
| lucasagomes | JayF, yeah, fair enough | 19:41 |
| * lucasagomes have to test that patch | 19:41 | |
| NobodyCam | jroll: missed the update doc comment: are you thinking here : http://docs.openstack.org/developer/ironic/deploy/install-guide.html#enrollment-process | 19:42 |
| jroll | NobodyCam: yep | 19:43 |
| NobodyCam | sweet adding now :) | 19:43 |
| jroll | thanks! | 19:43 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Ironic third party CI spec https://review.openstack.org/241294 | 19:45 |
| jroll | rloo: spec had a small formatting thing, also a doc8 thing and fixed your nit ^ | 19:46 |
| jroll | formatting http://docs-draft.openstack.org/94/241294/9/check/gate-ironic-specs-docs/2eb4734//doc/build/html/specs/approved/third-party-ci.html | 19:46 |
| * jroll lunches and stuff | 19:46 | |
| rloo | jroll: oh. Under 'Testing' it still sez 'None'. I must have missed that. | 19:47 |
| jroll | rloo: doh, I missed that | 19:49 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-specs: Ironic third party CI spec https://review.openstack.org/241294 | 19:49 |
| jroll | rloo: fixeded | 19:49 |
| rloo | jroll: thx. will wait for jenkins and then bam. | 19:49 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add port option support for ipmitool https://review.openstack.org/254899 | 19:50 |
| jroll | thanks :) | 19:50 |
| * jroll actually goes away now | 19:50 | |
| *** penick has quit IRC | 19:52 | |
| * NobodyCam steps afk and makes soup ... brb | 19:56 | |
| *** thrash is now known as thrash|biab | 19:58 | |
| lucasagomes | folks I'm going to call it a day, have a great night everyone! | 19:59 |
| NobodyCam | have a good night too lucasagomes | 19:59 |
| TheJulia | devananda: thats a 2.0 only feature, you need to go to 2.0 for it to work :( | 19:59 |
| devananda | TheJulia: oki doki | 20:00 |
| TheJulia | env-setup should make the world happy w/r/t ansible version | 20:00 |
| sinval | night lucasagomes | 20:01 |
| lucasagomes | thanks see y'all | 20:01 |
| *** kozhukalov_ has quit IRC | 20:01 | |
| lucasagomes | devananda, oh quick request if you have some time, mind taking a look at https://review.openstack.org/#/c/253605/ ? That's the indexable json fields stuff, I just want to know if the spec is aligned with what you proposed before | 20:02 |
| lucasagomes | maybe we could have only 1 table for the field names as well that are shared with both cross join tables | 20:03 |
| devananda | lucasagomes: looking | 20:03 |
| lucasagomes | devananda, thanks a lot! (/me trusts deva with db stuff) | 20:03 |
| devananda | lucasagomes: while i'm reading, i'm curious whether you see similarity between this and the tags work | 20:04 |
| *** garthb has joined #openstack-ironic | 20:04 | |
| lucasagomes | devananda, tags is going to do something similar right? The way they structure the db | 20:05 |
| *** aginwala has quit IRC | 20:05 | |
| devananda | lucasagomes: yah. and filtering by tags | 20:05 |
| lucasagomes | devananda, right | 20:05 |
| lucasagomes | devananda, but you think we don't need to make properties & driver_info indexable too? | 20:05 |
| lucasagomes | so when we introduce the filter API we can filter on tags, properties and driver_info | 20:05 |
| devananda | lucasagomes: I think all three tables could (dunno if they should) reference the same fact table | 20:06 |
| devananda | eg, tags, properties, driver_info -- these could all be one table | 20:06 |
| devananda | with 3 cross-ref tables | 20:07 |
| lucasagomes | devananda, right, yeah I was thinking about that for properties + driver_info | 20:07 |
| lucasagomes | I can take a look at the tags too | 20:07 |
| * lucasagomes checks | 20:07 | |
| devananda | I dont know how sqla will handle that, though | 20:07 |
| lucasagomes | right, I can try, I have a patch up doing ^ already | 20:08 |
| lucasagomes | well kinda-ish | 20:08 |
| lucasagomes | devananda, https://review.openstack.org/#/c/252531/ | 20:08 |
| lucasagomes | I can try to use the same fact table, I let you know tomorrow if that works | 20:09 |
| *** aginwala has joined #openstack-ironic | 20:09 | |
| *** pece has joined #openstack-ironic | 20:09 | |
| devananda | lucasagomes: great, thanks | 20:09 |
| lucasagomes | devananda, the only problem with the fact table is that we somehow will need to add a mechanism to clean it up | 20:10 |
| devananda | lucasagomes: eh, i'm not sure we do | 20:10 |
| lucasagomes | that delete-orphan cascade thing doesn't work as I thought | 20:10 |
| lucasagomes | devananda, just leave it grows? | 20:10 |
| lucasagomes | s/leave/let | 20:11 |
| devananda | yah | 20:11 |
| devananda | well. in most of our use cases, it's not going to grow indefinitely | 20:12 |
| devananda | only if someone is loading a unique tag/property/driverinfo parameter for every node -- and then deleting it | 20:12 |
| devananda | *deleting the node | 20:12 |
| lucasagomes | yeah, I think it most cases it should be fine | 20:12 |
| lucasagomes | perhaps we can just leave a low bug open about it in case someone hit that problem | 20:13 |
| lucasagomes | devananda, thanks again! Tomorrow I will give it a go | 20:13 |
| devananda | lucasagomes: cheers. g'night! | 20:13 |
| lucasagomes | night | 20:13 |
| *** lucasagomes is now known as lucas-dinner | 20:13 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Agent vendor handles manual cleaning https://review.openstack.org/247695 | 20:16 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Replace config 'clean_nodes' with 'automated_clean' https://review.openstack.org/247701 | 20:16 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: API to manually clean nodes https://review.openstack.org/251995 | 20:17 |
| *** yarkot has joined #openstack-ironic | 20:18 | |
| * krotscheck is trying to figure out where to put error messages. | 20:34 | |
| * krotscheck figured it out just by typing that message. | 20:34 | |
| * krotscheck goes to do that thing. | 20:34 | |
| *** jcoufal has quit IRC | 20:37 | |
| *** thrash|biab is now known as thrash | 20:46 | |
| *** sergek has quit IRC | 20:46 | |
| openstackgerrit | Bertrand Lallau proposed openstack/ironic: Numerous debug messages due to iso8601 log level https://review.openstack.org/254951 | 20:47 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add port option support for ipmitool https://review.openstack.org/254899 | 20:48 |
| *** yarkot has quit IRC | 20:49 | |
| openstackgerrit | Merged openstack/python-ironicclient: Fix exceptions.from_response() parameter https://review.openstack.org/250675 | 20:51 |
| jlvillal | NobodyCam, I'll wait for Jenkins but it looks good to me :) | 20:58 |
| NobodyCam | jlvillal: awesome Thak you :) | 21:01 |
| jlvillal | NobodyCam, Thanks for the patch! | 21:01 |
| NobodyCam | :) ran in to that while looking into ipmi testing | 21:04 |
| *** aginwala has quit IRC | 21:05 | |
| *** Sukhdev has quit IRC | 21:08 | |
| *** aginwala has joined #openstack-ironic | 21:09 | |
| *** jaybeale has joined #openstack-ironic | 21:10 | |
| jlvillal | NobodyCam, Bad news: https://jenkins04.openstack.org/job/gate-ironic-python27/856/console | 21:12 |
| * NobodyCam clicks | 21:13 | |
| jlvillal | Looks like there is a check of all the options or something. So doesn't expect 'ipmi_port' to be in the list. I'm guessing | 21:14 |
| jlvillal | NobodyCam, ^ | 21:14 |
| NobodyCam | jlvillal: yep. forgot to run the pxe test locally | 21:14 |
| NobodyCam | fixing | 21:14 |
| jlvillal | NobodyCam, I've done it too many times to count ;) | 21:15 |
| *** [1]cdearborn has joined #openstack-ironic | 21:15 | |
| jlvillal | mrda, Anytime you want to meet just let me know! | 21:17 |
| openstackgerrit | Chris Krelle proposed openstack/ironic: Add port option support for ipmitool https://review.openstack.org/254899 | 21:18 |
| NobodyCam | jlvillal: ^^^ | 21:18 |
| NobodyCam | :) | 21:18 |
| jlvillal | NobodyCam, Thanks | 21:22 |
| *** yarkot has joined #openstack-ironic | 21:24 | |
| *** cdearborn has quit IRC | 21:30 | |
| *** dprince has quit IRC | 21:30 | |
| *** dims has joined #openstack-ironic | 21:31 | |
| *** vishwanathj has joined #openstack-ironic | 21:32 | |
| *** aginwala has quit IRC | 21:33 | |
| *** Marga_ has quit IRC | 21:45 | |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added driver and port information to node detail page https://review.openstack.org/254971 | 21:51 |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/ironic: Improving capabilities_to_dict and adding helpful tests https://review.openstack.org/254972 | 21:52 |
| NobodyCam | jlvillal: just fyi I have a ping out to the Ilo folks RE: the 3rd party test | 21:53 |
| *** dims has quit IRC | 21:59 | |
| *** Marga_ has joined #openstack-ironic | 22:08 | |
| *** david-ly_ has joined #openstack-ironic | 22:10 | |
| *** david-lyle has quit IRC | 22:13 | |
| *** david-ly_ is now known as david-lyle | 22:13 | |
| jlvillal | NobodyCam, Great :) | 22:16 |
| *** ndipanov has quit IRC | 22:21 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector-specs: High Availability for Ironic Inspector https://review.openstack.org/253675 | 22:22 |
| NobodyCam | changing locations back in a bit | 22:23 |
| *** mkovacik has quit IRC | 22:26 | |
| *** e0ne has quit IRC | 22:26 | |
| *** mgoddard has quit IRC | 22:29 | |
| *** cdearborn has joined #openstack-ironic | 22:33 | |
| *** yarkot has quit IRC | 22:39 | |
| *** aginwala has joined #openstack-ironic | 22:42 | |
| *** [1]cdearborn has quit IRC | 22:49 | |
| *** [1]cdearborn has joined #openstack-ironic | 22:55 | |
| *** samueldmq has quit IRC | 22:56 | |
| *** devananda has quit IRC | 22:57 | |
| *** NobodyCam has quit IRC | 22:57 | |
| *** alexpilotti has quit IRC | 22:57 | |
| *** hblixt has quit IRC | 22:58 | |
| *** alexpilotti has joined #openstack-ironic | 22:58 | |
| *** alexpilotti has quit IRC | 22:59 | |
| *** alexpilotti has joined #openstack-ironic | 23:00 | |
| *** yuanying has joined #openstack-ironic | 23:00 | |
| *** kkoski has quit IRC | 23:06 | |
| *** chlong has quit IRC | 23:10 | |
| *** cdearborn has quit IRC | 23:10 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Introduce libvirt dirver https://review.openstack.org/253096 | 23:14 |
| *** baoli has quit IRC | 23:18 | |
| *** ionutbalutoiu has quit IRC | 23:20 | |
| *** alexpilotti has quit IRC | 23:23 | |
| *** alexpilotti has joined #openstack-ironic | 23:24 | |
| *** naohirot has joined #openstack-ironic | 23:24 | |
| *** [1]cdearborn has quit IRC | 23:29 | |
| *** baoli has joined #openstack-ironic | 23:32 | |
| *** alexpilotti has quit IRC | 23:33 | |
| *** alexpilotti has joined #openstack-ironic | 23:33 | |
| *** baoli has quit IRC | 23:35 | |
| *** hblixt has joined #openstack-ironic | 23:35 | |
| *** Sukhdev has joined #openstack-ironic | 23:35 | |
| *** baoli has joined #openstack-ironic | 23:36 | |
| *** alex_xu has quit IRC | 23:36 | |
| *** aginwala has quit IRC | 23:37 | |
| *** alex_xu has joined #openstack-ironic | 23:41 | |
| *** mbound_ has joined #openstack-ironic | 23:42 | |
| mrda | jlvillal: hey, sorry traveling today. You're not in training today? | 23:43 |
| *** pece has quit IRC | 23:44 | |
| openstackgerrit | Merged openstack/ironic: Add port option support for ipmitool https://review.openstack.org/254899 | 23:45 |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/ironic: Adding oneview.common tests for some method not well tested https://review.openstack.org/255006 | 23:53 |
| sinval | hey folks, a quick review: https://review.openstack.org/255006 :) | 23:53 |
| *** baoli has quit IRC | 23:54 | |
| *** jerrygb has quit IRC | 23:55 | |
| *** aginwala has joined #openstack-ironic | 23:55 | |
| *** jerrygb has joined #openstack-ironic | 23:56 | |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added driver and port information to node detail page https://review.openstack.org/254971 | 23:56 |
| *** smoriya has joined #openstack-ironic | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!