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

Updating docs for copy and sync commands #2440

Merged
merged 7 commits into from
Nov 6, 2023

Conversation

normesta
Copy link
Contributor

Including Gen2 in the list of supported sources and destinations for both copy and sync commands

@normesta
Copy link
Contributor Author

normesta commented Nov 3, 2023

Yikes. Didn't realize that I had this PR still open so I accidentally made another commit to it. Would any previous reviewers, mind reviewing the additional delta here? Thanks!

@gapra-msft
Copy link
Member

@normesta lgtm!

@normesta
Copy link
Contributor Author

normesta commented Nov 3, 2023

THanks @gapra-msft . Curious about the process here. Do we sign off on our own contributions, or is there a designated admin o your team that signs off on these? I'm not sure if your repo uses the same "# sign-off" semantics. Thx!

@gapra-msft
Copy link
Member

@normesta you should be good since you have 2 sign offs on this PR. Please merge this PR at your convenience.

@normesta
Copy link
Contributor Author

normesta commented Nov 6, 2023

Ok Thank you @gapra-msft !

#sign-off

@normesta
Copy link
Contributor Author

normesta commented Nov 6, 2023

#sign-off

@normesta
Copy link
Contributor Author

normesta commented Nov 6, 2023

Hmm. The "# sign-off" semantic indeed does not work the same here. @gapra-msft can you please advise me as to how I can merge this PR? I see no option in the UI of this issue that I can use to do that. I attempted to use the sign-off hash tag that we use in the docs.microsoft collection and that doesn't appear to kick off any sort of merge automation.

@gapra-msft
Copy link
Member

Oh interesting, it might be that you don't have merge permissions on this repo. I can manually merge this for you if that's alright? @normesta

@normesta
Copy link
Contributor Author

normesta commented Nov 6, 2023

Yes, please do merge for me. Thank you! I don't believe I have permission to do that myself. That's probably a good thing :-)

@gapra-msft gapra-msft merged commit 60fc3bd into Azure:dev Nov 6, 2023
9 checks passed
@Daiannec
Copy link

Daiannec commented Dec 7, 2023

c214f6c3-dcc7-4049-5e7e-36af557d17d3.log

Hello team,
I still have error 403 ... someone can help me?
This is test account

2023/12/07 18:53:44 ==> REQUEST/RESPONSE (Try=1/41.5623ms, OpTime=41.5623ms) -- RESPONSE STATUS CODE ERROR
HEAD https://c57d47f6c9e44a79acccc01.blob.core.windows.net/ingestiondata/PST%2Fazcopy.exe?se=2024-01-06T18%3A33%3A56Z&si=IngestionSasForAzCopy202311271809445768&sig=-REDACTED-&sr=c&sv=2015-04-05
Accept: application/xml
User-Agent: AzCopy/10.22.0 azsdk-go-service.Client/v1.2.0 (go1.19.12; Windows_NT)
X-Ms-Client-Request-Id: a7d25825-3d5f-48ff-59ab-d6c8a5e518fc
x-ms-version: 2023-08-03

RESPONSE Status: 403 This request is not authorized to perform this operation using this permission.
Date: Thu, 07 Dec 2023 18:53:43 GMT
Server: Windows-Azure-Blob/1.0 Microsoft-HTTPAPI/2.0
Vary: Origin
X-Ms-Client-Request-Id: a7d25825-3d5f-48ff-59ab-d6c8a5e518fc
X-Ms-Error-Code: AuthorizationPermissionMismatch
X-Ms-Request-Id: 14eff5e5-301e-006e-043e-2939ee000000
X-Ms-Version: 2023-08-03

@gapra-msft
Copy link
Member

@Daiannec please create a new issue with details of what you are experiencing, vs what you expect.

@Daiannec
Copy link

Daiannec commented Dec 7, 2023 via email

@normesta normesta deleted the normesta-updates branch December 11, 2023 19:01
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.

4 participants