Skip to content

chore(deps): bump github.com/hashicorp/packer-plugin-sdk from 0.6.0 to 0.6.1 #520

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Apr 1, 2025

Bumps github.com/hashicorp/packer-plugin-sdk from 0.6.0 to 0.6.1.

Release notes

Sourced from github.com/hashicorp/packer-plugin-sdk's releases.

v0.6.1

What's Changed

Full Changelog: hashicorp/packer-plugin-sdk@v0.6.0...v0.6.1

Commits
  • d037325 version: cut release v0.6.1
  • 9800105 go.mod: bump go-jose to 4.0.5
  • 99a6159 go.mod: remove toolchain statement
  • 9417a76 deps: update golang depends and bump go version to >=1.23
  • ae49fa1 ci: remove go 1.21 and go 1.22 from test matrix
  • ebb7876 ci: run tests against go 1.23 and 1.24
  • bbf8adc version: prepare v0.6.1-dev
  • 6f42498 version: cut release v0.6.0
  • 6207578 go.mod: tidy up go.mod/go.sum
  • See full diff in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added dependencies go Pull requests that update go code labels Apr 1, 2025
@dependabot dependabot bot requested a review from a team as a code owner April 1, 2025 14:03
@tenthirtyam
Copy link
Collaborator

tenthirtyam commented Apr 1, 2025

This PR will superseded #518 and #519, but needs #517 to allow the validation to pass due to failures in golanglint configuration.

@tenthirtyam tenthirtyam changed the title build(deps): bump github.com/hashicorp/packer-plugin-sdk from 0.6.0 to 0.6.1 chore(deps): bump github.com/hashicorp/packer-plugin-sdk from 0.6.0 to 0.6.1 Apr 1, 2025
@tenthirtyam tenthirtyam added this to the v2.0.0 milestone Apr 1, 2025
Copy link
Collaborator

@tenthirtyam tenthirtyam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove:

toolchain go1.24.1

@tenthirtyam tenthirtyam self-requested a review April 1, 2025 18:42
@tenthirtyam tenthirtyam dismissed their stale review April 1, 2025 18:43

Addressed in bf99742.

@tenthirtyam tenthirtyam force-pushed the dependabot/go_modules/github.com/hashicorp/packer-plugin-sdk-0.6.1 branch 3 times, most recently from 1a04572 to f3bc723 Compare April 2, 2025 15:17
@tenthirtyam
Copy link
Collaborator

@dependabot recreate

@dependabot dependabot bot force-pushed the dependabot/go_modules/github.com/hashicorp/packer-plugin-sdk-0.6.1 branch from f3bc723 to 24c8825 Compare April 2, 2025 15:20
Bumps [github.com/hashicorp/packer-plugin-sdk](https://github.com/hashicorp/packer-plugin-sdk) from 0.6.0 to 0.6.1.
- [Release notes](https://github.com/hashicorp/packer-plugin-sdk/releases)
- [Changelog](https://github.com/hashicorp/packer-plugin-sdk/blob/main/CHANGELOG.md)
- [Commits](hashicorp/packer-plugin-sdk@v0.6.0...v0.6.1)

---
updated-dependencies:
- dependency-name: github.com/hashicorp/packer-plugin-sdk
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
@lbajolet-hashicorp lbajolet-hashicorp force-pushed the dependabot/go_modules/github.com/hashicorp/packer-plugin-sdk-0.6.1 branch from 24c8825 to 56f93d4 Compare April 2, 2025 15:46
@lbajolet-hashicorp
Copy link
Contributor

Huh, the generation changes are worrying a bit, I think we'll need to investigate why those changes happened before merging this, it could be a bug with packer-sdc, or a recent change on the plugin that exposes something it shouldn't.

@tenthirtyam
Copy link
Collaborator

Huh, the generation changes are worrying a bit, I think we'll need to investigate why those changes happened before merging this, it could be a bug with packer-sdc, or a recent change on the plugin that exposes something it shouldn't.

Yeah, it's really acting whacky. Generation fails with the toolchain directive, but then if you generate you will see changes to the ssh_file_transfer_x and an http_ item. too which should nto be there.

Version 1.54.2 of the golangci-lint tool is not compatible with go
1.23, which is the version the project is now compiled with.

In order for CI to be able to run correctly, we are therefore bumping
the golangci-lint dependency to 1.60.1 now, so that the lint job can
run again.
tenthirtyam
tenthirtyam previously approved these changes Apr 2, 2025
A series of changes have made the current .golangci.yml configuration
file non-conformant with latest updates, causing warnings and errors
when attempting to run the tool on the plugin.

This commit fixes all the reported issues with the configurtion,
removing dead checkers and updating the deprecated/removed options in
the config file.
With the 1.60.1 update to golangci-lint and the configuration changes,
some more linting errors were reported by the linter now, so we fix
those that appeared.
@lbajolet-hashicorp
Copy link
Contributor

Thanks for the approval @tenthirtyam, just to be clear, I think this PR should not be immediately merged because still of the changes in the generated code. I'm worried this could be a side-effect of a change in the Go libs which causes some unexpected changes in this case, so we should understand why those changes suddenly appear, and assess if they're legit or due to a bug in packer-sdc

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies go Pull requests that update go code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants