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

Feature/kubernetes 1.18 #129

Merged
merged 14 commits into from
Dec 28, 2020
Merged

Conversation

gfrntz
Copy link
Contributor

@gfrntz gfrntz commented Dec 24, 2020

[examples/argocd-with-applications]

  • terraform 0.14

[modules/cicd/argo/modules/cd]

  • terraform-aws-modules/iam/aws//modules/iam-assumable-role-with-oidc v.3.6.0
  • chart_version 2.11.0

[modules/cicd/jenkins]

  • change chart repo url

[modules/ingress/nginx]

  • add aws lb tags

[modules/ingress/oauth]

  • change chart repo url

[modules/kubernetes]

  • terraform-aws-modules/eks/aws v13.2.1
  • temporary disable enable_irsa option
  • add cluster_oidc_arn and cluster_output outputs
  • set 1.18 as default cluster version

[modules/logging/efk]

  • change repo url

[modules/network]

  • terraform-aws-modules/vpc/aws v2.64.0

[modules/scaling/cluster-autoscaler]

  • terraform-aws-modules/iam/aws//modules/iam-assumable-role-with-oidc v3.6.0
  • change repo url

[modules/system/external-dns]

  • terraform-aws-modules/iam/aws//modules/iam-assumable-role-with-oidc v3.6.0

[modules/system/external-secrets]

  • Fix template
  • Add namespace creation
  • Change cluster_output type

Remove kubeflow module

Test

  • Install rvm
curl -sSL https://get.rvm.io | bash -s stable
  • Install ruby

  • Run bundle install

  • Run terraform plan && terraform apply

  • Run AWS_PROFILE=YOUR_PROFILE rake spec

* terraform 0.14

[modules/cicd/argo/modules/cd]
* terraform-aws-modules/iam/aws//modules/iam-assumable-role-with-oidc v.3.6.0
* chart_version 2.11.0

[modules/cicd/jenkins]
* change chart repo url

[modules/ingress/nginx]
* add aws lb tags

[modules/ingress/oauth]
* change chart repo url

[modules/kubernetes]
* terraform-aws-modules/eks/aws v13.2.1
* temporary disable enable_irsa option
* add cluster_oidc_arn and cluster_output outputs
* set 1.18 as default cluster version

[modules/logging/efk]
* change repo url

[modules/network]
* terraform-aws-modules/vpc/aws v2.64.0

[modules/scaling/cluster-autoscaler]
* terraform-aws-modules/iam/aws//modules/iam-assumable-role-with-oidc v3.6.0
* change repo url

[modules/system/external-dns]
* terraform-aws-modules/iam/aws//modules/iam-assumable-role-with-oidc v3.6.0

[modules/system/external-secrets]
* Fix template
* Add namespace creation
* Change cluster_output type

Signed-off-by: Gleb Rusakov <[email protected]>
gfrntz and others added 5 commits December 24, 2020 21:05
* terraform 0.14

[modules/cicd/argo/modules/cd]
* terraform-aws-modules/iam/aws//modules/iam-assumable-role-with-oidc v.3.6.0
* chart_version 2.11.0

[modules/cicd/jenkins]
* change chart repo url

[modules/ingress/nginx]
* add aws lb tags

[modules/ingress/oauth]
* change chart repo url

[modules/kubernetes]
* terraform-aws-modules/eks/aws v13.2.1
* temporary disable enable_irsa option
* add cluster_oidc_arn and cluster_output outputs
* set 1.18 as default cluster version

[modules/logging/efk]
* change repo url

[modules/network]
* terraform-aws-modules/vpc/aws v2.64.0

[modules/scaling/cluster-autoscaler]
* terraform-aws-modules/iam/aws//modules/iam-assumable-role-with-oidc v3.6.0
* change repo url

[modules/system/external-dns]
* terraform-aws-modules/iam/aws//modules/iam-assumable-role-with-oidc v3.6.0

[modules/system/external-secrets]
* Fix template
* Add namespace creation
* Change cluster_output type

Update README, delete kubeflow module, add default variables

Fix reviewdog terraform version

Fix jenkinsfile

Add awspec tests for argocd-with-applications
@gfrntz gfrntz changed the title WIP: Feature/kubernetes 1.18 Feature/kubernetes 1.18 Dec 25, 2020
akastav
akastav previously approved these changes Dec 28, 2020
Copy link
Contributor

@akastav akastav left a comment

Choose a reason for hiding this comment

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

LGTM

RustamGimadiev
RustamGimadiev previously approved these changes Dec 28, 2020
@akastav akastav dismissed stale reviews from RustamGimadiev and themself via 9592eb0 December 28, 2020 08:19
@akastav akastav merged commit ca62650 into provectus:master Dec 28, 2020
@akastav akastav added the enhancement New feature or request label Dec 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants