-
Notifications
You must be signed in to change notification settings - Fork 182
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
Add Travis - Multiarch Builds, Autopublish on Docker-Hub and Security Scan with trivy #70
Closed
thiscantbeserious
wants to merge
130
commits into
skooner-k8s:master
from
thiscantbeserious:merge-herbrandson
Closed
Changes from all commits
Commits
Show all changes
130 commits
Select commit
Hold shift + click to select a range
17b56f7
Merge pull request #3 from zetaab/feature/oidc_group
herbrandson 8ac2f21
typo fix
complexsplit fb88fba
Merge pull request #4 from complexsplit/master
herbrandson 24b2d47
Lazily request docs (so we don't request them before we've auth-ed)
74ce499
Chart added
c2f5d8c
Merge pull request #8 from IvanKirianov/dev
herbrandson dcfd2f1
Refactoring. Better charts. Fixed a bug parsing cpu usage. Using a be…
3642a26
nodeport yaml provided and readme updated for faster up-and-running o…
StevenPG 9f4970a
Merge pull request #9 from StevenPG/dev
herbrandson 24a6001
Adding sorting by cpu/ram support to nodes view. Some better error ha…
6464e70
Merge branch 'dev' of github.com:herbrandson/k8dash into dev
ef55ce7
Adding ingress.yaml to helm chart
42d5c26
Fix parsing disk space / ram
frohikey 7c2f6d7
Switching router to solve issue w/ routing to urls with colons in the…
9287513
Helper function parseUnitsOfBytes
frohikey e93925f
Merge pull request #10 from frohikey/dev
herbrandson d59174f
Some styling efforts
08803dc
Merge branch 'dev' of github.com:herbrandson/k8dash into dev
b9b4907
Fixing issue w/ opening links in new tabs showing a 'page not found' …
5696146
Fixing bug with viewing namespaces
b4d816b
Initial work on responsive/mobile design
512c5a0
Wrapping up responsive design overhaul
275a977
Improving ios installed app experience
fc00861
Cleaning up a few small issues w/ responsive design
aad1405
Small fix to colSpan on workloads.
3749f07
A lot of charts work
f5233b5
Merge pull request #18 from herbrandson/charts
herbrandson 24028d0
More stying of charts
9d99053
Small styling change for nodesPanel
0326db2
Adding more columns to pods/nodes to better display reservations and …
8e6d22c
Creating custom donut chart with animations
f291f32
Adding missing file
a9f1055
Merge branch 'stats' into dev
502bc40
Refactoring. More chart clean up. Improving sorting by pod resources.
b0b1a14
Set theme jekyll-theme-cayman
herbrandson 89a7751
Packaging helm charts for deployment via helm hub
c44a2b4
Removing an old comment
60b075a
Merge branch 'master' of github.com:herbrandson/k8dash
f443819
Moving helm files into specific directory
d028f1c
Actually adding the helm files this time
6308d00
Fixing issue w/ text being cut off in the menu
0a60825
One more small fix for menu text cutoff
8de4c88
Better support for non-admin RBAC roles
9dcf91c
Fixing bug with Yaml not wrapping correctly in Firefox/Safari
65a424d
Enabling ability for k8dash to be served from non-root urls
ee663ca
Setting a 'base' tag to index.html to enable serving k8dash from non-…
d8524aa
Replacing 'page' npm package with custom router
3dbff38
Fixing bug with 'logs' and 'exec' paths (missing shebangs)
6381411
ensure k8dash scheduled on linux nodes
KnicKnic cc3eb0d
Merge pull request #24 from KnicKnic/master
herbrandson 7695d15
Initial work at hide menu items based on user roles
3dbfbf0
Small fix to helm chart
ed0ee73
Merge branch 'master' into dev
da90409
Small fix to Helm chart
b5c1d32
Making names in EventPanel into links
dd9ed6d
Update npm and node dependencies
910f79b
Smart login doesn't loose the original url
d290b91
Improvements to the logging ui. Correctly decoding unicode chars. Ini…
5fd5a4d
Updating the readme to indicate that kubectl proxy cannot be used to …
a9854e9
Better icon styling to indicate various states
e7dab82
Adding ability to pass authorization headers recieved from the GET fo…
9ca05ad
A bit of performance work
f1f9cf5
Removing moment.js (a bit of perf gain)
4dadf69
Merge branch 'dev' into rules
c13552b
Limit menu options based on accounts permissions
14986ca
Fixing bug with displaying correct menu items when using oidc. Other …
ce84ab6
Fixing regression with invalid jwt being passed to the 'watch' apis
611a1a1
Support k8s cluster with parameter
titilambert 67c4938
Merge pull request #33 from titilambert/master
herbrandson 7d77419
Use humanize-duration to better format duration (mo, m, ms)
olivergg cbd9aa4
#47: Treat token as password field
paullryan b756157
Merge pull request #48 from nomadreservations/feature/47-autocomplete
herbrandson 67493fb
Merge pull request #46 from olivergg/improve_duration_frmat
herbrandson 5d72fd9
Bump lodash from 4.17.11 to 4.17.15 in /server
dependabot[bot] 3c29c52
Merge pull request #51 from herbrandson/dependabot/npm_and_yarn/serve…
herbrandson 8e3c4ba
Add anchor link for metadata values that match valid URLs
olivergg 4fb83ca
Show job duration in job detail
olivergg 06770cc
Merge pull request #54 from olivergg/show_job_duration
herbrandson 98d7a20
Merge pull request #53 from olivergg/href_for_urls_in_metadata
herbrandson 00c49c2
Support for multiple ingress paths
c78962c
Typo in chart archive
d89ad48
helmignore
657a43f
Typo
ac31064
Merge branch 'helm0.0.1'
ba83355
Service account yaml
a7835c8
fix deprecated api versions
1d5bf7c
Update Dockerfile for Multiarch support
7544884
Update Dockerfile for Multiarch support
c6509b3
Merge pull request #64 from ConnorGraham/helm0.0.1
herbrandson 27fa02d
Merge pull request #63 from ConnorGraham/master
herbrandson 7867dbd
Merge pull request #65 from turbosquid/issues/apiVersion
herbrandson fdb9190
Merge branch 'master' of git://github.com/herbrandson/k8dash
ccf5320
Merge branch 'master' of git://github.com/herbrandson/k8dash
4993395
update deps, replace node-sass with sass
53a3924
update deps, replace node-sass with sass
55abb42
Fix security issues reported by trivy
a075ccb
Add travis config
a962cc9
Update Travis & Dockerfile
ef050cc
Merge branch 'master' of https://github.com/thiscantbeserious/k8dash
c8ebb63
Fix brain
d03f06e
Update Travis to build
487cd62
Update Travis
7ac2d42
Update travis
c1297f9
Update Travis
54b825d
Update travis
fa185f2
Update travis
b101ff2
Update travis
eb83710
Update travis
57386c3
Update travis
93cf4de
Update travis
6bb4876
Update travis
c4d79cb
Update travis
eeb5dbe
Update travis
f605dd2
Update travis
a87417f
Update travis
e08a8a4
Update travis
bc5d97e
Update Dockerfile to improve Caching for NPM-Packages
8579de6
Update Dockerfile to improve Caching for NPM-Packages
c684399
Merge branch 'master' of https://github.com/thiscantbeserious/k8dash
c3ed9f6
Speed up Multiarch-Build for Travis
6c481d3
Update travis
5f00168
Finalize Dockerfile for Multiarch-Build & Optimize Travis-Config to p…
7500d1b
Finalize Travis config
ad43afd
Finalize Travis config
ec87f03
Finalize Travis config
8a34812
Added Travis build-status
thiscantbeserious 4a76336
Update README.md
thiscantbeserious 548e48a
Switch to herbrandson/k8dash
thiscantbeserious 9fde86e
Update README.md
thiscantbeserious 6c3db24
Code-review fixes
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
language: generic | ||
services: | ||
- docker | ||
env: | ||
global: | ||
- HUB_REPO=herbrandson/k8dash | ||
- BINFMT_TAG=a7996909642ee92942dcd6cff44b9b95f08dad64 | ||
- COMMIT=${TRAVIS_COMMIT::8} | ||
- DOCKER_CLI_EXPERIMENTAL=enabled | ||
cache: | ||
directories: | ||
- docker_images | ||
- $HOME/.cache/trivy | ||
before_cache: | ||
- docker save -o docker_images/images.tar $(docker images -a -q) | ||
before_install: | ||
- sudo apt-get -y update || true | ||
- sudo apt-get -y install qemu-system-arm qemu-user-static binfmt-support | ||
- docker run --rm --privileged docker/binfmt:${BINFMT_TAG} | ||
- sudo update-binfmts --display | ||
- curl -fsSL https://get.docker.com -o get-docker.sh | ||
- sudo sh get-docker.sh | ||
- echo ${HUB_TOKEN} | docker login --username ${HUB_USER} --password-stdin | ||
- docker load -i docker_images/images.tar || true | ||
- docker buildx create --use | ||
- docker buildx build ./ --platform=linux/arm/v6,linux/arm/v7,linux/arm64/v8,linux/amd64,linux/i386 -t ${HUB_REPO}:${COMMIT} -t ${HUB_REPO}:latest --push | ||
- docker save -o docker_images/images.tar $(docker images -a -q) | ||
- export VERSION=$(curl --silent "https://api.github.com/repos/aquasecurity/trivy/releases/latest" | grep '"tag_name":' | sed -E 's/.*"v([^"]+)".*/\1/') | ||
- wget https://github.com/aquasecurity/trivy/releases/download/v${VERSION}/trivy_${VERSION}_Linux-64bit.tar.gz | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just curious... any reason you're using wget here and curl above? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Copy & Paste from the individual snippets :)) |
||
- tar zxvf trivy_${VERSION}_Linux-64bit.tar.gz | ||
script: | ||
- ./trivy --exit-code 0 --severity HIGH --no-progress ${HUB_REPO}:${COMMIT} | ||
- ./trivy --exit-code 1 --severity CRITICAL --no-progress ${HUB_REPO}:${COMMIT} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've not tried using travis-ci w/ an open source project before, but I absolutely love that you've added this :)
Is there anything that will needs to be configured at travis-ci.com to make this work?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes you need to generate an API-Token on Docker Hub and add that to the Travis Project via the HUB_TOKEN Environment Variable and you also need to set HUB_USER to your accoutname.
See here:
https://docs.travis-ci.com/user/environment-variables/#defining-variables-in-repository-settings
Afterwards Travis will publish it automatically on Docker Hub after each commit.
Btw. I only used it once myself but I loved the idea of being able to work remotely on my Chromebook and Travis doing the heavy duty for me (not just testing) so I gave it a go.
Guess I'll do that more often from now on, since I like the result. :) ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, I kinda love this idea :)
However, I need a bit to get this all wired up. It also makes for multiple large and only semi related changes in the PR. How would you feel about pulling the
travis.yml
file out into a separate PR? Everything else LGTM and I'd love to be able to merge it asap.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, sorry. One other comment on your last comment above. I currently have it configured so DockerHub watches GitHub for changes and automatically does this build there. I think that's the same thing you were going for via Travis. Any advantages to moving that build out of DockerHub and into Travis?