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

Invalid storage volume content type name #8203

Closed
jamielsharief opened this issue Dec 1, 2020 · 9 comments · Fixed by #8204
Closed

Invalid storage volume content type name #8203

jamielsharief opened this issue Dec 1, 2020 · 9 comments · Fixed by #8204
Assignees

Comments

@jamielsharief
Copy link
Contributor

jamielsharief commented Dec 1, 2020

Previously I could create a volume on a storage pool via the API by posting the following data to /storage-pools/default/volumes, however now I am getting the error Invalid storage volume content type name

Array
(
    [name] => volume-58da
    [type] => custom
    [config] => Array
        (
            [size] => 5GB
        )

    [driver] => zfs
)

Note: the same error occurs if using the example from the REST docs

{
    "config": {},
    "name": "vol1",
    "type": "custom"
}
@stgraber
Copy link
Contributor

stgraber commented Dec 1, 2020

@tomponline can you take a quick look? Feels like we're failing to default to fs content type when not specified?

@tomponline
Copy link
Member

@stgraber yep will do

@tomponline
Copy link
Member

@jamielsharief also what version of LXD please

@tomponline tomponline self-assigned this Dec 1, 2020
@tomponline
Copy link
Member

I think ive found the issue

@tomponline
Copy link
Member

@jamielsharief by the way in the interim you can also complete the request by providing the content_type field set to "filesystem" (which will be the default if empty) of "block" for block volumes.

@jamielsharief
Copy link
Contributor Author

Sorry for the delay

  • LXD version 4.8

note. The manual says set to custom but I had tried filesystem earlier but it said I don't have permissions. I am trying to look into now, just LXD is giving EOF errors on everything. I will let you know shortly.

@tomponline
Copy link
Member

tomponline commented Dec 1, 2020

volume type (custom) and content_type are different concepts.

@tomponline
Copy link
Member

@jamielsharief do you have a reproducer using lxc query command?

@jamielsharief
Copy link
Contributor Author

The EOF error was caused because of change I made before i left earlier, ignore that comment.

I have added the content_type and it now works again, my PHPUnit tests from 2-3 months ago stopped working as well, so I presume this is something to do with a recent update?

tomponline added a commit that referenced this issue Jul 22, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [github.com/pkg/xattr](https://github.com/pkg/xattr) | `v0.4.9` ->
`v0.4.10` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fpkg%2fxattr/v0.4.10?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fpkg%2fxattr/v0.4.10?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fpkg%2fxattr/v0.4.9/v0.4.10?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fpkg%2fxattr/v0.4.9/v0.4.10?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>pkg/xattr (github.com/pkg/xattr)</summary>

### [`v0.4.10`](https://github.com/pkg/xattr/releases/tag/v0.4.10)

[Compare
Source](https://github.com/pkg/xattr/compare/v0.4.9...v0.4.10)

#### What's Changed

- Fix FIFO handling on Solaris by
[@&#8203;greatroar](https://github.com/greatroar) in
[https://github.com/pkg/xattr/pull/70](https://github.com/pkg/xattr/pull/70)
- Add support for go 1.21 by
[@&#8203;kuba--](https://github.com/kuba--) in
[https://github.com/pkg/xattr/pull/72](https://github.com/pkg/xattr/pull/72)
- Detect the need to increase buffer size on TrueNAS SCALE. by
[@&#8203;erikrose](https://github.com/erikrose) in
[https://github.com/pkg/xattr/pull/73](https://github.com/pkg/xattr/pull/73)

#### New Contributors

- [@&#8203;erikrose](https://github.com/erikrose) made their first
contribution in
[https://github.com/pkg/xattr/pull/73](https://github.com/pkg/xattr/pull/73)

**Full Changelog**:
pkg/xattr@v0.4.9...v0.4.10

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 4am 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, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/canonical/lxd).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40MzguMCIsInVwZGF0ZWRJblZlciI6IjM3LjQzOC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->
tomponline added a commit that referenced this issue Jul 22, 2024
)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [github.com/minio/minio-go/v7](https://github.com/minio/minio-go) |
`v7.0.73` -> `v7.0.74` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fminio%2fminio-go%2fv7/v7.0.74?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fminio%2fminio-go%2fv7/v7.0.74?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fminio%2fminio-go%2fv7/v7.0.73/v7.0.74?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fminio%2fminio-go%2fv7/v7.0.73/v7.0.74?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>minio/minio-go (github.com/minio/minio-go/v7)</summary>

###
[`v7.0.74`](https://github.com/minio/minio-go/releases/tag/v7.0.74):
Bugfix Release

[Compare
Source](https://github.com/minio/minio-go/compare/v7.0.73...v7.0.74)

#### What's Changed

- return appropriate error type for generic handling by
[@&#8203;harshavardhana](https://github.com/harshavardhana) in
[https://github.com/minio/minio-go/pull/1981](https://github.com/minio/minio-go/pull/1981)
- feat: add SetContentDisposition method for postPolicy Upload by
[@&#8203;lianggx6](https://github.com/lianggx6) in
[https://github.com/minio/minio-go/pull/1982](https://github.com/minio/minio-go/pull/1982)
- docs: fix doc about default alias by
[@&#8203;emil2k](https://github.com/emil2k) in
[https://github.com/minio/minio-go/pull/1984](https://github.com/minio/minio-go/pull/1984)
- fix: GCS endpoint validation to allow ports by
[@&#8203;ammarhusain-sol](https://github.com/ammarhusain-sol) in
[https://github.com/minio/minio-go/pull/1985](https://github.com/minio/minio-go/pull/1985)

#### New Contributors

- [@&#8203;lianggx6](https://github.com/lianggx6) made their first
contribution in
[https://github.com/minio/minio-go/pull/1982](https://github.com/minio/minio-go/pull/1982)
- [@&#8203;emil2k](https://github.com/emil2k) made their first
contribution in
[https://github.com/minio/minio-go/pull/1984](https://github.com/minio/minio-go/pull/1984)
- [@&#8203;ammarhusain-sol](https://github.com/ammarhusain-sol) made
their first contribution in
[https://github.com/minio/minio-go/pull/1985](https://github.com/minio/minio-go/pull/1985)

**Full Changelog**:
minio/minio-go@v7.0.73...v7.0.74

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 4am 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, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/canonical/lxd).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40MzguMCIsInVwZGF0ZWRJblZlciI6IjM3LjQzOC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->
tomponline added a commit that referenced this issue Jul 29, 2024
)

[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [github.com/go-jose/go-jose/v4](https://github.com/go-jose/go-jose)
| `v4.0.3` -> `v4.0.4` |
[![age](https://developer.mend.io/api/mc/badges/age/go/github.com%2fgo-jose%2fgo-jose%2fv4/v4.0.4?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/go/github.com%2fgo-jose%2fgo-jose%2fv4/v4.0.4?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/go/github.com%2fgo-jose%2fgo-jose%2fv4/v4.0.3/v4.0.4?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/go/github.com%2fgo-jose%2fgo-jose%2fv4/v4.0.3/v4.0.4?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>go-jose/go-jose (github.com/go-jose/go-jose/v4)</summary>

###
[`v4.0.4`](https://github.com/go-jose/go-jose/blob/HEAD/CHANGELOG.md#v404)

[Compare
Source](https://github.com/go-jose/go-jose/compare/v4.0.3...v4.0.4)

#### Fixed

- Reverted "Allow unmarshalling JSONWebKeySets with unsupported key
types" as a
breaking change. See
[#&#8203;136](https://github.com/go-jose/go-jose/issues/136) /
[#&#8203;137](https://github.com/go-jose/go-jose/issues/137).

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 4am 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, check
this box

---

This PR was generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View the
[repository job log](https://developer.mend.io/github/canonical/lxd).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40NDAuNyIsInVwZGF0ZWRJblZlciI6IjM3LjQ0MC43IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants