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

[Feature STG 85/86] Cold Tier #21024

Conversation

siminsavani-msft
Copy link
Contributor

@siminsavani-msft siminsavani-msft commented Jun 15, 2023

  1. Added AccessTierCold to constants.go in blob
  2. Added tests or updated tests for the following APIs:
  • Container - Flat List Blob, Hierarchy List Blobs
  • BlockBlob - Upload(), UploadBlobFromURL/PutBlobFromURL(), CommitBlockList()
  • Blob - SetTier(), GetPropertiest(), CopyFromURL(), StartCopyFromURL()
  • The purpose of this PR is explained in this or a referenced issue.
  • The PR does not update generated files.
  • Tests are included and/or updated for code changes.
  • Updates to CHANGELOG.md are included.
  • MIT license headers are included in each file.

@github-actions github-actions bot added the Storage Storage Service (Queues, Blobs, Files) label Jun 15, 2023
@siminsavani-msft
Copy link
Contributor Author

/azp run go - azblob

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@siminsavani-msft siminsavani-msft changed the title Cold tier [Feature STG 85/86] Cold Tier Jun 19, 2023
@siminsavani-msft
Copy link
Contributor Author

New PR here: #21032

@siminsavani-msft siminsavani-msft deleted the cold-tier branch June 19, 2023 17:51
@@ -3,6 +3,7 @@
## 1.1.0-beta.2 (Unreleased)

### Features Added
* Added `AccessTierCold`.
Copy link
Member

Choose a reason for hiding this comment

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

Added support for Cold tier.

//// testcommon.ValidateBlobErrorCode(c, err, bloberror.InvalidHeaderValue)
////}
////
// //func (s *BlobRecordedTestsSuite) TestBlobTierInferred() {
Copy link
Member

Choose a reason for hiding this comment

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

Why is this test commented out?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AzBlob Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants