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.

Sign up for GitHub

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

Jump to bottom

yq snap should optionally allow connecting to the removable-media snapd interface slot #1618

Closed
brlin-tw opened this issue Mar 27, 2023 · 0 comments · Fixed by #1619
Closed

yq snap should optionally allow connecting to the removable-media snapd interface slot #1618

brlin-tw opened this issue Mar 27, 2023 · 0 comments · Fixed by #1619
Labels
bug v4

Comments

@brlin-tw
Copy link
Contributor

brlin-tw commented Mar 27, 2023 •
edited

Describe the bug

Currently the yq snap does not allow connecting to the removable-media snapd interface slot, making it unable to process YAML documents under /mnt or /media paths using the regular syntax even if the user wanted to.

By making the plug declaration snap users can optionally allows the snap to do so, expect a pull request to be filed soon.

Version of yq: 4.33.1
Operating system: linux
Installed via: snap

Input Yaml

/mnt/data1.yml:

any: yaml file should do

Command
The command you ran:

yq e '.any' /mnt/data1.yml

Actual behavior

Error: open /mnt/data1.yml: permission denied

Expected behavior

yaml file should do
brlin-tw added a commit to brlin-tw/yq that referenced this issue Mar 27, 2023
@brlin-tw
Add removable-media interface plug declaration to the snap packaging(m…
f1e5f1f 
…ikefarah#1618)

Allow snap users to process YAML documents under /media or /mnt paths after connecting to the `removable-media` snapd slot:

```
sudo snap connect yq :removable-media
```

Fixes mikefarah#1618.
@brlin-tw brlin-tw mentioned this issue Mar 27, 2023
Merged
mikefarah pushed a commit that referenced this issue Mar 28, 2023
@brlin-tw
Add removable-media interface plug declaration to the snap packaging(#…
311622d 
…1618) (#1619)

Allow snap users to process YAML documents under /media or /mnt paths after connecting to the `removable-media` snapd slot:

```
sudo snap connect yq :removable-media
```

Fixes #1618.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug v4
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add removable-media interface plug declaration to the snap packaging(#1618) brlin-tw/yq
1 participant
@brlin-tw