You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/06/03 05:05:04 UTC
[GitHub] [pulsar] lhotari opened a new pull request #10793: [Build] Optimize website build by mounting ~/.m2 in the docker container
lhotari opened a new pull request #10793:
URL: https://github.com/apache/pulsar/pull/10793
### Motivation
Pulsar website build fails. It is slow. One reason is that Pulsar is built within the docker container without maven caches.
### Modifications
- prebuild Pulsar before running website build scripts
- mount ~/.m2 -> /root/.m2 inside containe
- use the GitHub Actions cache key with `-m2-dependencies-all` as the primary source since it contains all dependencies
- remove `mvn install` from the script that runs in the container
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] eolivelli commented on pull request #10793: [Build] Optimize website build by mounting ~/.m2 in the docker container
Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #10793:
URL: https://github.com/apache/pulsar/pull/10793#issuecomment-853707332
Sure.
Regarding the milestone.
This change is only about CI and website.
I am not sure it has some value to cite it in the list of changes for a release.
Not a problem from my side.
In this case I just forgot (I really would like to have something like the BK merge script)
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] sijie commented on pull request #10793: [Build] Optimize website build by mounting ~/.m2 in the docker container
Posted by GitBox <gi...@apache.org>.
sijie commented on pull request #10793:
URL: https://github.com/apache/pulsar/pull/10793#issuecomment-853700497
@eolivelli Can you please update the assignee, labels, and milestone?
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] eolivelli merged pull request #10793: [Build] Optimize website build by mounting ~/.m2 in the docker container
Posted by GitBox <gi...@apache.org>.
eolivelli merged pull request #10793:
URL: https://github.com/apache/pulsar/pull/10793
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] sijie commented on pull request #10793: [Build] Optimize website build by mounting ~/.m2 in the docker container
Posted by GitBox <gi...@apache.org>.
sijie commented on pull request #10793:
URL: https://github.com/apache/pulsar/pull/10793#issuecomment-853700497
@eolivelli Can you please update the assignee, labels, and milestone?
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] eolivelli merged pull request #10793: [Build] Optimize website build by mounting ~/.m2 in the docker container
Posted by GitBox <gi...@apache.org>.
eolivelli merged pull request #10793:
URL: https://github.com/apache/pulsar/pull/10793
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] eolivelli commented on pull request #10793: [Build] Optimize website build by mounting ~/.m2 in the docker container
Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #10793:
URL: https://github.com/apache/pulsar/pull/10793#issuecomment-853707332
Sure.
Regarding the milestone.
This change is only about CI and website.
I am not sure it has some value to cite it in the list of changes for a release.
Not a problem from my side.
In this case I just forgot (I really would like to have something like the BK merge script)
--
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org