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

TES-305 DNS Module #11

Merged
merged 8 commits into from
Dec 1, 2023
Merged

TES-305 DNS Module #11

merged 8 commits into from
Dec 1, 2023

Conversation

kristianiliev1
Copy link
Contributor

@kristianiliev1 kristianiliev1 commented Nov 15, 2023

Description

Introduced new Private DNS Module

Related Issues

TES-305

Changes

Added Private DNS Module
Added script, which creates DNS records based on the attached disk
Changed disk naming to "Disk-$${RESOURCE_GROUP}-$${ZONE_ID}-$${DISK_ORDER}"
Fixed an issue where the attach disk script was failing

Checklist

  • I have tested these changes thoroughly.
  • My code follows the project's coding style.
  • I have added appropriate comments to my code, especially in complex areas.
  • All new and existing tests passed locally.

@kristianiliev1 kristianiliev1 self-assigned this Nov 15, 2023
@viktor-ribchev viktor-ribchev changed the title Tes 305 dns module TES-305 DNS Module Nov 17, 2023
modules/load_balancer/main.tf Outdated Show resolved Hide resolved
variables.tf Outdated Show resolved Hide resolved
variables.tf Outdated Show resolved Hide resolved
modules/dns/variables.tf Outdated Show resolved Hide resolved
modules/dns/variables.tf Outdated Show resolved Hide resolved
modules/dns/main.tf Outdated Show resolved Hide resolved
main.tf Outdated Show resolved Hide resolved
main.tf Outdated Show resolved Hide resolved
main.tf Show resolved Hide resolved
main.tf Outdated Show resolved Hide resolved
main.tf Outdated Show resolved Hide resolved
modules/dns/variables.tf Outdated Show resolved Hide resolved
modules/dns/variables.tf Outdated Show resolved Hide resolved
modules/dns/main.tf Outdated Show resolved Hide resolved
variables.tf Outdated Show resolved Hide resolved
modules/dns/versions.tf Outdated Show resolved Hide resolved
@viktor-ribchev
Copy link
Contributor

Rebase and remove data sources in favor of variables.

@kristianiliev1 kristianiliev1 marked this pull request as ready for review November 22, 2023 12:22
@viktor-ribchev viktor-ribchev force-pushed the TES-305-DNS-Module branch 2 times, most recently from 02bd874 to 39b5cab Compare November 27, 2023 13:25
.gitignore Outdated Show resolved Hide resolved
modules/dns/versions.tf Outdated Show resolved Hide resolved
kristianiliev1 and others added 5 commits November 29, 2023 15:49
Initial commit for DNS module - TES-305

fix formating

update half of the comments from the pull request review

fix A records for instances

changes from comments and fix records

TES-305

add changes from comment suggestions

remove dns auto register

TES-305

add DNS module

Initial commit for DNS module - TES-305

fix formating

update half of the comments from the pull request review

fix A records for instances

changes from comments and fix records

TES-305

add changes from comment suggestions
Added additional variables
Added script, which creates DNS records based on the attached disk
Changed disk naming to "Disk-$${RESOURCE_GROUP}-$${ZONE_ID}-$${DISK_ORDER}"
Fixed an issue where the attach disk script was failing
… of Disk name

Updated .gitignore
removed versions.tf for DNS module

Moved the DNS role assignment to Roles module
modules/dns/variables.tf Outdated Show resolved Hide resolved
modules/vm/templates/entrypoint.sh.tpl Outdated Show resolved Hide resolved
modules/vm/templates/entrypoint.sh.tpl Outdated Show resolved Hide resolved
@viktor-ribchev viktor-ribchev merged commit 21f04c2 into main Dec 1, 2023
1 check passed
@viktor-ribchev viktor-ribchev deleted the TES-305-DNS-Module branch December 1, 2023 08:13
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.

4 participants