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

Add 'Injecting credentials' section to provider docs #318

Merged
merged 2 commits into from
Sep 10, 2023

Conversation

rdarcy1
Copy link
Contributor

@rdarcy1 rdarcy1 commented Jun 30, 2023

Closes #196

Like the author of the above issue, I also spent some time looking for a solution to provide credentials to this provider so have documented the options based on cyrilgdn's comment.

Despite some of this being built-in Terraform functionality, I (and I imagine many others) have mainly used cloud providers such as AWS, where there's only a need to provide a couple of well-documented environment variables. It's much more likely with the PostgreSQL provider that you'll want to dynamically reference a data source or resource (e.g. an RDS instance that's created in another module) so I feel it's valuable to document examples of this.

Open to edits/suggestions if there are better ways of expressing this.

Copy link
Owner

@cyrilgdn cyrilgdn left a comment

Choose a reason for hiding this comment

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

Thanks a lot 🙏

@cyrilgdn cyrilgdn merged commit e7499ce into cyrilgdn:master Sep 10, 2023
6 checks passed
@rdarcy1 rdarcy1 deleted the patch-1 branch September 10, 2023 21:30
dduportal referenced this pull request in jenkins-infra/azure Sep 23, 2023
<Actions>
<action
id="16c2f665ec8f38b263f8d1f01847791b753aaf4c15655cd80097752f2ecae989">
        <h3>Bump Terraform `postgresql` provider version</h3>
<details
id="1d9343c012f5434ac9fe8a98135bae3667b399259be16d9b14302ea3bd424a24">
            <summary>Update Terraform lock file</summary>
<p>&#34;cyrilgdn/postgresql&#34; updated from &#34;1.21.0&#34; to
&#34;1.21.0&#34; in file &#34;.terraform.lock.hcl&#34;</p>
            <details>
                <summary>1.21.0</summary>
<pre>Changelog retrieved
from:&#xA;&#x9;https://github.com/cyrilgdn/terraform-provider-postgresql/releases/tag/v1.21.0&#xA;##
What&#39;s Changed&#xD;&#xA;* Make function security, strictness,
volatility, and parallel safety configurable by @tommyzli in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/307&#xD;&#xA;*
postgresql_publication.markdown - table ordering by @RoseateSpoonbill in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/265&#xD;&#xA;*
Bump gocloud dependency to fix aws CA validation by @toadjaune in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/340&#xD;&#xA;*
feat: Allow for inlining client cert by @vr in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/342&#xD;&#xA;*
Update postgresql_extension.html.markdown by @tspearconquest in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/337&#xD;&#xA;*
Use uint32 for OID to stop conversion errors with pguint32 by @mat285 in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/324&#xD;&#xA;*
feat: Postgresql 15 support by @cyrilgdn in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/348&#xD;&#xA;*
Added MS Azure passwordless authentication by @librucha in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/346&#xD;&#xA;*
Add &#39;Injecting credentials&#39; section to provider docs by @rdarcy1
in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/318&#xD;&#xA;*
fixed escaping for postgresql user info by @librucha in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/344&#xD;&#xA;&#xD;&#xA;##
New Contributors&#xD;&#xA;* @tommyzli made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/307&#xD;&#xA;*
@RoseateSpoonbill made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/265&#xD;&#xA;*
@toadjaune made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/340&#xD;&#xA;*
@vr made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/342&#xD;&#xA;*
@tspearconquest made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/337&#xD;&#xA;*
@mat285 made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/324&#xD;&#xA;*
@librucha made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/346&#xD;&#xA;*
@rdarcy1 made their first contribution in
https://github.com/cyrilgdn/terraform-provider-postgresql/pull/318&#xD;&#xA;&#xD;&#xA;**Full
Changelog**:
https://github.com/cyrilgdn/terraform-provider-postgresql/compare/v1.20.0...v1.21.0</pre>
            </details>
        </details>
    </action>
</Actions>

---

<table>
  <tr>
    <td width="77">
<img src="https://www.updatecli.io/images/updatecli.png" alt="Updatecli
logo" width="50" height="50">
    </td>
    <td>
      <p>
Created automatically by <a
href="https://www.updatecli.io/">Updatecli</a>
      </p>
      <details><summary>Options:</summary>
        <br />
<p>Most of Updatecli configuration is done via <a
href="https://www.updatecli.io/docs/prologue/quick-start/">its
manifest(s)</a>.</p>
        <ul>
<li>If you close this pull request, Updatecli will automatically reopen
it, the next time it runs.</li>
<li>If you close this pull request and delete the base branch, Updatecli
will automatically recreate it, erasing all previous commits made.</li>
        </ul>
        <p>
Feel free to report any issues at <a
href="https://github.com/updatecli/updatecli/issues">github.com/updatecli/updatecli</a>.<br
/>
If you find this tool useful, do not hesitate to star <a
href="https://github.com/updatecli/updatecli/stargazers">our GitHub
repository</a> as a sign of appreciation, and/or to tell us directly on
our <a
href="https://matrix.to/#/#Updatecli_community:gitter.im">chat</a>!
        </p>
      </details>
    </td>
  </tr>
</table>

Co-authored-by: Jenkins Infra Bot (updatecli) <[email protected]>
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.

Document support for authenticating using environment variables
2 participants