8000 Implement azure source by gerardsegarra · Pull Request #517 · hashicorp/go-getter · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Implement azure source #517

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

Draft
wants to merge 4 commits into
base: main
Choose a base branch
8000
from

Conversation

gerardsegarra
Copy link

WIP implement azure as a source module.

Copy link
hashicorp-cla-app bot commented Feb 12, 2025

CLA assistant check
All committers have signed the CLA.

Copy link

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes

Have you signed the CLA already but the status is still pending? Recheck it.

@crw
Copy link
Contributor
crw commented Feb 13, 2025

Related to #222?

@gerardsegarra
Copy link
Author

Related to #222?

That's true. To be honest I didn't see it, but the PR is like 4 years old. An on the other hand, in this PR #395 manicminer commented that we should use giovanni's library, which I'm trying to use.

How would you suggest for me to continue? Should I contribute somehow to #222? I think it's not using giovanni's either.

Thanks!

@crw
Copy link
Contributor
crw commented Feb 18, 2025

Hi @gerardsegarra, I'd say no, keep this PR open. If we review one, we can close both of them. I am not sure why the previous did not get fully reviewed and included, other than it being a low priority for the Azure Provider team (at HashiCorp). It can be hard to get changes through in go-getter.

@tony-harverson-moonpig
Copy link

Just for tracking, it's probably worth linking this to the original issue #33, that this is intended to solve.

@tony-harverson-moonpig
Copy link

@gerardsegarra How are you getting on with this? Is there anything you could use help with to help get this over the line?

@gerardsegarra
Copy link
Author

hey @tony-harverson-moonpig , I haven't had time to continue on my free time so far. I got stuck with the testing as if you look at the other cloud modules, they use private keys to run integration tests with private resources.

Maybe @crw has a proposal on how we can do that? Maybe the team from hashicrop could collaborate creating the internal testing?

@crw
Copy link
Contributor
crw commented Mar 25, 2025

Apologies I do not have a suggestion. I am going to see if I can find someone who can decide on this PR. As I mentioned earlier, it can be hard to get changes in go-getter.

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

Successfully merging this pull request may close these issues.

3 participants
0