[ 
https://issues.apache.org/jira/browse/YETUS-799?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16795326#comment-16795326
 ] 

Yetus QA commented on YETUS-799:
--------------------------------

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
23s{color} | {color:blue} Docker mode activated. {color} |
|| || || || {color:brown} Prechecks {color} ||
| {color:green}+1{color} | {color:green} dupname {color} | {color:green}  0m  
0s{color} | {color:green} No case conflicting files found. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:red}-1{color} | {color:red} test4tests {color} | {color:red}  0m  
0s{color} | {color:red} The patch doesn't appear to include any new or modified 
tests. Please justify why no new tests are needed for this patch. Also please 
list what manual steps were performed to verify this patch. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:red}-1{color} | {color:red} mvninstall {color} | {color:red}  0m 
11s{color} | {color:red} root in master failed. {color} |
| {color:orange}-0{color} | {color:orange} javadoc {color} | {color:orange}  0m 
17s{color} | {color:orange} precommit in master failed. {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  0m 
10s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} hadolint {color} | {color:green}  0m  
1s{color} | {color:green} There were no new hadolint issues. {color} |
| {color:green}+1{color} | {color:green} shellcheck {color} | {color:green}  0m 
 4s{color} | {color:green} There were no new shellcheck issues. {color} |
| {color:green}+1{color} | {color:green} shelldocs {color} | {color:green}  0m  
0s{color} | {color:green} There were no new shelldocs issues. {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  0m  
8s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m  
7s{color} | {color:green} precommit in the patch passed. {color} |
| {color:blue}0{color} | {color:blue} asflicense {color} | {color:blue}  0m  
6s{color} | {color:blue} ASF License check generated no output? {color} |
| {color:black}{color} | {color:black} {color} | {color:black}  1m 36s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=17.05.0-ce Server=17.05.0-ce base: 
https://builds.apache.org/job/PreCommit-YETUS-Build/977/artifact/patchprocess/Dockerfile
 |
| JIRA Issue | YETUS-799 |
| JIRA Patch URL | 
https://issues.apache.org/jira/secure/attachment/12962866/YETUS-799.00.patch |
| Optional Tests | dupname asflicense shellcheck shelldocs javac javadoc unit 
hadolint |
| uname | Linux 19d7b166dec3 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 
17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | precommit/src/main/shell/personality/yetus.sh |
| git revision | master / bdb0f18 |
| maven | version: Apache Maven 3.3.9 |
| Default Java | 1.8.0_191 |
| mvninstall | 
https://builds.apache.org/job/PreCommit-YETUS-Build/977/artifact/patchprocess/branch-mvninstall-root.txt
 |
| shellcheck | v0.6.0 |
| javadoc | 
https://builds.apache.org/job/PreCommit-YETUS-Build/977/artifact/patchprocess/branch-javadoc-precommit.txt
 |
|  Test Results | 
https://builds.apache.org/job/PreCommit-YETUS-Build/977/testReport/ |
| Max. process+thread count | 53 (vs. ulimit of 1000) |
| modules | C: precommit U: precommit |
| Console output | 
https://builds.apache.org/job/PreCommit-YETUS-Build/977/console |
| Powered by | Apache Yetus 0.10.0-SNAPSHOT http://yetus.apache.org |


This message was automatically generated.



> Use real paths in docker container
> ----------------------------------
>
>                 Key: YETUS-799
>                 URL: https://issues.apache.org/jira/browse/YETUS-799
>             Project: Yetus
>          Issue Type: Improvement
>          Components: Test Patch
>            Reporter: Allen Wittenauer
>            Assignee: Allen Wittenauer
>            Priority: Major
>         Attachments: YETUS-799.00.patch
>
>
> The --dockerind support doesn't really work for docker containers that need 
> access to the file system because the paths inside the Yetus container are 
> different than the ones that dockerd expects/the host uses externally.  In 
> order to fix this, we should just change our setup such that (minimally) the 
> $BASEDIR inside the container is the same as the $BASEDIR outside the 
> container and maximally, dump /testptch for almost everything and just use 
> the same paths inside our container.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to