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

chore: update Sumo Logic Kubernetes Setup to v3.15.1 (#3793) #3811

Merged
merged 3 commits into from
Jul 26, 2024
Merged

chore: update Sumo Logic Kubernetes Setup to v3.15.1 (#3793) #3811

merged 3 commits into from
Jul 26, 2024

Conversation

rnishtala-sumo
Copy link
Contributor

No description provided.

@rnishtala-sumo rnishtala-sumo requested a review from a team as a code owner July 25, 2024 17:09
@github-actions github-actions bot added the documentation documentation label Jul 25, 2024
@rnishtala-sumo rnishtala-sumo force-pushed the backport-3793-to-release-v3.19 branch 2 times, most recently from 293bc1c to e6069c2 Compare July 25, 2024 17:18
@github-actions github-actions bot removed the documentation documentation label Jul 25, 2024
@github-actions github-actions bot added the documentation documentation label Jul 25, 2024
@rnishtala-sumo rnishtala-sumo added do-not-merge/hold and removed documentation documentation labels Jul 25, 2024
github-actions[bot]
github-actions bot requested changes Jul 25, 2024
View reviewed changes
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This pull request contains invalid labels. Please remove all of the following labels: ['do-not-merge/hold']

@sumo-drosiek
Copy link
Contributor

It requires this PR: https://github.com/SumoLogic/sumologic-kubernetes-collection/pull/3780/files

@github-actions github-actions bot added the documentation documentation label Jul 26, 2024
@rnishtala-sumo rnishtala-sumo force-pushed the backport-3793-to-release-v3.19 branch 2 times, most recently from 014dfc2 to 9e564aa Compare July 26, 2024 19:11
@github-actions github-actions bot removed the documentation documentation label Jul 26, 2024
@sumo-drosiek @rnishtala-sumo
chore: update Sumo Logic Kubernetes Setup to v3.15.1 (#3793)
382778c 
---------

Signed-off-by: Dominik Rosiek <[email protected]>
(cherry picked from commit 3c7e1e6)
@github-actions github-actions bot added the documentation documentation label Jul 26, 2024
@rnishtala-sumo
Copy link
Contributor Author

@sumo-drosiek the setup image 3.15.1 doesn't seem to be compatible with helm chart v3

Test_Helm_Default_OT 2024-07-26T19:21:49Z logger.go:66: cp: can't create '/terraform/locals.tf': Permission denied
Test_Helm_Default_OT 2024-07-26T19:21:49Z logger.go:66: cp: can't create '/terraform/main.tf': Permission denied
Test_Helm_Default_OT 2024-07-26T19:21:49Z logger.go:66: cp: can't create '/terraform/providers.tf': Permission denied
Test_Helm_Default_OT 2024-07-26T19:21:49Z logger.go:66: cp: can't create '/terraform/resources.tf': Permission denied
Test_Helm_Default_OT 2024-07-26T19:21:49Z logger.go:66: cp: can't create '/terraform/variables.tf': Permission denied
Test_Helm_Default_OT 2024-07-26T19:21:49Z logger.go:66: cp: can't create '/terraform/fields.tf': Permission denied

@rnishtala-sumo
Copy link
Contributor Author

Found the issue, looks like the securityContext has changed for the job definitions.

@rnishtala-sumo rnishtala-sumo merged commit 68fb649 into release-v3.19 Jul 26, 2024
39 checks passed
@rnishtala-sumo rnishtala-sumo deleted the backport-3793-to-release-v3.19 branch July 26, 2024 20:17
@sumo-drosiek
Copy link
Contributor

@sumo-drosiek the setup image 3.15.1 doesn't seem to be compatible with helm chart v3

Test_Helm_Default_OT 2024-07-26T19:21:49Z logger.go:66: cp: can't create '/terraform/locals.tf': Permission denied
Test_Helm_Default_OT 2024-07-26T19:21:49Z logger.go:66: cp: can't create '/terraform/main.tf': Permission denied
Test_Helm_Default_OT 2024-07-26T19:21:49Z logger.go:66: cp: can't create '/terraform/providers.tf': Permission denied
Test_Helm_Default_OT 2024-07-26T19:21:49Z logger.go:66: cp: can't create '/terraform/resources.tf': Permission denied
Test_Helm_Default_OT 2024-07-26T19:21:49Z logger.go:66: cp: can't create '/terraform/variables.tf': Permission denied
Test_Helm_Default_OT 2024-07-26T19:21:49Z logger.go:66: cp: can't create '/terraform/fields.tf': Permission denied

#3811 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@echlebek echlebek echlebek approved these changes

@github-actions GitHub Actions Awaiting requested review from GitHub Actions

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

Successfully merging this pull request may close these issues.
3 participants
@rnishtala-sumo @sumo-drosiek @echlebek