-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ARROW-13323: [Archery] Validate docker compose configuration #10692
Conversation
Thanks for opening a pull request! If this is not a minor PR. Could you open an issue for this pull request on JIRA? https://issues.apache.org/jira/browse/ARROW Opening JIRAs ahead of time contributes to the Openness of the Apache Arrow project. Then could you also rename pull request title in the following format?
or
See also: |
.github/workflows/archery.yml
Outdated
@@ -62,7 +62,7 @@ jobs: | |||
working-directory: dev/archery | |||
run: pytest -v archery | |||
- name: Archery Docker Validation | |||
run: archery docker | |||
run: archery docker validate-config |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Click hasn't been executing the command initialization with a subcommand, effectively omitting the validation.
Although it has not caused any problems since the docker build and run commands verify the configuration.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should rename to check-config similarly to the crossbow config check command.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, merging on green
Will create a JIRA for it.