You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@atlas.apache.org by "erdenebayrd (via GitHub)" <gi...@apache.org> on 2025/02/22 03:23:04 UTC

[PR] Fix permission error on ${HOME}/.m2 directory before building Atlas [atlas]

erdenebayrd opened a new pull request, #297:
URL: https://github.com/apache/atlas/pull/297

   Fix permission error on ${HOME}/.m2 directory before building Atlas
   
   ## What changes were proposed in this pull request?
   
   Added a step in the README to ensure the ${HOME}/.m2 directory exists and has 
   the correct ownership before running `docker-compose -f docker-compose.atlas-base.yml -f docker-compose.atlas-build.yml up`. 
   
   This prevents permission errors during the build process.
   
   - Created the ${HOME}/.m2 directory if it does not exist.
   - Updated ownership to prevent permission errors.
   
   ## How was this patch tested?
   
   Manually tested by verifying:
   - The directory ${HOME}/.m2 is correctly created if missing.
   - Ownership is properly set to prevent permission errors.
   - The build process runs without permission issues.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@atlas.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix permission error on ${HOME}/.m2 directory before building Atlas [atlas]

Posted by "erdenebayrd (via GitHub)" <gi...@apache.org>.
erdenebayrd closed pull request #297: Fix permission error on ${HOME}/.m2 directory before building Atlas
URL: https://github.com/apache/atlas/pull/297


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@atlas.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix permission error on ${HOME}/.m2 directory before building Atlas [atlas]

Posted by "mneethiraj (via GitHub)" <gi...@apache.org>.
mneethiraj commented on code in PR #297:
URL: https://github.com/apache/atlas/pull/297#discussion_r1966437608


##########
dev-support/atlas-docker/README.md:
##########
@@ -36,24 +36,31 @@ Docker files in this folder create docker images and run them to build Apache At
    ./download-archives.sh
    ~~~
 
-5.  Execute following commands to set environment variables to build Apache Atlas docker containers:
+5. Ensure that the `${HOME}/.m2` directory exists and has the correct ownership:
+   ~~~
+   mkdir -p ${HOME}/.m2
+   sudo chown -R $(whoami):$(whoami) ${HOME}/.m2

Review Comment:
   `.m2/repository` directory can have large number of files and directories (> 50k files is not uncommon). Line # 42 can take a long time to complete.
   
   Instead of inserting step #5 (and updating numbers of subsequent steps), I suggest to update 6.1 with inclusion of following line before starting build:
   
   ```
   mkdir -p ${HOME}/.m2
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@atlas.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org