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

Get rid of advanced Docker image and change cloud deployments to use ENABLE_PRIMITIVE=false #428

Merged
merged 26 commits into from
Mar 25, 2022

Conversation

Amet13
Copy link
Contributor

@Amet13 Amet13 commented Mar 25, 2022

Description

  • Fix kubeval bug with linting all yamls in repo
  • Cleanup all mentions for advanced image from repo
  • Optimize k8s and terraform deployments to use ENABLE_PRIMITIVE=false

Fixes #421

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update

How Has This Been Tested?

$ helm upgrade --install \
    --create-namespace \
    --namespace=db1000n \
    -f values.yaml db1000n .
Release "db1000n" has been upgraded. Happy Helming!
NAME: db1000n
LAST DEPLOYED: Fri Mar 25 15:28:29 2022
NAMESPACE: db1000n
STATUS: deployed
REVISION: 3
TEST SUITE: None

$ kg po
NAME                       READY   STATUS    RESTARTS   AGE
db1000n-77b8f49569-j92q8   1/1     Running   0          24s

$ k apply -f daemonset.yaml 
namespace/db1000n unchanged
daemonset.apps/db1000n created
$ k apply -f deployment.yaml 
namespace/db1000n unchanged
deployment.apps/db1000n created

@Amet13 Amet13 requested a review from arriven as a code owner March 25, 2022 13:51
ansible/linux/setup.yaml Show resolved Hide resolved
.github/workflows/release-docker-advanced.yaml Outdated Show resolved Hide resolved
@Amet13 Amet13 requested a review from arriven March 25, 2022 14:09
Dockerfile Outdated Show resolved Hide resolved
@Amet13 Amet13 requested a review from arriven March 25, 2022 14:21
@arriven arriven merged commit 9798c1e into arriven:main Mar 25, 2022
@neatdecisions neatdecisions mentioned this pull request Mar 25, 2022
4 tasks
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.

Change cloud guides and deployments to use regular image with ENABLE_PRIMITIVE=false
2 participants