#1403 notification: Add compose_path into the messages
Merged 3 years ago by lsedlar. Opened 3 years ago by hlin.
hlin/pungi master  into  master

file modified
+1
@@ -52,6 +52,7 @@ 

          data.setdefault("compose_type", self.compose.compose_type)

          data.setdefault("compose_respin", self.compose.compose_respin)

          data.setdefault("compose_label", self.compose.compose_label)

+         data.setdefault("compose_path", self.compose.topdir)

          data.setdefault("release_short", self.compose.conf["release_short"])

          data.setdefault("release_name", self.compose.conf["release_name"])

          data.setdefault("release_version", self.compose.conf["release_version"])

file modified
+2
@@ -29,6 +29,7 @@ 

              compose_respin=1,

              compose_label="Updates-20171031.1021",

              compose_type="production",

+             topdir="/a/b",

              log_warning=mock.Mock(),

              conf={

                  "release_name": "Layer",
@@ -56,6 +57,7 @@ 

          data["compose_type"] = "production"

          data["compose_respin"] = 1

          data["compose_label"] = "Updates-20171031.1021"

+         data["compose_path"] = "/a/b"

          data["release_short"] = "L"

          data["release_name"] = "Layer"

          data["release_version"] = "27"

It's needed by the notification script added in RHELCMP-401.

Signed-off-by: Haibo Lin hlin@redhat.com

I'm not sure if this is a good change. It will has same similar value with location if translate_paths is not configured.

I don't think having the value twice if there is no path translation is a problem.

rebased onto d98158149b8f6a179815fb41e2e6b5c23bd96282

3 years ago

1 new commit added

  • Allow only creating unified ISO for specified arch
3 years ago

1 new commit added

  • notification: Add compose_path into the messages
3 years ago

rebased onto f5bfd50

3 years ago

Pull-Request has been merged by lsedlar

3 years ago