Skip to content
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

Azure to Core 4.8.x #817

Open
wants to merge 17 commits into
base: 5.9.x
Choose a base branch
from
Open

Azure to Core 4.8.x #817

wants to merge 17 commits into from

Conversation

sdelamo
Copy link
Contributor

@sdelamo sdelamo commented Feb 17, 2025

No description provided.

@sdelamo sdelamo requested a review from yawkat February 17, 2025 15:07
@sdelamo sdelamo added the type: dependency-upgrade Upgrade a dependency label Feb 17, 2025
test testErrorHandlerWithFluxChunkedSignaledDelayedError  fails
`io.micronaut.http.server.tck.tests.forms.FormsJacksonAnnotationsTest#httpClientFormSubmissionsDoesNotSupportJacksonAnnotations` test fails
@@ -32,6 +33,7 @@ class AzureLoggingSpec extends Specification {
@Inject
ApplicationConfiguration applicationConfiguration

@PendingFeature
void 'test Azure logging'() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

passes for me

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok it breaks when running all tests at the same time. the test just looks broken to me. it relies on AzureAppender being initialized only once, but if the appender was initialized in another test, it breaks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: dependency-upgrade Upgrade a dependency
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants