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

[BUG] API diagnostics are not being updated on deployment #703

Open
tomsauret21 opened this issue Nov 12, 2024 · 3 comments
Open

[BUG] API diagnostics are not being updated on deployment #703

tomsauret21 opened this issue Nov 12, 2024 · 3 comments

Comments

@tomsauret21
Copy link

Release version

V6.0.1.3

Describe the bug

When deploying an API, the value from the JSON file is being used and the value from the configuration file is being ignored. This is causing deployments to fail if the API has the diagnostic settings enabled. This is the same issue that was reported in #672

Expected behavior

Be able to deploy APIs with diagnostic settings into other environments

Actual behavior

The deployment fails out with error it cannot find the logger ID in that environment

Reproduction Steps

Check the box in the settings of the API to enable diagnostics on the API
Extract the API and ensure it has a diagnostics folder with it
Update the config file to have the diagnostics setting for the environment you are deploying to
Run the publisher and wait for the error

Copy link

  Thank you for opening this issue! Please be patient while we will look into it and get back to you as this is an open source project. In the meantime make sure you take a look at the [closed issues](https://github.com/Azure/apiops/issues?q=is%3Aissue+is%3Aclosed) in case your question has already been answered. Don't forget to provide any additional information if needed (e.g. scrubbed logs, detailed feature requests,etc.).
  Whenever it's feasible, please don't hesitate to send a Pull Request (PR) our way. We'd greatly appreciate it, and we'll gladly assess and incorporate your changes.

@LukaszHryniewiecki
Copy link

I can confirm it seems that issue still exists in newest release

@mathim
Copy link

mathim commented Dec 18, 2024

This issue is also blocking is from upgrading to APIOps v6

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

No branches or pull requests

3 participants