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

Update Python requirements for projects #1030

Merged
merged 2 commits into from
Jul 12, 2021
Merged

Conversation

t0yv0
Copy link
Member

@t0yv0 t0yv0 commented Jun 28, 2021

Would this be appropriate to raise the lower bound of Python examples to the latest released (semi-automatically) or do we prefer to manage it in a different way?

I was curious to try this in response to GHA failure. The context:


   Requirement.parse('pulumi<3.4.0,>=3.0.0'), ***'pulumi-kubernetes'***)
   https://github.com/pulumi/examples/runs/2845226942?check_suite_focus=true

  pkg_resources.ContextualVersionConflict: (pulumi 3.5.1
    (/tmp/p-it-fv-az216-3-simple-87a0419a-304933166/venv/lib/python3.7/site-packages),
    Requirement.parse('pulumi<3.4.0,>=3.0.0'),
    ***'pulumi-kubernetes'***) error: an unhandled error occurred:
    Program exited with non-zero exit code: 1

So there was a requirements conflict in some Kubernetes example. I will try to run GHA on this branch to see if this helps it.

@t0yv0 t0yv0 requested review from pierskarsenbarg and stack72 June 28, 2021 14:50
@t0yv0
Copy link
Member Author

t0yv0 commented Jun 28, 2021

Hrm... fixed some issues with this, put Packet issues in #1031 - but the remainder is still failing

     kubernetes:apps/v1:Deployment (redis-leader): error: configured
    Kubernetes cluster is unreachable: unable to load schema
    information from the API server: Get
    "https://35.225.153.116/openapi/v2?timeout=32s": dial tcp
    35.225.153.116:443: i/o timeout

Let me run it a few more times.

@t0yv0 t0yv0 force-pushed the t0yv0/bump-python-requirements branch from bb199e0 to 65f16d2 Compare June 30, 2021 15:30
@t0yv0
Copy link
Member Author

t0yv0 commented Jun 30, 2021

Trying one more time on top of master.

@t0yv0 t0yv0 merged commit 8fa3392 into master Jul 12, 2021
@t0yv0 t0yv0 deleted the t0yv0/bump-python-requirements branch July 12, 2021 14:00
dixler pushed a commit that referenced this pull request Jan 21, 2022
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.

2 participants