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 my current IP address #409

Merged
merged 1 commit into from
Jun 14, 2023
Merged

Conversation

daniel-beck
Copy link
Contributor

This seems impractical IMO.

@daniel-beck daniel-beck requested a review from a team June 14, 2023 09:13
@daniel-beck daniel-beck marked this pull request as draft June 14, 2023 09:14
Copy link
Collaborator

@smerle33 smerle33 left a comment

Choose a reason for hiding this comment

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

🚀

@dduportal
Copy link
Contributor

For info: #410 to solve this

dduportal added a commit that referenced this pull request Jun 14, 2023
…c IPs (#410)

Related to jenkins-infra/helpdesk#3624 (short
fix).

It avoids impractical PRs such as #409 and the subsequent Terraform
rework that would have make a list concatenation between "admin" and
"trusted.ci users" public IPs.

---------

Signed-off-by: Damien Duportal <[email protected]>
Co-authored-by: Hervé Le Meur <[email protected]>
dduportal pushed a commit that referenced this pull request Apr 19, 2024
<Actions>
<action
id="296d75eab55b9d23bd1e94dc34cea43b964c29945c12fefcb674e3c068a0a767">
        <h3>Bump Terraform `random` provider version</h3>
<details
id="1d9343c012f5434ac9fe8a98135bae3667b399259be16d9b14302ea3bd424a24">
            <summary>Update Terraform lock file</summary>
<p>changes detected:&#xA;&#x9;&#34;hashicorp/random&#34; updated from
&#34;3.6.0&#34; to &#34;3.6.1&#34; in file
&#34;.terraform.lock.hcl&#34;</p>
            <details>
                <summary>3.6.1</summary>
<pre>Changelog retrieved
from:&#xA;&#x9;https://github.com/hashicorp/terraform-provider-random/releases/tag/v3.6.1&#xA;BUG
FIXES:&#xA;&#xA;* all: Prevent `keepers` from triggering an in-place
update following import
([#385](hashicorp/terraform-provider-random#385
resource/random_shuffle: Prevent inconsistent result after apply when
result_count is set to 0
([#409](hashicorp/terraform-provider-random#409
provider/random_password: Fix bug which causes panic when special,
upper, lower and number/numeric are all false
([#551](hashicorp/terraform-provider-random#551
provider/random_string: Fix bug which causes panic when special, upper,
lower and number/numeric are all false
([#551](https://github.com/hashicorp/terraform-provider-random/issues/551))&#xA;&#xA;&#xA;</pre>
            </details>
        </details>
<a
href="https://infra.ci.jenkins.io/job/updatecli/job/azure/job/main/118/">Jenkins
pipeline link</a>
    </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.

3 participants