Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[CI][Packaging] Enable Azure in more tests and builds #40079

Closed
Tom-Newton opened this issue Feb 14, 2024 · 0 comments · Fixed by #40080
Closed

[CI][Packaging] Enable Azure in more tests and builds #40079

Tom-Newton opened this issue Feb 14, 2024 · 0 comments · Fixed by #40080
Assignees
@Tom-Newton
Labels
Component: Continuous Integration Type: enhancement
Milestone
16.0.0

Comments

@Tom-Newton
Copy link
Contributor

Tom-Newton commented Feb 14, 2024 •
edited
Loading

Describe the enhancement requested

Child issue of: #39317

There are more builds and tests where I should have enabled Azure in #39352. I started tacking stuff onto #40021 but it was turning into a bit of a mess so I decided to split out the CI stuff.

Importantly we need Azure to be enabled in python CI tests. That means enabling Azure in the conda builds.

Component(s)

Continuous Integration
@Tom-Newton Tom-Newton mentioned this issue Feb 14, 2024
Open
@Tom-Newton Tom-Newton changed the title [CI][Packaging] Enable Azure is conda builds [CI][Packaging] Enable Azure in more tests and builds Feb 14, 2024
@github-actions github-actions bot mentioned this issue Feb 14, 2024
Merged
kou pushed a commit that referenced this issue Feb 22, 2024
@Tom-Newton
GH-40079: [CI][Packaging] Enable Azure in more tests and builds (#40080)
b089c6a 
### Rationale for this change
We want python side tests of `AzureFileSystem` to run in CI. 

### What changes are included in this PR?
- Add missing `export` to enable Azure pyarrow tests
- Enable azure in sdist tests.
- Enable Azure on macos python builds
- Enable azure in conda builds and install dependencies (Azure C++ SDK and azurite)
- Enable retries on C++ tests to mitigate #40121

Probably all of this should have been included in #39971

### Are these changes tested?
There is no new functionality to test

### Are there any user-facing changes?
No

* Closes: #40079
* GitHub Issue: #40079

Authored-by: Thomas Newton <thomas.w.newton@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
@kou kou added this to the 16.0.0 milestone Feb 22, 2024
zanmato1984 pushed a commit to zanmato1984/arrow that referenced this issue Feb 28, 2024
@Tom-Newton @zanmato1984
apacheGH-40079: [CI][Packaging] Enable Azure in more tests and builds (…
eeacd01 
…apache#40080)

### Rationale for this change
We want python side tests of `AzureFileSystem` to run in CI. 

### What changes are included in this PR?
- Add missing `export` to enable Azure pyarrow tests
- Enable azure in sdist tests.
- Enable Azure on macos python builds
- Enable azure in conda builds and install dependencies (Azure C++ SDK and azurite)
- Enable retries on C++ tests to mitigate apache#40121

Probably all of this should have been included in apache#39971

### Are these changes tested?
There is no new functionality to test

### Are there any user-facing changes?
No

* Closes: apache#40079
* GitHub Issue: apache#40079

Authored-by: Thomas Newton <thomas.w.newton@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
thisisnic pushed a commit to thisisnic/arrow that referenced this issue Mar 8, 2024
@Tom-Newton @thisisnic
apacheGH-40079: [CI][Packaging] Enable Azure in more tests and builds (…
3fa227b 
…apache#40080)

### Rationale for this change
We want python side tests of `AzureFileSystem` to run in CI. 

### What changes are included in this PR?
- Add missing `export` to enable Azure pyarrow tests
- Enable azure in sdist tests.
- Enable Azure on macos python builds
- Enable azure in conda builds and install dependencies (Azure C++ SDK and azurite)
- Enable retries on C++ tests to mitigate apache#40121

Probably all of this should have been included in apache#39971

### Are these changes tested?
There is no new functionality to test

### Are there any user-facing changes?
No

* Closes: apache#40079
* GitHub Issue: apache#40079

Authored-by: Thomas Newton <thomas.w.newton@gmail.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Tom-Newton Tom-Newton

Labels
Component: Continuous Integration Type: enhancement
Projects
None yet
Milestone
16.0.0
Development

Successfully merging a pull request may close this issue.

GH-40079: [CI][Packaging] Enable Azure in more tests and builds Tom-Newton/arrow
2 participants
@kou @Tom-Newton