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

NO-ISSUE: moving images to be multiplatform #19

Open
wants to merge 43 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
43 commits
Select commit Hold shift + click to select a range
809c74c
OCPBUGSM-30066: Retry wait for operator in case it failed (#319)
machacekondra Jul 4, 2021
56359ed
MGMT-6663: Update progress of operators only when there's a new progr…
YuviGold Jul 5, 2021
df41ab5
NO-ISSUE: Adding sagidayan as a code reviewer (#322)
sagidayan Jul 11, 2021
a905189
Bug 1979009: Change log message about EFI support (#323)
sagidayan Jul 11, 2021
ebf29ac
MGMT-7201: Wrap PostInstallConfigs logs and pass the entire failure m…
YuviGold Jul 11, 2021
706bbf2
Bug 1979009: Change log message about EFI support (#326)
sagidayan Jul 13, 2021
ec000b2
MGMT-7210: Upgrade Go version to 1.16 (#325)
omertuc Jul 14, 2021
e87d07c
NO-ISSUE: Log message before uploading logs to service (#330)
flaper87 Jul 16, 2021
1f26de3
MGMT-7149: Merge wait for operators (#331)
YuviGold Jul 19, 2021
90c1e8d
OCPBUGSM-31942: Return error on failed status operator update (#332)
machacekondra Jul 20, 2021
a1057b1
MGMT-4893: Add Must-Gather reports when olm controllers fail (#329)
slaviered Jul 21, 2021
57c4c94
Bug 1981465: Assisted installer wait for ready master nodes on bootst…
eranco74 Jul 25, 2021
e285f30
OCPBUGSM-27526: Cluster deployment freeze on Finalizing stage forever…
tsorya Jul 27, 2021
5273a2c
MGMT-4893: quote must-gather-image env variable in controller manifes…
slaviered Jul 28, 2021
8c2351a
MGMT-3817: asssited-controller should always send logs (#337)
tsorya Aug 4, 2021
2403dad
MGMT-7452: Remove token from assisted-installer-controller log (#338)
tsorya Aug 5, 2021
424639f
OCPBUGSM-32117: Adding ipv6 support to dns busy address workaroung (#…
tsorya Aug 5, 2021
99a6321
MGMT-7417: Change installer and installer controller to support infra…
yevgeny-shnaidman Aug 8, 2021
f3800cf
MGMT-7450: Removing pull secret token from failure logs (#340)
tsorya Aug 9, 2021
4c04303
NO-ISSUE: remove obsolete installation-timeout parameter (#341)
eranco74 Aug 9, 2021
3f66cb7
OCPBUGSM-31802: hosts that were moved to ready state in k8s, didn't s…
tsorya Aug 10, 2021
52a722d
MGMT-7178: Assisted-installer should not reboot worker till 2 masters…
ori-amizur Aug 17, 2021
f663d71
MGMT-7597: Remove ronniel1, razregev, and asalkeld from OWNERS (#345)
YuviGold Aug 22, 2021
bbbebda
MGMT-7635 Fix logs gathering on SNO when failing to complete bootstra…
eranco74 Aug 22, 2021
e346bc3
MGMT-7292: Wait for the OLM to be initilized before CR apply (#333)
machacekondra Aug 23, 2021
cc8d5bb
OCPBUGSM-33782: Fix the manifest parsing (#347)
machacekondra Aug 25, 2021
f2d2ade
MGMT-7713: Worker won't get updated to 'Waiting for ignition' when in…
eranco74 Sep 1, 2021
3673218
MGMT-7750: get efi file according to cpu architecture (#351)
danielerez Sep 5, 2021
dbbd369
NO-ISSUE: Use -v for efibootmgr to get detailed output (#354)
flaper87 Sep 8, 2021
4f87783
NO-ISSUE: upload controller logs with operator status before must-gat…
tsorya Sep 19, 2021
f4397ab
MGMT-7315: Support IP-based matching of nodes during installation (#359)
mkowalski Sep 22, 2021
6f4c222
MGMT-7504: move assisted-installer/controller to use V2 APIs for comm…
yevgeny-shnaidman Sep 22, 2021
b2d7e1e
MGMT-7760: Switch to stream8 (#360)
flaper87 Sep 23, 2021
be3b5fd
NO-ISSUE: Add lranjbar to OWNERS_ALIASES (#356)
lranjbar Sep 29, 2021
04b57c5
Bug 1966621: Do not use run-level label for assisted-installer namesp…
mkowalski Oct 5, 2021
f88da83
MGMT-4078: Validate console operator in parallel to CVO (#362)
YuviGold Oct 6, 2021
a4fc5e8
NO-ISSUE: Add commit-message prefix NO-ISSUE to dependanbot (#364)
YuviGold Oct 7, 2021
08f2d48
NO-ISSUE: Add `replaces` entry in go.mod for `irifrance/gini` which h…
omertuc Oct 10, 2021
a6debec
Bug 2004633: cluster deployment failed on connection error (#366)
eranco74 Oct 10, 2021
aa6c55f
NO-ISSUE: Add omertuc to code-approvers (#367)
eranco74 Oct 10, 2021
e6d6c7b
Bug 2010183: LogURL field of the AgentClusterInstall is not filled up…
yevgeny-shnaidman Oct 11, 2021
149411c
NO-ISSUE: Remove user yuvigold (#370)
omertuc Oct 12, 2021
87aec67
NO-ISSUE: moving images to be multiplatform
tsorya Jun 28, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,12 @@ updates:
- "ok-to-test"
- "go"
- "dependencies"
commit-message:
prefix: "NO-ISSUE"

- package-ecosystem: "docker"
directory: "/"
schedule:
interval: "weekly"
commit-message:
prefix: "NO-ISSUE"
12 changes: 9 additions & 3 deletions Dockerfile.assisted-installer
Original file line number Diff line number Diff line change
@@ -1,4 +1,9 @@
FROM registry.ci.openshift.org/openshift/release:golang-1.15 AS builder
FROM --platform=$BUILDPLATFORM golang:1.16 AS builder

ARG TARGETPLATFORM

WORKDIR /go/src/github.com/openshift/assisted-installer-agent

ENV GOFLAGS=-mod=mod
WORKDIR /go/src/github.com/openshift/assisted-installer

Expand All @@ -9,9 +14,10 @@ COPY go.sum go.sum
RUN go mod download

COPY . .
RUN make installer

FROM quay.io/centos/centos:centos8
RUN if [ "$TARGETPLATFORM" = "linux/arm64" ]; then GO_BUILD_ENV_PARAMS="GOOS=linux GOARCH=arm64" make installer;else make installer; fi

FROM --platform=$TARGETPLATFORM quay.io/centos/centos:centos8

COPY --from=builder /go/src/github.com/openshift/assisted-installer/build/installer /usr/bin/installer
COPY --from=builder /go/src/github.com/openshift/assisted-installer/deploy/assisted-installer-controller /assisted-installer-controller/deploy
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile.assisted-installer-build
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM registry.ci.openshift.org/openshift/release:golang-1.15
FROM registry.ci.openshift.org/openshift/release:golang-1.16
ENV GO111MODULE=on
ENV GOFLAGS=""

Expand Down
12 changes: 8 additions & 4 deletions Dockerfile.assisted-installer-controller
Original file line number Diff line number Diff line change
@@ -1,7 +1,10 @@
FROM quay.io/openshift/origin-cli:4.9.0 AS cli
FROM --platform=$BUILDPLATFORM quay.io/openshift/origin-cli:4.9.0 AS cli

FROM registry.ci.openshift.org/openshift/release:golang-1.15 AS builder
FROM --platform=$BUILDPLATFORM golang:1.16 AS builder

ARG TARGETPLATFORM
ENV GOFLAGS=-mod=mod

WORKDIR /go/src/github.com/openshift/assisted-installer

# Bring in the go dependencies before anything else so we can take
Expand All @@ -11,9 +14,10 @@ COPY go.sum go.sum
RUN go mod download

COPY . .
RUN make controller

FROM quay.io/centos/centos:centos8
RUN if [ "$TARGETPLATFORM" = "linux/arm64" ]; then GO_BUILD_ENV_PARAMS="GOOS=linux GOARCH=arm64" make controller;else make controller; fi

FROM --platform=$TARGETPLATFORM quay.io/centos/centos:centos8

RUN yum -y install make gcc unzip wget curl rsync && yum clean all
COPY --from=builder /go/src/github.com/openshift/assisted-installer/build/assisted-installer-controller /usr/bin/assisted-installer-controller
Expand Down
10 changes: 5 additions & 5 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ NAMESPACE := $(or ${NAMESPACE},assisted-installer)
GIT_REVISION := $(shell git rev-parse HEAD)
PUBLISH_TAG := $(or ${GIT_REVISION})

CONTAINER_BUILD_PARAMS = --network=host --label git_revision=${GIT_REVISION}
CONTAINER_BUILD_PARAMS = --label git_revision=${GIT_REVISION}

REPORTS ?= $(ROOT_DIR)/reports
CI ?= false
Expand Down Expand Up @@ -53,18 +53,18 @@ endif
build: installer controller

installer:
CGO_ENABLED=0 go build -o build/installer src/main/main.go
$(GO_BUILD_ENV_PARAMS) CGO_ENABLED=0 go build -o build/installer src/main/main.go

controller:
CGO_ENABLED=0 go build -o build/assisted-installer-controller src/main/assisted-installer-controller/assisted_installer_main.go
$(GO_BUILD_ENV_PARAMS) CGO_ENABLED=0 go build -o build/assisted-installer-controller src/main/assisted-installer-controller/assisted_installer_main.go

build-images: installer-image controller-image

installer-image:
$(CONTAINER_COMMAND) build $(CONTAINER_BUILD_PARAMS) -f Dockerfile.assisted-installer . -t $(INSTALLER)
docker buildx build --platform=linux/arm64,linux/amd64 $(CONTAINER_BUILD_PARAMS) -f Dockerfile.assisted-installer . -t $(INSTALLER) --push

controller-image:
$(CONTAINER_COMMAND) build $(CONTAINER_BUILD_PARAMS) -f Dockerfile.assisted-installer-controller . -t $(CONTROLLER)
docker buildx build --platform=linux/arm64,linux/amd64 $(CONTAINER_BUILD_PARAMS) -f Dockerfile.assisted-installer-controller . -t $(CONTROLLER) --push

push-installer: installer-image
$(CONTAINER_COMMAND) push $(INSTALLER)
Expand Down
8 changes: 3 additions & 5 deletions OWNERS_ALIASES
Original file line number Diff line number Diff line change
Expand Up @@ -9,26 +9,24 @@ aliases:
- gamli75
- ori-amizur
- oshercc
- razregev
- romfreiman
- ronniel1
- tsorya
- yevgeny-shnaidman
- yuvigold
- masayag
- nmagnezi
- carbonin
- rollandf
- danielerez
- ybettan
- slaviered
- osherdp
- asalkeld
- flaper87
- mkowalski
- lranjbar
- omertuc
code-reviewers:
- jakub-dzon
- pkliczewski
- masayag
- jordigilh
- machacekondra
- sagidayan
Original file line number Diff line number Diff line change
Expand Up @@ -12,4 +12,4 @@ data:
ca-cert-path: '{{.CACertPath}}'
check-cluster-version: '{{.CheckCVO}}'
high-availability-mode: {{.HaMode}}
must-gather-image: {{.MustGatherImage}}
must-gather-image: '{{.MustGatherImage}}'
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,3 @@ apiVersion: v1
kind: Namespace
metadata:
name: assisted-installer
labels:
openshift.io/run-level: "0"
Original file line number Diff line number Diff line change
Expand Up @@ -154,3 +154,17 @@ rules:
- pods
verbs:
- deletecollection
- apiGroups:
- "security.openshift.io"
resourceNames:
- "anyuid"
- "nonroot"
- "hostmount-anyuid"
- "machine-api-termination-handler"
- "hostnetwork"
- "hostaccess"
- "node-exporter"
resources:
- securitycontextconstraints
verbs:
- use
19 changes: 10 additions & 9 deletions go.mod
Original file line number Diff line number Diff line change
@@ -1,13 +1,14 @@
module github.com/openshift/assisted-installer

go 1.14
go 1.16

require (
github.com/Microsoft/go-winio v0.4.15-0.20200113171025-3fe6c5262873 // indirect
github.com/PuerkitoBio/rehttp v1.0.0
github.com/aybabtme/iocontrol v0.0.0-20150809002002-ad15bcfc95a0 // indirect
github.com/benbjohnson/clock v1.0.3 // indirect
github.com/coreos/ignition/v2 v2.10.1
github.com/go-logr/logr v0.4.0 // indirect
github.com/go-openapi/runtime v0.19.28
github.com/go-openapi/strfmt v0.20.1
github.com/go-openapi/swag v0.19.9
Expand All @@ -16,11 +17,11 @@ require (
github.com/hashicorp/go-version v1.3.0
github.com/kelseyhightower/envconfig v1.4.0
github.com/metal3-io/baremetal-operator v0.0.0
github.com/onsi/ginkgo v1.16.2
github.com/onsi/gomega v1.12.0
github.com/onsi/ginkgo v1.16.4
github.com/onsi/gomega v1.13.0
github.com/openshift/api v3.9.1-0.20191111211345-a27ff30ebf09+incompatible
github.com/openshift/assisted-installer-agent v0.0.0-20200811180147-bc9c7b899b8a
github.com/openshift/assisted-service v1.0.10-0.20210526082015-cf99d1fca3fe
github.com/openshift/assisted-service v1.0.10-0.20210921191140-7dcb60579fdf
github.com/openshift/client-go v0.0.0-20201020074620-f8fd44879f7c
github.com/openshift/machine-api-operator v0.2.1-0.20201002104344-6abfb5440597
github.com/operator-framework/api v0.8.0
Expand All @@ -30,20 +31,20 @@ require (
github.com/thoas/go-funk v0.8.0
github.com/vincent-petithory/dataurl v0.0.0-20191104211930-d1553a71de50
golang.org/x/net v0.0.0-20210428140749-89ef3d95e781
golang.org/x/sync v0.0.0-20201020160332-67f06af15bc9
golang.org/x/sync v0.0.0-20210220032951-036812b2e83c
gopkg.in/yaml.v2 v2.4.0
honnef.co/go/tools v0.0.1-2020.1.6 // indirect
k8s.io/api v0.21.0
k8s.io/apimachinery v0.21.0
k8s.io/api v0.21.1
k8s.io/apimachinery v0.21.1
k8s.io/client-go v12.0.0+incompatible
sigs.k8s.io/controller-runtime v0.8.3
sigs.k8s.io/controller-runtime v0.9.0
)

replace (
github.com/metal3-io/baremetal-operator => github.com/openshift/baremetal-operator v0.0.0-20200715132148-0f91f62a41fe // Use OpenShift fork
github.com/openshift/api => github.com/openshift/api v0.0.0-20200901182017-7ac89ba6b971
github.com/openshift/hive/pkg/apis => github.com/carbonin/hive/pkg/apis v0.0.0-20210209195732-57e8c3ae12d1
github.com/openshift/machine-api-operator => github.com/openshift/machine-api-operator v0.2.1-0.20201026110925-50ea569da51b
github.com/irifrance/gini => github.com/go-air/gini v1.0.1
k8s.io/api => k8s.io/api v0.19.2
k8s.io/apimachinery => k8s.io/apimachinery v0.19.2
k8s.io/client-go => k8s.io/client-go v0.19.2
Expand Down
Loading