Skip to content

Draft: [Azure storage] as a part of fileUploader mode#146

Closed
mykhailiukVitalii wants to merge 14 commits intomasterfrom
100-azure-store
Closed

Draft: [Azure storage] as a part of fileUploader mode#146
mykhailiukVitalii wants to merge 14 commits intomasterfrom
100-azure-store

Conversation

@mykhailiukVitalii
Copy link
Copy Markdown
Contributor

  1. I have added a new Azure store processing using AZURE_STORAGE_CONNECTION_STRING & AZURE_CONTAINER_NAME
  2. Big fileUploader mode refactoring

WIP:

  • intagreation tests
  • test the functionality on all examples from /examples folder (all adapter tests)

Comment thread lib/fileUploader/uploadManager.js Outdated
* @param {string} filepath - The filepath to be uploaded.
* @param {string} runId - The ID of the run or identifier for the file.
*/
const handleFileUploadByPath = async (runId, filepath) => {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please transform into class

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

in my opinion, this is not a good idea. It is better to use the uploadManager to handle different download options such as S3 or Azure

@mykhailiukVitalii
Copy link
Copy Markdown
Contributor Author

It remains to delete the old fileUploadfile and carefully recheck the reporter based on the popular frameworks!!!

@olexandr13 olexandr13 deleted the branch master February 10, 2026 13:43
@olexandr13 olexandr13 closed this Feb 10, 2026
@olexandr13 olexandr13 deleted the 100-azure-store branch February 10, 2026 13:43
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.

3 participants