| *** praneshp has left #openstack-ironic | 00:01 | |
| *** rpioso has quit IRC | 00:06 | |
| *** rpioso has joined #openstack-ironic | 00:07 | |
| *** daemontool has joined #openstack-ironic | 00:17 | |
| *** hblixt has quit IRC | 00:19 | |
| *** rpioso has quit IRC | 00:26 | |
| saripurigopi | how can I submit changes for existing open bp from different machine? I've tried 'git fetch **' when I do git commit -a and git-review it shows two pending commits. | 00:34 |
|---|---|---|
| jlvillal | saripurigopi: Can I get more details | 00:35 |
| jlvillal | maybe a pastebin of your output | 00:35 |
| saripurigopi | I want to submit changes to existing review that is in progress. | 00:36 |
| jlvillal | saripurigopi: Can you pastebin your error messages? | 00:36 |
| saripurigopi | I've tried git clone and git fetch ssh://saripurigopi@review.openstack.org:29418/openstack/ironic-specs refs/changes/42/192142/13 && git cherry-pick FETCH_HEAD | 00:36 |
| saripurigopi | git bash-3.2$ git review | 00:37 |
| saripurigopi | You are about to submit multiple commits. This is expected if you are | 00:37 |
| saripurigopi | submitting a commit that is dependent on one or more in-review | 00:37 |
| saripurigopi | commits. Otherwise you should consider squashing your changes into one | 00:37 |
| saripurigopi | commit before submitting. | 00:37 |
| saripurigopi | The outstanding commits are: | 00:37 |
| saripurigopi | d5ed8c4 (HEAD -> master) Add out-of-band RAID configuration support for Cisco Hardware | 00:37 |
| saripurigopi | 586543f Add out-of-band RAID configuration support for Cisco Hardware | 00:37 |
| jlvillal | Prefer pastebin. for that stuff. | 00:37 |
| saripurigopi | sorry | 00:37 |
| saripurigopi | jlvillal : http://paste.openstack.org/show/486345/ | 00:37 |
| jlvillal | So you have two unmerged patches. | 00:38 |
| jlvillal | saripurigopi: Do you only want to submit one patch? | 00:39 |
| jlvillal | Ah | 00:39 |
| jlvillal | I see issue | 00:39 |
| jlvillal | Sorry. | 00:39 |
| jlvillal | The problem is that you did a 2nd commit | 00:39 |
| saripurigopi | jlvillal: yes, I'm trying this from a new machine. | 00:39 |
| jlvillal | The correct procedure to update a commit is: | 00:39 |
| jlvillal | $ git add <files> | 00:39 |
| jlvillal | $ git commit --amend | 00:39 |
| saripurigopi | right | 00:39 |
| jlvillal | This then updates the commit with the changed files | 00:40 |
| saripurigopi | okay. | 00:40 |
| jlvillal | saripurigopi: I think what you need to do now is squash it together. | 00:40 |
| jlvillal | $ git rebase --interactive HEAD^^^ | 00:40 |
| saripurigopi | jlvillal : okay, let me try. | 00:40 |
| jlvillal | Then read what it says. | 00:40 |
| jlvillal | The top commit will want to be squashed in I believe | 00:40 |
| jlvillal | saripurigopi: You should be able to change the top line 'pick' to 's' (for squash) | 00:41 |
| jlvillal | And then hopefully you now have one commit | 00:41 |
| ijw | If you want to fix what you have, try 'git rebase -i HEAD~3' - then I think you shoul dbe able to squash one into the other | 00:43 |
| ijw | I see I should read to the bottom before replying. | 00:44 |
| jlvillal | :) | 00:46 |
| jlvillal | saripurigopi: Any luck? | 00:47 |
| saripurigopi | jlvillal : I dont see that file anymore under approved directory. | 00:47 |
| jlvillal | saripurigopi: I don't understand. | 00:49 |
| jlvillal | Not sure what file you are talking about. | 00:49 |
| saripurigopi | jlvillal: looks like on track, I'm trying cherry pick of my existing change. | 00:50 |
| jlvillal | saripurigopi: Okay. Good luck! :) | 00:50 |
| jlvillal | ijw: The reviews for the console cable said it works with Mac. Well at least one review I read :) | 00:50 |
| saripurigopi | jlvillal: still seeing the same error http://paste.openstack.org/show/486348/ | 00:53 |
| jlvillal | saripurigopi: Seems like you have other patches in your tree | 00:54 |
| jlvillal | 3334dc4 Clarify stable branch testing for third party CI | 00:55 |
| jlvillal | Is that yours? | 00:55 |
| saripurigopi | no | 00:55 |
| jroll | git rebase origin/master should fix that | 00:56 |
| jlvillal | saripurigopi: Why are you doing cherry pick? Is patch in merge conflict? | 00:56 |
| saripurigopi | jlvillal: I'm not sure what method should I use to pull the exisiting review. so I've tried cherry pick. | 00:57 |
| jlvillal | saripurigopi: I try to always do checkout, unless I have a merge conflict | 00:57 |
| jlvillal | saripurigopi: The reason being to make things easier for reviewers to see changes between my patch sets. | 00:57 |
| openstackgerrit | Gopi Krishna S proposed openstack/ironic-specs: Add out-of-band RAID configuration support for Cisco Hardware https://review.openstack.org/192142 | 00:57 |
| jlvillal | If keep pulling to tip and then doing a cherry-pick, comparing patch revisions gets difficult. | 00:58 |
| saripurigopi | jlvillal: okay. will follow that. | 00:58 |
| jlvillal | saripurigopi: So I recommend the 'check-out' command line. | 00:58 |
| saripurigopi | jlvillal: okay | 00:59 |
| jlvillal | saripurigopi: Hopefully that works :) | 00:59 |
| saripurigopi | jlvillal: sure. | 01:00 |
| saripurigopi | jroll, jlvillal: able to submit my changes, please take a look @ Gopi Krishna S proposed openstack/ironic-specs: Add out-of-band RAID configuration support for Cisco Hardware https://review.openstack.org/192142 | 01:00 |
| jlvillal | saripurigopi: So reading that patch set. It looks like rebasing might be a good idea. From comment from rloo. | 01:01 |
| jlvillal | Not 100% sure though. | 01:01 |
| *** chaustin has joined #openstack-ironic | 01:01 | |
| jlvillal | saripurigopi: Since seems like need to have content in specs-not-implemented. | 01:02 |
| jlvillal | If you read comment from Ruby Loo | 01:02 |
| jlvillal | saripurigopi: You might try what jroll suggested: git rebase origin/master | 01:03 |
| *** garthb has quit IRC | 01:03 | |
| jlvillal | And then create that symbolic link as mentioned in the comment. | 01:03 |
| jlvillal | And update the patch/commit | 01:04 |
| jlvillal | And then submit it again | 01:04 |
| *** chaustin has quit IRC | 01:06 | |
| *** chaustin_ has joined #openstack-ironic | 01:06 | |
| *** jaybeale has joined #openstack-ironic | 01:11 | |
| *** chaustin_ has quit IRC | 01:11 | |
| saripurigopi | jlvillal: sure, will do. | 01:12 |
| *** izaakk has quit IRC | 01:13 | |
| openstackgerrit | Gopi Krishna S proposed openstack/ironic-specs: Add out-of-band RAID configuration support for Cisco Hardware https://review.openstack.org/192142 | 01:15 |
| *** chaustin has joined #openstack-ironic | 01:15 | |
| *** Sukhdev has quit IRC | 01:20 | |
| *** saripurigopi has quit IRC | 01:23 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Trying py27/34 with oslo-master https://review.openstack.org/277650 | 01:26 |
| *** mtanino has quit IRC | 01:34 | |
| *** Sukhdev has joined #openstack-ironic | 01:35 | |
| jlvillal | ijw, As an FYI. I just got home and hooked the cable to my EdgeRouter Lite and my MikroTik Cloud Router Switch. And the serial connection works with both of them. I am using a Linux box with minicom. | 01:45 |
| *** Sukhdev has quit IRC | 01:51 | |
| *** chaustin has quit IRC | 01:51 | |
| *** ChubYann has quit IRC | 01:56 | |
| *** chaustin has joined #openstack-ironic | 01:58 | |
| *** chaustin has quit IRC | 02:00 | |
| *** chaustin has joined #openstack-ironic | 02:00 | |
| *** chaustin has quit IRC | 02:00 | |
| *** [1]cdearborn has joined #openstack-ironic | 02:09 | |
| openstackgerrit | Arun S A G proposed openstack/ironic-specs: Add spec for deploying OS tarballs https://review.openstack.org/248968 | 02:20 |
| *** thrash is now known as thrash|g0ne | 02:22 | |
| *** Marga_ has quit IRC | 02:30 | |
| ijw | jlvillal: cheers | 02:33 |
| *** Marga_ has joined #openstack-ironic | 02:34 | |
| *** chaustin has joined #openstack-ironic | 02:38 | |
| *** Marga_ has quit IRC | 02:40 | |
| *** chaustin has quit IRC | 02:43 | |
| *** ijw has quit IRC | 02:55 | |
| *** saripurigopi has joined #openstack-ironic | 03:17 | |
| *** rloo has quit IRC | 03:19 | |
| *** dims has joined #openstack-ironic | 03:27 | |
| *** dims_ has quit IRC | 03:28 | |
| *** dims has quit IRC | 03:30 | |
| *** Nisha has joined #openstack-ironic | 03:34 | |
| *** spandhe has quit IRC | 03:37 | |
| *** achanda has joined #openstack-ironic | 03:40 | |
| *** saripurigopi has quit IRC | 03:40 | |
| *** saripurigopi1 has joined #openstack-ironic | 03:40 | |
| *** links has joined #openstack-ironic | 03:41 | |
| *** dims has joined #openstack-ironic | 03:43 | |
| *** achanda has quit IRC | 03:45 | |
| *** dims has quit IRC | 03:47 | |
| *** saripurigopi1 has quit IRC | 03:48 | |
| *** saripurigopi has joined #openstack-ironic | 03:48 | |
| *** [1]cdearborn has quit IRC | 03:53 | |
| *** Marga_ has joined #openstack-ironic | 03:54 | |
| *** Nisha has quit IRC | 04:06 | |
| *** Nisha has joined #openstack-ironic | 04:07 | |
| *** baoli_ has quit IRC | 04:11 | |
| *** shu-mutou is now known as shu-mutou-AFK | 04:31 | |
| *** daemontool has quit IRC | 04:32 | |
| *** daemontool has joined #openstack-ironic | 04:32 | |
| *** shu-mutou-AFK is now known as shu-mutou | 04:34 | |
| *** saripurigopi has quit IRC | 04:57 | |
| *** saripurigopi has joined #openstack-ironic | 04:57 | |
| *** Sukhdev has joined #openstack-ironic | 04:57 | |
| *** amotoki has joined #openstack-ironic | 05:07 | |
| *** amotoki_ has joined #openstack-ironic | 05:09 | |
| *** amotoki has quit IRC | 05:12 | |
| *** kalpase has joined #openstack-ironic | 05:12 | |
| *** amotoki_ has quit IRC | 05:15 | |
| *** daemontool_ has joined #openstack-ironic | 05:21 | |
| *** daemontool has quit IRC | 05:22 | |
| *** Marga_ has quit IRC | 05:24 | |
| *** Nisha has quit IRC | 05:24 | |
| *** saripurigopi has quit IRC | 05:31 | |
| *** saripurigopi has joined #openstack-ironic | 05:32 | |
| *** ukalifon has joined #openstack-ironic | 05:34 | |
| *** saripurigopi has quit IRC | 05:35 | |
| *** ukalifon has quit IRC | 05:40 | |
| *** ukalifon1 has joined #openstack-ironic | 05:40 | |
| *** ukalifon2 has joined #openstack-ironic | 05:41 | |
| *** ukalifon1 has quit IRC | 05:41 | |
| *** ukalifon has joined #openstack-ironic | 05:41 | |
| *** yushiro has joined #openstack-ironic | 05:45 | |
| *** jaybeale has quit IRC | 05:47 | |
| *** NobodyCam has quit IRC | 05:48 | |
| *** amotoki has joined #openstack-ironic | 05:49 | |
| *** NobodyCam has joined #openstack-ironic | 05:50 | |
| *** Marga_ has joined #openstack-ironic | 06:06 | |
| *** Nisha has joined #openstack-ironic | 06:15 | |
| *** achanda has joined #openstack-ironic | 06:18 | |
| *** ukalifon has quit IRC | 06:21 | |
| *** ijw has joined #openstack-ironic | 06:27 | |
| *** achanda has quit IRC | 06:35 | |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 06:41 |
| yushiro | vsaienko, ping | 06:42 |
| *** achanda has joined #openstack-ironic | 06:43 | |
| *** ukalifon has joined #openstack-ironic | 06:48 | |
| *** amotoki has quit IRC | 06:56 | |
| *** amotoki has joined #openstack-ironic | 06:58 | |
| *** achanda has quit IRC | 06:58 | |
| *** itamarl has joined #openstack-ironic | 07:10 | |
| *** ukalifon has quit IRC | 07:13 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add head backslash for node path patch https://review.openstack.org/276693 | 07:19 |
| *** Sukhdev has quit IRC | 07:19 | |
| *** athomas has joined #openstack-ironic | 07:40 | |
| *** deray has joined #openstack-ironic | 07:42 | |
| *** dnikishov has joined #openstack-ironic | 07:50 | |
| *** moshele has joined #openstack-ironic | 07:53 | |
| *** arturas has joined #openstack-ironic | 07:57 | |
| *** arturas has quit IRC | 07:57 | |
| *** arturas has joined #openstack-ironic | 07:58 | |
| *** jcoufal has joined #openstack-ironic | 07:59 | |
| vsaienko | yushiro: Morning | 08:01 |
| yushiro | vsaienko, Good morning. I'd like to talk you about https://review.openstack.org/#/c/139687/ | 08:02 |
| vsaienko | yushiro: sure | 08:04 |
| yushiro | vsaienko, Currently, I'm implementing ML2 plugin for my vendor's switch and testing provisioning baremetal with LAG. | 08:04 |
| yushiro | vsaienko, http://paste.openstack.org/show/486363/ This is the ironic-port and portgroup configuration. | 08:05 |
| yushiro | vsaienko, In this case, 2 neutron port created when deploying provisioning_network | 08:09 |
| vsaienko | yushiro right for port: 90:1b:0e:0e:c1:04 and port 90:1b:0e:0e:c1:05 | 08:12 |
| yushiro | vsaienko, Yeah. However, in this case, local_link_information for each neutron-port includes both physical_port information. | 08:13 |
| yushiro | vsaienko, {"local_link_information": [{"switch_info": "xxxxxx", "port_id": "1/1/0/7", "switch_id": "00:e0:00:79:a2:ad"}, {"switch_info": "xxxxxx", "port_id": "1/1/0/5", "switch_id": "00:e0:00:79:a2:ad"}]} | 08:14 |
| vsaienko | yushiro: it looks that you are using old version of the patch | 08:15 |
| yushiro | vsaienko, Oh... Just a moment, please? | 08:15 |
| *** ukalifon1 has joined #openstack-ironic | 08:17 | |
| yushiro | vsaienko, I applied NOT LATEST version patch in my environment. I'm sorry. | 08:18 |
| openstackgerrit | Debayan Ray proposed openstack/ironic: Augmenting the hashing strategy https://review.openstack.org/277408 | 08:19 |
| *** mkovacik has joined #openstack-ironic | 08:24 | |
| deray | hello and good morning Ironicers! | 08:25 |
| deray | yuriyz, hi | 08:28 |
| deray | vdrok, o/ | 08:29 |
| *** e0ne has joined #openstack-ironic | 08:49 | |
| *** ndipanov has joined #openstack-ironic | 08:55 | |
| *** bkumar has joined #openstack-ironic | 08:59 | |
| *** mbound has joined #openstack-ironic | 08:59 | |
| aarefiev | morning all! | 09:03 |
| mkovacik | morning aarefiev , guys! | 09:05 |
| aarefiev | mkovacik: morning | 09:05 |
| *** moorryan has quit IRC | 09:07 | |
| *** lucas-dinner is now known as lucasagomes | 09:10 | |
| lucasagomes | morning all :-) | 09:10 |
| *** yushiro has quit IRC | 09:12 | |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 09:15 |
| *** ifarkas has joined #openstack-ironic | 09:19 | |
| *** e0ne has quit IRC | 09:19 | |
| *** mgoddard has joined #openstack-ironic | 09:21 | |
| *** vaishnavi has joined #openstack-ironic | 09:27 | |
| *** arturas has quit IRC | 09:28 | |
| *** rcarrillocruz has joined #openstack-ironic | 09:28 | |
| openstackgerrit | Merged openstack/ironic: Update ironic.conf.samle https://review.openstack.org/276631 | 09:35 |
| *** max_lobur has quit IRC | 09:38 | |
| *** daemontool has joined #openstack-ironic | 09:39 | |
| *** derekh has joined #openstack-ironic | 09:39 | |
| openstackgerrit | Merged openstack/ironic: Log warning if ipmi_username/ipmi_password missing https://review.openstack.org/273891 | 09:39 |
| *** daemontool_ has quit IRC | 09:40 | |
| openstackgerrit | Merged openstack/ironic: Disable clean step 'reset_ilo' for iLO drivers by default https://review.openstack.org/277288 | 09:41 |
| *** dtantsur|afk is now known as dtantsur | 09:42 | |
| dtantsur | Morning Ironic | 09:42 |
| * dtantsur forgot to open IRC in the morning | 09:42 | |
| aarefiev | lucasagomes, dtantsur: morning | 09:46 |
| lucasagomes | dtantsur, morning | 09:46 |
| lucasagomes | dtantsur, re docs, staging can't not use docs.openstack.org :-( only official projects can | 09:46 |
| *** mbound has quit IRC | 09:47 | |
| dtantsur | lucasagomes, why aren't we under ironic unbrella? | 09:47 |
| lucasagomes | dtantsur, talking to jroll we agreed that ironic-core is not responsible for the code in there | 09:47 |
| lucasagomes | so we are not under ironic's umbrella | 09:47 |
| dtantsur | lucasagomes, how is this related? | 09:47 |
| dtantsur | ironic-core is also not responsible for python-dracclient and ironic-inspector | 09:48 |
| dtantsur | and for web UI work | 09:48 |
| lucasagomes | dtantsur, yeah it's something we can talk about | 09:48 |
| lucasagomes | dtantsur, or even pyghmi | 09:48 |
| dtantsur | I think we understood project umbrella somewhat wider than what core team is responsible for | 09:48 |
| *** Nisha has quit IRC | 09:48 | |
| lucasagomes | in any case, the agreement *right now* is that ironic staging drivers is not under ironic's umbrella | 09:49 |
| dtantsur | well, I +2ed that patch :) | 09:49 |
| lucasagomes | dtantsur, http://ironic-staging-drivers.readthedocs.org/en/latest/README.html#what-the-ironic-staging-drivers-is-not | 09:49 |
| lucasagomes | dtantsur, yeah, we can talk with jroll to see his position on making it part of the umbrealla, but until then we have to keep it out | 09:50 |
| * lucasagomes is ok just having a place to put stuff | 09:50 | |
| *** NobodyCam has quit IRC | 09:52 | |
| *** mbound has joined #openstack-ironic | 09:55 | |
| aarefiev | dtantsur: hi, have a min? question about discovery testing | 09:57 |
| dtantsur | aarefiev, yep | 09:57 |
| *** max_lobur has joined #openstack-ironic | 09:57 | |
| aarefiev | dtantsur: AFAIU discovery won't work for dib ramdisk | 09:57 |
| dtantsur | why? | 09:57 |
| *** deray has quit IRC | 09:59 | |
| aarefiev | dtantsur: I'm trying to figure out why dib job don't send introspection data https://review.openstack.org/#/c/276107 | 09:59 |
| *** NobodyCam has joined #openstack-ironic | 09:59 | |
| aarefiev | dtantsur: actually I have never used dib ramdisk thought | 10:02 |
| dtantsur | aarefiev, maybe the wrong vm gets started? for ipa we have only one vm in gate.. | 10:02 |
| dtantsur | anyway, I'm fine if the new job will be for IPA only (actually I don't think we should create more jobs for DIB) | 10:02 |
| dtantsur | aarefiev, this is suspicious: http://logs.openstack.org/07/276107/8/check/gate-ironic-inspector-dsvm-dib/3c6ca9c/console.html#_2016-02-08_15_08_30_176 | 10:03 |
| aarefiev | dtantsur: hmm, missing this; so I should create new job with IPA only | 10:06 |
| aarefiev | right? | 10:06 |
| dtantsur | aarefiev, yep, it should be a new job (non-voting for now), which will enable the hook and run a new exercise script | 10:07 |
| aarefiev | dtantsur: ok, about test scenario, | 10:08 |
| aarefiev | verify only it was enrolled in ironic | 10:08 |
| dtantsur | yep, lets keep it simple | 10:08 |
| aarefiev | dtantsur: ok, thanks, will do | 10:09 |
| *** Marga_ has quit IRC | 10:12 | |
| *** electrofelix has joined #openstack-ironic | 10:18 | |
| openstackgerrit | Merged openstack/ironic: Agent vendor handles manual cleaning https://review.openstack.org/247695 | 10:20 |
| openstackgerrit | Merged openstack/ironic: Updated iLO documentation for boot mode capability https://review.openstack.org/274618 | 10:21 |
| *** e0ne has joined #openstack-ironic | 10:21 | |
| dtantsur | awesome ^^ | 10:21 |
| openstackgerrit | Sergii Turivnyi proposed openstack/python-ironicclient: Add sanity tests for testing actions with Chassis https://review.openstack.org/240562 | 10:21 |
| *** deray has joined #openstack-ironic | 10:24 | |
| *** deray has left #openstack-ironic | 10:29 | |
| *** deva_ has joined #openstack-ironic | 10:29 | |
| *** deray has joined #openstack-ironic | 10:30 | |
| deray | lucasagomes, I have posted my comments in https://review.openstack.org/#/c/277408 | 10:30 |
| lucasagomes | deray, thanks will take a look | 10:31 |
| deray | lucasagomes, cool :) | 10:31 |
| *** NobodyCa1 has joined #openstack-ironic | 10:33 | |
| *** samueldmq1 has joined #openstack-ironic | 10:33 | |
| *** amotoki has quit IRC | 10:34 | |
| lucasagomes | deray, answers looks good. But would be nice to have them in the commit message, so folks looking at this change will understand the motivations | 10:36 |
| lucasagomes | git log is our history of changes, so it we have to document it there | 10:37 |
| *** samueldmq1 has quit IRC | 10:38 | |
| *** NobodyCa1 has quit IRC | 10:38 | |
| *** deva_ has quit IRC | 10:38 | |
| deray | lucasagomes, I have updated the commit message stating ``md5`` being the default.. | 10:38 |
| lucasagomes | deray, it's ok to say that the method wasn't used at all, and therefore changing the default algorithm is fine | 10:38 |
| deray | lucasagomes, okay.. | 10:39 |
| lucasagomes | deray, thanks for that | 10:40 |
| openstackgerrit | Merged openstack/ironic: Show transitions initiated by API requests https://review.openstack.org/259099 | 10:45 |
| *** dims has joined #openstack-ironic | 10:45 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [WIP] Do not update firewall rules if list of MAC's did not change https://review.openstack.org/277777 | 10:47 |
| *** sambetts has joined #openstack-ironic | 10:48 | |
| *** dims has quit IRC | 10:49 | |
| *** amotoki has joined #openstack-ironic | 10:49 | |
| *** mgould has joined #openstack-ironic | 10:49 | |
| openstackgerrit | Debayan Ray proposed openstack/ironic: Augmenting the hashing strategy https://review.openstack.org/277408 | 10:49 |
| *** dims has joined #openstack-ironic | 10:50 | |
| sambetts | dtantsur: I've just had to fix a colleagues devstack with inspector by changing the os_auth_url back to having /v2.0 on the end again, has something changed in devstack again making that not work | 10:50 |
| dtantsur | >_< | 10:51 |
| deray | lucasagomes, done. Updated the commit message. tyt to check. | 10:51 |
| *** dprince has joined #openstack-ironic | 10:52 | |
| lucasagomes | deray, ta much! Will take a look in a few | 10:52 |
| mgould | morning Ironic | 10:53 |
| deray | gm, mgould | 10:54 |
| *** amotoki has quit IRC | 10:55 | |
| mgould | deray, morning | 10:56 |
| deray | mgould, o/ | 10:56 |
| *** ukalifon1 has quit IRC | 10:57 | |
| sambetts | dtantsur: the error we were seeing on the inspector side was a 404 can not find resource from keystone | 11:00 |
| dtantsur | facepalm.... | 11:00 |
| *** mbound has quit IRC | 11:02 | |
| deray | lucasagomes, just in case you have some free time, will appreciate if you can have a look at: https://review.openstack.org/#/c/211019/ | 11:13 |
| *** vaishnavi has quit IRC | 11:13 | |
| *** mbound has joined #openstack-ironic | 11:15 | |
| *** dnikishov has quit IRC | 11:19 | |
| *** dnikishov has joined #openstack-ironic | 11:20 | |
| *** ukalifon1 has joined #openstack-ironic | 11:22 | |
| *** pas-ha has quit IRC | 11:25 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Replace ifconfig with ip https://review.openstack.org/277791 | 11:25 |
| *** shu-mutou is now known as shu-mutou-AFK | 11:28 | |
| *** bkumar has quit IRC | 11:32 | |
| openstackgerrit | Satoru Moriya proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 11:35 |
| *** baoli has joined #openstack-ironic | 11:36 | |
| *** baoli has quit IRC | 11:37 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Remove deprecated support for passing patches lists into hooks https://review.openstack.org/277796 | 11:38 |
| *** jcoufal has quit IRC | 11:38 | |
| vdrok | morning ironic! and morning deray mkovacik lucasagomes dtantsur and sambetts | 11:39 |
| sambetts | o/ vdrok | 11:39 |
| deray | gm, vdrok :) | 11:39 |
| mkovacik | vdrok, morning! | 11:39 |
| mkovacik | and morning all! :) | 11:39 |
| dtantsur | morning vdrok, mkovacik, and everyone else :) | 11:40 |
| *** dims_ has joined #openstack-ironic | 11:40 | |
| deray | gm, dtantsur o/ | 11:40 |
| *** dims has quit IRC | 11:40 | |
| sambetts | dtantsur: https://review.openstack.org/#/c/274703/ but I thought that the method we implemented for the inspector worked for either version :/ | 11:41 |
| lucasagomes | vdrok, morning | 11:41 |
| dtantsur | sambetts, it still works in our gate | 11:41 |
| dtantsur | so I dunno | 11:42 |
| sambetts | Yeah I have no idea... | 11:42 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Do not update firewall rules if list of MAC's did not change https://review.openstack.org/277777 | 11:42 |
| dtantsur | sambetts, I got complaints that we're spamming with iptables calls, so ^^ | 11:42 |
| sambetts | dtantsur: I kind of agree :-P it did make the logs hard to read | 11:43 |
| dtantsur | yep | 11:43 |
| dtantsur | I even wonder, if we merge this, would it qualify for a liberty backport | 11:43 |
| *** boris-42 has quit IRC | 11:43 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introduce API for aborting introspection https://review.openstack.org/265275 | 11:49 |
| sambetts | yeah it definitely has potential and its only an optimisation not a feature change | 11:49 |
| *** MattMan has quit IRC | 11:50 | |
| *** xek_ is now known as xek | 11:51 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introduce API for aborting introspection https://review.openstack.org/265275 | 11:54 |
| *** trown has joined #openstack-ironic | 11:58 | |
| *** trown has quit IRC | 11:59 | |
| *** ukalifon1 has quit IRC | 12:00 | |
| *** trown has joined #openstack-ironic | 12:00 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 12:02 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 12:02 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 12:02 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 12:02 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 12:02 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 12:02 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 12:02 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 12:02 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 12:02 |
| *** trown is now known as trown|outtypewww | 12:02 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add network provider interface and implementations https://review.openstack.org/139687 | 12:02 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Add a missing unit test on 'introspection status' command https://review.openstack.org/277807 | 12:05 |
| openstackgerrit | Satoru Moriya proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 12:09 |
| *** dims_ has quit IRC | 12:11 | |
| *** lucasagomes is now known as lucas-hungry | 12:13 | |
| *** _degorenko|afk is now known as degorenko | 12:16 | |
| *** dims_ has joined #openstack-ironic | 12:19 | |
| TheJulia | Good morning | 12:24 |
| deray | TheJulia, good morning :) | 12:26 |
| *** vinm213 has joined #openstack-ironic | 12:27 | |
| *** pas-ha has joined #openstack-ironic | 12:27 | |
| sambetts | o/ TheJulia | 12:27 |
| *** mtanino has joined #openstack-ironic | 12:28 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introduce API for aborting introspection https://review.openstack.org/265275 | 12:34 |
| *** openstackgerrit_ has joined #openstack-ironic | 12:39 | |
| *** jcoufal has joined #openstack-ironic | 12:44 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introduce API for aborting introspection https://review.openstack.org/265275 | 12:44 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Update example.conf https://review.openstack.org/277811 | 12:48 |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introduce API for aborting introspection https://review.openstack.org/265275 | 12:49 |
| *** Marga_ has joined #openstack-ironic | 12:55 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Try to fetch inspector URL from the service catalog https://review.openstack.org/277814 | 12:56 |
| openstackgerrit | Satoru Moriya proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 12:56 |
| *** thrash|g0ne is now known as thrash | 12:57 | |
| *** trown|outtypewww is now known as trown | 12:58 | |
| openstackgerrit | Moshe Levi proposed openstack/ironic: Adding InfiniBand Support https://review.openstack.org/264263 | 12:59 |
| openstackgerrit | Satoru Moriya proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 13:01 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Try to fetch inspector URL from the service catalog https://review.openstack.org/277814 | 13:05 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Add a missing unit test on 'introspection status' command https://review.openstack.org/277807 | 13:06 |
| *** lucas-hungry is now known as lucasagomes | 13:07 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Try to fetch inspector URL from the service catalog https://review.openstack.org/277814 | 13:13 |
| *** deray has quit IRC | 13:13 | |
| openstackgerrit | Satoru Moriya proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 13:14 |
| *** links has quit IRC | 13:17 | |
| *** deray has joined #openstack-ironic | 13:17 | |
| *** smoriya_ has quit IRC | 13:18 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Try to fetch inspector URL from the service catalog https://review.openstack.org/277814 | 13:20 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironic-inspector-client: Try to fetch inspector URL from the service catalog https://review.openstack.org/277814 | 13:23 |
| *** chaustin has joined #openstack-ironic | 13:37 | |
| *** chaustin has left #openstack-ironic | 13:38 | |
| *** chaustin has joined #openstack-ironic | 13:38 | |
| *** chaustin has quit IRC | 13:41 | |
| *** daemontool has quit IRC | 13:42 | |
| *** MattMan has joined #openstack-ironic | 13:44 | |
| *** [1]cdearborn has joined #openstack-ironic | 13:46 | |
| *** chaustin has joined #openstack-ironic | 13:47 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add auto-discovery feature https://review.openstack.org/262199 | 13:51 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add discover nodes exercise https://review.openstack.org/276107 | 13:51 |
| *** ukalifon has joined #openstack-ironic | 13:52 | |
| *** vinm213 has quit IRC | 13:53 | |
| *** openstackgerrit_ has quit IRC | 13:56 | |
| *** daemontool has joined #openstack-ironic | 13:59 | |
| *** cdearborn has joined #openstack-ironic | 14:00 | |
| *** dims_ has quit IRC | 14:03 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Do not update firewall rules if list of MAC's did not change https://review.openstack.org/277777 | 14:05 |
| dtantsur | mkovacik, ^^ | 14:05 |
| *** kalpase has left #openstack-ironic | 14:06 | |
| *** mtanino has quit IRC | 14:07 | |
| trown | dtantsur: saw this on HN this morning and thought of you :p https://deadlockempire.github.io/ | 14:08 |
| mkovacik | dtantsur, ack | 14:09 |
| * mkovacik looks | 14:09 | |
| *** dims has joined #openstack-ironic | 14:10 | |
| dtantsur | lol | 14:11 |
| *** baoli has joined #openstack-ironic | 14:14 | |
| *** [1]cdearborn has quit IRC | 14:15 | |
| *** daemontool_ has joined #openstack-ironic | 14:16 | |
| *** links has joined #openstack-ironic | 14:16 | |
| *** baoli_ has joined #openstack-ironic | 14:17 | |
| *** daemontool has quit IRC | 14:17 | |
| *** rloo has joined #openstack-ironic | 14:18 | |
| lucasagomes | trown, that's quite awesome | 14:20 |
| *** baoli has quit IRC | 14:20 | |
| trown | lucasagomes: ya I may actually learn something from that | 14:20 |
| lucasagomes | "You've done it, Scheduler! The armies have broken apart and our soldiers are driving them back. But do not grow complacent - this is merely the beginning of a war - The Great Concurrency War - and we will win it!" | 14:20 |
| lucasagomes | hah | 14:21 |
| *** amotoki has joined #openstack-ironic | 14:21 | |
| *** NobodyCa1 has joined #openstack-ironic | 14:22 | |
| *** chaustin has quit IRC | 14:24 | |
| *** chaustin has joined #openstack-ironic | 14:24 | |
| aarefiev | dtantsur: about this one https://review.openstack.org/#/c/277843/ , is there any reason to have separate python-ironic-inspector-client-dsvm | 14:28 |
| dtantsur | aarefiev, it used python-ironic-inspector-client from source, not from pypi | 14:28 |
| openstackgerrit | Moshe Levi proposed openstack/ironic: Adding InfiniBand Support https://review.openstack.org/264263 | 14:28 |
| *** stendulker has joined #openstack-ironic | 14:29 | |
| openstackgerrit | Sam Betts proposed openstack/ironic-specs: Add spec for VLAN Aware Baremetal Instances https://review.openstack.org/277853 | 14:29 |
| *** NobodyCa1 has quit IRC | 14:29 | |
| aarefiev | dtantsur: as I get your point, you propose to rename arg ramdisk -> type, and add if type == discovery, right? | 14:29 |
| dtantsur | aarefiev, yep. but I don't insist actually, maybe it would be too much a mess | 14:30 |
| *** amotoki has quit IRC | 14:31 | |
| *** chaustin has quit IRC | 14:32 | |
| aarefiev | dtantsur: only 2 additional if statement :) | 14:32 |
| sambetts | lucasagomes: is it expected for tox -e docs to fail on ironic-specs because of a warning about the specs/not-implemented files? | 14:33 |
| *** chaustin_ has joined #openstack-ironic | 14:33 | |
| *** mgould has quit IRC | 14:34 | |
| NobodyCam | good morning dtantsur lucasagomes jroll devananda TheJulia jlvillal sambetts vdrok rloo mgould and all others Ironicers not listed here :) | 14:34 |
| dtantsur | morning NobodyCam | 14:34 |
| NobodyCam | :p | 14:34 |
| sambetts | p/ NobodyCam | 14:34 |
| NobodyCam | hehehe :) | 14:35 |
| *** amotoki has joined #openstack-ironic | 14:35 | |
| *** chaustin_ has quit IRC | 14:36 | |
| jlvillal | Good morning NobodyCam dtantsur sambetts aarefiev lucasagomes trown sambetts TheJulia deray mkovacik and everyone else | 14:36 |
| *** aline_ has joined #openstack-ironic | 14:36 | |
| dtantsur | morning jlvillal | 14:36 |
| mkovacik | jlvillal, o/ | 14:36 |
| jroll | morning y'all | 14:36 |
| jlvillal | :) | 14:36 |
| NobodyCam | o/ | 14:36 |
| deray | gm NobodyCam, jlvillal and all :) | 14:36 |
| deray | jroll, gm :) | 14:37 |
| *** chaustin_ has joined #openstack-ironic | 14:37 | |
| *** amotoki has quit IRC | 14:37 | |
| NobodyCam | morning deray | 14:39 |
| *** ijw has quit IRC | 14:39 | |
| deray | NobodyCam, o/ :) | 14:39 |
| rloo | hi and a cool mornin' NobodyCam, sambetts, aarefiev, dtantsur, jlvillal, mkovacik, jroll, deray, lucasagomes, trown and all the other fine folks out there | 14:39 |
| NobodyCam | :) | 14:39 |
| deray | hey rloo, g'morning | 14:39 |
| jlvillal | hi rloo jroll | 14:39 |
| mkovacik | rloo, o/ :) | 14:40 |
| jroll | \o | 14:40 |
| aarefiev | NobodyCam, jlvillal, jroll, rloo: good morning | 14:41 |
| sambetts | o/ rloo | 14:41 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: API to manually clean nodes https://review.openstack.org/251995 | 14:41 |
| *** amotoki has joined #openstack-ironic | 14:41 | |
| NobodyCam | good morning aarefiev :) | 14:41 |
| *** amotoki has quit IRC | 14:41 | |
| *** chaustin_ has quit IRC | 14:43 | |
| *** awiddersheim has joined #openstack-ironic | 14:44 | |
| deray | rloo, could you have a look at-> if your time permits.. https://review.openstack.org/#/c/211019/ | 14:44 |
| *** moshele has quit IRC | 14:44 | |
| *** moshele has joined #openstack-ironic | 14:46 | |
| *** stendulker has quit IRC | 14:46 | |
| *** chaustin has joined #openstack-ironic | 14:48 | |
| rloo | deray: sure. time doesn't permit now though; I'm trying to review the high priority items. Also, I won't review this until after the manual cleaning patches get merged since this doesn't make sense w/o those. | 14:48 |
| vdrok | morning NobodyCam jlvillal jroll rloo ! | 14:48 |
| dtantsur | rloo, should I W+1 https://review.openstack.org/#/c/251995/ ? | 14:48 |
| rloo | dtantsur: yes please | 14:49 |
| *** mgould has joined #openstack-ironic | 14:49 | |
| dtantsur | done! | 14:49 |
| rloo | dtantsur: thanks! | 14:49 |
| dtantsur | we're on track for getting it in Mitaka, I guess :) | 14:50 |
| rloo | dtantsur: yes. I'm trying to get most of our priority items in for Mitaka... http://specs.openstack.org/openstack/ironic-specs/priorities/mitaka-priorities.html | 14:51 |
| *** chaustin has left #openstack-ironic | 14:51 | |
| *** chaustin has joined #openstack-ironic | 14:51 | |
| deray | rloo, sure. Regarding manual cleaning any item you are waiting to get merged barring this (API https://review.openstack.org/#/c/251995/)? Is cli also supposed to get into? | 14:51 |
| rloo | dtantsur: ok, maybe I'm trying to get half of our priorities in for M. | 14:51 |
| *** amit213 has quit IRC | 14:51 | |
| dtantsur | we have good chances :) the futurist thing is moving slowly tho | 14:52 |
| rloo | deray: the cli should get in too but that isn't a blocker (for me) wrt reviewing other manual-clean related patches. | 14:52 |
| rloo | deray: i think i have to update the cli patch too but want to do some reviewing of the network patches first. | 14:52 |
| *** chaustin has left #openstack-ironic | 14:52 | |
| *** chaustin has joined #openstack-ironic | 14:53 | |
| *** chaustin has left #openstack-ironic | 14:53 | |
| *** chaustin has joined #openstack-ironic | 14:53 | |
| *** chaustin has left #openstack-ironic | 14:53 | |
| *** chaustin has joined #openstack-ironic | 14:53 | |
| *** mtanino has joined #openstack-ironic | 14:54 | |
| *** chaustin has left #openstack-ironic | 14:54 | |
| *** chaustin has joined #openstack-ironic | 14:54 | |
| *** chaustin has left #openstack-ironic | 14:54 | |
| *** links has quit IRC | 14:54 | |
| *** chaustin has joined #openstack-ironic | 14:54 | |
| openstackgerrit | Milan Kováčik proposed openstack/ironic-inspector: Introduce API for aborting introspection https://review.openstack.org/265275 | 14:56 |
| deray | rloo, makes sense | 14:57 |
| *** chaustin has left #openstack-ironic | 14:58 | |
| *** chaustin has joined #openstack-ironic | 14:58 | |
| dtantsur | sambetts, will you have a second to take a look at ^^ | 14:59 |
| dtantsur | please? | 14:59 |
| rcarrillocruz | hi folks | 14:59 |
| lucasagomes | rloo, jlvillal morning | 14:59 |
| rcarrillocruz | question | 14:59 |
| *** mgoddard_ has joined #openstack-ironic | 14:59 | |
| rcarrillocruz | what process or mechanism does create the /httpboot/pxelinux.cfg/<mac address> on bifrost deploys | 14:59 |
| rcarrillocruz | ? | 14:59 |
| *** chaustin has left #openstack-ironic | 15:00 | |
| dtantsur | rcarrillocruz, on any ironic deploys this is created by ironic-conductor itself (the pxe or agent driver) | 15:00 |
| *** chaustin has joined #openstack-ironic | 15:00 | |
| rcarrillocruz | ok | 15:00 |
| rcarrillocruz | so | 15:00 |
| rcarrillocruz | on two new servers | 15:01 |
| rcarrillocruz | i see ironic-conductor creating file for just one nic | 15:01 |
| rcarrillocruz | not the two that both servers have | 15:01 |
| rcarrillocruz | which eventually cause the pxe boot to fail, as there's no corresponding pxelinux.cfg/<mac> file to match | 15:01 |
| rcarrillocruz | any hint dtantsur ? | 15:01 |
| dtantsur | rcarrillocruz, do you have ironic ports for both nics? | 15:02 |
| *** mgoddard has quit IRC | 15:02 | |
| rcarrillocruz | hmmm... o-k, i think it must be some idempontent issue with the enroll playbook | 15:02 |
| rcarrillocruz | fair enough, let me delete the record altogether | 15:03 |
| rcarrillocruz | thx! | 15:03 |
| rcarrillocruz | i think you put me on good track | 15:03 |
| *** dtantsur is now known as dtantsur|brb | 15:03 | |
| sambetts | rloo: Do you know if I've missed something in this patch which is causing docs to fail https://review.openstack.org/#/c/277853/ I think I've done things right :/ | 15:05 |
| TheJulia | rcarrillocruz: the actual deploy process will create the mac address association based on ports supplied by enrollment | 15:06 |
| rcarrillocruz | yeah...however, i've been adding/removing the prov mac to inventory | 15:06 |
| rcarrillocruz | reprovisioned | 15:06 |
| rcarrillocruz | yet just one mac created on pxelinux.cfg | 15:07 |
| rcarrillocruz | which makes me think the enrollment port creation may not be idempotent | 15:07 |
| rcarrillocruz | i did a node-delete to afresh | 15:07 |
| TheJulia | did you somehow get a stray port created? | 15:07 |
| rcarrillocruz | on first enrollment (which makes the node/port creation), the node only had one mac | 15:08 |
| *** [1]cdearborn has joined #openstack-ironic | 15:08 | |
| rcarrillocruz | then prov failed | 15:08 |
| rcarrillocruz | then added back the prov mac to inventory | 15:08 |
| rcarrillocruz | reprovisioned, but failed again | 15:08 |
| rcarrillocruz | and spotted the pxelinux.cfg/<mac> for prov was not being created | 15:08 |
| *** ijw has joined #openstack-ironic | 15:08 | |
| TheJulia | that doesn't seem right since it should be parsing and doing multiple macs, unless there is an orphaned port entry somehow. | 15:09 |
| TheJulia | anything in last_error for the node that fails to deploy? | 15:09 |
| *** chaustin has left #openstack-ironic | 15:10 | |
| *** chaustin has joined #openstack-ironic | 15:10 | |
| rcarrillocruz | hmm,... enroll afresh doesn't create the port no... | 15:10 |
| rcarrillocruz | even though it's there | 15:10 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add auto-discovery feature https://review.openstack.org/262199 | 15:10 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add discover nodes exercise https://review.openstack.org/276107 | 15:10 |
| rcarrillocruz | can i see what ports are associated to a node | 15:10 |
| rcarrillocruz | something like ironic-node show verbose or somth | 15:10 |
| rcarrillocruz | ? | 15:10 |
| TheJulia | ironic port-list | 15:10 |
| TheJulia | with --detail | 15:11 |
| rcarrillocruz | k, thx | 15:13 |
| TheJulia | althought the re-assertion of the node should fail if there is a port conflict | 15:13 |
| rloo | sambetts: was on phone. looking now... | 15:13 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 15:14 |
| rcarrillocruz | hmm, yeah | 15:15 |
| rcarrillocruz | ironic port-list --detail |grep <node uuid> | 15:15 |
| rcarrillocruz | that shows one nic | 15:15 |
| rcarrillocruz | that's odd... | 15:15 |
| TheJulia | any ports missing nodes? | 15:15 |
| rloo | sambetts: nothing obvious to me. | 15:17 |
| sambetts | rloo: :( | 15:18 |
| rloo | sambetts: is the file in not-implemented a soft link? it doesn't look like it | 15:19 |
| rloo | sambetts: yeah, it is actually a file with ../approved/... in it. | 15:19 |
| *** raddaoui_ has joined #openstack-ironic | 15:19 | |
| *** raddaoui has joined #openstack-ironic | 15:19 | |
| *** raddaoui__ has joined #openstack-ironic | 15:19 | |
| sambetts | rloo: ooh, I'm looking at the other patches for ironic-specs and I didn't realise it was a ln -s | 15:20 |
| rloo | sambetts: it should be 'ln -s ../approved/...' | 15:20 |
| rcarrillocruz | TheJulia: no, all rows containing UUID... | 15:20 |
| rloo | sambetts: yup. | 15:20 |
| sambetts | rloo: they seem to appear as real files in gerrit :-P | 15:21 |
| rcarrillocruz | i wonder if i hit some sort of port limit ? | 15:21 |
| rcarrillocruz | i'll double check ironic logs during enroll | 15:21 |
| rloo | sambetts: yeah, i only noticed it when I downloaded it. | 15:21 |
| *** piet has joined #openstack-ironic | 15:22 | |
| *** saripurigopi has joined #openstack-ironic | 15:22 | |
| openstackgerrit | Debayan Ray proposed openstack/ironic: Firmware update(iLO) as manual cleaning step https://review.openstack.org/211019 | 15:22 |
| *** cdearborn has quit IRC | 15:22 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Cache agent clean steps on node https://review.openstack.org/267918 | 15:23 |
| openstackgerrit | Sam Betts proposed openstack/ironic-specs: Add spec for VLAN Aware Baremetal Instances https://review.openstack.org/277853 | 15:24 |
| *** mgoddard_ has quit IRC | 15:26 | |
| *** mgoddard has joined #openstack-ironic | 15:26 | |
| *** chaustin has quit IRC | 15:29 | |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 15:29 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 15:30 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: During cleaning, store clean step index https://review.openstack.org/270986 | 15:32 |
| deray | jroll, just got to see your comment on the hashing strategy change.. | 15:34 |
| *** saripurigopi has quit IRC | 15:34 | |
| *** saripurigopi1 has joined #openstack-ironic | 15:34 | |
| deray | jroll, it is getting used here: https://review.openstack.org/#/c/211019/18/ironic/drivers/modules/ilo/common.py I705 | 15:34 |
| deray | jroll, as this oob firmware update patch is turning out to be already a big one.. thatsy thought of raising a new one for the modification of hashing strategy. | 15:38 |
| rcarrillocruz | TheJulia: glean honours the order of the nic entries of the inventory, does it? | 15:40 |
| rcarrillocruz | i.e. | 15:40 |
| rcarrillocruz | first nic i put on inventory for a node, glean will create eth0 | 15:40 |
| rcarrillocruz | second eth1 | 15:40 |
| rcarrillocruz | etc | 15:40 |
| rcarrillocruz | ? | 15:40 |
| rcarrillocruz | double-checking something | 15:40 |
| *** ijw has quit IRC | 15:41 | |
| *** krtaylor has quit IRC | 15:42 | |
| deray | jroll, should I place a 'Depends On' of hashing (https://review.openstack.org/#/c/277408) in the firmware update patch (https://review.openstack.org/#/c/211019) | 15:42 |
| jroll | deray: meh, I guess it's fine then... but yeah those need to depend on each other | 15:42 |
| jroll | deray: you know if it's in the same project, you can just chain the patches instead of depends-on, right? :) | 15:43 |
| jroll | as in, have two commits in a row and submit them together | 15:43 |
| jroll | much easier for reviewers to see too | 15:43 |
| *** izaakk has joined #openstack-ironic | 15:43 | |
| rcarrillocruz | agree, depends-on is really meant for cross-project dependency handling | 15:44 |
| * jlvillal just always agrees with jroll. Makes jlvilllal's life simpler :) | 15:44 | |
| jroll | that's a terrible idea :D | 15:44 |
| jlvillal | :P | 15:45 |
| *** chaustin has joined #openstack-ironic | 15:45 | |
| *** chaustin_ has joined #openstack-ironic | 15:45 | |
| deray | jroll, any idea link to educate myself on patch chaining? | 15:45 |
| *** saripurigopi1 has quit IRC | 15:45 | |
| deray | *any link | 15:45 |
| jroll | deray: it's just making two commits in a row | 15:45 |
| jlvillal | deray: I think that is about it. | 15:46 |
| jlvillal | Make a commit | 15:46 |
| jlvillal | make a 2nd commit | 15:46 |
| jroll | git checkout master; git checkout -b my-branch; <do work>; git commit; <do work>; git commit; git review | 15:46 |
| jlvillal | and push it (git review) | 15:46 |
| rloo | our ironic.conf.sample *was* up to date earlier today, but a patch merged that makes it out of date again. Sigh. | 15:46 |
| jlvillal | Inconceivable! | 15:46 |
| jroll | deray: it's fine for now, if you don't want to shuffle the patches around, just as a note for the future, it's much easier for both devs and reviewers :) | 15:46 |
| rloo | jlvillal: princess bride sez yes (i forgot whether she said anything) | 15:47 |
| * jlvillal does enjoy the movie The Princess Bride | 15:47 | |
| jlvillal | rloo: I just remember Andre the Giant saying something like, You keep using that word. I don't think it means what you think it means. | 15:47 |
| jroll | rloo: https://s-media-cache-ak0.pinimg.com/originals/97/71/87/977187aed56f1983bfea6611b8ec2029.gif | 15:48 |
| rloo | jlvillal: am trying to rebase one of my patches that touches the .sample. sigh. not going to submit a separate patch to only update the .sample, doesn't seem worth it. | 15:48 |
| rloo | jlvillal: :D | 15:48 |
| *** chaustin_ has quit IRC | 15:49 | |
| jlvillal | Makes sense to me. I don't think a config file update requires a patch of its own. But that's just me. | 15:49 |
| rloo | jlvillal: well, it would be nice if my patch *only* showed the change in the .sample that was due to the patch. | 15:49 |
| jlvillal | rloo: Yeah. I guess you could do a patch for the config file, then your patch. And submit them as a series. If you wanted too. | 15:50 |
| rloo | jlvillal: well, we merged a patch that only updated the config. but then another patch was merged that had an older copy of the config. | 15:51 |
| jlvillal | :( | 15:51 |
| rloo | jlvillal: so yes, i could do a patch for the config, but by the time my actual patch gets merged, who knows what the state of the config might be. | 15:51 |
| jlvillal | rloo: :( Yeah, hard to predict how long a patch will take to get merged. | 15:52 |
| rloo | jlvillal: exactly. | 15:52 |
| deray | jroll, sure.. will try doing a patch chaining exercise sometime. Btw, I was under the impression that ``git review`` only picks up the latest commit from the log | 15:52 |
| * jlvillal wonders if a class on Software Craftsmanship will be any good. But he clicks link and it says page not found. Nice job IT :) | 15:53 | |
| jlvillal | deray: It should pick up all commits since the tip. | 15:53 |
| jroll | deray: it picks up anything not on the branch you're running against (default master) | 15:53 |
| jroll | deray: left a comment on the patch, mostly lgtm | 15:53 |
| *** itamarl has quit IRC | 15:54 | |
| deray | jlvillal, jroll, got to know now. thx :) | 15:54 |
| deray | jroll, thx again :) | 15:55 |
| jroll | np :) | 15:55 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Replace config 'clean_nodes' with 'automated_clean' https://review.openstack.org/247701 | 15:56 |
| deray | jroll, can i go ahead making the code more exhaustive (including the other hashes like: sha224, sha256, sha384 and sha512) ? | 15:58 |
| jroll | deray: I don't think there's any need to do that unless we need them | 15:59 |
| jroll | I guess I don't care, but I don't see the point of adding code that is unused | 15:59 |
| *** moshele has quit IRC | 15:59 | |
| deray | jroll, agreed and point taken.. will raise an exception thne | 16:00 |
| * deray will be leaving IRC now, g'bye all and have a good day ahead :) | 16:01 | |
| *** Nisha has joined #openstack-ironic | 16:02 | |
| *** garthb has joined #openstack-ironic | 16:02 | |
| *** deray has quit IRC | 16:02 | |
| jlvillal | derekh: Ciao | 16:03 |
| jlvillal | sorry derekh wrong <tab> completion | 16:03 |
| derekh | jlvillal: no prob | 16:03 |
| *** stendulker has joined #openstack-ironic | 16:05 | |
| *** ijw has joined #openstack-ironic | 16:05 | |
| *** Nisha_away has joined #openstack-ironic | 16:08 | |
| *** Nisha has quit IRC | 16:09 | |
| *** boris-42 has joined #openstack-ironic | 16:11 | |
| *** daemontool__ has joined #openstack-ironic | 16:12 | |
| *** daemontool__ is now known as daemontool | 16:13 | |
| *** krtaylor has joined #openstack-ironic | 16:14 | |
| *** mbound has quit IRC | 16:14 | |
| *** daemontool_ has quit IRC | 16:15 | |
| openstackgerrit | Merged openstack/ironic: API to manually clean nodes https://review.openstack.org/251995 | 16:18 |
| *** saripurigopi has joined #openstack-ironic | 16:20 | |
| devananda | ^ !! \o/ | 16:21 |
| devananda | also - good morning, all | 16:21 |
| *** klindgren__ is now known as klindgren | 16:21 | |
| *** saripurigopi1 has joined #openstack-ironic | 16:22 | |
| *** saripurigopi has quit IRC | 16:22 | |
| *** dtantsur|brb is now known as dtantsur | 16:24 | |
| sambetts | anyone elses gerrit being super slow all of a sudden? | 16:24 |
| rloo | sambetts: +1 | 16:24 |
| sambetts | oh good not just me then | 16:24 |
| sambetts | :) | 16:25 |
| *** ukalifon has quit IRC | 16:26 | |
| rloo | sambetts: seems ok now | 16:26 |
| sambetts | yeah must of had a lag spike | 16:26 |
| *** ijw has quit IRC | 16:29 | |
| *** chaustin has quit IRC | 16:30 | |
| *** saripurigopi has joined #openstack-ironic | 16:30 | |
| *** chaustin has joined #openstack-ironic | 16:31 | |
| dtantsur | morning devananda, jroll, jlvillal and everyone I missed while was afk :) | 16:31 |
| sambetts | jlvillal: you left almost the exact same comment as I was about to on the inspector rules patch, luckly I refresh the page before I posted :-P | 16:33 |
| jlvillal | sambetts: :D | 16:33 |
| *** saripurigopi1 has quit IRC | 16:34 | |
| *** jaybeale has joined #openstack-ironic | 16:34 | |
| *** aline_ has quit IRC | 16:37 | |
| jlvillal | yolanda: Are you @hp.com or @hpe.com ? Just noticing patch uses @hp.com | 16:39 |
| *** mbound has joined #openstack-ironic | 16:40 | |
| *** piet has quit IRC | 16:40 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-python-agent: Clear GPT and MBR data structures on disk before imaging https://review.openstack.org/277949 | 16:41 |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add head backslash for node path patch https://review.openstack.org/276693 | 16:42 |
| *** krtaylor has quit IRC | 16:44 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Remove "ramdisk" mentioning in func test contrib https://review.openstack.org/277384 | 16:44 |
| *** piet has joined #openstack-ironic | 16:46 | |
| *** mgoddard_ has joined #openstack-ironic | 16:46 | |
| jlvillal | sambetts: I'm also noticing Gerrit slowness now | 16:47 |
| sambetts | jlvillal: I'm getting proxy errors now... I'm not even going through a proxy my end ... | 16:47 |
| jlvillal | sambetts: The proxy is on their side, not yours. | 16:47 |
| jlvillal | I think. | 16:48 |
| sambetts | jlvillal: yeah | 16:48 |
| sambetts | proably a load balancer or something | 16:48 |
| *** mgoddard has quit IRC | 16:49 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-python-agent: Clear GPT and MBR data structures on disk before imaging https://review.openstack.org/277949 | 16:49 |
| * jlvillal goes and gets some not so good coffee. | 16:49 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Add head backslash for node path patch https://review.openstack.org/276693 | 16:50 |
| *** vishwanathj has joined #openstack-ironic | 16:51 | |
| *** stendulker has quit IRC | 16:54 | |
| betherly | hi all! re-installing my ironic setup using | 16:54 |
| betherly | http://docs.openstack.org/developer/ironic/dev/dev-quickstart.html | 16:54 |
| betherly | following ./stack.sh in my devstack environment i am now getting errors | 16:54 |
| betherly | # query the image id of the default cirros image | 16:55 |
| betherly | image=$(nova image-list | egrep "$DEFAULT_IMAGE_NAME"'[^-]' | awk '{ print $2 }') | 16:55 |
| betherly | ERROR (InternalServerError): An unexpected error prevented the server from fulfilling your request. (HTTP 500) (Request-ID: req-741499f6-d7ce-456e-a830-8ef601c94d65) | 16:55 |
| *** krtaylor has joined #openstack-ironic | 16:55 | |
| mgould | betherly, what's in the nova logs? | 16:56 |
| jlvillal | betherly: What mgould said. Nova CPU log and Nova conductor log are good places to look | 16:57 |
| devananda | in this case, also nova-api | 16:57 |
| devananda | and glance, possibly | 16:57 |
| devananda | betherly: do any other queries succed? | 16:57 |
| devananda | eg, can you "nova list" | 16:57 |
| devananda | or "glance image-list" | 16:58 |
| betherly | nope | 16:58 |
| devananda | can you authenticate with keystone and get the service catalog? | 16:58 |
| mgould | IIRC devstack starts all the services under a screen session, so you have to attach to it to see the logs | 16:58 |
| betherly | no neither of those | 16:58 |
| betherly | ok thanks ill look into that | 16:58 |
| devananda | ok ... have you attached tothe devstack screen session and looked at the status of the services? the error is probably apparently if you scroll through those | 16:58 |
| betherly | devananda: no not yet thanks ill take a look through there | 16:59 |
| *** achanda has joined #openstack-ironic | 16:59 | |
| devananda | because it sounds like something just failed to start :) | 16:59 |
| betherly | awesome. ill run through again and let you know how it goes | 16:59 |
| *** achanda has quit IRC | 16:59 | |
| betherly | multitasking with meeting too so excuse the delay :) | 17:00 |
| *** d0ugal has quit IRC | 17:02 | |
| *** raildo-afk is now known as raildo | 17:03 | |
| *** saripurigopi has quit IRC | 17:04 | |
| *** daemontool has quit IRC | 17:06 | |
| *** praneshp has joined #openstack-ironic | 17:06 | |
| sambetts | jlvillal: w00p tinyipa :) | 17:06 |
| *** spandhe has joined #openstack-ironic | 17:07 | |
| jlvillal | sambetts: Yep :) next step will be actually using it :) | 17:07 |
| sambetts | yeah, once the project-config patch merges then we can see if it works in the gate (it should) (i hope) (it worked last time I checked ;)) | 17:08 |
| jlvillal | sambetts: For TinyIPA, how much is a reasonable amount of memory to give a VM that uses it? | 17:08 |
| jlvillal | Currently we allocate 1GB to the VM | 17:09 |
| sambetts | jlvillal: I've sucessfully booted a virtual box VM with tinyipa using 384mb of RAM, and thats the lowest I pushed it | 17:09 |
| jlvillal | sambetts: So 512MB should be a safe amount? | 17:09 |
| *** piet has quit IRC | 17:10 | |
| jlvillal | sambetts: I was hoping we could launch 3 VMs in our test runs. Currently we launch 1. Some tempest tests expect 3 available. Not the ones we currently run. | 17:11 |
| vdrok | devananda, jroll if you know smth about it - I see 3 options in ironic client that are marked as deprecated: https://github.com/openstack/python-ironicclient/blob/master/ironicclient/shell.py#L126-L139 | 17:12 |
| vdrok | but there is no --os-cert key added to parser | 17:12 |
| vdrok | should these deprecation warnings be removed or should we these options to list of allowed keys for client | 17:13 |
| devananda | vdrok: I believe these options are superceded by oslo things | 17:13 |
| sambetts | jlvillal: I'd hope that 512 would be plenty | 17:14 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Replace config 'clean_nodes' with 'automated_clean' https://review.openstack.org/247701 | 17:14 |
| *** chaustin has quit IRC | 17:14 | |
| vdrok | devananda, but then when I try ironic --os-cert 123 node-list it says ironic: error: argument <subcommand>: invalid choice: '123' (choose from 'chassis-create', 'chassis-delete',... | 17:14 |
| jlvillal | sambetts: Great :) Then 3 of them would only take 1.5GB | 17:15 |
| vdrok | so it does not recognize that | 17:15 |
| sambetts | jlvillal: yup :) that would be great for tempest | 17:15 |
| *** ijw has joined #openstack-ironic | 17:15 | |
| jlvillal | +1! | 17:16 |
| *** jcoufal has quit IRC | 17:19 | |
| jlvillal | sambetts: Looks like infra will likely restart Gerrit | 17:20 |
| *** Nisha_brb has joined #openstack-ironic | 17:20 | |
| NobodyCam | :) | 17:23 |
| *** athomas has quit IRC | 17:23 | |
| dtantsur | yep, notice will come in a second | 17:23 |
| -openstackstatus- NOTICE: Gerrit is restarting now, to alleviate current performance impact and WebUI errors. | 17:24 | |
| *** Nisha_away has quit IRC | 17:24 | |
| jlvillal | dtantsur: I was waiting for it :) I saw in infra that it should come... | 17:24 |
| NobodyCam | hehehe | 17:24 |
| NobodyCam | was just look at http://logs.openstack.org/85/276785/1/check/gate-tempest-dsvm-ironic-pxe_ipa/ecfd144/console.html#_2016-02-09_15_34_47_907 | 17:24 |
| jlvillal | NobodyCam: Should be back up they say | 17:24 |
| NobodyCam | ack | 17:26 |
| *** mbound has quit IRC | 17:30 | |
| *** e0ne has quit IRC | 17:32 | |
| lucasagomes | harlowja, yo, around? I was investigating tooz a little to see how we can fit it in Ironic and I think I found a nasty behavior with the zookeeper driver: https://bugs.launchpad.net/python-tooz/+bug/1543688 | 17:32 |
| openstack | Launchpad bug 1543688 in tooz "multiple acquire(blocking=False) can lead to a deadlock" [Undecided,New] | 17:32 |
| dtantsur | lucasagomes, he's usually around much later | 17:34 |
| lucasagomes | dtantsur, thanks. Yeah he might see the ping later on then | 17:35 |
| *** ijw has quit IRC | 17:36 | |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 17:36 |
| *** ijw has joined #openstack-ironic | 17:37 | |
| mkovacik | lucasagomes, nice one! :) | 17:40 |
| lucasagomes | mkovacik, yeah, I'm afraid that needs to be checked, because even tho the lock objects has a "is_acquired" flag, that's racy | 17:40 |
| lucasagomes | needs to be fixed* | 17:41 |
| mkovacik | right | 17:41 |
| *** ijw has quit IRC | 17:41 | |
| *** degorenko is now known as _degorenko|afk | 17:43 | |
| *** mkovacik has quit IRC | 17:44 | |
| *** achanda has joined #openstack-ironic | 17:50 | |
| *** saripurigopi has joined #openstack-ironic | 17:56 | |
| *** derekh has quit IRC | 17:58 | |
| *** penick has joined #openstack-ironic | 18:00 | |
| *** vishwanathj has quit IRC | 18:00 | |
| *** mgoddard_ has quit IRC | 18:02 | |
| *** mgoddard has joined #openstack-ironic | 18:02 | |
| *** bnemec has joined #openstack-ironic | 18:02 | |
| *** penick has quit IRC | 18:05 | |
| sambetts | night all | 18:05 |
| *** sambetts has quit IRC | 18:06 | |
| openstackgerrit | Merged openstack/ironic-inspector: Introduce API for aborting introspection https://review.openstack.org/265275 | 18:06 |
| openstackgerrit | Merged openstack/ironic-inspector: Remove "ramdisk" mentioning in func test contrib https://review.openstack.org/277384 | 18:07 |
| *** electrofelix has quit IRC | 18:07 | |
| saripurigopi | morning Ironic | 18:07 |
| mgould | saripurigopi, morning | 18:07 |
| * mgould -> home; good night! | 18:07 | |
| saripurigopi | mgould: hii | 18:07 |
| *** mgould has quit IRC | 18:08 | |
| lucasagomes | harlowja, this fix the deadlock: https://review.openstack.org/#/c/278001/ | 18:08 |
| lucasagomes | tested it locally, I need to update the tests but it's late here so I will do it tomorrow | 18:08 |
| * lucasagomes marks as WIP | 18:08 | |
| lucasagomes | calling it a day folks, have a great night! | 18:11 |
| *** lucasagomes is now known as lucas-dinner | 18:11 | |
| jlvillal | lucas-dinner: Ciao! | 18:11 |
| *** Nisha_brb has quit IRC | 18:16 | |
| *** e0ne has joined #openstack-ironic | 18:17 | |
| *** Nisha has joined #openstack-ironic | 18:17 | |
| Nisha | Hello | 18:17 |
| *** raddaoui_ has quit IRC | 18:18 | |
| *** raddaoui has quit IRC | 18:18 | |
| *** raddaoui__ has quit IRC | 18:18 | |
| Nisha | we had a requirement to have swift and glance https based for deploy. we are using agent_ilo driver for it. So the swift temp url generated for the instance image is https based. The agent fails to download the image because of lack of certificate validation | 18:20 |
| Nisha | We added a code in IPA to verify the certificate, | 18:20 |
| Nisha | and the copied the "certificate" in the deploy ramdisk at a common path say "/etc/ssl/cert/<cert>" | 18:21 |
| Nisha | The deploy and boot goes through with this | 18:21 |
| Nisha | But now the question is how do we pass certificate to the agent for downloading the instance image from a https url | 18:22 |
| JayF | so you have a self-signed https cert | 18:23 |
| *** piet has joined #openstack-ironic | 18:23 | |
| *** piet has quit IRC | 18:23 | |
| Nisha | copying the certificate in the image is obviously just good in testing env, but not in production env | 18:23 |
| JayF | and you need to ship the cert with the agent | 18:23 |
| Nisha | Jayf | 18:23 |
| Nisha | yes | 18:23 |
| JayF | What agent image are you using? | 18:23 |
| Nisha | the deploy iso is agent based | 18:23 |
| Nisha | we opend up the iso and copied the cert there | 18:23 |
| JayF | that doesn't really answer my question :) | 18:23 |
| Nisha | what do u mean by "agent image" | 18:24 |
| JayF | You can build IPA inside a DIB ramdisk, a CoreOS ramdisk, or now a TinyIPA ramdisk | 18:24 |
| Nisha | oh | 18:24 |
| Nisha | DIB | 18:24 |
| JayF | which one you use is going to change the proper method for getting things inside | 18:24 |
| JayF | I don't know much about the DIB build :( Sorry. If CoreOS I know how to help b/c we embed lots of things in that during the build | 18:24 |
| vdrok | have a good night everyone! | 18:24 |
| Nisha | but the issue is in IPA and not whic method we use to build the iso | 18:24 |
| JayF | Well, it's environmental, not code | 18:25 |
| Nisha | CoreOS also will have similar thing...i am actually using an old iso(bios one) and not sure which method i used to create it | 18:25 |
| JayF | IPA needs that cert, so it has to be embedded into the ramdisk during the build | 18:25 |
| *** daemontool has joined #openstack-ironic | 18:25 | |
| JayF | if this were a CoreOS image, I'd suggest you'd want to inject your ramdisk in the middle of the build process | 18:25 |
| Nisha | or otherway, can we make ironic pass it | 18:25 |
| JayF | Today there's no way whatsoever to pass that in from Ironic | 18:26 |
| Nisha | because the certificate may vary from env to env | 18:26 |
| JayF | and it looks like https verification can't be toggled off for IPA either | 18:26 |
| JayF | I think a patch to disable https verification would be reasonable | 18:26 |
| Nisha | yes, but i am asking for the correct way of handling this | 18:26 |
| dtantsur | JayF, Nisha, DIB build of agent is not substantially different from DIB build of a user image, if that's the question | 18:26 |
| dtantsur | i.e. you embed things there by creating an element that copies them | 18:26 |
| dtantsur | or using something like -p package-name | 18:27 |
| JayF | There are two ways imo that are correct: 1) Embed your certificate at build time, by modifying the build process. or 2) Add a setting to IPA to disable https cert validation, and teach Ironic how to pass that over | 18:27 |
| JayF | dtantsur: I just don't know how DIB builds work generally :) | 18:27 |
| Nisha | dtantsur, correct, but the certificate may vary from env to env | 18:27 |
| dtantsur | ah, so it's not in build time, right? | 18:27 |
| Nisha | so u mean DIB should take the certificate as an input? | 18:27 |
| JayF | Nisha: IMO yep, that would be the best route. | 18:27 |
| Nisha | JayF, i dont think thats a feasible one | 18:28 |
| JayF | Nisha: but if you need to vary cert at deploy time and not build time, that's not something IPA supports at all now | 18:28 |
| Nisha | ppl may change the certificate if required | 18:28 |
| JayF | you'd have to disable https verification in that case, I think, which is unwise, but at least you're still somewhat protected by the hash | 18:28 |
| dtantsur | if build time works for you, I can imagine an element for doing it, though I'm not sure if one exists right now | 18:28 |
| Nisha | it would require them to build the image just because of that | 18:28 |
| Nisha | dtantsur, it doesnt exst | 18:28 |
| Nisha | it can be one option, i dont deny | 18:28 |
| Nisha | i am just trying to figure out if we can make ironic pass that information to agent | 18:29 |
| Nisha | not sure though | 18:29 |
| JayF | I do not think it's reasonable for Ironic to pass it through to the agent | 18:29 |
| Nisha | dont know which one is more secure | 18:29 |
| JayF | it's basically a "turtles all the way down" problem -- unless at some point you have a valid cert, signed by a public CA shipped by the agent | 18:30 |
| JayF | at some point you're going to have to either 1) build in a cert for one of your self-signed things | 18:30 |
| JayF | or 2) use an untrusted connection to bootstrap into certificates (at this point, you're better off just disabling https verification on the download) | 18:30 |
| Nisha | Jayf i didnt get 2nd one | 18:30 |
| dtantsur | an example of how to inject things into IPA via DIB: https://review.openstack.org/#/c/245082/ | 18:31 |
| Nisha | what do u mean by bootstrap | 18:31 |
| JayF | What I'm saying is there's an inherent problem with trying to secure things in IPA: | 18:31 |
| JayF | the only *secure channel* you have into the ramdisk is when you attach it via the ilo (because you're using the virtual media driver) | 18:31 |
| JayF | if you are unwilling to put the cert in the ramdisk that's transmitted securely via the oob | 18:32 |
| *** aginwala has joined #openstack-ironic | 18:32 | |
| JayF | there's no real good way to get secure information back in without using something like a publically signed SSL certificate | 18:32 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Add network provider interface and implementations https://review.openstack.org/139687 | 18:33 |
| Nisha | hmmm Jayf, same i was thinking that we let user provide the certificate in say driver_info, and then the driver passes that info in the floppy image (similar to data like coductor ip address etc ), and then agent uses that | 18:34 |
| Nisha | its irrespective of agent_ilo driver | 18:34 |
| Nisha | the issue is common for any driver as all the drivers use agent based ramdisk | 18:35 |
| Nisha | and you can have swift configured as https, then the swift temp urls will be https based | 18:35 |
| *** ijw has joined #openstack-ironic | 18:35 | |
| Nisha | the issue can be seen even if you provide the instance image input as the "https url" | 18:36 |
| JayF | Nisha: the real problem is using an invalid, not signed by a public CA certificate, and/or not having a reliable internal CA cert you can embed at build time | 18:36 |
| JayF | I think a cert is too big to put in driver_info, and if you just reference it, you have the "secure bootstrap" problem I talked about above | 18:36 |
| Nisha | could you give pointers for this | 18:36 |
| Nisha | not signed by a public CA certificate | 18:36 |
| JayF | so you go to https://google.com | 18:36 |
| JayF | and it just works | 18:37 |
| JayF | because teh cert is signed by a public CA | 18:37 |
| JayF | if you were using a valid cert signed by a public CA, the agent would also accept it and it would just work | 18:37 |
| JayF | the problem is whoever is deploying certs in your environment are not deploying signed SSL certs, and IPA is rightfully rejecting them | 18:37 |
| Nisha | but the agent doesnt has the intelligenceto understand any certificate correct | 18:37 |
| Nisha | ? | 18:37 |
| JayF | Certificates are basically just electronic documents saying "X attests that I am somewebsite.com" | 18:38 |
| Nisha | yes true | 18:38 |
| JayF | IPA is making sure that the "X" is in the trusted list | 18:38 |
| JayF | is it's not, it's going to reject the certificate | 18:38 |
| JayF | like any sane and properly configured https client :) | 18:38 |
| *** chaustin has joined #openstack-ironic | 18:38 | |
| Nisha | so currently what is its "trusted list"? | 18:38 |
| JayF | well that depends somewhat on how your image is built | 18:39 |
| JayF | in the CoreOS agent, since the base OS of the image is a Debian serrver, you basically get Debian's default trusted list | 18:39 |
| JayF | IDK what goes into the DIB image; or if it gets a base set of ca certs installed at all | 18:40 |
| *** baoli_ has quit IRC | 18:40 | |
| Nisha | Jayf, ok. but how do you know if swift will be configured with same default debian certificates | 18:40 |
| JayF | whoever configures your swift had to provide a ca certificate | 18:41 |
| JayF | if it were signed by a major CA, you'd have no trouble because it'd be trusted | 18:41 |
| JayF | this is sorta basics of https/ssl :/ | 18:41 |
| Nisha | agent needs to verify thecertificate with which your swift/https webswerver is configured. | 18:41 |
| Nisha | yes that i understand | 18:41 |
| JayF | your swift provides a cert | 18:41 |
| JayF | that is crypto signed by someone elses' cert | 18:42 |
| Nisha | so u mean the agent verifies the certificates today? | 18:42 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 18:42 |
| JayF | yes | 18:42 |
| JayF | absolutely | 18:42 |
| *** trown is now known as trown|lunch | 18:42 | |
| Nisha | ok, i will try this. so ultimately you mean that the certificate has to be embedded at the time of building | 18:43 |
| *** ijw has quit IRC | 18:43 | |
| Nisha | so say i build an image with certificate embedded, but my verify fails | 18:44 |
| Madasi | if you are using self-signed, or non CA-signed, then you would have to embed the signer into the trusted cert store, which gets annoyingly tricky at times | 18:44 |
| *** jaybeale has quit IRC | 18:44 | |
| Nisha | IOW, how do we prevent customers from not using self signed certificates in their production env | 18:44 |
| *** ChubYann has joined #openstack-ironic | 18:44 | |
| Madasi | or turn off validation entirely, which has huge security implications, i.e. anyone can claim to be the endpoint server | 18:44 |
| *** aginwala has quit IRC | 18:45 | |
| Nisha | a small piece of code change in IPA resolves this issue | 18:45 |
| Nisha | so in my opinion we should add validation in IPA | 18:45 |
| Nisha | JayF, ^^^ | 18:45 |
| JayF | Validation already exists in IPA. The code that would "fix" your problem is an option to /disable/ validation | 18:45 |
| Nisha | could you give me pointers in IPA code where validation already happens for certificates | 18:46 |
| jroll | it's built into requests | 18:46 |
| jroll | Nisha: can you show me the "small piece of code change in IPA resolves this issue" | 18:47 |
| Nisha | yes but without "verify" how does it validate? | 18:47 |
| Nisha | yes just amin | 18:47 |
| openstackgerrit | Sivaramakrishna Garimella proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 18:47 |
| Nisha | jroll, its just adding verify in the requests.get() | 18:47 |
| Nisha | we did this "resp = requests.get(url, stream=True, proxies=proxies, verify='/etc/ssl/certs/httpsserver.pem')" | 18:48 |
| Nisha | just added verify | 18:48 |
| jroll | Nisha: http://docs.python-requests.org/en/latest/user/advanced/#ssl-cert-verification | 18:48 |
| jroll | "By default, verify is set to True" | 18:48 |
| jroll | now, you're passing in a specific cert to use | 18:49 |
| Nisha | yes thast for testing | 18:49 |
| jroll | it doesn't work by just putting it in /etc/ssl/certs/ because requests ships its own cert bundle | 18:49 |
| Nisha | so obviosuly it has to be replaced with common place | 18:49 |
| jroll | see http://docs.python-requests.org/en/latest/user/advanced/#ca-certificates | 18:49 |
| *** aginwala has joined #openstack-ironic | 18:49 | |
| jroll | we have a version of requests > 2.4, https://github.com/openstack/ironic-python-agent/blob/master/requirements.txt#L20 | 18:50 |
| jroll | so you can use certifi, whatever that is, at build time to cause requests to use your custom CA cert | 18:50 |
| Nisha | we are using 2.7 | 18:51 |
| jroll | right, that's fine | 18:51 |
| JayF | jroll: certifi is just a separately-packaged set of certs from Mozilla, functionally equivalent to what ships with reqs alreayd | 18:51 |
| jroll | again, see http://docs.python-requests.org/en/latest/user/advanced/#ca-certificates | 18:51 |
| JayF | jroll: tl;dr probably the first step in unbundling the cert bundle from requests-proper | 18:52 |
| Madasi | certifi is a standalon cert repo, pulled from the mozilla root store | 18:52 |
| jroll | JayF: right, "From Requests version 2.4.0 onwards, Requests will attempt to use certificates from certifi if it is present on the system. This allows for users to update their trusted certificates without having to change the code that runs on their system." | 18:52 |
| Nisha | but on the baremetal mozilla may not be there | 18:52 |
| jroll | though I guess you have to pass in the CA bundle path | 18:52 |
| Madasi | it's the ones mozilla uses, not pulled from mozilla when you run it | 18:52 |
| Nisha | deploy iso is supposed to be mini OS... | 18:52 |
| dtantsur | oh voting is open for talks | 18:53 |
| JayF | They just are basically using Mozilla as the arbiter of who to trust; it doesn't mean mozilla firefox has to be installed | 18:53 |
| jroll | Nisha: so, to answer your question, your CA cert needs to be bundled in with the operating system. with how requests uses certifi, you can just replace /path/to/site-packages/certifi/cacert.pem and it should just work | 18:55 |
| jroll | does that make sense? | 18:56 |
| Madasi | https://github.com/certifi/python-certifi/issues/22 < doesn't look like certifi currently supports adding your own cert | 18:56 |
| Nisha | jroll, ok...let me try this way | 18:56 |
| Nisha | jroll, i will have to try... | 18:56 |
| Nisha | will try and update... | 18:56 |
| *** krtaylor has quit IRC | 18:57 | |
| jroll | Nisha: okay, alternatively I think I would allow an ipa-ca-cert kernel param that sets the path to CA cert bundles passed to request | 18:58 |
| jroll | JayF: ^ wdyt | 18:58 |
| jroll | I don't find that unreasonable | 18:58 |
| harlowja | lucas-dinner interesting | 18:58 |
| JayF | jroll: Nisha: What's the value of that over just adding a cert to the "right place" on the filesystem? | 18:58 |
| JayF | ooooh | 18:59 |
| Madasi | hmm, a closer read of the certifi bug, it sounds like it is only a problem when certifi package is updated, so could be done at image creation time | 18:59 |
| JayF | jroll: you could also set that to "False" to fully disable verification, couldn't you? | 18:59 |
| jroll | JayF: because 'the right place' is unclear to me at this moment :) | 18:59 |
| jroll | JayF: yes :( | 18:59 |
| jroll | well | 18:59 |
| JayF | I think that's a plus to the change, not a minus | 18:59 |
| jroll | not really, because we'd pass the string | 18:59 |
| jroll | not the boolean | 18:59 |
| JayF | IMO we should do it if we can make a disable work too | 18:59 |
| JayF | adding an arg but not having it have a disable option is borderline silly | 18:59 |
| JayF | even though we'd obviously never want anyone to run a production that way | 19:00 |
| Nisha | jroll, where will that param fit? in IPA? | 19:00 |
| Nisha | and how do we pass it? | 19:00 |
| JayF | we'd add the param to IPA, IPA can take params via kernel command line | 19:01 |
| JayF | so you'd add it to the append_kernel_params (may have the name wrong) option in Ironic | 19:01 |
| JayF | to pass it through to IPA | 19:01 |
| jroll | Nisha: see agent.py | 19:01 |
| Nisha | JayF, exactly i was saying the same thing earlier | 19:01 |
| *** suro-patz has joined #openstack-ironic | 19:01 | |
| Nisha | jroll, so but the param value will be certifictae | 19:02 |
| Nisha | will it be a very large value to pass thru ironic? | 19:02 |
| jroll | Nisha: the value will be the path to the certificate that is built in the agent | 19:02 |
| *** raildo is now known as raildo-afk | 19:02 | |
| jroll | rather, that is stored in the agent image at build time | 19:02 |
| Nisha | ohk, u mean the param will just pass the path and IPA can validate that path | 19:03 |
| Nisha | the certificate has to be there in the image | 19:03 |
| jroll | yes | 19:04 |
| Nisha | hmmm | 19:04 |
| dtantsur | jroll, why can't we embed the path as well? Oo | 19:04 |
| dtantsur | I mean, if we already embed files themselves | 19:04 |
| jroll | dtantsur: yeah, that's a good point | 19:04 |
| jroll | I guess I'd rather that :D | 19:04 |
| Madasi | not sure about embedding, but requests will take an ENV variable | 19:05 |
| JayF | dtantsur: jroll: That's why I said the argument should also take a "False" to disable the verification | 19:05 |
| jroll | yeah, but thinking about it, ironic doesn't need to be involved at all | 19:06 |
| dtantsur | what if IPA: 1. if /etc/ironic-python-agent/certificate exists use it, 2. otherwise use whatever the default is | 19:06 |
| dtantsur | or something like that | 19:06 |
| dtantsur | + kernel parameter to disable verification at all | 19:06 |
| dtantsur | if I'm suggesting non-sense just ask me to finally go have dinner :D | 19:07 |
| jroll | dtantsur: yeah, that sounds reasonable to me | 19:08 |
| dtantsur | Nisha, JayF ^^ | 19:08 |
| dtantsur | and I'm calling it a day, see you :) | 19:08 |
| *** krtaylor has joined #openstack-ironic | 19:09 | |
| JayF | g'night | 19:09 |
| *** dtantsur is now known as dtantsur|afk | 19:09 | |
| Nisha | dtantsur, thanks | 19:09 |
| Nisha | jroll, yes..looks reasonable | 19:09 |
| harlowja | lucas-dinner what's zookeeper being used for :-P | 19:09 |
| Nisha | will try it out | 19:10 |
| *** daemontool_ has joined #openstack-ironic | 19:10 | |
| Nisha | jroll, JayF, dtantsur|afk , Madasi thanks for the discussion | 19:10 |
| jroll | Nisha: np | 19:10 |
| JayF | np | 19:11 |
| *** daemontool__ has joined #openstack-ironic | 19:11 | |
| *** daemontool has quit IRC | 19:12 | |
| *** daemontool_ has quit IRC | 19:14 | |
| *** alex_xu has quit IRC | 19:15 | |
| *** chaustin has quit IRC | 19:16 | |
| *** penick has joined #openstack-ironic | 19:17 | |
| *** dims_ has joined #openstack-ironic | 19:17 | |
| *** jcook has quit IRC | 19:17 | |
| *** dims has quit IRC | 19:17 | |
| *** alex_xu has joined #openstack-ironic | 19:18 | |
| *** jcook has joined #openstack-ironic | 19:20 | |
| *** Nisha_away has joined #openstack-ironic | 19:21 | |
| *** chaustin has joined #openstack-ironic | 19:22 | |
| *** Nisha has quit IRC | 19:24 | |
| *** dims_ has quit IRC | 19:27 | |
| *** Sukhdev has joined #openstack-ironic | 19:27 | |
| *** dims has joined #openstack-ironic | 19:30 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add support for providing custom Swift endpoint URLs https://review.openstack.org/251277 | 19:31 |
| *** max_lobur has quit IRC | 19:32 | |
| *** ChubYann has quit IRC | 19:33 | |
| *** mkovacik has joined #openstack-ironic | 19:37 | |
| openstackgerrit | Surojit Pathak proposed openstack/ironic: Fix stacking failure on RHEL7 https://review.openstack.org/278062 | 19:40 |
| *** dims has quit IRC | 19:41 | |
| *** dims has joined #openstack-ironic | 19:42 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic: Add support for dib based agent ramdisk in lib/ironic https://review.openstack.org/264579 | 19:43 |
| *** ChubYann has joined #openstack-ironic | 19:44 | |
| *** trown|lunch is now known as trown | 19:44 | |
| *** dims_ has joined #openstack-ironic | 19:46 | |
| gmmaha | JayF: might i bother you with a follow on question about network interfaces? :) | 19:47 |
| *** vishwanathj has joined #openstack-ironic | 19:47 | |
| JayF | You're always welcome to ask the channel questions :) | 19:47 |
| gmmaha | :) | 19:47 |
| gmmaha | i found the stable-interface-names element and enabled that on my ubuntu image along with dhcp-all-interfaces.. Now almost all the interfaces are coming up with the right name except for 1 :( | 19:48 |
| gmmaha | that alone comes up as eth0.. the rest are p513p1, p787p1, p787p2 | 19:48 |
| JayF | I don't know specifics about that element | 19:48 |
| gmmaha | aah ok | 19:48 |
| JayF | heh, I'm willing to bet | 19:48 |
| JayF | that element is /why/ you have eth0 as your first interface | 19:48 |
| JayF | if you can link the code I'll confirm | 19:48 |
| * gmmaha goes to find the element | 19:49 | |
| *** dims has quit IRC | 19:49 | |
| gmmaha | https://github.com/openstack/diskimage-builder/tree/master/elements/stable-interface-names | 19:49 |
| *** e0ne has quit IRC | 19:49 | |
| *** aginwala has quit IRC | 19:49 | |
| gmmaha | without that element, they were all enumerating as ethX and was random.. never the same names :( | 19:49 |
| JayF | interesting. | 19:50 |
| JayF | grep -r eth0 /etc/udev/ | 19:50 |
| JayF | (making some assumptions about your distro, so if that doens't work lmk) | 19:51 |
| JayF | I bet you'll find a single file in there forcing that one interface to eht0 | 19:51 |
| *** jaybeale has joined #openstack-ironic | 19:52 | |
| gmmaha | you are right | 19:52 |
| gmmaha | etc/udev/rules.d/70-persistent-net.rules:SUBSYSTEM=="net", ACTION=="add", DRIVERS=="?*", ATTR{address}=="00:1e:67:d0:d8:22", ATTR{dev_id}=="0x0", ATTR{type}=="1", KERNEL=="eth*", NAME="eth0" | 19:53 |
| * gmmaha hates the persistent rules file now!! | 19:53 | |
| JayF | so here's a suggestion | 19:53 |
| JayF | mv /etc/udev/rules.d/70-per* /root/backup-70 # move the old file out of the way | 19:53 |
| JayF | ln -s /dev/null /etc/udev/rules.d/70-... | 19:53 |
| JayF | then reboot | 19:53 |
| * gmmaha goes to test it | 19:54 | |
| gmmaha | and need to find a way to dump that file off the system when i make the image using diskimage-builder | 19:54 |
| *** aginwala has joined #openstack-ironic | 19:55 | |
| *** achanda has quit IRC | 19:55 | |
| *** dims_ has quit IRC | 19:57 | |
| *** dprince has quit IRC | 19:59 | |
| JayF | gmmaha: I mean, you can look at the element you linked me, make another one almost identical with another path, and you're done | 19:59 |
| JayF | gmmaha: assuming that works :) | 19:59 |
| gmmaha | JayF: :) | 19:59 |
| gmmaha | what if i make this element delete that file? | 20:00 |
| *** Nisha_away has quit IRC | 20:00 | |
| * gmmaha wonders if he is stretching boundaries he shouldnt | 20:00 | |
| *** dprince has joined #openstack-ironic | 20:00 | |
| JayF | gmmaha: that file likely doesn't exist until first boot | 20:01 |
| JayF | gmmaha: the symlinking dev/null into it is to prevent your startup scripts from creating it | 20:01 |
| gmmaha | JayF: aah ok.. | 20:02 |
| *** ndipanov has quit IRC | 20:02 | |
| *** ifarkas has quit IRC | 20:02 | |
| gmmaha | JayF: thanks.. testing it now.. will probably add that into an additional element to disable persistent rules generation | 20:05 |
| *** rcernin has joined #openstack-ironic | 20:08 | |
| *** piet has joined #openstack-ironic | 20:09 | |
| mrda | Morning Ironic | 20:10 |
| *** piet has quit IRC | 20:13 | |
| *** piet has joined #openstack-ironic | 20:13 | |
| *** krtaylor has quit IRC | 20:19 | |
| jlvillal | Hello mrda aka Mr Tim Tam Slam Man :) | 20:19 |
| NobodyCam | morning mrda | 20:21 |
| *** Marga_ has quit IRC | 20:23 | |
| mrda | jlvillal: so you got them? | 20:25 |
| mrda | NobodyCam: o/ | 20:25 |
| jlvillal | mrda: Yep. And thanks :) | 20:25 |
| JayF | gmmaha: did it work? | 20:25 |
| mrda | \o/ | 20:25 |
| NobodyCam | hummm: https://www.openstack.org/summit/austin-2016/vote-for-speakers/presentation/7808 :) | 20:26 |
| jlvillal | mrda: Though I am trying to lose some weight. So I will save them for when my wife gets back and we can have them with coffee (or tea) | 20:26 |
| mrda | jlvillal: ...and try a tim tam slam :) | 20:27 |
| *** Sukhdev has quit IRC | 20:27 | |
| * jlvillal finds it very annoying that the part to fix his dishwasher has gone in a "Processing Exception" loop with the US Postal Service and now has no idea when it will arrive. | 20:27 | |
| jlvillal | mrda: We will :) | 20:28 |
| mrda | Noting that the one you get in the USA don't taste the same as the ones you buy in Australia | 20:28 |
| jlvillal | They sell them here too? | 20:28 |
| mrda | (I don't think the Australian made ones have corn syrup or depleted uranium - it makes a difference to the taste) | 20:28 |
| mrda | jlvillal: I believe so | 20:29 |
| jlvillal | mrda: Yep :) http://www.amazon.com/s/ref=nb_sb_noss_1?url=search-alias%3Daps&field-keywords=tim+tam | 20:29 |
| *** raildo-afk is now known as raildo | 20:29 | |
| *** Sukhdev has joined #openstack-ironic | 20:30 | |
| *** ndipanov has joined #openstack-ironic | 20:31 | |
| * jlvillal just now reads about depleted uranium :) | 20:31 | |
| *** krtaylor has joined #openstack-ironic | 20:31 | |
| NobodyCam | http://www.digplanet.com/wiki/Frog_cake | 20:33 |
| *** NobodyCa1 has joined #openstack-ironic | 20:34 | |
| mrda | NobodyCam: they are totally a thing, and they are delicious | 20:34 |
| NobodyCam | :) | 20:34 |
| BadCub | "an artificial cream" that statement alone would concern me lol | 20:39 |
| NobodyCam | lol | 20:39 |
| *** aginwala has quit IRC | 20:40 | |
| *** NobodyCa1 has quit IRC | 20:41 | |
| *** aginwala has joined #openstack-ironic | 20:43 | |
| *** Sukhdev has quit IRC | 20:45 | |
| *** piet has quit IRC | 20:46 | |
| *** NobodyCa1 has joined #openstack-ironic | 20:48 | |
| *** deva_ has joined #openstack-ironic | 20:49 | |
| *** krtaylor has quit IRC | 20:53 | |
| *** NobodyCa1 has quit IRC | 20:53 | |
| *** chaustin has quit IRC | 20:54 | |
| *** deva_ has quit IRC | 20:54 | |
| *** aginwala has quit IRC | 20:57 | |
| *** Marga_ has joined #openstack-ironic | 20:59 | |
| *** daemontool has joined #openstack-ironic | 21:02 | |
| *** daemontool__ has quit IRC | 21:02 | |
| *** chaustin has joined #openstack-ironic | 21:03 | |
| *** dims has joined #openstack-ironic | 21:03 | |
| *** krotscheck is now known as krotscheck_dcm | 21:03 | |
| *** e0ne has joined #openstack-ironic | 21:03 | |
| *** krtaylor has joined #openstack-ironic | 21:07 | |
| rcarrillocruz | hi folks | 21:07 |
| rcarrillocruz | anyone around to look at this failure ? | 21:08 |
| rcarrillocruz | http://paste.openstack.org/show/486488/ | 21:08 |
| rcarrillocruz | not sure what that is, quite cryptic | 21:08 |
| rcarrillocruz | i get that on some servers | 21:08 |
| rcarrillocruz | Command execution failed: list index out of range | 21:08 |
| devananda | rcarrillocruz: anything in the ironic-conductor log file? | 21:08 |
| gmmaha | JayF: testing it now.. sorry got pulled into a meeting | 21:09 |
| *** max_lobur has joined #openstack-ironic | 21:10 | |
| rcarrillocruz | devananda: i'm afraid not, no exception that i can see | 21:11 |
| rcarrillocruz | time to increase conductor debugging? | 21:11 |
| rcarrillocruz | log level to debugging or whchever level that is | 21:11 |
| devananda | rcarrillocruz: clearly this is coming from a node deployment ... what about gathering agent logs? | 21:12 |
| rcarrillocruz | how can i get those? the node gets powered off right after the deploy failed | 21:12 |
| devananda | rcarrillocruz: but yea, if INFO level isn't giving you enough, I would try debugging | 21:12 |
| devananda | *DEBUG level | 21:12 |
| rcarrillocruz | can you point me to docs about gathering agent logs? | 21:13 |
| rcarrillocruz | i'll go fish | 21:13 |
| devananda | JayF: ya'll have a patch (or two) up that helps with ^ right? | 21:13 |
| *** daemontool_ has joined #openstack-ironic | 21:24 | |
| jroll | devananda: rcarrillocruz add systemd.journald.forward_to_console=??? can't remember if it's "yes" or e.g. "ttys0" | 21:26 |
| jroll | I think the former | 21:26 |
| jroll | that's a kernel cmd line param btw | 21:26 |
| devananda | aahhh right | 21:26 |
| rcarrillocruz | oh | 21:26 |
| rcarrillocruz | sweet | 21:26 |
| rcarrillocruz | thx folks | 21:26 |
| jroll | np | 21:27 |
| *** daemontool has quit IRC | 21:27 | |
| jroll | devananda: so I couldn't get jack + pulseaudio + all these conferencing systems to play nice :/ | 21:27 |
| *** dprince has quit IRC | 21:31 | |
| devananda | jroll: :( | 21:32 |
| *** achanda has joined #openstack-ironic | 21:33 | |
| gmmaha | JayF: didnt work :( | 21:33 |
| gmmaha | the file is a symlink but still the interface comes up as eth0 | 21:33 |
| gmmaha | there is a suggestion to write '#' to 75-persistent-rules-generator.. let me try that | 21:33 |
| devananda | jroll: other ideas? | 21:34 |
| devananda | IME, external mic will result in terrible quality for folks | 21:34 |
| jroll | devananda: we can either only have VOIP, or we can convince people to telephone in to a VC system | 21:34 |
| jroll | yeah I'm not bridging the two without SW | 21:34 |
| devananda | jroll: I do not believe "telephone internationally into this US-based VC system" is a fair thing to ask for | 21:35 |
| *** baoli has joined #openstack-ironic | 21:38 | |
| jroll | devananda: I agree :) | 21:38 |
| jroll | though do keep in mind there are many international numbers available | 21:38 |
| jroll | but yes I agree | 21:39 |
| openstackgerrit | Merged openstack/ironic-python-agent: Fix vfd mount for capitalized device name https://review.openstack.org/275511 | 21:40 |
| *** davideagnello has quit IRC | 21:42 | |
| *** davideagnello has joined #openstack-ironic | 21:43 | |
| *** achanda has quit IRC | 21:43 | |
| *** achanda has joined #openstack-ironic | 21:48 | |
| *** Marga_ has quit IRC | 21:49 | |
| *** raildo is now known as raildo-afk | 21:49 | |
| *** Marga_ has joined #openstack-ironic | 21:50 | |
| cinerama | one good point is that some audio conferencing systems only have tollfree numbers that work with certain carriers in that country | 21:50 |
| cinerama | er, only work with certain carriers | 21:50 |
| cinerama | which is splitting hairs a bit but people may not be aware of that and it has affected people i've worked with in the past | 21:51 |
| openstackgerrit | Merged openstack/bifrost: Force type casting to boolean in dhcp flag https://review.openstack.org/275785 | 21:55 |
| openstackgerrit | Merged openstack/bifrost: Allow to define the number of test vms to be created https://review.openstack.org/275371 | 21:55 |
| jroll | cinerama: yeah, to be clear I don't think it's a good idea, just an option if needed :) | 21:56 |
| *** Marga_ has quit IRC | 21:57 | |
| *** Marga_ has joined #openstack-ironic | 21:57 | |
| jroll | devananda: NobodyCam: I know y'all have been interested in policy things in the past so: https://review.openstack.org/#/c/245629 | 21:58 |
| openstackgerrit | Baodong (Robert) Li proposed openstack/ironic-python-agent: Build socket list right before select call https://review.openstack.org/267537 | 22:01 |
| *** Sukhdev has joined #openstack-ironic | 22:03 | |
| openstackgerrit | Ruby Loo proposed openstack/python-ironicclient: Add 'node-set-provision-state <node> clean' https://review.openstack.org/258694 | 22:04 |
| *** chaustin has quit IRC | 22:07 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 22:09 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 22:09 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 22:09 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 22:09 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 22:09 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 22:09 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 22:09 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 22:09 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 22:09 |
| * NobodyCam clicks jroll's link | 22:11 | |
| devananda | vsaienko: curious, is there a reason you always push the whole patch chain, not just the one you want to change? | 22:12 |
| devananda | vsaienko: it makes reviewing harder | 22:12 |
| devananda | vsaienko: also, what happened to this patch?? https://review.openstack.org/#/c/139687/ | 22:15 |
| devananda | it looks like it got lost in the middle of the others | 22:15 |
| *** raddaoui_ has joined #openstack-ironic | 22:17 | |
| *** raddaoui has joined #openstack-ironic | 22:17 | |
| *** raddaoui__ has joined #openstack-ironic | 22:17 | |
| gmmaha | JayF: none of those worked.. but good thing is the two interfaces are always eth0 and p**** so thats good enough to start with | 22:18 |
| gmmaha | :) | 22:19 |
| JayF | heh good luck :) | 22:19 |
| JayF | that's all pretty much udev and distro dependent | 22:19 |
| *** achanda has quit IRC | 22:19 | |
| gmmaha | JayF: seems like it.. | 22:19 |
| *** achanda has joined #openstack-ironic | 22:21 | |
| NobodyCam | jroll: TY for the pointer :) | 22:22 |
| jroll | np | 22:24 |
| jlvillal | Am I doing something wrong or does the help kind of suck for python-ironicclient? | 22:29 |
| jlvillal | $ ironic node-set-provision-state --help | 22:29 |
| jlvillal | Does not give useful info :( | 22:29 |
| devananda | jlvillal: ironic help node-set-provision-state | 22:31 |
| jlvillal | devananda: Thanks. I'm used to '--help' :) | 22:31 |
| * jlvillal may not be alone... | 22:31 | |
| devananda | yah. --help for the main command options. "help <thing>" for subcommands | 22:31 |
| jlvillal | devananda: Great! Thanks for that info. | 22:32 |
| devananda | that's fairly standard IME ... but YMMV ... | 22:32 |
| *** trown is now known as trown|outtypewww | 22:32 | |
| *** aginwala has joined #openstack-ironic | 22:33 | |
| *** dims_ has joined #openstack-ironic | 22:39 | |
| *** dims has quit IRC | 22:40 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Remove 'zapping' from code https://review.openstack.org/278135 | 22:42 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Allow to build user image with DIB https://review.openstack.org/256363 | 22:44 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Link-Local-Connection info to ironic port https://review.openstack.org/256365 | 22:44 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Added operator documentation for ironic portgroups https://review.openstack.org/228496 | 22:44 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 22:44 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: refactor ironic enroll-node code https://review.openstack.org/256364 | 22:44 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add configure_provision_network function https://review.openstack.org/256367 | 22:44 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update Ironic VM network connection https://review.openstack.org/256366 | 22:44 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 22:44 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add Ironic/Neutron integration documentation https://review.openstack.org/258596 | 22:44 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic: Add network provider interface and implementations https://review.openstack.org/139687 | 22:44 |
| *** achanda has quit IRC | 22:45 | |
| *** Sukhdev has quit IRC | 22:47 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Remove 'zapping' from code https://review.openstack.org/278135 | 22:47 |
| *** baoli has quit IRC | 22:50 | |
| *** baoli has joined #openstack-ironic | 22:55 | |
| *** trown|outtypewww is now known as trown | 22:55 | |
| *** Sukhdev has joined #openstack-ironic | 22:57 | |
| *** rcernin has quit IRC | 22:58 | |
| *** Sukhdev has quit IRC | 22:58 | |
| *** Sukhdev has joined #openstack-ironic | 22:59 | |
| *** jaypipes has quit IRC | 23:00 | |
| *** Sukhdev has quit IRC | 23:01 | |
| *** daemontool__ has joined #openstack-ironic | 23:03 | |
| *** Sukhdev has joined #openstack-ironic | 23:04 | |
| *** daemontool_ has quit IRC | 23:04 | |
| *** Sukhdev has quit IRC | 23:09 | |
| *** Sukhdev has joined #openstack-ironic | 23:09 | |
| *** Sukhdev has quit IRC | 23:10 | |
| *** Sukhdev has joined #openstack-ironic | 23:10 | |
| *** Sukhdev has quit IRC | 23:12 | |
| *** thrash is now known as thrash|g0ne | 23:12 | |
| *** Sukhdev has joined #openstack-ironic | 23:13 | |
| *** mkovacik has quit IRC | 23:13 | |
| *** aginwala has quit IRC | 23:14 | |
| *** aginwala has joined #openstack-ironic | 23:16 | |
| *** penick has quit IRC | 23:18 | |
| *** mkovacik has joined #openstack-ironic | 23:19 | |
| *** daemontool_ has joined #openstack-ironic | 23:20 | |
| *** baoli has quit IRC | 23:20 | |
| *** daemontool__ has quit IRC | 23:22 | |
| *** praneshp has quit IRC | 23:23 | |
| *** raddaoui__ has quit IRC | 23:25 | |
| *** raddaoui_ has quit IRC | 23:25 | |
| *** raddaoui has quit IRC | 23:25 | |
| *** daemontool_ has quit IRC | 23:28 | |
| *** daemontool has joined #openstack-ironic | 23:29 | |
| *** Sukhdev has quit IRC | 23:30 | |
| *** Sukhdev has joined #openstack-ironic | 23:31 | |
| *** Sukhdev has quit IRC | 23:32 | |
| *** smoriya_ has joined #openstack-ironic | 23:46 | |
| *** Sukhdev has joined #openstack-ironic | 23:47 | |
| *** ndipanov has quit IRC | 23:49 | |
| *** Sukhdev has quit IRC | 23:54 | |
| anteaya | NobodyCam: you still about? | 23:58 |
| NobodyCam | maybe | 23:59 |
| NobodyCam | :p | 23:59 |
| anteaya | yay | 23:59 |
| NobodyCam | hehehe | 23:59 |
| anteaya | someone I would like you to meet | 23:59 |
| anteaya | give me a minute or two | 23:59 |
| anteaya | we | 23:59 |
| NobodyCam | whats up | 23:59 |
| anteaya | 'll be right back | 23:59 |
| NobodyCam | sure | 23:59 |
| anteaya | thanks | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
