From 22506e39a2ebb12cc2be04cb34c84643fbf54491 Mon Sep 17 00:00:00 2001 From: Pavel Abramov Date: Tue, 16 Apr 2024 08:25:50 +0200 Subject: [PATCH] EVE AppInst Discovery: move `allow_to_discover` to AppInst port This is more granular approach. From appInstance perspective, you also know which network port will be used. Signed-off-by: Pavel Abramov --- proto/config/appconfig.proto | 7 +++---- proto/config/netconfig.proto | 4 ++++ 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/proto/config/appconfig.proto b/proto/config/appconfig.proto index 3af6fa34..77b039a5 100644 --- a/proto/config/appconfig.proto +++ b/proto/config/appconfig.proto @@ -103,6 +103,9 @@ message SnapshotConfig { // can also be increased to cause an application instance restart without // any other change to the application instance. message AppInstanceConfig { + // deprecated fields + reserved 14, 24; + UUIDandVersion uuidandversion = 1; string displayname = 2; // User-friendly name VmConfig fixedresources = 3; @@ -206,10 +209,6 @@ message AppInstanceConfig { // PatchEnvelopeRef is used to track all patch envelopes used by the app instance PatchEnvelopeRef patchRef = 23; - - // allow AppInstance to discover other AppInstances - // attached to its network instances - bool allow_to_discover = 24; } message PatchEnvelopeRef { diff --git a/proto/config/netconfig.proto b/proto/config/netconfig.proto index 3b5573e7..05daab75 100644 --- a/proto/config/netconfig.proto +++ b/proto/config/netconfig.proto @@ -55,6 +55,10 @@ message NetworkAdapter { // valid vlan id range: 2 - 4093 // vlan id 1 is implicitly used by linux bridges uint32 access_vlan_id = 41; + + // allow AppInstance to discover other AppInstances + // attached to this network instance. Default is false + bool allow_to_discover = 42; } message WirelessConfig {