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

Enable --device directory as src device #1937

Closed
wants to merge 1 commit into from

Conversation

QiWang19
Copy link
Contributor

Enables --device accepte directory path as source device. Add the devices under the source directory to the destination directory.

complete card test criteria: https://jira.coreos.com/browse/RUN-497
related podman issue: containers/podman#2380

Signed-off-by: Qi Wang qiwan@redhat.com

@QiWang19
Copy link
Contributor Author

@TomSweeneyRedHat @rhatdan PTAL

@TomSweeneyRedHat
Copy link
Member

We should have some kind of change to the man page(s) too.

_, err = DeviceFromPath("/dev/pts")
assert.Error(t, err)
assert.NoError(t, err)
Copy link
Member

Choose a reason for hiding this comment

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

Should add another test for a file

_, err = DeviceFromPath("/etc/passwd") 	
    assert.Error(t, err)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

test added

@rhatdan
Copy link
Member

rhatdan commented Oct 23, 2019

@rh-atomic-bot r+

@rh-atomic-bot
Copy link
Collaborator

📌 Commit 4db2a49 has been approved by rhatdan

@rh-atomic-bot
Copy link
Collaborator

⌛ Testing commit 4db2a49 with merge 8195b03...

rh-atomic-bot pushed a commit that referenced this pull request Oct 23, 2019
Enables --device accepte directory path as source device. Add the devices under the source directory to the destination directory.

complete card test criteria: https://jira.coreos.com/browse/RUN-497
related podman issue: containers/podman#2380

Signed-off-by: Qi Wang <qiwan@redhat.com>

Closes: #1937
Approved by: rhatdan
@rh-atomic-bot
Copy link
Collaborator

💔 Test failed - status-travis

@TomSweeneyRedHat
Copy link
Member

@rh-atomic-bot retry

@rh-atomic-bot
Copy link
Collaborator

⌛ Testing commit 4db2a49 with merge 6b4b163...

rh-atomic-bot pushed a commit that referenced this pull request Oct 23, 2019
Enables --device accepte directory path as source device. Add the devices under the source directory to the destination directory.

complete card test criteria: https://jira.coreos.com/browse/RUN-497
related podman issue: containers/podman#2380

Signed-off-by: Qi Wang <qiwan@redhat.com>

Closes: #1937
Approved by: rhatdan
@rh-atomic-bot
Copy link
Collaborator

💔 Test failed - status-travis

@QiWang19 QiWang19 force-pushed the dir_device branch 2 times, most recently from e51ea79 to ab3e596 Compare October 24, 2019 02:32
@QiWang19
Copy link
Contributor Author

does this error message mean some other tool does not support adding a directory of devices?
https://travis-ci.org/containers/buildah/jobs/601935493#L1986

@QiWang19
Copy link
Contributor Author

@TomSweeneyRedHat Can I add another Dockerfile under tests/bud/device/? I want to try to see if
ci fails because current Dockerfile does not work for my test.
git doesn't track the new file after I only add it under /tests/bud/device

@QiWang19 QiWang19 force-pushed the dir_device branch 2 times, most recently from 83e6880 to 31ba7bd Compare October 24, 2019 18:45
Enables --device accepte directory path as source device. Add the devices under the source directory to the destination directory.

complete card test criteria: https://jira.coreos.com/browse/RUN-497
related podman issue: containers/podman#2380

Signed-off-by: Qi Wang <qiwan@redhat.com>
@QiWang19
Copy link
Contributor Author

@TomSweeneyRedHat tests are green now. Can we get this in

@rhatdan
Copy link
Member

rhatdan commented Oct 25, 2019

@rh-atomic-bot r+

@rh-atomic-bot
Copy link
Collaborator

📌 Commit 6f6f32b has been approved by rhatdan

@rh-atomic-bot
Copy link
Collaborator

⚡ Test exempted: pull fully rebased and already tested.

@TomSweeneyRedHat
Copy link
Member

I'm jumping in after the barn door shut, but LGTM and nice job @QiWang19

caiges pushed a commit to caiges/buildah that referenced this pull request Nov 12, 2019
Enables --device accepte directory path as source device. Add the devices under the source directory to the destination directory.

complete card test criteria: https://jira.coreos.com/browse/RUN-497
related podman issue: containers/podman#2380

Signed-off-by: Qi Wang <qiwan@redhat.com>

Closes: containers#1937
Approved by: rhatdan
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants