Skip to content
This repository was archived by the owner on Feb 5, 2020. It is now read-only.

Add govcloud support#2608

Merged
enxebre merged 1 commit intocoreos:masterfrom
enxebre:govcloud-m
Jan 4, 2018
Merged

Add govcloud support#2608
enxebre merged 1 commit intocoreos:masterfrom
enxebre:govcloud-m

Conversation

@enxebre
Copy link
Contributor

@enxebre enxebre commented Dec 20, 2017

This introduces support for aws govcloud:

  • Duplication has been generally chosen over abstraction to keep it as simple as possible and being non intrusive with conventional aws platform for now.
  • As a follow up, both internal aws and govcloud could use the same bastion infrastructure created here contrib/govcloud
  • Jenkins credentials need to be setup

@coreosbot
Copy link

Can one of the admins verify this patch?

@cpanato
Copy link
Contributor

cpanato commented Dec 21, 2017

ok to test

@enxebre enxebre force-pushed the govcloud-m branch 3 times, most recently from 93a10fb to 6c6f8a6 Compare December 22, 2017 12:39
@cpanato cpanato closed this Dec 22, 2017
@cpanato cpanato reopened this Dec 22, 2017
@coreosbot
Copy link

Can one of the admins verify this patch?

@cpanato cpanato force-pushed the govcloud-m branch 2 times, most recently from f32ff88 to 1055761 Compare December 22, 2017 14:44
description = "See tectonic_self_hosted_etcd in config.tf"
}

variable "tectonic_vanilla_k8s" {
Copy link
Contributor

Choose a reason for hiding this comment

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

Will need to remove these once your other PR lands.

@cpanato
Copy link
Contributor

cpanato commented Dec 22, 2017

retest this please

@cpanato
Copy link
Contributor

cpanato commented Dec 22, 2017

retest this please

@enxebre
Copy link
Contributor Author

enxebre commented Jan 3, 2018

retest this please

@enxebre
Copy link
Contributor Author

enxebre commented Jan 3, 2018

retest this please

1 similar comment
@enxebre
Copy link
Contributor Author

enxebre commented Jan 3, 2018

retest this please

Copy link
Contributor

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

lgtm

@cpanato
Copy link
Contributor

cpanato commented Jan 3, 2018

waiting for @sym3tri / @alexsomesan

@spangenberg spangenberg self-requested a review January 4, 2018 14:20
Copy link
Contributor

@spangenberg spangenberg left a comment

Choose a reason for hiding this comment

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

LGTM with hint to make bastion instance type configurable in the near future.

@sym3tri
Copy link
Contributor

sym3tri commented Jan 4, 2018

ship it

@enxebre enxebre merged commit 26fd55f into coreos:master Jan 4, 2018
@sym3tri
Copy link
Contributor

sym3tri commented Jan 4, 2018

@enxebre don't forget to cherry-pick into track-1

@enxebre
Copy link
Contributor Author

enxebre commented Jan 4, 2018

track-1 PR #2660

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants