aws-cli icon indicating copy to clipboard operation
aws-cli copied to clipboard

Update supported PyYAML range.

Open zozoens31 opened this issue 3 years ago • 3 comments

Issue #, if available:

Description of changes: Update supported PyYAML range to include 6.0.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

zozoens31 avatar Jan 10 '22 09:01 zozoens31

Codecov Report

Merging #6648 (ae78bf3) into develop (3ecbd27) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #6648   +/-   ##
========================================
  Coverage    92.84%   92.84%           
========================================
  Files          204      204           
  Lines        16293    16293           
========================================
  Hits         15128    15128           
  Misses        1165     1165           

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3ecbd27...ae78bf3. Read the comment docs.

codecov-commenter avatar Jan 12 '22 20:01 codecov-commenter

Any update on this PR? What is the next step to get it merged?

pcorpet avatar Mar 21 '22 10:03 pcorpet

Any updates? This is somewhat of a blocker when using alongside other dependencies which require PyYAML>=6.0.

dhalenok avatar Sep 12 '22 13:09 dhalenok

Hi, will this be resolved at some point? It seems like a trivial issue that will improve the user experience by a lot.

kdewald avatar Jan 30 '23 23:01 kdewald

Hello @stealthycoin, sorry for pinging you in this old PR, but you seem to be actively maintaining this repo.

I was wondering if there is an underlying issue blocking the merge of this PR? Or if maybe it just fell through the cracks? If possible, it would be very useful to merge this PR since many libraries ask now for PyYAML>=6.0 and aws-cli impedes installing it.

Thank you very much in advance!

dbuades avatar Apr 07 '23 14:04 dbuades

@stealthycoin I think you should be aware that due to the fact that AWS released the newest Managed Airflow (v2.5.1) that is using PyYAML v6+ without merging this pull, AWS customers won't be able to install awscli in newest MWAA. awscli is required to handle communication between MWAA and EKS.

phajduk avatar Apr 20 '23 09:04 phajduk

Pitching in, it's a dealbreaker for me to upgrade my team's Airflow instance. Doesn't seem to be a huge change for @aws-cli team to merge this pr

vlazuka avatar May 25 '23 21:05 vlazuka

AWS released the newest Managed Airflow (v2.5.1) that is using PyYAML v6+ without merging this pull, AWS customers won't be able to install awscli in newest MWAA

@phajduk - MWAA pre-installs aws cli v2, which doesn't have PyYAML as dependency. Were you trying to install cli v1 on MWAA? If yes, you should move over to pre-installed v2.

shubham22 avatar Jun 29 '23 17:06 shubham22

@shubham22 but what about other platforms, not MWAA?

vlazuka avatar Jun 29 '23 18:06 vlazuka

@vlazuka - I was only answering w.r.t. MWAA. Based on the latest comment here from AWS cli member, it does seem like cli v2 is the only workaround right now.

shubham22 avatar Jun 29 '23 19:06 shubham22

Resolved in https://github.com/aws/aws-cli/pull/8037. Closing.

jonathan343 avatar Jul 28 '23 05:07 jonathan343