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 a linux VM as a jumpbox to Bastion SSH into the Hub network #219

Merged
merged 4 commits into from
May 24, 2021

Conversation

glennmusa
Copy link
Contributor

Description

Adds a Linux virtual machine that defaults to Ubuntu 18.04 LTS to use as a jumpbox from a Bastion host in the Hub network.

Issue reference

The issue this PR will close: #188

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles or validates correctly
  • BASH scripts have been validated using shellcheck
  • All tests pass (manual and automated)
  • The documentation is updated to cover any new or changed features
  • Markdown files have been linted using the recommended linter. (See .vscode/extensions.json.)
  • Relevant issues are linked to this PR

@glennmusa glennmusa changed the title add a linux virtual machine as a hub network jumpbox add a linux VM as a jumpbox to Bastion SSH into the Hub network May 21, 2021
@glennmusa glennmusa enabled auto-merge (squash) May 21, 2021 23:37
Copy link
Contributor

@brooke-hamilton brooke-hamilton left a comment

Choose a reason for hiding this comment

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

@glennmusa Just a question: why do the names of the Bastion instance and the VMs (and related resources) have a default prefix of "mlzDemo"? Is it because we expect users to modify the Bastion and jump box resources before using them?

@glennmusa glennmusa merged commit 00f91d8 into main May 24, 2021
@glennmusa glennmusa deleted the glennmusa/linuxjumpbox branch May 24, 2021 13:09
@glennmusa
Copy link
Contributor Author

glennmusa commented May 24, 2021

@glennmusa Just a question: why do the names of the Bastion instance and the VMs (and related resources) have a default prefix of "mlzDemo"? Is it because we expect users to modify the Bastion and jump box resources before using them?

@brooke-hamilton I don't think there was ever consensus or intentional design around default .tfvars values. "mlzDemo" seems to have emerged as a naming convention.

I want to say that for the most part the Terraform in the solution has treated most configurable settings as input via Terraform .tfvars files, but I don't see why there isn't a future where the configuration(s) could be made up of calculated values: A) the input via .tfvars files with B) calculated values that include environment variables like mlz_env_name.

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.

Provision a Linux VM as a Bastion jumpbox
2 participants