-
Notifications
You must be signed in to change notification settings - Fork 3
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
refactor(driver)!: drop otel libs (#127)
## Description <!-- What code changes are made? What problem does this PR addresses, or what feature this PR adds? --> This pull request removes dependency on OTEL instrumentation, to reduce binary size and simplify dependencies. Additionally it updates reference to new COSI repository, so that CRD and Controller can be deployed. <!-- Usage: `Resolves #<issue number>`, or `Resolves <link to the issue>`. If PR is about `failing-tests`, please post the related tests in a comment and do not use `Resolves` --> N/A Signed-off-by: Mateusz Urbanek <[email protected]>
- Loading branch information
Showing
28 changed files
with
113 additions
and
919 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,94 +1,60 @@ | ||
| run: | ||
| # Timeout for analysis, e.g. 30s, 5m. | ||
| timeout: 5m | ||
|
|
||
| # Include test files. | ||
| tests: true | ||
|
|
||
| # List of build tags to pass to all linters. | ||
| build-tags: | ||
| - integration | ||
|
|
||
| issues: | ||
| # Set to 0 to not skip any issues. | ||
| max-issues-per-linter: 0 | ||
|
|
||
| # Set to 0 to not skip any issues. | ||
| max-same-issues: 0 | ||
|
|
||
| output: | ||
| # Sort results by: filepath, then line, then column. | ||
| sort-results: true | ||
|
|
||
| # Make issues output unique by line. | ||
| uniq-by-line: false | ||
|
|
||
| linters: | ||
| # Enable specific linter | ||
| enable: | ||
| # Detect context.Context contained in structs. | ||
| - containedctx | ||
| # Check whether a function uses a non-inherited context. | ||
| - contextcheck | ||
| # Find declarations and assignments with too many blank identifiers. | ||
| - dogsled | ||
| # Check for unchecked errors. | ||
| - errcheck | ||
| # Find code that will cause problems with the error wrapping scheme. | ||
| - errorlint | ||
| # Inspects source code for security problems. | ||
| - gci | ||
| - gosec | ||
| # Check that compiler directives are valid. | ||
| - gocheckcompilerdirectives | ||
| # Calculate cognitive complexities of functions. | ||
| - gocognit | ||
| # Find repeated strings that could be replaced by a constant. | ||
| - goconst | ||
| # Provides functionalities missing from other linters. | ||
| - gocritic | ||
| # Calculates cyclomatic complexity of a function. | ||
| - gocyclo | ||
| # Check if comments end with a dot. | ||
| - godot | ||
| # A stricter replacement for gofmt. | ||
| - gofumpt | ||
| # Simplify the code. | ||
| - gosimple | ||
| # Check for correctness of programs. | ||
| - govet | ||
| # Detect ineffectual assignments. | ||
| - ineffassign | ||
| # Correct commonly misspelled English words in source files. | ||
| - misspell | ||
| # Magic Number Detector. | ||
| - mnd | ||
| # Finds the code that returns nil even if it checks that the error is not nil. | ||
| - nilerr | ||
| # Checks that there is no simultaneous return of nil error and an invalid value. | ||
| - nilnil | ||
| # Find incorrect usages of t.Parallel(). | ||
| - paralleltest | ||
| # Reports direct reads from proto message fields when getters should be used. | ||
| - protogetter | ||
| # Drop-in replacement of golint. | ||
| - revive | ||
| # Ensure consistent code style when using log/slog. | ||
| - sloglint | ||
| # Find bugs and performance issues statically. | ||
| - staticcheck | ||
| # Checks Go code for unused constants, variables, functions and types. | ||
| - unused | ||
| # Empty lines linter. | ||
| - wsl | ||
|
|
||
| # Setting of specific linters. | ||
| linters-settings: | ||
| paralleltest: | ||
| # Ignore missing calls to `t.Parallel()` and only report incorrect uses of it. | ||
| ignore-missing: false | ||
|
|
||
| sloglint: | ||
| # Enforce using key-value pairs only (incompatible with attr-only). | ||
| kv-only: true | ||
| # Enforce a single key naming convention. | ||
| key-naming-case: snake | ||
|
|
||
| gci: | ||
| sections: | ||
| - standard | ||
| - default | ||
| - blank | ||
| - dot | ||
| - prefix(github.com/linode) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.