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

[Draft]Verify: verification for splunk upgrade scenario #1195

Open
wants to merge 92 commits into
base: feature-level2
Choose a base branch
from
Open

[Draft]Verify: verification for splunk upgrade scenario #1195

wants to merge 92 commits into from

Conversation

vivekr-splunk
Copy link
Collaborator

@vivekr-splunk vivekr-splunk commented Jul 18, 2023 •
edited
Loading

Verification step in Splunk Upgrade Scenario

  1. License manager check license information
  2. cluster manager check splunk health

All these information is shown in Status.Conditions.

tgarg-splunk and others added 30 commits June 14, 2023 14:28
@tgarg-splunk
Added changeAnnotation method
bed6401
@tgarg-splunk
Refined changeClusterManagerAnnotations
7de6f36
@vivekr-splunk @tgarg-splunk
test case for upgrade scenario
f4453a1
@tgarg-splunk
Modified kuttl cases
32c385d
@tgarg-splunk
Added kuttl tests; Updated LicenseMaster
8efb456
@tgarg-splunk
Fixed uninstall kuttl test
3c75c99
@tgarg-splunk
Fixed unit test
ee474fc
@tgarg-splunk
Removed changeAnnotation from licenseMaster
35a2eb0
@tgarg-splunk
Added branch in int-tests
d9540fd
@tgarg-splunk
Completed code coverage tests
5a17a5f
@tgarg-splunk
Added upgradeScenario and related methods for CM
706dc96
@tgarg-splunk
Added label selectors to get Current Image
c5af670
@tgarg-splunk
Changed pod.Spec to pod.Status
237ecdf
@tgarg-splunk
Added changeAnnotations for MC
5966d87
@tgarg-splunk
Added kuttl test cases
b827bd9
@tgarg-splunk
Fixed unit test
a1159a8
@tgarg-splunk
Fixed SmartStore unit test
3aa032b
@tgarg-splunk
Added code coverage test
f0e73c8
@vivekr-splunk @tgarg-splunk
using fake client instead of mock
53f6f68
@vivekr-splunk @tgarg-splunk
removed creating statefulset and service
8301fd9
@tgarg-splunk
Corrected LMCurrentImage method
8353d25
@tgarg-splunk
Completed Coverage tests for CM
dab93db
@tgarg-splunk
Refined changeClusterManagerAnnotations
fb77880
@vivekr-splunk @tgarg-splunk
test case for upgrade scenario
b6c70d5
@tgarg-splunk
Modified kuttl cases
6957966
@tgarg-splunk
Added kuttl tests; Updated LicenseMaster
70c73c2
@tgarg-splunk
Fixed unit test
4a945eb
@tgarg-splunk
Removed changeAnnotation from licenseMaster
a2c9f6d
@tgarg-splunk
Completed code coverage tests
86baa21
@tgarg-splunk
Resolved all conflict issues
b24fca7
@semgrep-app
Copy link

semgrep-app bot commented Jul 26, 2023

Semgrep found 6 G304-1 findings:

File open from tainted variable

Ignore this finding from G304-1.

vivekr-splunk and others added 6 commits July 26, 2023 08:47
@vivekr-splunk
fixed meta object argument
64c601f 
Signed-off-by: vivekr-splunk <94569031+vivekr-splunk@users.noreply.github.com>
@vivekr-splunk
fixed meta object argument
37b1f6e 
Signed-off-by: vivekr-splunk <94569031+vivekr-splunk@users.noreply.github.com>
@tgarg-splunk
Added maintenance mode
b6620b6
@tgarg-splunk
Fixed controller code
59ad0aa
@vivekr-splunk
changes to support lm
5f96a5c 
Signed-off-by: vivekr-splunk <94569031+vivekr-splunk@users.noreply.github.com>
@vivekr-splunk
working license manager verification code
1855c51 
Signed-off-by: vivekr-splunk <94569031+vivekr-splunk@users.noreply.github.com>
@vivekr-splunk
working license manager verification code after PR review
0b0989a 
Signed-off-by: vivekr-splunk <94569031+vivekr-splunk@users.noreply.github.com>
@vivekr-splunk vivekr-splunk changed the title [Draft]: Verify Verify: verification for splunk upgrade scenario Aug 3, 2023
@vivekr-splunk
Merge branch 'develop' into verify
ef92864
@vivekr-splunk
fixed some test cases
cb0b5e7 
Signed-off-by: vivekr-splunk <94569031+vivekr-splunk@users.noreply.github.com>
@vivekr-splunk
fixed some test cases
74937ec 
Signed-off-by: vivekr-splunk <94569031+vivekr-splunk@users.noreply.github.com>
@vivekr-splunk
upgrading go 1.21
fe0cce8 
Signed-off-by: vivekr-splunk <94569031+vivekr-splunk@users.noreply.github.com>
@vivekr-splunk
changed ioutils to os
fed6f8a 
Signed-off-by: vivekr-splunk <94569031+vivekr-splunk@users.noreply.github.com>
@vivekr-splunk
changed few formatting issues
ad0882a 
Signed-off-by: vivekr-splunk <94569031+vivekr-splunk@users.noreply.github.com>
@vivekr-splunk vivekr-splunk changed the title Verify: verification for splunk upgrade scenario [Draft]Verify: verification for splunk upgrade scenario Sep 14, 2023
@vivekr-splunk
adding lm to upgrade testing
a5ae87f 
Signed-off-by: vivekr-splunk <vivekr@splunk.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tgarg-splunk tgarg-splunk tgarg-splunk left review comments

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

Successfully merging this pull request may close these issues.
2 participants
@vivekr-splunk @tgarg-splunk