Simplify registry service component updates & repo (#2850)

* Simplify registry service component updates & repo

* update openapi

* fix

Author: mschuwalow

cli/golem-cli/src/command_handler/app/deploy_diff.rs

@@ -732,7 +732,6 @@ fn normalized_diff_deployment(
                         Some(component) => diff::Component {
                             metadata: match component.metadata.as_value() {
                                 Some(metadata) => diff::ComponentMetadata {
-                                    version: metadata.version.clone(),
                                     env: safe_env(&metadata.env),
                                     config_vars: metadata.config_vars.clone(),
                                 }

cli/golem-cli/src/command_handler/component/mod.rs

@@ -1010,7 +1010,6 @@ impl ComponentCommandHandler {
 
         Ok(diff::Component {
             metadata: diff::ComponentMetadata {
-                version: Some("".to_string()), // TODO: atomic
                 env: properties
                     .env
                     .iter()

golem-api-grpc/proto/golem/component/component.proto

@@ -29,17 +29,13 @@ message Component {
   golem.common.EnvironmentId environment_id = 6;
   google.protobuf.Timestamp created_at = 7;
   repeated golem.component.InitialComponentFile files = 8;
-  repeated golem.component.InitialComponentFile original_files = 9;
   repeated golem.component.PluginInstallation installed_plugins = 10;
   golem.common.AccountId account_id = 11;
   map<string, string> env = 12;
-  map<string, string> original_env = 13;
-  map<string, string> config_vars = 14;
-  map<string, string> original_config_vars = 15;
-  repeated LocalAgentConfigEntry local_agent_config = 16;
-  golem.common.Hash wasm_hash = 17;
-  golem.common.ApplicationId application_id = 18;
-  golem.common.Hash hash = 19;
-  string transformed_object_store_key = 20;
-  string object_store_key = 21;
+  map<string, string> config_vars = 13;
+  repeated LocalAgentConfigEntry local_agent_config = 14;
+  golem.common.Hash wasm_hash = 15;
+  golem.common.ApplicationId application_id = 16;
+  golem.common.Hash hash = 17;
+  string object_store_key = 18;
 }

golem-common/src/base_model/component.rs

@@ -131,12 +131,9 @@ declare_structs! {
         pub metadata: ComponentMetadata,
         pub created_at: chrono::DateTime<chrono::Utc>,
         pub files: Vec<InitialComponentFile>,
-        pub original_files: Vec<InitialComponentFile>,
         pub installed_plugins: Vec<InstalledPlugin>,
         pub env: BTreeMap<String, String>,
-        pub original_env: BTreeMap<String, String>,
         pub config_vars: BTreeMap<String, String>,
-        pub original_config_vars: BTreeMap<String, String>,
         pub local_agent_config: Vec<LocalAgentConfigEntry>,
         pub wasm_hash: diff::Hash,
     }

golem-common/src/model/component.rs

@@ -22,7 +22,6 @@ impl ComponentDto {
     pub fn to_diffable(&self) -> diff::Component {
         diff::Component {
             metadata: diff::ComponentMetadata {
-                version: Some("".to_string()), // TODO: atomic
                 env: self
                     .env
                     .iter()

golem-common/src/model/diff/component.rs

@@ -99,8 +99,6 @@ impl Diffable for ComponentFile {
 #[derive(Debug, Clone, PartialEq, Serialize)]
 #[serde(rename_all = "camelCase")]
 pub struct ComponentMetadata {
-    #[serde(skip_serializing_if = "Option::is_none")]
-    pub version: Option<String>,
     #[serde(skip_serializing_if = "BTreeMap::is_empty")]
     pub env: BTreeMap<String, String>,
     #[serde(skip_serializing_if = "BTreeMap::is_empty")]

golem-registry-service/db/migration/postgres/004_local_agent_config.sql

@@ -1,2 +1,16 @@
 ALTER TABLE component_revisions
-    ADD local_agent_config BYTEA NOT NULL;
+    ADD local_agent_config BYTEA NOT NULL;
+
+ALTER TABLE component_revisions
+    DROP COLUMN original_env;
+
+ALTER TABLE component_revisions
+    DROP COLUMN original_config_vars;
+
+ALTER TABLE component_revisions
+    DROP COLUMN transformed_object_store_key;
+
+ALTER TABLE component_revisions
+    DROP COLUMN version;
+
+DROP TABLE original_component_files;

golem-registry-service/db/migration/sqlite/004_local_agent_config.sql

@@ -1,2 +1,16 @@
 ALTER TABLE component_revisions
     ADD local_agent_config BYTEA NOT NULL;
+
+ALTER TABLE component_revisions
+    DROP COLUMN original_env;
+
+ALTER TABLE component_revisions
+    DROP COLUMN original_config_vars;
+
+ALTER TABLE component_revisions
+    DROP COLUMN transformed_object_store_key;
+
+ALTER TABLE component_revisions
+    DROP COLUMN version;
+
+DROP TABLE original_component_files;

golem-registry-service/src/model/component.rs

@@ -13,7 +13,6 @@
 // limitations under the License.
 
 pub mod api_definition;
-pub mod component;
 pub mod component_slug;
 pub mod login;
 pub mod security_scheme;