Skip to content

Commit

Permalink
Update builder image
Browse files Browse the repository at this point in the history
* go 1.18 -> 1.20
* node 4.16.0 -> 4.21.3
* kubectl 1.20.4 -> 1.27.1
  • Loading branch information
spadgett committed May 18, 2023
1 parent 3e0bb09 commit fab8b77
Show file tree
Hide file tree
Showing 10 changed files with 18 additions and 18 deletions.
6 changes: 3 additions & 3 deletions .ci-operator.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
build_root_image:
name: release
namespace: openshift
tag: tectonic-console-builder-v24
name: tectonic-console-builder
namespace: ci
tag: v25
6 changes: 3 additions & 3 deletions Dockerfile.builder
Original file line number Diff line number Diff line change
Expand Up @@ -8,15 +8,15 @@
# You can test the image using `./builder-run.sh`. For instance:
# $ ./builder-run.sh ./build-backend.sh

FROM golang:1.18-stretch
FROM golang:1.20-bullseye

MAINTAINER Ed Rooth - CoreOS

### For golang testing stuff
RUN go install github.com/jstemmer/go-junit-report@latest

### Install NodeJS and yarn
ENV NODE_VERSION="v14.16.0"
ENV NODE_VERSION="v14.21.3"
ENV YARN_VERSION="v1.22.10"

# yarn needs a home writable by any user running the container
Expand All @@ -30,7 +30,7 @@ RUN apt-get update \
libgtk2.0-0 libgtk-3-0 libgbm-dev libnotify-dev libgconf-2-4 libnss3 libxss1 libasound2 libxtst6 xauth xvfb
# ^^ additional Cypress dependencies: https://docs.cypress.io/guides/guides/continuous-integration.html#Dependencies

RUN curl -LO https://storage.googleapis.com/kubernetes-release/release/v1.20.4/bin/linux/$(go env GOARCH)/kubectl && \
RUN curl -LO https://storage.googleapis.com/kubernetes-release/release/v1.27.1/bin/linux/$(go env GOARCH)/kubectl && \
chmod +x ./kubectl && \
mv ./kubectl /usr/local/bin/kubectl

Expand Down
2 changes: 1 addition & 1 deletion Dockerfile.dev
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Dockerfile to build console image from pre-built front end.

FROM quay.io/coreos/tectonic-console-builder:v24 AS build
FROM quay.io/coreos/tectonic-console-builder:v25 AS build
RUN mkdir -p /go/src/github.com/openshift/console/
ADD . /go/src/github.com/openshift/console/
WORKDIR /go/src/github.com/openshift/console/
Expand Down
2 changes: 1 addition & 1 deletion Dockerfile.plugins.demo
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
# See dynamic-demo-plugin/README.md for details.

# Stage 0: build the demo plugin
FROM quay.io/coreos/tectonic-console-builder:v24 AS build
FROM quay.io/coreos/tectonic-console-builder:v25 AS build

RUN mkdir -p /src/console
COPY . /src/console
Expand Down
2 changes: 1 addition & 1 deletion builder-run.sh
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ set -e
# Without env vars:
# ./builder-run.sh ./my-script --my-script-arg1 --my-script-arg2

BUILDER_IMAGE="quay.io/coreos/tectonic-console-builder:v24"
BUILDER_IMAGE="quay.io/coreos/tectonic-console-builder:v25"

# forward whitelisted env variables to docker
ENV_STR=()
Expand Down
2 changes: 1 addition & 1 deletion go.mod
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
module github.com/openshift/console

go 1.18
go 1.20

require (
github.com/coreos/go-oidc v2.1.0+incompatible
Expand Down
2 changes: 1 addition & 1 deletion pkg/server/resource_lister.go
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ type resourceLister struct {
responseFilter FilterFunction
}

//HandleResources handles resource requests
// HandleResources handles resource requests
func (l *resourceLister) HandleResources(w http.ResponseWriter, r *http.Request) {
if r.Method != "GET" {
serverutils.SendResponse(w, http.StatusMethodNotAllowed, serverutils.ApiError{Err: "invalid method: only GET is allowed"})
Expand Down
4 changes: 2 additions & 2 deletions pkg/serverconfig/metrics.go
Original file line number Diff line number Diff line change
Expand Up @@ -190,7 +190,7 @@ func (m *Metrics) calculatePluginInfo(

if lastPluginInfo != nil {
for lastPluginVendor, lastPluginStates := range *lastPluginInfo {
for lastPluginState, _ := range lastPluginStates {
for lastPluginState := range lastPluginStates {
if pluginInfo[lastPluginVendor] == nil {
pluginInfo[lastPluginVendor] = make(map[PluginState]int)
}
Expand Down Expand Up @@ -223,7 +223,7 @@ func (m *Metrics) calculatePluginInfo(
}

if m.config != nil && m.config.Plugins != nil {
for pluginName, _ := range m.config.Plugins {
for pluginName := range m.config.Plugins {
if found := consolePluginNames[pluginName]; !found {
vendor := knownPluginVendors[pluginName]
if vendor == "" {
Expand Down
6 changes: 3 additions & 3 deletions pkg/usersettings/helpers.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ func newUserSettingMeta(userInfo *unstructured.Unstructured) (*UserSettingMeta,
if uid != "" {
resourceIdentifier = string(uid)
ownerReferences = []meta.OwnerReference{
meta.OwnerReference{
{
APIVersion: userInfo.GetAPIVersion(),
Kind: userInfo.GetKind(),
Name: name,
Expand Down Expand Up @@ -51,7 +51,7 @@ func createRole(userSettingMeta *UserSettingMeta) *rbac.Role {
OwnerReferences: userSettingMeta.OwnerReferences,
},
Rules: []rbac.PolicyRule{
rbac.PolicyRule{
{
APIGroups: []string{
"", // Core group, not "v1"
},
Expand Down Expand Up @@ -84,7 +84,7 @@ func createRoleBinding(userSettingMeta *UserSettingMeta) *rbac.RoleBinding {
OwnerReferences: userSettingMeta.OwnerReferences,
},
Subjects: []rbac.Subject{
rbac.Subject{
{
APIGroup: "rbac.authorization.k8s.io",
Kind: "User",
Name: userSettingMeta.Username,
Expand Down
4 changes: 2 additions & 2 deletions pkg/usersettings/helpers_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ func TestNewUserSettingsMeta(t *testing.T) {
UID: "1234",
ResourceIdentifier: "1234",
OwnerReferences: []meta.OwnerReference{
meta.OwnerReference{
{
APIVersion: "user.openshift.io/v1",
Kind: "User",
Name: "kube:admin",
Expand All @@ -81,7 +81,7 @@ func TestNewUserSettingsMeta(t *testing.T) {
UID: "1234",
ResourceIdentifier: "1234",
OwnerReferences: []meta.OwnerReference{
meta.OwnerReference{
{
APIVersion: "user.openshift.io/v1",
Kind: "User",
Name: "developer",
Expand Down

0 comments on commit fab8b77

Please sign in to comment.