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.

Sign up for GitHub

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

added prow job for depstat #22137

Merged
merged 12 commits into from
May 14, 2021
Merged

added prow job for depstat #22137

Changes from 4 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
c81bf38
added prow job for depstat
RinkiyaKeDad May 11, 2021
4425308
go get -> go install
RinkiyaKeDad May 11, 2021
f9c651a
working locally
RinkiyaKeDad May 12, 2021
77628b0
fix shell errors
RinkiyaKeDad May 12, 2021
04928e9
fix arg specifying method
RinkiyaKeDad May 13, 2021
8a27d9b
removed '\' at the end
RinkiyaKeDad May 13, 2021
0ae55e0
partly working locally
RinkiyaKeDad May 13, 2021
cfb6940
removed hardcoded
RinkiyaKeDad May 14, 2021
5d4a6a7
major changes
RinkiyaKeDad May 14, 2021
6b670bd
major changes 2
RinkiyaKeDad May 14, 2021
70c516c
added annotations
RinkiyaKeDad May 14, 2021
abe925c
added new line
RinkiyaKeDad May 14, 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
24 changes: 24 additions & 0 deletions config/jobs/kubernetes/sig-arch/kubernetes-depstat.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
presubmits:
kubernetes/kubernetes:
- name: kubernetes-depstat
decorate: true
always_run: true
skip_report: true
spec:
containers:
- image: golang
command: ["/bin/bash"]
args: ["-c", "set -euo;
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think one of the tests is failing because it expects a ] at the end of this line. So should I have the entire thing in one line to fix this? '-'

@dims @spiffxp

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See the other pattern in general use : https://cs.k8s.io/?q=-%20%5C%7C&i=nope&files=&excludeFiles=&repos=kubernetes/test-infra

BASE_SHA="${PULL_BASE_SHA:-""}"; ARTIFACTS="${ARTIFACTS:-/tmp/artifacts}";
mkdir -p "${ARTIFACTS}";
cd "$ARTIFACTS"
go install github.com/kubernetes-sigs/depstat@latest;
cd /go/src/k8s.io/kubernetes
function write_report() { depstat stats --json; };
function diff_reports() { diff "$1" "$2"; };
write_report > "${ARTIFACTS}/stats.json";
if [ -n "${BASE_SHA}" ]; then
git checkout -b base "${BASE_SHA}";
write_report > "${ARTIFACTS}/stats-base.json";
diff_reports "${ARTIFACTS}"/stats-base.json "${ARTIFACTS}"/stats.json;
fi;"]