-
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 4 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 IOracleDatabaseResource : IResourceWithConnectionString | ||
| { | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,81 @@ | ||
| // 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_DATABASE_PASSWORD"; | ||
|
|
||
| /// <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"); | ||
| var oracleDatabaseContainer = new OracleDatabaseContainerResource(name, password); | ||
| return builder.AddResource(oracleDatabaseContainer) | ||
| .WithAnnotation(new ManifestPublishingCallbackAnnotation(WriteOracleDatabaseContainerToManifest)) | ||
| .WithAnnotation(new ServiceBindingAnnotation(ProtocolType.Tcp, port: port, containerPort: 1521)) | ||
| .WithAnnotation(new ContainerImageAnnotation { Image = "database/free", Tag = "latest", Registry = "container-registry.oracle.com" }) | ||
| .WithEnvironment(PasswordEnvVarName, () => oracleDatabaseContainer.Password); | ||
| } | ||
|
|
||
| /// <summary> | ||
| /// Adds a Oracle Database connection to the application model. Connection strings can also be read from the connection string section of the configuration using the name of the resource. | ||
| /// </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="connectionString">The Oracle Database connection string (optional).</param> | ||
| /// <returns>A reference to the <see cref="IResourceBuilder{OracleDatabaseConnectionResource}"/>.</returns> | ||
| public static IResourceBuilder<OracleDatabaseConnectionResource> AddOracleDatabaseConnection(this IDistributedApplicationBuilder builder, string name, string? connectionString = null) | ||
| { | ||
| var oracleDatabaseConnection = new OracleDatabaseConnectionResource(name, connectionString); | ||
|
|
||
| return builder.AddResource(oracleDatabaseConnection) | ||
| .WithAnnotation(new ManifestPublishingCallbackAnnotation((context) => WriteOracleDatabaseConnectionToManifest(context, oracleDatabaseConnection))); | ||
| } | ||
|
|
||
| /// <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<OracleDatabaseContainerResource> builder, string name) | ||
| { | ||
| var oracleDatabase = new OracleDatabaseResource(name, builder.Resource); | ||
| return builder.ApplicationBuilder.AddResource(oracleDatabase) | ||
| .WithAnnotation(new ManifestPublishingCallbackAnnotation( | ||
| (json) => WriteOracleDatabaseToManifest(json, oracleDatabase))); | ||
| } | ||
|
|
||
| private static void WriteOracleDatabaseConnectionToManifest(ManifestPublishingContext context, OracleDatabaseConnectionResource oracleDatabaseConnection) | ||
| { | ||
| context.Writer.WriteString("type", "oracle.connection.v0"); | ||
| context.Writer.WriteString("connectionString", oracleDatabaseConnection.GetConnectionString()); | ||
| } | ||
|
|
||
| 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); | ||
| } | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| // 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 connection. | ||
| /// </summary> | ||
| /// <param name="name">The name of the resource.</param> | ||
| /// <param name="connectionString">The Oracle Database connection string.</param> | ||
| public class OracleDatabaseConnectionResource(string name, string? connectionString) : Resource(name), IOracleDatabaseResource | ||
|
||
| { | ||
| private readonly string? _connectionString = connectionString; | ||
|
|
||
| /// <summary> | ||
| /// Gets the connection string for the Oracle Database server. | ||
| /// </summary> | ||
| /// <returns>The specified connection string.</returns> | ||
| public string? GetConnectionString() => _connectionString; | ||
| } | ||
| 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), IOracleDatabaseResource | ||
| { | ||
| 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="oracleContainer">The Oracle Database server resource associated with this database.</param> | ||
| public class OracleDatabaseResource(string name, OracleDatabaseContainerResource oracleContainer) : Resource(name), IOracleDatabaseResource, IResourceWithParent<OracleDatabaseContainerResource> | ||
| { | ||
| public OracleDatabaseContainerResource Parent { get; } = oracleContainer; | ||
|
|
||
| /// <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,24 @@ | ||||||||||
| <Project Sdk="Microsoft.NET.Sdk"> | ||||||||||
|
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. Why the
Contributor
Author
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 tried to follow the existing pattern of other EF components where we have Aspire.X.EntityFrameworkCore.Y where: X is the company/project Oracle calls its product Oracle Database, so I followed this pattern to leave it open for other Oracle products that may support EntityFramework.
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. Fair enough. I'm happy with that as a justification as long as @eerhardt is happy as the czar of Aspire components ;)
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. The naming pattern we use is in aspire/src/Components/README.md Lines 27 to 30 in 23e168d
This should be called
Contributor
Author
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. Done |
||||||||||
|
|
||||||||||
| <PropertyGroup> | ||||||||||
| <TargetFramework>$(NetCurrent)</TargetFramework> | ||||||||||
| <IsPackable>true</IsPackable> | ||||||||||
| <PackageTags>$(ComponentEfCorePackageTags) sqlserver sql</PackageTags> | ||||||||||
eerhardt marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||||||||||
| <Description>A Oracle Database provider for Entity Framework Core that integrates with Aspire, including connection pooling, health check, logging, and telemetry.</Description> | ||||||||||
eerhardt marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||||||||||
| </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.