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

FYI: installing on MacOS Ventura (13.0.1) fails on grpcio #11424

Closed
georgettica opened this issue Nov 19, 2022 · 21 comments
Closed

FYI: installing on MacOS Ventura (13.0.1) fails on grpcio #11424

georgettica opened this issue Nov 19, 2022 · 21 comments
Assignees
Labels
kind/bug Some behavior is incorrect or out of spec needs-repro Needs repro steps before it can be triaged or fixed p1 Bugs severe enough to be the next item assigned to an engineer resolution/fixed This issue was fixed
Milestone

Comments

@georgettica
Copy link

What happened?

followed the instructions and failed on grpcio

Steps to reproduce

pulumi new aws-python

Expected Behavior

that is will create the stack

Actual Behavior

tried to pull the full log but it seems I cannot sanitize it correctly, so I will work on that tomorrow, sry about that

Output of pulumi about

$ pulumi about
CLI          
Version      3.47.1
Go Version   go1.19.2
Go Compiler  gc

Plugins
NAME    VERSION
python  unknown

Host     
OS       darwin
Version  13.0.1
Arch     arm64

This project is written in python: executable='/opt/homebrew/bin/python3' version='3.10.8
'

Backend        
Name           YYY
URL            file://
User           XXX
Organizations  

Dependencies:
NAME        VERSION
dill        0.3.6
parver      0.4.0
pip         22.3.1
protobuf    4.21.9
PyYAML      6.0.0
semver      2.13.0
setuptools  65.6.0
six         1.16.0
wheel       0.38.4

Pulumi locates its logs in XXYYZZ by default

Additional context

No response

Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@georgettica georgettica added kind/bug Some behavior is incorrect or out of spec needs-triage Needs attention from the triage team labels Nov 19, 2022
@georgettica
Copy link
Author

basicallly an issue with grpcio, and it's solved on the prerelease, but not pushed to brew and pypi. once it's out I will check if this reproduces

@t0yv0
Copy link
Member

t0yv0 commented Nov 21, 2022

Could you provide a little bit more context? I've tried to reproduce on Ventura 13.0.1, Pulumi 3.47.1 albeit my architecture is not arm64. Pulumi selects grpcio==1.47. Should Pulumi be upgraded to a more recent version of grpcio?

@guineveresaenger guineveresaenger added needs-repro Needs repro steps before it can be triaged or fixed and removed needs-triage Needs attention from the triage team labels Nov 21, 2022
@guineveresaenger guineveresaenger transferred this issue from pulumi/pulumi-aws Nov 21, 2022
@t0yv0
Copy link
Member

t0yv0 commented Nov 21, 2022

Guessing the repro here might be just "pulumi new aws-python && pulumi up" on an M1-powered (arm) Mac machine.

@justinvp
Copy link
Member

Just want to confirm that pulumi new aws-python && pulumi up works OK on an M1 mac on Monterey. Checking to see if anyone on the team has already upgraded to Ventura, otherwise, we can look into running in a VM (or possibly CI) to repro.

@RobbieMcKinstry
Copy link
Contributor

RobbieMcKinstry commented Nov 21, 2022

Also confirmed that pulumi new aws-python && pulumi up works on my M1 mac on Ventura 13.0.1.
Was not able to reproduce.
https://app.warp.dev/block/NXK69cORCxjbQBVp9IYcrZ

@georgettica
Copy link
Author

@RobbieMcKinstry how is your pulumi installed?

@georgettica
Copy link
Author

I think I solved the issue by going through Homebrew/discussions#3920

but I am verifying now

@RobbieMcKinstry
Copy link
Contributor

I compiled Pulumi from source.

@georgettica
Copy link
Author

seems the fix I thought I had didn't work out correctly :/
thanks @RobbieMcKinstry for the response though!

@georgettica
Copy link
Author

creating the grpcio issue and added a log to it
grpc/grpc#31761

@georgettica
Copy link
Author

the logs for your convinience https://gist.github.com/georgettica/80cc8d8197d02bcbbefa4bd084d12cfe

@georgettica
Copy link
Author

seems the issue with grpcio has been solved. now I am retrying :)

@georgettica
Copy link
Author

@t0yv0 so it seems upgrading to the latest version solves this issue :/

@t0yv0 t0yv0 added the resolution/fixed This issue was fixed label Dec 1, 2022
@t0yv0 t0yv0 closed this as completed Dec 1, 2022
@georgettica
Copy link
Author

georgettica commented Dec 1, 2022

(latest version of grpcio)
is that in the pipeline?
@t0yv0

@t0yv0
Copy link
Member

t0yv0 commented Dec 2, 2022

Apologies, reopening. I misunderstood this as latest version of Pulumi.

@t0yv0 t0yv0 reopened this Dec 2, 2022
@t0yv0
Copy link
Member

t0yv0 commented Dec 2, 2022

@georgettica confirming It's actively being worked on - #11431 - there's several tickets that are related here including #11276 - apologies for the delay in getting this fixed!

@georgettica
Copy link
Author

georgettica commented Dec 2, 2022

No worries, as long as I know where to look I can close this as duplicate

@joeduffy joeduffy added the p1 Bugs severe enough to be the next item assigned to an engineer label Dec 21, 2022
@joeduffy
Copy link
Member

marking p1 per discussion with @lukehoban as this is blocking usage on a common configuration.

@mikhailshilkov mikhailshilkov removed the resolution/fixed This issue was fixed label Dec 21, 2022
@kpitzen
Copy link
Contributor

kpitzen commented Dec 21, 2022

@joeduffy @georgettica @t0yv0 @RobbieMcKinstry @justinvp I believe this should be fixed in our next release. #11431 was merged, and I was able to both repro the original issue pre-fix, and verify its resolution post-fix in Python 3.11 on an M1 machine.

@RobbieMcKinstry RobbieMcKinstry added this to the 0.82 milestone Dec 22, 2022
@RobbieMcKinstry RobbieMcKinstry added the resolution/fixed This issue was fixed label Dec 22, 2022
@RobbieMcKinstry
Copy link
Contributor

Since we believe this fixed, I'll close the issue again and apply the label. Feel free to reopen if Pulumi v3.50.2 does not solve the problem.

@georgettica
Copy link
Author

georgettica commented Dec 26, 2022

sadly I got another failure while trying to re-use it with the latest pulumi.

I will update once I gathered all of the facts again to see where the build fails

and open a different issue if the problem is not the same

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Some behavior is incorrect or out of spec needs-repro Needs repro steps before it can be triaged or fixed p1 Bugs severe enough to be the next item assigned to an engineer resolution/fixed This issue was fixed
Projects
None yet
Development

No branches or pull requests

8 participants