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

Improvement/zenko 2429 backbeat direct data access #964

Open
wants to merge 2 commits into
base: development/8.1
Choose a base branch
from

Conversation

jonathan-gramain
Copy link
Contributor

  • improvement: ZENKO-2429 copy location management to arsenal

Copy the logic to load locations based on orbit location types in
arsenal, to be used by Backbeat to have direct GET/PUT access to
locations.

  • improvement: ZENKO-2429 extend AwsClient.get() API

Allow the caller not to pass a callback, but retrieve the request
object directly.

This is needed for Backbeat to manage request canceling (abort) of one
end when the other end raises an error.

Copy the logic to load locations based on orbit location types in
arsenal, to be used by Backbeat to have direct GET/PUT access to
locations.
Allow the caller not to pass a callback, but retrieve the request
object directly.

This is needed for Backbeat to manage request canceling (abort) of one
end when the other end raises an error.
@bert-e
Copy link
Contributor

bert-e commented Mar 23, 2020

Hello jonathan-gramain,

My role is to assist you with the merge of this
pull request. Please type @bert-e help to get information
on this process, or consult the user documentation.

Status report is not available.

@bert-e
Copy link
Contributor

bert-e commented Mar 23, 2020

Waiting for approval

The following approvals are needed before I can proceed with the merge:

  • the author

  • 2 peers

Peer approvals must include at least 1 approval from the following list:

@bert-e
Copy link
Contributor

bert-e commented Mar 23, 2020

Waiting for approval

The following approvals are needed before I can proceed with the merge:

  • the author

  • 2 peers

Peer approvals must include at least 1 approval from the following list:

The following options are set: approve

@rahulreddy
Copy link
Collaborator

/wait

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants