Skip to content

License issue warning after update to 0.9.4 (audit logging is enabled) #4383

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.

Sign up for GitHub

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

Jump to bottom
Closed
ntimo opened this issue Oct 6, 2022 · 1 comment · Fixed by #4387
Closed

License issue warning after update to 0.9.4 (audit logging is enabled) #4383

ntimo opened this issue Oct 6, 2022 · 1 comment · Fixed by #4387
Assignees
@kylecarbs
Labels
s0 Major regression, all-hands-on-deck to fix

Comments

@ntimo
Copy link
Contributor

ntimo commented Oct 6, 2022

Description

I updated my coder setup to version 0.9.4 and now when I view the dashboard I see a huge license violation warning because the audit feature is enabled.
image
I did not enabled this feature nor would I know how after a license.

Desired solution

The warning should not appear when the feature is not actively configured or used.

My coder env settings:

      CODER_PG_CONNECTION_URL: "postgresql://{{ coder__postgres_user }}:{{ coder__postgres_password }}@postgres/{{ coder__postgres_db }}?sslmode=disable"
      CODER_ADDRESS: "0.0.0.0:7080"
      CODER_ACCESS_URL: "{{ coder__url_with_proto }}"
      CODER_WILDCARD_ACCESS_URL: "{{ coder__wildcard_url }}"
      CODER_PROMETHEUS_ENABLE: "true"
      CODER_PROMETHEUS_ADDRESS: "0.0.0.0:7090"
      CODER_TAILSCALE: true
@ntimo ntimo changed the title License issue warning after update to 0.9.4 License issue warning after update to 0.9.4 (audit logging is enabled) Oct 6, 2022
@kylecarbs
Copy link
Member

Oofta, we'll be changing the way we dogfood to make sure we fix this.

kylecarbs added a commit that referenced this issue Oct 6, 2022
@kylecarbs
fix: Remove audit warning if unlicensed
888f592 
Fixes #4383.
@kylecarbs kylecarbs mentioned this issue Oct 6, 2022
Merged
@kylecarbs kylecarbs added the s0 Major regression, all-hands-on-deck to fix label Oct 6, 2022
kylecarbs added a commit that referenced this issue Oct 6, 2022
@kylecarbs
fix: Remove audit warning if unlicensed
77d4c7a 
Fixes #4383.
kylecarbs added a commit that referenced this issue Oct 6, 2022
@kylecarbs
fix: Remove audit warning if unlicensed
8a93572 
Fixes #4383.
@kylecarbs kylecarbs self-assigned this Oct 6, 2022
@matifali matifali added the bug label May 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kylecarbs kylecarbs

Labels
s0 Major regression, all-hands-on-deck to fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Remove audit warning if unlicensed coder/coder
3 participants
@ntimo @kylecarbs @matifali