-
Notifications
You must be signed in to change notification settings - Fork 166
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
Restructure filtering and add new filters #744
Conversation
6eead6f
to
515942a
Compare
191ee50
to
d0ed381
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm too new to Go to give relevant feedback/review.
OK, but you've been doing a good job so far with your Go reviews 😉 |
Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
This changes the way filters are applied. Prior to this change, filters were applied in various places. With this change, all filters are applied at start by modifying the loaded definition. Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
Signed-off-by: Thomas Hipp <thomas.hipp@canonical.com>
d0ed381
to
2e6a1a1
Compare
With this PR, meta.tar.xz ends up not having config.* or templates. If the fix is simple, let's do that, but it's probably best to revert this PR for now, as lxc-create -t download is broken for the world right now. |
Hi - when you resubmit this, it would be great if you could add a test in the github actions workflow which does a pack-lxc and verifies that the required files are generated in meta.tar. |
Fixes #742