2:02 p.m.
We need this patch to fix ostree in our f28 updates testing runs.
See https://pagure.io/dusty/failed-composes/issue/63
Dusty
2:02 p.m.
New subject: [PATCH] bodhi pungi ostree: point to different repos for f28
The f28 repos we need to tap into are not in the same locations
as for post release (f26/f27) runs. We also don't need to include
the 'updates' repo because there is no updates repo for f28 yet.
Signed-off-by: Dusty Mabe <dusty(a)dustymabe.com>
---
roles/bodhi2/backend/templates/pungi.rpm.conf.j2 | 28 +++++++++++++++++-------
1 file changed, 20 insertions(+), 8 deletions(-)
diff --git a/roles/bodhi2/backend/templates/pungi.rpm.conf.j2
b/roles/bodhi2/backend/templates/pungi.rpm.conf.j2
index 42e3f6206..ec0b36c9c 100644
--- a/roles/bodhi2/backend/templates/pungi.rpm.conf.j2
+++ b/roles/bodhi2/backend/templates/pungi.rpm.conf.j2
@@ -123,11 +123,17 @@ ostree = {
"config_branch": "f[[ release.version ]]",
"repo": [
"Everything",
- [% if request.name == 'testing' %]
- # In the case of testing, also inject the last stable updates
- "https://kojipkgs{{ env_suffix
}}.fedoraproject.org/compose/updates/f[[ release.version_int
]]-updates/compose/Everything/$basearch/os/",
+ # If f28 then don't yet point to updates repo for testing composes
+ # Also the compose location is under /compose/branched/
+ [% if release.version_int >= 28 %]
+ "https://kojipkgs{{ env_suffix
}}.fedoraproject.org/compose/branched/latest-Fedora-[[ release.version_int
]]/compose/Everything/$basearch/os/"
+ [% else %]
+ [% if request.name == 'testing' %]
+ # In the case of testing, also inject the last stable updates
+ "https://kojipkgs{{ env_suffix
}}.fedoraproject.org/compose/updates/f[[ release.version_int
]]-updates/compose/Everything/$basearch/os/",
+ [% endif %]
+ "https://kojipkgs{{ env_suffix }}.fedoraproject.org/compose/[[
release.version_int ]]/latest-Fedora-[[ release.version_int
]]/compose/Everything/$basearch/os/"
[% endif %]
- "https://kojipkgs{{ env_suffix }}.fedoraproject.org/compose/[[
release.version_int ]]/latest-Fedora-[[ release.version_int
]]/compose/Everything/$basearch/os/"
]
"ostree_repo": "/mnt/koji/compose/atomic/repo",
"tag_ref": False,
@@ -151,11 +157,17 @@ ostree = {
"config_branch": "f[[ release.version ]]",
"repo": [
"Everything",
- [% if request.name == 'testing' %]
- # In the case of testing, also inject the last stable updates
- "https://kojipkgs{{ env_suffix
}}.fedoraproject.org/compose/updates/f[[ release.version_int
]]-updates/compose/Everything/$basearch/os/",
+ # If f28 then don't yet point to updates repo for testing composes
+ # Also the compose location is under /compose/branched/
+ [% if release.version_int >= 28 %]
+ "https://kojipkgs{{ env_suffix
}}.fedoraproject.org/compose/branched/latest-Fedora-[[ release.version_int
]]/compose/Everything/$basearch/os/"
+ [% else %]
+ [% if request.name == 'testing' %]
+ # In the case of testing, also inject the last stable updates
+ "https://kojipkgs{{ env_suffix
}}.fedoraproject.org/compose/updates/f[[ release.version_int
]]-updates/compose/Everything/$basearch/os/",
+ [% endif %]
+ "https://kojipkgs{{ env_suffix }}.fedoraproject.org/compose/[[
release.version_int ]]/latest-Fedora-[[ release.version_int
]]/compose/Everything/$basearch/os/"
[% endif %]
- "https://kojipkgs{{ env_suffix }}.fedoraproject.org/compose/[[
release.version_int ]]/latest-Fedora-[[ release.version_int
]]/compose/Everything/$basearch/os/"
]
"ostree_repo": "/mnt/koji/compose/atomic/repo",
"tag_ref": False,
--
2.14.3
3:03 p.m.
On 03/09/2018 12:02 PM, Dusty Mabe wrote:
We need this patch to fix ostree in our f28 updates testing runs.
See https://pagure.io/dusty/failed-composes/issue/63
Seems ok, +1.
kevin
3:25 p.m.
New subject: [PATCH] bodhi pungi ostree: point to different repos for f28
The f28 repos we need to tap into are not in the same locations
as for post release (f26/f27) runs. We also don't need to include
the 'updates' repo because there is no updates repo for f28 yet.
This is actually an f28 repo, but I had made a mistake moving things around (I
accidentally moved things out of the os/ directory in the empty-repo, while this was
supposed to have happened in the /pub repo), so let's keep that.
Also, for the compose/[[ release.version_int ]]/latest-Fedora, I think we should just
create a symlink of /28 to /branched perhaps, but that'd be the only part we should
have in this patch.
+ [% if release.version_int >= 28 %]
Let's use an explicit release.version_int == 28, to indicate that this is only the
case for now while we are in branched state.
3:41 p.m.
New subject: [PATCH] bodhi pungi ostree: point to different repos for f28
On 03/09/2018 04:25 PM, Patrick マルタインアンドレアス Uiterwijk wrote:
> The f28 repos we need to tap into are not in the same locations
> as for post release (f26/f27) runs. We also don't need to include
> the 'updates' repo because there is no updates repo for f28 yet.
>
This is actually an f28 repo, but I had made a mistake moving things around (I
accidentally moved things out of the os/ directory in the empty-repo, while this was
supposed to have happened in the /pub repo), so let's keep that.
Also, for the compose/[[ release.version_int ]]/latest-Fedora, I think we should just
create a symlink of /28 to /branched perhaps, but that'd be the only part we should
have in this patch.
> + [% if release.version_int >= 28 %]
Let's use an explicit release.version_int == 28, to indicate that this is only the
case for now while we are in branched state.
ok I'm using a pull request for this:
https://gitlab.com/dustymabe/fedora-infra-ansible/merge_requests/1/diffs?w=1
here is the updated patch if you want to fight with pulling it in from email:
From 088ec129cd028b4c70e5a719280bfb3d7d1d2b48 Mon Sep 17 00:00:00 2001
From: Dusty Mabe <dusty(a)dustymabe.com>
Date: Fri, 9 Mar 2018 14:59:13 -0500
Subject: [PATCH] bodhi pungi ostree: point to different repos for f28
The f28 repos we need to tap into are not in the same locations
as for post release (f26/f27) runs. We also don't need to include
the 'updates' repo because there is no updates repo for f28 yet.
Signed-off-by: Dusty Mabe <dusty(a)dustymabe.com>
---
roles/bodhi2/backend/templates/pungi.rpm.conf.j2 | 14 ++++++++++++--
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/roles/bodhi2/backend/templates/pungi.rpm.conf.j2
b/roles/bodhi2/backend/templates/pungi.rpm.conf.j2
index 42e3f6206..ff31d400e 100644
--- a/roles/bodhi2/backend/templates/pungi.rpm.conf.j2
+++ b/roles/bodhi2/backend/templates/pungi.rpm.conf.j2
@@ -127,7 +127,12 @@ ostree = {
# In the case of testing, also inject the last stable updates
"https://kojipkgs{{ env_suffix
}}.fedoraproject.org/compose/updates/f[[ release.version_int
]]-updates/compose/Everything/$basearch/os/",
[% endif %]
- "https://kojipkgs{{ env_suffix }}.fedoraproject.org/compose/[[
release.version_int ]]/latest-Fedora-[[ release.version_int
]]/compose/Everything/$basearch/os/"
+ # For f28 the compose location is under /compose/branched/
+ [% if release.version_int == 28 %]
+ "https://kojipkgs{{ env_suffix
}}.fedoraproject.org/compose/branched/latest-Fedora-[[ release.version_int
]]/compose/Everything/$basearch/os/"
+ [% else %]
+ "https://kojipkgs{{ env_suffix }}.fedoraproject.org/compose/[[
release.version_int ]]/latest-Fedora-[[ release.version_int
]]/compose/Everything/$basearch/os/"
+ [% endif %]
]
"ostree_repo": "/mnt/koji/compose/atomic/repo",
"tag_ref": False,
@@ -155,7 +160,12 @@ ostree = {
# In the case of testing, also inject the last stable updates
"https://kojipkgs{{ env_suffix
}}.fedoraproject.org/compose/updates/f[[ release.version_int
]]-updates/compose/Everything/$basearch/os/",
[% endif %]
- "https://kojipkgs{{ env_suffix }}.fedoraproject.org/compose/[[
release.version_int ]]/latest-Fedora-[[ release.version_int
]]/compose/Everything/$basearch/os/"
+ # For f28 the compose location is under /compose/branched/
+ [% if release.version_int == 28 %]
+ "https://kojipkgs{{ env_suffix
}}.fedoraproject.org/compose/branched/latest-Fedora-[[ release.version_int
]]/compose/Everything/$basearch/os/"
+ [% else %]
+ "https://kojipkgs{{ env_suffix }}.fedoraproject.org/compose/[[
release.version_int ]]/latest-Fedora-[[ release.version_int
]]/compose/Everything/$basearch/os/"
+ [% endif %]
]
"ostree_repo": "/mnt/koji/compose/atomic/repo",
"tag_ref": False,
--
2.14.3
3:44 p.m.
New subject: [PATCH] bodhi pungi ostree: point to different repos for f28
On 03/09/2018 04:25 PM, Patrick マルタインアンドレアス Uiterwijk wrote:
ok I'm using a pull request for this:
https://gitlab.com/dustymabe/fedora-infra-ansible/merge_requests/1/diffs?w=1
here is the updated patch if you want to fight with pulling it in from email:
From 088ec129cd028b4c70e5a719280bfb3d7d1d2b48 Mon Sep 17 00:00:00 2001
From: Dusty Mabe <dusty(a)dustymabe.com>
Date: Fri, 9 Mar 2018 14:59:13 -0500
Subject: [PATCH] bodhi pungi ostree: point to different repos for f28
The f28 repos we need to tap into are not in the same locations
as for post release (f26/f27) runs. We also don't need to include
the 'updates' repo because there is no updates repo for f28 yet.
This still refers to the updates repo, but you can fix that on push.
+1
3:45 p.m.
New subject: [PATCH] bodhi pungi ostree: point to different repos for f28
On 03/09/2018 04:44 PM, Patrick マルタインアンドレアス Uiterwijk wrote:
> On 03/09/2018 04:25 PM, Patrick マルタインアンドレアス Uiterwijk wrote:
>
>
> ok I'm using a pull request for this:
>
> https://gitlab.com/dustymabe/fedora-infra-ansible/merge_requests/1/diffs?w=1
>
> here is the updated patch if you want to fight with pulling it in from email:
>
>
>
>
> From 088ec129cd028b4c70e5a719280bfb3d7d1d2b48 Mon Sep 17 00:00:00 2001
> From: Dusty Mabe <dusty(a)dustymabe.com>
> Date: Fri, 9 Mar 2018 14:59:13 -0500
> Subject: [PATCH] bodhi pungi ostree: point to different repos for f28
>
> The f28 repos we need to tap into are not in the same locations
> as for post release (f26/f27) runs. We also don't need to include
> the 'updates' repo because there is no updates repo for f28 yet.
This still refers to the updates repo, but you can fix that on push.
I will fix the commit message.. thanks
2241
days inactive
2241
days old
infrastructure@lists.fedoraproject.org
7 comments
4 participants
participants (4)
-
Dusty Mabe -
Kevin Fenzi -
Mohan Boddu -
Patrick マルタインアンドレアス Uiterwijk