Skip to content

v2/v3: Replace very old method of creating a LoggerFactory#360

Merged
TimHess merged 4 commits intomainfrom
config-server-logging
Mar 26, 2025
Merged

v2/v3: Replace very old method of creating a LoggerFactory#360
TimHess merged 4 commits intomainfrom
config-server-logging

Conversation

@TimHess
Copy link
Member

@TimHess TimHess commented Mar 7, 2025

Update Logging-related content for Config Server client

@github-actions
Copy link

github-actions bot commented Mar 7, 2025

Preview link: https://docs-steeltoe-pr-360.azurewebsites.net

  • Your changes have been deployed to the preview site. The preview site will update as you add more commits to this branch.
  • The preview link is shareable, but will be deleted when the pull request is merged or closed.

Need to pair with a custom main site address?

A team member can use this az CLI command, replacing and for valid numbers:

az webapp config appsettings set --resource-group steeltoe --name docs-steeltoe --slot pr-<this-PR-number> --settings mainsite_host=https://www-steeltoe-pr-<other-pr-number>.azurewebsites.net

This is an automated message.

@TimHess TimHess force-pushed the config-server-logging branch from e6f7d3a to 039b6a3 Compare March 21, 2025 19:27
@TimHess TimHess marked this pull request as ready for review March 21, 2025 19:27
@TimHess TimHess requested a review from bart-vmware March 24, 2025 21:19
Copy link
Member

@bart-vmware bart-vmware left a comment

Choose a reason for hiding this comment

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

Updated link text, LGTM otherwise.

Co-authored-by: Bart Koelman <104792814+bart-vmware@users.noreply.github.com>
@TimHess TimHess requested a review from bart-vmware March 26, 2025 13:37
@TimHess TimHess merged commit 74b30fd into main Mar 26, 2025
6 checks passed
@TimHess TimHess deleted the config-server-logging branch March 26, 2025 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants