Skip to content

Commit edd6370

Browse files
sharath-srikanth-chellappaSharath Srikanth Chellappa
andauthored
cdi: Renaming binaries in the SPEC file to align with upstream naming conventions (#12214)
Co-authored-by: Sharath Srikanth Chellappa <sharathsr@microsoft.com>
1 parent b0a482b commit edd6370

7 files changed

Lines changed: 16 additions & 31 deletions

File tree

.pipelines/containerSourceData/cdi/Dockerfile-cdi-apiserver

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,6 @@ ARG USER
1010

1111
@INCLUDE_MAIN_RUN_INSTRUCTION@
1212

13-
# workaround till binaries rename is merged
14-
RUN [ -f /usr/bin/virt-cdi-apiserver ] && mv -f /usr/bin/virt-cdi-apiserver /usr/bin/cdi-apiserver
15-
1613
#simple smoke test
1714
RUN ls /usr/bin/cdi-apiserver
1815

.pipelines/containerSourceData/cdi/Dockerfile-cdi-controller

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,6 @@ ARG USER
1010

1111
@INCLUDE_MAIN_RUN_INSTRUCTION@
1212

13-
# workaround till binaries rename is merged
14-
RUN [ -f /usr/bin/virt-cdi-controller ] && mv -f /usr/bin/virt-cdi-controller /usr/bin/cdi-controller
15-
1613
#simple smoke test
1714
RUN ls /usr/bin/cdi-controller
1815

.pipelines/containerSourceData/cdi/Dockerfile-cdi-importer

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,6 @@ ARG USER
1010

1111
@INCLUDE_MAIN_RUN_INSTRUCTION@
1212

13-
# workaround till binaries rename is merged
14-
RUN [ -f /usr/bin/virt-cdi-importer ] && mv -f /usr/bin/virt-cdi-importer /usr/bin/cdi-importer
15-
1613
#simple smoke test
1714
RUN ls /usr/bin/cdi-importer
1815

.pipelines/containerSourceData/cdi/Dockerfile-cdi-operator

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,6 @@ ARG USER
1010

1111
@INCLUDE_MAIN_RUN_INSTRUCTION@
1212

13-
# workaround till binaries rename is merged
14-
RUN [ -f /usr/bin/virt-cdi-operator ] && mv -f /usr/bin/virt-cdi-operator /usr/bin/cdi-operator
15-
1613
#simple smoke test
1714
RUN ls /usr/bin/cdi-operator
1815

.pipelines/containerSourceData/cdi/Dockerfile-cdi-uploadproxy

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,6 @@ ARG USER
1010

1111
@INCLUDE_MAIN_RUN_INSTRUCTION@
1212

13-
# workaround till binaries rename is merged
14-
RUN [ -f /usr/bin/virt-cdi-uploadproxy ] && mv -f /usr/bin/virt-cdi-uploadproxy /usr/bin/cdi-uploadproxy
15-
1613
#simple smoke test
1714
RUN ls /usr/bin/cdi-uploadproxy
1815

.pipelines/containerSourceData/cdi/Dockerfile-cdi-uploadserver

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,6 @@ ARG USER
1010

1111
@INCLUDE_MAIN_RUN_INSTRUCTION@
1212

13-
# workaround till binaries rename is merged
14-
RUN [ -f /usr/bin/virt-cdi-uploadserver ] && mv -f /usr/bin/virt-cdi-uploadserver /usr/bin/cdi-uploadserver
15-
1613
#simple smoke test
1714
RUN ls /usr/bin/cdi-uploadserver
1815

SPECS/containerized-data-importer/containerized-data-importer.spec

Lines changed: 16 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
Summary: Container native virtualization
1919
Name: containerized-data-importer
2020
Version: 1.57.0
21-
Release: 9%{?dist}
21+
Release: 10%{?dist}
2222
License: ASL 2.0
2323
Vendor: Microsoft Corporation
2424
Distribution: Azure Linux
@@ -149,20 +149,20 @@ CGO_ENABLED=0 ./hack/build/build-go.sh build \
149149
%install
150150
mkdir -p %{buildroot}%{_bindir}
151151

152-
install -p -m 0755 _out/cmd/cdi-apiserver/cdi-apiserver %{buildroot}%{_bindir}/virt-cdi-apiserver
152+
install -p -m 0755 _out/cmd/cdi-apiserver/cdi-apiserver %{buildroot}%{_bindir}/cdi-apiserver
153153

154154
install -p -m 0755 cmd/cdi-cloner/cloner_startup.sh %{buildroot}%{_bindir}/
155155
install -p -m 0755 _out/cmd/cdi-cloner/cdi-cloner %{buildroot}%{_bindir}/
156156

157-
install -p -m 0755 _out/cmd/cdi-controller/cdi-controller %{buildroot}%{_bindir}/virt-cdi-controller
157+
install -p -m 0755 _out/cmd/cdi-controller/cdi-controller %{buildroot}%{_bindir}/cdi-controller
158158

159-
install -p -m 0755 _out/cmd/cdi-importer/cdi-importer %{buildroot}%{_bindir}/virt-cdi-importer
159+
install -p -m 0755 _out/cmd/cdi-importer/cdi-importer %{buildroot}%{_bindir}/cdi-importer
160160

161-
install -p -m 0755 _out/cmd/cdi-operator/cdi-operator %{buildroot}%{_bindir}/virt-cdi-operator
161+
install -p -m 0755 _out/cmd/cdi-operator/cdi-operator %{buildroot}%{_bindir}/cdi-operator
162162

163-
install -p -m 0755 _out/cmd/cdi-uploadproxy/cdi-uploadproxy %{buildroot}%{_bindir}/virt-cdi-uploadproxy
163+
install -p -m 0755 _out/cmd/cdi-uploadproxy/cdi-uploadproxy %{buildroot}%{_bindir}/cdi-uploadproxy
164164

165-
install -p -m 0755 _out/cmd/cdi-uploadserver/cdi-uploadserver %{buildroot}%{_bindir}/virt-cdi-uploadserver
165+
install -p -m 0755 _out/cmd/cdi-uploadserver/cdi-uploadserver %{buildroot}%{_bindir}/cdi-uploadserver
166166

167167
install -p -m 0755 _out/tools/cdi-containerimage-server/cdi-containerimage-server %{buildroot}%{_bindir}/cdi-containerimage-server
168168

@@ -180,7 +180,7 @@ install -m 0644 _out/manifests/release/cdi-cr.yaml %{buildroot}%{_datadir}/cdi/m
180180
%files api
181181
%license LICENSE
182182
%doc README.md
183-
%{_bindir}/virt-cdi-apiserver
183+
%{_bindir}/cdi-apiserver
184184

185185
%files cloner
186186
%license LICENSE
@@ -191,31 +191,31 @@ install -m 0644 _out/manifests/release/cdi-cr.yaml %{buildroot}%{_datadir}/cdi/m
191191
%files controller
192192
%license LICENSE
193193
%doc README.md
194-
%{_bindir}/virt-cdi-controller
194+
%{_bindir}/cdi-controller
195195

196196
%files importer
197197
%license LICENSE
198198
%doc README.md
199-
%{_bindir}/virt-cdi-importer
199+
%{_bindir}/cdi-importer
200200
%{_bindir}/cdi-containerimage-server
201201
%{_bindir}/cdi-image-size-detection
202202
%{_bindir}/cdi-source-update-poller
203203

204204
%files operator
205205
%license LICENSE
206206
%doc README.md
207-
%{_bindir}/virt-cdi-operator
207+
%{_bindir}/cdi-operator
208208
%{_bindir}/csv-generator
209209

210210
%files uploadproxy
211211
%license LICENSE
212212
%doc README.md
213-
%{_bindir}/virt-cdi-uploadproxy
213+
%{_bindir}/cdi-uploadproxy
214214

215215
%files uploadserver
216216
%license LICENSE
217217
%doc README.md
218-
%{_bindir}/virt-cdi-uploadserver
218+
%{_bindir}/cdi-uploadserver
219219

220220
%files manifests
221221
%license LICENSE
@@ -226,6 +226,9 @@ install -m 0644 _out/manifests/release/cdi-cr.yaml %{buildroot}%{_datadir}/cdi/m
226226
%{_datadir}/cdi/manifests
227227

228228
%changelog
229+
* Mon Feb 03 2025 Sharath Srikanth Chellappa <sharathsr@microsoft.com> - 1.57.0-10
230+
- Rename cdi binaries to be inline with upstream.
231+
229232
* Wed Jan 29 2025 Kanishk Bansal <kanbansal@microsoft.com> - 1.57.0-9
230233
- Fix CVE-2024-28180 with an upstream patch
231234

0 commit comments

Comments
 (0)