Skip to content

build(deps): bump github.com/go-resty/resty/v2 from 2.17.1 to 2.17.2 in the meraki group across 1 directory#50162

Closed
dependabot[bot] wants to merge 2 commits into
mainfrom
dependabot/go_modules/main/meraki-4e4938d4bd
Closed

build(deps): bump github.com/go-resty/resty/v2 from 2.17.1 to 2.17.2 in the meraki group across 1 directory#50162
dependabot[bot] wants to merge 2 commits into
mainfrom
dependabot/go_modules/main/meraki-4e4938d4bd

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Apr 16, 2026

Copy link
Copy Markdown
Contributor

Bumps the meraki group with 1 update in the / directory: github.com/go-resty/resty/v2.

Updates github.com/go-resty/resty/v2 from 2.17.1 to 2.17.2

Release notes

Sourced from github.com/go-resty/resty/v2's releases.

v2.17.2

Release Notes

Backport

Release

Full Changelog: go-resty/resty@v2.17.1...v2.17.2

Commits

@dependabot dependabot Bot added automation dependabot GitHub Dependabot Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team labels Apr 16, 2026
@dependabot dependabot Bot requested a review from a team as a code owner April 16, 2026 14:00
@dependabot dependabot Bot added automation Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team dependabot GitHub Dependabot labels Apr 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@mergify

mergify Bot commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @dependabot[bot]? 🙏.
For such, you'll need to label your PR with:

  • The upcoming major version of the Elastic Stack
  • The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@coderabbitai

coderabbitai Bot commented Apr 16, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 91212959-684e-4954-970b-19c0525d2585

📥 Commits

Reviewing files that changed from the base of the PR and between c828da9 and 2ab42fe.

📒 Files selected for processing (1)
  • NOTICE.txt
✅ Files skipped from review due to trivial changes (1)
  • NOTICE.txt

📝 Walkthrough

Walkthrough

The pull request updates the github.com/go-resty/resty/v2 dependency from v2.17.1 to v2.17.2. The go.mod entry for that module was changed accordingly, and the NOTICE.txt reference to the module’s license file was updated to reflect the new version. No other module requirements, directives, or exported/public declarations were modified.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dependabot/go_modules/main/meraki-4e4938d4bd
  • 🛠️ Update Documentation: Commit on current branch
  • 🛠️ Update Documentation: Create PR

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions

This comment has been minimized.

@dependabot dependabot Bot changed the title build(deps): bump github.com/go-resty/resty/v2 from 2.17.1 to 2.17.2 in the meraki group build(deps): bump github.com/go-resty/resty/v2 from 2.17.1 to 2.17.2 in the meraki group across 1 directory Apr 17, 2026
@dependabot dependabot Bot force-pushed the dependabot/go_modules/main/meraki-4e4938d4bd branch from 1e0c970 to 1738f4f Compare April 17, 2026 03:05
@github-actions

This comment has been minimized.

@dependabot dependabot Bot force-pushed the dependabot/go_modules/main/meraki-4e4938d4bd branch from c24fc56 to c828da9 Compare April 17, 2026 10:25
@github-actions

This comment has been minimized.

@dependabot dependabot Bot force-pushed the dependabot/go_modules/main/meraki-4e4938d4bd branch from 2ab42fe to 0761515 Compare April 20, 2026 11:58
@dependabot dependabot Bot force-pushed the dependabot/go_modules/main/meraki-4e4938d4bd branch 4 times, most recently from 8e23893 to 6ebc7f3 Compare April 24, 2026 10:25
@github-actions

This comment has been minimized.

@dependabot dependabot Bot force-pushed the dependabot/go_modules/main/meraki-4e4938d4bd branch 4 times, most recently from 57d0a2f to dc46bd7 Compare May 1, 2026 12:01
@dependabot dependabot Bot force-pushed the dependabot/go_modules/main/meraki-4e4938d4bd branch from d2df65a to 6f8a3c1 Compare May 4, 2026 12:43
@github-actions

github-actions Bot commented May 4, 2026

Copy link
Copy Markdown
Contributor

TL;DR

The Buildkite failure is isolated to winlogbeat/eventlog on Windows and does not match this PR’s resty dependency bump. The failing test (TestWindowsEventLogAPI) aborts with Access is denied. when opening the test event source.

Remediation

  • Re-run the failing Windows job first (this looks like an environment/permission-sensitive test failure outside the changed files).
  • If it reproduces, harden winlogbeat/eventlog/wineventlog_test.go by handling access-denied setup failures explicitly (for example, skip with a clear message when event source creation/open is blocked on the worker image).
Investigation details

Root Cause

The failure occurs in Winlogbeat test setup while creating/opening a Windows Event Log source, not in dependency-resolution or resty call paths.

  • Failing callsite: winlogbeat/eventlog/wineventlog_test.go:336-341
    • log, err = eventlog.Open(source)
    • t.Fatal(err) on error
  • PR changes are only dependency metadata updates (go.mod, go.sum, NOTICE.txt), with no winlogbeat code changes.

Evidence

  • Build: https://buildkite.com/elastic/beats/builds/45371
  • Job/step: Winlogbeat: Win 2025 Unit Tests (beats-winlogbeat build 28728)
  • Key log excerpt:
    • === FAIL: winlogbeat/eventlog TestWindowsEventLogAPI (0.08s)
    • wineventlog_test.go:341: Access is denied.

Verification

  • Not run locally: this runner is Linux, and the failing test path is Windows-specific (eventlog API).

Follow-up

If this starts recurring on Win 2025 workers, consider making setup failures deterministic in test output (explicit skip reason on permission denial) to reduce false-red CI for dependency-only PRs.


What is this? | From workflow: PR Buildkite Detective

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

@dependabot dependabot Bot force-pushed the dependabot/go_modules/main/meraki-4e4938d4bd branch from f31bf4e to 3f4d5ef Compare May 5, 2026 10:25
Bumps the meraki group with 1 update in the / directory: [github.com/go-resty/resty/v2](https://github.com/go-resty/resty).


Updates `github.com/go-resty/resty/v2` from 2.17.1 to 2.17.2
- [Release notes](https://github.com/go-resty/resty/releases)
- [Commits](go-resty/resty@v2.17.1...v2.17.2)

---
updated-dependencies:
- dependency-name: github.com/go-resty/resty/v2
  dependency-version: 2.17.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: meraki
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot force-pushed the dependabot/go_modules/main/meraki-4e4938d4bd branch from 078559b to bfccf80 Compare May 6, 2026 10:27
@dependabot @github

dependabot Bot commented on behalf of github May 6, 2026

Copy link
Copy Markdown
Contributor Author

Looks like github.com/go-resty/resty/v2 is no longer updatable, so this is no longer needed.

@dependabot dependabot Bot closed this May 6, 2026
@dependabot dependabot Bot deleted the dependabot/go_modules/main/meraki-4e4938d4bd branch May 6, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation dependabot GitHub Dependabot Team:Obs-InfraObs Label for the Observability Infrastructure Monitoring team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants