tests: run tests on a running cluster #2735

cpanato · 2018-01-16T14:26:05Z

fixes: https://jira.coreos.com/browse/INST-520

coreosbot · 2018-01-16T14:26:07Z

Can one of the admins verify this patch?

squat

Two small changes. Otherwise looks good

squat · 2018-01-16T15:42:52Z

tests/smoke/azure/README.md

@@ -33,3 +33,10 @@ To run the entire test suite for Azure, use this command:
 export TEST_VARS=vars/Azure-exp.tfvars
 make tests/smoke TEST='azure_*'
 ```
+
+
+## Running the rspec tests in a existing cluster


This should either go in all the READMEs or in a general one

yes, wrong place. my bad

squat · 2018-01-16T15:44:27Z

tests/rspec/lib/shared_examples/k8s.rb

@@ -24,20 +25,29 @@
  include_examples('withRunningClusterExistingBuildFolder', vpn_tunnel)
 end

-RSpec.shared_examples 'withRunningClusterExistingBuildFolder' do |vpn_tunnel = false|
+RSpec.shared_examples 'withRunningClusterExistingBuildFolder' do |vpn_tunnel = false, exist_tf = nil, exist_plat = nil|


can we make the order of exist_tf and exist_plat the same here and for ClusterFactory.from_variable

mxinden · 2018-01-17T09:26:55Z

tests/rspec/lib/cluster.rb

+    end
+  rescue Timeout::Error
+    forensic(false)
+    raise 'Applying cluster failed'


s/Applying cluster failed/terraform init failed/

done, thanks!

mxinden · 2018-01-17T09:35:02Z

tests/rspec/lib/shared_examples/k8s.rb

@@ -145,6 +155,7 @@
      # remove platform AZURE when the JIRA https://jira.prod.coreos.systems/browse/INST-619 is fixed
      skip_platform = %w[metal azure gcp]
      skip "This test is not ready to run in #{platform}" if skip_platform.include?(platform)
+      skip 'Skipping this tests. running locally' unless Jenkins.environment?


Why are you skipping this test, when it is run locally?

because this is the scale up, and if you run for example 3-4 times or more it will scale up your cluster a lot. also takes some time and the developer maybe need just a fast feedback loop.

the scale-up tests are used for a single test or in CI. not for local tests and debug.
if the dev really need this he can comment out this part

mxinden · 2018-01-17T09:39:22Z

tests/rspec/spec/existing_cluster_spec.rb

+
+RSpec.describe 'existing_cluster' do
+  before(:all) do
+    raise 'Missing existing platform. Please set PLATFORM environment variable' unless EnvVar.set?(%w[PLATFORM])


Great, I like the "fail fast" approach!
Can you do the same with the CLUSTER variable?

mxinden · 2018-01-18T08:37:18Z

@cpanato One general idea: The console password and username are both saved inside a Kubernetes secret in the tectonic-system namespace. How about reading it from there instead of saving it into a .json file locally and reading it from there.

This would make the process more robust, because users don't need to worry about the environment variables. What do you think?

cpanato · 2018-01-18T13:35:54Z

@mxinden this definitely is a good idea, however, the password is a hash and I cannot decode that
for example, this is the configmap looks like:

<....>
 staticPasswords:
    - email: "zora_fisher@zieme.biz"
      hash: "$2a$12$msOJPEYv.agnG6Yf6FYmYeKS7zIy93ft0oqqxiG5bX6JfKXzTvNxK"
      username: "admin"
      userID: "Yqo-0IiDywGQJzkghXaGKw"
<....>

mxinden · 2018-01-22T10:02:05Z

tests/rspec/lib/shared_examples/k8s.rb

@@ -145,6 +155,7 @@
      # remove platform AZURE when the JIRA https://jira.prod.coreos.systems/browse/INST-619 is fixed
      skip_platform = %w[metal azure gcp]
      skip "This test is not ready to run in #{platform}" if skip_platform.include?(platform)
+      skip 'Skipping this tests. running locally' unless Jenkins.environment?


* module/update-payload: Include TNO and its appversion into the payload. A follow up PR to the release automation repo is required to fully update the payload. * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * bazel: fix terraform-examples * bazel: refactor examples and docs This commit refactors the Bazel rules for examples and Documentation to accomplish several goals: 1. Openstack artifacts are now correctly compiled; 2. build rules for examples are in the examples directory and build rules for Documentation are in the Documentation directory. The last point allows us to easily compile all examples with: ```sh bazel build examples:all ``` Similarly, we can compile all Documentation by issuing: ```sh bazel build Documentation:all ``` * Update tectonic-cluo-operator to v0.3.0 * Update CLUO to v0.5.0 * Add support for Kubernetes v1.9 * config.tf: Update etcd version from 3.1.8 to 3.2.14 (#2738) * bazel: create tarball target (#2750) This commit adds tarball targets for Bazel. We can now create tarballs using: ```sh bazel build tarball ``` This will create a gzip compressed tarball named tectonic.tar.gz. The contents of the tarball will be nested in a directory named `tectonic`. To create a tarball with a version string in the directory name, we can build the tarball using: ```sh export VERSION=<some version> bazel build tarball --action_env=VERSION ``` In this case, the file structure will be nested in a directory called `tectonic_<some version>`. This commit also makes the installer binaries pure go so that they are statically linked, which is nicer for containers. * *: Remove option to deploy etcd without TLS (#2763) * frontend: Add space-before-function-paren eslint rule Makes space character before function parameters consistent. * tests: run tests on a running cluster (#2735) * Add terraform fmt test rule to bazel * tests: collect machine console logs (aws) (#2754) * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition

* module/update-payload: Include TNO and its appversion into the payload. A follow up PR to the release automation repo is required to fully update the payload. * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * bazel: fix terraform-examples * bazel: refactor examples and docs This commit refactors the Bazel rules for examples and Documentation to accomplish several goals: 1. Openstack artifacts are now correctly compiled; 2. build rules for examples are in the examples directory and build rules for Documentation are in the Documentation directory. The last point allows us to easily compile all examples with: ```sh bazel build examples:all ``` Similarly, we can compile all Documentation by issuing: ```sh bazel build Documentation:all ``` * Update tectonic-cluo-operator to v0.3.0 * Update CLUO to v0.5.0 * Add support for Kubernetes v1.9 * config.tf: Update etcd version from 3.1.8 to 3.2.14 (#2738) * bazel: create tarball target (#2750) This commit adds tarball targets for Bazel. We can now create tarballs using: ```sh bazel build tarball ``` This will create a gzip compressed tarball named tectonic.tar.gz. The contents of the tarball will be nested in a directory named `tectonic`. To create a tarball with a version string in the directory name, we can build the tarball using: ```sh export VERSION=<some version> bazel build tarball --action_env=VERSION ``` In this case, the file structure will be nested in a directory called `tectonic_<some version>`. This commit also makes the installer binaries pure go so that they are statically linked, which is nicer for containers. * *: Remove option to deploy etcd without TLS (#2763) * frontend: Add space-before-function-paren eslint rule Makes space character before function parameters consistent. * tests: run tests on a running cluster (#2735) * frontend: Rename REMOVE_AT action to REMOVE_FIELD_LIST_ROW and simplify This Redux action is now specifically for removing a row from a field list, so rename it to make that clear. Simplify the reducers by assuming they take a field list id and an index, not an arbitrary path. Also stop clearing any associated error data, since a REMOVE_FIELD_LIST_ROW action will need to trigger a re-validation of the field list anyway. * frontend: Fix 2 htmlFor attributes * platforms/govcloud: add s3 support include in CL alpha * Add terraform fmt test rule to bazel * frontend: Only skip AWS VPC validation for 2 fields This fixes a bug where the "Private Route 53 Zones must use an existing private VPC." error was not cleared when switching to a valid hosted zone. This approach should be less error prone and easier to maintain. * tests: collect machine console logs (aws) (#2754) * tests: add max log file size (1GB) feature * add initial aws support for bootstrapping through ncg (#2758) * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition * Ut2 integration (#2800) * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition * Inject kubeconfig file from ignition

* module/update-payload: Include TNO and its appversion into the payload. A follow up PR to the release automation repo is required to fully update the payload. * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * bazel: fix terraform-examples * bazel: refactor examples and docs This commit refactors the Bazel rules for examples and Documentation to accomplish several goals: 1. Openstack artifacts are now correctly compiled; 2. build rules for examples are in the examples directory and build rules for Documentation are in the Documentation directory. The last point allows us to easily compile all examples with: ```sh bazel build examples:all ``` Similarly, we can compile all Documentation by issuing: ```sh bazel build Documentation:all ``` * Update tectonic-cluo-operator to v0.3.0 * Update CLUO to v0.5.0 * Add support for Kubernetes v1.9 * config.tf: Update etcd version from 3.1.8 to 3.2.14 (coreos#2738) * bazel: create tarball target (coreos#2750) This commit adds tarball targets for Bazel. We can now create tarballs using: ```sh bazel build tarball ``` This will create a gzip compressed tarball named tectonic.tar.gz. The contents of the tarball will be nested in a directory named `tectonic`. To create a tarball with a version string in the directory name, we can build the tarball using: ```sh export VERSION=<some version> bazel build tarball --action_env=VERSION ``` In this case, the file structure will be nested in a directory called `tectonic_<some version>`. This commit also makes the installer binaries pure go so that they are statically linked, which is nicer for containers. * *: Remove option to deploy etcd without TLS (coreos#2763) * frontend: Add space-before-function-paren eslint rule Makes space character before function parameters consistent. * tests: run tests on a running cluster (coreos#2735) * Add terraform fmt test rule to bazel * tests: collect machine console logs (aws) (coreos#2754) * add initial aws support for bootstrapping through ncg (coreos#2758) * Inject kubeconfig file from ignition

* module/update-payload: Include TNO and its appversion into the payload. A follow up PR to the release automation repo is required to fully update the payload. * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * bazel: fix terraform-examples * bazel: refactor examples and docs This commit refactors the Bazel rules for examples and Documentation to accomplish several goals: 1. Openstack artifacts are now correctly compiled; 2. build rules for examples are in the examples directory and build rules for Documentation are in the Documentation directory. The last point allows us to easily compile all examples with: ```sh bazel build examples:all ``` Similarly, we can compile all Documentation by issuing: ```sh bazel build Documentation:all ``` * Update tectonic-cluo-operator to v0.3.0 * Update CLUO to v0.5.0 * Add support for Kubernetes v1.9 * config.tf: Update etcd version from 3.1.8 to 3.2.14 (coreos#2738) * bazel: create tarball target (coreos#2750) This commit adds tarball targets for Bazel. We can now create tarballs using: ```sh bazel build tarball ``` This will create a gzip compressed tarball named tectonic.tar.gz. The contents of the tarball will be nested in a directory named `tectonic`. To create a tarball with a version string in the directory name, we can build the tarball using: ```sh export VERSION=<some version> bazel build tarball --action_env=VERSION ``` In this case, the file structure will be nested in a directory called `tectonic_<some version>`. This commit also makes the installer binaries pure go so that they are statically linked, which is nicer for containers. * *: Remove option to deploy etcd without TLS (coreos#2763) * frontend: Add space-before-function-paren eslint rule Makes space character before function parameters consistent. * tests: run tests on a running cluster (coreos#2735) * frontend: Rename REMOVE_AT action to REMOVE_FIELD_LIST_ROW and simplify This Redux action is now specifically for removing a row from a field list, so rename it to make that clear. Simplify the reducers by assuming they take a field list id and an index, not an arbitrary path. Also stop clearing any associated error data, since a REMOVE_FIELD_LIST_ROW action will need to trigger a re-validation of the field list anyway. * frontend: Fix 2 htmlFor attributes * platforms/govcloud: add s3 support include in CL alpha * Add terraform fmt test rule to bazel * frontend: Only skip AWS VPC validation for 2 fields This fixes a bug where the "Private Route 53 Zones must use an existing private VPC." error was not cleared when switching to a valid hosted zone. This approach should be less error prone and easier to maintain. * tests: collect machine console logs (aws) (coreos#2754) * tests: add max log file size (1GB) feature * add initial aws support for bootstrapping through ncg (coreos#2758) * add initial aws support for bootstrapping through ncg (coreos#2758) * Inject kubeconfig file from ignition * Ut2 integration (coreos#2800) * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * add initial aws support for bootstrapping through ncg (coreos#2758) * Inject kubeconfig file from ignition * Inject kubeconfig file from ignition

* module/update-payload: Include TNO and its appversion into the payload. A follow up PR to the release automation repo is required to fully update the payload. * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * bazel: fix terraform-examples * bazel: refactor examples and docs This commit refactors the Bazel rules for examples and Documentation to accomplish several goals: 1. Openstack artifacts are now correctly compiled; 2. build rules for examples are in the examples directory and build rules for Documentation are in the Documentation directory. The last point allows us to easily compile all examples with: ```sh bazel build examples:all ``` Similarly, we can compile all Documentation by issuing: ```sh bazel build Documentation:all ``` * Update tectonic-cluo-operator to v0.3.0 * Update CLUO to v0.5.0 * Add support for Kubernetes v1.9 * config.tf: Update etcd version from 3.1.8 to 3.2.14 (#2738) * bazel: create tarball target (#2750) This commit adds tarball targets for Bazel. We can now create tarballs using: ```sh bazel build tarball ``` This will create a gzip compressed tarball named tectonic.tar.gz. The contents of the tarball will be nested in a directory named `tectonic`. To create a tarball with a version string in the directory name, we can build the tarball using: ```sh export VERSION=<some version> bazel build tarball --action_env=VERSION ``` In this case, the file structure will be nested in a directory called `tectonic_<some version>`. This commit also makes the installer binaries pure go so that they are statically linked, which is nicer for containers. * *: Remove option to deploy etcd without TLS (#2763) * frontend: Add space-before-function-paren eslint rule Makes space character before function parameters consistent. * tests: run tests on a running cluster (#2735) * frontend: Rename REMOVE_AT action to REMOVE_FIELD_LIST_ROW and simplify This Redux action is now specifically for removing a row from a field list, so rename it to make that clear. Simplify the reducers by assuming they take a field list id and an index, not an arbitrary path. Also stop clearing any associated error data, since a REMOVE_FIELD_LIST_ROW action will need to trigger a re-validation of the field list anyway. * frontend: Fix 2 htmlFor attributes * platforms/govcloud: add s3 support include in CL alpha * Add terraform fmt test rule to bazel * frontend: Only skip AWS VPC validation for 2 fields This fixes a bug where the "Private Route 53 Zones must use an existing private VPC." error was not cleared when switching to a valid hosted zone. This approach should be less error prone and easier to maintain. * tests: collect machine console logs (aws) (#2754) * tests: add max log file size (1GB) feature * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition * Ut2 integration (#2800) * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition * Ut2 integration (#2801) * module/update-payload: Include TNO and its appversion into the payload. A follow up PR to the release automation repo is required to fully update the payload. * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * bazel: fix terraform-examples * bazel: refactor examples and docs This commit refactors the Bazel rules for examples and Documentation to accomplish several goals: 1. Openstack artifacts are now correctly compiled; 2. build rules for examples are in the examples directory and build rules for Documentation are in the Documentation directory. The last point allows us to easily compile all examples with: ```sh bazel build examples:all ``` Similarly, we can compile all Documentation by issuing: ```sh bazel build Documentation:all ``` * Update tectonic-cluo-operator to v0.3.0 * Update CLUO to v0.5.0 * Add support for Kubernetes v1.9 * config.tf: Update etcd version from 3.1.8 to 3.2.14 (#2738) * bazel: create tarball target (#2750) This commit adds tarball targets for Bazel. We can now create tarballs using: ```sh bazel build tarball ``` This will create a gzip compressed tarball named tectonic.tar.gz. The contents of the tarball will be nested in a directory named `tectonic`. To create a tarball with a version string in the directory name, we can build the tarball using: ```sh export VERSION=<some version> bazel build tarball --action_env=VERSION ``` In this case, the file structure will be nested in a directory called `tectonic_<some version>`. This commit also makes the installer binaries pure go so that they are statically linked, which is nicer for containers. * *: Remove option to deploy etcd without TLS (#2763) * frontend: Add space-before-function-paren eslint rule Makes space character before function parameters consistent. * tests: run tests on a running cluster (#2735) * frontend: Rename REMOVE_AT action to REMOVE_FIELD_LIST_ROW and simplify This Redux action is now specifically for removing a row from a field list, so rename it to make that clear. Simplify the reducers by assuming they take a field list id and an index, not an arbitrary path. Also stop clearing any associated error data, since a REMOVE_FIELD_LIST_ROW action will need to trigger a re-validation of the field list anyway. * frontend: Fix 2 htmlFor attributes * platforms/govcloud: add s3 support include in CL alpha * Add terraform fmt test rule to bazel * frontend: Only skip AWS VPC validation for 2 fields This fixes a bug where the "Private Route 53 Zones must use an existing private VPC." error was not cleared when switching to a valid hosted zone. This approach should be less error prone and easier to maintain. * tests: collect machine console logs (aws) (#2754) * tests: add max log file size (1GB) feature * add initial aws support for bootstrapping through ncg (#2758) * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition * Ut2 integration (#2800) * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition * Inject kubeconfig file from ignition

* module/update-payload: Include TNO and its appversion into the payload. A follow up PR to the release automation repo is required to fully update the payload. * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * bazel: fix terraform-examples * bazel: refactor examples and docs This commit refactors the Bazel rules for examples and Documentation to accomplish several goals: 1. Openstack artifacts are now correctly compiled; 2. build rules for examples are in the examples directory and build rules for Documentation are in the Documentation directory. The last point allows us to easily compile all examples with: ```sh bazel build examples:all ``` Similarly, we can compile all Documentation by issuing: ```sh bazel build Documentation:all ``` * Update tectonic-cluo-operator to v0.3.0 * Update CLUO to v0.5.0 * Add support for Kubernetes v1.9 * config.tf: Update etcd version from 3.1.8 to 3.2.14 (coreos#2738) * bazel: create tarball target (coreos#2750) This commit adds tarball targets for Bazel. We can now create tarballs using: ```sh bazel build tarball ``` This will create a gzip compressed tarball named tectonic.tar.gz. The contents of the tarball will be nested in a directory named `tectonic`. To create a tarball with a version string in the directory name, we can build the tarball using: ```sh export VERSION=<some version> bazel build tarball --action_env=VERSION ``` In this case, the file structure will be nested in a directory called `tectonic_<some version>`. This commit also makes the installer binaries pure go so that they are statically linked, which is nicer for containers. * *: Remove option to deploy etcd without TLS (coreos#2763) * frontend: Add space-before-function-paren eslint rule Makes space character before function parameters consistent. * tests: run tests on a running cluster (coreos#2735) * Add terraform fmt test rule to bazel * tests: collect machine console logs (aws) (coreos#2754) * add initial aws support for bootstrapping through ncg (coreos#2758) * Inject kubeconfig file from ignition

* module/update-payload: Include TNO and its appversion into the payload. A follow up PR to the release automation repo is required to fully update the payload. * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * bazel: fix terraform-examples * bazel: refactor examples and docs This commit refactors the Bazel rules for examples and Documentation to accomplish several goals: 1. Openstack artifacts are now correctly compiled; 2. build rules for examples are in the examples directory and build rules for Documentation are in the Documentation directory. The last point allows us to easily compile all examples with: ```sh bazel build examples:all ``` Similarly, we can compile all Documentation by issuing: ```sh bazel build Documentation:all ``` * Update tectonic-cluo-operator to v0.3.0 * Update CLUO to v0.5.0 * Add support for Kubernetes v1.9 * config.tf: Update etcd version from 3.1.8 to 3.2.14 (#2738) * bazel: create tarball target (#2750) This commit adds tarball targets for Bazel. We can now create tarballs using: ```sh bazel build tarball ``` This will create a gzip compressed tarball named tectonic.tar.gz. The contents of the tarball will be nested in a directory named `tectonic`. To create a tarball with a version string in the directory name, we can build the tarball using: ```sh export VERSION=<some version> bazel build tarball --action_env=VERSION ``` In this case, the file structure will be nested in a directory called `tectonic_<some version>`. This commit also makes the installer binaries pure go so that they are statically linked, which is nicer for containers. * *: Remove option to deploy etcd without TLS (#2763) * frontend: Add space-before-function-paren eslint rule Makes space character before function parameters consistent. * tests: run tests on a running cluster (#2735) * frontend: Rename REMOVE_AT action to REMOVE_FIELD_LIST_ROW and simplify This Redux action is now specifically for removing a row from a field list, so rename it to make that clear. Simplify the reducers by assuming they take a field list id and an index, not an arbitrary path. Also stop clearing any associated error data, since a REMOVE_FIELD_LIST_ROW action will need to trigger a re-validation of the field list anyway. * frontend: Fix 2 htmlFor attributes * platforms/govcloud: add s3 support include in CL alpha * Add terraform fmt test rule to bazel * frontend: Only skip AWS VPC validation for 2 fields This fixes a bug where the "Private Route 53 Zones must use an existing private VPC." error was not cleared when switching to a valid hosted zone. This approach should be less error prone and easier to maintain. * tests: collect machine console logs (aws) (#2754) * tests: add max log file size (1GB) feature * add initial aws support for bootstrapping through ncg (#2758) * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition * Ut2 integration (#2800) * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition * Inject kubeconfig file from ignition

* module/update-payload: Include TNO and its appversion into the payload. A follow up PR to the release automation repo is required to fully update the payload. * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * bazel: fix terraform-examples * bazel: refactor examples and docs This commit refactors the Bazel rules for examples and Documentation to accomplish several goals: 1. Openstack artifacts are now correctly compiled; 2. build rules for examples are in the examples directory and build rules for Documentation are in the Documentation directory. The last point allows us to easily compile all examples with: ```sh bazel build examples:all ``` Similarly, we can compile all Documentation by issuing: ```sh bazel build Documentation:all ``` * Update tectonic-cluo-operator to v0.3.0 * Update CLUO to v0.5.0 * Add support for Kubernetes v1.9 * config.tf: Update etcd version from 3.1.8 to 3.2.14 (#2738) * bazel: create tarball target (#2750) This commit adds tarball targets for Bazel. We can now create tarballs using: ```sh bazel build tarball ``` This will create a gzip compressed tarball named tectonic.tar.gz. The contents of the tarball will be nested in a directory named `tectonic`. To create a tarball with a version string in the directory name, we can build the tarball using: ```sh export VERSION=<some version> bazel build tarball --action_env=VERSION ``` In this case, the file structure will be nested in a directory called `tectonic_<some version>`. This commit also makes the installer binaries pure go so that they are statically linked, which is nicer for containers. * *: Remove option to deploy etcd without TLS (#2763) * frontend: Add space-before-function-paren eslint rule Makes space character before function parameters consistent. * tests: run tests on a running cluster (#2735) * Add terraform fmt test rule to bazel * tests: collect machine console logs (aws) (#2754) * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition

* module/update-payload: Include TNO and its appversion into the payload. A follow up PR to the release automation repo is required to fully update the payload. * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * bazel: fix terraform-examples * bazel: refactor examples and docs This commit refactors the Bazel rules for examples and Documentation to accomplish several goals: 1. Openstack artifacts are now correctly compiled; 2. build rules for examples are in the examples directory and build rules for Documentation are in the Documentation directory. The last point allows us to easily compile all examples with: ```sh bazel build examples:all ``` Similarly, we can compile all Documentation by issuing: ```sh bazel build Documentation:all ``` * Update tectonic-cluo-operator to v0.3.0 * Update CLUO to v0.5.0 * Add support for Kubernetes v1.9 * config.tf: Update etcd version from 3.1.8 to 3.2.14 (coreos#2738) * bazel: create tarball target (coreos#2750) This commit adds tarball targets for Bazel. We can now create tarballs using: ```sh bazel build tarball ``` This will create a gzip compressed tarball named tectonic.tar.gz. The contents of the tarball will be nested in a directory named `tectonic`. To create a tarball with a version string in the directory name, we can build the tarball using: ```sh export VERSION=<some version> bazel build tarball --action_env=VERSION ``` In this case, the file structure will be nested in a directory called `tectonic_<some version>`. This commit also makes the installer binaries pure go so that they are statically linked, which is nicer for containers. * *: Remove option to deploy etcd without TLS (coreos#2763) * frontend: Add space-before-function-paren eslint rule Makes space character before function parameters consistent. * tests: run tests on a running cluster (coreos#2735) * frontend: Rename REMOVE_AT action to REMOVE_FIELD_LIST_ROW and simplify This Redux action is now specifically for removing a row from a field list, so rename it to make that clear. Simplify the reducers by assuming they take a field list id and an index, not an arbitrary path. Also stop clearing any associated error data, since a REMOVE_FIELD_LIST_ROW action will need to trigger a re-validation of the field list anyway. * frontend: Fix 2 htmlFor attributes * platforms/govcloud: add s3 support include in CL alpha * Add terraform fmt test rule to bazel * frontend: Only skip AWS VPC validation for 2 fields This fixes a bug where the "Private Route 53 Zones must use an existing private VPC." error was not cleared when switching to a valid hosted zone. This approach should be less error prone and easier to maintain. * tests: collect machine console logs (aws) (coreos#2754) * tests: add max log file size (1GB) feature * add initial aws support for bootstrapping through ncg (coreos#2758) * add initial aws support for bootstrapping through ncg (coreos#2758) * Inject kubeconfig file from ignition * Ut2 integration (coreos#2800) * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * add initial aws support for bootstrapping through ncg (coreos#2758) * Inject kubeconfig file from ignition * Inject kubeconfig file from ignition

* modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition * Ut2 integration (#2800) * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition * Ut2 integration (#2801) * module/update-payload: Include TNO and its appversion into the payload. A follow up PR to the release automation repo is required to fully update the payload. * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * bazel: fix terraform-examples * bazel: refactor examples and docs This commit refactors the Bazel rules for examples and Documentation to accomplish several goals: 1. Openstack artifacts are now correctly compiled; 2. build rules for examples are in the examples directory and build rules for Documentation are in the Documentation directory. The last point allows us to easily compile all examples with: ```sh bazel build examples:all ``` Similarly, we can compile all Documentation by issuing: ```sh bazel build Documentation:all ``` * Update tectonic-cluo-operator to v0.3.0 * Update CLUO to v0.5.0 * Add support for Kubernetes v1.9 * config.tf: Update etcd version from 3.1.8 to 3.2.14 (#2738) * bazel: create tarball target (#2750) This commit adds tarball targets for Bazel. We can now create tarballs using: ```sh bazel build tarball ``` This will create a gzip compressed tarball named tectonic.tar.gz. The contents of the tarball will be nested in a directory named `tectonic`. To create a tarball with a version string in the directory name, we can build the tarball using: ```sh export VERSION=<some version> bazel build tarball --action_env=VERSION ``` In this case, the file structure will be nested in a directory called `tectonic_<some version>`. This commit also makes the installer binaries pure go so that they are statically linked, which is nicer for containers. * *: Remove option to deploy etcd without TLS (#2763) * frontend: Add space-before-function-paren eslint rule Makes space character before function parameters consistent. * tests: run tests on a running cluster (#2735) * frontend: Rename REMOVE_AT action to REMOVE_FIELD_LIST_ROW and simplify This Redux action is now specifically for removing a row from a field list, so rename it to make that clear. Simplify the reducers by assuming they take a field list id and an index, not an arbitrary path. Also stop clearing any associated error data, since a REMOVE_FIELD_LIST_ROW action will need to trigger a re-validation of the field list anyway. * frontend: Fix 2 htmlFor attributes * platforms/govcloud: add s3 support include in CL alpha * Add terraform fmt test rule to bazel * frontend: Only skip AWS VPC validation for 2 fields This fixes a bug where the "Private Route 53 Zones must use an existing private VPC." error was not cleared when switching to a valid hosted zone. This approach should be less error prone and easier to maintain. * tests: collect machine console logs (aws) (#2754) * tests: add max log file size (1GB) feature * add initial aws support for bootstrapping through ncg (#2758) * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition * Ut2 integration (#2800) * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * add initial aws support for bootstrapping through ncg (#2758) * Inject kubeconfig file from ignition * Inject kubeconfig file from ignition

* module/update-payload: Include TNO and its appversion into the payload. A follow up PR to the release automation repo is required to fully update the payload. * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * bazel: fix terraform-examples * bazel: refactor examples and docs This commit refactors the Bazel rules for examples and Documentation to accomplish several goals: 1. Openstack artifacts are now correctly compiled; 2. build rules for examples are in the examples directory and build rules for Documentation are in the Documentation directory. The last point allows us to easily compile all examples with: ```sh bazel build examples:all ``` Similarly, we can compile all Documentation by issuing: ```sh bazel build Documentation:all ``` * Update tectonic-cluo-operator to v0.3.0 * Update CLUO to v0.5.0 * Add support for Kubernetes v1.9 * config.tf: Update etcd version from 3.1.8 to 3.2.14 (coreos#2738) * bazel: create tarball target (coreos#2750) This commit adds tarball targets for Bazel. We can now create tarballs using: ```sh bazel build tarball ``` This will create a gzip compressed tarball named tectonic.tar.gz. The contents of the tarball will be nested in a directory named `tectonic`. To create a tarball with a version string in the directory name, we can build the tarball using: ```sh export VERSION=<some version> bazel build tarball --action_env=VERSION ``` In this case, the file structure will be nested in a directory called `tectonic_<some version>`. This commit also makes the installer binaries pure go so that they are statically linked, which is nicer for containers. * *: Remove option to deploy etcd without TLS (coreos#2763) * frontend: Add space-before-function-paren eslint rule Makes space character before function parameters consistent. * tests: run tests on a running cluster (coreos#2735) * frontend: Rename REMOVE_AT action to REMOVE_FIELD_LIST_ROW and simplify This Redux action is now specifically for removing a row from a field list, so rename it to make that clear. Simplify the reducers by assuming they take a field list id and an index, not an arbitrary path. Also stop clearing any associated error data, since a REMOVE_FIELD_LIST_ROW action will need to trigger a re-validation of the field list anyway. * frontend: Fix 2 htmlFor attributes * platforms/govcloud: add s3 support include in CL alpha * Add terraform fmt test rule to bazel * frontend: Only skip AWS VPC validation for 2 fields This fixes a bug where the "Private Route 53 Zones must use an existing private VPC." error was not cleared when switching to a valid hosted zone. This approach should be less error prone and easier to maintain. * tests: collect machine console logs (aws) (coreos#2754) * tests: add max log file size (1GB) feature * add initial aws support for bootstrapping through ncg (coreos#2758) * add initial aws support for bootstrapping through ncg (coreos#2758) * Inject kubeconfig file from ignition * Ut2 integration (coreos#2800) * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * add initial aws support for bootstrapping through ncg (coreos#2758) * Inject kubeconfig file from ignition * Inject kubeconfig file from ignition

* module/update-payload: Include TNO and its appversion into the payload. A follow up PR to the release automation repo is required to fully update the payload. * modules/aws: add ec2_ami_override to etcd, master, worker nodes Adds three terraform variables which can be used to override the AMI rather than filtering out the latest Container Linux version for the given node type. * platform/aws: remove image_re unused variable * bazel: fix terraform-examples * bazel: refactor examples and docs This commit refactors the Bazel rules for examples and Documentation to accomplish several goals: 1. Openstack artifacts are now correctly compiled; 2. build rules for examples are in the examples directory and build rules for Documentation are in the Documentation directory. The last point allows us to easily compile all examples with: ```sh bazel build examples:all ``` Similarly, we can compile all Documentation by issuing: ```sh bazel build Documentation:all ``` * Update tectonic-cluo-operator to v0.3.0 * Update CLUO to v0.5.0 * Add support for Kubernetes v1.9 * config.tf: Update etcd version from 3.1.8 to 3.2.14 (coreos#2738) * bazel: create tarball target (coreos#2750) This commit adds tarball targets for Bazel. We can now create tarballs using: ```sh bazel build tarball ``` This will create a gzip compressed tarball named tectonic.tar.gz. The contents of the tarball will be nested in a directory named `tectonic`. To create a tarball with a version string in the directory name, we can build the tarball using: ```sh export VERSION=<some version> bazel build tarball --action_env=VERSION ``` In this case, the file structure will be nested in a directory called `tectonic_<some version>`. This commit also makes the installer binaries pure go so that they are statically linked, which is nicer for containers. * *: Remove option to deploy etcd without TLS (coreos#2763) * frontend: Add space-before-function-paren eslint rule Makes space character before function parameters consistent. * tests: run tests on a running cluster (coreos#2735) * Add terraform fmt test rule to bazel * tests: collect machine console logs (aws) (coreos#2754) * add initial aws support for bootstrapping through ncg (coreos#2758) * Inject kubeconfig file from ignition

cpanato added area/testing platform/aws run-smoke-tests labels Jan 16, 2018

cpanato force-pushed the INST-520 branch from 3378052 to 811ecd5 Compare January 16, 2018 14:39

cpanato requested review from mxinden and squat January 16, 2018 15:25

squat suggested changes Jan 16, 2018

View reviewed changes

mxinden reviewed Jan 17, 2018

View reviewed changes

tests: run tests on a running cluster

3d92f6e

cpanato force-pushed the INST-520 branch from 811ecd5 to 3d92f6e Compare January 17, 2018 11:40

squat approved these changes Jan 22, 2018

View reviewed changes

mxinden approved these changes Jan 22, 2018

View reviewed changes

cpanato merged commit 9e04b61 into coreos:master Jan 22, 2018

cpanato deleted the INST-520 branch January 22, 2018 10:11

cpanato added a commit to cpanato/tectonic-installer that referenced this pull request Jan 22, 2018

tests: run tests on a running cluster (coreos#2735)

35add1a

cpanato mentioned this pull request Jan 22, 2018

Track1 aws boot logs #2790

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

tests: run tests on a running cluster #2735

tests: run tests on a running cluster #2735

cpanato commented Jan 16, 2018

coreosbot commented Jan 16, 2018

squat left a comment

squat Jan 16, 2018

cpanato Jan 17, 2018

squat Jan 16, 2018

cpanato Jan 17, 2018

mxinden Jan 17, 2018

cpanato Jan 17, 2018

mxinden Jan 17, 2018

cpanato Jan 17, 2018

mxinden Jan 22, 2018

mxinden Jan 17, 2018

cpanato Jan 17, 2018

mxinden commented Jan 18, 2018

cpanato commented Jan 18, 2018

mxinden Jan 22, 2018

tests: run tests on a running cluster #2735

tests: run tests on a running cluster #2735

Conversation

cpanato commented Jan 16, 2018

coreosbot commented Jan 16, 2018

squat left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

mxinden commented Jan 18, 2018

cpanato commented Jan 18, 2018

Choose a reason for hiding this comment