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.

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

[main] Rebuild for aws_crt_cpp0201, google_cloud_cpp2101, re220230302 #1053

Conversation

regro-cf-autotick-bot
Copy link
Contributor

@regro-cf-autotick-bot regro-cf-autotick-bot commented May 14, 2023

This PR has been triggered in an effort to update aws_crt_cpp0201.

Notes and instructions for merging this PR:

  1. Please merge the PR only after the tests have passed.
  2. Feel free to push to the bot's branch to update this PR if needed.

Please note that if you close this PR we presume that the feedstock has been rebuilt, so if you are going to perform the rebuild yourself don't close this PR until the your rebuild has been merged.

Closes #1059

If this PR was opened in error or needs to be updated please add the bot-rerun label to this PR. The bot will close this PR and schedule another one. If you do not have permissions to add this label, you can use the phrase @conda-forge-admin, please rerun bot in a PR comment to have the conda-forge-admin add it for you.

This PR was created by the regro-cf-autotick-bot. The regro-cf-autotick-bot is a service to automatically track the dependency graph, migrate packages, and propose package version updates for conda-forge. Feel free to drop us a line if there are any issues! This PR was generated by https://github.com/regro/cf-scripts/actions/runs/4970484114, please use this URL for debugging.

@conda-forge-webservices
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@h-vetinari h-vetinari changed the title Rebuild for aws_crt_cpp0201 [main] Rebuild for aws_crt_cpp0201 May 14, 2023
@h-vetinari h-vetinari marked this pull request as draft May 14, 2023 07:30
@h-vetinari
Copy link
Member

h-vetinari commented May 14, 2023

Let's wait with this round of PRs this until we figured out the abseil situation a bit better all sorted now

@h-vetinari h-vetinari force-pushed the rebuild-aws_crt_cpp0201-0-1_hf82ccc branch 3 times, most recently from aaee1d3 to 86a1a87 Compare May 17, 2023 08:17
@h-vetinari h-vetinari changed the title [main] Rebuild for aws_crt_cpp0201 [main] Rebuild for aws_crt_cpp0201, google_cloud_cpp2101, re220230302 May 17, 2023
@h-vetinari h-vetinari marked this pull request as ready for review May 17, 2023 08:42
@h-vetinari h-vetinari added automerge Merge the PR when CI passes and removed automerge Merge the PR when CI passes labels May 17, 2023
@h-vetinari
Copy link
Member

Single test failure here, and only on linux. Normally I'd go with a skip, but this looks like it might be leaking memory? Any ideas? The test in question has no comments or documentation AFAICS.
@kou @pitrou @jorisvandenbossche @xhochy

=================================== FAILURES ===================================
__________________________ test_total_bytes_allocated __________________________

    def test_total_bytes_allocated():
>       assert pa.total_allocated_bytes() == 0
E       assert 1216 == 0
E        +  where 1216 = <built-in function total_allocated_bytes>()
E        +    where <built-in function total_allocated_bytes> = pa.total_allocated_bytes

It's also interesting that this doesn't appear in any of the equivalent PRs to the maintenance branches, e.g. #1057.

@xhochy
Copy link
Member

xhochy commented May 17, 2023

This is a test that shouldn't be flaky. It would be good if someone from upstream can look into it or we can hope that with updated dependencies, it also shows up upstream.

@h-vetinari
Copy link
Member

we can hope that with updated dependencies, it also shows up upstream.

Checking this in apache/arrow#35637

@h-vetinari
Copy link
Member

we can hope that with updated dependencies, it also shows up upstream.

Checking this in apache/arrow#35637

Does not reproduce with current upstream main branch...

@h-vetinari h-vetinari force-pushed the rebuild-aws_crt_cpp0201-0-1_hf82ccc branch from 624118a to 373246f Compare May 24, 2023 01:36
@h-vetinari h-vetinari added the automerge Merge the PR when CI passes label May 24, 2023
@h-vetinari h-vetinari removed the automerge Merge the PR when CI passes label May 24, 2023
@h-vetinari h-vetinari merged commit 9a85508 into conda-forge:main May 24, 2023
@regro-cf-autotick-bot regro-cf-autotick-bot deleted the rebuild-aws_crt_cpp0201-0-1_hf82ccc branch May 24, 2023 05:54
kou pushed a commit to apache/arrow that referenced this pull request Jul 26, 2023
Corresponds to conda-forge/arrow-cpp-feedstock#1053 for pyarrow (one failing test on our infra that needs debugging), as well as the state of the feedstock for r-arrow after conda-forge/r-arrow-feedstock#65
* Closes: #35658

Authored-by: H. Vetinari <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
R-JunmingChen pushed a commit to R-JunmingChen/arrow that referenced this pull request Aug 20, 2023
…e#35637)

Corresponds to conda-forge/arrow-cpp-feedstock#1053 for pyarrow (one failing test on our infra that needs debugging), as well as the state of the feedstock for r-arrow after conda-forge/r-arrow-feedstock#65
* Closes: apache#35658

Authored-by: H. Vetinari <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
loicalleyne pushed a commit to loicalleyne/arrow that referenced this pull request Nov 13, 2023
…e#35637)

Corresponds to conda-forge/arrow-cpp-feedstock#1053 for pyarrow (one failing test on our infra that needs debugging), as well as the state of the feedstock for r-arrow after conda-forge/r-arrow-feedstock#65
* Closes: apache#35658

Authored-by: H. Vetinari <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants