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

github: use ppa:ubuntu-lxc/daily instead of ppa:ubuntu-lxc/lxc-git-master #12396

Merged

Conversation

mihalicyn
Copy link
Member

@mihalicyn mihalicyn commented Oct 17, 2023

Let's be consistent with go-lxc [1] and use ppa:ubuntu-lxc/daily to fetch and install development build of liblxc.

I have checked that builds in ppa:ubuntu-lxc/daily and ppa:ubuntu-lxc/lxc-git-master right now are equal. Likely we need to get rid of ubuntu-lxc/lxc-git-master at some point, but as a first step let's just start using ubuntu-lxc/daily for LXD CI builds.

[1] https://github.com/lxc/go-lxc/blob/ccae595aa49e779f7ecc9250329967aa546acd31/.github/workflows/test.yml#L29

…ster

Let's be consistent with go-lxc [1] and use ppa:ubuntu-lxc/daily to fetch
and install development build of liblxc.

I have checked that builds in ppa:ubuntu-lxc/daily and ppa:ubuntu-lxc/lxc-git-master
right now are equal. Likely we need to get rid of ubuntu-lxc/lxc-git-master at some point,
but as a first step let's just start using ubuntu-lxc/daily for LXD CI builds.

[1] https://github.com/lxc/go-lxc/blob/ccae595aa49e779f7ecc9250329967aa546acd31/.github/workflows/test.yml#L29

Signed-off-by: Alexander Mikhalitsyn <aleksandr.mikhalitsyn@canonical.com>
@tomponline tomponline changed the title github: use ppa:ubuntu-lxc/daily instead of ppa:ubuntu-lxc/lxc-git-ma… github: use ppa:ubuntu-lxc/daily instead of ppa:ubuntu-lxc/lxc-git-master Oct 17, 2023
@tomponline tomponline merged commit b6ec0f2 into canonical:main Oct 17, 2023
25 of 26 checks passed
This pull request was closed.
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.

2 participants