Merge main into proj/linodego_v2.0

.github/workflows/ci.yml

@@ -17,13 +17,18 @@ jobs:
           go-version: stable
 
       - name: golangci-lint
-        uses: golangci/golangci-lint-action@v6
+        uses: golangci/golangci-lint-action@v8
         with:
           version: latest
 
       - name: Run tidy
         run: make tidy
 
+      - name: Run formatter
+        run: |
+          go install mvdan.cc/gofumpt@86bffd62437a3c437c0b84d5d5ab244824e762fc
+          gofumpt -l -w .
+
       - name: Fail if changes
         run: git diff-index --exit-code HEAD
 
@@ -85,7 +90,7 @@ jobs:
 
     steps:
       - name: Notify Slack
-        uses: slackapi/slack-github-action@v2.0.0
+        uses: slackapi/slack-github-action@v2.1.0
         with:
           method: chat.postMessage
           token: ${{ secrets.SLACK_BOT_TOKEN }}

.github/workflows/labeler.yml

@@ -21,7 +21,7 @@ jobs:
         uses: actions/checkout@v4
       -
         name: Run Labeler
-        uses: crazy-max/ghaction-github-labeler@31674a3852a9074f2086abcf1c53839d466a47e7
+        uses: crazy-max/ghaction-github-labeler@24d110aa46a59976b8a7f35518cb7f14f434c916
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
           yaml-file: .github/labels.yml

.github/workflows/nightly_smoke_tests.yml

@@ -38,7 +38,7 @@ jobs:
 
       - name: Notify Slack
         if: (success() || failure()) && github.repository == 'linode/linodego'
-        uses: slackapi/slack-github-action@v2.0.0
+        uses: slackapi/slack-github-action@v2.1.0
         with:
           method: chat.postMessage
           token: ${{ secrets.SLACK_BOT_TOKEN }}

.github/workflows/release-notify-slack.yml

@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Notify Slack - Main Message
-        uses: slackapi/slack-github-action@v2.0.0
+        uses: slackapi/slack-github-action@v2.1.0
         with:
           method: chat.postMessage
           token: ${{ secrets.SLACK_BOT_TOKEN }}

.golangci.yml

@@ -1,75 +1,85 @@
+version: "2"
 run:
   tests: false
-
-linters-settings:
-  errcheck:
-    check-type-assertions: true
-    check-blank: true
-
-  govet:
-    enable:
-      - atomicalign
-      - shadow
-    enable-all: false
-    disable-all: false
-  gocyclo:
-    min-complexity: 30
-  gocognit:
-    min-complexity: 30
-  dupl:
-    threshold: 100
-
 linters:
-  enable-all: true
+  default: all
   disable:
-    # deprecated linters
-    - exportloopref
-    ####################
-
-    # conflicted with go fmt
-    - nolintlint
-
-    # workaround to avoid linter failures of getting malformed json
-    - musttag
-
-    - err113
-
     - bodyclose
     - contextcheck
-    - nilerr
-    - noctx
-    - rowserrcheck
-    - sqlclosecheck
-    - tparallel
+    - cyclop
+    - depguard
+    - dupl
+    - durationcheck
+    - err113
+    - errcheck
     - errname
+    - errorlint
+    - exhaustive
+    - exhaustruct
     - forcetypeassert
+    - gochecknoglobals
+    - gochecknoinits
     - gocyclo
-    - unparam
-    - nakedret
-    - lll
-    - dupl
+    - godot
     - gosec
-    - gochecknoinits
-    - gochecknoglobals
-    - exhaustruct
+    - inamedparam
+    - lll
+    - musttag
+    - nakedret
+    - nilerr
+    - nlreturn
+    - noctx
+    - nolintlint
     - nonamedreturns
-    - errcheck
+    - perfsprint
+    - recvcheck
+    - rowserrcheck
+    - sqlclosecheck
     - staticcheck
-    - stylecheck
-    - wsl
-    - varnamelen
+    - tagalign
     - tagliatelle
-    - nlreturn
-    - wrapcheck
+    - tparallel
+    - unparam
+    - varnamelen
     - wastedassign
-    - durationcheck
-    - errorlint
-    - cyclop
-    - godot
-    - exhaustive
-    - depguard
-    - tagalign
-    - inamedparam
-    - perfsprint
-    - recvcheck
-  fast: false
+    - wrapcheck
+    - wsl
+  settings:
+    dupl:
+      threshold: 100
+    errcheck:
+      check-type-assertions: true
+      check-blank: true
+    gocognit:
+      min-complexity: 30
+    gocyclo:
+      min-complexity: 30
+    govet:
+      enable:
+        - atomicalign
+        - shadow
+      enable-all: false
+      disable-all: false
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    paths:
+      - third_party$
+      - builtin$
+      - examples$
+formatters:
+  enable:
+    - gci
+    - gofmt
+    - gofumpt
+    - goimports
+  exclusions:
+    generated: lax
+    paths:
+      - third_party$
+      - builtin$
+      - examples$

Makefile

@@ -4,6 +4,7 @@ BIN_DIR := $(GOPATH)/bin
 INTEGRATION_DIR := ./test/integration
 FIXTURES_DIR    := $(INTEGRATION_DIR)/fixtures
 
+TEST_TAGS := integration
 TEST_TIMEOUT := 5h
 
 SKIP_DOCKER       ?= 0
@@ -28,7 +29,7 @@ test-unit:
 	cd test && make test-unit
 
 test-int:
-	cd test && make test-int
+	cd test && make test-int TEST_TIMEOUT=$(TEST_TIMEOUT)
 
 testcov-func:
 	@go test -v -coverprofile="coverage.txt" . > /dev/null 2>&1
@@ -40,7 +41,7 @@ testcov-html:
 	@go tool cover -html coverage.txt
 
 test-smoke:
-	cd test && make test-smoke
+	cd test && make test-smoke TEST_TIMEOUT=$(TEST_TIMEOUT)
 
 build: vet lint
 	go build ./...
@@ -77,7 +78,7 @@ run_fixtures:
 	LINODE_API_VERSION="v4beta" \
 	LINODE_URL="$(LINODE_URL)" \
 	GO111MODULE="on" \
-	go test --tags $(TEST_TAGS) -timeout=$(TEST_TIMEOUT) -v $(TEST_ARGS)
+	go test -timeout=$(TEST_TIMEOUT) -v $(TEST_ARGS)
 
 sanitize:
 	@echo "* Sanitizing fixtures"

account_events.go

@@ -72,6 +72,7 @@ const (
 	ActionDatabaseDelete                          EventAction = "database_delete"
 	ActionDatabaseFailed                          EventAction = "database_failed"
 	ActionDatabaseUpdate                          EventAction = "database_update"
+	ActionDatabaseResize                          EventAction = "database_resize"
 	ActionDatabaseCreateFailed                    EventAction = "database_create_failed"
 	ActionDatabaseUpdateFailed                    EventAction = "database_update_failed"
 	ActionDatabaseBackupCreate                    EventAction = "database_backup_create"
@@ -302,6 +303,9 @@ func (c *Client) GetEvent(ctx context.Context, eventID int) (*Event, error) {
 }
 
 // MarkEventRead marks a single Event as read.
+// Deprecated: `MarkEventRead` is a deprecated API, please consider using `MarkEventsSeen` instead.
+// Please note that the `MarkEventsSeen` API functions differently and will mark all events up to and
+// including the referenced event-id as "seen" rather than individual events.
 func (c *Client) MarkEventRead(ctx context.Context, event *Event) error {
 	e := formatAPIPath("account/events/%d/read", event.ID)
 	return doPOSTRequestNoRequestResponseBody(ctx, c, e)