Skip to contentSkip to conversation

cbuild: Add Fedora 27 container

PATCH In review Leon Romanovsky 4 participants 6 messages Jan 12, 2018 – Jan 17, 2018 4 messages not in the archive InfiniBand

lore
Share & tools
lore

Leon Romanovsky

author
#0 lore raw

PATCH 1/1 cbuild: Add Fedora 27 container

From: Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/[email protected]>

Add Fedora 27 to the list of cbuild targets.

Signed-off-by Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/[email protected]>
---
 buildlib/cbuild | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
buildlib/cbuild +15 -0
index 3f659bf8..0ef80b60 100755
@@ -141,6 +141,20 @@ class fc26(Environment):
             " ".join(sorted(self.pkgs))));
         return res;
 
+class fc27(Environment):
+    docker_parent = "fedora:27";
+    pkgs = (centos7.pkgs - {"make"}) | {"ninja-build"};
+    name = "fc27";
+    specfile = "redhat/rdma-core.spec";
+    ninja_cmd = "ninja-build";
+    is_rpm = True;
+
+    def get_docker_file(self):
+        res = DockerFile(self.docker_parent);
+        res.lines.append("RUN dnf install -y %s && dnf clean all"%(
+            " ".join(sorted(self.pkgs))));
+        return res;
+
 # -------------------------------------------------------------------------
 
 class APTEnvironment(Environment):

@@ -337,6 +351,7 @@ environments = [centos6(),
                 jessie(),
                 stretch(),
                 fc26(),
+                fc27(),
                 leap(),
                 tumbleweed(),
                 debian_experimental(),

--
2.15.1

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html

A message that is not in the archive (sent off-list or not yet received): 20180112115209.17437-1-leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org

#2 lore raw
Quoted 33 lines · Leon Romanovsky

From: Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/[email protected]>

Add Fedora 27 to the list of cbuild targets.

Signed-off-by Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/[email protected]>
 buildlib/cbuild | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
buildlib/cbuild +15 -0
index 3f659bf8..0ef80b60 100755
+++ b/buildlib/cbuild

@@ -141,6 +141,20 @@ class fc26(Environment):
             " ".join(sorted(self.pkgs))));
         return res;
 
+class fc27(Environment):
+    docker_parent = "fedora:27";
+    pkgs = (centos7.pkgs - {"make"}) | {"ninja-build"};
+    name = "fc27";
+    specfile = "redhat/rdma-core.spec";
+    ninja_cmd = "ninja-build";
+    is_rpm = True;
+
+    def get_docker_file(self):
+        res = DockerFile(self.docker_parent);
+        res.lines.append("RUN dnf install -y %s && dnf clean all"%(
+            " ".join(sorted(self.pkgs))));
+        return res;
+
 # -------------------------------------------------------------------------
 
 class APTEnvironment(Environment):

buildlib/cbuild environments = [centos6(), · line 354

Quoted 4 lines
buildlib/cbuild +1 -0
@@ -337,6 +351,7 @@ environments = [centos6(),
                 jessie(),
                 stretch(),
                 fc26(),
+                fc27(),

I've been keeping only 1 fc container, no need to test old fedora.

Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html

A message that is not in the archive (sent off-list or not yet received): 20180112180855.GF15974-uk2M96/[email protected]

author
#4 lore raw
Quoted 32 lines · Jason Gunthorpe
Quoted 31 lines · Leon Romanovsky

From: Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/[email protected]>

Add Fedora 27 to the list of cbuild targets.

Signed-off-by Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/[email protected]>
 buildlib/cbuild | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
buildlib/cbuild +15 -0
index 3f659bf8..0ef80b60 100755
+++ b/buildlib/cbuild

@@ -141,6 +141,20 @@ class fc26(Environment):
             " ".join(sorted(self.pkgs))));
         return res;
 
+class fc27(Environment):
+    docker_parent = "fedora:27";
+    pkgs = (centos7.pkgs - {"make"}) | {"ninja-build"};
+    name = "fc27";
+    specfile = "redhat/rdma-core.spec";
+    ninja_cmd = "ninja-build";
+    is_rpm = True;
+
+    def get_docker_file(self):
+        res = DockerFile(self.docker_parent);
+        res.lines.append("RUN dnf install -y %s && dnf clean all"%(
+            " ".join(sorted(self.pkgs))));
+        return res;
+
 # -------------------------------------------------------------------------
Quoted 8 lines

buildlib/cbuild environments = [centos6(), · line 354

Quoted 6 lines
buildlib/cbuild +1 -0
 class APTEnvironment(Environment):

@@ -337,6 +351,7 @@ environments = [centos6(),
                 jessie(),
                 stretch(),
                 fc26(),
+                fc27(),

I've been keeping only 1 fc container, no need to test old fedora.

I'll remove the rest.

Quoted 2 lines

Jason

#5 lore raw
In reply to a message not in the archive
Quoted 14 lines · Leon Romanovsky

From: Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/[email protected]>

Add Fedora 27 to the list of cbuild targets.

Signed-off-by Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/[email protected]>
---
 buildlib/cbuild | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
buildlib/cbuild +0 -0
index 3f659bf8..0ef80b60 100755

buildlib/cbuild class fc26(Environment): · line 146

Quoted 6 lines
buildlib/cbuild +3 -0
@@ -141,6 +141,20 @@ class fc26(Environment):
             " ".join(sorted(self.pkgs))));
         return res;
 
+class fc27(Environment):
+    docker_parent = "fedora:27";
+    pkgs = (centos7.pkgs - {"make"}) | {"ninja-build"};

Is there some reason why are you pulling a fedora:27 container and then centos packages?

Signature · 3 lines
Doug Ledford <[email protected]>
    GPG KeyID: B826A3330E572FDD
    Key fingerprint = AE6B 1BDA 122B 23B4 265B  1274 B826 A333 0E57 2FDD

A message that is not in the archive (sent off-list or not yet received): 1516202077.3403.254.camel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org

#7 lore raw
Quoted 16 lines · Doug Ledford
Quoted 15 lines · Leon Romanovsky

From: Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/[email protected]>

Add Fedora 27 to the list of cbuild targets.

Signed-off-by Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/[email protected]>
 buildlib/cbuild | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
buildlib/cbuild +1 -0
index 3f659bf8..0ef80b60 100755
+++ b/buildlib/cbuild

@@ -141,6 +141,20 @@ class fc26(Environment):
             " ".join(sorted(self.pkgs))));
         return res;
Quoted 6 lines

buildlib/cbuild class fc26(Environment): · line 146

Quoted 3 lines
buildlib/cbuild +3 -0
+class fc27(Environment):
+    docker_parent = "fedora:27";
+    pkgs = (centos7.pkgs - {"make"}) | {"ninja-build"};

Is there some reason why are you pulling a fedora:27 container and then centos packages?

That is the pattern, centos7 has a list of required packages that is most similar to fc27. It will make more sense when the patch is respun to have fc27 replace fc26..

Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html

A message that is not in the archive (sent off-list or not yet received): 20180117161614.GA3764-VPRAkNaXOzVWk0Htik3J/[email protected]

author
#9 lore raw
Quoted 24 lines · Jason Gunthorpe
Quoted 16 lines · Doug Ledford
Quoted 15 lines · Leon Romanovsky

From: Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/[email protected]>

Add Fedora 27 to the list of cbuild targets.

Signed-off-by Leon Romanovsky <leonro-VPRAkNaXOzVWk0Htik3J/[email protected]>
 buildlib/cbuild | 15 +++++++++++++++
 1 file changed, 15 insertions(+)
buildlib/cbuild +1 -0
index 3f659bf8..0ef80b60 100755
+++ b/buildlib/cbuild

@@ -141,6 +141,20 @@ class fc26(Environment):
             " ".join(sorted(self.pkgs))));
         return res;
Quoted 6 lines

buildlib/cbuild class fc26(Environment): · line 146

Quoted 3 lines
buildlib/cbuild +3 -0
+class fc27(Environment):
+    docker_parent = "fedora:27";
+    pkgs = (centos7.pkgs - {"make"}) | {"ninja-build"};

Is there some reason why are you pulling a fedora:27 container and then centos packages?

Quoted 3 lines

That is the pattern, centos7 has a list of required packages that is most similar to fc27. It will make more sense when the patch is respun to have fc27 replace fc26..

I'll do.

Thanks

Quoted 6 lines

Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html