profile
viewpoint
cnorman clnperez @IBM Austin, TX IBMer and RH Partner Engineer

clnperez/cli 3

The Docker CLI

clnperez/envoy-ppc64le 1

Build envoy for power

clnperez/abseil-cpp 0

Abseil Common Libraries (C++)

clnperez/app-pet-breed-demo 0

nvidia-docker version of the pet breed demo app from https://github.com/Junsong-Wang/pet-breed

clnperez/backward-cpp 0

A beautiful stack trace pretty printer for C++

clnperez/bazel 0

a fast, scalable, multi-language and extensible build system

clnperez/bazel-gazelle 0

Gazelle is a Bazel build file generator for Go projects.

clnperez/ci-tools 0

DPTP Tooling

clnperez/cni 0

Istio CNI to setup kubernetes pod namespaces to redirect traffic to sidecar proxy.

clnperez/compose 0

Define and run multi-container applications with Docker

Pull request review commentjaegertracing/jaeger

Build binaries for Linux on IBM Power pcc64le architecture

 ALL_SRC := $(shell find . -name '*.go' \ # ALL_PKGS is used with 'go cover' and 'golint' ALL_PKGS := $(shell go list $(sort $(dir $(ALL_SRC)))) UNAME := $(shell uname -m)-#Race flag is not supported on s390x architecture-ifeq ($(UNAME), s390x)+#Race flag is not supported on s390x/ppc64le architecture+ifeq ($(UNAME),$(filter $(UNAME), s390x ppc64le))

actually -- and sorry for not catching this sooner, the race flag has been supported for Power for a few releases now. I've asked someone else to carry this PR and will have her link to this one when she submits. Thanks @yurishkuro for the review.

redmark-alt

comment created time in 4 days

pull request commentdocker/distribution-library-image

Multi-Arch Builds via Multi-Stage

@dmcgowan if memory serves me correctly, there are some registry tests in moby ci -- and power and z do have jobs there (e.g. https://ci-next.docker.com/public/blue/organizations/jenkins/moby/detail/PR-41155/3/pipeline). It's not the distribution CI but it's not nothing. :D

james-crowley

comment created time in 8 days

pull request commentopenshift/installer

maintenance: remove pools and volumes

@cfergeau is mergebot not going to take this one?

clnperez

comment created time in 11 days

issue commentdocker/for-linux

docker 18.09.x missing packages for power platform

@lazystone That's all ongoing. It's just been very bogged down with process. I appreciate the enthusiasm, though. :D

mkumatag

comment created time in 12 days

push eventclnperez/release

Christy Norman

commit sha a1fa1a6528a7902bff809252e6bfa83d9de327c3

use cron to align ppc64le libvirt jobs So that we can be careful of system usage, restrict these jobs to run every twelve hours, staggered by 4 hours (by release). Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 16 days

PR opened openshift/release

use cron to align ppc64le libvirt jobs

So that we can be careful of system usage, restrict these jobs to run every twelve hours, staggered by 4 hours (by release).

Signed-off-by: Christy Norman christy@linux.vnet.ibm.com

+2 -0

0 comment

2 changed files

pr created time in 16 days

create barnchclnperez/release

branch : ppc64le-libvirt-cron

created branch time in 16 days

push eventclnperez/release

Christy Norman

commit sha 2be6ff07fd6252dddcdeae5952931ad0d8917f65

reduce to four boskos leases Temporarily, until we have more capcity in place. Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 16 days

pull request commentopenshift/installer

maintenance: remove pools and volumes

@cfergeau -- missed your review earlier, but just made that changed and pushed it.

clnperez

comment created time in 16 days

push eventclnperez/installer

David Igou

commit sha 02e050f30658d2e168e9f430b63509ccec1c9cde

upi: update upi/vsphere/README.md Clarify that a DHCP server is still required at the beginning of the installation to download the ignition files. In response to #2552

view details

David Eads

commit sha 5259a9d0abd106a22a1a9e627ea2c3befa3babbf

bootstrap: restore 30 minute bootstrap timeout We made improvements in the etcd operator to bring the bootstrap timeout back to 30 minutes.

view details

Dan Winship

commit sha 4a9769fbffb04b8c6fada6acc19c5707d35d4cec

Give a proper error when misusing OPENSHIFT_INSTALL_AZURE_EMULATE_SINGLESTACK_IPV6

view details

Martin André

commit sha 1aedcd8aa9d10e91537786c8314b8cb52b3492b7

OpenStack: Remove DNS VIP Depends on https://github.com/openshift/machine-config-operator/pull/1666

view details

Adrian Moreno

commit sha 7827bab98c2b8fe474d523fe2ee0bb4ec5bd6d39

GCP: add support for image licenses Add an optional parameter in the GCP install-config that contains a list of license URLs to be added to the compute image Credits: Based on the work by Colin Walters Signed-off-by: Adrian Moreno <amorenoz@redhat.com>

view details

W. Trevor King

commit sha 0344dd85c45262066cb1326c3cb60c679809b3e1

cmd/openshift-install: Drop unused 'directory' arguments Looks like I forgot to clean these up when pivoting from destroyBootstrap to waitForBootstrapComplete and removing the only directory consumer in bfc40f87fb (cmd/openshift-install/upi: Add a user-provided-infrastructure subcommand, 2019-03-08, #1393).

view details

John Hixson

commit sha dfd34eb0cb0dc47b23c60f13dbac29b069e9af1b

aws: update the bootstrap ignition fetching to use custom region endpoints Update the S3 bucket that stores the ignition config to use a presigned URL. This allows the S3 bucket to be accesseed via HTTP(s) similar to Azure and GCP thus allowing the installer to pick the correct endpoint based on region/user specification. https://issues.redhat.com/browse/CORS-1322

view details

Luis Tomas Bolivar

commit sha 3c301690d71597e12be71c31b0ad1e8815ab140f

Add information about ovn-octavia

view details

Ben Nemec

commit sha 32382506d1dd36a46d5cf944f2b8e2143a1754d2

baremetal: Remove DNS VIP Use of this setting is being removed in [0]. Once that goes in we can remove it from the installer as well. 0: https://github.com/openshift/machine-config-operator/pull/1569

view details

patrickdillon

commit sha b652e585a46630647679ad96991af3ae44b5ae49

Bug 1836042: set folder absolute path in vSphere cloud provider This is a partial revision of 814f64f4bce13a1c4472a104f72b7658b0e76abe, where we update to use an absolute path rather than a relative path. Using a relative path results in an error when trying to provision a PVC and a subfolder was used for install.

view details

patrickdillon

commit sha e5e94e017727c3d1aca249faed56031f0f977e84

vendor: replace terraform-provider-vsphere with OpenShift fork With https://bugzilla.redhat.com/show_bug.cgi?id=1825323 we see terraform failing with an inconsistent plan when the vSAN storage policy enforces thin-provisioned disks but for some reason the terraform provider produces a plan stating they are thick-provisioned. Terraform docs indicates these types of problems require a fix to CustomizeDiff in the provider: https://www.terraform.io/docs/extend/terraform-0.12-compatibility.html#inaccurate-plans In a recent PR https://github.com/terraform-providers/terraform-provider-vsphere/pull/1052 we see that when the VM is being imported from an OVA, the DiskDiffOperation is simply skipped. DiskDiffOperation is the function that produces the inconsistent plan in the bugzilla mentioned before. We have created a similar PR to skip DiskDiffOperation when VMs are being cloned: https://github.com/terraform-providers/terraform-provider-vsphere/pull/1075 This commit carries that patch until it has merged upstream or resolved otherwise The commit was created by running: go mod edit -replace github.com/terraform-providers/terraform-provider-vsphere=github.com/openshift/terraform-provider-vsphere@v1.18.1-openshift-1 go mod edit -replace github.com/vmware/govmomi=github.com/vmware/govmomi@v0.22.2-0.20200420222347-5fceac570f29 go mod tidy go mod vendor This commit also updates the vsphereprivate provider to use the updated libraries.

view details

Abhinav Dahiya

commit sha 4781ef62d6d30f363a1562345deaf8d726515089

pkg: add a diagnostics error The diagnostics error allows the providing important context to provide better error reporting for the the users. The error allows the installer assets etc. to provide structural information, - Source: the source of the error, the installer assets have errors from cloud providers or internal errors, the source allows providing hat context to better categorize these errors. - Reason: is a single word reason that corrrectly summarizes the type of error, allows the users to quickly understand the type of error. also should allow internal metrics tracking to tracks these error types.

view details

Abhinav Dahiya

commit sha 699becad7cdfad0c6241fc18e92fd5661f777340

pkg/terraform: add diagnostics errors for terraform apply operations The terraform errors are tracked in a buffer. This buffers is then used to match against various known conditions to understand the reasons for the errors. This now allows the terraform apply to return specific errors in these cases instead of previous `failed to apply Terraform` constant string message.

view details

patrickdillon

commit sha ea5e1bf4c2b2c7f1ce85525bc953ab59f5bb72c2

Bug 1834966: update vSphere UPI docs to remove machinesets To enable infrastructure provisioning in vSphere, the installer now creates machine object and machineset manifests. This commit updates UPI docs to remove those manifests, as the skew between the installer produced manifests and user-provisioned control-plane causes errors.

view details

Abhinav Dahiya

commit sha e91df626c10e569e7613249053b7b9b264db42df

installconfig: relax GCP cluster id restriction The infra id of the clusters on GCP was reduced to 12 in https://github.com/openshift/installer/pull/2088 because we couldn't handle the hostname seen by rhcos machine to be greater than 64. More details on this are available in https://bugzilla.redhat.com/show_bug.cgi?id=1809345 now since BZ 1809345 is fixed by https://github.com/openshift/machine-config-operator/pull/1711 and https://github.com/openshift/cluster-api-provider-gcp/pull/88 the installer can relax the restriction on the infra-id to match the other platforms. Why is it important? On GCP all resources are prefixed with infra-id, which currently is 12 chars with 6 chars used by random bit, leaving only 6 chars from cluster name. This causes trouble associating the cluster to jobs in CI as most of the identifyable characters are dropped from the resource names in CI due to this restriction. Also because of the previous restriction, only one char are used from pool's name, making is higly likely to collide in cases there are more.

view details

OpenShift Merge Robot

commit sha b4d4379684a870872c7fc21fd6976eff6f90363c

Merge pull request #3619 from patrickdillon/bz-1834966-vsphere-upi-machinesets Bug 1834966: update vSphere UPI docs to remove machinesets

view details

Roy Golan

commit sha e0c0ee856135e1811fbf9400f294dcf7551258be

docs: oVirt install_ipi.md revisit install_ipi.md Signed-off-by: Roy Golan <rgolan@redhat.com>

view details

OpenShift Merge Robot

commit sha 2dc5c6266f836e7009fea55b9381690b7fa772ce

Merge pull request #3603 from patrickdillon/vsphere-vsan-bz-1825323 Bug 1825323: replace terraform-provider-vsphere with OpenShift fork

view details

Stephen Benjamin

commit sha be1f13580b593b2ea1829f292549b7fd1604a6a0

gather: enable bootstrap log gathering for baremetal This enables bootstrap log gathering for baremetal. We fetch the IP's for the control plane hosts using ironic introspection data, and pick the best one. We prefer to find an IP in a machine network, but otherwise we'll use the first valid IP found. For the bootstrap, the only IP address we can know is the provisioning IP for the bootstrap.

view details

Stephen Benjamin

commit sha 4550c2512d14b8715e29c49ca10c2d29e3e01535

vendor: update terraform-provider-ironic to v0.2.1 This updates terraform-provider-ironic to 0.2.1. This version of the terraform provider has several fixes that we need related to introspection. The installer will now use introspection data to collect IP's for the control plane hosts for bootstrap log gathering. After updating the version in go.mod, I ran `go mod vendor` and `go mod tidy`.

view details

push time in 16 days

push eventclnperez/installer

Christy Norman

commit sha f570343650a201145df12741ac88af73c8405d0e

maintenance: remove pools and volumes This will clean up all volumes under all non-default pools. The openshift CI creates a pool for each cluster. Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 16 days

pull request commentopenshift/release

restrict ppc64le libvirt env to 4 bastion services

@jaypoulz is this good enough to keep us down to 4 jobs max at a time?

clnperez

comment created time in 16 days

PR opened openshift/release

restrict ppc64le libvirt env to 4 bastion services

Temporarily, until we have more capcity in place.

Signed-off-by: Christy Norman christy@linux.vnet.ibm.com

+0 -23

0 comment

1 changed file

pr created time in 16 days

create barnchclnperez/release

branch : four-max-bastion-svcs

created branch time in 16 days

issue openedprometheus/promu

crossbuild platform flag only accepts two platforms

Hopefully I'm not just misusing the flags. I think I remember this working fine on this laptop last week (though before I upgraded to Fedora 32).

promu crossbuild --platforms=linux/ppc64le linux/s390x linux/amd64 -v
promu: error: unexpected linux/amd64, try --help
promu crossbuild -v --platforms=linux/ppc64le linux/s390x linux/amd64
promu: error: unexpected linux/amd64, try --help
promu crossbuild -v --platforms=linux/ppc64le darwin/amd64 linux/s390x
promu: error: unexpected linux/s390x, try --help
promu crossbuild --platforms=linux/ppc64le darwin/amd64 linux/s390x linux/s390x
promu: error: unexpected linux/s390x, try --help

It complains about whatever you put in the last spot.

os: fedora 32 arch: x86_64 docker version: 19.03.11 go version: go version go1.13.9 linux/amd64

promu downloaded from https://github.com/prometheus/promu/releases. Both v0.4 and v0.5 fail the same way.

created time in 25 days

pull request commentopenshift/release

Enable oc adm must-gather tests

/lgtm

mkumatag

comment created time in a month

pull request commentopenshift/release

Fix skip test list

/retest

mkumatag

comment created time in a month

PR closed openshift/release

Reviewers
Testing pr 9179
+25 -25

1 comment

2 changed files

clnperez

pr closed time in a month

pull request commentdocker/docker-credential-helpers

Enabled ppc64le as an architecture

Hi @vrushaliinamdar -- this is a dup of https://github.com/docker/docker-credential-helpers/issues/144. Have you considered submitting a PR for this update yourself? We know that this project does build easily on ppc64le.

vrushaliinamdar

comment created time in a month

PR opened openshift/release

Testing pr 9179
+25 -25

0 comment

2 changed files

pr created time in a month

create barnchclnperez/release

branch : testing-pr-9179

created branch time in a month

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

Can this be merged? The jobs run (:tada:) , and @mkumatag is working on paring down the failing tests for 4.4 in a separate PR.

clnperez

comment created time in a month

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

/retest

clnperez

comment created time in a month

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

/retest

clnperez

comment created time in a month

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

/retest

clnperez

comment created time in a month

pull request commentopenshift/installer

maintenance: remove pools and volumes

It doesn't seem like any of those failures are related. The tf-lint one says it passed. :woman_shrugging: @jcpowermac @jhixson74 can you chime in?

clnperez

comment created time in a month

pull request commentopenshift/installer

maintenance: remove pools and volumes

/retest

clnperez

comment created time in a month

push eventclnperez/installer

Christy Norman

commit sha eb67ba1fe122444a8a15359b3e51ac744c4cd201

maintenance: remove pools and volumes This will clean up all volumes under all non-default pools. The openshift CI creates a pool for each cluster. Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in a month

PR opened openshift/installer

remove pools and volumes

This will clean up all volumes under all non-default pools. The openshift CI creates a pool for each cluster.

Signed-off-by: Christy Norman christy@linux.vnet.ibm.com

+9 -1

0 comment

1 changed file

pr created time in a month

create barnchclnperez/installer

branch : libvirt-pool-cleanup

created branch time in a month

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

https://github.com/openshift/release/pull/9097 merged. still waiting on some backports

clnperez

comment created time in 2 months

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

split out another part of this. now need https://github.com/openshift/release/pull/9097 to land before this one

clnperez

comment created time in 2 months

push eventclnperez/release

Christy Norman

commit sha d0ab2ce800b145ce9e0f30a92bcfff57bb809fdc

patched installer image Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

PR opened openshift/release

enable lease usage by ppc64le jobs

So that Power libvirt jobs can run multiple simultaneous leases.

Signed-off-by: Christy Norman christy@linux.vnet.ibm.com

+2 -2

0 comment

1 changed file

pr created time in 2 months

create barnchclnperez/release

branch : libvirt-template-lease-ppc64le

created branch time in 2 months

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

/retest

clnperez

comment created time in 2 months

push eventclnperez/release

Christy Norman

commit sha 1cc88ec33e01300138bf719bf4cda14db7659bcc

patched installer image Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

push eventclnperez/release

Christy Norman

commit sha 0882764168e7a37966aab1b6f2aa537ebb41ad59

patched installer image Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

push eventclnperez/dockerfiles

Christy Norman

commit sha acd74869b05fc265080067a2355c72c6f1c3516d

pushing up local debug work from months ago Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

pull request commentopenshift/release

Update libvirt job for 4.4 release branch

@praveenkumar do you have an ETA on this one? I need it for https://github.com/openshift/release/pull/8919.

praveenkumar

comment created time in 2 months

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

/retest

clnperez

comment created time in 2 months

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

/retest

clnperez

comment created time in 2 months

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

/retest

clnperez

comment created time in 2 months

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

/retest

clnperez

comment created time in 2 months

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

/retest

clnperez

comment created time in 2 months

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

/retest

clnperez

comment created time in 2 months

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

/retest

clnperez

comment created time in 2 months

pull request commentopenshift/release

Parameterize the installer image used for ppc64le OpenStack testing

/lgtm

hamzy

comment created time in 2 months

pull request commentopenshift/release

add ppc64le-libvirt to boskos resources

@petr-muller @droslean PTAL This is needed for https://github.com/openshift/release/pull/8919

clnperez

comment created time in 2 months

push eventclnperez/release

Christy Norman

commit sha 17c2f16782841558573861b5b2227568131d9832

add ppc64le-libvirt to boskos resources For use with the Power CI libvirt jobs, in order to have concurrent release testing. Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

push eventclnperez/release

Christy Norman

commit sha f7badcba8173cb9886dd6386ac6097d0232f00e2

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

PR opened openshift/release

add ppc64le-libvirt to boskos resources

For use with the Power CI libvirt jobs, in order to have concurrent release testing.

Signed-off-by: Christy Norman christy@linux.vnet.ibm.com

+10 -2

0 comment

3 changed files

pr created time in 2 months

push eventclnperez/release

Christy Norman

commit sha b69f1ca3507d4656f64f0b68b68ec57c045554fc

add ppc64le-libvirt to boskos resources For use with the Power CI libvirt jobs, in order to have concurrent release testing. Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

create barnchclnperez/release

branch : ppc64le-libvirt-boskos

created branch time in 2 months

push eventclnperez/release

Mark Hamzy

commit sha 908808cbdd2e67ffb8bcf3c87fd8cd1987de8513

Fix wrong ConfigMap for prow-job-cluster-launch-installer-openstack-e2e-ppc64le

view details

Mark Hamzy

commit sha 861af955987e37a6990652f1b9892ea7d99d7e5f

Use sshd-43.bastion-ppc64le as hostAliases NOTE: I cannot do something like the following: echo $(nslookup sshd-43.bastion-ppc64le | grep 172 | awk '{print $2}') or echo $(nslookup sshd-43.bastion-ppc64le | awk -F': ' 'NF==2 { print $2 }') because it is part of the pod spec of openshift-release-release-4.3-periodics.yaml

view details

akoserwa

commit sha 4d84713e63a14a8c3c94ce3071e3ff39a062f5ee

adding config & job for rhi-operator

view details

akoserwa

commit sha 2d6beeaa10cf44077aaf150e902db6beefd18452

fixing the name

view details

akoserwa

commit sha 2a02e04bbcb1b5126bcaba530a5029fb00eb4137

adding owners config

view details

akoserwa

commit sha 0b29872957b95bfa9ce019b00028984ab5a8dbdf

fixing owners file format

view details

akoserwa

commit sha f943fcac16d03da25fd2008618b9cc9a165525f6

moving owners file to rhi directory

view details

akoserwa

commit sha b623c70c41d359e778f65f5d6bc194b9022e0273

removing sentry-dsn by make jobs regenerating

view details

akoserwa

commit sha 12ba50bee91ec96e5bca2fc1ad33834cea4bb789

updating reference to integreatly

view details

akoserwa

commit sha 56bc67b32a3615755f9ca5d6b1b452225965e2e0

changing reference to rhi

view details

akoserwa

commit sha b99070b10adc9ffa4a4ce5efb090ad5f20c2d405

re-run make jobs

view details

akoserwa

commit sha 8e94455f39ec07ccb0a6bbd82b7e47b340d36e2d

adding roles and role binding

view details

akoserwa

commit sha 23d49e42e4f3f3fc8fdf01f651e983f6a49d33cb

changes by make config job

view details

akoserwa

commit sha 172eda86c53ea4ed2ca004e4eac83abec1a05380

updating compile build reference

view details

akoserwa

commit sha 063443d975ae06cde8ad01009a7abf75eda5d988

updating config

view details

akoserwa

commit sha 2640f0994e5e8b09b249af082c68a10c0a2194a2

updating the compile target name change

view details

akoserwa

commit sha 51052c480f9a658ac69c8d1616bf01c73873da11

updating target build name

view details

akoserwa

commit sha cc9269d82842f69d9f3d927b965baebc0240cca7

add export component for operator image

view details

akoserwa

commit sha 48b06afbecff0c48aca4aa0481e3026ec4652e89

fixing identifier

view details

akoserwa

commit sha e02db8f19a5aef34a13aba9ea04c3adaf6f24799

updating jobs

view details

push time in 2 months

pull request commentopenshift/release

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

/retest

clnperez

comment created time in 2 months

push eventclnperez/release

Christy Norman

commit sha 4e427aa6673a562f8c833cbf1cd62042d1ff5643

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

push eventclnperez/release

Christy Norman

commit sha d07bc305cd75f3d26edb7bd50d00bdcd9e2284d4

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

push eventclnperez/release

Dr. Stefan Schimanski

commit sha fd9fd2a1d40d60556e127ba8867fd7cb9e3a7795

ci-operator/jobs: move openshift-release-release-4.4-periodics to broken bucket

view details

Phil Cameron

commit sha 3e85e8a4c6eea1f14375b621ed752b1fc4d213ac

CI Artifacts - add output from "oc get" The commands that CI extra runs capture the results in .json format which is complete but not easy to read. This captures the reports in files in the (new) oc_cmds directory. https://issues.redhat.com/browse/SDN-926 Signed-off-by: Phil Cameron <pcameron@redhat.com>

view details

Matthias Wessendorf

commit sha 855cf6c1864b235c7437764ab598f812d4612c1a

Adding the missing upgrader image Signed-off-by: Matthias Wessendorf <mwessend@redhat.com>

view details

Federico Paolinelli

commit sha cc56d1959d8878b44c43360063c601f0cc302b98

Bump 4.5 changes and align master as 4.6 branch. This includes removing the sctptester image as it's being consumed directly from quay mirror. Signed-off-by: Federico Paolinelli <fpaoline@redhat.com>

view details

Paul Morie

commit sha 019aea65ebf59365949dde3cd3708d79ba0127c9

Enable tide for ocm/nucleus

view details

Ray Harris

commit sha 0c31812eaf173db5a24da2cb8abbc608e488f1eb

acm: mco-app add image

view details

Ray Harris

commit sha 942e987e2540af1e629ef5d207eb8fd6b72a7170

acm: mco-app fix config metadata

view details

Ray Harris

commit sha 383088d2bf2c59ef7b149ccb81c9839aeb1cef43

acm: mco-app update images.from

view details

Ray Harris

commit sha a4b80f39cab6f6fcea56c804e676165edd34967e

acm: mco-app update images.from

view details

Christy Norman

commit sha 30b8a54e7103813c831c8c18922ffde83bf6da02

add clnperez as sshd bastion owner for p libvirt Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

openshift-bot

commit sha 701145fdae3dfe6c94e457f340fbf72706ac5f93

config-brancher --current-release 4.5 --future-release 4.6

view details

Ray Harris

commit sha 4dad01c8b482b618c4ccf9aa63ee045fd0ea91ab

acm: mco-app update binary build command

view details

openshift-bot

commit sha 1c276fae8ffa0b64c0aef6e152148a4a2e571b92

ci-operator-config-mirror --config-path ./ci-operator/config --to-org openshift-priv

view details

openshift-bot

commit sha 3c7ddfe506c4e312c4da2f6ce9e794593f8e0c14

determinize-ci-operator --confirm

view details

OpenShift Merge Robot

commit sha 25b623506ad675ab6a5e88992e6d3bc875ad9a58

Merge pull request #8564 from clnperez/clnperez-libvirt-p-admin add clnperez as sshd bastion owner for p libvirt

view details

OpenShift Merge Robot

commit sha 4843d7e8e7b31a1f89bb64f7d6e47d06e096b2c1

Merge pull request #8913 from openshift-bot/auto-config-brancher Automate config brancher by auto-config-brancher job at Wed, 06 May 2020 20:02:34 UTC

view details

OpenShift Merge Robot

commit sha 8b04960174ca328eb3e83c559f56e3f30975f595

Merge pull request #8909 from pmorie/ocm-nucleus-tide Enable tide for ocm/nucleus

view details

openshift-bot

commit sha cc8d30b9b457e01693423d890ce26e430c8e609f

config-brancher --current-release 4.5 --future-release 4.6

view details

openshift-bot

commit sha 188b1f064519f348d94f27eface9eb9feec9e2a4

ci-operator-config-mirror --config-path ./ci-operator/config --to-org openshift-priv

view details

openshift-bot

commit sha 8fd5d631fc8543d834dfd649c7821156264000f6

determinize-ci-operator --confirm

view details

push time in 2 months

pull request commentopenshift/ci-tools

add remote libvirt profile for ppc64le

/retest

clnperez

comment created time in 2 months

pull request commentopenshift/ci-tools

add remote libvirt profile for ppc64le

looks like this is hitting the flaky test mentioned here: https://github.com/openshift/ci-tools/pull/776

@alvaroaleman @petr-muller -- can you give this one a review? It's needed before we can get https://github.com/openshift/release/pull/8919 int

clnperez

comment created time in 2 months

pull request commentopenshift/ci-tools

add remote libvirt profile for ppc64le

/retest

clnperez

comment created time in 2 months

push eventclnperez/ci-tools

Christy Norman

commit sha 68c3b8fc98bd7be7af7afbcd376ed395b73db866

add remote libvirt profile for ppc64le Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

push eventclnperez/release

Christy Norman

commit sha 5c2cabb1aaa4f5fddb497b03bea425961542eff6

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

push eventclnperez/release

Christy Norman

commit sha 89049768d41e63cec07e430f97aa49189f97337d

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

push eventclnperez/release

Christy Norman

commit sha 8dd424752f2ab689798c485b5216a3ab097f4085

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

push eventclnperez/release

Christy Norman

commit sha 5be8781fc105e032979b402932d8a4ed20cc332f

add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

PR opened openshift/ci-tools

add remote libvirt profile for ppc64le

Signed-off-by: Christy Norman christy@linux.vnet.ibm.com

+11 -3

0 comment

3 changed files

pr created time in 2 months

create barnchclnperez/ci-tools

branch : ppc64le-libvirt-profile

created branch time in 2 months

push eventclnperez/ci-tools

Christy Norman

commit sha 9b39a2e5b7d1af38409f531ce4950e0b771c5047

deprovision a libvirt cluster this is for the p system currently testing the 4.3 releases. the plan is to be able to set additional ips if we have different bastions. we could add an outer for-loop for that in the future if needed. the journaclctl way of checking for age/existence in time is there because there is no way to get the age of a libvirt resource unless metadata was added to the resource upon creation. Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

pull request commentopenshift/release

add clnperez as sshd bastion owner for p libvirt

I'm g2g. Thanks @jaypoulz

clnperez

comment created time in 2 months

pull request commentopenshift/release

[WIP] add concurrent 4.3 and 4.4 e2e jobs for ppc64le-libvirt

@jaypoulz is there anything else I need to do PR-wise for these? I still need to set up the haproxy and add the right IPs to the ssh tunnel once that's done. What about anything for leases, etc.?

We've got enough capacity in our environment to handle the multiple releases, too, JFYI.

clnperez

comment created time in 2 months

create barnchclnperez/release

branch : ppc64le-libvirt-concurrent

created branch time in 2 months

pull request commentmulti-arch/ocp-remote-ci

add ppc64le-libvirt configs

Just figured it would be good to keep these in the same place!

clnperez

comment created time in 2 months

PR opened multi-arch/ocp-remote-ci

add ppc64le-libvirt configs

Signed-off-by: Christy Norman christy@linux.vnet.ibm.com

+79 -0

0 comment

2 changed files

pr created time in 2 months

create barnchclnperez/ocp-remote-ci

branch : ppc64le-libvirt

created branch time in 2 months

fork clnperez/ocp-remote-ci

Configuration for our CI setup in api.ci.openshift.org.

fork in 2 months

push eventclnperez/release

Christy Norman

commit sha 30b8a54e7103813c831c8c18922ffde83bf6da02

add clnperez as sshd bastion owner for p libvirt Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

pull request commentopenshift/ci-tools

deprovision a libvirt cluster

I think that for this to work we would have to modify the sshd bastion to listen on a port that we then include in the ssh tunnel created on the ci host(s). Right now, if I ssh from inside the pod like :

    - /usr/local/bin/mock-nss.sh
    - /bin/sh
    - -c
    - |
      #!/bin/sh
      ssh -T -v root@<internal-pod-ip> -p 2222 -i /.ssh/id_rsa journalctl \
         -u libvirtd.service --until "4 hours ago"

I get a verification error:

debug1: Reading configuration data /etc/crypto-policies/back-ends/openssh.config
debug1: configuration requests final Match pass
debug1: re-parsing configuration
debug1: Reading configuration data /etc/ssh/ssh_config
debug1: Reading configuration data /etc/ssh/ssh_config.d/05-redhat.conf
debug1: Reading configuration data /etc/crypto-policies/back-ends/openssh.config
debug1: Connecting to 172.16.29.28 [172.16.29.28] port 2222.
debug1: Connection established.
debug1: identity file /.ssh/id_rsa type 0
debug1: identity file /.ssh/id_rsa-cert type -1
debug1: Local version string SSH-2.0-OpenSSH_8.0
debug1: Remote protocol version 2.0, remote software version OpenSSH_7.4
debug1: match: OpenSSH_7.4 pat OpenSSH_7.0*,OpenSSH_7.1*,OpenSSH_7.2*,OpenSSH_7.3*,OpenSSH_7.4*,OpenSSH_7.5*,OpenSSH_7.6*,OpenSSH_7.7* compat 0x04000002
debug1: Authenticating to 172.16.29.28:2222 as 'root'
debug1: SSH2_MSG_KEXINIT sent
debug1: SSH2_MSG_KEXINIT received
debug1: kex: algorithm: curve25519-sha256
debug1: kex: host key algorithm: ecdsa-sha2-nistp256
debug1: kex: server->client cipher: aes256-gcm@openssh.com MAC: <implicit> compression: none
debug1: kex: client->server cipher: aes256-gcm@openssh.com MAC: <implicit> compression: none
debug1: kex: curve25519-sha256 need=32 dh_need=32
debug1: kex: curve25519-sha256 need=32 dh_need=32
debug1: expecting SSH2_MSG_KEX_ECDH_REPLY
debug1: Server host key: ecdsa-sha2-nistp256 SHA256:V3v9eJLf57wfDRc2td4Vv++G53OFIgnFws3350ok7Ek
debug1: checking without port identifier
debug1: read_passphrase: can't open /dev/tty: No such device or address
Host key verification failed.

The levels of port-forward and reverse-tunneling and NAT are making my head hurt a little. :D

clnperez

comment created time in 2 months

pull request commentopenshift/release

add clnperez as sshd bastion owner for p libvirt

@hamzy @crawford PTAL

clnperez

comment created time in 2 months

push eventclnperez/release

Christy Norman

commit sha 76561d15c1fa04d637517cac5b118c69768e5ecd

add clnperez as sshd bastion owner for p libvirt Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

PR opened openshift/release

add clnperez as sshd bastion owner for p libvirt

Signed-off-by: Christy Norman christy@linux.vnet.ibm.com

+9 -1

0 comment

1 changed file

pr created time in 2 months

create barnchclnperez/release

branch : clnperez-libvirt-p-admin

created branch time in 2 months

pull request commentopenshift/ci-tools

deprovision a libvirt cluster

i'm not quite sure how to test it locally. i was assuming that only the prow jobs could ssh to that ip. (i'm talking about the ip of the sshd bastion we set up, in case that wasn't obvious before)

clnperez

comment created time in 2 months

Pull request review commentopenshift/ci-tools

deprovision a libvirt cluster

 EOF   echo "will deprovision GCE cluster ${infraID} in region ${region}" done +libvirt_p_cluster_age_cutoff="$(TZ=":America/New_York" date --date="-11 hours" '+%Y-%m-%dT%H:%M%z')"+echo "deprovisioning clusters with creation time before ${libvirt_p_cluster_age_cutoff} in libvirt-p"+for network in $(ssh ${bastion_p} virsh net-list --all --name | grep -v default) ; do

yah it's static. for the ci it's hard-coded into the release job: https://github.com/openshift/release/blob/master/ci-operator/jobs/openshift/release/openshift-release-release-4.3-periodics.yaml#L6912

clnperez

comment created time in 2 months

Pull request review commentopenshift/ci-tools

deprovision a libvirt cluster

 EOF   echo "will deprovision GCE cluster ${infraID} in region ${region}" done +libvirt_p_cluster_age_cutoff="$(TZ=":America/New_York" date --date="-11 hours" '+%Y-%m-%dT%H:%M%z')"+echo "deprovisioning clusters with creation time before ${libvirt_p_cluster_age_cutoff} in libvirt-p"+for network in $(ssh ${bastion_p} virsh net-list --all --name | grep -v default) ; do+        if [[ ! -z $(ssh ${bastion_p} journalctl -u libvirtd.service --until \"11 hours ago\" 2>/dev/null | grep ${network}) ]] ; then

re time cutoff format: so can i just set it to 4 hours ago then, since that's what journalctl is using? re bash: didn't catch that you were talking about two things there. i'll fix that, thanks.

clnperez

comment created time in 2 months

push eventclnperez/ci-tools

Christy Norman

commit sha c1127a9a92a91c1ed8d18ad001d6512d09759f56

deprovision a libvirt cluster this is for the p system currently testing the 4.3 releases. the plan is to be able to set additional ips if we have different bastions. we could add an outer for-loop for that in the future if needed. the journaclctl way of checking for age/existence in time is there because there is no way to get the age of a libvirt resource unless metadata was added to the resource upon creation. Signed-off-by: Christy Norman <christy@linux.vnet.ibm.com>

view details

push time in 2 months

Pull request review commentopenshift/ci-tools

deprovision a libvirt cluster

 EOF   echo "will deprovision GCE cluster ${infraID} in region ${region}" done +libvirt_p_cluster_age_cutoff="$(TZ=":America/New_York" date --date="-11 hours" '+%Y-%m-%dT%H:%M%z')"+echo "deprovisioning clusters with creation time before ${libvirt_p_cluster_age_cutoff} in libvirt-p"+for network in $(ssh ${bastion_p} virsh net-list --all --name | grep -v default) ; do+        if [[ ! -z $(ssh ${bastion_p} journalctl -u libvirtd.service --until \"11 hours ago\" 2>/dev/null | grep ${network}) ]] ; then
journalctl -u libvirtd --until $libvirt_p_cluster_age_cutoff
Failed to parse timestamp: 2020-04-23T04:48-0400
clnperez

comment created time in 2 months

pull request commentopenshift/ci-tools

deprovision a libvirt cluster

@stevekuznetsov thanks for the first review. another thing i'm not sure about with the approach is, will those ssh calls get to the right place? iow -- will the ssh to the bastion ip call journalctl on the libvirt host?

clnperez

comment created time in 2 months

Pull request review commentopenshift/ci-tools

deprovision a libvirt cluster

 EOF   echo "will deprovision GCE cluster ${infraID} in region ${region}" done +libvirt_p_cluster_age_cutoff="$(TZ=":America/New_York" date --date="-11 hours" '+%Y-%m-%dT%H:%M%z')"+echo "deprovisioning clusters with creation time before ${libvirt_p_cluster_age_cutoff} in libvirt-p"+for network in $(ssh ${bastion_p} virsh net-list --all --name | grep -v default) ; do+        if [[ ! -z $(ssh ${bastion_p} journalctl -u libvirtd.service --until \"11 hours ago\" 2>/dev/null | grep ${network}) ]] ; then

yes, that's the only way i can think of to know how old something is (if it was created before a certain time).

and i can double-check, but i think i tried using that, but iirc the timestamp format's different than what journalctl expects -- so i was just using that var as a logging output. --until is so much simpler, it's harder to mess up, so I just went with that. do you mind if i keep that strategy?

clnperez

comment created time in 2 months

Pull request review commentopenshift/ci-tools

deprovision a libvirt cluster

 EOF   echo "will deprovision GCE cluster ${infraID} in region ${region}" done +libvirt_p_cluster_age_cutoff="$(TZ=":America/New_York" date --date="-11 hours" '+%Y-%m-%dT%H:%M%z')"+echo "deprovisioning clusters with creation time before ${libvirt_p_cluster_age_cutoff} in libvirt-p"+for network in $(ssh ${bastion_p} virsh net-list --all --name | grep -v default) ; do

i was hoping that'd be something we could add to the cleanup run. is that possible? i'm not familiar with how these values get set. i noticed that CLUSTER_TTL was set somewhere.

clnperez

comment created time in 2 months

Pull request review commentopenshift/ci-tools

deprovision a libvirt cluster

 EOF   echo "will deprovision GCE cluster ${infraID} in region ${region}" done +libvirt_p_cluster_age_cutoff="$(TZ=":America/New_York" date --date="-11 hours" '+%Y-%m-%dT%H:%M%z')"

yah, 4's better. i won't get into why i picked 11 to start but will change.

clnperez

comment created time in 2 months

Pull request review commentopenshift/ci-tools

deprovision a libvirt cluster

 EOF   echo "will deprovision GCE cluster ${infraID} in region ${region}" done +libvirt_p_cluster_age_cutoff="$(TZ=":America/New_York" date --date="-11 hours" '+%Y-%m-%dT%H:%M%z')"+echo "deprovisioning clusters with creation time before ${libvirt_p_cluster_age_cutoff} in libvirt-p"+for network in $(ssh ${bastion_p} virsh net-list --all --name | grep -v default) ; do+        if [[ ! -z $(ssh ${bastion_p} journalctl -u libvirtd.service --until \"11 hours ago\" 2>/dev/null | grep ${network}) ]] ; then

oh yah, definitely meant to fix that. i was focusing more on big-picture "will this work" with this submission.

clnperez

comment created time in 2 months

pull request commentopenshift/ci-tools

deprovision a libvirt cluster

@stevekuznetsov -- this is a first pass at this. i'm not sure exactly how to go about integrating the bastion with this script. this approach is kind of a guess and might be impossible. thoughts?

clnperez

comment created time in 2 months

PR opened openshift/ci-tools

deprovision a libvirt cluster

this is for the p system currently testing the 4.3 releases. the plan is to be able to set additional ips if we have different bastions. we could add an outer for-loop for that in the future if needed.

the journaclctl way of checking for age/existence in time is there because there is no way to get the age of a libvirt resource unless metadata was added to the resource upon creation.

Signed-off-by: Christy Norman christy@linux.vnet.ibm.com

+18 -0

0 comment

1 changed file

pr created time in 2 months

create barnchclnperez/ci-tools

branch : libvirt-deprovision

created branch time in 2 months

fork clnperez/ci-tools

DPTP Tooling

fork in 2 months

Pull request review commentopenshift/release

ppc64le libvirt ci job

 objects:          wait "${openshift_install}" +        mock-nss.sh openshift-install --dir=/tmp/artifacts/installer --log-level=debug wait-for install-complete 2>&1 &

Thanks @crawford! I think we can all say the same thing about our minds these past couple of weeks. :)

mkumatag

comment created time in 3 months

Pull request review commentopenshift/release

ppc64le libvirt ci job

 objects:          wait "${openshift_install}" +        mock-nss.sh openshift-install --dir=/tmp/artifacts/installer --log-level=debug wait-for install-complete 2>&1 &

Also, @manojnkumar said that "in theory" no customers should see this version of the installer. It's very much unsupported (dev only), just to make sure that's clear. That makes me okay with this approach.

mkumatag

comment created time in 3 months

more