-
Notifications
You must be signed in to change notification settings - Fork 4
STAC-22970: Installation scripts support STS_SKIP_SSL, STS_CA_CERT_BASE64_DATA, and STS_CA_CERT_path #105
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
Conversation
…SE64_DATA, and STS_CA_CERT_path
|
Tests results Testing Bash script for SUSE Observability CLI installation with different SSL certificate handling scenarios in Macos. 1. FAIL, the unknown certificaterm ~/.config/stackstate-cli/config.yaml 2. SUCCESS, skip SSL validationrm ~/.config/stackstate-cli/config.yaml 3. SUCCESS, connect with the self-signed certificate from the filerm ~/.config/stackstate-cli/config.yaml 4. SUCCESS, connect with the self-signed certificate passed via command line argumentrm ~/.config/stackstate-cli/config.yaml Testing PowerShell script for SUSE Observability CLI installation with different SSL certificate handling scenarios. 1. FAIL, the unknown certificaterm C:\Users\Administrator.config\stackstate-cli\config.yaml 2. SUCCESS, skip SSL validation. { iwr -useb https://raw.githubusercontent.com/StackVista/stackstate-cli/a63c7604f8ccd795cb913b6345be14a9d815c25b/scripts/publish/installers/install.ps1 } | iex; install -StsUrl "https://viliakov.sandbox.stackstate.io" -StsApiToken "..." -StsSkipSsl true 3. SUCCESS, connect with the self-signed certificate from the file. { iwr -useb https://raw.githubusercontent.com/StackVista/stackstate-cli/a63c7604f8ccd795cb913b6345be14a9d815c25b/scripts/publish/installers/install.ps1 } | iex; install -StsUrl "https://viliakov.sandbox.stackstate.io" -StsApiToken "..." -StsCaCertPath cert.crt 4. SUCCESS, connect with the self-signed certificate passed via command line argument. { iwr -useb https://raw.githubusercontent.com/StackVista/stackstate-cli/a63c7604f8ccd795cb913b6345be14a9d815c25b/scripts/publish/installers/install.ps1 } | iex; install -StsUrl "https://viliakov.sandbox.stackstate.io" -StsApiToken "..." -StsCaCertBase64Data "LS0tLS1CRUdJTiBDRVJUSUZJQ0FURS0tLS0tDQpNSUlEcnpDQ0FwZWdBd0lCQWdJVURNUGtMT0xHSjEyNDM4TWJJMzJleWtidzJ4b3dEUVlKS29aSWh2Y05BUUVMDQpCUUF3S1RFbk1DVUdBMVVFQXd3ZWRtbHNhV0ZyYjNZdWMyRnVaR0p2ZUM1emRHRmphM04wWVhSbExtbHZNQjRYDQpEVEkxTURjeE56RXpNamd6TjFvWERUSTJNRGN4TnpFek1qZ3pOMW93S1RFbk1DVUdBMVVFQXd3ZWRtbHNhV0ZyDQpiM1l1YzJGdVpHSnZlQzV6ZEdGamEzTjBZWFJsTG1sdk1JSUJJakFOQmdrcWhraUc5dzBCQVFFRkFBT0NBUThBDQpNSUlCQ2dLQ0FRRUEwTUlkUE94ckNwWEIrRjZQNk5ZN015T2ltdVZpVldKR0RXOWNrejRtWFpZQ0pENGlxcktTDQpZNGJQNk9ETzRCZ1d4S0ZFbHhOZHdOSXFoTG1JN1JSMU1XU1JvNDdveHdQTG5xdzNJTmxzWDB0MXJCcDZrNnpLDQpLNFlZK3dHZFVIL2tldWcwM3VNUzdIeEJYRW1oQ2FNbkdQajJCQmZCNFVSYzQxRGtGZXhHVS9GaTFjeXYwYUNxDQpDZ3hiVGhOL2ZHU0dOMmV2THVhYms5bWZ3NEFIM0s4aXNRK2tTOWkzTzQ1OUJnREdIOHlqYnJXZkJVZFBYVng1DQppRmlZakdKalZNMHBUUDFkTnJpVGM4OGxwYWpYUksrKzZPMmdtakw5a2JmMFBHelJzdnFxVmdJMDd5Ujh1VjFJDQowTWFVd00yL1ZKclZCNnQ4MHdCdUMxVGl2K1JpWW10Slh3SURBUUFCbzRIT01JSExNQjBHQTFVZERnUVdCQlNoDQppS0JDbXJwOGpIU0NNdlVuSHYvV2dnN0x5REFmQmdOVkhTTUVHREFXZ0JTaGlLQkNtcnA4akhTQ012VW5Idi9XDQpnZzdMeURBUEJnTlZIUk1CQWY4RUJUQURBUUgvTUhnR0ExVWRFUVJ4TUcrQ0huWnBiR2xoYTI5MkxuTmhibVJpDQpiM2d1YzNSaFkydHpkR0YwWlM1cGI0SWpiM1JzY0MxMmFXeHBZV3R2ZGk1ellXNWtZbTk0TG5OMFlXTnJjM1JoDQpkR1V1YVcrQ0tHOTBiSEF0YUhSMGNDMTJhV3hwWVd0dmRpNXpZVzVrWW05NExuTjBZV05yYzNSaGRHVXVhVzh3DQpEUVlKS29aSWh2Y05BUUVMQlFBRGdnRUJBSXVCRlZxSnNKSW1PQjR0aFJrK0ZGZDdVSmxLMWtRbmE5d29LdjIzDQpqdStmcEVXZ1paUTBVL3hHUzlmM0p2eENVSnY4b2ozSFlrZlBRUWd0UG1ld0FUVkJ4MmNUUnBvZ1Y2SkZjQW83DQpmUFNMQ3pPdVN0M2M0U00xT3REbnlUb1VhQWY2WVFRVDRtK1Y0SUtiNlFvMFhXZkN4aGt1S0psT2ZtRHRxTmcvDQp1VllqZkc3K0tPWnMrNkNUSndxZEl3cE5EYkxEK0ROZm8zYi9jNzMxUWExYjlvOFo4cklyTnJZWGo0a2x5M0QxDQo5N1FpVkpDTDB1L2ZDKy9Lc1V4cTl5bkFZU1BneWQyQ0JueG5RRGNxOGFRQVRWVGxBYWZTZmswc2h2dWNnUW1KDQpLSUw5eGFNM2lUZHZyV0d0V2VBaUVRb2NzUkJKTTV4anF0bnUwUjV4RGxMVS9UUT0NCi0tLS0tRU5EIENFUlRJRklDQVRFLS0tLS0=" |
deontaljaard
left a comment
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.
LGTM.
Add SSL/CA certificate support to installation scripts
Summary
Changes Made
Bash script (install.sh):
PowerShell script (install.ps1):
Validation Rules
Test Plan