-
Notifications
You must be signed in to change notification settings - Fork 732
Adds Aspire Oracle EntityFrameworkCore Database component. #1295
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 11 commits
92a7922
0f2f3f8
12ee396
0f384e8
773b457
06d7a4c
4db7868
824ed28
91e0b7b
74a10a0
ae61db3
9ac8eab
e04cee1
8b84294
5d99c70
7c09cbd
9abcdd8
5f0b689
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| // Licensed to the .NET Foundation under one or more agreements. | ||
eerhardt marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| // The .NET Foundation licenses this file to you under the MIT license. | ||
|
|
||
| namespace Aspire.Hosting.ApplicationModel; | ||
|
|
||
| /// <summary> | ||
| /// Represents a Oracle Database resource that requires a connection string. | ||
| /// </summary> | ||
| public interface IOracleDatabaseParentResource : IResourceWithConnectionString, IResourceWithEnvironment | ||
| { | ||
| } | ||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,105 @@ | ||||||
| // Licensed to the .NET Foundation under one or more agreements. | ||||||
| // The .NET Foundation licenses this file to you under the MIT license. | ||||||
|
|
||||||
| using System.Net.Sockets; | ||||||
| using Aspire.Hosting.ApplicationModel; | ||||||
| using Aspire.Hosting.Publishing; | ||||||
|
|
||||||
| namespace Aspire.Hosting; | ||||||
|
|
||||||
| /// <summary> | ||||||
| /// Provides extension methods for adding Oracle Database resources to an <see cref="IDistributedApplicationBuilder"/>. | ||||||
| /// </summary> | ||||||
| public static class OracleDatabaseBuilderExtensions | ||||||
| { | ||||||
| private const string PasswordEnvVarName = "ORACLE_PWD"; | ||||||
|
|
||||||
| /// <summary> | ||||||
| /// Adds a Oracle Database container to the application model. The default image is "database/free" and the tag is "latest". | ||||||
| /// </summary> | ||||||
| /// <param name="builder">The <see cref="IDistributedApplicationBuilder"/>.</param> | ||||||
| /// <param name="name">The name of the resource. This name will be used as the connection string name when referenced in a dependency.</param> | ||||||
| /// <param name="port">The host port for Oracle Database.</param> | ||||||
| /// <param name="password">The password for the Oracle Database container. Defaults to a random password.</param> | ||||||
| /// <returns>A reference to the <see cref="IResourceBuilder{OracleDatabaseContainerResource}"/>.</returns> | ||||||
| public static IResourceBuilder<OracleDatabaseContainerResource> AddOracleDatabaseContainer(this IDistributedApplicationBuilder builder, string name, int? port = null, string? password = null) | ||||||
| { | ||||||
| password = password ?? Guid.NewGuid().ToString("N").Substring(0, 30); | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We should centralize this generated password logic into
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Note that it is different than SqlServer.
and MySql
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We should probably just come up with a decent password generator that we can use. What I did for SQL Server was half baked and really only appropriate for inner loop (which luckily is where it is used). I'd be happy to see this go in without change this password generation logic and instead see an issue created where we properly develop out PasswordUtils than can meet our various use cases (needs to be flexible enough to side step various limitations around what the containerized tool can used, and what we can escape in the connection string.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I know an issue hasn't been created yet, but I saw this comment and opened a PR that might address it: #1469. |
||||||
| var oracleDatabaseContainer = new OracleDatabaseContainerResource(name, password); | ||||||
| return builder.AddResource(oracleDatabaseContainer) | ||||||
| .WithManifestPublishingCallback(context => WriteOracleDatabaseContainerResourceToManifest(context, oracleDatabaseContainer)) | ||||||
| .WithAnnotation(new ServiceBindingAnnotation(ProtocolType.Tcp, port: port, containerPort: 1521)) | ||||||
| .WithAnnotation(new ContainerImageAnnotation { Image = "database/free", Tag = "latest", Registry = "container-registry.oracle.com" }) | ||||||
| .WithEnvironment(context => | ||||||
| { | ||||||
| if (context.PublisherName == "manifest") | ||||||
| { | ||||||
| context.EnvironmentVariables.Add(PasswordEnvVarName, $"{{{oracleDatabaseContainer.Name}.inputs.password}}"); | ||||||
| } | ||||||
| else | ||||||
| { | ||||||
| context.EnvironmentVariables.Add(PasswordEnvVarName, oracleDatabaseContainer.Password); | ||||||
| } | ||||||
| }); | ||||||
| } | ||||||
|
|
||||||
| /// <summary> | ||||||
| /// Adds a Oracle Database resource to the application model. A container is used for local development. | ||||||
| /// </summary> | ||||||
| /// <param name="builder">The <see cref="IDistributedApplicationBuilder"/>.</param> | ||||||
| /// <param name="name">The name of the resource. This name will be used as the connection string name when referenced in a dependency.</param> | ||||||
| /// <returns>A reference to the <see cref="IResourceBuilder{OracleDatabaseServerResource}"/>.</returns> | ||||||
| public static IResourceBuilder<OracleDatabaseServerResource> AddOracleDatabase(this IDistributedApplicationBuilder builder, string name) | ||||||
| { | ||||||
| var password = Guid.NewGuid().ToString("N").Substring(0, 30); | ||||||
| var oracleDatabaseServer = new OracleDatabaseServerResource(name, password); | ||||||
| return builder.AddResource(oracleDatabaseServer) | ||||||
| .WithManifestPublishingCallback(WriteOracleDatabaseContainerToManifest) | ||||||
| .WithAnnotation(new ServiceBindingAnnotation(ProtocolType.Tcp, containerPort: 1521)) | ||||||
| .WithAnnotation(new ContainerImageAnnotation { Image = "database/free", Tag = "latest", Registry = "container-registry.oracle.com" }) | ||||||
| .WithEnvironment(PasswordEnvVarName, () => oracleDatabaseServer.Password); | ||||||
| } | ||||||
|
|
||||||
| /// <summary> | ||||||
| /// Adds a Oracle Database database to the application model. | ||||||
| /// </summary> | ||||||
| /// <param name="builder">The Oracle Database server resource builder.</param> | ||||||
| /// <param name="name">The name of the resource. This name will be used as the connection string name when referenced in a dependency.</param> | ||||||
| /// <returns>A reference to the <see cref="IResourceBuilder{OracleDatabaseResource}"/>.</returns> | ||||||
| public static IResourceBuilder<OracleDatabaseResource> AddDatabase(this IResourceBuilder<IOracleDatabaseParentResource> builder, string name) | ||||||
| { | ||||||
| var oracleDatabase = new OracleDatabaseResource(name, builder.Resource); | ||||||
| return builder.ApplicationBuilder.AddResource(oracleDatabase) | ||||||
| .WithManifestPublishingCallback(context => WriteOracleDatabaseToManifest(context, oracleDatabase)); | ||||||
| } | ||||||
|
|
||||||
| private static void WriteOracleDatabaseContainerToManifest(ManifestPublishingContext context) | ||||||
| { | ||||||
| context.Writer.WriteString("type", "oracle.server.v0"); | ||||||
| } | ||||||
|
|
||||||
| private static void WriteOracleDatabaseToManifest(ManifestPublishingContext context, OracleDatabaseResource oracleDatabase) | ||||||
| { | ||||||
| context.Writer.WriteString("type", "oracle.database.v0"); | ||||||
| context.Writer.WriteString("parent", oracleDatabase.Parent.Name); | ||||||
| } | ||||||
|
|
||||||
| private static void WriteOracleDatabaseContainerResourceToManifest(ManifestPublishingContext context, OracleDatabaseContainerResource resource) | ||||||
| { | ||||||
| context.WriteContainer(resource); | ||||||
| context.Writer.WriteString( // "connectionString": "...", | ||||||
| "connectionString", | ||||||
| $"user id=system;password={{{resource.Name}.inputs.password}};data source={{{resource.Name}.bindings.tcp.host}}:{{{resource.Name}.bindings.tcp.port}};"); | ||||||
| context.Writer.WriteStartObject("inputs"); // "inputs": { | ||||||
| context.Writer.WriteStartObject("password"); // "password": { | ||||||
| context.Writer.WriteString("type", "string"); // "type": "string", | ||||||
| context.Writer.WriteBoolean("secret", true); // "secret": true, | ||||||
| context.Writer.WriteStartObject("default"); // "default": { | ||||||
| context.Writer.WriteStartObject("generate"); // "generate": { | ||||||
| context.Writer.WriteNumber("minLength", 10); // "minLength": 10, | ||||||
| context.Writer.WriteEndObject(); // } | ||||||
| context.Writer.WriteEndObject(); // } | ||||||
| context.Writer.WriteEndObject(); // } | ||||||
| context.Writer.WriteEndObject(); // } | ||||||
| } | ||||||
| } | ||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,31 @@ | ||
| // Licensed to the .NET Foundation under one or more agreements. | ||
| // The .NET Foundation licenses this file to you under the MIT license. | ||
|
|
||
| namespace Aspire.Hosting.ApplicationModel; | ||
|
|
||
| /// <summary> | ||
| /// A resource that represents a Oracle Database container. | ||
| /// </summary> | ||
| /// <param name="name">The name of the resource.</param> | ||
| /// <param name="password">The Oracle Database server password.</param> | ||
| public class OracleDatabaseContainerResource(string name, string password) : ContainerResource(name), IOracleDatabaseParentResource | ||
| { | ||
| public string Password { get; } = password; | ||
|
|
||
| /// <summary> | ||
| /// Gets the connection string for the Oracle Database server. | ||
| /// </summary> | ||
| /// <returns>A connection string for the Oracle Database server in the form "user id=system;password=password;data source=localhost:port".</returns> | ||
| public string? GetConnectionString() | ||
| { | ||
| if (!this.TryGetAllocatedEndPoints(out var allocatedEndpoints)) | ||
| { | ||
| throw new DistributedApplicationException("Expected allocated endpoints!"); | ||
| } | ||
|
|
||
| var allocatedEndpoint = allocatedEndpoints.Single(); // We should only have one endpoint for Oracle Database. | ||
|
|
||
| var connectionString = $"user id=system;password={Password};data source={allocatedEndpoint.Address}:{allocatedEndpoint.Port}"; | ||
eerhardt marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| return connectionString; | ||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,30 @@ | ||
| // Licensed to the .NET Foundation under one or more agreements. | ||
| // The .NET Foundation licenses this file to you under the MIT license. | ||
|
|
||
| namespace Aspire.Hosting.ApplicationModel; | ||
|
|
||
| /// <summary> | ||
| /// A resource that represents a Oracle Database database. This is a child resource of a <see cref="OracleDatabaseContainerResource"/>. | ||
| /// </summary> | ||
| /// <param name="name">The name of the resource.</param> | ||
| /// <param name="oracleParentResource">The Oracle Database parent resource associated with this database.</param> | ||
| public class OracleDatabaseResource(string name, IOracleDatabaseParentResource oracleParentResource) : Resource(name), IResourceWithParent<IOracleDatabaseParentResource>, IResourceWithConnectionString | ||
| { | ||
| public IOracleDatabaseParentResource Parent { get; } = oracleParentResource; | ||
|
|
||
| /// <summary> | ||
| /// Gets the connection string for the Oracle Database. | ||
| /// </summary> | ||
| /// <returns>A connection string for the Oracle Database.</returns> | ||
| public string? GetConnectionString() | ||
| { | ||
| if (Parent.GetConnectionString() is { } connectionString) | ||
| { | ||
| return $"{connectionString}/{Name}"; | ||
| } | ||
| else | ||
| { | ||
| throw new DistributedApplicationException("Parent resource connection string was null."); | ||
| } | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,33 @@ | ||
| // Licensed to the .NET Foundation under one or more agreements. | ||
| // The .NET Foundation licenses this file to you under the MIT license. | ||
|
|
||
| using Aspire.Hosting.Utils; | ||
|
|
||
| namespace Aspire.Hosting.ApplicationModel; | ||
|
|
||
| /// <summary> | ||
| /// A resource that represents a Oracle Database container. | ||
| /// </summary> | ||
| /// <param name="name">The name of the resource.</param> | ||
| /// <param name="password">The Oracle Database server password.</param> | ||
| public class OracleDatabaseServerResource(string name, string password) : Resource(name), IOracleDatabaseParentResource | ||
| { | ||
| public string Password { get; } = password; | ||
|
|
||
| /// <summary> | ||
| /// Gets the connection string for the Oracle Database server. | ||
| /// </summary> | ||
| /// <returns>A connection string for the Oracle Database server in the form "user id=system;password=password;data source=host:port".</returns> | ||
| public string? GetConnectionString() | ||
| { | ||
| if (!this.TryGetAllocatedEndPoints(out var allocatedEndpoints)) | ||
| { | ||
| throw new DistributedApplicationException("Expected allocated endpoints!"); | ||
| } | ||
|
|
||
| var allocatedEndpoint = allocatedEndpoints.Single(); // We should only have one endpoint for Oracle. | ||
|
|
||
| var connectionString = $"user id=system;password={PasswordUtil.EscapePassword(Password)};data source={allocatedEndpoint.Address}:{allocatedEndpoint.Port}"; | ||
| return connectionString; | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,24 @@ | ||
| <Project Sdk="Microsoft.NET.Sdk"> | ||
|
|
||
| <PropertyGroup> | ||
| <TargetFramework>$(NetCurrent)</TargetFramework> | ||
| <IsPackable>true</IsPackable> | ||
| <PackageTags>$(ComponentEfCorePackageTags) oracle sql</PackageTags> | ||
| <Description>An Oracle Database provider for Entity Framework Core that integrates with Aspire, including connection pooling, health check, logging, and telemetry.</Description> | ||
| </PropertyGroup> | ||
|
|
||
| <ItemGroup> | ||
| <Compile Include="..\Common\HealthChecksExtensions.cs" Link="HealthChecksExtensions.cs" /> | ||
| </ItemGroup> | ||
|
|
||
| <ItemGroup> | ||
| <PackageReference Include="Microsoft.Extensions.Configuration.Binder" /> | ||
| <PackageReference Include="Microsoft.Extensions.Diagnostics.HealthChecks.EntityFrameworkCore" /> | ||
| <PackageReference Include="Microsoft.Extensions.Hosting.Abstractions" /> | ||
| <PackageReference Include="OpenTelemetry.Extensions.Hosting" /> | ||
| <PackageReference Include="OpenTelemetry.Instrumentation.EntityFrameworkCore" /> | ||
| <PackageReference Include="OpenTelemetry.Instrumentation.EventCounters" /> | ||
| <PackageReference Include="Oracle.EntityFrameworkCore" /> | ||
| </ItemGroup> | ||
|
|
||
| </Project> |
Uh oh!
There was an error while loading. Please reload this page.