[GitHub] [arrow] kszucs commented on pull request #7021: ARROW-8628: [Dev] Wrap docker-compose commands with archery
kszucs commented on pull request #7021: URL: https://github.com/apache/arrow/pull/7021#issuecomment-624178442 Hopefully we won't have any regressions, if we do I can handle those in follow-up PRs. Thanks for the reviews! Merging. 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: us...@infra.apache.org
[GitHub] [arrow] kszucs commented on pull request #7021: ARROW-8628: [Dev] Wrap docker-compose commands with archery
kszucs commented on pull request #7021: URL: https://github.com/apache/arrow/pull/7021#issuecomment-623695509 @github-actions crossbow submit -g test 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: us...@infra.apache.org
[GitHub] [arrow] kszucs commented on pull request #7021: ARROW-8628: [Dev] Wrap docker-compose commands with archery
kszucs commented on pull request #7021: URL: https://github.com/apache/arrow/pull/7021#issuecomment-623684032 > Some UX improvement I'd like to see: > > * I often fail to run `docker-compose` from the root of the sources leading to cryptic errors because it can't find the `.env`. Could you automatically fix this with a setcwd in archery? This is a nice UX feature for user who don't know this detail. `archery docker run` now works from any subdirectory of arrow, even when it is installed in non-editable mode. The script handles the reading of dotenv defaults for better integration, so we always override those values. > * It would be nice if upon failure, we'd get the list of environment variables changes affecting the docker build, e.g. PANDAS=master... printed somewhere such that we can easily reproduce. Added. > * Add a `archery docker list` command so we don't need to look at docker-compose.yml to get the list of available targets. Bonus point if you add a short description Added and the bonus is deferred to https://issues.apache.org/jira/browse/ARROW-8697 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: us...@infra.apache.org
[GitHub] [arrow] kszucs commented on pull request #7021: ARROW-8628: [Dev] Wrap docker-compose commands with archery
kszucs commented on pull request #7021: URL: https://github.com/apache/arrow/pull/7021#issuecomment-622320430 @github-actions crossbow submit -g test 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: us...@infra.apache.org
[GitHub] [arrow] kszucs commented on pull request #7021: ARROW-8628: [Dev] Wrap docker-compose commands with archery
kszucs commented on pull request #7021: URL: https://github.com/apache/arrow/pull/7021#issuecomment-621957531 @github-actions crossbow submit test-debian-10-cpp 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: us...@infra.apache.org
[GitHub] [arrow] kszucs commented on pull request #7021: ARROW-8628: [Dev] Wrap docker-compose commands with archery
kszucs commented on pull request #7021: URL: https://github.com/apache/arrow/pull/7021#issuecomment-621946996 @github-actions crossbow submit test-debian-10-cpp 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: us...@infra.apache.org
[GitHub] [arrow] kszucs commented on pull request #7021: ARROW-8628: [Dev] Wrap docker-compose commands with archery
kszucs commented on pull request #7021: URL: https://github.com/apache/arrow/pull/7021#issuecomment-621883711 @github-actions crossbow submit test-debian-10-cpp test-debian-10-go-1.12 test-conda-python-3.7 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: us...@infra.apache.org
[GitHub] [arrow] kszucs commented on pull request #7021: ARROW-8628: [Dev] Wrap docker-compose commands with archery
kszucs commented on pull request #7021: URL: https://github.com/apache/arrow/pull/7021#issuecomment-621759490 @github-actions crossbow submit test-debian-10-cpp test-debian-10-go-1.12 test-conda-python-3.7 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: us...@infra.apache.org
[GitHub] [arrow] kszucs commented on pull request #7021: ARROW-8628: [Dev] Wrap docker-compose commands with archery
kszucs commented on pull request #7021: URL: https://github.com/apache/arrow/pull/7021#issuecomment-621696833 @github-actions crossbow submit test-conda-python-3.7-pandas-latest test-debian-10-cpp test-conda-python-3.7 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: us...@infra.apache.org
[GitHub] [arrow] kszucs commented on pull request #7021: ARROW-8628: [Dev] Wrap docker-compose commands with archery
kszucs commented on pull request #7021: URL: https://github.com/apache/arrow/pull/7021#issuecomment-621349417 @github-actions crossbow submit test-conda-python-3.7-pandas-latest test-debian-10-cpp test-conda-python-3.7 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: us...@infra.apache.org