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

feat: add allow_replace to server resource #712

Merged
merged 8 commits into from
Nov 21, 2024

Conversation

cristiGuranIonos
Copy link
Collaborator

What does this fix or implement?

Checklist

  • PR name added as appropriate (e.g. feat:/fix:/doc:/test:/refactor:)
  • Tests added or updated
  • Documentation updated
  • Changelog updated and version incremented (label: upcoming release)
  • Github Issue linked if any
  • Jira task updated

@cristiGuranIonos cristiGuranIonos marked this pull request as draft November 20, 2024 10:09
@cristiGuranIonos cristiGuranIonos marked this pull request as ready for review November 20, 2024 11:43
adeatcu-ionos
adeatcu-ionos previously approved these changes Nov 20, 2024
@@ -198,7 +200,7 @@
- Fix documentation for `ionoscloud_server`, `ionoscloud_volume`, `ionoscloud_lan` resources and `ionoscloud_image` data sources

### Enhancements
- Add configurable fields to `ionoscloud_share` resource. Fields that can be set on creation: `description`, `sec_auth_protection`, `licence_type`.
- Add configurable fields to `ionoscloud_share` resource. Fields that can be set on creation: `description`, `sec_auth_protection`, `licence_type`.
Copy link
Contributor

Choose a reason for hiding this comment

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

@cristiGuranIonos do you use some kind of formatting tool for this file? I'm just curious where these changes are coming from.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

no idea how they got added back. Maybe on merge in some way. Will leave as is, it does not matter.

- Split tests with build tags
- Improve http client performance and timeouts

### Documentations:
### Documentations:
Copy link
Contributor

Choose a reason for hiding this comment

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

@cristiGuranIonos these look like some new white spaces, not sure if you actually wanted to add these. Anyway, it doesn't matter.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

a previous commit had removed these. Not sure how they got added back. Will take a look

Co-authored-by: Deatcu Andrei-Cristian <[email protected]>
@cristiGuranIonos cristiGuranIonos merged commit 3354c9d into master Nov 21, 2024
7 checks passed
@cristiGuranIonos cristiGuranIonos deleted the feat/add_allow_replace_to_server branch November 21, 2024 09:33
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.

2 participants