Skip to content

Revert "Fix Trace ID environment variable"#2730

Merged
kyleknap merged 2 commits intodevelopfrom
revert-2727-fix-trace-id
Jul 30, 2022
Merged

Revert "Fix Trace ID environment variable"#2730
kyleknap merged 2 commits intodevelopfrom
revert-2727-fix-trace-id

Conversation

@kyleknap
Copy link
Copy Markdown
Contributor

Reverts #2727

Redshift Data users running in Lambda get 500's not that the environment variable is being respected:

$ AWS_LAMBDA_FUNCTION_NAME=foo _X_AMZN_TRACE_ID=samcheck aws redshift-data list-statements --region us-east-1

An error occurred (InternalFailure) when calling the ListStatements operation (reached max retries: 4): 

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 30, 2022

Codecov Report

Merging #2730 (0075e5b) into develop (c61e661) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop    #2730   +/-   ##
========================================
  Coverage    95.25%   95.25%           
========================================
  Files           60       60           
  Lines        12278    12278           
========================================
  Hits         11696    11696           
  Misses         582      582           
Impacted Files Coverage Δ
botocore/handlers.py 95.05% <100.00%> (ø)

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 c61e661...0075e5b. Read the comment docs.

@kyleknap kyleknap merged commit 315c835 into develop Jul 30, 2022
@kyleknap kyleknap deleted the revert-2727-fix-trace-id branch July 30, 2022 18:57
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