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

import1: Implement importd methods #310

Merged
merged 1 commit into from
Jun 20, 2019
Merged

import1: Implement importd methods #310

merged 1 commit into from
Jun 20, 2019

Conversation

Xuanwo
Copy link
Contributor

@Xuanwo Xuanwo commented Jun 17, 2019

systemd-importd service is used to import, export and download
additional system images, for running them as local containers
using tools such as systemd-nspawn.

This PR allows users to import/export/pull images via
systemd-importd.

Fix #232

@Xuanwo
Copy link
Contributor Author

Xuanwo commented Jun 17, 2019

ping @lucab, this PR is ready for review.

return c, nil
}

func (c *Conn) initConnection() error {
Copy link
Contributor

Choose a reason for hiding this comment

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

Note to self: this is a connection helper duplicated in a few places, we should later move it to a common sub-package under internal.

import1/dbus.go Show resolved Hide resolved
import1/dbus.go Outdated Show resolved Hide resolved
import1/dbus.go Outdated Show resolved Hide resolved
import1/dbus.go Outdated Show resolved Hide resolved
import1/dbus.go Outdated Show resolved Hide resolved
import1/dbus.go Outdated Show resolved Hide resolved
import1/dbus_test.go Outdated Show resolved Hide resolved
import1/dbus.go Show resolved Hide resolved
@lucab
Copy link
Contributor

lucab commented Jun 19, 2019

Two minor things left, looks ready otherwise. @Xuanwo feel free to squash into a single commit.

systemd-importd service is used to import, export and download
additional system images, for running them as local containers
using tools such as systemd-nspawn.

This commit allow users to import/export/pull images via
systemd-importd.
@Xuanwo
Copy link
Contributor Author

Xuanwo commented Jun 19, 2019

This PR is ready for merge now 😄

@lucab lucab changed the title Implement importd import1: Implement importd methods Jun 20, 2019
Copy link
Contributor

@lucab lucab left a comment

Choose a reason for hiding this comment

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

LGTM, thanks a lot!

@lucab lucab merged commit e64a0ec into coreos:master Jun 20, 2019
@lucab lucab added this to the v19 milestone Jun 20, 2019
@Xuanwo Xuanwo deleted the add-importd branch June 20, 2019 07:15
@lucab lucab modified the milestones: v19, v20 Jun 20, 2019
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 this pull request may close these issues.

Systemd-Importd support
2 participants