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

[Tables] changing QueryOptions param to be a ptr #14961

Merged
merged 2 commits into from
Jul 8, 2021

Conversation

seankane-msft
Copy link
Member

No description provided.

@check-enforcer
Copy link

check-enforcer bot commented Jul 1, 2021

This repository is protected by Check Enforcer. The check-enforcer check-run will not pass until there is at least one more check-run successfully passing. Check Enforcer supports the following comment commands:

  • /check-enforcer evaluate; tells Check Enforcer to evaluate this pull request.
  • /check-enforcer override; by-pass Check Enforcer (approvals still required).

Copy link
Member

@catalinaperalta catalinaperalta left a comment

Choose a reason for hiding this comment

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

Looking good, just switch to using nil when there are no options to be configured. Then in the code you can initialize an empty version of the options to be used to not cause a panic.

Initialize the options internally like this in the ctor (example taken from azidentity):

        if options == nil {
		options = &ClientSecretCredentialOptions{}
	}

sdk/tables/aztable/table_client.go Outdated Show resolved Hide resolved
sdk/tables/aztable/table_service_client.go Outdated Show resolved Hide resolved
sdk/tables/aztable/zt_table_recorded_tests.go Outdated Show resolved Hide resolved
sdk/tables/aztable/table_client.go Show resolved Hide resolved
@seankane-msft seankane-msft merged commit 6bc983b into Azure:track2-tables Jul 8, 2021
@seankane-msft seankane-msft deleted the tables-query-ptr branch July 8, 2021 16:27
seankane-msft added a commit that referenced this pull request Sep 2, 2021
* Initial generation of the aztables module (#14441)

* initial generation

* Update ci.yml

* Introduce internal/testFramework and test aztables with it (#14491)

* testFramework initial implementation

* sanitizer fixes (#14600)

* Progress on aztables (#14676)

* implement batch operations and all entity operations

* AsModels

* rename

* comments

* comment tweaks, options as pointer

* add EntityMapAsModel

* refactors to AddEntity

* update some code docs

* doc fixes

* doc fixes

* doc improvements

* rename testframework to recording

* move ci.yml

* ci path

* ci path

* [Tables] Update from testframework -> recording (#14888)

* Initial generation of the aztables module (#14441)

* initial generation

* Update ci.yml

* Introduce internal/testFramework and test aztables with it (#14491)

* testFramework initial implementation

* sanitizer fixes (#14600)

* Progress on aztables (#14676)

* implement batch operations and all entity operations

* AsModels

* rename

* comments

* comment tweaks, options as pointer

* add EntityMapAsModel

* refactors to AddEntity

* update some code docs

* doc fixes

* doc fixes

* doc improvements

* rename testframework to recording

* fixing tables to go from testframework -> recording

* renaming recordingInstance -> r

* file name change

Co-authored-by: Christopher Scott <[email protected]>
Co-authored-by: catalinaperalta <[email protected]>

* [Tables] ci fix (#14956)

* [Tables] Add post-request validation of entity (#14960)

* adds a secondary validation if the entity doesnt have pk/rk

* creating an errors file just for handling errors

* removing extra line

* moving error to constants file, renaming constants file

* improving test

* [Tables] changing QueryOptions param to be a ptr (#14961)

* changing QueryOptions param to be a ptr

* feedback from catalina

* [Tables] linting Fixes (#15028)

* small fixes

* small fixes

* small fixes

* small fixes

* small fixes

* small fixes

* adding error checking to transactional batching

* adding error checking to test set up

* adding more error checking

* adding more error checking

* fixing two tests

* fixing error reporting

* fixing error reporting

* fixing error reporting

* fixing last lint failure

* [Tables] Adding swagger.md  (#15016)

* adding swagger

* updating swagger with autorest command

* more fixes

* fixing directive

* updating swagger.md

* fixed swagger

* fixing directive

* add dataplane flag

* updating swagger

* commented out tests, passes remaining

* addressing chris hamons feedback

* undoing changes

* undoing the rest of changes

* updating generated code

* formatting gen code

* more formatting corrections

* one last formatting correction

* [Tables] Add get/set access policy (#15027)

* adding a way to skip cosmos tests, adding access policies

* adding support for getting/setting access policies

* tests working

* does not work for multiple access policies

* adding post validation

* added error validation, note about possible issue

* set policy works

* removing table access policy file

* [Tables] Add GetStats, GetProperties, and SetProperties methods (#15112)

* adding GetProperties

* basic test of set properties

* added logging and metrics tests

* tests for setting service properties

* adding test for retention too long

* adding samples and docs

* [Tables] Add support for Entity struct (#15003)

* adding test for custom entity

* further

* starting access policy

* working for basic entity

* adding to test

* new generated code, removed one more test

* AddEntity now takes []bytes

* adding directive to change entity to []byte, changes to compile and be able to pass

* converting Value map[string]interface{} -> []byte in convenience layer

* paging works the same way now too

* fixing two tests, update complex and basic test entities, need to add type hints for int64/float64 types

* adding test for delete method, changed delete etag to be nil

* working test for merge

* Added Merge test

* fix up test query simple entity

* complex query test

* everything except for batch tests

* basic batch functionality

* fixing batch mixed test

* fixed last batch test

* added separate file for testing batch actions

* fixing query on table service client to take a pointer as well

* working on annotating entities

* fixing recordings

* working with odata annotations

* working test for using odata to annotate entity

* adding options to get

* working with embedded entity now too

* adding to entity, peeling off prints

* converted to a type

* first change to add EdmEntity type

* updated unmarshalling

* removing two files

* fix editLink

* removing sample

* cleaning up

* removing more code

* cleaning up

* formatting

* formatting

* [Tables] change Query to List (#15099)

* adding List methods

* moving helper to zt file

* improving docstrings

* changing all query methods to list

* updating docstrings

* [Tables] Update transactions method (#15125)

* adding test for custom entity

* further

* starting access policy

* working for basic entity

* adding to test

* new generated code, removed one more test

* AddEntity now takes []bytes

* adding directive to change entity to []byte, changes to compile and be able to pass

* converting Value map[string]interface{} -> []byte in convenience layer

* paging works the same way now too

* fixing two tests, update complex and basic test entities, need to add type hints for int64/float64 types

* adding test for delete method, changed delete etag to be nil

* working test for merge

* Added Merge test

* fix up test query simple entity

* complex query test

* everything except for batch tests

* basic batch functionality

* fixing batch mixed test

* fixed last batch test

* added separate file for testing batch actions

* fixing query on table service client to take a pointer as well

* working on annotating entities

* fixing recordings

* working with odata annotations

* working test for using odata to annotate entity

* adding options to get

* working with embedded entity now too

* adding to entity, peeling off prints

* converted to a type

* first change to add EdmEntity type

* updated unmarshalling

* removing two files

* fix editLink

* removing sample

* cleaning up

* removing more code

* cleaning up

* re-record and remove methods

* more cleanup

* formatting

* having issues with cosmos tests

* Adding Table suffix to Create, Delete, and List methods on service client (#15181)

* [Tables] Architect Feedback (#15196)

* architect feedback

* docstrings

* feedback

* fixing eng pipelines

* figuring out pipeline failure

* missing recording

* restore eng changes

* [EngSys] Test proxy (#15101)

* [Tables] Add authentication via SAS credential (#15256)

* changing recording to use test-proxy

* updating client and test proxy

* updated Transport Do to send actual request

* whitespace

* working implementation

* updating internal recording

* now I have a blank recording...

* cleaning up

* more clean up

* transport Do is repeating, proxy is not parsing URL correctly

* small changes

* adding a policy to do the same thing

* adding a second test for proxy and transport

* further progress with Jeff, switching to only using a policy right now

* cleaning up file

* cleaning up

* added more debugging, added two required headers that I was missing

* proxy working on a single test, need to convert all tests to test proxy

* cleaning up

* moved over successfully :)

* more helper functions

* working for a subtest functionality

* fixed up a service client test

* converting more service tests

* converted all service client tests

* converted table client tests

* fixed last two client tests

* entity test

* entity tests

* converting access policy tests

* all passing in record mode

* batch tests

* liveonly for batch tests

* adding a sanitizer and more methods for recording

* adding main method to start and stop the proxy automatically

* adding test proxy step to pipeline

* double dash on version arg

* need a better way to get userful error reporting

* explicitly running test-proxy in the background

* adjust the proxy start and install to be compatible with specific process

* dont run test proxy in background

* didnt remove an import

* header to proxy_test.go

* fixes for pipeline

* added loggin on accident

* use nohup on linux machines

* further simplify nohup usage

* there are no exes on windows

* fixing errors, passing in default env variable

* force background for linux task. may need another iteration to place it within the quotes

* fake aad credential

* clean up after test-proxy

* add print for record mode

* another different attempt at starting the proxy

* jobs apparently is an unrecognized command on our linux boxes

* correct dumb typo in run tests

* replace nuget install with docker

* working fake credentials for client delete entity test

* fixing fake credentials portion, passing locally

* trying an insecureskipverify transport

* debugging statements to find the cwd

* removing blank recording

* working implementation of sas

* working sas credential for account signature

* table sas is failing

* leverage proxy tool, not docker

* remove apostrophes

* moving recordings one directory up

* transitioning to using the sas written by storage team

* using storage methods

* copied storage code, sas needs to be appended in a different way

* changing directory for test-proxy to start from

* removing parenthesis

* adding steps for validating ssl

* switching to docker

* adding start server script

* working implementation if we can fix the prepending of 'recording' to the docker request finder

* big refactor, moving stuff into recording file, cleaning up proxy_test.go file

* last fixings

* convert back to docker. windows and linux images now present

* double condition

* moving configuration to a separate file. preparing for eng/common move

* tier 0 of trust. I don't expect this to work, but it's still worth a shot. maybe the devops agents do something differently

* update condition to use variable syntax

* guess I'm not using variables. removing

* call start-server.ps1

* adjust build-test and configure-proxy to run the docker container in context

* wrap in quotes for the container create. it's apparently a bad

* adjust the initialize call

* disable vet temporarily

* looking for cert file in env variable PROXY_CERT

* set proxy_cert environment variable to find that certificate

* changing to crt file

* use crt cert

* want to see errors

* powershell errors

* correcting how the volume binds to the windows container

* small modification for windows container

* finally got the magic sauce

* cleaning up to remove azcore from internal

* removing recordings that use vcr

* issue with the url creation

* persist query params correctly in azcore.JoinPaths

* removing print statements

* return root

* changing location of script

* forgot the stop command

* working table level sas implementation

* adding start/end rk/pk functionality

* removing print statements

* removing more non-tables code

* docstrings and removing storage only code

* making sas tests live only

* Fixing sas table name to be lowercase always, adding test to verify read only

* Fixing sas table name to be lowercase always, adding test to verify read only

* adding cosmos sas test

* fix for params

* apiview fixes

* fixing test

* undoing change

* updates

* error naming

* fixed service client, was adding percallpolicies twice

* simplification

* aligning with main

* converting from query pager to list pager for naming consistency

* updating objects for consistency

* jeffs feedback

* small fixes

* autorest.md changes

* fixing tablename

Co-authored-by: scbedd <[email protected]>

* fixing internal (#15315)

* Fixes2 (#15316)

Makes a fix to azcore in track2-tables branch

* [Tables] bump deps (#15350)

* Sync eng/common directory with azure-sdk-tools for PR 1912 (#15314)

* Attempt to purge all vaults, managed HSMs

Reverts #1910. Vaults and managed HSMs are automatically purged on their purge date. The point was to purge them daily to preserve capacity. The default purge date is +90 days.

* Add timeout and more logging

* Pass required -Resource

* Fix log message

* Ensure the $Resource is correctly captured

Added comment to new code explaining why, since ScriptBlock.GetNewClosure() is not working as expected.

* Add -ErrorAction to Receive-Job

Worked without terminating when run locally, but failed on the first error in the AzDO agent.

* Use $using:r instead of creating ScriptBlock

More idiomatic for passing ScriptBlocks to jobs.

* Resolve PR feedback

* Change default DeleteAfterHours to 120

Resolves #1917

* Use the Az cmdlets built-in -AsJob

Co-authored-by: Heath Stewart <[email protected]>

* Fix query batch processing (#15319)

Co-authored-by: Wes Haggard <[email protected]>

* fix: three dots path not work in linux in build script (#15297)

* Release v56.3.0 1629350585 (#15323)

* Generated from specification/apimanagement/resource-manager/readme.md tag package-preview-2021-01 (commit hash: 30d1537f0c9aa49e6e04401b1d16f612b31231e7)

* Generated from specification/purview/resource-manager/readme.md tag package-2021-07-01 (commit hash: 30d1537f0c9aa49e6e04401b1d16f612b31231e7)

* v56.3.0

* fix

* Remove old unused update changelog script (#15326)

Co-authored-by: Wes Haggard <[email protected]>

* add clear output flag to build.ps1 (#15255)

* add clear output flag

* remove the path parameter and make sure the process-sdk function only runs in current directory

* change flag name

* [tools/generator] add automation and release command for track2 (#15271)

* [tools/generator] add automation and release command for track2

* add copyright and do gofmt

* Use semver to pump version, move git operation to repo package and fix some typo

* Add specfic version to release cmd, seperate package for automation and remove test config file

* add automation script and config for track2

* fix: three dots path not work in linux in build script

* fix: ln pwsh.exe to pwsh to run in linux

* fix: ignore dep check for go-git

* chore: label name change for track2

* fix: wrong package path result in empty git commit

* fix: repo addr replacement in autorest.md

* chore: move script to eng folder and change name

* fix: onboard generation need to replace NewClientMethod

* chore: change all naming from track2 to v2

* fix: add new build constraint

* fix: template compat with go 1.17 and change commit msg for release

* chore: add v2 to readme begin regex to support future change of `track2` naming

* chore: remove track2 naming

* fix: duplicated and wrong expression

* fix: add package title param to release cmd

* fix: ci

* Prevent ManagedIdentityCredential mutating GetToken arguments (#15331)

* [Azcore] bumps version of internal (#15347)

* bump internal to latest

* upgrading internal

* not the correct one

* changelog update

* [azidentity] bump azcore and internal dependencies (#15348)

* updating to latest core and identity

* trying to fix merge

* undoing changes to identity that should not be there

* more changes

* more changes

* removing a bunch more

* one last file

* formatting

* linting checks

* linting

* formatting

Co-authored-by: Azure SDK Bot <[email protected]>
Co-authored-by: Heath Stewart <[email protected]>
Co-authored-by: Wes Haggard <[email protected]>
Co-authored-by: tadelesh <[email protected]>
Co-authored-by: JiahuiPeng <[email protected]>
Co-authored-by: Arcturus <[email protected]>
Co-authored-by: Charles Lowell <[email protected]>

* [Tables] connection string parser (#15187)

* adding conn str parser

* Basic conn str parser

* adding support for custom domains

* invalid tests

* adding self note

* Update sdk/tables/aztable/connection_string.go

Co-authored-by: Richard Park <[email protected]>

* Update sdk/tables/aztable/connection_string.go

Co-authored-by: Richard Park <[email protected]>

* improving testing

* finishing connection string portion

* formatting and improved error handling

* docstring addition for private method

* connection string error

* splitn instead of split

* sas comment

* updating proxy-server script

* configure proxy with hash of file, correct file location

* using certs in main

* trying download with no password

* trying without the pfx file altogether

* adding pfx portion back

* manual copy of testproxy to common

* using common

* fixing path

* adding trust ps1 script

* adding step to run proxy

* formatting and go mod x/net

* fixing for latest azcore

* fixing rqeuire

* removing eng stuff

* conn str

* running tests

* run tests = true

* adding build sourcesDir

* relative path

* fixing up path

* build-test

* moving byte array response inside table pager

* env var playing to fix ci lint

* changing package name

Co-authored-by: Richard Park <[email protected]>

* [Tables] simplify code space by moving generated to `internal` module (#15299)

* changing to internal package, everything except batching works

* back to working

* converting more things to private

* renamings

* still working, checkpoint

* most of tableclient done

* finished table client options/responses

* finished with service client

* cors rule done

* retentionpolicy

* retentionpolicy and cors

* last piece

* final comments

* updating with newest autorest

* formatting

* fixing go mod in internal package

* update versions

* flipping param order

* convert internal to a subpackage

* fixing telemetry and bumping go version

* updating proxy, adding test-resources.json file

* cert issues

* formatting and x/net

* fixing build-test

* correcting file path

* lint and formatting

* go fmt

* dropping coverage for internal

* updates from pre-arch board

* checking error

* doing scopes work in generated code

* linting

* removing old file

* consistent namings

* removing types and simplifying pageresponse for list entities

* formatting

* renames, removing Table prefix, removing TableTRansactionError

* fixed error type, placeholder while azcore is in PR

* running tests live, fixing one test, converting to always using shared key

* formatting

* linter

* fixing playback for storage

* fixing playback credential

* fixing serviceURL to expect tableName in service url

* moving update mode into options for update/insert entity

* return *Client from CreateTable method

* cleaning up structs

* flipping order of strings in conn str constructor

* renaming param of transactionAction from ETag -> IfMatch

* remove XMS prefix, change Value to Entities/Tables

* moving GetTableSASToken to Client struct

* making MinimalOdata the default, removing option to set it, new recordings

* renaming ETag -> IfMatch for Options

* formatting

* renaming file, adding version to constants.go file

* adding version

* updating to latest azcore

* updating autorest

* formatting

* updates for SAS

* fixing tests, fixing conn str parser

* formatting

* missing error check

* [Tables] Docs and cleanup (#15428)

* removing to library

* cleanup

* fixing two tests for live

* removing options, responses, and table_pagers files

* removing table_ prefix on files

* removing byte_array_response.go file

* fixing up two reponses, adding proper comments and fixing samples

* removing NewClientFromConnStr

* wrong 'to' library

* [Tables]Rename package (#15448)

<!--
Thank you for contributing to the Azure SDK for Go.

Please verify the following before submitting your PR, thank you!
-->

- [ ] The purpose of this PR is explained in this or a referenced issue.
- [ ] The PR does not update generated files.
   - These files are managed by the codegen framework at [Azure/autorest.go][].
- [ ] Tests are included and/or updated for code changes.
- [ ] Updates to [CHANGELOG.md][] are included.
- [ ] MIT license headers are included in each file.

[Azure/autorest.go]: https://github.com/Azure/autorest.go
[CHANGELOG.md]: https://github.com/Azure/azure-sdk-for-go/blob/main/CHANGELOG.md

* [Tables] Merging internal (#15451)

* updating

* formatting

Co-authored-by: Christopher Scott <[email protected]>
Co-authored-by: catalinaperalta <[email protected]>
Co-authored-by: scbedd <[email protected]>
Co-authored-by: Azure SDK Bot <[email protected]>
Co-authored-by: Heath Stewart <[email protected]>
Co-authored-by: Wes Haggard <[email protected]>
Co-authored-by: tadelesh <[email protected]>
Co-authored-by: JiahuiPeng <[email protected]>
Co-authored-by: Arcturus <[email protected]>
Co-authored-by: Charles Lowell <[email protected]>
Co-authored-by: Richard Park <[email protected]>
jay-most pushed a commit to jay-most/azure-sdk-for-go that referenced this pull request Sep 18, 2021
* Initial generation of the aztables module (Azure#14441)

* initial generation

* Update ci.yml

* Introduce internal/testFramework and test aztables with it (Azure#14491)

* testFramework initial implementation

* sanitizer fixes (Azure#14600)

* Progress on aztables (Azure#14676)

* implement batch operations and all entity operations

* AsModels

* rename

* comments

* comment tweaks, options as pointer

* add EntityMapAsModel

* refactors to AddEntity

* update some code docs

* doc fixes

* doc fixes

* doc improvements

* rename testframework to recording

* move ci.yml

* ci path

* ci path

* [Tables] Update from testframework -> recording (Azure#14888)

* Initial generation of the aztables module (Azure#14441)

* initial generation

* Update ci.yml

* Introduce internal/testFramework and test aztables with it (Azure#14491)

* testFramework initial implementation

* sanitizer fixes (Azure#14600)

* Progress on aztables (Azure#14676)

* implement batch operations and all entity operations

* AsModels

* rename

* comments

* comment tweaks, options as pointer

* add EntityMapAsModel

* refactors to AddEntity

* update some code docs

* doc fixes

* doc fixes

* doc improvements

* rename testframework to recording

* fixing tables to go from testframework -> recording

* renaming recordingInstance -> r

* file name change

Co-authored-by: Christopher Scott <[email protected]>
Co-authored-by: catalinaperalta <[email protected]>

* [Tables] ci fix (Azure#14956)

* [Tables] Add post-request validation of entity (Azure#14960)

* adds a secondary validation if the entity doesnt have pk/rk

* creating an errors file just for handling errors

* removing extra line

* moving error to constants file, renaming constants file

* improving test

* [Tables] changing QueryOptions param to be a ptr (Azure#14961)

* changing QueryOptions param to be a ptr

* feedback from catalina

* [Tables] linting Fixes (Azure#15028)

* small fixes

* small fixes

* small fixes

* small fixes

* small fixes

* small fixes

* adding error checking to transactional batching

* adding error checking to test set up

* adding more error checking

* adding more error checking

* fixing two tests

* fixing error reporting

* fixing error reporting

* fixing error reporting

* fixing last lint failure

* [Tables] Adding swagger.md  (Azure#15016)

* adding swagger

* updating swagger with autorest command

* more fixes

* fixing directive

* updating swagger.md

* fixed swagger

* fixing directive

* add dataplane flag

* updating swagger

* commented out tests, passes remaining

* addressing chris hamons feedback

* undoing changes

* undoing the rest of changes

* updating generated code

* formatting gen code

* more formatting corrections

* one last formatting correction

* [Tables] Add get/set access policy (Azure#15027)

* adding a way to skip cosmos tests, adding access policies

* adding support for getting/setting access policies

* tests working

* does not work for multiple access policies

* adding post validation

* added error validation, note about possible issue

* set policy works

* removing table access policy file

* [Tables] Add GetStats, GetProperties, and SetProperties methods (Azure#15112)

* adding GetProperties

* basic test of set properties

* added logging and metrics tests

* tests for setting service properties

* adding test for retention too long

* adding samples and docs

* [Tables] Add support for Entity struct (Azure#15003)

* adding test for custom entity

* further

* starting access policy

* working for basic entity

* adding to test

* new generated code, removed one more test

* AddEntity now takes []bytes

* adding directive to change entity to []byte, changes to compile and be able to pass

* converting Value map[string]interface{} -> []byte in convenience layer

* paging works the same way now too

* fixing two tests, update complex and basic test entities, need to add type hints for int64/float64 types

* adding test for delete method, changed delete etag to be nil

* working test for merge

* Added Merge test

* fix up test query simple entity

* complex query test

* everything except for batch tests

* basic batch functionality

* fixing batch mixed test

* fixed last batch test

* added separate file for testing batch actions

* fixing query on table service client to take a pointer as well

* working on annotating entities

* fixing recordings

* working with odata annotations

* working test for using odata to annotate entity

* adding options to get

* working with embedded entity now too

* adding to entity, peeling off prints

* converted to a type

* first change to add EdmEntity type

* updated unmarshalling

* removing two files

* fix editLink

* removing sample

* cleaning up

* removing more code

* cleaning up

* formatting

* formatting

* [Tables] change Query to List (Azure#15099)

* adding List methods

* moving helper to zt file

* improving docstrings

* changing all query methods to list

* updating docstrings

* [Tables] Update transactions method (Azure#15125)

* adding test for custom entity

* further

* starting access policy

* working for basic entity

* adding to test

* new generated code, removed one more test

* AddEntity now takes []bytes

* adding directive to change entity to []byte, changes to compile and be able to pass

* converting Value map[string]interface{} -> []byte in convenience layer

* paging works the same way now too

* fixing two tests, update complex and basic test entities, need to add type hints for int64/float64 types

* adding test for delete method, changed delete etag to be nil

* working test for merge

* Added Merge test

* fix up test query simple entity

* complex query test

* everything except for batch tests

* basic batch functionality

* fixing batch mixed test

* fixed last batch test

* added separate file for testing batch actions

* fixing query on table service client to take a pointer as well

* working on annotating entities

* fixing recordings

* working with odata annotations

* working test for using odata to annotate entity

* adding options to get

* working with embedded entity now too

* adding to entity, peeling off prints

* converted to a type

* first change to add EdmEntity type

* updated unmarshalling

* removing two files

* fix editLink

* removing sample

* cleaning up

* removing more code

* cleaning up

* re-record and remove methods

* more cleanup

* formatting

* having issues with cosmos tests

* Adding Table suffix to Create, Delete, and List methods on service client (Azure#15181)

* [Tables] Architect Feedback (Azure#15196)

* architect feedback

* docstrings

* feedback

* fixing eng pipelines

* figuring out pipeline failure

* missing recording

* restore eng changes

* [EngSys] Test proxy (Azure#15101)

* [Tables] Add authentication via SAS credential (Azure#15256)

* changing recording to use test-proxy

* updating client and test proxy

* updated Transport Do to send actual request

* whitespace

* working implementation

* updating internal recording

* now I have a blank recording...

* cleaning up

* more clean up

* transport Do is repeating, proxy is not parsing URL correctly

* small changes

* adding a policy to do the same thing

* adding a second test for proxy and transport

* further progress with Jeff, switching to only using a policy right now

* cleaning up file

* cleaning up

* added more debugging, added two required headers that I was missing

* proxy working on a single test, need to convert all tests to test proxy

* cleaning up

* moved over successfully :)

* more helper functions

* working for a subtest functionality

* fixed up a service client test

* converting more service tests

* converted all service client tests

* converted table client tests

* fixed last two client tests

* entity test

* entity tests

* converting access policy tests

* all passing in record mode

* batch tests

* liveonly for batch tests

* adding a sanitizer and more methods for recording

* adding main method to start and stop the proxy automatically

* adding test proxy step to pipeline

* double dash on version arg

* need a better way to get userful error reporting

* explicitly running test-proxy in the background

* adjust the proxy start and install to be compatible with specific process

* dont run test proxy in background

* didnt remove an import

* header to proxy_test.go

* fixes for pipeline

* added loggin on accident

* use nohup on linux machines

* further simplify nohup usage

* there are no exes on windows

* fixing errors, passing in default env variable

* force background for linux task. may need another iteration to place it within the quotes

* fake aad credential

* clean up after test-proxy

* add print for record mode

* another different attempt at starting the proxy

* jobs apparently is an unrecognized command on our linux boxes

* correct dumb typo in run tests

* replace nuget install with docker

* working fake credentials for client delete entity test

* fixing fake credentials portion, passing locally

* trying an insecureskipverify transport

* debugging statements to find the cwd

* removing blank recording

* working implementation of sas

* working sas credential for account signature

* table sas is failing

* leverage proxy tool, not docker

* remove apostrophes

* moving recordings one directory up

* transitioning to using the sas written by storage team

* using storage methods

* copied storage code, sas needs to be appended in a different way

* changing directory for test-proxy to start from

* removing parenthesis

* adding steps for validating ssl

* switching to docker

* adding start server script

* working implementation if we can fix the prepending of 'recording' to the docker request finder

* big refactor, moving stuff into recording file, cleaning up proxy_test.go file

* last fixings

* convert back to docker. windows and linux images now present

* double condition

* moving configuration to a separate file. preparing for eng/common move

* tier 0 of trust. I don't expect this to work, but it's still worth a shot. maybe the devops agents do something differently

* update condition to use variable syntax

* guess I'm not using variables. removing

* call start-server.ps1

* adjust build-test and configure-proxy to run the docker container in context

* wrap in quotes for the container create. it's apparently a bad

* adjust the initialize call

* disable vet temporarily

* looking for cert file in env variable PROXY_CERT

* set proxy_cert environment variable to find that certificate

* changing to crt file

* use crt cert

* want to see errors

* powershell errors

* correcting how the volume binds to the windows container

* small modification for windows container

* finally got the magic sauce

* cleaning up to remove azcore from internal

* removing recordings that use vcr

* issue with the url creation

* persist query params correctly in azcore.JoinPaths

* removing print statements

* return root

* changing location of script

* forgot the stop command

* working table level sas implementation

* adding start/end rk/pk functionality

* removing print statements

* removing more non-tables code

* docstrings and removing storage only code

* making sas tests live only

* Fixing sas table name to be lowercase always, adding test to verify read only

* Fixing sas table name to be lowercase always, adding test to verify read only

* adding cosmos sas test

* fix for params

* apiview fixes

* fixing test

* undoing change

* updates

* error naming

* fixed service client, was adding percallpolicies twice

* simplification

* aligning with main

* converting from query pager to list pager for naming consistency

* updating objects for consistency

* jeffs feedback

* small fixes

* autorest.md changes

* fixing tablename

Co-authored-by: scbedd <[email protected]>

* fixing internal (Azure#15315)

* Fixes2 (Azure#15316)

Makes a fix to azcore in track2-tables branch

* [Tables] bump deps (Azure#15350)

* Sync eng/common directory with azure-sdk-tools for PR 1912 (Azure#15314)

* Attempt to purge all vaults, managed HSMs

Reverts Azure#1910. Vaults and managed HSMs are automatically purged on their purge date. The point was to purge them daily to preserve capacity. The default purge date is +90 days.

* Add timeout and more logging

* Pass required -Resource

* Fix log message

* Ensure the $Resource is correctly captured

Added comment to new code explaining why, since ScriptBlock.GetNewClosure() is not working as expected.

* Add -ErrorAction to Receive-Job

Worked without terminating when run locally, but failed on the first error in the AzDO agent.

* Use $using:r instead of creating ScriptBlock

More idiomatic for passing ScriptBlocks to jobs.

* Resolve PR feedback

* Change default DeleteAfterHours to 120

Resolves Azure#1917

* Use the Az cmdlets built-in -AsJob

Co-authored-by: Heath Stewart <[email protected]>

* Fix query batch processing (Azure#15319)

Co-authored-by: Wes Haggard <[email protected]>

* fix: three dots path not work in linux in build script (Azure#15297)

* Release v56.3.0 1629350585 (Azure#15323)

* Generated from specification/apimanagement/resource-manager/readme.md tag package-preview-2021-01 (commit hash: 30d1537f0c9aa49e6e04401b1d16f612b31231e7)

* Generated from specification/purview/resource-manager/readme.md tag package-2021-07-01 (commit hash: 30d1537f0c9aa49e6e04401b1d16f612b31231e7)

* v56.3.0

* fix

* Remove old unused update changelog script (Azure#15326)

Co-authored-by: Wes Haggard <[email protected]>

* add clear output flag to build.ps1 (Azure#15255)

* add clear output flag

* remove the path parameter and make sure the process-sdk function only runs in current directory

* change flag name

* [tools/generator] add automation and release command for track2 (Azure#15271)

* [tools/generator] add automation and release command for track2

* add copyright and do gofmt

* Use semver to pump version, move git operation to repo package and fix some typo

* Add specfic version to release cmd, seperate package for automation and remove test config file

* add automation script and config for track2

* fix: three dots path not work in linux in build script

* fix: ln pwsh.exe to pwsh to run in linux

* fix: ignore dep check for go-git

* chore: label name change for track2

* fix: wrong package path result in empty git commit

* fix: repo addr replacement in autorest.md

* chore: move script to eng folder and change name

* fix: onboard generation need to replace NewClientMethod

* chore: change all naming from track2 to v2

* fix: add new build constraint

* fix: template compat with go 1.17 and change commit msg for release

* chore: add v2 to readme begin regex to support future change of `track2` naming

* chore: remove track2 naming

* fix: duplicated and wrong expression

* fix: add package title param to release cmd

* fix: ci

* Prevent ManagedIdentityCredential mutating GetToken arguments (Azure#15331)

* [Azcore] bumps version of internal (Azure#15347)

* bump internal to latest

* upgrading internal

* not the correct one

* changelog update

* [azidentity] bump azcore and internal dependencies (Azure#15348)

* updating to latest core and identity

* trying to fix merge

* undoing changes to identity that should not be there

* more changes

* more changes

* removing a bunch more

* one last file

* formatting

* linting checks

* linting

* formatting

Co-authored-by: Azure SDK Bot <[email protected]>
Co-authored-by: Heath Stewart <[email protected]>
Co-authored-by: Wes Haggard <[email protected]>
Co-authored-by: tadelesh <[email protected]>
Co-authored-by: JiahuiPeng <[email protected]>
Co-authored-by: Arcturus <[email protected]>
Co-authored-by: Charles Lowell <[email protected]>

* [Tables] connection string parser (Azure#15187)

* adding conn str parser

* Basic conn str parser

* adding support for custom domains

* invalid tests

* adding self note

* Update sdk/tables/aztable/connection_string.go

Co-authored-by: Richard Park <[email protected]>

* Update sdk/tables/aztable/connection_string.go

Co-authored-by: Richard Park <[email protected]>

* improving testing

* finishing connection string portion

* formatting and improved error handling

* docstring addition for private method

* connection string error

* splitn instead of split

* sas comment

* updating proxy-server script

* configure proxy with hash of file, correct file location

* using certs in main

* trying download with no password

* trying without the pfx file altogether

* adding pfx portion back

* manual copy of testproxy to common

* using common

* fixing path

* adding trust ps1 script

* adding step to run proxy

* formatting and go mod x/net

* fixing for latest azcore

* fixing rqeuire

* removing eng stuff

* conn str

* running tests

* run tests = true

* adding build sourcesDir

* relative path

* fixing up path

* build-test

* moving byte array response inside table pager

* env var playing to fix ci lint

* changing package name

Co-authored-by: Richard Park <[email protected]>

* [Tables] simplify code space by moving generated to `internal` module (Azure#15299)

* changing to internal package, everything except batching works

* back to working

* converting more things to private

* renamings

* still working, checkpoint

* most of tableclient done

* finished table client options/responses

* finished with service client

* cors rule done

* retentionpolicy

* retentionpolicy and cors

* last piece

* final comments

* updating with newest autorest

* formatting

* fixing go mod in internal package

* update versions

* flipping param order

* convert internal to a subpackage

* fixing telemetry and bumping go version

* updating proxy, adding test-resources.json file

* cert issues

* formatting and x/net

* fixing build-test

* correcting file path

* lint and formatting

* go fmt

* dropping coverage for internal

* updates from pre-arch board

* checking error

* doing scopes work in generated code

* linting

* removing old file

* consistent namings

* removing types and simplifying pageresponse for list entities

* formatting

* renames, removing Table prefix, removing TableTRansactionError

* fixed error type, placeholder while azcore is in PR

* running tests live, fixing one test, converting to always using shared key

* formatting

* linter

* fixing playback for storage

* fixing playback credential

* fixing serviceURL to expect tableName in service url

* moving update mode into options for update/insert entity

* return *Client from CreateTable method

* cleaning up structs

* flipping order of strings in conn str constructor

* renaming param of transactionAction from ETag -> IfMatch

* remove XMS prefix, change Value to Entities/Tables

* moving GetTableSASToken to Client struct

* making MinimalOdata the default, removing option to set it, new recordings

* renaming ETag -> IfMatch for Options

* formatting

* renaming file, adding version to constants.go file

* adding version

* updating to latest azcore

* updating autorest

* formatting

* updates for SAS

* fixing tests, fixing conn str parser

* formatting

* missing error check

* [Tables] Docs and cleanup (Azure#15428)

* removing to library

* cleanup

* fixing two tests for live

* removing options, responses, and table_pagers files

* removing table_ prefix on files

* removing byte_array_response.go file

* fixing up two reponses, adding proper comments and fixing samples

* removing NewClientFromConnStr

* wrong 'to' library

* [Tables]Rename package (Azure#15448)

<!--
Thank you for contributing to the Azure SDK for Go.

Please verify the following before submitting your PR, thank you!
-->

- [ ] The purpose of this PR is explained in this or a referenced issue.
- [ ] The PR does not update generated files.
   - These files are managed by the codegen framework at [Azure/autorest.go][].
- [ ] Tests are included and/or updated for code changes.
- [ ] Updates to [CHANGELOG.md][] are included.
- [ ] MIT license headers are included in each file.

[Azure/autorest.go]: https://github.com/Azure/autorest.go
[CHANGELOG.md]: https://github.com/Azure/azure-sdk-for-go/blob/main/CHANGELOG.md

* [Tables] Merging internal (Azure#15451)

* updating

* formatting

Co-authored-by: Christopher Scott <[email protected]>
Co-authored-by: catalinaperalta <[email protected]>
Co-authored-by: scbedd <[email protected]>
Co-authored-by: Azure SDK Bot <[email protected]>
Co-authored-by: Heath Stewart <[email protected]>
Co-authored-by: Wes Haggard <[email protected]>
Co-authored-by: tadelesh <[email protected]>
Co-authored-by: JiahuiPeng <[email protected]>
Co-authored-by: Arcturus <[email protected]>
Co-authored-by: Charles Lowell <[email protected]>
Co-authored-by: Richard Park <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants