Merge pull request #506 from UffizziCloud/feature/505-add-node-select…

…or-to-cluster

[505] added node selector to cluster

CONTRIBUTING.md

@@ -84,7 +84,7 @@ In order to add a new migration do the following steps:
 2. Run the command inside the `web` container
 
 ```bash
-rake app:uffizzi_core:install
+rake uffizzi_core:install:migrations
 ```
 
 This command copies the new migration to the `db/migrate` folder

core/app/forms/uffizzi_core/api/cli/v1/cluster/create_form.rb

@@ -3,7 +3,7 @@
 class UffizziCore::Api::Cli::V1::Cluster::CreateForm < UffizziCore::Cluster
   include UffizziCore::ApplicationForm
 
-  permit :name, :manifest, :creation_source
+  permit :name, :manifest, :creation_source, :node_selector
 
   validate :check_manifest, if: -> { manifest.present? }
 

core/app/lib/uffizzi_core/concerns/models/cluster.rb

@@ -19,6 +19,7 @@ module UffizziCore::Concerns::Models::Cluster
     validates :name, presence: true, format: { with: /\A[a-zA-Z0-9-]*\z/ }
 
     enumerize :creation_source, in: UffizziCore.cluster_creation_sources, scope: true, predicates: true
+    enumerize :node_selector, in: [:gvisor, :vanilla]
     attribute :creation_source, :string, default: :manual
     validates :creation_source, presence: true
     belongs_to :kubernetes_distribution, optional: true

core/app/serializers/uffizzi_core/controller/create_cluster/cluster_serializer.rb

@@ -4,7 +4,7 @@ class UffizziCore::Controller::CreateCluster::ClusterSerializer < UffizziCore::B
   include UffizziCore::DependencyInjectionConcern
   include_module_if_exists('UffizziCore::Controller::CreateCluster::ClusterSerializerModule')
 
-  attributes :name, :manifest, :base_ingress_host, :distro, :image
+  attributes :name, :manifest, :base_ingress_host, :distro, :image, :node_selector
 
   def base_ingress_host
     managed_dns_zone = controller_settings_service.vcluster_settings_by_vcluster(object).managed_dns_zone

core/db/migrate/20240301200235_add_node_selector_to_cluster.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+class AddNodeSelectorToCluster < ActiveRecord::Migration[6.1]
+  def change
+    add_column(:uffizzi_core_clusters, :node_selector, :string)
+  end
+end

core/test/dummy/db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2023_10_09_182412) do
+ActiveRecord::Schema.define(version: 2024_03_01_200235) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -85,6 +85,7 @@
     t.string "host"
     t.string "creation_source"
     t.integer "kubernetes_distribution_id"
+    t.string "node_selector"
     t.index ["project_id"], name: "index_cluster_on_project_id"
   end
 

db/migrate/20240301200916_add_node_selector_to_cluster.uffizzi_core.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+# This migration comes from uffizzi_core (originally 20240301200235)
+class AddNodeSelectorToCluster < ActiveRecord::Migration[6.1]
+  def change
+    add_column(:uffizzi_core_clusters, :node_selector, :string)
+  end
+end

db/schema.rb

@@ -10,7 +10,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 2023_10_09_163516) do
+ActiveRecord::Schema.define(version: 2024_03_01_200916) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -84,6 +84,8 @@
     t.datetime "updated_at", precision: 6, null: false
     t.string "host"
     t.string "creation_source"
+    t.integer "kubernetes_distribution_id"
+    t.string "node_selector"
     t.index ["project_id"], name: "index_cluster_on_project_id"
   end