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

Backport to 5.17 #8296

Merged
merged 7 commits into from
Aug 19, 2024
Merged

Backport to 5.17 #8296

merged 7 commits into from
Aug 19, 2024

Conversation

jackyalbo
Copy link
Contributor

Explain the changes

Issues: Fixed #xxx / Gap #xxx

Testing Instructions:

  • Doc added/updated
  • Tests added

shirady and others added 5 commits August 19, 2024 15:08
1. list_config_files change function signature (remove unused config_path) and edit JSDoc (add missing type)
2. rename the option from should_decrypt to decrypt_secret_key as the destructor of options uses

Signed-off-by: shirady <[email protected]>
(cherry picked from commit 6268072)
1. In list_config_files "mock" the entries array to have one entry and it is the name by the filter (for performance).
2. Exclude the action list from fetch_bucket_data and fetch_account_data.
3. Edit the account list tests to validate that we receive an array in the response.

Signed-off-by: shirady <[email protected]>
(cherry picked from commit cd5d80e)
Signed-off-by: nadav mizrahi <[email protected]>
(cherry picked from commit 4898af4)
1. Handle the case where the endpoint port was wrong - for example:
  - S3 port with IAM action
  - IAM port with S3 action

Signed-off-by: shirady <[email protected]>
(cherry picked from commit 5d071d4)
Signed-off-by: jackyalbo <[email protected]>
Signed-off-by: jackyalbo <[email protected]>
(cherry picked from commit 8c21f68)
@nimrod-becker nimrod-becker merged commit 93aa33f into noobaa:5.17 Aug 19, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants