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

Merge Python 3.10 upgrade up to production #125

Merged
merged 46 commits into from
Sep 26, 2023
Merged

Merge Python 3.10 upgrade up to production #125

merged 46 commits into from
Sep 26, 2023

Conversation

dmannarino
Copy link
Member

Pull request checklist

Please check if your PR fulfills the following requirements:

  • Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your master!
  • Make sure you are making a pull request against the develop branch (left side). Also you should start your branch off our develop.
  • Check the commit's or even all commits' message styles matches our requested structure.
  • Check your code additions will fail neither code linting checks nor unit test.

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this introduce a breaking change?

  • Yes
  • No

Other information

@dmannarino
Update packages, lambda layers used
983495d
@dmannarino
Roll back starlette, fastapi versions
4bbda9d
@dmannarino
Pin jinja2 dep for old starlette
df4f9b4
@dmannarino
Bump rasterio, pillow versions
3c78121
@dmannarino
Bump rasterio to 1.3.4
a24f088
@dmannarino
Attempt to upgrade to Python 3.10
8aa3f59
@dmannarino
Merge branch 'dev' into rasterio_1_3_2
928c5e1
@dmannarino
Upgrade GFW TF image used
34bf9de
@dmannarino
Upgrade to rasterio 1.3.7
1117416
@dmannarino
Lambda Edge only supports up to Python 3.9
b0b7aa2
@dmannarino
Upgrade to rasterio 1.3.8
bf810c2
@dmannarino
Upgrade to rasterio 1.3.8
e8a8f6e
@dmannarino
Cosmetic changes (mostly from linters)
639d43f
@dmannarino
Missed a reference to rasterio 1.3.7
4ba56ba
@dmannarino
Cap TF version
ecd15f1
@dmannarino
Downgrade from problematic TF image
69d1640
@dmannarino
Downgrade from problematic TF image
10b6ef6
@dmannarino
Fix references to old rasterio upgrade (to 1.3.4)
9de5000
@dmannarino
Merge branch 'dev' into rasterio_1_3_2
cd16e9a
@dmannarino
Forgot to update Pipfile.lock
a3c38fe
@dmannarino
Fix straggling reference to old rasterio
85f3a46
@dmannarino
Explicitly install older terraform
718e352
@dmannarino
Don't use tee to suppress garbage in logs; add --no-install-recommend…
c920577 
…s to reduce bloat
@dmannarino
Try un-pinning jinja2
86c731b
@dmannarino
jinja2 still not detected: Try unpinning starlette and fastapi (grasp…
9df601d 
…ing at straws)
@dmannarino
Try --no-upgrade to avoid upgrading python to 3.11
3f4033e
@dmannarino
Revert no-upgrade clause (which didn't preveent Python 3.11 from inst…
5f4cf25 
…alling); Forgot to update PIpfile.lock. Oops
@dmannarino
Try to fix new error...
5d55cb3
@dmannarino
Try upgrading localstack for Python 3.10 lambda support. Slowly.
9538dbe
@dmannarino
Try reducing version of Python for lambdas
7a99d3f
@dmannarino
Lost patience, increment localstack to 0.14.5 (and revert previous ch…
45ff7a5 
…ange to lambda runtimes)
@dmannarino
Try adding 4566 to ports
34e5f0c
@dmannarino
Revert foolish TF upgrade (even going up a minor version is problematic)
204d2d3
@dmannarino
Fix removing semicolons (note to self: Stop touching things
4e9ce7e
@dmannarino
Upgrade to localstack 2.2
ac70266
@dmannarino
Change back to older TF provider schema
61dd1db
@dmannarino
Revert changes to TF docker compose file, as they break things in GHA
272c7c6
@dmannarino
Factor-out Python, package versions in test script
ff3abb9
@dmannarino
Use lambda_runtime and lambda_edge runtime to reduce explicit hard-co…
e0f4769 
…ding of Python versions in TF
@dmannarino
Make further use of variables to decrease instances of explicitly spe…
ffc4c40 
…cifying Python versions
@dmannarino
Forgot a file
cef82ab
@dmannarino
Use variables to specify versions of lambda layers in tests
f6a3b53
@dmannarino
WIP (error running tests): Replace attempt to use function in variabl…
b97859d 
…es (not allowed) with locals in test terraform
@dmannarino
Merge pull request #109 from wri/rasterio_1_3_2
5c9cb17 
Update, add packages for Python 3.10
@dmannarino
Merge branch 'staging' into dev
2894f01
@dmannarino
Merge pull request #124 from wri/dev
beba7ce 
Merge Python 3.10 upgrade up to staging
@dmannarino dmannarino merged commit 6c8a614 into production Sep 26, 2023
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@solomon-negusse solomon-negusse solomon-negusse approved these changes

@gtempus gtempus Awaiting requested review from gtempus

@jterry64 jterry64 Awaiting requested review from jterry64

@danscales danscales Awaiting requested review from danscales

@manukala6 manukala6 Awaiting requested review from manukala6

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dmannarino @solomon-negusse