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

Update flaky test in 0.44.x #1250

Merged
merged 3 commits into from
Jun 20, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
1 change: 1 addition & 0 deletions .github/workflows/release-process.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ jobs:
uses: carvel-dev/setup-action@v1
with:
only: imgpkg
token: ${{ secrets.GITHUB_TOKEN }}

- name: Login to GitHub Container Registry
uses: docker/login-action@v2
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/release-published.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ jobs:
with:
only: kctrl
kctrl: ${{ github.event.release.tag_name }}
token: ${{ secrets.GITHUB_TOKEN }}
- run: |
kctrl version
version=`kctrl version`
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/test-kctrl-gh.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ jobs:
uses: carvel-dev/setup-action@v1
with:
only: ytt, kbld, imgpkg, vendir, kapp
token: ${{ secrets.GITHUB_TOKEN }}
- name: Run Tests
run: |
set -e -x
Expand Down
6 changes: 4 additions & 2 deletions hack/dependencies_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,13 @@ import (
)

func init() {
client = http.DefaultClient
client = &http.Client{
// try to use a Github API token from the environment to avoid rate-limiting
Transport: newGithubTransport(),
}
}

func TestDependencyDownload(t *testing.T) {
client = http.DefaultClient
for _, tc := range []struct {
name string
dep dependency
Expand Down
31 changes: 0 additions & 31 deletions test/e2e/assets/https-server/server.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,6 @@ kind: Service
metadata:
namespace: https-server
name: https-svc
annotations:
kapp.k14s.io/change-group: server
spec:
selector:
app: self-signed-https-server
Expand All @@ -23,8 +21,6 @@ kind: Deployment
metadata:
name: self-signed-https-server
namespace: https-server
annotations:
kapp.k14s.io/change-group: server
spec:
replicas: 1
selector:
Expand Down Expand Up @@ -131,30 +127,3 @@ binaryData:
# deploy:
# - kapp: {}
packages.tar: "H4sIAHczxmIAA+2SS2rEMAyGZ51T6AL22KmTQg5RuupetZVJyGOM4wRC6d0bx5M+oAylDJRCvo0sS/+PjGVRN3ii4Wjjgc9de7gxQohcKQjxPs/WKNKYr2QiA6nSXKo7makUhFSZzA4gbj3Id4yDR7eMYprrfUtbWV6px6fAe/wnMMYStPUTuaE+9wUY9MjjLtT9iWt0E7Xc0HScJLa2Qpk0dW8KeIz7knTkMYiKBKDHjgq4bBJ7pmrpZBqZJuc/WXHJBRfJYEkHlaPy4WfCpXnaBo0eAJ4626Kn4ASweQZK8rraEgaV93bLAqNrCxj7cSBzuf1qFTSz9wW8vF5yQ7Y9zx/VBq1dy3/9iTs7Ozu/4A1GdDfIAAgAAA=="

# TODO should we make vendir's http retry within App CR, to avoid
# transient failure when Service=>Deployment networking is not ready?
---
apiVersion: batch/v1
kind: Job
metadata:
name: check-nginx-conn
namespace: default
annotations:
kapp.k14s.io/update-strategy: always-replace
kapp.k14s.io/change-rule: upsert after upserting server
spec:
backoffLimit: 0
template:
metadata:
name: check-nginx-conn
spec:
containers:
- name: check
image: busybox
command:
- /bin/sh
- "-c"
- |
wget --tries=10 --no-check-certificate https://https-svc.https-server.svc.cluster.local/deployment.yml
restartPolicy: Never