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

support azure driver for metadata management #1410

Open
wants to merge 3 commits into
base: metadata-management
Choose a base branch
from

Conversation

devanshjain7
Copy link
Contributor

What type of PR is this?

/kind new feature

What this PR does / why we need it:
This PR adds support for Azure Blob Storage for metadata collection.

Please Note:
Currently not able to collect the following object level parameters:
Tags, ACL, Expires Date, Encryption, VersionID
Reason: Due to usage of old version of Azure SDK ([email protected])

Test Report Added?:

/kind TESTED

Test Report:
image
image

Special notes for your reviewer:

@codecov
Copy link

codecov bot commented Mar 6, 2023

Codecov Report

Merging #1410 (9ecf57e) into metadata-management (c372b6d) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@                 Coverage Diff                  @@
##           metadata-management    #1410   +/-   ##
====================================================
  Coverage                48.12%   48.12%           
====================================================
  Files                       10       10           
  Lines                     1571     1571           
====================================================
  Hits                       756      756           
  Misses                     756      756           
  Partials                    59       59           

@devanshjain7 devanshjain7 changed the title add azure driver - first iteration support azure driver for metadata management Mar 7, 2023
@PravinRanjan10
Copy link
Collaborator

Please separate the vendor pkg out of this PR

Copy link
Collaborator

@PravinRanjan10 PravinRanjan10 left a comment

Choose a reason for hiding this comment

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

Please separate the vendor pkg out of this PR

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

Successfully merging this pull request may close these issues.

2 participants