You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@metron.apache.org by nickwallen <gi...@git.apache.org> on 2017/04/05 21:02:26 UTC
[GitHub] incubator-metron pull request #514: METRON-829 Use Fastcapa with Kerberos
GitHub user nickwallen opened a pull request:
https://github.com/apache/incubator-metron/pull/514
METRON-829 Use Fastcapa with Kerberos
Yes, you can use Fastcapa with Kerberos with no code changes. I describe the entire process in the README.
This PR is dependent on #509 which explains the extra commits that are currently visible.
## Pull Request Checklist
- [ ] Is there a JIRA ticket associated with this PR? If not one needs to be created at [Metron Jira](https://issues.apache.org/jira/browse/METRON/?selectedTab=com.atlassian.jira.jira-projects-plugin:summary-panel).
- [ ] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
- [ ] Has your PR been rebased against the latest commit within the target branch (typically master)?
- [ ] Have you included steps to reproduce the behavior or problem that is being changed or addressed?
- [ ] Have you included steps or a guide to how the change may be verified and tested manually?
- [ ] Have you ensured that the full suite of tests and checks have been executed in the root incubating-metron folder via:
- [ ] Have you written or updated unit tests and or integration tests to verify your changes?
- [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)?
- [ ] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?
### For documentation related changes:
- [ ] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via `site-book/target/site/index.html`:
```
cd site-book
bin/generate-md.sh
mvn site:site
```
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/nickwallen/incubator-metron METRON-829
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-metron/pull/514.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #514
----
commit 6410229e6a6a031f470d01ced2eb316d43bae29a
Author: Nick Allen <ni...@nickallen.org>
Date: 2017-03-21T17:51:55Z
METRON-822 Improve Fastcapa Performance
commit 2492bc70833f40512b7bfbac34d03ef3f862242c
Author: Nick Allen <ni...@nickallen.org>
Date: 2017-04-05T01:00:42Z
Updated README
commit 324b487a98f1d4efa99b737a29c3150d2f3b1d53
Author: Nick Allen <ni...@nickallen.org>
Date: 2017-04-05T14:34:02Z
Added multi-burst for receive workers
commit e9a952fe54132b5e28dbdd302a4b3d46ab64dee0
Author: Nick Allen <ni...@nickallen.org>
Date: 2017-04-05T16:16:13Z
Error messages to stderr
commit ee84379ee28f71b79cc7fecbc0c08ad72765ea79
Author: Nick Allen <ni...@nickallen.org>
Date: 2017-04-05T16:19:33Z
Created a JIRA to replace the TODO comment in the code
commit 28c439e598dde9b8c042cbeb9a9495d1a9e6db78
Author: Nick Allen <ni...@nickallen.org>
Date: 2017-04-05T20:55:42Z
METRON-829 Use Fastcapa with Kerberos
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #514: METRON-829 Use Fastcapa with Kerberos
Posted by JonZeolla <gi...@git.apache.org>.
Github user JonZeolla commented on the issue:
https://github.com/apache/incubator-metron/pull/514
Ah, I missed that, I was only reviewing the Kerberos section from the two commits.
It's because there are indented blank lines in the code block. That's a known issue, if you remove the indentation or the empty lines it should fix it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #514: METRON-829 Use Fastcapa with Kerberos
Posted by nickwallen <gi...@git.apache.org>.
Github user nickwallen commented on the issue:
https://github.com/apache/incubator-metron/pull/514
@JonZeolla I notice that the rendering of the code under the following text looks wonky. Although, I have no idea why. Any thoughts?
> 6. Ensure that the device was bound. It should be shown as a \u201cnetwork device using DPDK-compatible driver.\u201d
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron pull request #514: METRON-829 Use Fastcapa with Kerberos
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-metron/pull/514
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #514: METRON-829 Use Fastcapa with Kerberos
Posted by nickwallen <gi...@git.apache.org>.
Github user nickwallen commented on the issue:
https://github.com/apache/incubator-metron/pull/514
That was it. Thanks @JonZeolla !
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #514: METRON-829 Use Fastcapa with Kerberos
Posted by nickwallen <gi...@git.apache.org>.
Github user nickwallen commented on the issue:
https://github.com/apache/incubator-metron/pull/514
Ok. The instructions should be ready-to-go now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #514: METRON-829 Use Fastcapa with Kerberos
Posted by JonZeolla <gi...@git.apache.org>.
Github user JonZeolla commented on the issue:
https://github.com/apache/incubator-metron/pull/514
#511 should be in master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #514: METRON-829 Use Fastcapa with Kerberos
Posted by nickwallen <gi...@git.apache.org>.
Github user nickwallen commented on the issue:
https://github.com/apache/incubator-metron/pull/514
@cestella I had thought that the setup for the Bro Plugin (the work you did in #501) would be almost the same as the setup for Fastcapa. My assumption was wrong though. I had to create a JAAS configuration file which I don't see in the instructions that you created for the Bro Plugin. Any idea why you didn't need to do that? Are my instructions incorrect?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #514: METRON-829 Use Fastcapa with Kerberos
Posted by cestella <gi...@git.apache.org>.
Github user cestella commented on the issue:
https://github.com/apache/incubator-metron/pull/514
That shouldn't be required as librdkafka doesn't know anything about JAAS since JAAS is a Java thing and librdkafka is decidedly not. It does ticket renewal literally by having you specify the keytab and the kinit command (the default one works). I validated that by botching the kinit command in the librdkafka config and having it scream bloody murder in the logs periodically and stop working.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron pull request #514: METRON-829 Use Fastcapa with Kerberos
Posted by nickwallen <gi...@git.apache.org>.
GitHub user nickwallen reopened a pull request:
https://github.com/apache/incubator-metron/pull/514
METRON-829 Use Fastcapa with Kerberos
Yes, you can use Fastcapa with Kerberos with no code change. I describe the entire process in the README.
This PR is dependent on #509 which explains the extra commits that are currently visible.
## Pull Request Checklist
- [x] Is there a JIRA ticket associated with this PR? If not one needs to be created at [Metron Jira](https://issues.apache.org/jira/browse/METRON/?selectedTab=com.atlassian.jira.jira-projects-plugin:summary-panel).
- [x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
- [x] Has your PR been rebased against the latest commit within the target branch (typically master)?
- [x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?
- [x] Have you included steps or a guide to how the change may be verified and tested manually?
- [x] Have you ensured that the full suite of tests and checks have been executed in the root incubating-metron folder via:
- [x] Have you written or updated unit tests and or integration tests to verify your changes?
- [x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)?
- [x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?
- [x] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via `site-book/target/site/index.html`:
```
cd site-book
bin/generate-md.sh
mvn site:site
```
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/nickwallen/incubator-metron METRON-829
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-metron/pull/514.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #514
----
commit a062c440751088bb5b3ea9df1cfa90136e820fce
Author: Nick Allen <ni...@nickallen.org>
Date: 2017-04-05T20:55:42Z
METRON-829 Use Fastcapa with Kerberos
commit 3165e5f4c5aad51560be966a26b97aae65ef1c7d
Author: Nick Allen <ni...@nickallen.org>
Date: 2017-04-05T21:08:53Z
Grammar is king
commit d9575a3481cb5cda4217a8b8d68590109e9c6c6b
Author: Nick Allen <ni...@nickallen.org>
Date: 2017-04-10T14:37:01Z
Removed JAAS configuration steps which are not needed with Fastcapa.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron pull request #514: METRON-829 Use Fastcapa with Kerberos
Posted by nickwallen <gi...@git.apache.org>.
Github user nickwallen closed the pull request at:
https://github.com/apache/incubator-metron/pull/514
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #514: METRON-829 Use Fastcapa with Kerberos
Posted by nickwallen <gi...@git.apache.org>.
Github user nickwallen commented on the issue:
https://github.com/apache/incubator-metron/pull/514
@cestella Thanks. Duh. That makes sense.
Thinking back, I probably just needed the JAAS stuff since I was trying to test with the Kafka Console Producer before landing data with Fastcapa. I'll work through the steps again and remove the JAAS stuff so as not confuse the instructions.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #514: METRON-829 Use Fastcapa with Kerberos
Posted by nickwallen <gi...@git.apache.org>.
Github user nickwallen commented on the issue:
https://github.com/apache/incubator-metron/pull/514
Kicking Travis
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #514: METRON-829 Use Fastcapa with Kerberos
Posted by cestella <gi...@git.apache.org>.
Github user cestella commented on the issue:
https://github.com/apache/incubator-metron/pull/514
+1 by inspection
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-metron issue #514: METRON-829 Use Fastcapa with Kerberos
Posted by cestella <gi...@git.apache.org>.
Github user cestella commented on the issue:
https://github.com/apache/incubator-metron/pull/514
@nickwallen kick travis; that's an intermittent error.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---