[GitHub] incubator-metron pull request #384: METRON-603 Update website to use Apache ...

[mattf-horton]

Github user mattf-horton commented on a diff in the pull request:

https://github.com/apache/incubator-metron/pull/384#discussion_r90739762
  
--- Diff: site/_includes/primary-nav-items.html ---
@@ -10,6 +10,6 @@
 News
 
 
- https://github.com/apache/incubator-metron/releases; 
target="new">Download 
+ https://dist.apache.org/repos/dist/release/incubator/metron/; 
target="new">Download 
--- End diff --

@kylerichardson , I agree, a downloads page is more or less the only way to 
satisfy the simultaneous requirements of directing the code download through a 
mirror and the signature download through the dist site.

I would suggest keeping jira METRON-603 open (adding "via Mirror script" to 
the title) and using it for the correct enhancement later on.  Meanwhile, I 
have no objection to committing this PR as an improvement to the current 
situation of having the actual site out of sync with the source code.  We are 
still sufficiently low volume that we aren't likely to cause problems with 
Infra.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-metron pull request #384: METRON-603 Update website to use Apache ...

[ptgoetz]

Github user ptgoetz commented on a diff in the pull request:

https://github.com/apache/incubator-metron/pull/384#discussion_r90726311
  
--- Diff: site/_includes/primary-nav-items.html ---
@@ -10,6 +10,6 @@
 News
 
 
- https://github.com/apache/incubator-metron/releases; 
target="new">Download 
+ https://dist.apache.org/repos/dist/release/incubator/metron/; 
target="new">Download 
--- End diff --

@kylerichardson A separate downloads page a good approach that a lot of 
projects take.

Basically you have links to the source/binary artifacts that go through 
closer.cgi, and the links to the checksums/signatures go to dist.apache.org. 
The reason behind that is that checksums/signatures are not synced with the 
mirrors for security reasons.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

HDP 2.5.3

[Otto Fowler]

How do we feel about moving to HDP 2.5.3?

http://docs.hortonworks.com/HDPDocuments/HDP2/HDP-2.5.3/bk_release-notes/bk_release-notes-20161130.pdf

and Ambari 2.4.2.0

http://docs.hortonworks.com/HDPDocuments/Ambari-2.4.2.0/bk_ambari-release-notes/bk_ambari-release-notes-20161128.pdf

[GitHub] incubator-metron issue #379: METRON-595: ES Writer uses more than one IP

[dlyle65535]

Github user dlyle65535 commented on the issue:

https://github.com/apache/incubator-metron/pull/379
  
Oh, thanks for that. Let me run it up really quickly then. Totally mis-read 
you, @ottobackwards. 


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-metron pull request #386: METRON-607: Enrichment doc improvement a...

[mmiklavc]

GitHub user mmiklavc opened a pull request:

https://github.com/apache/incubator-metron/pull/386

METRON-607: Enrichment doc improvement and test cleanup

This PR addresses https://issues.apache.org/jira/browse/METRON-607


You can merge this pull request into a Git repository by running:

$ git pull https://github.com/mmiklavc/incubator-metron METRON-607

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-metron/pull/386.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 #386


commit 45cc113f3bd123e0b667f2b7d5fd58bbeb59adbf
Author: Michael Miklavcic 
Date:   2016-12-02T19:03:49Z

METRON-607: Enrichment doc improvement and test cleanup




---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-metron issue #379: METRON-595: ES Writer uses more than one IP

[ottobackwards]

Github user ottobackwards commented on the issue:

https://github.com/apache/incubator-metron/pull/379
  
@dlyle65535 I did not test, I was answering the question of if quick-dev 
works at all


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-metron issue #379: METRON-595: ES Writer uses more than one IP

[dlyle65535]

Github user dlyle65535 commented on the issue:

https://github.com/apache/incubator-metron/pull/379
  
Nope, looks like you did a test run with changes and @ottobackwards did the 
clean one I was going to do, so I'm +1.
Thanks!



---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-metron issue #385: METRON-604 Mpack installs do not work on clean ...

[dlyle65535]

Github user dlyle65535 commented on the issue:

https://github.com/apache/incubator-metron/pull/385
  
+1 Travis pending. Like importing Ambari's format as ambari_format, btw. 
Thanks!


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-metron issue #383: METRON-597 Sporadic Failures of Profiler Integr...

[ottobackwards]

Github user ottobackwards commented on the issue:

https://github.com/apache/incubator-metron/pull/383
  
+1
Ran pr on 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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-metron issue #379: METRON-595: ES Writer uses more than one IP

[JonathanRider]

Github user JonathanRider commented on the issue:

https://github.com/apache/incubator-metron/pull/379
  
@dlyle65535 is there anything specific that you wanted me to test on 
quickdev that I haven't already done?


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-metron issue #376: METRON-593: Enable an automated static analysis...

[dlyle65535]

Github user dlyle65535 commented on the issue:

https://github.com/apache/incubator-metron/pull/376
  
+1, thanks!


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-metron pull request #375: METRON-592: Change popup to Warning mess...

[asfgit]

Github user asfgit closed the pull request at:

https://github.com/apache/incubator-metron/pull/375


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] incubator-metron pull request #384: METRON-603 Update website to use Apache ...

[mattf-horton]

Github user mattf-horton commented on a diff in the pull request:

https://github.com/apache/incubator-metron/pull/384#discussion_r90615484
  
--- Diff: site/_includes/primary-nav-items.html ---
@@ -10,6 +10,6 @@
 News
 
 
- https://github.com/apache/incubator-metron/releases; 
target="new">Download 
+ https://dist.apache.org/repos/dist/release/incubator/metron/; 
target="new">Download 
--- End diff --

Hi Kyle, this change is on the right path, since we definitely should be 
publishing downloads on "dist".  However, 
http://www.apache.org/dev/release-download-pages.html says:
- All links to the distribution artifacts **must not** reference the main 
Apache web site ["dist"]. They should use the standard mechanisms to distribute 
the load between the mirror sites [see http://www.apache.org/dyn/closer.cgi/ ].
- All links to checksums, detached signatures and public keys **must** 
reference the main Apache web site and should use https (SSL).

The web page then goes on in some detail about how to set up a script to 
automatically redirect the user to an appropriate mirror.  

If the above isn't clear, there is a better written explanation at 
http://incubator.apache.org/guides/releasemanagement.html#understanding-mirroring
But it points back to http://www.apache.org/dev/release-download-pages.html 
to talk about the script.

An example of a good download page that uses an automatic mirror-direction 
script is
https://maven.apache.org/download.cgi
An example of a download page that is "dumber" but therefore easier to set 
up is
https://hadoop.apache.org/releases.html

The quick-and-dirty fix is to simply point at 
http://www.apache.org/dyn/closer.cgi/incubator/metron/ instead of 
https://dist.apache.org/repos/dist/release/incubator/metron/ , BUT that 
doesn't give the user access to the secure signatures, and that's not 
acceptable.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---