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

Jump to bottom

updating pkg versions initial commit #93

Merged
merged 19 commits into from
Oct 14, 2024
Merged

updating pkg versions initial commit #93

merged 19 commits into from
Oct 14, 2024

Conversation

ibrahimkk-moideen
Copy link
Collaborator

@ibrahimkk-moideen ibrahimkk-moideen commented Sep 3, 2024 •
edited
Loading

  1. Updated few packages to resolved high/medium sec vulns.
  2. Fixed capturing model in finished log
  3. Included option to skip TLS verification.
  4. Updated Helm chart with to include the above option.

ibrahimkk-moideen and others added 19 commits September 3, 2024 16:30
@ibrahimkk-moideen
updating pkg versions initial commit
79bdb63
@ibrahimkk-moideen
fix model field in scrape finish log entry
787d44f
@ibrahimkk-moideen
include ssl verify option
8e3689a
@ibrahimkk-moideen
update changelog
1f43db4
@ibrahimkk-moideen
update readme with new ssl verify flag
2b05881
@ibrahimkk-moideen
update changelog with skip TLS info
a21af9d
@ibrahimkk-moideen
remove newly added param in exporter func
b2a0e63
@ibrahimkk-moideen
updating helm chart
4df49bf
@ibrahimkk-moideen
setting ssl verify default to true
73f12c1
@ibrahimkk-moideen
setting back skip ssl verify default value to false
55e8986
@ibrahimkk-moideen
modify insecure flag and fix Dockerfile formatting
1265a90
@ibrahimkk-moideen
restore ssl verify flag
09cf5b4
@ibrahimkk-moideen
few updates to helm chart in fixing issue
464fb6e
@ibrahimkk-moideen
update helm chart parameters
63e95a2
@ibrahimkk-moideen
add ssl verify option to separate config
fe8f221
@ibrahimkk-moideen
remove default for ssl verify bool
52c57ba
@derrick-dacosta
upgrade go to 1.23, fix config init ordering, quote all string helm c…
3ea3b0d 
…hart values
@ibrahimkk-moideen
update helm readme, update to GH workflow
3c77bbb
@ibrahimkk-moideen
deprecate module query param in favor of model
a24e13b
derrick-dacosta
derrick-dacosta approved these changes Oct 14, 2024
View reviewed changes
Copy link
Collaborator

@derrick-dacosta derrick-dacosta left a comment

Choose a reason for hiding this comment

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

LGTM, we just need to make sure our prometheus config reflects the new model name in the scrape url

@ibrahimkk-moideen ibrahimkk-moideen merged commit 6ae8d65 into main Oct 14, 2024
3 checks passed
@ibrahimkk-moideen ibrahimkk-moideen linked an issue Oct 14, 2024 that may be closed by this pull request
Package update and Fix model field in final log message #94
Closed
@ibrahimkk-moideen ibrahimkk-moideen deleted the upgrade_pkgs_secvulns branch October 15, 2024 20:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@derrick-dacosta derrick-dacosta derrick-dacosta approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Package update and Fix model field in final log message
2 participants
@ibrahimkk-moideen @derrick-dacosta