-
Notifications
You must be signed in to change notification settings - Fork 0
Khushi/watch poc 2 #1
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
base: ci_dev
Are you sure you want to change the base?
Conversation
| @serviceRecords = nil | ||
|
|
||
| # Adding telemetry to send pod telemetry every 5 minutes | ||
| timeDifference = (DateTime.now.to_time.to_i - @@podTelemetryTimeTracker).abs |
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.
Since enumerate only called whenever watch gets disconnected. We will have to move the telemetry code in enumerate function to merge_updates as this being called for every minute.
| touch /var/opt/microsoft/docker-cimprov/log/fluentd.log | ||
| chmod 666 /var/opt/microsoft/docker-cimprov/log/fluentd.log | ||
|
|
||
| touch /var/opt/microsoft/docker-cimprov/log/testing-podinventory.json |
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.
why do we need these files?
| ENV RUBY_GC_HEAP_OLDOBJECT_LIMIT_FACTOR 0.9 | ||
| RUN /usr/bin/apt-get update && /usr/bin/apt-get install -y libc-bin wget openssl curl sudo python-ctypes init-system-helpers net-tools rsyslog cron vim dmidecode apt-transport-https gnupg && rm -rf /var/lib/apt/lists/* | ||
| COPY setup.sh main.sh defaultpromenvvariables defaultpromenvvariables-rs defaultpromenvvariables-sidecar mdsd.xml envmdsd $tmpdir/ | ||
| COPY kubeclient-4.9.2.gem setup.sh main.sh defaultpromenvvariables defaultpromenvvariables-rs defaultpromenvvariables-sidecar mdsd.xml envmdsd $tmpdir/ |
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.
Not urgent, but will have to update the build code to build the gem automatically instead of copying manually.
| @@ -0,0 +1,44 @@ | |||
| { | |||
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.
Not urgent, these files needs to be removed.
No description provided.