forked from posuch/zabbix-kubernetes-1
-
Notifications
You must be signed in to change notification settings - Fork 9
updating enhancement #24
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
Open
blodone
wants to merge
88
commits into
zabbix-tooling:master
Choose a base branch
from
vico-research-and-consulting:main
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
88 commits
Select commit
Hold shift + click to select a range
220242b
fix monitor-user rights, jenkinsfile
blodone 0c4d6ad
refactor
blodone c5a6c2f
refactor start_data_threads
blodone cba9d6c
fix dockerfile
blodone c30872d
fix build.sh
blodone 716ff9c
config_default fix
blodone e0716ac
.dockerignore config fix
blodone b37c7ae
readme.md
b279df3
run shfmt -i 4 -w
89824f2
merge from zabbix-tooling
blodone a8af412
Merge pull request #1 from zabbix-tooling/master
blodone b5f0f21
remove .idea
blodone 5e18cc3
Falsch-Positive Zählungen raus
e39dad1
Falsch-Positive Zählungen raus
60d482b
Falsch-Positive Zählungen raus
f70e022
alle Änderungen raus, die nicht wichtig
46dfc7a
dangling Spaces, bare Except
bdb4124
nonexistent hash throws TypeError
6a6ccae
parentheses
156209a
isort, re
8a8536f
kind dict prüfen
39ca576
uninitialized kind
84cfe87
isort uber alles
f4c9ab7
isort uber alles
d8eb4f7
import ordering
blodone 7511a42
base -l 120, broken isort reverted
0eebc9b
ignore local test config
d26a0a0
py3.11
ce93c03
sent/send
d050160
pods/containers faster
4b8fbf7
Noch volles Logging, status waiting maskiert
c84b71a
better, shorter logic
1a8ff66
even simpler logic
abdb7eb
Bulk logging raus, nur noch Errorstate loggen.
29dcc21
5 Pod Phases, 3 Container States
62ca872
Logging auch deployment
55eedcc
dangling lines
8955764
ignore
e19e412
ini file in Docker
05f1765
changes
73c4fe7
Merge SRE-720 -> main
33d3e61
get_zabbix_metrics for pods
bd4c846
worky
38b5790
fix logging
9345146
logger fixed, pods fixed
feaa54f
pod monitoring
9155e6f
fix pod
ef1f105
config_default.ini
blodone 06f4faf
fix build
blodone 6d26384
set zabbix_dry_run default to False
6bc2b9a
fix pod resource_data json
f2f78e2
progress containers
blodone f745fdb
container reimplement
eea7f82
debug containers
1d2b8ce
fix report_global containers
blodone 2eb3525
Dockerfile
blodone 0e1a4ab
template update
531bf5c
remove template_group
8389d2d
update versions, fix node values
blodone 0ca04ac
excepthook
blodone 9d6c314
add resource_obj.added datetime comparison with discovery_sent
ee09337
fix resend added discovery_sent
9f93886
fix resend discovery check
4de6ff6
longer wait on discovery first run
blodone c6114fb
fix first run timedthread
blodone 01e8fd0
log sending of data
f78258f
update recovery expressions
c31f0e7
speed up deployment resource data, better error log
9fe678c
fix terminated and imagepullbackoff
blodone 8bdaa1a
refactor if for pod
blodone 50cd37c
prepare re-check discovery items still there
blodone 77d74fe
uid refresh check enable
ab1c102
obj_uid typo
blodone 73e280b
fix real_name pod, del_obj orphan
c441971
del obj by string
9898f3c
del obj by string fix
51216c6
add data_refreshed timestamp
17e419b
copy resource.objects to delete
blodone cd9950c
secret for monitoring user
9f77eeb
refresh data with discovery after some time
blodone e2392fd
merge zabbix-tooling
blodone d98af23
fix pod, remove idea/workspace.xml
blodone 7261354
remove duplicate .idea in .gitignore
blodone 7b9d4d3
check before division by zero in pvc
blodone 3ff0aae
fix discovery data
blodone 9e06cf8
hotfix nodes name_space check
blodone 6223d4b
prepare for zabbix-tooling merge
blodone 9c10f67
fix components
blodone 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 hidden or 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 |
---|---|---|
|
@@ -6,4 +6,4 @@ build.sh | |
.gitignore | ||
kubernetes/* | ||
venv/* | ||
config_* | ||
configd_* |
This file contains hidden or 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 |
---|---|---|
|
@@ -10,4 +10,3 @@ venv/* | |
*.log | ||
/config*.py | ||
/config*.ini | ||
.idea |
This file was deleted.
Oops, something went wrong.
This file contains hidden or 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 |
---|---|---|
@@ -1,5 +1,5 @@ | ||
FROM python:3.11.0a3 | ||
LABEL maintainer="[email protected]" | ||
FROM python:3.11 | ||
LABEL maintainer="[email protected]" | ||
LABEL Description="zabbix-kubernetes - efficent kubernetes monitoring for zabbix" | ||
|
||
MAINTAINER [email protected] | ||
|
@@ -10,23 +10,23 @@ ENV ZABBIX_SERVER "zabbix" | |
ENV ZABBIX_HOST "k8s" | ||
ENV CRYPTOGRAPHY_DONT_BUILD_RUST "1" | ||
|
||
COPY --chown=nobody:users requirements.txt /app/requirements.txt | ||
WORKDIR /app | ||
COPY --chown=nobody:users Pipfile /app/ | ||
|
||
RUN apt-get update -y && \ | ||
apt-get install libffi-dev libffi7 libssl-dev bash screen ncdu -y && \ | ||
pip3 install --upgrade pip && \ | ||
pip3 install -r /app/requirements.txt && \ | ||
apt-get upgrade -y && \ | ||
apt-get dist-upgrade -y && \ | ||
apt-get remove base libssl-dev libffi-dev gcc -y && \ | ||
apt-get autoremove -y && \ | ||
rm -rf /var/lib/apt/lists/* /root/.cache | ||
RUN apt-get update -y | ||
scoopex marked this conversation as resolved.
Show resolved
Hide resolved
|
||
RUN apt-get upgrade -y | ||
RUN apt-get dist-upgrade -y | ||
RUN apt-get install libffi-dev libffi8 libssl-dev bash screen ncdu -y | ||
RUN pip install --root-user-action=ignore --upgrade pip && pip install --root-user-action=ignore pipenv | ||
RUN PIPENV_USE_SYSTEM=1 pipenv install --skip-lock --system | ||
RUN apt-get remove base libssl-dev libffi-dev gcc -y | ||
RUN apt-get autoremove -y | ||
RUN rm -rf /var/lib/apt/lists/* /root/.cache | ||
|
||
COPY --chown=nobody:users base /app/base | ||
COPY --chown=nobody:users k8sobjects /app/k8sobjects | ||
COPY --chown=nobody:users check_kubernetesd /app/check_kubernetesd | ||
COPY --chown=nobody:users config_default.ini /app/config_default.ini | ||
|
||
USER nobody | ||
WORKDIR /app | ||
|
||
ENTRYPOINT [ "/app/check_kubernetesd" ] |
This file contains hidden or 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 hidden or 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,22 @@ | ||
[[source]] | ||
url = "https://pypi.org/simple" | ||
verify_ssl = true | ||
name = "pypi" | ||
|
||
[packages] | ||
kubernetes = "==21.7.0" | ||
cryptography = "==36.0.1" | ||
types-cryptography = "==3.3.10" | ||
py-zabbix = "==1.1.7" | ||
sentry-sdk = "==1.5.1" | ||
adal = "==1.2.7" | ||
urllib3 = "==1.26.7" | ||
pytest = "==6.2.5" | ||
mypy = "==0.930" | ||
flake8 = "==4.0.1" | ||
coloredlogs = "==15.0.1" | ||
|
||
[dev-packages] | ||
|
||
[requires] | ||
python_version = "3.11" |
This file contains hidden or 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 hidden or 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.
Just remove the maintainer reference.
Probably updating to python 3.13 is something we can do in this step (but not mandatory from review perspective)