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 aws.md with errors #2226

Merged
merged 3 commits into from
Mar 11, 2025
Merged

Update aws.md with errors #2226

merged 3 commits into from
Mar 11, 2025

Conversation

grusev
Copy link
Collaborator

@grusev grusev commented Mar 10, 2025

added couple more error messages

Reference Issues/PRs

What does this implement or fix?

Any other comments?

Checklist

Checklist for code changes...
  • Have you updated the relevant docstrings, documentation and copyright notice?
  • Is this contribution tested against all ArcticDB's features?
  • Do all exceptions introduced raise appropriate error messages?
  • Are API changes highlighted in the PR description?
  • Is the PR labelled as enhancement or bug so it appears in autogenerated release notes?

added couple more error messages
@grusev grusev requested review from phoebusm and alexowens90 March 10, 2025 08:53
This is usually due to an incorrect IAM account setup or an incorrect configuration file.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think in terms of the errors and causes, it requires a separate section, rather put everything in monolithic text. Can you please separate those.

This is usually due to an incorrect IAM account setup or an incorrect configuration file.

'Access Denied' error most probably mean incorrect aws configuration file:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this line can be removed

Actual: arcticdb_ext.exceptions.StorageException: E_S3_RETRYABLE Retry-able error: S3Error#99 : Encountered network error when sending http request for object '_arctic_cfg/cref/'
```
A loss of network connectivity would triggers such error. Note, that this error will appear after several attempts to re-establish the connection
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
A loss of network connectivity would triggers such error. Note, that this error will appear after several attempts to re-establish the connection
A loss of network connectivity could trigger such an error. Note, that this error will appear after several attempts to re-establish the connection.

@grusev grusev added the patch Small change, should increase patch version label Mar 11, 2025
@grusev grusev merged commit 1c5ffa6 into master Mar 11, 2025
146 of 147 checks passed
@grusev grusev deleted the grusev-patch-1 branch March 11, 2025 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Small change, should increase patch version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants