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

fix: allow deleting reverse DNS entries #814

Closed
phm07 opened this issue Jul 11, 2024 · 0 comments · Fixed by #839 or #842
Closed

fix: allow deleting reverse DNS entries #814

phm07 opened this issue Jul 11, 2024 · 0 comments · Fixed by #839 or #842
Labels

Comments

@phm07
Copy link
Contributor

phm07 commented Jul 11, 2024

At the moment, set-rdns only allow adding reverse DNS entries, but they don't allow deleting them. This should be made possible because the API supports it.

@phm07 phm07 added the bug label Jul 11, 2024
phm07 added a commit that referenced this issue Aug 14, 2024
This PR adds a `--reset` flag to the `set-rdns` commands which allows
passing a null value as the hostname. This resets the reverse DNS entry
to the original entry for the IP.

Fixes #814
phm07 pushed a commit that referenced this issue Aug 30, 2024
🤖 I have created a release *beep* *boop*
---


##
[1.47.0](v1.46.0...v1.47.0)
(2024-08-30)


### Features

* allow deleting rDNS entries
([#839](#839))
([129f51d](129f51d)),
closes [#814](#814)
* **certificate:** allow retrying managed certificate issuance
([#847](#847))
([0223f7d](0223f7d))
* improve datacenter describe server types readability
([#854](#854))
([fc0002c](fc0002c)),
closes [#852](#852)


### Bug Fixes

* `quiet` option sometimes only working as a flag
([#846](#846))
([f7c9ac6](f7c9ac6))
* **image:** only accept numerical ids for update, delete, label
([#844](#844))
([7355571](7355571))
* **load-balancer:** show message if none of --server, --label-selector
or --ip is set ([#849](#849))
([2df45e0](2df45e0))
* **primary-ip:** change protection commands do not allow protection
levels ([#851](#851))
([697a87a](697a87a))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
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 a pull request may close this issue.

1 participant