| JayF | impossible to debug just looking through the lens of the gate | 00:00 |
|---|---|---|
| jroll | yeah | 00:00 |
| jroll | er | 00:00 |
| jroll | what I meant was, without running locally | 00:00 |
| Haomeng | zer0c00l: yes | 00:00 |
| zer0c00l | JayF: I run devstack on a RHEL7 box | 00:01 |
| zer0c00l | I was able to get the systemd-nspawn working | 00:01 |
| zer0c00l | fine | 00:01 |
| *** sacharya_ has quit IRC | 00:01 | |
| zer0c00l | it did the installation | 00:01 |
| JayF | zer0c00l: we've def seen issues downstream, when implementing some hardwaremanagers | 00:01 |
| zer0c00l | If you look at the ironic-bm-logs, all i see is tftp loading the deploy kernel | 00:01 |
| JayF | zer0c00l: that nspawn has some security pieces that are nearly-impossible to disable to implement some cleaning bits | 00:02 |
| JayF | zer0c00l: may be unrelated to the pain you're seeing though :/ | 00:02 |
| zer0c00l | yeah | 00:02 |
| zer0c00l | Again, the setup just works fine with the changes to cloud-config.yml. With kernel 4.1.x they introduced a new cgroups fs | 00:03 |
| zer0c00l | so all i needed to do was just mount /sys/fs/cgroup into /opt/ironic_python_agent/sys | 00:03 |
| zer0c00l | All is left to do is | 00:04 |
| zer0c00l | running the gate locally | 00:04 |
| zer0c00l | is it possible? | 00:04 |
| zer0c00l | Any pointers? | 00:04 |
| jroll | zer0c00l: keep in mind whenever the "baremetal" node in the gate reboots, it truncates the console log :( | 00:05 |
| jroll | zer0c00l: otherwise, yes it's possible. I'd recommend just running tempest locally though I forget the specific incantation | 00:06 |
| zer0c00l | i see | 00:07 |
| zer0c00l | So is there a way to look at the logs while the gate is running | 00:07 |
| zer0c00l | I mean the "baremetal" logs | 00:07 |
| jroll | not that I know of | 00:08 |
| jroll | so when you run it locally, a full provision works? | 00:08 |
| Haomeng | zer0c00l: I think you run on RHEL, right, it works | 00:09 |
| zer0c00l | Haomeng: Yeah, i run it on RHEL | 00:09 |
| zer0c00l | jroll: yes | 00:09 |
| zer0c00l | it works | 00:09 |
| zer0c00l | I have a devstack setup | 00:09 |
| jroll | wtf | 00:10 |
| zer0c00l | i can post the logs if you want ;) | 00:10 |
| jroll | heh | 00:10 |
| Haomeng | zer0c00l: but our gate does not run on rhel I think | 00:10 |
| jroll | I mean, the gate runs on ubuntu | 00:10 |
| zer0c00l | yes it runs on ubuntu | 00:10 |
| jroll | which shouldn't matter at all, because this is running in a vm | 00:10 |
| Haomeng | yes, so there are some differents | 00:10 |
| zer0c00l | jroll: Yes it could be a hypervisor thing? | 00:11 |
| jroll | zer0c00l: that would be very odd, but it could be I guess | 00:11 |
| JayF | I mean, honestly, more often our gate fails bceause it's a slow-thing | 00:11 |
| JayF | moreso than a hypervisor specific thing | 00:11 |
| zer0c00l | i see | 00:11 |
| JayF | but we've def had failures specific to running teh gate in VMs | 00:11 |
| jroll | not repeatedly like this though | 00:11 |
| JayF | that's why after writing an image we run both partprobe and partx | 00:11 |
| JayF | jroll: ^ see that bug | 00:11 |
| jroll | this definitely smells like a real break | 00:11 |
| Haomeng | zer0c00l: looks like the systemd-nspawn not working as well | 00:11 |
| Haomeng | and the issue is docker related I think | 00:12 |
| jroll | docker only does the build, not the running of the container | 00:12 |
| jroll | zer0c00l: I'll run this on ubuntu and see what I can do | 00:12 |
| Haomeng | jroll: ok | 00:13 |
| *** naohirot has joined #openstack-ironic | 00:13 | |
| *** derekh has joined #openstack-ironic | 00:13 | |
| Haomeng | is this /opt/ironic-python-agent/sys/fs/cgroup/blkio folder created by systemd-nspawn? | 00:13 |
| jroll | I think so? | 00:14 |
| zer0c00l | Haomeng: that has been fixed by editing the cloud-config.yml | 00:15 |
| Haomeng | zer0c00l: should related with /sys/fs/cgroup which we changed in this patch | 00:15 |
| zer0c00l | yes | 00:15 |
| zer0c00l | here is a successful boot with the latest installer build | 00:15 |
| zer0c00l | https://paste.fedoraproject.org/291642/14478056/ | 00:15 |
| Haomeng | zer0c00l: yes, maybe this patch is just working on rhel and not working on ubuntu | 00:16 |
| Haomeng | zer0c00l: we dont think other ipa gate failed, it should not be common issue I think | 00:16 |
| jroll | zer0c00l: hm, that has [ 549.917839] systemd-nspawn[821]: Failed to mount to /opt/ironic-python-agent/sys/fs/cgroup/blkio: No such file or directory | 00:16 |
| jroll | but it eventually succeeds? | 00:16 |
| zer0c00l | oh sorry, | 00:17 |
| zer0c00l | it was one of test runs | 00:17 |
| jroll | ah ok | 00:17 |
| *** harshs has quit IRC | 00:17 | |
| zer0c00l | i ssh'd into the running VM | 00:17 |
| zer0c00l | and edited the systemd unit file | 00:17 |
| Haomeng | zer0c00l: dont worry | 00:17 |
| zer0c00l | added /sys/fs/cgroup | 00:17 |
| zer0c00l | then it worked | 00:17 |
| zer0c00l | lol | 00:17 |
| Haomeng | zer0c00l: so we have workaround? | 00:17 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/246090 | 00:18 |
| zer0c00l | Haomeng: Yes, It has been already in-corporated in the code | 00:18 |
| zer0c00l | In linux kernel 4.1.7 (which is the latest coreos stable) they added cgroups fs | 00:19 |
| zer0c00l | systemd-nspawn requires it to be mounted in order for it work | 00:19 |
| Haomeng | zer0c00l: https://review.openstack.org/#/c/191505/9/imagebuild/coreos/oem/cloud-config.yml ? | 00:19 |
| zer0c00l | Yes that is the work around | 00:19 |
| *** penick has quit IRC | 00:20 | |
| *** derekh has quit IRC | 00:20 | |
| Haomeng | zer0c00l: you mean we have to ssh'd to vm and add " | 00:22 |
| zer0c00l | Haomeng: When created the core OS ramdisk with my authorized_keys, I was noticing the logs and systemd-nspawn failed | 00:23 |
| zer0c00l | So i sshed' into the VM as core@ user | 00:23 |
| Haomeng | zer0c00l: ok,got | 00:24 |
| zer0c00l | Then edited the systemd unit file in /etc/systemd/system/ironic-python-agent.service | 00:24 |
| zer0c00l | Updated the systemd-nspawn command in there to bind mount /sys/fs/cgroups | 00:24 |
| zer0c00l | Then i did a systemctl daemon-reload to reload the unit files | 00:24 |
| *** Pavo has quit IRC | 00:24 | |
| zer0c00l | IPA started working now | 00:24 |
| zer0c00l | okay! | 00:25 |
| JayF | Awesome :) | 00:25 |
| JayF | I built a lot of that coreos builder stuff initially and have been one of the sole maintainers | 00:25 |
| JayF | warms my heart to see other people troubleshooting and knowing how it all ticks | 00:25 |
| zer0c00l | :) | 00:25 |
| JayF | you just have to accept the love of systemd into your heart | 00:25 |
| zer0c00l | hahaha | 00:26 |
| Haomeng | JayF: :) | 00:26 |
| * jroll foresees JayF handing out pamphlets door to door soon | 00:26 | |
| JayF | no just black boxes | 00:26 |
| Haomeng | :) | 00:26 |
| JayF | you just have to trust the pamplets are inside, and communicating truth | 00:26 |
| jroll | lolol | 00:26 |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/python-ironicclient: Refactoring and removing duplicate code of "base.Manager" heirs https://review.openstack.org/246692 | 00:27 |
| *** penick has joined #openstack-ironic | 00:27 | |
| Haomeng | zer0c00l: so can we do some code change to modify ironic-python-agent.service defination, it is our ipa service defination, not sure, ingored me if does make sense:) | 00:30 |
| JayF | Haomeng: he did that change; all those units for systemd are created in the cloud-config.yml file | 00:30 |
| JayF | just waiting to see if tempest passes before tossing on my +2 | 00:30 |
| Haomeng | JayF: yes, got:) | 00:31 |
| JayF | jroll: re: the coreos stable patch -- we're OK knowing CoreOS can break the gate at any point? | 00:31 |
| *** harshs has joined #openstack-ironic | 00:31 | |
| Haomeng | JayF: this one - https://review.openstack.org/#/c/191505/9/imagebuild/coreos/oem/cloud-config.yml ? | 00:31 |
| JayF | yes | 00:31 |
| Haomeng | JayF: ok,got,thk:) | 00:31 |
| JayF | heh, I haven't looked at the upstream version of that file in a long time :) | 00:32 |
| JayF | much more clean than the one in my downstream ramdisk :) | 00:32 |
| *** harshs_ has joined #openstack-ironic | 00:34 | |
| *** alexpilotti has joined #openstack-ironic | 00:35 | |
| *** harshs has quit IRC | 00:36 | |
| *** harshs_ is now known as harshs | 00:36 | |
| *** harshs has quit IRC | 00:37 | |
| *** sinval has joined #openstack-ironic | 00:37 | |
| *** alexpilotti has quit IRC | 00:39 | |
| jroll | JayF: right, that's a good question, probably not (I previously thought maybe) | 00:39 |
| JayF | it would only break -src | 00:40 |
| JayF | b/c we couldn't get a "clean build" uplaoded if that gate was broken | 00:40 |
| JayF | breaking IPA gate seems less bad, but I'm on the fence | 00:40 |
| jroll | right | 00:40 |
| *** Sukhdev has quit IRC | 00:41 | |
| *** Sukhdev has joined #openstack-ironic | 00:43 | |
| *** harshs has joined #openstack-ironic | 00:44 | |
| *** Sukhdev has quit IRC | 00:44 | |
| *** harshs has quit IRC | 00:44 | |
| zer0c00l | I don't have ubuntu with me, we are kind of a redhat shop , let me know how it goes jroll | 00:46 |
| JayF | zer0c00l: ^ we're talking about that change a little up there too | 00:46 |
| JayF | tl;dr: linking to a coreos channel means they can do a deploy that breaks our IPA gate | 00:47 |
| JayF | which is a bad thing, but the alternatives are bad too, so meh | 00:47 |
| jroll | zer0c00l: yeah, i'm trying to test | 00:48 |
| jroll | zer0c00l: dang, totally works for me too | 00:54 |
| * jroll sighs | 00:55 | |
| *** jerrygb has quit IRC | 00:55 | |
| zer0c00l | :) | 00:55 |
| zer0c00l | So something's up with the gate? | 00:55 |
| jroll | JayF: also random data point, this took 3 minutes to get from power on to agent heartbeating :/ | 00:55 |
| jroll | zer0c00l: I have no clue. in general the gate is working afaik | 00:56 |
| zer0c00l | jroll: 3 min is too slow? | 00:57 |
| zer0c00l | what is the normal time is like? | 00:57 |
| zer0c00l | Also is the timeout less than 3 mins? | 00:57 |
| jroll | I don't think it's too slow, but not sure | 00:57 |
| jroll | just a random data point :) | 00:57 |
| jroll | zer0c00l: default deploy timeout is 1800 seconds, so I don't think that's the problem here | 00:58 |
| jroll | :) | 00:58 |
| harlowja | zer0c00l u need to make that script that uses kexec to switch out vms into ubuntu :-P | 00:59 |
| harlowja | get er done, lol | 00:59 |
| harlowja | *switch our vms | 00:59 |
| *** hemna is now known as hemnafk | 01:04 | |
| *** jamielennox is now known as jamielennox|away | 01:10 | |
| zer0c00l | jroll: If you look at the conductor log it looks like it is provisioning it fine | 01:10 |
| zer0c00l | http://logs.openstack.org/05/191505/9/check/gate-tempest-dsvm-ironic-pxe_ipa-src/2bc210a/logs/screen-ir-cond.txt.gz | 01:10 |
| zer0c00l | 2015-11-17 20:15:18.507 12254 INFO ironic.drivers.modules.agent_base_vendor [-] Deployment to node 91247dfc-5ace-47a0-9a06-835d33e11b6d done | 01:11 |
| zer0c00l | It looks like things get stuck on clean wait? | 01:13 |
| jroll | odd | 01:16 |
| jroll | so maybe the job timed out? | 01:16 |
| *** penick has quit IRC | 01:19 | |
| zer0c00l | The timeout is 300s? | 01:20 |
| zer0c00l | https://github.com/openstack/tempest/blob/88d0e77b25dfc79becf50116fd72dec8750a1d05/tempest/config.py#L1225 | 01:20 |
| jroll | oh wow | 01:23 |
| jroll | that's good to know :) | 01:23 |
| *** Pavo has joined #openstack-ironic | 01:23 | |
| jroll | I had no clue that was there | 01:23 |
| lintan_ | morning Ironic | 01:23 |
| jroll | such enlightenment | 01:23 |
| lintan_ | morning jroll, Haomeng,JayF zer0c00l | 01:24 |
| jroll | zer0c00l: we also have a seemingly random bug where the node seems to fail to pxe | 01:24 |
| jroll | which I thuoght was slowness, but couldn't find a small timeout like this | 01:24 |
| jroll | lintan_: hey, good morning | 01:24 |
| zer0c00l | lintan_: Morning | 01:25 |
| zer0c00l | From the conductor it seems the node starts booting at 20:16 | 01:26 |
| zer0c00l | Finishes to deploying at 20:21 | 01:26 |
| zer0c00l | or almost 20:21:58 | 01:26 |
| zer0c00l | ~6 mins | 01:26 |
| zer0c00l | The timeout is 300 secs | 01:27 |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/python-ironicclient: Refactoring and removing duplicate code of "base.Manager" heirs https://review.openstack.org/246692 | 01:27 |
| zer0c00l | 360 secs to deploy | 01:27 |
| zer0c00l | This could be a problem to | 01:27 |
| jroll | zer0c00l: yep, I think you nailed it | 01:27 |
| zer0c00l | jroll: So we should increase the timeout? | 01:28 |
| zer0c00l | in tempest | 01:28 |
| jroll | zer0c00l: I'm thinking so, yes | 01:29 |
| jroll | zer0c00l: maybe add a related-bug for bug 1393099 | 01:30 |
| openstack | bug 1393099 in OpenStack-Gate "test_baremetal_server_ops fails with timeout waiting for provision_state to change" [Undecided,Confirmed] https://launchpad.net/bugs/1393099 | 01:30 |
| jroll | since it might fix that as well | 01:30 |
| zer0c00l | Looks like that has something to do with tftp timeout | 01:32 |
| zer0c00l | In our case there is no tftp timeout | 01:32 |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/python-ironicclient: Refactoring and removing duplicate code of "base.Manager" heirs https://review.openstack.org/246692 | 01:32 |
| jroll | zer0c00l: that's the guess, but hard to tell | 01:32 |
| zer0c00l | Looks like our case the host boots.. We see and actually does the provisioning according to the conductor | 01:33 |
| jroll | zer0c00l: so I think what's happening in that bug is that the deploy times out during the boot, and it looks like a tftp failure | 01:33 |
| zer0c00l | yes | 01:34 |
| zer0c00l | It wipes out the console log as well | 01:34 |
| zer0c00l | Thats why we half-baked logs in baremetal_0_console.log | 01:34 |
| jroll | right, so I think that timeout is what is causing the bug I linked | 01:34 |
| jroll | because someitmes we get slow gate hosts | 01:35 |
| zer0c00l | okay | 01:35 |
| zer0c00l | jroll: There are some timeouts here https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml | 01:40 |
| zer0c00l | Where should we change this? | 01:40 |
| jroll | zer0c00l: those timeouts are in minutes, those should be fine | 01:40 |
| jroll | I think just need to change the tempest one | 01:41 |
| *** zhenguo has joined #openstack-ironic | 01:42 | |
| Haomeng | lintan_: morning:) | 01:49 |
| *** boris-42 has joined #openstack-ironic | 01:50 | |
| *** harshs has joined #openstack-ironic | 01:52 | |
| lintan_ | hey Haomeng :) | 01:52 |
| openstackgerrit | Haomeng,Wang proposed openstack/python-ironicclient: support multi-nodes input for node-set-boot-device https://review.openstack.org/246207 | 01:58 |
| *** penick has joined #openstack-ironic | 01:58 | |
| Haomeng | lintan_: :) | 01:58 |
| *** cemason has joined #openstack-ironic | 01:59 | |
| *** harshs has quit IRC | 02:02 | |
| *** baoli has joined #openstack-ironic | 02:03 | |
| *** Haomeng has quit IRC | 02:04 | |
| *** sacharya has joined #openstack-ironic | 02:05 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/python-ironicclient: Drop explicit Python 2.6 support https://review.openstack.org/246512 | 02:06 |
| jroll | ^ easy one | 02:06 |
| *** thrash is now known as thrash|g0ne | 02:06 | |
| zer0c00l | jroll: https://review.openstack.org/#/c/246718/ | 02:07 |
| jroll | zer0c00l: mind if I update the commit message? | 02:07 |
| *** lazy_prince has joined #openstack-ironic | 02:10 | |
| *** Marga_ has quit IRC | 02:11 | |
| *** Haomeng has joined #openstack-ironic | 02:12 | |
| zer0c00l | jroll: I don't. please go ahead | 02:12 |
| *** praneshp has quit IRC | 02:13 | |
| jroll | zer0c00l: done, thanks | 02:14 |
| *** MattMan has quit IRC | 02:18 | |
| *** MattMan has joined #openstack-ironic | 02:18 | |
| zer0c00l | jroll: There is a spell error there s/enuogh/enough ? | 02:21 |
| jroll | zer0c00l: ha, thanks | 02:21 |
| jroll | I'll leave it for now | 02:22 |
| openstackgerrit | Jim Rollenhagen proposed openstack/ironic-python-agent: Use latest CoreOS stable when building https://review.openstack.org/191505 | 02:22 |
| jroll | zer0c00l: ^ added a depends-on there, that will tell us if this works as we expect | 02:22 |
| jroll | ]/b 71 | 02:22 |
| jroll | oops | 02:22 |
| zer0c00l | jroll: Cool! Thanks. | 02:23 |
| jroll | no problem, thanks for working on this :D | 02:23 |
| zer0c00l | :) | 02:23 |
| zer0c00l | jroll: Do you know the current status of hardware raid in IPA? | 02:24 |
| zer0c00l | I remeber there was a spec to enable hw raid | 02:25 |
| Haomeng | jroll: good night:) | 02:25 |
| *** baoli_ has joined #openstack-ironic | 02:25 | |
| jroll | hiya Haomeng :) | 02:25 |
| jroll | zer0c00l: all the pieces are there for RAID, we just need to finish the "manual cleaning" work so ops can trigger it. that should be done this cycle | 02:26 |
| Haomeng | jroll: :) | 02:26 |
| *** penick has quit IRC | 02:27 | |
| zer0c00l | jroll: oh i see, what type of raid controllers are supported now? | 02:27 |
| zer0c00l | HP? DELL? | 02:27 |
| *** baoli has quit IRC | 02:28 | |
| jroll | zer0c00l: good question, I believe those all live out of tree as agent hardware managers. I know HP has support, not sure who else off the top of my head | 02:28 |
| Haomeng | jroll: last question, if you have time please help to check with lucas and dmitry for this bp patch - https://review.openstack.org/#/c/228301/ ,thank you:) | 02:28 |
| *** davideagnello has quit IRC | 02:28 | |
| jroll | Haomeng: oh my, I'm sorry I forgot about that bp | 02:29 |
| *** davideagnello has joined #openstack-ironic | 02:29 | |
| *** jerrygb has joined #openstack-ironic | 02:29 | |
| jroll | Haomeng: approved the bp and removed -2 | 02:29 |
| jroll | thanks for reminder | 02:30 |
| *** dims_ has quit IRC | 02:30 | |
| zer0c00l | jroll: Looks like they want us to set it locally https://review.openstack.org/#/c/246718/ | 02:31 |
| Haomeng | jroll: np, it just a client minor patch:) thank you very much:) | 02:31 |
| openstackgerrit | Tan Lin proposed openstack/python-ironicclient: Add '--node' to port-list https://review.openstack.org/246216 | 02:31 |
| Haomeng | jroll: and I am still working on backup bp spec proposal | 02:31 |
| *** meghal has joined #openstack-ironic | 02:33 | |
| jroll | Haomeng: cool, thanks | 02:33 |
| Haomeng | jroll: but the 'backup' function is more complex bp, will take care and commit the bp spec:) | 02:34 |
| Haomeng | jroll: :) | 02:34 |
| jroll | Haomeng: indeed :) | 02:34 |
| Haomeng | jroll: it is late for youon local time I think | 02:34 |
| jroll | Haomeng: a bit late, I'll be leaving soon :) | 02:35 |
| Haomeng | jroll: ok, good night, and nice dream:) | 02:35 |
| jroll | Haomeng: have a good day | 02:35 |
| Haomeng | jroll: :) | 02:35 |
| jroll | zer0c00l: let's revisit tomorrow? | 02:35 |
| zer0c00l | jroll: I agree! | 02:36 |
| zer0c00l | Good night! | 02:36 |
| *** yonglihe has quit IRC | 02:42 | |
| sinval | I'm leaving folks, see you later o/ | 02:44 |
| openstackgerrit | Merged openstack/python-ironicclient: Use keystoneclient.exception directly https://review.openstack.org/244491 | 02:45 |
| *** sinval has left #openstack-ironic | 02:48 | |
| *** deray has joined #openstack-ironic | 02:55 | |
| openstackgerrit | Tan Lin proposed openstack/python-ironicclient: Correct node-port-list help info https://review.openstack.org/246729 | 02:58 |
| *** meghal has quit IRC | 02:58 | |
| *** NobodyCam has quit IRC | 02:59 | |
| *** NobodyCam has joined #openstack-ironic | 03:00 | |
| *** liliars has quit IRC | 03:01 | |
| *** meghal has joined #openstack-ironic | 03:03 | |
| *** liliars has joined #openstack-ironic | 03:05 | |
| *** ijw has quit IRC | 03:10 | |
| *** meghal has quit IRC | 03:11 | |
| *** yuanying_ has joined #openstack-ironic | 03:17 | |
| *** cemason has quit IRC | 03:19 | |
| *** yuanying has quit IRC | 03:19 | |
| *** yuanying_ has quit IRC | 03:21 | |
| *** achanda has joined #openstack-ironic | 03:24 | |
| openstackgerrit | Tan Lin proposed openstack/python-ironicclient: Add more unit tests for clituils https://review.openstack.org/246732 | 03:25 |
| *** cemason has joined #openstack-ironic | 03:30 | |
| *** cdearborn has quit IRC | 03:35 | |
| *** sacharya has quit IRC | 03:46 | |
| *** achanda has quit IRC | 03:48 | |
| *** cemason has quit IRC | 03:55 | |
| *** lazy_prince has quit IRC | 03:55 | |
| *** achanda has joined #openstack-ironic | 03:55 | |
| *** cemason has joined #openstack-ironic | 03:56 | |
| *** cemason has quit IRC | 03:57 | |
| *** meghal has joined #openstack-ironic | 04:04 | |
| *** lazy_prince has joined #openstack-ironic | 04:05 | |
| *** meghal1 has joined #openstack-ironic | 04:05 | |
| *** meghal has quit IRC | 04:08 | |
| *** yuanying has joined #openstack-ironic | 04:10 | |
| jroll | zer0c00l: progress! http://logs.openstack.org/05/191505/10/check/gate-tempest-dsvm-ironic-pxe_ipa-src/bfee19e/logs/ironic-bm-logs/baremetalbrbm_0_console.txt.gz | 04:10 |
| jroll | odd progress, but it's progress | 04:11 |
| * jroll actually goes away now | 04:11 | |
| *** rloo has quit IRC | 04:17 | |
| *** lazy_prince has quit IRC | 04:30 | |
| *** amotoki has joined #openstack-ironic | 04:34 | |
| *** deray has quit IRC | 04:46 | |
| *** amotoki has quit IRC | 04:49 | |
| *** kalpase has joined #openstack-ironic | 04:50 | |
| *** harshs has joined #openstack-ironic | 04:52 | |
| *** harshs_ has joined #openstack-ironic | 04:53 | |
| *** deray has joined #openstack-ironic | 04:53 | |
| *** jamielennox|away is now known as jamielennox | 04:54 | |
| *** harshs has quit IRC | 04:56 | |
| *** harshs_ is now known as harshs | 04:56 | |
| *** amotoki has joined #openstack-ironic | 05:01 | |
| *** HimanshuGarg has quit IRC | 05:01 | |
| *** yuikotakada has joined #openstack-ironic | 05:07 | |
| *** mtanino has quit IRC | 05:09 | |
| *** amotoki has quit IRC | 05:10 | |
| *** achanda has quit IRC | 05:12 | |
| *** amotoki has joined #openstack-ironic | 05:14 | |
| *** Sukhdev has joined #openstack-ironic | 05:17 | |
| *** Marga_ has joined #openstack-ironic | 05:20 | |
| *** Marga_ has quit IRC | 05:20 | |
| *** Marga_ has joined #openstack-ironic | 05:21 | |
| *** praneshp has joined #openstack-ironic | 05:23 | |
| *** jerrygb has quit IRC | 05:25 | |
| *** achanda has joined #openstack-ironic | 05:25 | |
| *** krtaylor has quit IRC | 05:29 | |
| *** achanda has quit IRC | 05:29 | |
| *** achanda has joined #openstack-ironic | 05:35 | |
| Sukhdev | lazy_prince: Ping | 05:36 |
| *** amotoki has quit IRC | 05:40 | |
| *** harshs_ has joined #openstack-ironic | 05:44 | |
| *** harshs has quit IRC | 05:44 | |
| *** harshs_ is now known as harshs | 05:44 | |
| *** praneshp_ has joined #openstack-ironic | 05:50 | |
| *** Sukhdev has quit IRC | 05:50 | |
| openstackgerrit | Debayan Ray proposed openstack/ironic-specs: Add Manual cleaning support to iLO drivers https://review.openstack.org/145404 | 05:51 |
| *** amotoki has joined #openstack-ironic | 05:51 | |
| *** achanda has quit IRC | 05:53 | |
| *** praneshp has quit IRC | 05:53 | |
| *** praneshp_ is now known as praneshp | 05:53 | |
| *** achanda has joined #openstack-ironic | 05:58 | |
| openstackgerrit | Debayan Ray proposed openstack/ironic-specs: Add Manual cleaning support to iLO drivers https://review.openstack.org/145404 | 06:01 |
| *** yuikotakada has quit IRC | 06:15 | |
| *** xiexs has joined #openstack-ironic | 06:15 | |
| openstackgerrit | Tan Lin proposed openstack/python-ironicclient: Add a new cmd method node-get-vendor-passthru-methods https://review.openstack.org/237444 | 06:32 |
| openstackgerrit | Tan Lin proposed openstack/python-ironicclient: Add driver-get-vendor-passthru-methods cmd https://review.openstack.org/246765 | 06:32 |
| openstackgerrit | Haomeng,Wang proposed openstack/python-ironicclient: support multi-nodes operation for set_power_state command https://review.openstack.org/228301 | 06:34 |
| *** rcernin has joined #openstack-ironic | 06:37 | |
| *** Marga_ has quit IRC | 07:03 | |
| *** Marga_ has joined #openstack-ironic | 07:05 | |
| *** alexpilotti has joined #openstack-ironic | 07:10 | |
| openstackgerrit | Haomeng,Wang proposed openstack/python-ironicclient: support multi-nodes operation for node-set-power-state command https://review.openstack.org/228301 | 07:12 |
| *** alexpilotti has quit IRC | 07:15 | |
| openstackgerrit | Haomeng,Wang proposed openstack/python-ironicclient: support multi-nodes operation for node-set-power-state command https://review.openstack.org/228301 | 07:17 |
| *** ionutbalutoiu has joined #openstack-ironic | 07:18 | |
| openstackgerrit | Haomeng,Wang proposed openstack/python-ironicclient: support multi-nodes input for node-set-boot-device https://review.openstack.org/246207 | 07:19 |
| openstackgerrit | Haomeng,Wang proposed openstack/python-ironicclient: support multi-nodes input for node-set-maintenance https://review.openstack.org/242299 | 07:22 |
| *** subscope has joined #openstack-ironic | 07:24 | |
| *** jerrygb has joined #openstack-ironic | 07:25 | |
| openstackgerrit | Haomeng,Wang proposed openstack/python-ironicclient: support multi-nodes input for node-set-provision-state https://review.openstack.org/242301 | 07:26 |
| *** jerrygb has quit IRC | 07:30 | |
| *** alexpilotti has joined #openstack-ironic | 07:31 | |
| *** achanda has quit IRC | 07:32 | |
| *** alexpilotti has quit IRC | 07:35 | |
| *** harshs has quit IRC | 07:36 | |
| *** ukalifon1 has joined #openstack-ironic | 07:38 | |
| *** ionutbalutoiu has quit IRC | 07:40 | |
| openstackgerrit | Tan Lin proposed openstack/python-ironicclient: A minor change for driver_shell test https://review.openstack.org/246784 | 07:42 |
| openstackgerrit | Haomeng,Wang proposed openstack/python-ironicclient: support multi-nodes operation for node-set-power-state command https://review.openstack.org/228301 | 08:02 |
| *** ifarkas has joined #openstack-ironic | 08:08 | |
| *** pas-ha has quit IRC | 08:10 | |
| *** GheRivero has left #openstack-ironic | 08:15 | |
| *** deray has quit IRC | 08:21 | |
| *** deray has joined #openstack-ironic | 08:21 | |
| *** pas-ha has joined #openstack-ironic | 08:22 | |
| *** ionutbalutoiu has joined #openstack-ironic | 08:24 | |
| *** ziggypup has joined #openstack-ironic | 08:24 | |
| *** ziggypup has left #openstack-ironic | 08:24 | |
| *** kozhukalov_ has joined #openstack-ironic | 08:26 | |
| openstackgerrit | Debayan Ray proposed openstack/ironic: Firmware update as part of iLO manual cleaning process https://review.openstack.org/211019 | 08:26 |
| *** alexpilotti has joined #openstack-ironic | 08:30 | |
| *** yog_ has joined #openstack-ironic | 08:34 | |
| *** jcoufal has joined #openstack-ironic | 08:35 | |
| *** deray has quit IRC | 08:35 | |
| *** dtantsur|afk is now known as dtantsur | 08:36 | |
| dtantsur | Morning Ironic | 08:37 |
| ionutbalutoiu | Morning dtantsur | 08:38 |
| *** sacharya has joined #openstack-ironic | 08:49 | |
| *** kozhukalov_ has quit IRC | 08:51 | |
| Haomeng | morning dtantsur, ionutbalutoiu :) | 08:52 |
| *** sacharya has quit IRC | 08:53 | |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Validate all tcp/udp port numbers https://review.openstack.org/244713 | 09:00 |
| aarefiev | morning folks | 09:04 |
| *** achanda has joined #openstack-ironic | 09:05 | |
| aarefiev | dtantsur: hi, have a min? | 09:07 |
| aarefiev | Is it possible nodes auto discovery? | 09:07 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add documentation on how to run tests https://review.openstack.org/240866 | 09:07 |
| dtantsur | aarefiev, hi! not right now, see ML for discussion. we may end up with it available this cycle, but it's pretty controversial | 09:08 |
| aarefiev | dtantsur: ahh I see, I've found Lucas demo and wondering how it works | 09:09 |
| aarefiev | https://www.youtube.com/watch?v=gZGYeo7sJhQ | 09:09 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add documentation on how to run tests https://review.openstack.org/240866 | 09:10 |
| dtantsur | aarefiev, it used to be an early prototype of ironic-inspector. we didn't implement it that way for various reasons, not we're discussing this feature again | 09:11 |
| *** romainh has joined #openstack-ironic | 09:11 | |
| *** lucas-afk is now known as lucasagomes | 09:12 | |
| aarefiev | dtantsur: thanks for clarification, looking forward to see this feature, I suggest you are working on this | 09:14 |
| *** _degorenko is now known as degorenko | 09:14 | |
| dtantsur | aarefiev, sambetts plans on doing it, as soon as we polish the design details | 09:14 |
| openstackgerrit | Tan Lin proposed openstack/python-ironicclient: A minor change for driver_shell test https://review.openstack.org/246784 | 09:16 |
| aarefiev | dtantsur: ok thanks, hope I can help with this feature, I'm interesting in | 09:18 |
| *** ionutbalutoiu has quit IRC | 09:19 | |
| betherly | Morning all! | 09:19 |
| *** ionutbalutoiu has joined #openstack-ironic | 09:20 | |
| dtantsur | morning betherly | 09:21 |
| *** Piet has quit IRC | 09:21 | |
| dtantsur | aarefiev, cool! please start with a ML discussion, it started a couple of weeks ago, but stalled. I also need to get back to it. | 09:21 |
| *** achanda has quit IRC | 09:22 | |
| aarefiev | dtantsur: sure | 09:22 |
| *** ziggypup has joined #openstack-ironic | 09:24 | |
| *** ziggypup has left #openstack-ironic | 09:24 | |
| *** athomas has joined #openstack-ironic | 09:29 | |
| *** naohirot has quit IRC | 09:30 | |
| *** e0ne has joined #openstack-ironic | 09:32 | |
| *** xek has joined #openstack-ironic | 09:37 | |
| *** mkovacik has joined #openstack-ironic | 09:41 | |
| *** jistr has joined #openstack-ironic | 09:42 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic-python-agent: Create partition at max msdos limit for disks > 2TB https://review.openstack.org/246253 | 09:48 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Follow up patch for da9c3b0adc67efa916fc534d975823c0a45948a1 https://review.openstack.org/246827 | 09:56 |
| *** ionutbalutoiu has quit IRC | 09:56 | |
| lucasagomes | morning all :-) | 09:56 |
| dtantsur | morning lucasagomes! | 09:58 |
| openstackgerrit | Merged openstack/ironic: Use wsgi from oslo.service for Ironic API https://review.openstack.org/219298 | 10:00 |
| openstackgerrit | Merged openstack/ironic: Sort root device hints when parsing https://review.openstack.org/246422 | 10:02 |
| openstackgerrit | Merged openstack/ironic: Document 'erase_devices_iterations' config option https://review.openstack.org/240137 | 10:02 |
| *** mgould has joined #openstack-ironic | 10:07 | |
| mgould | morning Ironicers! | 10:07 |
| *** ionutbalutoiu has joined #openstack-ironic | 10:08 | |
| dtantsur | morning mgould | 10:11 |
| openstackgerrit | Merged openstack/ironic: Add and document two new root device hints: wwn_{with, vendor}_extension https://review.openstack.org/245870 | 10:15 |
| *** kozhukalov_ has joined #openstack-ironic | 10:24 | |
| *** jerrygb has joined #openstack-ironic | 10:26 | |
| *** deray has joined #openstack-ironic | 10:26 | |
| *** daemontool has joined #openstack-ironic | 10:28 | |
| *** daemontool has quit IRC | 10:30 | |
| *** daemontool has joined #openstack-ironic | 10:30 | |
| *** jerrygb has quit IRC | 10:31 | |
| *** daemontool has quit IRC | 10:31 | |
| *** thrash|g0ne has quit IRC | 10:34 | |
| yolanda | hi, newbie question... i'm trying to generate a custom image with bifrost, to add an extra element. I add the element on /etc/bifrost/bifrost_global_vars. The element is just a custom ironic-agent one, that cherry-picks a fix : https://review.openstack.org/246253 | 10:36 |
| yolanda | it doesn't seem to be applied, am i missing some logic? | 10:36 |
| yolanda | what i do is to add the element on /etc/bifrost/bifrost_global_vars, and regenerate the image | 10:37 |
| *** dtantsur is now known as dtantsur|brb | 10:37 | |
| yolanda | lucasagomes, i amended the change following your comments, thanks for the review | 10:41 |
| *** e0ne has quit IRC | 10:43 | |
| *** mgould has quit IRC | 10:44 | |
| *** amotoki has quit IRC | 10:47 | |
| *** sacharya has joined #openstack-ironic | 10:54 | |
| *** mgould has joined #openstack-ironic | 10:57 | |
| *** sacharya has quit IRC | 10:58 | |
| *** meghal1 has quit IRC | 11:02 | |
| lucasagomes | yolanda, thanks I will look in a min | 11:04 |
| *** HimanshuGarg has joined #openstack-ironic | 11:04 | |
| *** e0ne has joined #openstack-ironic | 11:04 | |
| *** meghal has joined #openstack-ironic | 11:05 | |
| *** dims has joined #openstack-ironic | 11:07 | |
| *** deray has quit IRC | 11:09 | |
| *** dims has quit IRC | 11:12 | |
| *** daemontool has joined #openstack-ironic | 11:17 | |
| *** dims has joined #openstack-ironic | 11:18 | |
| *** daemontool has quit IRC | 11:18 | |
| *** daemontool has joined #openstack-ironic | 11:19 | |
| *** romainh has quit IRC | 11:19 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Refactor the image download and checksum computation bits https://review.openstack.org/234338 | 11:19 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-python-agent: Add support for streaming raw images directly onto the disk https://review.openstack.org/234339 | 11:19 |
| *** meghal has quit IRC | 11:20 | |
| *** daemontool has quit IRC | 11:21 | |
| *** daemontool has joined #openstack-ironic | 11:24 | |
| yolanda | lucasagomes, thanks | 11:25 |
| yolanda | i tested the change manually, running the script on a machine with 5tb disk, and worked. I am trying to bring a custom element to my bifrost setup, to test it, but no luck | 11:26 |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Add missing translation markers https://review.openstack.org/246875 | 11:26 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Wake up AMT interface before send request https://review.openstack.org/234190 | 11:27 |
| *** zhenguo has quit IRC | 11:27 | |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Add missing translation markers https://review.openstack.org/246875 | 11:28 |
| *** sinval has joined #openstack-ironic | 11:30 | |
| sinval | good morning folks | 11:30 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic: Add SSL support to the Ironic API https://review.openstack.org/240492 | 11:32 |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/python-ironicclient: Refactoring and removing duplicate code of "base.Manager" heirs https://review.openstack.org/246692 | 11:37 |
| *** subscope has quit IRC | 11:41 | |
| sambetts | Morning all o/ | 11:41 |
| sambetts | How was your holiday dtantsur | 11:41 |
| sinval | hey sambetts | 11:41 |
| sambetts | o/ sinval | 11:42 |
| *** subscope has joined #openstack-ironic | 11:42 | |
| *** dtantsur|brb is now known as dtantsur | 11:43 | |
| dtantsur | morning sambetts! nothing special, mostly cleaning at home :D still I feel relieved, not sure why :) | 11:43 |
| lucasagomes | yolanda, right, yeah I don't know much about ansible playbooks :-( | 11:43 |
| lucasagomes | sinval, morning | 11:43 |
| sambetts | dtantsur: :D its always good to get that stuff out of the way | 11:43 |
| lucasagomes | sambetts, morning too :-) | 11:43 |
| sambetts | o/ lucasagomes | 11:44 |
| sinval | hey lucasagomes o/ | 11:44 |
| sinval | jlvillal, hey I updated https://review.openstack.org/246692 | 11:46 |
| sinval | jlvillal, take a look when you have time | 11:46 |
| *** daemontool has quit IRC | 11:47 | |
| *** daemontool has joined #openstack-ironic | 11:47 | |
| *** amotoki has joined #openstack-ironic | 11:48 | |
| *** amotoki has quit IRC | 11:49 | |
| * mgould could also use another look at https://review.openstack.org/#/c/240202/, which now addresses dtantsur's comments | 11:50 | |
| *** mgould is now known as mgould|afk | 11:53 | |
| *** romainh has joined #openstack-ironic | 11:57 | |
| openstackgerrit | Yuiko Takada proposed openstack/ironic-specs: Add allowed transitions to API https://review.openstack.org/224022 | 11:57 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic: Add SSL support to the Ironic API https://review.openstack.org/240492 | 12:04 |
| *** smoriya_ has quit IRC | 12:20 | |
| *** subscope has quit IRC | 12:20 | |
| *** subscope has joined #openstack-ironic | 12:21 | |
| *** subscope has quit IRC | 12:21 | |
| openstackgerrit | Haozhi Cui proposed openstack/python-ironicclient: fix a typo in node_shell.py https://review.openstack.org/246899 | 12:24 |
| *** aarefiev has quit IRC | 12:25 | |
| *** aarefiev has joined #openstack-ironic | 12:27 | |
| sinval | vdrok, are you there? | 12:28 |
| vdrok | sinval, morning, yes ^) | 12:28 |
| sinval | vdrok, morning | 12:28 |
| sinval | hey, I have some questions about your last comment on https://review.openstack.org/#/c/246692/4/ironicclient/common/base.py | 12:28 |
| sinval | you suggested a common function _vendor_passthru for driver and node, to avoid the problem of relying update, delete and get functions defined only on the heirs | 12:29 |
| openstackgerrit | Ionut-Madalin Balutoiu proposed openstack/ironic: Add clean up method for the DHCP factory https://review.openstack.org/245911 | 12:29 |
| vdrok | sinval, yep, not sure if it is the nest solution, but at least it will not be in the base class :) | 12:30 |
| sinval | vdrok, what do you think about the same idea of _create and _get methods, that would be nice to pass the specific function as a parameter | 12:31 |
| vdrok | *best | 12:31 |
| jlvillal | Good morning Ironic | 12:31 |
| vdrok | sinval, why do you need a function? It seems that just path to resource would be enough? | 12:32 |
| vdrok | morning jlvillal | 12:32 |
| sinval | morning jlvillal | 12:32 |
| jlvillal | :) | 12:33 |
| sinval | vdrok, could you be clearer about the path to resource approach? I'm missing something about this... | 12:33 |
| vdrok | sinval, on L 48 in base.py for example add path argument and do self.api.json_request to this path. and e.g. in chassis.py L172 pass self._path() as an argument | 12:35 |
| vdrok | def _create(self, creation_attributes, path, **kwargs): | 12:36 |
| *** lucasagomes is now known as lucas-hungry | 12:37 | |
| sinval | vdrok, I see, for def _create I agree with this | 12:38 |
| *** kalpase has left #openstack-ironic | 12:38 | |
| *** mbound has joined #openstack-ironic | 12:39 | |
| sinval | vdrok, but for _vendor_passthru, the problem was the self.update, self.get and self.delete calls, so to me, passing those functions as parameters, just like you're saying about _create would be good enough | 12:40 |
| *** mbound has quit IRC | 12:41 | |
| vdrok | sinval, yes, it is an option, my suggestion was to add some util function that will get node or driver object as an argument and call methods on it | 12:44 |
| vdrok | sinval, it's up to you to decide :) | 12:45 |
| sinval | vdrok, ok, I'll maintain the _vendor_passthru on base.Manager, make more sense once we are already using inheritance to deal with this, just a nit... | 12:45 |
| sinval | vdrok, thnaks for your help o/ | 12:46 |
| vdrok | np | 12:46 |
| *** romainh has quit IRC | 12:47 | |
| *** mbound has joined #openstack-ironic | 12:50 | |
| aarefiev | jlvillal: hi, could you take a look one more time, please: https://review.openstack.org/#/c/229286/ | 12:54 |
| *** electrofelix has joined #openstack-ironic | 12:57 | |
| *** e0ne has quit IRC | 12:57 | |
| *** jerrygb has joined #openstack-ironic | 13:06 | |
| openstackgerrit | Syed Ismail Faizan Barmawer proposed openstack/ironic-lib: Replace rootwrap_config and rootwrap_helper_cmd with root_helper https://review.openstack.org/245741 | 13:07 |
| TheJulia | good morning | 13:10 |
| *** jerrygb has quit IRC | 13:10 | |
| sambetts | o/ TheJulia | 13:10 |
| *** e0ne has joined #openstack-ironic | 13:10 | |
| *** mgould|afk is now known as mgould | 13:14 | |
| *** dprince has joined #openstack-ironic | 13:19 | |
| *** sacharya has joined #openstack-ironic | 13:19 | |
| jroll | morning all :) | 13:20 |
| dtantsur | morning TheJulia, jroll | 13:21 |
| *** praneshp has quit IRC | 13:21 | |
| jroll | \o | 13:21 |
| sinval | morning TheJulia, dtantsur, jroll | 13:22 |
| liliars | morning TheJulia sambetts jroll dtantsur and everyone else I don't see :)) | 13:23 |
| sinval | hey liliars o/ | 13:23 |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Add missing translation markers https://review.openstack.org/246875 | 13:23 |
| dtantsur | sinval, o/ | 13:23 |
| jroll | hiya liliars, sinval :) | 13:23 |
| dtantsur | morning liliars as well :) do you still plan to update that IPA documentation patch? | 13:23 |
| *** sacharya has quit IRC | 13:24 | |
| liliars | dtantsur: hey! was just about to talk to you. sorry I didn't have the time to go over and update that IPA patch. I left some comments though. actually I was thinking a follow up patch, at least for the swift part, is a better idea. especially to ease the review process as folks seem to have mixed opinions about this storage backend thing. so we can move the discussion to a new patch, wdyt? | 13:28 |
| liliars | dtantsur: but I can update the patch as well with the other comments :) | 13:28 |
| liliars | the existing patch* | 13:29 |
| liliars | dtantsur: don't know if you had the time to see the reviews yet but there is some discussion going on over there | 13:29 |
| liliars | hey sinval :) | 13:30 |
| dtantsur | liliars, I'm a bit busy right now, so I didn't have a chance to look at comments yet | 13:31 |
| liliars | dtantsur, yeah you were also on a holiday yesterday so I kinda figured. when you have some time take a look and let me know. but I'm still up for it :) was just trying to figure a better way to do it considering the new reviews | 13:34 |
| dtantsur | liliars, if you feel like updating - just go ahead :) I definitely won't have time for it today and unlikely tomorrows | 13:34 |
| *** kbyrne has quit IRC | 13:36 | |
| liliars | dtantsur: ok then, I'll update the swift unrelated things and afterwards we can move the discussion regarding this to a new patch, I think it's the best way. thanks! | 13:36 |
| dtantsur | thank you! | 13:36 |
| *** afaranha has joined #openstack-ironic | 13:37 | |
| *** ndipanov has quit IRC | 13:38 | |
| *** ndipanov has joined #openstack-ironic | 13:39 | |
| *** kbyrne has joined #openstack-ironic | 13:43 | |
| *** achanda has joined #openstack-ironic | 13:44 | |
| *** krtaylor has joined #openstack-ironic | 13:46 | |
| *** achanda has quit IRC | 13:46 | |
| ifarkas | dtantsur, hi, could you please review https://review.openstack.org/#/c/235350/ ? | 13:47 |
| dtantsur | ifarkas, on my list, will try to find some time :) | 13:47 |
| ifarkas | dtantsur, thanks! | 13:47 |
| *** lucas-hungry is now known as lucasagomes | 13:48 | |
| *** thrash has joined #openstack-ironic | 13:48 | |
| *** thrash has quit IRC | 13:48 | |
| *** thrash has joined #openstack-ironic | 13:48 | |
| jlvillal | aarefiev, I will try to look at it today. I am in training this week. Which is good for me, but bad for my reviewing... | 13:48 |
| *** achanda has joined #openstack-ironic | 13:49 | |
| *** ionutbalutoiu has quit IRC | 13:50 | |
| aarefiev | jlvillal: cool, have fun:); don't worry so.. | 13:50 |
| *** thiagop has joined #openstack-ironic | 13:52 | |
| *** absubram has joined #openstack-ironic | 13:52 | |
| *** achanda has quit IRC | 13:53 | |
| thiagop | Good morning Ironic | 13:57 |
| sambetts | o/ ifarkas, liliars, thiagop | 13:57 |
| ifarkas | o/ | 14:00 |
| *** alexpilotti has quit IRC | 14:03 | |
| *** alexpilotti has joined #openstack-ironic | 14:03 | |
| jroll | hey cores, got a couple client patches that need landing pls | 14:05 |
| jroll | https://review.openstack.org/#/c/246512/ | 14:05 |
| jroll | https://review.openstack.org/#/c/237444/ | 14:05 |
| *** kozhukalov_ has quit IRC | 14:07 | |
| *** bapalm has joined #openstack-ironic | 14:08 | |
| *** boris-42 has quit IRC | 14:08 | |
| jroll | for context I'd like to release a client after those land | 14:09 |
| jroll | to drop py26 | 14:09 |
| liliars | dtantsur: np :) | 14:10 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Check shellinabox started successfully or not https://review.openstack.org/246283 | 14:10 |
| *** achanda has joined #openstack-ironic | 14:12 | |
| *** mgoddard__ has joined #openstack-ironic | 14:18 | |
| *** kozhukalov_ has joined #openstack-ironic | 14:19 | |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Add missing translation markers https://review.openstack.org/246875 | 14:19 |
| *** romainh has joined #openstack-ironic | 14:20 | |
| *** mgoddard_ has quit IRC | 14:20 | |
| *** cdearborn has joined #openstack-ironic | 14:24 | |
| *** daemontool_ has joined #openstack-ironic | 14:25 | |
| *** daemontool_ has quit IRC | 14:26 | |
| *** lsmola has quit IRC | 14:26 | |
| *** daemontool_ has joined #openstack-ironic | 14:27 | |
| *** absubram has quit IRC | 14:27 | |
| *** daemontool has quit IRC | 14:27 | |
| *** mgould has quit IRC | 14:27 | |
| *** daemontool_ has quit IRC | 14:28 | |
| *** mgoddard__ has quit IRC | 14:28 | |
| *** rloo has joined #openstack-ironic | 14:29 | |
| *** mgoddard has joined #openstack-ironic | 14:29 | |
| *** daemontool_ has joined #openstack-ironic | 14:29 | |
| *** lsmola has joined #openstack-ironic | 14:29 | |
| *** lsmola has quit IRC | 14:31 | |
| *** openstackgerrit has quit IRC | 14:31 | |
| *** openstackgerrit has joined #openstack-ironic | 14:32 | |
| jroll | lucasagomes: when you have a sec https://review.openstack.org/#/c/246512/ | 14:33 |
| lucasagomes | jroll, will do, I'm in a call at the moment | 14:33 |
| jroll | cool, thanks | 14:33 |
| *** mgould has joined #openstack-ironic | 14:40 | |
| *** bapalm has quit IRC | 14:43 | |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Add missing translation markers https://review.openstack.org/246875 | 14:43 |
| *** bapalm has joined #openstack-ironic | 14:43 | |
| openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Add missing translation markers https://review.openstack.org/246875 | 14:47 |
| *** kozhukalov_ has quit IRC | 14:51 | |
| *** daemontool_ has quit IRC | 14:51 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Add System Event Log support to management interface https://review.openstack.org/246965 | 14:52 |
| aarefiev | rloo: hi, have a min to talk about https://review.openstack.org/#/c/226201/ ? | 14:57 |
| rloo | aarefiev: sure | 14:57 |
| aarefiev | rloo: not sure I understand your opinion about exception changed, how it can break something | 14:58 |
| *** harshs has joined #openstack-ironic | 14:58 | |
| rloo | aarefiev: if an exception is raised -- that usually means some code won't be run. | 14:58 |
| rloo | aarefiev: there are i think cases where if you had not made those changes, the code would still 'work'. | 14:58 |
| *** mbound has quit IRC | 14:58 | |
| *** jasonamyers has joined #openstack-ironic | 14:59 | |
| rloo | aarefiev: so i think i am fine with making the changes but i am not fine with not documenting the effect of those changes to the users. | 14:59 |
| rloo | aarefiev: ie, there may be cases where a 'not-correct' value still worked before, and now when someone upgrades, it will break for them. | 14:59 |
| rloo | aarefiev: and i don't have time to go through all the configs that you modified, to see which ones will break and which won't. | 15:00 |
| rloo | aarefiev: do you know what i mean? | 15:00 |
| aarefiev | rloo: hmm, hot it works with 'not-correct' value ? | 15:01 |
| aarefiev | s/hot/how | 15:01 |
| rloo | aarefiev: did you test/run any of this, or did you just make code changes? | 15:01 |
| *** daemontool has joined #openstack-ironic | 15:01 | |
| aarefiev | rloo: tested not all, but there are checks for them already in code | 15:03 |
| *** kozhukalov_ has joined #openstack-ironic | 15:03 | |
| rloo | aarefiev: for example, https://review.openstack.org/#/c/226201/6/ironic/api/app.py | 15:03 |
| rloo | aarefiev: line 98 | 15:04 |
| rloo | aarefiev: that line only checks if the value is 'keystone'. | 15:04 |
| rloo | aarefiev: so if it had been 'foo' before, it would have been fine. | 15:04 |
| rloo | aarefiev: but it cannot be 'foo' with your change. | 15:04 |
| rloo | aarefiev: and i think with your change, an exception will be raised at line 98, right? | 15:04 |
| aarefiev | rloo: sure, but it value = 'foo', it's wrong behavior | 15:06 |
| aarefiev | should it be error? | 15:06 |
| rloo | aarefiev: i am not disagreeing with your code change. i am saying that we need to document the effect this change will have on users. | 15:06 |
| aarefiev | rloo: ahh ok, I see , where do you suggest to document it? | 15:07 |
| rloo | aarefiev: in the commit. that seems easiest. | 15:07 |
| jroll | reno! | 15:07 |
| rloo | aarefiev: sorry, i was afraid jroll would be listening to this... in reno. | 15:08 |
| jroll | lol | 15:08 |
| *** subscope has joined #openstack-ironic | 15:08 | |
| rloo | aarefiev: (I was hoping to make more work for jroll so he'd extract stuff from commits and put them in reno.) | 15:08 |
| * jroll cries | 15:08 | |
| jroll | rloo: there's enough of that to do already :/ | 15:09 |
| rloo | aarefiev, jroll: anyway, in addition to reno, i think it makes sense to put it in the commit itself. | 15:09 |
| jroll | yeah, agree | 15:09 |
| aarefiev | rloo: :) ok I get it, thanks for clarification | 15:09 |
| rloo | jroll: yeah, i know. just that we're starting with reno and don't quite have whatever process in place for that, do we? | 15:09 |
| jroll | rloo: the process is "add a release note with reno", not sure what you're asking for | 15:10 |
| rloo | aarefiev: sorry if my comment in that review wasn't clear. (hard to know when to blab on, or keep it short.) | 15:10 |
| jroll | http://docs.openstack.org/developer/reno/usage.html | 15:10 |
| *** harshs has quit IRC | 15:10 | |
| *** kromanenko has joined #openstack-ironic | 15:11 | |
| aarefiev | rloo: np, thank you for help | 15:11 |
| rloo | jroll: well, do we need to worry about formatting or anything? or i guess we can just get info in there and it can be cleaned up later. | 15:11 |
| jroll | rloo: meh, the most important thing is the info | 15:12 |
| rloo | jroll: i haven't had a chance to read about reno etc. so if i haven't, just didn't think i should be asking others to do it yet. | 15:13 |
| jroll | rloo: well, the longer we wait to enforce it, the more work we have to do to go back and pick things up | 15:14 |
| jroll | so please do read about it, it's important | 15:14 |
| rloo | jroll: agree. on my list... | 15:14 |
| *** ukalifon1 has quit IRC | 15:16 | |
| *** jcoufal has quit IRC | 15:17 | |
| *** Nakato has quit IRC | 15:17 | |
| *** ndipanov has quit IRC | 15:18 | |
| *** Nakato has joined #openstack-ironic | 15:18 | |
| *** ndipanov has joined #openstack-ironic | 15:19 | |
| *** ukalifon has joined #openstack-ironic | 15:19 | |
| jroll | stepping away for a while | 15:20 |
| dhellmann | jroll, rloo : reno notes are expected to be restructuredtext. The test jobs should fail if the output can't be parsed. | 15:23 |
| *** sacharya has joined #openstack-ironic | 15:23 | |
| *** jcoufal has joined #openstack-ironic | 15:23 | |
| rloo | dhellmann: good to know. I'm sure it'll all be easy as pie once we start using it :) | 15:23 |
| dhellmann | rloo : there's no time like the present ;-) | 15:24 |
| rloo | dhellmann: i was just wondering about the 'overhead' for people to land a bug/feature. Anyway... | 15:24 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: Use server_profile_template_uri at scheduling https://review.openstack.org/237122 | 15:25 |
| dhellmann | rloo : overhead? | 15:25 |
| rloo | dhellmann: well, they maybe have to have a spec or open a bug, do the code changes. know how to submit patches, write proper commit messages, and now add something to reno. | 15:26 |
| rloo | dhellmann: oh, and update documentation. and maybe i'm missing some other stuff :) | 15:27 |
| dhellmann | rloo : someone else was doing all of this work before. Distributing the load will, eventually, make the release process smoother and ensure we don't miss things by waiting until the end of the release to write about them. | 15:27 |
| dhellmann | rloo : yep, building complicated things well takes effort | 15:27 |
| rloo | dhellmann: oh, i don't disagree with it. just wondering how/if we can make it easier for submitters. | 15:27 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic: Use oslo_config choices support https://review.openstack.org/226201 | 15:28 |
| rloo | dhellmann: not that i am spending many seconds wondering about it. | 15:28 |
| *** sacharya has quit IRC | 15:28 | |
| openstackgerrit | Serge Kovaleff proposed openstack/ironic-inspector: Generate Sphinx docs https://review.openstack.org/246985 | 15:28 |
| dhellmann | rloo : yeah, we'll find a balance as we go | 15:28 |
| *** yonglihe has joined #openstack-ironic | 15:29 | |
| *** mbound has joined #openstack-ironic | 15:29 | |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Add limits support to management interface https://review.openstack.org/209612 | 15:29 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Intel Node Manager policies support https://review.openstack.org/214657 | 15:30 |
| *** ifarkas has quit IRC | 15:30 | |
| *** ifarkas has joined #openstack-ironic | 15:30 | |
| *** ifarkas has quit IRC | 15:30 | |
| *** ifarkas has joined #openstack-ironic | 15:31 | |
| rloo | aarefiev: hey, quick question for you or devananda. This bug can be closed now, right? https://bugs.launchpad.net/ironic/+bug/1296833 | 15:31 |
| openstack | Launchpad bug 1296833 in Ironic "client disconnect should not log stack trace in API" [Low,Opinion] | 15:31 |
| aarefiev | rloo: sure, we don't use wsgiref anymore | 15:32 |
| *** lazy_prince has joined #openstack-ironic | 15:32 | |
| aarefiev | rloo: I've updated commit msg, now trying to figure out how reno works :) | 15:35 |
| rloo | aarefiev: let me know when you find out! (or I'll just look at your patch) | 15:35 |
| *** pas-ha has quit IRC | 15:38 | |
| *** kozhukalov_ has quit IRC | 15:40 | |
| *** deray has joined #openstack-ironic | 15:47 | |
| *** deray_ has joined #openstack-ironic | 15:47 | |
| *** Sukhdev has joined #openstack-ironic | 15:48 | |
| *** gabriel-bezerra is now known as gabriel | 15:49 | |
| *** mgould has quit IRC | 15:54 | |
| *** kozhukalov_ has joined #openstack-ironic | 15:56 | |
| *** jerrygb has joined #openstack-ironic | 15:57 | |
| *** daemontool has quit IRC | 15:58 | |
| *** daemontool has joined #openstack-ironic | 15:59 | |
| vdrok | rloo, morning, are you currently working on manual cleaning? I'd like to understand cleaning stuff better, so maybe I can help with it? | 15:59 |
| *** mgoddard has quit IRC | 15:59 | |
| rloo | vdrok: yes, i'm working on it. the code is done, i "just" have to write more unit tests and break it up into several patches... | 15:59 |
| *** dtantsur is now known as dtantsur|brb | 16:00 | |
| rloo | vdrok: am hoping to do that stuff later today. will see how it goes. | 16:00 |
| sambetts | jroll, krtaylor: QA Meeting on today at 17:00? | 16:00 |
| vdrok | rloo, great :) | 16:00 |
| lucasagomes | jroll, patch is already merged, I'm late :-) thanks rloo | 16:01 |
| lucasagomes | s/merged/approved | 16:01 |
| rloo | lucasagomes: :D | 16:02 |
| openstackgerrit | Merged openstack/python-ironicclient: Drop explicit Python 2.6 support https://review.openstack.org/246512 | 16:06 |
| *** hemnafk is now known as hemna | 16:06 | |
| *** mgould has joined #openstack-ironic | 16:07 | |
| deray | hello ironic folks .. | 16:07 |
| mgould | hi deray | 16:07 |
| deray | mgould: hi .. | 16:08 |
| *** rcernin has quit IRC | 16:09 | |
| deray | I am currently working on a topic related to manual cleaning .. https://review.openstack.org/#/c/145404/ | 16:09 |
| deray | "Add Manual cleaning support to iLO drivers" | 16:09 |
| deray | would like you to have a look at the spec, to validate my understanding about manual cleaning | 16:11 |
| *** wuhg has quit IRC | 16:11 | |
| openstackgerrit | Merged openstack/ironic: Remove duplicated unit tests in test_manager https://review.openstack.org/246442 | 16:13 |
| krtaylor | sambetts, yes, I'm planning on it :) | 16:18 |
| sambetts | krtaylor: :) cool | 16:18 |
| krtaylor | I'm hopeful that jlvillal will be there, else I'll run it again | 16:18 |
| *** yog_ has quit IRC | 16:19 | |
| deray | vdrok: can you have a cursory glance at https://review.openstack.org/#/c/145404/, when you are a little free? | 16:20 |
| jlvillal | krtaylor, If you can run it again that would be most helpful. I am in class all day long today. | 16:22 |
| *** dims_ has joined #openstack-ironic | 16:22 | |
| krtaylor | jlvillal, no worries, I'll take care of it | 16:23 |
| jroll | deray: I think you will like my review :) | 16:23 |
| *** dims has quit IRC | 16:24 | |
| jroll | krtaylor: so I went hunting for the list of driver contacts, it isn't as complete as I remember: https://wiki.openstack.org/wiki/Ironic/Drivers | 16:25 |
| deray | jroll: lol, no no please spare me, i am just a new kid on the block :) .. just kidding, please do. | 16:25 |
| jroll | deray: haha, I tried to make your life easy :) | 16:25 |
| deray | jroll: appreciate ;) | 16:26 |
| krtaylor | jroll, ouch, well that makes direct contact harder, but not impossible, we can call front desks if necessary like thingee said :) | 16:26 |
| deray | jroll: please have a look .. my first contribution to Ironic :) | 16:27 |
| jroll | krtaylor: yeah, we'll do the best we can. should be able to dig in git history a bit for email addresses too :) | 16:27 |
| deray | "goign to be" of course | 16:27 |
| sambetts | krtaylor, jroll some people have started putting their IRC handles next to their drivers on the etherpad too :) | 16:27 |
| jroll | deray: I mean I left a review already, please look at it :) | 16:27 |
| JayF | deray: jroll gave you a get-out-of-spec-free comment on that spec, pretty much the best -1 ever | 16:28 |
| sambetts | krtaylor, jroll: Hopefully we can build up a comprehensive list on there | 16:28 |
| krtaylor | sambetts, agreed, that should be the place to "work" on it | 16:28 |
| deray | oh .. brb | 16:28 |
| jlvillal | krtaylor, Thanks! | 16:28 |
| JayF | jroll: re: the 3rd party CI spec, if the 3rd party drivers are going 100% outside of the ironic umbrella, I think that should be explicit | 16:29 |
| sambetts | jroll, can you have a look on that IronicQA pad and see if there are any drivers you think we've missed in our list | 16:29 |
| krtaylor | sambetts, we will eventually need the CI systems to use the infra wiki page for listing their contact info also | 16:29 |
| JayF | jroll: My assumption reading through it would be we might end up with openstack/ironic-CrappyHardware2000-driver repos :) | 16:29 |
| sambetts | krtaylor: Agreed, maybe something we need to put into the spec? | 16:30 |
| krtaylor | JayF, thanks for bringing that up, I don't have an answer | 16:30 |
| krtaylor | JayF, we discussed it briefly, and the summary was that they just stop being there an not be moved anywhere | 16:30 |
| jroll | sambetts: link please :) | 16:30 |
| sambetts | JayF: I've already got a cisco-ironic-contrib repo for my out-of-tree one, maybe we need to define a convention in the spec | 16:30 |
| *** alexpilotti has quit IRC | 16:30 | |
| sambetts | jroll: https://etherpad.openstack.org/p/IronicCI | 16:31 |
| krtaylor | JayF, I had proposed an attic approach to be a bit nicer | 16:31 |
| JayF | I don't care about if there's a convention; just if the 3rd party drivers aren't even allowed under the "big umbrella" of openstack bare metal we should say so. | 16:31 |
| jroll | JayF: openstack/ironic-CrappyHardware2000-driver is totally fine. big tent. it just won't be under the baremetal program (aka ironic umbrella) | 16:31 |
| JayF | jroll: aha, okay | 16:31 |
| JayF | jroll: not 100% sure how those lines are drawn now, part of why I was asking | 16:31 |
| *** mgoddard has joined #openstack-ironic | 16:31 | |
| jroll | JayF: and we should document that. but I don't care if it's in the spec or not | 16:31 |
| jroll | ah | 16:31 |
| jroll | lemme link | 16:31 |
| krtaylor | but, we don't want to create that repo for them, right? | 16:32 |
| sambetts | JayF: its a document stored by infra, http://git.openstack.org/cgit/openstack/governance/tree/reference/projects.yaml | 16:32 |
| jroll | JayF: ironic umbrella: https://github.com/openstack/governance/blob/master/reference/projects.yaml#L1459-L1545 | 16:32 |
| JayF | well I knew that piece | 16:32 |
| JayF | I didn't know there were allowed to be projects sorta just hanging out | 16:32 |
| JayF | not under any umbrella | 16:33 |
| jroll | JayF: and project existence https://github.com/openstack-infra/project-config/blob/master/gerrit/projects.yaml#L2307 | 16:33 |
| jroll | yep | 16:33 |
| jroll | krtaylor: correct | 16:33 |
| JayF | got it | 16:33 |
| krtaylor | if a driver team cares enough to continue with a driver that has been removed from tree, shouldn't they create a out-of-tree repo for it> | 16:33 |
| krtaylor | ok, I can update the spec with this info, I think that is a valid approach | 16:33 |
| sambetts | krtaylor: Something that happened with the neutron decomposion was that people wanted to keep their git history when they decomposed, are we going to support that? | 16:35 |
| rloo | jroll: there is some other web page that has ironic drivers listed (besides https://wiki.openstack.org/wiki/Ironic/Drivers). let me see if i can find it | 16:35 |
| *** dprince has quit IRC | 16:35 | |
| jroll | rloo: yeah, I couldn't find it :/ | 16:35 |
| krtaylor | sambetts, I don't believe it is possible across namespaces/projects, hm... | 16:36 |
| *** Nisha has joined #openstack-ironic | 16:36 | |
| jroll | sambetts: they can do that on their own, since we aren't moving the thing | 16:36 |
| krtaylor | rloo, you may be thinking of the etherpad? | 16:36 |
| rloo | jroll: found it. http://stackalytics.com/report/driverlog?project_id=openstack%2Fironic | 16:36 |
| jroll | sambetts: btw, I added msftocs and seamicro drivers to the pad | 16:36 |
| krtaylor | ah, yes, that too :) | 16:36 |
| sambetts | jroll: :D awesome thanks | 16:36 |
| rloo | jroll: (linked from our main ironic page. we did something right!) | 16:36 |
| jroll | rloo: aha, thanks! krtaylor check that link out | 16:36 |
| jroll | heh | 16:37 |
| *** alexpilotti has joined #openstack-ironic | 16:37 | |
| krtaylor | jroll, at some point, we need to say the spec is good enough :) else we will keep refining it instead of JFDI | 16:38 |
| *** Piet has joined #openstack-ironic | 16:38 | |
| jroll | krtaylor: that's what my +2 was meant to say :) | 16:40 |
| *** Marga_ has quit IRC | 16:40 | |
| *** Marga_ has joined #openstack-ironic | 16:42 | |
| *** mkovacik has quit IRC | 16:42 | |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Add TLS support and TSM remote video (WIP) https://review.openstack.org/246596 | 16:43 |
| openstackgerrit | Merged openstack/ironic-lib: Updated from global requirements https://review.openstack.org/246091 | 16:44 |
| *** mgoddard_ has joined #openstack-ironic | 16:45 | |
| *** sacharya has joined #openstack-ironic | 16:45 | |
| *** mkovacik has joined #openstack-ironic | 16:46 | |
| *** lazy_prince has quit IRC | 16:47 | |
| *** lazy_prince has joined #openstack-ironic | 16:47 | |
| *** mgoddard has quit IRC | 16:48 | |
| *** mgould has quit IRC | 16:51 | |
| rloo | jroll: what/are there guidelines wrt what we can log in debug? we shouldn't do passwords, but what about eg swift temp url. is it a security risk? : https://review.openstack.org/#/c/243141/ | 16:52 |
| jroll | rloo: great question | 16:53 |
| jroll | so, the security risk there is that someone with that url can fetch a private image until the url expires | 16:53 |
| *** rpioso has joined #openstack-ironic | 16:53 | |
| jroll | I think I would prefer to not do that, but I'm not sure | 16:54 |
| jroll | basically the risk profile is someone with logs access that does not have ir-api/ir-db/glance admin access | 16:54 |
| rloo | jroll: someone that has access to the logs and wants to do that. i feel like we should err on the side of caution but i dunno too. | 16:54 |
| jroll | rloo: I feel like that too | 16:55 |
| rloo | jroll: i'm passing the hot potato to you :) | 16:55 |
| jroll | nooooo | 16:55 |
| lucasagomes | lol, I'm actually OK with that | 16:57 |
| jroll | lucasagomes: okay with that risk profile, you mean? | 16:58 |
| lucasagomes | yeah | 16:58 |
| rloo | jroll, lucasagomes: i was thinking. what about yet another config to show sensitive info in debug. | 16:58 |
| *** dtantsur|brb is now known as dtantsur | 16:59 | |
| lucasagomes | cause he can only get that image, he can't modify or do anything right? | 16:59 |
| jroll | lucasagomes: sure, but that image may have private data | 16:59 |
| openstackgerrit | Debayan Ray proposed openstack/ironic: Firmware update as part of iLO manual cleaning process https://review.openstack.org/211019 | 16:59 |
| lucasagomes | jroll, say configdrive, when we log the string for it | 16:59 |
| lucasagomes | someone can also unzip it and have the config drive right? Can it have sensitive information? | 16:59 |
| lucasagomes | but that said, I kinda like rloo opnion on that | 17:00 |
| lucasagomes | rloo, maybe not even a config option but a policy thing? | 17:00 |
| lucasagomes | that we can enable/disable debug logs for sensitive information | 17:00 |
| * lucasagomes jumps on another call (I will keep an eye in the channel) | 17:00 | |
| rloo | lucasagomes: policy is based on user/tenant? | 17:00 |
| jroll | policy is for api | 17:00 |
| lucasagomes | oh true, it's only for api | 17:01 |
| * lucasagomes facepalm | 17:01 | |
| *** e0ne has quit IRC | 17:01 | |
| jroll | maybe it's fine to log, idk, I lean toward not logging it | 17:01 |
| jroll | we should probably not log configdrive either >.> | 17:01 |
| lucasagomes | yeah | 17:01 |
| rloo | i also think not logging. is this something we can ask the security group? | 17:01 |
| *** penick has joined #openstack-ironic | 17:02 | |
| jroll | we can ask them anything :) | 17:02 |
| openstackgerrit | Merged openstack/ironic-python-agent: Follow up patch for da9c3b0adc67efa916fc534d975823c0a45948a1 https://review.openstack.org/246827 | 17:02 |
| rloo | jroll: I mean, is this a 'reasonable question' to ask the security group? :D | 17:02 |
| jroll | sure! | 17:02 |
| lucasagomes | this also raises the question whether debug is the right log level for production | 17:02 |
| jroll | rloo: or even mailing list, I'm sure some folks would dogpile on it | 17:02 |
| * lucasagomes knows jroll runs it in debug | 17:02 | |
| jroll | it is because openstack is impossible to debug without it | 17:03 |
| lucasagomes | yeah | 17:03 |
| jroll | as long as nothing breaks, info is fine | 17:03 |
| jroll | but | 17:03 |
| jroll | ya know. | 17:03 |
| lucasagomes | heh | 17:03 |
| rloo | lucasagomes: we cannot prevent what values folks use in production. | 17:03 |
| rloo | lucasagomes: but we can describe what info is shown, and let them decide | 17:03 |
| lucasagomes | rloo, right | 17:04 |
| rloo | unfortunately we need to debug cuz it doesn't work perfectly :-( | 17:05 |
| lucasagomes | yeah, it's another problem that triggers this | 17:05 |
| *** mgould has joined #openstack-ironic | 17:05 | |
| lucasagomes | cause ideally, debug could have sensitive info | 17:05 |
| *** HimanshuGarg has quit IRC | 17:06 | |
| dtantsur | sensitive info can be useful for debugging too | 17:06 |
| jroll | lucasagomes: I'm just going to land this if you're okay with that https://review.openstack.org/#/c/237444/ | 17:06 |
| dtantsur | so I'd say people running in debug should secure their logs (actually I think everyone should) | 17:06 |
| jroll | dtantsur: sure, it's just yet another attack vector | 17:06 |
| lucasagomes | jroll, ++ I'm totally godo with it, I've tested locally | 17:06 |
| lucasagomes | works fine | 17:06 |
| jroll | cool | 17:06 |
| *** kozhukalov_ has quit IRC | 17:07 | |
| dtantsur | jroll, I just tend to consider logs as sensitive no matter what... we never know what information can be deduced from them | 17:07 |
| jroll | dtantsur: agree, that doesn't mean we should intentionally put credentials etc there | 17:08 |
| dtantsur | definitely... is this conversation about swift tempurls? | 17:08 |
| lucasagomes | dtantsur, yeah | 17:09 |
| dtantsur | I see. I can't make up my mind about this issue, but I do agree that logging them could be handy... could be a configuration option? | 17:10 |
| dtantsur | in some cases (e.g. tripleo) logging these should be totally fine | 17:10 |
| jroll | right | 17:10 |
| jroll | it's fine for rackspace too, as we don't support custom images | 17:10 |
| jroll | just trying to think of the general case | 17:11 |
| *** jistr has quit IRC | 17:11 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Node https://review.openstack.org/239506 | 17:12 |
| dtantsur | jroll, option defaulting to false would make at least some people happy. dunno.. | 17:13 |
| deray | jroll: sorry to break into.. registered 2 new blueprints (bp/ilo-firmware-update-in-manual-cleaning and bp/ilo-license-activate-in-manual-cleaning) .. thanks | 17:14 |
| jroll | dtantsur: yeah, I guess that's fine, I just don't like options :P | 17:14 |
| jroll | deray: perfect, thanks! | 17:14 |
| * dtantsur creates options for everything usually :D | 17:14 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Node https://review.openstack.org/239506 | 17:15 |
| *** romainh has quit IRC | 17:15 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic: Use oslo_config choices support https://review.openstack.org/226201 | 17:16 |
| jroll | deray: could you add some words to those about how it works, maybe just "add a clean step with this function signature"? | 17:18 |
| deray | jroll: will definitely do .. | 17:18 |
| *** meghal has joined #openstack-ironic | 17:19 | |
| *** mbound has quit IRC | 17:19 | |
| *** kozhukalov_ has joined #openstack-ironic | 17:19 | |
| *** HimanshuGarg has joined #openstack-ironic | 17:19 | |
| jroll | deray: thanks, I can approve them when that's done | 17:19 |
| *** daemontool has quit IRC | 17:20 | |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Node https://review.openstack.org/239506 | 17:20 |
| jroll | aarefiev: your release note on 226201 lgtm, but I think the category should be 'upgrade', not 'other' | 17:21 |
| aarefiev | jroll: I thought about upgrade at first time too, but now not sure, actually we don't need to upgrade anything | 17:22 |
| aarefiev | from doc For example, if a database schema alteration is needed. | 17:23 |
| jroll | aarefiev: but it could become a problem at upgrade time, that is for notes about things operators need to check when they upgrade | 17:23 |
| jroll | so config changes should always have an upgrade: section | 17:23 |
| aarefiev | jroll: ok | 17:24 |
| *** kromanenko has quit IRC | 17:24 | |
| *** lazy_prince has quit IRC | 17:25 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic: Use oslo_config choices support https://review.openstack.org/226201 | 17:25 |
| *** Sukhdev has quit IRC | 17:31 | |
| deray | jroll: have just hastily updated it. let me know wyt. | 17:33 |
| deray | folks, going to log off now, good day ahead | 17:34 |
| jroll | thanks deray :) | 17:34 |
| *** Sukhdev has joined #openstack-ironic | 17:36 | |
| deray | jroll: thank you! have a good day! | 17:37 |
| jroll | you too! | 17:38 |
| *** deray_ has quit IRC | 17:38 | |
| *** deray has quit IRC | 17:38 | |
| *** mgoddard has joined #openstack-ironic | 17:38 | |
| *** mgoddard_ has quit IRC | 17:38 | |
| jroll | TIL this is a thing http://grafana.openstack.org/dashboard/db/zuul-status | 17:39 |
| *** ionutbalutoiu has joined #openstack-ironic | 17:44 | |
| *** ukalifon has quit IRC | 17:49 | |
| *** jerrygb_ has joined #openstack-ironic | 17:54 | |
| *** jerrygb_ has quit IRC | 17:54 | |
| *** rloo has quit IRC | 17:54 | |
| *** verdurin has quit IRC | 17:55 | |
| *** jerrygb_ has joined #openstack-ironic | 17:55 | |
| *** mkovacik has quit IRC | 17:55 | |
| *** jerrygb has quit IRC | 17:57 | |
| sambetts | well thats pretty! | 17:57 |
| *** jerrygb_ has quit IRC | 17:59 | |
| *** achanda has quit IRC | 18:00 | |
| liliars | woo nice one! *looking forward* | 18:00 |
| *** jerrygb_ has joined #openstack-ironic | 18:01 | |
| *** jerrygb_ has quit IRC | 18:01 | |
| *** jerrygb_ has joined #openstack-ironic | 18:01 | |
| *** degorenko is now known as _degorenko | 18:02 | |
| *** achanda has joined #openstack-ironic | 18:02 | |
| *** dprince has joined #openstack-ironic | 18:03 | |
| *** achanda has quit IRC | 18:03 | |
| *** meghal has quit IRC | 18:04 | |
| *** meghal has joined #openstack-ironic | 18:04 | |
| *** meghal has quit IRC | 18:04 | |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/246090 | 18:04 |
| *** meghal has joined #openstack-ironic | 18:04 | |
| openstackgerrit | Merged openstack/python-ironicclient: Add a new cmd method node-get-vendor-passthru-methods https://review.openstack.org/237444 | 18:04 |
| openstackgerrit | Merged openstack/python-ironicclient: Add driver-get-vendor-passthru-methods cmd https://review.openstack.org/246765 | 18:04 |
| *** daemontool has joined #openstack-ironic | 18:05 | |
| JayF | lucasagomes: thanks for fixing my bug in the past (get_os_install_device used to return None in some edge cases, just looked at the upstream agent, and you fixed it when you factored out guess_root_disk) | 18:08 |
| *** mgould has quit IRC | 18:10 | |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi: Add TLS support and TSM remote video (WIP) https://review.openstack.org/246596 | 18:11 |
| *** HimanshuGarg has quit IRC | 18:11 | |
| *** Marga_ has quit IRC | 18:13 | |
| lucasagomes | JayF, hi there, morning! No problem. I'm sure you added some FIXME/TODO in the code so thank you for that :-) | 18:14 |
| JayF | lucasagomes: nope, I literally just ran over the bug, b/c we had something in a downstream hardware manager using get_os_install_device and it bugged out when None was returned instead of a string containing the block device :) | 18:14 |
| JayF | lucasagomes: we have pretty significant downstream hardware managers in the agent as I'm sure you can imagine :) | 18:14 |
| *** e0ne has joined #openstack-ironic | 18:15 | |
| lucasagomes | JayF, heh yeah I've looked at some of it actually to see the clean tasks you guys have | 18:15 |
| lucasagomes | onmetal_ironic_hardware_manager or something like that | 18:16 |
| JayF | Yeah, we actually have a more up to date version downstream | 18:16 |
| JayF | had to start using some vendor tools only given to us under NDA, so we had to close up our newer version o fit | 18:16 |
| *** daemontool has quit IRC | 18:16 | |
| JayF | we've refactored it since then to map to the new api -- one hardware manager per piece of hardware + one generic hardware manager to run things not-hardware-specific | 18:16 |
| * JayF is implementing something in the non-hw-specific hw manager now to validate no hardware is missing from any machine before OnMetal provisions it | 18:17 | |
| *** meghal has quit IRC | 18:17 | |
| *** jerrygb_ has quit IRC | 18:17 | |
| JayF | I honestly would like to talk publically more about the coolness of the hardware manager stuff we have downstream. Good stuff from JoshNang and natorious in there that not many upstream folks get to see | 18:17 |
| dtantsur | wonderful evening news: coreos IPA image goes to dracut emergency shell for me... I didn't think it even uses dracut, but ok.. | 18:18 |
| *** Sukhdev has quit IRC | 18:19 | |
| JayF | CoreOS uses dracut when booting | 18:19 |
| JayF | That's a really strange problem though | 18:19 |
| JayF | I don't think I've *ever* seen CoreOS pxe fail to boot | 18:19 |
| dtantsur | I see... so it fails with something like "systemd not found" or similar... will investigate on | 18:19 |
| JayF | I've seen IPA fail to load and other failure cases, but never it fail to run dracut | 18:19 |
| dtantsur | everything is possible in ironic-inspector gates :D | 18:19 |
| JayF | hah | 18:19 |
| sambetts | haha | 18:19 |
| JayF | If you get stuck, feel free to toss some gists my way and I'll see if I can help | 18:20 |
| lucasagomes | JayF, oh that's interesting... cause I most tools for firmware management is going to be non-foss | 18:20 |
| JayF | I know a few things about how all that stuff works | 18:20 |
| *** dprince has quit IRC | 18:20 | |
| lucasagomes | so having an abstraction to isolate those is very useful | 18:20 |
| dtantsur | yeah, I'll torture myself for tomorrow as well, then will cry for help :) thanks! | 18:20 |
| *** dprince has joined #openstack-ironic | 18:20 | |
| JayF | lucasagomes: it's almost like when I wrote the hardware manager interface to allow arbitrary things to be done I knew we'd have this problem ;) | 18:20 |
| lucasagomes | dtantsur, systemd not gound!? | 18:21 |
| lucasagomes | found* | 18:21 |
| jroll | lucasagomes: it isn't super interesting, it's like a clean step to run a thing provided by the vendor | 18:21 |
| lucasagomes | lol | 18:21 |
| jroll | not much we can abstract :P | 18:21 |
| dtantsur | lucasagomes, yeah... systemd complaining that systemd is not found, or something of the same level of insanity... I'll figure out tomorrows, maybe | 18:21 |
| JayF | lucasagomes: part of why I've always insisted on the agent being able to to drive what gets run | 18:21 |
| dtantsur | g'night everyone | 18:21 |
| JayF | dtantsur: that honestly just sounds like a bad build | 18:21 |
| jroll | night dtantsur | 18:21 |
| *** meghal has joined #openstack-ironic | 18:22 | |
| lucasagomes | jroll, yes, but that's good because if you can abstract that say in another file... you can opensource all the rest of the hardware manager except those bits | 18:22 |
| dtantsur | JayF, yeah, but I only download it, no build is supposed to happen. maybe I did something wrong, not sure.. | 18:22 |
| dtantsur | will see | 18:22 |
| sambetts | night dtantsur | 18:22 |
| *** dtantsur is now known as dtantsur|afk | 18:22 | |
| *** verdurin has joined #openstack-ironic | 18:22 | |
| jroll | lucasagomes: sure, there just isn't much left when we remove the vendor stuff, that isn't in our v1 manager | 18:22 |
| lucasagomes | oh ok, yeah that's sad heh | 18:23 |
| jroll | lucasagomes: we did build it in multiple classes so it's easy to open up some of those if needed | 18:23 |
| lucasagomes | btw JayF, if you have some time your input would be good here https://review.openstack.org/#/c/246253/ | 18:24 |
| JayF | lucasagomes: I do anticipate a time coming when we'll be able to talk more about some of the stuff in the manager. It's just a complete non-starter today | 18:25 |
| lucasagomes | and I feel like we want to translate that script to python, it keep growing | 18:25 |
| sambetts | jroll: has anyone else ever seen a devstack ironic install fail to download the required image from glance and just hang at deploying?? | 18:25 |
| lucasagomes | jroll, yeah that's perfect | 18:25 |
| *** HimanshuGarg has joined #openstack-ironic | 18:26 | |
| lucasagomes | sambetts, seems like a bug... you can see the failure in the logs but the state wans't moved to deploy fail? | 18:26 |
| sambetts | there isn't a failure in the logs, I see Master cache miss for image d9e4ee2a-138e-4fbf-bda0-76101b26cd82, starting download and then its just sat and not completed the download | 18:26 |
| JayF | lucasagomes: I really hate that code getting more complex w/o being refactored into python | 18:26 |
| jroll | sambetts: idk | 18:26 |
| lucasagomes | JayF, ++ | 18:26 |
| JayF | lucasagomes: but then you get into the loop of waiting for ironiclib, and it just gets more sticky from there | 18:27 |
| JayF | sambetts: there's little/no error checking in the agent client code | 18:27 |
| JayF | sambetts: If the agent failed in the middle of downloading the image, I wouldn't expect anything to fail it short of a timeout | 18:27 |
| sambetts | JayF: this is the conductor downloading the image for the cache | 18:27 |
| JayF | hah okay | 18:27 |
| * JayF goes back into his agent hole | 18:28 | |
| lucasagomes | problem being I don't think we have timeout for nodes in "deploying"... do we? | 18:28 |
| * lucasagomes checks | 18:28 | |
| JayF | lucasagomes: yep. | 18:28 |
| JayF | lucasagomes: that would be a correct statement | 18:28 |
| JayF | lucasagomes: and we may or may not monitor downstream for various "stuck" states that come from the lack of error handling in the agent | 18:28 |
| *** dprince has quit IRC | 18:29 | |
| *** dprince has joined #openstack-ironic | 18:29 | |
| lucasagomes | right yeah things to improve | 18:30 |
| *** HimanshuGarg has quit IRC | 18:30 | |
| *** ifarkas has quit IRC | 18:30 | |
| *** trown is now known as trown|lunch | 18:31 | |
| sambetts | lucasagomes: I can't abort a node in deploying either can I? | 18:31 |
| lucasagomes | sambetts, nop | 18:31 |
| sambetts | :'( | 18:31 |
| * lucasagomes gotta get back to the spec to be able to abort stuff while *ing | 18:32 | |
| lucasagomes | sambetts, but it worth some investigation if you can... it feels like glance should error out eventually? | 18:33 |
| lucasagomes | or maybe it did and ironic somehow did bubble that exception up to the manager.py | 18:33 |
| sambetts | lucasagomes: Yeah, I need to check all the logs | 18:33 |
| lucasagomes | didn't * | 18:33 |
| lucasagomes | sambetts, it also may be some lock in the filesystem? | 18:34 |
| lucasagomes | some leftover somewhere, cause I think we use some fs locks to help synchronize stuff | 18:35 |
| lucasagomes | maybe the code is just there waiting for it | 18:35 |
| *** meghal has quit IRC | 18:35 | |
| * lucasagomes is guessing now | 18:35 | |
| *** athomas has quit IRC | 18:36 | |
| *** alexpilotti has quit IRC | 18:36 | |
| lucasagomes | sambetts, https://github.com/openstack/oslo.concurrency/blob/master/oslo_concurrency/lockutils.py#L188-L190 | 18:37 |
| *** alexpilotti has joined #openstack-ironic | 18:40 | |
| lucasagomes | I will call it a day, have a good night everyone! | 18:41 |
| *** lucasagomes is now known as lucas-dinner | 18:41 | |
| thiagop | bye lucas-dinner, see you tomorrow | 18:41 |
| lucas-dinner | see ya :-) | 18:41 |
| *** harshs has joined #openstack-ironic | 18:43 | |
| *** HimanshuGarg has joined #openstack-ironic | 18:44 | |
| *** jerrygb has joined #openstack-ironic | 18:44 | |
| thiagop | vdrok: Hey! Did you see my comments on that OneView patch? | 18:45 |
| *** dprince has quit IRC | 18:49 | |
| JoshNang | JayF: yeah the multiple hardware manager stuff worked out really well. i'm sad i didn't get to give my talk about that (and other stuff) at tokyo | 18:53 |
| *** rloo has joined #openstack-ironic | 18:54 | |
| sambetts | cya lucas-dinner | 18:56 |
| *** mbound has joined #openstack-ironic | 18:56 | |
| *** harlowja has quit IRC | 19:05 | |
| *** harlowja has joined #openstack-ironic | 19:08 | |
| *** dprince has joined #openstack-ironic | 19:09 | |
| *** mbound has quit IRC | 19:10 | |
| *** mbound has joined #openstack-ironic | 19:12 | |
| *** dprince has quit IRC | 19:14 | |
| *** dprince has joined #openstack-ironic | 19:15 | |
| *** absubram has joined #openstack-ironic | 19:18 | |
| *** Marga_ has joined #openstack-ironic | 19:19 | |
| *** dprince has quit IRC | 19:19 | |
| *** ndipanov has quit IRC | 19:20 | |
| *** penick has quit IRC | 19:20 | |
| *** penick has joined #openstack-ironic | 19:20 | |
| *** meghal has joined #openstack-ironic | 19:23 | |
| *** bapalm has quit IRC | 19:23 | |
| *** bapalm has joined #openstack-ironic | 19:26 | |
| *** dprince has joined #openstack-ironic | 19:27 | |
| *** [1]rpioso has joined #openstack-ironic | 19:27 | |
| *** rpioso has quit IRC | 19:28 | |
| *** trown|lunch is now known as trown | 19:28 | |
| *** vishwanathj has quit IRC | 19:29 | |
| *** alexpilotti has quit IRC | 19:36 | |
| *** Sukhdev has joined #openstack-ironic | 19:38 | |
| *** alexpilotti has joined #openstack-ironic | 19:42 | |
| sambetts | night all | 19:43 |
| *** Marga_ has quit IRC | 19:47 | |
| *** Sukhdev has quit IRC | 19:49 | |
| *** Sukhdev has joined #openstack-ironic | 20:05 | |
| *** subscope has quit IRC | 20:06 | |
| *** subscope has joined #openstack-ironic | 20:06 | |
| *** subscope has quit IRC | 20:09 | |
| *** subscope has joined #openstack-ironic | 20:10 | |
| *** e0ne has quit IRC | 20:13 | |
| *** Marga_ has joined #openstack-ironic | 20:14 | |
| *** subscope has quit IRC | 20:17 | |
| sinval | night sambetts | 20:18 |
| thiagop | night sambetts | 20:18 |
| *** kozhukalov_ has quit IRC | 20:27 | |
| zer0c00l | Morning folks | 20:27 |
| *** dprince has quit IRC | 20:32 | |
| thiagop | morning zer0c00l | 20:33 |
| *** penick has quit IRC | 20:34 | |
| zer0c00l | :) | 20:35 |
| *** harlowja has quit IRC | 20:36 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/247111 | 20:39 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/247112 | 20:39 |
| *** harlowja has joined #openstack-ironic | 20:41 | |
| openstackgerrit | Merged openstack/ironic-python-agent: Create partition at max msdos limit for disks > 2TB https://review.openstack.org/246253 | 20:42 |
| *** Marga_ has quit IRC | 20:45 | |
| *** Nisha has quit IRC | 20:45 | |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: $$configuration mock https://review.openstack.org/247126 | 20:45 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Specifying target provision states in fsm https://review.openstack.org/247128 | 20:46 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/247111 | 20:47 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/247112 | 20:47 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/247130 | 20:47 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/247131 | 20:47 |
| *** verdurin has quit IRC | 20:49 | |
| *** ionutbalutoiu has quit IRC | 20:50 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/247152 | 20:51 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/247153 | 20:51 |
| *** rebase has joined #openstack-ironic | 20:53 | |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added Ironic API result transformation. https://review.openstack.org/241799 | 20:54 |
| *** Sukhdev has quit IRC | 20:55 | |
| *** sinvalv has joined #openstack-ironic | 20:56 | |
| *** Walts has joined #openstack-ironic | 20:56 | |
| Walts | Hi everyone, I have been running a POC of ironic for work, and have been deploying bare metal nodes from from a network with dhcp enabled. this is all fine but is there anyway I can statically assign IP addresses bare metal nodes | 21:00 |
| *** openstack has joined #openstack-ironic | 21:05 | |
| *** meghal has quit IRC | 21:05 | |
| *** meghal has joined #openstack-ironic | 21:05 | |
| *** meghal has quit IRC | 21:05 | |
| *** meghal has joined #openstack-ironic | 21:06 | |
| *** mbound has joined #openstack-ironic | 21:06 | |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added IronicDriverProperties resource https://review.openstack.org/247171 | 21:06 |
| *** e0ne has joined #openstack-ironic | 21:09 | |
| *** dprince has joined #openstack-ironic | 21:09 | |
| sinvalv | Walts: Hi, well, I'm not sure if you can provide static IP for your bare metal nodes, but I do know that during the deployment you'll need the DHCP anyway, he is the guy that send the nodes to the TFTP server (which contains the image to be deployed to your node) | 21:09 |
| jroll | Walts: use dhcp_provider=none and manage_tftp=False, and set up your own dhcp/tftp thing. | 21:09 |
| *** thiagop has left #openstack-ironic | 21:10 | |
| *** meghal has quit IRC | 21:10 | |
| jroll | Walts: that's for deployment anyway, there's ways for the tenant network as well (configdrive + cloud-init or similar) | 21:10 |
| *** thiagop has joined #openstack-ironic | 21:10 | |
| *** penick has joined #openstack-ironic | 21:10 | |
| *** meghal has joined #openstack-ironic | 21:10 | |
| *** jistr has joined #openstack-ironic | 21:10 | |
| *** mbound has quit IRC | 21:11 | |
| *** jerrygb_ has joined #openstack-ironic | 21:12 | |
| devananda | sambetts: for maintenance mode, wdyt of "ignore / include" ? | 21:12 |
| Walts | sinvalv, jroll: thanks I will look into those options, I was curious as to the extra specs that can be passed to the ironic port create command I wondered if an IP address could be passed in that way e.g. ronic port-create -a $MAC_ADDRESS -n $NODE_UUID -e vif_port_ID=$PORT_ID | 21:13 |
| devananda | Walts: you can pass in static IP through the config drive. I don't have a link handy for documentation on how to do it, but there is a runbook in bifrost that, I believe, does just that | 21:14 |
| devananda | Walts: no - Ironic can't set the IP itself | 21:14 |
| *** meghal has quit IRC | 21:15 | |
| Walts | devananda: thanks I will try to find it, sounds like config drive is a good bet for this | 21:15 |
| devananda | Walts: if you're using Neutron to manage the bare metal node's network, then it can set the IP | 21:15 |
| Walts | yes using neutron | 21:15 |
| *** penick has quit IRC | 21:15 | |
| *** jerrygb has quit IRC | 21:15 | |
| devananda | ah. then don't look into configdrive -- you should use neutron to manage the dhcp | 21:15 |
| Walts | devananda: sorry if this is a silly question but wont neutron still be handing out dynamic ips, it is doing this currently? | 21:17 |
| * krotscheck has somehow managed to create a circular set of dependencies in gerrit. | 21:17 | |
| devananda | krotscheck: well done | 21:17 |
| krotscheck | s/dependencies/patches/ | 21:17 |
| krotscheck | I have no idea how | 21:17 |
| *** verdurin has joined #openstack-ironic | 21:17 | |
| devananda | Walts: sorry, maybe I misunderstood. What are you using neutron for? | 21:18 |
| *** penick has joined #openstack-ironic | 21:19 | |
| *** jerrygb_ has quit IRC | 21:22 | |
| *** ijw has joined #openstack-ironic | 21:22 | |
| *** ijw has quit IRC | 21:22 | |
| Walts | devananda: my bad I'm new to this and not explaining myself well. With neutron I have a flat dhcp network mapped to a physical network, I am provisioning a bare metal node through nova which of course uses ironic, the node builds but has a dynamicly assigned IP I was just wondering if it is possible to set this statically. I if I made the network none dhcp then i would loose the ability to net boot | 21:22 |
| *** ijw has joined #openstack-ironic | 21:23 | |
| *** jerrygb_ has joined #openstack-ironic | 21:23 | |
| *** alexpilotti has quit IRC | 21:23 | |
| Walts | this may be an issue with my lack of understanding of the other openstack components not just ironic | 21:24 |
| devananda | Walts: gotcha. If you want statically assigned IPs, you would need neutron to _not_ assign dynamic IPs or manage DHCP for those nodes | 21:25 |
| *** ionutbalutoiu has joined #openstack-ironic | 21:26 | |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: $$configuration mock https://review.openstack.org/247126 | 21:26 |
| *** jerrygb__ has joined #openstack-ironic | 21:26 | |
| devananda | Walts: in that scenario, you could do what jroll suggested earlier -- in ironic, set dhcp_provider=none and manage_tftp=False, run your own dhcp service that is statically configured to respond to bootp requests with the ironic-python-agent ramdisk, and inform ironic that you'd like all nodes to boot from local disk after deployment | 21:27 |
| *** ukalifon1 has joined #openstack-ironic | 21:27 | |
| Walts | devananda: understood, thanks for you help | 21:27 |
| *** jerrygb__ has quit IRC | 21:27 | |
| *** jerrygb has joined #openstack-ironic | 21:28 | |
| *** jerrygb_ has quit IRC | 21:28 | |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added Ironic API result transformation. https://review.openstack.org/241799 | 21:30 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added IronicDriverProperties resource https://review.openstack.org/247171 | 21:30 |
| *** verdurin has quit IRC | 21:31 | |
| *** e0ne has quit IRC | 21:31 | |
| *** Marga_ has joined #openstack-ironic | 21:34 | |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Removed ports https://review.openstack.org/246578 | 21:37 |
| *** alexpilotti has joined #openstack-ironic | 21:38 | |
| *** Marga_ has quit IRC | 21:39 | |
| *** Marga_ has joined #openstack-ironic | 21:40 | |
| *** jistr has quit IRC | 21:41 | |
| *** Pavo has quit IRC | 21:41 | |
| *** Pavo has joined #openstack-ironic | 21:44 | |
| *** e0ne has joined #openstack-ironic | 21:47 | |
| *** kozhukalov_ has joined #openstack-ironic | 21:47 | |
| *** HimanshuGarg has quit IRC | 21:49 | |
| *** e0ne has quit IRC | 21:50 | |
| *** Walts has quit IRC | 21:53 | |
| *** verdurin has joined #openstack-ironic | 21:59 | |
| *** jerrygb has quit IRC | 22:00 | |
| *** penick has quit IRC | 22:01 | |
| *** HimanshuGarg has joined #openstack-ironic | 22:01 | |
| *** verdurin has quit IRC | 22:03 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Support arguments to clean step methods https://review.openstack.org/247201 | 22:06 |
| *** penick has joined #openstack-ironic | 22:24 | |
| *** harshs has quit IRC | 22:26 | |
| *** Sukhdev has joined #openstack-ironic | 22:26 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Add manual cleaning to state machine https://review.openstack.org/247213 | 22:30 |
| *** verdurin has joined #openstack-ironic | 22:31 | |
| *** cdearborn has quit IRC | 22:31 | |
| *** hemna has quit IRC | 22:35 | |
| *** penick has quit IRC | 22:36 | |
| *** sacharya has quit IRC | 22:38 | |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/python-ironicclient: Refactoring and removing duplicate code of "base.Manager" heirs https://review.openstack.org/246692 | 22:39 |
| *** ukalifon1 has quit IRC | 22:40 | |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Added IronicDriverProperties resource https://review.openstack.org/247171 | 22:54 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic-webclient: Upgraded eslint https://review.openstack.org/247223 | 22:54 |
| *** dprince has quit IRC | 23:00 | |
| *** ijw has quit IRC | 23:03 | |
| *** Sukhdev has quit IRC | 23:04 | |
| *** jerrygb has joined #openstack-ironic | 23:05 | |
| *** Piet has quit IRC | 23:05 | |
| *** penick has joined #openstack-ironic | 23:06 | |
| *** ijw has joined #openstack-ironic | 23:07 | |
| *** jerrygb has quit IRC | 23:09 | |
| *** jerrygb has joined #openstack-ironic | 23:14 | |
| *** Sukhdev has joined #openstack-ironic | 23:15 | |
| openstackgerrit | Sinval Vieira Mendes Neto proposed openstack/python-ironicclient: Refactoring and removing duplicate code of "base.Manager" heirs https://review.openstack.org/246692 | 23:15 |
| openstackgerrit | Haozhi Cui proposed openstack/python-ironicclient: update node shell help info https://review.openstack.org/246899 | 23:19 |
| *** ionutbalutoiu has quit IRC | 23:20 | |
| *** kozhukalov_ has quit IRC | 23:22 | |
| *** harshs has joined #openstack-ironic | 23:23 | |
| *** penick has quit IRC | 23:35 | |
| *** alexpilotti has quit IRC | 23:45 | |
| *** naohirot has joined #openstack-ironic | 23:51 | |
| *** smoriya_ has joined #openstack-ironic | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!