Skip to content
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

Undocumented breaking change in migrate/migrate Docker image #625

Closed
hairyhenderson opened this issue Sep 23, 2021 · 2 comments · Fixed by #628
Closed

Undocumented breaking change in migrate/migrate Docker image #625

hairyhenderson opened this issue Sep 23, 2021 · 2 comments · Fixed by #628
Labels
backwards incompatible Change is backwards incompatible

Comments

@hairyhenderson
Copy link

Describe the Bug

Starting with v4.15.0, the migrate binary is no longer present at /usr/local/bin/migrate, but has moved to /usr/bin/migrate.

Steps to Reproduce
Steps to reproduce the behavior:

  1. List the original location in the v4.14.1 image:
    $ docker run -it --rm --entrypoint '' migrate/migrate:v4.14.1 which migrate
    /usr/local/bin/migrate
  2. List the new location in the v4.15.0 image:
    $ docker run -it --rm --entrypoint '' migrate/migrate:v4.15.0 which migrate
    /usr/bin/migrate

Expected Behavior

Expected the binary to remain in /usr/local/bin/migrate since this was a minor version bump and compatibility was expected, since no mention was made of this change in the release notes.

Migrate Version

v4.15.0

Additional context

I bundle the migrate binary into another image like this:

FROM migrate/migrate:v4.15.0 AS migrate
FROM alpine:3.14.2

COPY --from=migrate /usr/local/bin/migrate /usr/local/bin/migrate
[...]

Of course, if the path changes, the image build breaks as well.

To be clear, this is a minor issue, and I'm totally OK with tweaking my Dockerfile to adjust, however this was a surprise, and not listed in the release notes.

@Fontinalis
Copy link
Member

Nice catch, thank you for reporting! I'm thinking about a quick fix for this in v4.15.1 as this probably causes an issue for many people.

@Fontinalis
Copy link
Member

For the time being, I updated the release notes with the breaking change! Thanks again for the report!

@Fontinalis Fontinalis added the backwards incompatible Change is backwards incompatible label Sep 24, 2021
@dhui dhui closed this as completed in #628 Sep 27, 2021
kodiakhq bot pushed a commit to cloudquery/cloudquery that referenced this issue Jun 20, 2022
…5.2 (#928)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [github.com/golang-migrate/migrate/v4](https://togithub.com/golang-migrate/migrate) | require | patch | `v4.15.0` -> `v4.15.2` |

---

### Release Notes

<details>
<summary>golang-migrate/migrate</summary>

### [`v4.15.2`](https://togithub.com/golang-migrate/migrate/releases/tag/v4.15.2)

[Compare Source](https://togithub.com/golang-migrate/migrate/compare/v4.15.1...v4.15.2)

#### Changes

##### Documentation

-   [`06c25a2`](https://togithub.com/golang-migrate/migrate/commit/06c25a2) Add pkger to README (by [@&#8203;glacials](https://togithub.com/glacials) )
-   [`ed8a8b0`](https://togithub.com/golang-migrate/migrate/commit/ed8a8b0) Include iofs in migration sources (by [@&#8203;bndw](https://togithub.com/bndw) )
-   [`b69b032`](https://togithub.com/golang-migrate/migrate/commit/b69b032) clarify the package name ([#&#8203;660](https://togithub.com/golang-migrate/migrate/issues/660)) (by [@&#8203;khash](https://togithub.com/khash) )
-   [`bfd0502`](https://togithub.com/golang-migrate/migrate/commit/bfd0502) Set syntax highlighting for pkger example (by [@&#8203;glacials](https://togithub.com/glacials) )
-   [`81cbc9c`](https://togithub.com/golang-migrate/migrate/commit/81cbc9c) fix: incorrect clickhouse dns example (by [@&#8203;xkisu](https://togithub.com/xkisu) )

##### Database Drivers

-   [`f621644`](https://togithub.com/golang-migrate/migrate/commit/f621644) Add WithConnection to Postgres similar to MySQL. (by [@&#8203;AndreasKl](https://togithub.com/AndreasKl) )
-   [`6e499cb`](https://togithub.com/golang-migrate/migrate/commit/6e499cb) Expose configuring cassandra connect timeout through query string (by [@&#8203;taywrobel](https://togithub.com/taywrobel) )
-   [`4ba6957`](https://togithub.com/golang-migrate/migrate/commit/4ba6957) Resolves [#&#8203;647](https://togithub.com/golang-migrate/migrate/issues/647) - Fixes typos in Mongo advisory locking parameters ([#&#8203;648](https://togithub.com/golang-migrate/migrate/issues/648)) (by [@&#8203;SJrX](https://togithub.com/SJrX) )
-   [`9d3babd`](https://togithub.com/golang-migrate/migrate/commit/9d3babd) Run SQL server tests on Azure SQL Edge (by [@&#8203;jfhbrook-at-work](https://togithub.com/jfhbrook-at-work) )
-   [`5798784`](https://togithub.com/golang-migrate/migrate/commit/5798784) Update Spanner to 1.27 (by [@&#8203;mccalltd](https://togithub.com/mccalltd) )
-   [`020eda4`](https://togithub.com/golang-migrate/migrate/commit/020eda4) Upgrade cloud.google.com/go/spanner to 1.28.0, which adds JSON_VALUE support (by [@&#8203;kylechadha](https://togithub.com/kylechadha) )
-   [`9f5ed82`](https://togithub.com/golang-migrate/migrate/commit/9f5ed82) Use DELETE FROM instead of TRUNCATE for MySQL ([#&#8203;656](https://togithub.com/golang-migrate/migrate/issues/656)) (by [@&#8203;martinarrieta](https://togithub.com/martinarrieta) and [@&#8203;antigremlin](https://togithub.com/antigremlin) )
-   [`608e4f6`](https://togithub.com/golang-migrate/migrate/commit/608e4f6) Use the recommended v4 in mysql README (by [@&#8203;mgenware](https://togithub.com/mgenware) )
-   [`a46677e`](https://togithub.com/golang-migrate/migrate/commit/a46677e) bump go-sqlite3 to v1.14.10 (by [@&#8203;pankona](https://togithub.com/pankona) )

##### Migration Sources

-   [`a2b0bda`](https://togithub.com/golang-migrate/migrate/commit/a2b0bda) change github auth to use oauth token instead of basic. (by [@&#8203;Greyeye](https://togithub.com/Greyeye) )

##### Builds

-   [`5aa1a03`](https://togithub.com/golang-migrate/migrate/commit/5aa1a03) Bump version of autorest/adal (by [@&#8203;jcmuller](https://togithub.com/jcmuller) )
-   [`169d0fc`](https://togithub.com/golang-migrate/migrate/commit/169d0fc) Include 2 letter query param key in unit test (by [@&#8203;suzaku](https://togithub.com/suzaku) )
-   [`331a15d`](https://togithub.com/golang-migrate/migrate/commit/331a15d) Update dktest from v0.3.9 to v0.3.10 to fix upstream security issues (by [@&#8203;dhui](https://togithub.com/dhui) )
-   [`c277388`](https://togithub.com/golang-migrate/migrate/commit/c277388) Update dktest to v0.3.9 for upstream security vulnerability fixes (by [@&#8203;dhui](https://togithub.com/dhui) )
-   [`cf68462`](https://togithub.com/golang-migrate/migrate/commit/cf68462) Run gofmt on internal build dir (by [@&#8203;dhui](https://togithub.com/dhui) )
-   [`fc7f720`](https://togithub.com/golang-migrate/migrate/commit/fc7f720) refactor: use sort.Slice (by [@&#8203;suzaku](https://togithub.com/suzaku) )
-   [`918e13a`](https://togithub.com/golang-migrate/migrate/commit/918e13a) Upgrade containerd and image-spec for the vulnerabilities ([#&#8203;662](https://togithub.com/golang-migrate/migrate/issues/662)) (by [@&#8203;mopp](https://togithub.com/mopp) )
-   [`4f82413`](https://togithub.com/golang-migrate/migrate/commit/4f82413) len(k) is guaranteed to be greater than 1 in this case (by [@&#8203;suzaku](https://togithub.com/suzaku) )

Thank you for your hard work and all your contributions!

### [`v4.15.1`](https://togithub.com/golang-migrate/migrate/releases/tag/v4.15.1)

[Compare Source](https://togithub.com/golang-migrate/migrate/compare/v4.15.0...v4.15.1)

#### Changes

##### Database Drivers

-   Update gosnowflake from v1.4.3 to v1.6.3 to fix security warnings ([#&#8203;632](https://togithub.com/golang-migrate/migrate/issues/632) & [#&#8203;637](https://togithub.com/golang-migrate/migrate/issues/637)). Thanks [@&#8203;nicholas-mainardi](https://togithub.com/nicholas-mainardi) and [@&#8203;obitech](https://togithub.com/obitech) for reporting the issue!
-   Add unofficial support for AWS Keyspaces to the Cassandra DB driver by `DELETE`ing the schema version table instead of using `TRUNCATE` thanks to [@&#8203;florentsolt](https://togithub.com/florentsolt) and [@&#8203;Baumanar](https://togithub.com/Baumanar) ([#&#8203;634](https://togithub.com/golang-migrate/migrate/issues/634))

##### Docker Image

-   Fixed CLI binary file location ([golang-migrate/migrate#625) thanks to [@&#8203;Fontinalis](https://togithub.com/Fontinalis) ([golang-migrate/migrate#628)

##### Builds

-   Update dktest to v0.3.7 to fix security warnings ([#&#8203;632](https://togithub.com/golang-migrate/migrate/issues/632)). Thanks [@&#8203;nicholas-mainardi](https://togithub.com/nicholas-mainardi) for reporting the issue!

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 3am on Monday" (UTC), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Renovate Bot](https://togithub.com/renovatebot/renovate).
kodiakhq bot pushed a commit to cloudquery/cq-provider-sdk that referenced this issue Jun 20, 2022
…5.2 (#348)

This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [github.com/golang-migrate/migrate/v4](https://togithub.com/golang-migrate/migrate) | require | patch | `v4.15.0` -> `v4.15.2` |

---

### Release Notes

<details>
<summary>golang-migrate/migrate</summary>

### [`v4.15.2`](https://togithub.com/golang-migrate/migrate/releases/tag/v4.15.2)

[Compare Source](https://togithub.com/golang-migrate/migrate/compare/v4.15.1...v4.15.2)

#### Changes

##### Documentation

-   [`06c25a2`](https://togithub.com/golang-migrate/migrate/commit/06c25a2) Add pkger to README (by [@&#8203;glacials](https://togithub.com/glacials) )
-   [`ed8a8b0`](https://togithub.com/golang-migrate/migrate/commit/ed8a8b0) Include iofs in migration sources (by [@&#8203;bndw](https://togithub.com/bndw) )
-   [`b69b032`](https://togithub.com/golang-migrate/migrate/commit/b69b032) clarify the package name ([#&#8203;660](https://togithub.com/golang-migrate/migrate/issues/660)) (by [@&#8203;khash](https://togithub.com/khash) )
-   [`bfd0502`](https://togithub.com/golang-migrate/migrate/commit/bfd0502) Set syntax highlighting for pkger example (by [@&#8203;glacials](https://togithub.com/glacials) )
-   [`81cbc9c`](https://togithub.com/golang-migrate/migrate/commit/81cbc9c) fix: incorrect clickhouse dns example (by [@&#8203;xkisu](https://togithub.com/xkisu) )

##### Database Drivers

-   [`f621644`](https://togithub.com/golang-migrate/migrate/commit/f621644) Add WithConnection to Postgres similar to MySQL. (by [@&#8203;AndreasKl](https://togithub.com/AndreasKl) )
-   [`6e499cb`](https://togithub.com/golang-migrate/migrate/commit/6e499cb) Expose configuring cassandra connect timeout through query string (by [@&#8203;taywrobel](https://togithub.com/taywrobel) )
-   [`4ba6957`](https://togithub.com/golang-migrate/migrate/commit/4ba6957) Resolves [#&#8203;647](https://togithub.com/golang-migrate/migrate/issues/647) - Fixes typos in Mongo advisory locking parameters ([#&#8203;648](https://togithub.com/golang-migrate/migrate/issues/648)) (by [@&#8203;SJrX](https://togithub.com/SJrX) )
-   [`9d3babd`](https://togithub.com/golang-migrate/migrate/commit/9d3babd) Run SQL server tests on Azure SQL Edge (by [@&#8203;jfhbrook-at-work](https://togithub.com/jfhbrook-at-work) )
-   [`5798784`](https://togithub.com/golang-migrate/migrate/commit/5798784) Update Spanner to 1.27 (by [@&#8203;mccalltd](https://togithub.com/mccalltd) )
-   [`020eda4`](https://togithub.com/golang-migrate/migrate/commit/020eda4) Upgrade cloud.google.com/go/spanner to 1.28.0, which adds JSON_VALUE support (by [@&#8203;kylechadha](https://togithub.com/kylechadha) )
-   [`9f5ed82`](https://togithub.com/golang-migrate/migrate/commit/9f5ed82) Use DELETE FROM instead of TRUNCATE for MySQL ([#&#8203;656](https://togithub.com/golang-migrate/migrate/issues/656)) (by [@&#8203;martinarrieta](https://togithub.com/martinarrieta) and [@&#8203;antigremlin](https://togithub.com/antigremlin) )
-   [`608e4f6`](https://togithub.com/golang-migrate/migrate/commit/608e4f6) Use the recommended v4 in mysql README (by [@&#8203;mgenware](https://togithub.com/mgenware) )
-   [`a46677e`](https://togithub.com/golang-migrate/migrate/commit/a46677e) bump go-sqlite3 to v1.14.10 (by [@&#8203;pankona](https://togithub.com/pankona) )

##### Migration Sources

-   [`a2b0bda`](https://togithub.com/golang-migrate/migrate/commit/a2b0bda) change github auth to use oauth token instead of basic. (by [@&#8203;Greyeye](https://togithub.com/Greyeye) )

##### Builds

-   [`5aa1a03`](https://togithub.com/golang-migrate/migrate/commit/5aa1a03) Bump version of autorest/adal (by [@&#8203;jcmuller](https://togithub.com/jcmuller) )
-   [`169d0fc`](https://togithub.com/golang-migrate/migrate/commit/169d0fc) Include 2 letter query param key in unit test (by [@&#8203;suzaku](https://togithub.com/suzaku) )
-   [`331a15d`](https://togithub.com/golang-migrate/migrate/commit/331a15d) Update dktest from v0.3.9 to v0.3.10 to fix upstream security issues (by [@&#8203;dhui](https://togithub.com/dhui) )
-   [`c277388`](https://togithub.com/golang-migrate/migrate/commit/c277388) Update dktest to v0.3.9 for upstream security vulnerability fixes (by [@&#8203;dhui](https://togithub.com/dhui) )
-   [`cf68462`](https://togithub.com/golang-migrate/migrate/commit/cf68462) Run gofmt on internal build dir (by [@&#8203;dhui](https://togithub.com/dhui) )
-   [`fc7f720`](https://togithub.com/golang-migrate/migrate/commit/fc7f720) refactor: use sort.Slice (by [@&#8203;suzaku](https://togithub.com/suzaku) )
-   [`918e13a`](https://togithub.com/golang-migrate/migrate/commit/918e13a) Upgrade containerd and image-spec for the vulnerabilities ([#&#8203;662](https://togithub.com/golang-migrate/migrate/issues/662)) (by [@&#8203;mopp](https://togithub.com/mopp) )
-   [`4f82413`](https://togithub.com/golang-migrate/migrate/commit/4f82413) len(k) is guaranteed to be greater than 1 in this case (by [@&#8203;suzaku](https://togithub.com/suzaku) )

Thank you for your hard work and all your contributions!

### [`v4.15.1`](https://togithub.com/golang-migrate/migrate/releases/tag/v4.15.1)

[Compare Source](https://togithub.com/golang-migrate/migrate/compare/v4.15.0...v4.15.1)

#### Changes

##### Database Drivers

-   Update gosnowflake from v1.4.3 to v1.6.3 to fix security warnings ([#&#8203;632](https://togithub.com/golang-migrate/migrate/issues/632) & [#&#8203;637](https://togithub.com/golang-migrate/migrate/issues/637)). Thanks [@&#8203;nicholas-mainardi](https://togithub.com/nicholas-mainardi) and [@&#8203;obitech](https://togithub.com/obitech) for reporting the issue!
-   Add unofficial support for AWS Keyspaces to the Cassandra DB driver by `DELETE`ing the schema version table instead of using `TRUNCATE` thanks to [@&#8203;florentsolt](https://togithub.com/florentsolt) and [@&#8203;Baumanar](https://togithub.com/Baumanar) ([#&#8203;634](https://togithub.com/golang-migrate/migrate/issues/634))

##### Docker Image

-   Fixed CLI binary file location ([golang-migrate/migrate#625) thanks to [@&#8203;Fontinalis](https://togithub.com/Fontinalis) ([golang-migrate/migrate#628)

##### Builds

-   Update dktest to v0.3.7 to fix security warnings ([#&#8203;632](https://togithub.com/golang-migrate/migrate/issues/632)). Thanks [@&#8203;nicholas-mainardi](https://togithub.com/nicholas-mainardi) for reporting the issue!

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 3am on Monday" (UTC), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, click this checkbox.

---

This PR has been generated by [Renovate Bot](https://togithub.com/renovatebot/renovate).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backwards incompatible Change is backwards incompatible
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants