-
Notifications
You must be signed in to change notification settings - Fork 0
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
External sftp code and local config #72
Conversation
…`copyfiles`; refactor some duplicate code; add placeholder env vars for SFTP; add placeholder calls to SFTP code Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com> Co-Authored-By: halprin <halprin@users.noreply.github.com> Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com>
… with folder structure Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com> Co-Authored-By: halprin <halprin@users.noreply.github.com> Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com> Co-Authored-By: jherrflexion <118225331+jherrflexion@users.noreply.github.com>
Co-Authored-By: halprin <halprin@users.noreply.github.com> Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com>
Co-Authored-By: halprin <halprin@users.noreply.github.com> Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
Co-authored-by: Sylvie <sschuresko@flexion.us> Co-authored-by: Rajeev Ranjan <rranjan@flexion.us> Co-authored-by: jcrichlake <145698165+jcrichlake@users.noreply.github.com> Co-authored-by: pluckyswan <96704946+pluckyswan@users.noreply.github.com> Co-authored-by: saquino0827 <saquino@flexion.us> Co-authored-by: James Herr <jherr@flexion.us>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com> Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com> Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com> Co-Authored-By: Samuel Aquino <saquino@flexion.us>
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com>
@@ -3,7 +3,7 @@ set -e | |||
|
|||
pushd ./src/ || exit | |||
|
|||
THRESHOLD=50 | |||
THRESHOLD=65 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we know what threshold we eventually want to reach? Is it the same as TI app?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't have a specific number in mind. It will depend on how much code we have that is just pointing to an external library that we wouldn't want to test.
Co-Authored-By: pluckyswan <96704946+pluckyswan@users.noreply.github.com> Co-Authored-By: Samuel Aquino <saquino@flexion.us> Co-Authored-By: jcrichlake <145698165+jcrichlake@users.noreply.github.com> Co-Authored-By: jherrflexion <118225331+jherrflexion@users.noreply.github.com>
Quality Gate passedIssues Measures |
External Partner SFTP and local config
We need to be able to connect to external SFTP clients and pull down files from them. This PR is the start of that work. It includes the code for retrieving data from an external SFTP server and copying it to our Azure storage container, plus config to enable local testing. We don't yet have credentials/info to set up connections to any customer's deployed environment.
Issue
SFTP issue
Checklist
Note: You may remove items that are not applicable