[GitHub] chrismattmann closed pull request #133: Streamline usage of junit among the project and add Travis CI build support

chrismattmann closed pull request #133: Streamline usage of junit among the project and add Travis CI build support URL: https://github.com/apache/drat/pull/133 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[GitHub] chrismattmann commented on issue #133: Streamline usage of junit among the project and add Travis CI build support

chrismattmann commented on issue #133: Streamline usage of junit among the project and add Travis CI build support URL: https://github.com/apache/drat/pull/133#issuecomment-398454278 thanks @ottlinger please file an INFRA ticket and report back the ISSUE id to the list. Thanks. I just merg

[GitHub] chrismattmann commented on issue #134: Upgrade to JDK8

chrismattmann commented on issue #134: Upgrade to JDK8 URL: https://github.com/apache/drat/issues/134#issuecomment-399504524 @ottlinger I fixed this. It builds fine now so please finish upgrading to JDK8. This is an automated

[GitHub] chrismattmann commented on issue #130: Updates for OODT 1.9 and fix for #112 (Do not merge)

chrismattmann commented on issue #130: Updates for OODT 1.9 and fix for #112 (Do not merge) URL: https://github.com/apache/drat/pull/130#issuecomment-399505091 @IMS94 I think we need to handle the issue you are seeing in OODT - with InstanceId being flowed down - did you fix that? ---

[GitHub] IMS94 commented on issue #130: Updates for OODT 1.9 and fix for #112 (Do not merge)

IMS94 commented on issue #130: Updates for OODT 1.9 and fix for #112 (Do not merge) URL: https://github.com/apache/drat/pull/130#issuecomment-399723389 @chrismattmann I'm in the process of doing that. Will need few more days.

[GitHub] chrismattmann commented on issue #130: Updates for OODT 1.9 and fix for #112 (Do not merge)

chrismattmann commented on issue #130: Updates for OODT 1.9 and fix for #112 (Do not merge) URL: https://github.com/apache/drat/pull/130#issuecomment-399727625 💯 This is an automated message from the Apache Git Service. To re

[GitHub] buggtb commented on issue #116: Set up DRAT to weekly scan the ASF full Github

buggtb commented on issue #116: Set up DRAT to weekly scan the ASF full Github URL: https://github.com/apache/drat/issues/116#issuecomment-400663234 Do we have, or can we get an ASF VM for this? This is an automated message fr

[GitHub] chrismattmann commented on issue #116: Set up DRAT to weekly scan the ASF full Github

chrismattmann commented on issue #116: Set up DRAT to weekly scan the ASF full Github URL: https://github.com/apache/drat/issues/116#issuecomment-400663558 yep we have one, drat.apache.org. Current projects running there: http://drat.apache.org/demo/ - Proteus live scan, pre-relea

[GitHub] buggtb commented on issue #134: Upgrade to JDK8

buggtb commented on issue #134: Upgrade to JDK8 URL: https://github.com/apache/drat/issues/134#issuecomment-400663725 Be worth testing JDK 10 also, because in some other projects JDK 10 broke Tomcat for a start. I'll give it a whirl this week at some point.

[GitHub] chrismattmann commented on issue #134: Upgrade to JDK8

chrismattmann commented on issue #134: Upgrade to JDK8 URL: https://github.com/apache/drat/issues/134#issuecomment-401645832 +1 thanks @buggtb This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] ahmedifhaam opened a new pull request #135: Initial Commit of adding new UI designed with vue

ahmedifhaam opened a new pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135 -Only works for local paths -Only intergrated proteus UI Viz features have to intergrated --

[GitHub] chrismattmann commented on issue #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-401850573 this includes generated JS and CSS minified. We need the original source, not the minified stuff. I can't review this. -

[GitHub] ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue

ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-402022677 @chrismattmann i have added the source This is an automated message f

[GitHub] chrismattmann commented on issue #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-402559104 whoah - i think you checked in like a fully built version of the project. Can you just check in the source code @ahmedifhaam ? It sho

[GitHub] ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue

ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-403012711 @chrismattmann Updated Please check again Sorry a bunch This is a

[GitHub] buggtb opened a new issue #136: Doesn't run under JDK10

buggtb opened a new issue #136: Doesn't run under JDK10 URL: https://github.com/apache/drat/issues/136 Tried to run DRAT on JDK 10 on Ubuntu and get: ` Using CATALINA_BASE: /home/bugg/drat/deploy/tomcat Using CATALINA_HOME: /home/bugg/drat/deploy/tomcat Using CATALINA_TMP

[GitHub] buggtb opened a new issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8

buggtb opened a new issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8 URL: https://github.com/apache/drat/issues/137 @chrismattmann has reported issues with DRAT on OODT 1.2.2 on the DRAT VM. Seg faults in the Lucene subsystem. -

[GitHub] chrismattmann commented on issue #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-404639338 will check today @ahmedifhaam This is an automated message from the

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202167804 ## File path: webapps/proteus-new/src/main/webapp/META-INF/MANIFEST.MF ## @@ -0,0 +1,6 @@ +Manifes

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202206545 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/HelloWorld.vue ## @@ -

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202207012 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/filelistcomp.vue ## @@

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202207293 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/projectstable.vue ## @

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202206616 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/barchartcomp.vue ## @@

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202206633 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/barchartcomp.vue ## @@

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202206945 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/filelistcomp.vue ## @@

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202167734 ## File path: webapps/proteus-new/pom.xml ## @@ -0,0 +1,18 @@ + Review comment: add ALv2 in

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202206580 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/barchartcomp.vue ## @@

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202207106 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/piechart.vue ## @@ -0,

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202206597 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/barchartcomp.vue ## @@

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202206979 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/filelistcomp.vue ## @@

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202207147 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/piechart.vue ## @@ -0,

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r202206852 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/barchartcomp.vue ## @@

[GitHub] buggtb commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8

buggtb commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8 URL: https://github.com/apache/drat/issues/137#issuecomment-404883956 Tested on the DRAT VM without error. Trying to figure out how to replicate. @chrismattmann got any tips? ---

[GitHub] chrismattmann commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8

chrismattmann commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8 URL: https://github.com/apache/drat/issues/137#issuecomment-404897871 it doesn't die from the command line. Try from Proteus at http://drat.apache.org/demo/ and then watch the logs. -

[GitHub] buggtb commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8

buggtb commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8 URL: https://github.com/apache/drat/issues/137#issuecomment-404937652 Found one @ryanmcdonagh SEVERE: Unable to obtain product by id: [19580d69-8695-11e8-b501-47ca7be11d25]: Message: Product: [19580d

[GitHub] buggtb edited a comment on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8

buggtb edited a comment on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8 URL: https://github.com/apache/drat/issues/137#issuecomment-404937652 Found one @ryanmcdonagh ``` SEVERE: Unable to obtain product by id: [19580d69-8695-11e8-b501-47ca7be11d25]: Message: Product

[GitHub] buggtb commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8

buggtb commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8 URL: https://github.com/apache/drat/issues/137#issuecomment-404941114 Okay @chrismattmann @ryanmcdonagh According to https://issues.apache.org/jira/browse/LUCENE-6943?focusedCommentId=16259353&page=com.atlas

[GitHub] buggtb commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8

buggtb commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8 URL: https://github.com/apache/drat/issues/137#issuecomment-404963227 Got bored and tried a build with lucene 6.5 in, got further bit still segfaults. Looked up the latest bomb and found this: https://issue

[GitHub] buggtb opened a new issue #138: Mapper doesn't run on Python 3

buggtb opened a new issue #138: Mapper doesn't run on Python 3 URL: https://github.com/apache/drat/issues/138 The mapper is python 2 only, so doesn't run on more modern systems currently. This is an automated message from the

[GitHub] buggtb opened a new issue #139: PGE failure

buggtb opened a new issue #139: PGE failure URL: https://github.com/apache/drat/issues/139 So i'm running it now and its working in terms of the Crawl via proteus. What I do see in the logs though is ``` SEVERE: PGETask FAILED!!! : Failed to build PgeConfig : Failed to par

[GitHub] buggtb commented on issue #139: PGE failure

buggtb commented on issue #139: PGE failure URL: https://github.com/apache/drat/issues/139#issuecomment-405053163 Okay tested against 1.1 and don't see it, so I guess its an OODT regression... debugging. This is an automated

[GitHub] buggtb removed a comment on issue #139: PGE failure

buggtb removed a comment on issue #139: PGE failure URL: https://github.com/apache/drat/issues/139#issuecomment-405053163 Okay tested against 1.1 and don't see it, so I guess its an OODT regression... debugging. This is an au

[GitHub] chrismattmann commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8

chrismattmann commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8 URL: https://github.com/apache/drat/issues/137#issuecomment-405577129 @buggtb any progress here? Very curious! This is an automated mes

[GitHub] buggtb commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8

buggtb commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8 URL: https://github.com/apache/drat/issues/137#issuecomment-405577558 Yeah I think this bit is done, but can you look at #139 and let me know your thoughts there. On 17 July 2018 at 15:14:07, Ch

[GitHub] chrismattmann commented on issue #139: PGE failure

chrismattmann commented on issue #139: PGE failure URL: https://github.com/apache/drat/issues/139#issuecomment-405597598 This looks like the RatAggregator Task is getting fired off before there are any RatLogs. You can check this in your Opsui FM browser. What do you see when you get this

[GitHub] buggtb commented on issue #139: PGE failure

buggtb commented on issue #139: PGE failure URL: https://github.com/apache/drat/issues/139#issuecomment-405696989 checking, I'll let you know shortly. This is an automated message from the Apache Git Service. To respond to the

[GitHub] ahmedifhaam commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

ahmedifhaam commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r203704148 ## File path: webapps/proteus-new/pom.xml ## @@ -0,0 +1,18 @@ + Review comment: initial one ?

[GitHub] buggtb commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

buggtb commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r203711098 ## File path: webapps/proteus-new/pom.xml ## @@ -0,0 +1,18 @@ + Review comment: https://github.com/

[GitHub] buggtb commented on issue #135: Initial Commit of adding new UI designed with vue

buggtb commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-406264661 I'm trying to process the PR but its a bit hairy. I would say, you still seem to have a minified index.html and some javascript stuff in the

[GitHub] buggtb opened a new issue #140: Can't select directory using the popup on mac

buggtb opened a new issue #140: Can't select directory using the popup on mac URL: https://github.com/apache/drat/issues/140 In proteus, if I try and select a local directory, it wont let me pick a directory to use. This is a

[GitHub] buggtb commented on issue #139: PGE failure

buggtb commented on issue #139: PGE failure URL: https://github.com/apache/drat/issues/139#issuecomment-406276471 Trying to figure out some of the wiring. The Lucene bug was from OODT using the lucene catalog, but there's a solr server and the mimepartitioner seems to query it? But its emp

[GitHub] chrismattmann commented on issue #139: PGE failure

chrismattmann commented on issue #139: PGE failure URL: https://github.com/apache/drat/issues/139#issuecomment-406353832 hmmm that suggests that the indexing process either didn't run or failed. After crawling, it has to run the Solr indexer, to ingest the CAS FM metadata into Solr. It run

[GitHub] chrismattmann commented on issue #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-406354905 @ahmedifhaam really need you to address these issues and put out a new PR in the next few days for review. Thanks. -

[GitHub] ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue

ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-406358547 minified files in the sense ; the files inside webapp/js ? This is

[GitHub] buggtb commented on issue #135: Initial Commit of adding new UI designed with vue

buggtb commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-406359338 @ahmedifhaam stuff like webapp/js/app.68e075fb.js Isn't that a file thats been compiled during the npm build phase?

[GitHub] ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue

ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-406365493 yes exactly otherwise anybody wish to run have to compile it again thats why i have added that source of those files are in the resourc

[GitHub] buggtb commented on issue #139: PGE failure

buggtb commented on issue #139: PGE failure URL: https://github.com/apache/drat/issues/139#issuecomment-406365893 Alright, as i mentioned in the slack channel I think this is a non issue and is infact cause by the docs being a bit stale and missing the alias stuff as I've stuck it in and i

[GitHub] buggtb commented on issue #135: Initial Commit of adding new UI designed with vue

buggtb commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-406366202 Well I tend to believe that these repos should only contain sourcecode and if people want to build it they can do. Be it with Maven or whate

[GitHub] ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue

ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-406367549 let me see the possible alternatives This is an automated message from

[GitHub] chrismattmann commented on issue #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-406374268 please just make it a Maven build step to call npm. Someone shouldn't have to do anything else other than calling mvn install on the

[GitHub] ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue

ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-406384931 will update soon Thank you @chrismattmann @buggtb This is an auto

[GitHub] buggtb commented on issue #109: Fresh install doesn't create the tomcat/temp directory

buggtb commented on issue #109: Fresh install doesn't create the tomcat/temp directory URL: https://github.com/apache/drat/issues/109#issuecomment-406566215 I've been deploying on ubuntu and mac osx for a week and haven't seen this so I think we can close this one down. --

[GitHub] buggtb closed issue #109: Fresh install doesn't create the tomcat/temp directory

buggtb closed issue #109: Fresh install doesn't create the tomcat/temp directory URL: https://github.com/apache/drat/issues/109 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] buggtb commented on issue #139: PGE failure

buggtb commented on issue #139: PGE failure URL: https://github.com/apache/drat/issues/139#issuecomment-406566343 Okay after a myriad of env vars getting set, I think this is a non bug. This is an automated message from the Ap

[GitHub] buggtb closed issue #139: PGE failure

buggtb closed issue #139: PGE failure URL: https://github.com/apache/drat/issues/139 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the

[GitHub] buggtb closed issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8

buggtb closed issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8 URL: https://github.com/apache/drat/issues/137 This is an automated message from the Apache Git Service. To respond to the message, please log on Gi

[GitHub] buggtb commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8

buggtb commented on issue #137: Stability issues with DRAT and OODT 1.2.2 on JDK 8 URL: https://github.com/apache/drat/issues/137#issuecomment-406567217 Closing this one out as we're tracking the issue on the OODT side and need to release OODT 1.2.3 to fix it.

[GitHub] buggtb opened a new issue #141: Proteus doesn't index

buggtb opened a new issue #141: Proteus doesn't index URL: https://github.com/apache/drat/issues/141 Not yet sure whats going on. Proteus crawls fine, then gets to teh index phase and blocks up. If you index manually then run the map and reduce phase in Proteus its fine. So somethi

[GitHub] buggtb commented on issue #141: Proteus doesn't index

buggtb commented on issue #141: Proteus doesn't index URL: https://github.com/apache/drat/issues/141#issuecomment-406569916 Ah found this in the proteus logs: SEVERE: Could not connect to Solr server null java.lang.InstantiationException: no protocol: null at org.apache

[GitHub] buggtb closed issue #141: Proteus doesn't index

buggtb closed issue #141: Proteus doesn't index URL: https://github.com/apache/drat/issues/141 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to

[GitHub] buggtb commented on issue #141: Proteus doesn't index

buggtb commented on issue #141: Proteus doesn't index URL: https://github.com/apache/drat/issues/141#issuecomment-406573061 Boom I was missing yet another env var. export SOLR_DRAT_URL=http://localhost:8080/solr/drat This

[GitHub] buggtb opened a new issue #142: Finish hooking up DRAT to Travis

buggtb opened a new issue #142: Finish hooking up DRAT to Travis URL: https://github.com/apache/drat/issues/142 Stick in some CI for DRAT and use some Selenium/Cypress stuff to run tests in Proteus to catch issues. This is an

[GitHub] ahmedifhaam commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

ahmedifhaam commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r204248783 ## File path: webapps/proteus-new/pom.xml ## @@ -0,0 +1,18 @@ + Review comment: will add the A

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r204282987 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/test.vue ## @@ -0,0 +1

[GitHub] chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r204282972 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/testii.vue ## @@ -0,0

[GitHub] chrismattmann commented on issue #140: Can't select directory using the popup on mac

chrismattmann commented on issue #140: Can't select directory using the popup on mac URL: https://github.com/apache/drat/issues/140#issuecomment-406940735 yeah never got this fully working, had to specify a path. Would be nice if you could just do a dir. --

[GitHub] chrismattmann commented on issue #136: Doesn't run under JDK10

chrismattmann commented on issue #136: Doesn't run under JDK10 URL: https://github.com/apache/drat/issues/136#issuecomment-406940813 yep this is an OODT issue. This is an automated message from the Apache Git Service. To resp

[GitHub] ahmedifhaam commented on a change in pull request #135: Initial Commit of adding new UI designed with vue

ahmedifhaam commented on a change in pull request #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#discussion_r204354015 ## File path: webapps/proteus-new/src/main/webapp/resources/src/components/testii.vue ## @@ -0,0 +1

[GitHub] ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue

ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-407578205 I have added the features promised but those need some more modifications with the next set of feedbacks i'll update.

[GitHub] RyanMcDonagh opened a new pull request #143: Set up Travis CI

RyanMcDonagh opened a new pull request #143: Set up Travis CI URL: https://github.com/apache/drat/pull/143 I've got Travis CI working for DRAT. It will currently fail to build because it's failing the Proteus test, but that can be resolved in a separate PR for now. ---

[GitHub] buggtb commented on issue #143: Set up Travis CI

buggtb commented on issue #143: Set up Travis CI URL: https://github.com/apache/drat/pull/143#issuecomment-408407909 LGTM 👍 This is an automated message from the Apache Git Service. To respond to the message, please log on Gi

[GitHub] RyanMcDonagh commented on issue #142: Finish hooking up DRAT to Travis

RyanMcDonagh commented on issue #142: Finish hooking up DRAT to Travis URL: https://github.com/apache/drat/issues/142#issuecomment-408411141 Addressed in PR 143: https://github.com/apache/drat/pull/143 This is an automated mes

[GitHub] ottlinger commented on issue #143: Set up Travis CI

ottlinger commented on issue #143: Set up Travis CI URL: https://github.com/apache/drat/pull/143#issuecomment-408414045 You do not need the script part in .travis-file (last lines) as we have a defaultGoal in our pom, which we want Travis CI to use.

[GitHub] RyanMcDonagh commented on issue #143: Set up Travis CI

RyanMcDonagh commented on issue #143: Set up Travis CI URL: https://github.com/apache/drat/pull/143#issuecomment-408414548 We tested it without it, but because of the way it bootstraps the tests, the second phase fails in the distribution. So we do need the script part there, or we need to

[GitHub] buggtb commented on issue #143: Set up Travis CI

buggtb commented on issue #143: Set up Travis CI URL: https://github.com/apache/drat/pull/143#issuecomment-408415529 @ottlinger you can replicate that actually in your own build by running mvn test -B which is what Travis runs. I don't think it hurts having the script in f

[GitHub] chrismattmann closed pull request #143: Set up Travis CI

chrismattmann closed pull request #143: Set up Travis CI URL: https://github.com/apache/drat/pull/143 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request (from a fork)

[GitHub] chrismattmann commented on issue #143: Set up Travis CI

chrismattmann commented on issue #143: Set up Travis CI URL: https://github.com/apache/drat/pull/143#issuecomment-408418182 awesome looks great! once we commit this I think we need to contact Apache infra to turn this on. @buggtb or someone else can you take care of that? -

[GitHub] buggtb commented on issue #143: Set up Travis CI

buggtb commented on issue #143: Set up Travis CI URL: https://github.com/apache/drat/pull/143#issuecomment-408418435 I shall enquire…. On 27 July 2018 at 14:27:56, Chris Mattmann (notificati...@github.com) wrote: Merged #143 .

[GitHub] buggtb closed issue #142: Finish hooking up DRAT to Travis

buggtb closed issue #142: Finish hooking up DRAT to Travis URL: https://github.com/apache/drat/issues/142 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the U

[GitHub] buggtb opened a new issue #144: Cypress Tests For Proteus

buggtb opened a new issue #144: Cypress Tests For Proteus URL: https://github.com/apache/drat/issues/144 Proteus should get some functional tests that allow us to test both Proteus and OODT under stress. This is an automated

[GitHub] buggtb commented on issue #143: Set up Travis CI

buggtb commented on issue #143: Set up Travis CI URL: https://github.com/apache/drat/pull/143#issuecomment-408420656 Filed INFRA-16831 to get it switched on This is an automated message from the Apache Git Service. To respond

[GitHub] ottlinger commented on issue #143: Set up Travis CI

ottlinger commented on issue #143: Set up Travis CI URL: https://github.com/apache/drat/pull/143#issuecomment-408524363 :+1: sorry - I though I already filed a ticket but seem to have forgotten about it. This is an automated

[GitHub] buggtb commented on issue #143: Set up Travis CI

buggtb commented on issue #143: Set up Travis CI URL: https://github.com/apache/drat/pull/143#issuecomment-408525518 No probs. It’s been done, builds looking good. On 27 July 2018 at 21:00:56, P. Ottlinger (notificati...@github.com) wrote: 👍 sorry - I though I already filed

[GitHub] ottlinger commented on issue #143: Set up Travis CI

ottlinger commented on issue #143: Set up Travis CI URL: https://github.com/apache/drat/pull/143#issuecomment-408527120 Builds look green - perfect ;-) This is an automated message from the Apache Git Service. To respond to th

[GitHub] ottlinger edited a comment on issue #143: Set up Travis CI

ottlinger edited a comment on issue #143: Set up Travis CI URL: https://github.com/apache/drat/pull/143#issuecomment-408527120 Builds look green - perfect ;-) https://travis-ci.org/apache/drat This is an automated message f

[GitHub] ottlinger opened a new pull request #145: Fix security vulnerabilities in proteus and webapps

ottlinger opened a new pull request #145: Fix security vulnerabilities in proteus and webapps URL: https://github.com/apache/drat/pull/145 Independent of the updates I have local test failures: ` Tests in error: TestProcessDratWrapper.testParseWorkflows:29 ExceptionInInitializer

[GitHub] ottlinger commented on issue #145: Fix security vulnerabilities in proteus and webapps

ottlinger commented on issue #145: Fix security vulnerabilities in proteus and webapps URL: https://github.com/apache/drat/pull/145#issuecomment-408669195 Travis does not show the test failures I've locally, thus ready 2B merged.

[GitHub] ahmedifhaam opened a new pull request #146: Completed Integration of Dratstat

ahmedifhaam opened a new pull request #146: Completed Integration of Dratstat URL: https://github.com/apache/drat/pull/146 This is an automated message from the Apache Git Service. To respond to the message, please log on Git

[GitHub] chrismattmann commented on issue #135: Initial Commit of adding new UI designed with vue

chrismattmann commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-410468602 ## Instruction for building DRAT proteus UI (new) ## Building vue front end 1. move to `$DRAT_ROOT/webapps/proteus-new/src/

[GitHub] ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue

ahmedifhaam commented on issue #135: Initial Commit of adding new UI designed with vue URL: https://github.com/apache/drat/pull/135#issuecomment-410479526 @chrismattmann we updated the build process now on _mvn clean build_ would do the build --

<    1   2   3   4   5   >