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

Migrated to track 2 datalake SDK #2366

Merged
merged 167 commits into from
Oct 2, 2023
Merged

Conversation

gapra-msft
Copy link
Member

No description provided.

adreed-msft and others added 30 commits August 31, 2022 14:01
#1872)

* Add check for 0 length, attempt to validate the returned object.

* Change to grabbing real SD length

* Add comment describing issue

* Prevent infinite loop upon listing failure

* Fix GCP error checking

* Fix GCP disable
* Manipulate URLs safely

* Fix folder deletion test

* Prevent infinite loop upon listing failure

* Fix GCP error checking

* Fix GCP disable

* Fail when errors listing/clearing bucket
* Microsoft mandatory file

* fixing small typo (,) in help of jobs clean

Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com>
Co-authored-by: Mohit Sharma <[email protected]>
* Implement MD OAuth testing

* Handle async on RevokeAccess, handle job cancel/failure better

* Prevent parallel testing of managed disks

* lint check

* Prevent infinite loop upon listing failure

* Fix GCP error checking

* Fix GCP disable

* Fail when errors listing/clearing bucket

* Add env vars

* Avoid revoking MD access, as it can be shared.

* Fix intermittent failures

* Disable MD OAuth testing temporarily.
* Add bugfixes to change log.

* Correct wording & punctuation
* Microsoft mandatory file

* Add info in error message for mkdir on Log/Plan

Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com>
Co-authored-by: Mohit Sharma <[email protected]>
* Microsoft mandatory file

* Fix fixupTokenJson

Co-authored-by: microsoft-github-policy-service[bot] <77245923+microsoft-github-policy-service[bot]@users.noreply.github.com>
Co-authored-by: Mohit Sharma <[email protected]>
Co-authored-by: Adam Orosz <[email protected]>
* Fix bad build + Prevent bad builds in the future

* Add Windows build
* Make sync use last write time for Azure Files

* Implement test
Merge pull request #1966 from Azure/gapra/codeowners
Merge pull request #1967 from Azure/gapra/cpkRetryFix
Fixed a bug where preserve permissions would not work with OAuth
common/defaultlog_syslog.go Dismissed Show dismissed Hide dismissed
common/defaultlog_syslog.go Dismissed Show dismissed Hide dismissed
common/defaultlog_syslog.go Dismissed Show dismissed Hide dismissed
@gapra-msft
Copy link
Member Author

These alerts are from files I added to keep supporting logging without pipeline.LogLevel. Should be safe to just dismiss the alerts. Thoughts?

@gapra-msft gapra-msft changed the title [WIP] Migrated to track 2 datalake SDK Migrated to track 2 datalake SDK Sep 18, 2023
Copy link
Contributor

@tasherif-msft tasherif-msft left a comment

Choose a reason for hiding this comment

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

ah thats nice to see :')

Copy link
Member

@nakulkar-msft nakulkar-msft left a comment

Choose a reason for hiding this comment

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

should merge to dev instead

@gapra-msft gapra-msft changed the base branch from main to dev September 20, 2023 15:48
@gapra-msft
Copy link
Member Author

should merge to dev instead

Good catch

@gapra-msft gapra-msft merged commit d1824f0 into dev Oct 2, 2023
17 checks passed
@gapra-msft gapra-msft deleted the gapra/track2_azdatalake_migration branch October 2, 2023 18:07
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.