add a no prompt for cases when dsn can be configured individually as …

…well

cli/cmd/env/configure.go

@@ -181,6 +181,9 @@ func VariablesFlow(ctx context.Context, ch *cmdutil.Helper, projectPath string)
 
 		for i := range c.Spec.ConfigProperties {
 			prop := c.Spec.ConfigProperties[i] // TODO: Move into range and turn into pointer
+			if prop.NoPrompt {
+				continue
+			}
 
 			key := fmt.Sprintf("connector.%s.%s", c.Name, prop.Key)
 			msg := key

proto/gen/rill/admin/v1/admin.swagger.yaml

@@ -3190,7 +3190,7 @@ definitions:
       `NullValue` is a singleton enumeration to represent the null value for the
       `Value` type union.
 
-      The JSON representation for `NullValue` is JSON `null`.
+       The JSON representation for `NullValue` is JSON `null`.
 
        - NULL_VALUE: Null value.
   rpcStatus:

proto/gen/rill/runtime/v1/runtime.swagger.yaml

@@ -2644,6 +2644,9 @@ definitions:
       secret:
         type: boolean
         title: If true, the property is a secret and should not be displayed in UIs
+      noPrompt:
+        type: boolean
+        title: If true, the property should not be prompted. The expectation is that only advanced users will use it and can set directly in the connector.yaml
     title: Property represents the spec of one of the driver's config properties
   ConnectorDriverPropertyType:
     type: string
@@ -2902,7 +2905,7 @@ definitions:
       `NullValue` is a singleton enumeration to represent the null value for the
       `Value` type union.
 
-      The JSON representation for `NullValue` is JSON `null`.
+       The JSON representation for `NullValue` is JSON `null`.
 
        - NULL_VALUE: Null value.
   rpcStatus:

proto/rill/runtime/v1/api.proto

@@ -677,6 +677,8 @@ message ConnectorDriver {
     string placeholder = 9;
     // If true, the property is a secret and should not be displayed in UIs
     bool secret = 10;
+    // If true, the property should not be prompted. The expectation is that only advanced users will use it and can set directly in the connector.yaml
+    bool no_prompt = 11;
   }
   // Name is the name of the connector
   string name = 1;

runtime/drivers/clickhouse/clickhouse.go

@@ -32,6 +32,7 @@ var spec = drivers.Spec{
 			Required:    false,
 			DisplayName: "Connection string",
 			Placeholder: "clickhouse://localhost:9000?username=default&password=",
+			NoPrompt:    true,
 		},
 		{
 			Key:         "host",

runtime/drivers/connectors.go

@@ -46,6 +46,7 @@ type PropertySpec struct {
 	Default     string
 	Placeholder string
 	Secret      bool
+	NoPrompt    bool
 }
 
 // PropertyType is an enum of types supported for connector properties.

runtime/drivers/druid/druid.go

@@ -34,6 +34,7 @@ var spec = drivers.Spec{
 			DisplayName: "Connection string",
 			Placeholder: "https://example.com/druid/v2/sql/avatica-protobuf?authentication=BASIC&avaticaUser=username&avaticaPassword=password",
 			Secret:      true,
+			NoPrompt:    true,
 		},
 		{
 			Key:         "host",

runtime/drivers/pinot/pinot.go

@@ -30,6 +30,7 @@ var spec = drivers.Spec{
 			DisplayName: "Connection string",
 			Placeholder: "http(s)://username:password@localhost:9000",
 			Secret:      true,
+			NoPrompt:    true,
 		},
 		{
 			Key:         "host",

runtime/server/connectors.go

@@ -200,6 +200,7 @@ func driverPropertySpecToPB(spec *drivers.PropertySpec) *runtimev1.ConnectorDriv
 		Default:     spec.Default,
 		Placeholder: spec.Placeholder,
 		Secret:      spec.Secret,
+		NoPrompt:    spec.NoPrompt,
 	}
 }
 

web-admin/src/client/gen/index.schemas.ts

@@ -1261,7 +1261,7 @@ export interface RpcStatus {
  * `NullValue` is a singleton enumeration to represent the null value for the
 `Value` type union.
 
-The JSON representation for `NullValue` is JSON `null`.
+ The JSON representation for `NullValue` is JSON `null`.
 
  - NULL_VALUE: Null value.
  */

web-common/src/proto/gen/rill/runtime/v1/api_pb.ts

@@ -3215,6 +3215,13 @@ export class ConnectorDriver_Property extends Message<ConnectorDriver_Property>
    */
   secret = false;
 
+  /**
+   * If true, the property should not be prompted. The expectation is that only advanced users will use it and can set directly in the connector.yaml
+   *
+   * @generated from field: bool no_prompt = 11;
+   */
+  noPrompt = false;
+
   constructor(data?: PartialMessage<ConnectorDriver_Property>) {
     super();
     proto3.util.initPartial(data, this);
@@ -3233,6 +3240,7 @@ export class ConnectorDriver_Property extends Message<ConnectorDriver_Property>
     { no: 8, name: "default", kind: "scalar", T: 9 /* ScalarType.STRING */ },
     { no: 9, name: "placeholder", kind: "scalar", T: 9 /* ScalarType.STRING */ },
     { no: 10, name: "secret", kind: "scalar", T: 8 /* ScalarType.BOOL */ },
+    { no: 11, name: "no_prompt", kind: "scalar", T: 8 /* ScalarType.BOOL */ },
   ]);
 
   static fromBinary(bytes: Uint8Array, options?: Partial<BinaryReadOptions>): ConnectorDriver_Property {

web-common/src/runtime-client/gen/index.schemas.ts

@@ -2134,7 +2134,7 @@ export interface Runtimev1Type {
  * `NullValue` is a singleton enumeration to represent the null value for the
 `Value` type union.
 
-The JSON representation for `NullValue` is JSON `null`.
+ The JSON representation for `NullValue` is JSON `null`.
 
  - NULL_VALUE: Null value.
  */
@@ -2329,6 +2329,7 @@ export interface ConnectorDriverProperty {
   default?: string;
   placeholder?: string;
   secret?: boolean;
+  noPrompt?: boolean;
 }
 
 export interface ColumnTimeSeriesRequestBasicMeasure {