Skip to content

Conversation

sgontla
Copy link
Contributor

@sgontla sgontla commented Jul 20, 2021

Merge feature-appframework to develop as part of the release 1.0.2

kumarajeet and others added 30 commits May 12, 2021 17:28
1) Log only if apps framework feature flag is true
2) Use scoped logging
it was not given. Enhanced the CRDs descriptions for apps framework for
better understanding of this interface.
1) Log only if apps framework feature flag is true
2) Use scoped logging
it was not given. Enhanced the CRDs descriptions for apps framework for
better understanding of this interface.
it was not given. Enhanced the CRDs descriptions for apps framework for
better understanding of this interface.
After upgrading operator-sdk to v0.18.2 the nightly builds started to fail due to kubernetes/kubernetes#91395 which is an issue with how Kubernetes 1.18 interprets the operator-sdk v0.18.2 generated CRDs.

The issue manifests as :
"The CustomResourceDefinition “clustermasters.enterprise.splunk.com” is invalid: spec.versions[0].schema.openAPIV3Schema.properties[spec].properties[serviceTemplate].properties[spec].properties[ports].items.properties[protocol].default: Required value: this property is in x-kubernetes-list-map-keys, so it must have a default or be a required property"
Manually fix nightly build after Operator-SDK v0.18.2 upgrade
CSPL-1027: Utility to download or upload file to s3
pdhanoya-splunk and others added 16 commits July 13, 2021 13:22
* Enforce TLS between Operator and AppRep

Make sure that secure connections are used when connection to the
AppRepo volumes from the Splunk Operator.

* Fix lint error

* Remove unnecessary setting of disableSSL
* CSPL-793: Add documentation for App Framework

* - addressing review comments

* - addressing a review comment

* - Addressing review comments
- As part of supporting the App Framework spec changes, increment the CRD version to v2.
- Increment Splunk Operator version to 1.0.2
* CSPL-1167: Operator changes to honor cluster scope

* addressed review comment
* CSPL-1193: update make_bundle.sh to fix removal of protocol field

* added a comment
Changing the Operator reference to 1.0.2 and and Splunk Enterprise ref to 8.2.1
CSPL-1200: Version changes
CSPL: Automation to check apps are not installed on deployer and ClusterMaster in ClusterWide Install
@pdhanoya-splunk
Copy link
Contributor

Expected failures in smoke test will pass when splunk:edge has ansible changes.

[1] c3appfw test
[1] /home/circleci/project/test/c3/appframework/appframework_test.go:29
[1]   Clustered deployment (C3 - clustered indexer, search head cluster)
[1]   /home/circleci/project/test/c3/appframework/appframework_test.go:233
[1]     smoke, c3, integration, appframework: can deploy a C3 SVA and have apps installed locally on CM and SHC Deployer [It]
[1]     /home/circleci/project/test/c3/appframework/appframework_test.go:234
[1] 
[1]     Timed out after 1500.000s.
[1]     Expected
[1]         <common.Phase>: Pending
[1]     to equal
[1]         <common.Phase>: Ready
[1] 
[1]     /home/circleci/project/test/testenv/verificationutils.go:103
[1] ------------------------------
[1] {"level":"info","ts":1626819302.8885179,"msg":"testenv teardown is skipped!\n","testenv":"5261045-c3appfw-dnj"}
[1] {"level":"info","ts":1626819302.888538,"msg":"testenv teardown is skipped!\n","testenv":"5261045-c3appfw-dnj"}
[1] 
[1] JUnit report was created: /home/circleci/project/test/c3/appframework/c3appfw-dnj_junit.xml
[1] 
[1] 
[1] Summarizing 1 Failure:
[1] 
[1] [Fail] c3appfw test Clustered deployment (C3 - clustered indexer, search head cluster) [It] smoke, c3, integration, appframework: can deploy a C3 SVA and have apps installed locally on CM and SHC Deployer 
[1] /home/circleci/project/test/testenv/verificationutils.go:103
[1] 
[1] Ran 1 of 2 Specs in 2022.382 seconds
[1] FAIL! -- 0 Passed | 1 Failed | 0 Pending | 1 Skipped

@pdhanoya-splunk pdhanoya-splunk merged commit 6b7594e into develop Jul 20, 2021
@smohan-splunk smohan-splunk changed the title Feature appframework CSPL-1085: Feature appframework Jul 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.