Skip to content

Commit

Permalink
Merge pull request #6515 from vrothberg/v2-enable-ubuntu
Browse files Browse the repository at this point in the history
V2 enable ubuntu
  • Loading branch information
openshift-merge-robot authored Jun 8, 2020
2 parents 2869cce + a5facca commit b8acc85
Show file tree
Hide file tree
Showing 7 changed files with 43 additions and 8 deletions.
13 changes: 6 additions & 7 deletions .cirrus.yml
Original file line number Diff line number Diff line change
Expand Up @@ -422,13 +422,12 @@ testing_task:
- name: "test ${PRIOR_FEDORA_NAME}"
gce_instance:
image_name: "${PRIOR_FEDORA_CACHE_IMAGE_NAME}"
# TODO:
# - name: "test ${UBUNTU_NAME}"
# gce_instance:
# image_name: "${UBUNTU_CACHE_IMAGE_NAME}"
# - name: "test ${PRIOR_UBUNTU_NAME}"
# gce_instance:
# image_name: "${PRIOR_UBUNTU_CACHE_IMAGE_NAME}"
- name: "test ${UBUNTU_NAME}"
gce_instance:
image_name: "${UBUNTU_CACHE_IMAGE_NAME}"
- name: "test ${PRIOR_UBUNTU_NAME}"
gce_instance:
image_name: "${PRIOR_UBUNTU_CACHE_IMAGE_NAME}"

timeout_in: 120m

Expand Down
6 changes: 5 additions & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -612,7 +612,7 @@ uninstall:
GIT_CHECK_EXCLUDE="./vendor:docs/make.bat" $(GOBIN)/git-validation -run DCO,short-subject,dangling-whitespace -range $(EPOCH_TEST_COMMIT)..$(HEAD)

.PHONY: install.tools
install.tools: .install.gitvalidation .install.md2man .install.ginkgo .install.golangci-lint ## Install needed tools
install.tools: .install.gitvalidation .install.md2man .install.ginkgo .install.golangci-lint .install.bats ## Install needed tools

define go-get
env GO111MODULE=off \
Expand All @@ -635,6 +635,10 @@ endef
.install.golangci-lint: .gopathok
VERSION=1.18.0 GOBIN=$(GOBIN) sh ./hack/install_golangci.sh

.PHONY: .install.bats
.install.bats: .gopathok
VERSION=v1.1.0 ./hack/install_bats.sh

.PHONY: .install.pre-commit
.install.pre-commit:
if [ -z "$(PRE_COMMIT)" ]; then \
Expand Down
1 change: 1 addition & 0 deletions contrib/cirrus/integration_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ case "$SPECIALMODE" in
make
make install PREFIX=/usr ETCDIR=/etc
make test-binaries
make .install.bats
if [[ "$TEST_REMOTE_CLIENT" == "true" ]]
then
make remote${TESTSUITE} VARLINK_LOG=$VARLINK_LOG
Expand Down
11 changes: 11 additions & 0 deletions contrib/cirrus/setup_environment.sh
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,17 @@ done
cd "${GOSRC}/"
case "${OS_RELEASE_ID}" in
ubuntu)
apt-get update
apt-get install -y containers-common
sed -ie 's/^\(# \)\?apparmor_profile =.*/apparmor_profile = ""/' /etc/containers/containers.conf
if [[ "$OS_RELEASE_VER" == "19" ]]; then
apt-get purge -y --auto-remove golang*
apt-get install -y golang-1.13
ln -s /usr/lib/go-1.13/bin/go /usr/bin/go
fi
if [[ "$OS_RELEASE_VER" == "20" ]]; then
apt-get install -y python-is-python3
fi
;;
fedora)
# All SELinux distros need this for systemd-in-a-container
Expand Down
16 changes: 16 additions & 0 deletions hack/install_bats.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,16 @@
#!/bin/bash

set -e

die() { echo "${1:-No error message given} (from $(basename $0))"; exit 1; }

buildDir=$(mktemp -d)
git clone https://github.com/bats-core/bats-core $buildDir

pushd $buildDir
pwd
git reset --hard ${VERSION}
./install.sh /usr/local
popd

rm -rf $buildDir
2 changes: 2 additions & 0 deletions test/e2e/checkpoint_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -232,6 +232,8 @@ var _ = Describe("Podman checkpoint", func() {
})

It("podman checkpoint container with established tcp connections", func() {
// Broken on Ubuntu.
SkipIfNotFedora()
localRunString := getRunString([]string{redis})
session := podmanTest.Podman(localRunString)
session.WaitWithDefaultTimeout()
Expand Down
2 changes: 2 additions & 0 deletions test/e2e/generate_kube_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -254,6 +254,8 @@ var _ = Describe("Podman generate kube", func() {
})

It("podman generate with user and reimport kube on pod", func() {
// This test fails on ubuntu due to https://github.com/seccomp/containers-golang/pull/27
SkipIfNotFedora()
podName := "toppod"
_, rc, _ := podmanTest.CreatePod(podName)
Expect(rc).To(Equal(0))
Expand Down

0 comments on commit b8acc85

Please sign in to comment.