Skip to content

Commit

Permalink
Stop testing the cOctopusSeqLogger component temporarily (#249)
Browse files Browse the repository at this point in the history
As it's stopped working with 2020.1, as Octopus got converted to netcore, and can no longer load a dll that is dropped into the application directory
  • Loading branch information
matt-richardson authored Apr 11, 2020
1 parent e7078a9 commit 17183a7
Show file tree
Hide file tree
Showing 7 changed files with 0 additions and 89 deletions.
13 changes: 0 additions & 13 deletions Tests/Scenarios/Server_Scenario_01_Install.ps1
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
$pass = ConvertTo-SecureString "SuperS3cretPassw0rd!" -AsPlainText -Force
$cred = New-Object System.Management.Automation.PSCredential ("OctoAdmin", $pass)

$seqPlainTextApiKey = ConvertTo-SecureString "MyMagicSeqApiKey" -AsPlainText -Force
$seqApiKey = New-Object System.Management.Automation.PSCredential ("ignored", $seqPlainTextApiKey)

Configuration Server_Scenario_01_Install
{
Import-DscResource -ModuleName OctopusDSC
Expand Down Expand Up @@ -46,16 +43,6 @@ Configuration Server_Scenario_01_Install
LogRequestMetrics = $false
}

cOctopusSeqLogger "Enable logging to seq"
{
InstanceType = "OctopusServer"
Ensure = "Present"
SeqServer = "http://localhost/seq"
SeqApiKey = $seqApiKey
Properties = @{ Application = "Octopus"; Server = "MyServer" }
DependsOn = "[cOctopusServer]OctopusServer"
}

cOctopusServerUsernamePasswordAuthentication "Enable Username/Password Auth"
{
InstanceName = "OctopusServer"
Expand Down
7 changes: 0 additions & 7 deletions Tests/Scenarios/Server_Scenario_03_Remove.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -14,12 +14,6 @@ Configuration Server_Scenario_03_Remove
ConfigurationMode = 'ApplyOnly'
}

cOctopusSeqLogger "Disable logging to seq"
{
InstanceType = 'OctopusServer'
Ensure = 'Absent'
}

cOctopusServer OctopusServer
{
Ensure = "Absent"
Expand All @@ -39,7 +33,6 @@ Configuration Server_Scenario_03_Remove

# dont mess with stats
AllowCollectionOfUsageStatistics = $false
DependsOn = "[cOctopusSeqLogger]Disable logging to seq"
}
}
}
13 changes: 0 additions & 13 deletions Tests/Scenarios/Tentacle_Scenario_01_Install.ps1
Original file line number Diff line number Diff line change
@@ -1,6 +1,3 @@
$seqPlainTextApiKey = ConvertTo-SecureString "MyMagicSeqApiKey" -AsPlainText -Force
$seqApiKey = New-Object System.Management.Automation.PSCredential ("ignored", $seqPlainTextApiKey)

$config = get-content "c:\temp\octopus-configured.marker" | ConvertFrom-Json
$OctopusServerUrl = $config.OctopusServerUrl
$ApiKey = $config.OctopusApiKey
Expand Down Expand Up @@ -52,16 +49,6 @@ Configuration Tentacle_Scenario_01_Install
Policy = "Test Policy"
}

cOctopusSeqLogger "Enable logging to seq"
{
InstanceType = "Tentacle"
Ensure = "Present"
SeqServer = "http://localhost/seq"
SeqApiKey = $seqApiKey
Properties = @{ Application = "Octopus"; Server = "MyServer" }
DependsOn = "[cTentacleAgent]ListeningTentacle"
}

cTentacleAgent PollingTentacle
{
Ensure = "Present";
Expand Down
11 changes: 0 additions & 11 deletions Tests/Scenarios/Tentacle_Scenario_02_Remove.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -18,12 +18,6 @@ Configuration Tentacle_Scenario_02_Remove
ConfigurationMode = 'ApplyOnly'
}

cOctopusSeqLogger "Disable logging to seq"
{
InstanceType = 'Tentacle'
Ensure = 'Absent'
}

cTentacleAgent ListeningTentacle
{
Ensure = "Absent";
Expand All @@ -43,7 +37,6 @@ Configuration Tentacle_Scenario_02_Remove
ListenPort = 10933;
DefaultApplicationDirectory = "C:\Applications"
TentacleHomeDirectory = "C:\Octopus\ListeningTentacleHome"
DependsOn = "[cOctopusSeqLogger]Disable logging to seq"
}

cTentacleAgent PollingTentacle
Expand All @@ -66,7 +59,6 @@ Configuration Tentacle_Scenario_02_Remove
DefaultApplicationDirectory = "C:\Applications"
CommunicationMode = "Poll"
TentacleHomeDirectory = "C:\Octopus\PollingTentacleHome"
DependsOn = "[cOctopusSeqLogger]Disable logging to seq"
}

cTentacleAgent ListeningTentacleWithoutAutoRegister
Expand All @@ -89,7 +81,6 @@ Configuration Tentacle_Scenario_02_Remove
TentacleHomeDirectory = "C:\Octopus\ListeningTentacleWithoutAutoRegisterHome"

RegisterWithServer = $false
DependsOn = "[cOctopusSeqLogger]Disable logging to seq"
}

cTentacleAgent ListeningTentacleWithThumbprintWithoutAutoRegister
Expand All @@ -113,7 +104,6 @@ Configuration Tentacle_Scenario_02_Remove

RegisterWithServer = $false
OctopusServerThumbprint = $ServerThumbprint
DependsOn = "[cOctopusSeqLogger]Disable logging to seq"
}

cTentacleAgent WorkerTentacle
Expand Down Expand Up @@ -165,7 +155,6 @@ Configuration Tentacle_Scenario_02_Remove
TentacleHomeDirectory = "C:\Octopus\ListeningTentacleWithCustomAccountHome"

TentacleServiceCredential = $serviceusercredential
DependsOn = "[cOctopusSeqLogger]Disable logging to seq"
}
}
}
15 changes: 0 additions & 15 deletions Tests/Spec/server_scenario_01_install_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -71,21 +71,6 @@
it { should exist }
end

#seq logging
describe file('C:/Program Files/Octopus Deploy/Octopus/Seq.Client.NLog.dll') do
it { should be_file }
end

describe file('C:/Program Files/Octopus Deploy/Octopus/Octopus.Server.exe.nlog') do
it { should be_file }
its(:content) { should match /<add assembly="Seq.Client.NLog" \/>/ }
its(:content) { should match /<logger name="\*" minlevel="Info" writeTo="seqbufferingwrapper" \/>/ }
its(:content) { should match /<target name=\"seqbufferingwrapper\" xsi:type=\"BufferingWrapper\" bufferSize=\"1000\" flushTimeout=\"2000\">/}
its(:content) { should match /<target name="seq" xsi:type="Seq" serverUrl="http:\/\/localhost\/seq" apiKey="MyMagicSeqApiKey">/ }
its(:content) { should match /<property name="Application" value="Octopus" \/>/ }
its(:content) { should match /<property name="Server" value="MyServer" \/>/ }
end

#dsc overall status
describe windows_dsc do
it { should be_able_to_get_dsc_configuration }
Expand Down
15 changes: 0 additions & 15 deletions Tests/Spec/server_scenario_02_install_second_node_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -74,21 +74,6 @@
it { should exist }
end

#seq logging
describe file('C:/Program Files/Octopus Deploy/Octopus/Seq.Client.NLog.dll') do
it { should be_file }
end

describe file('C:/Program Files/Octopus Deploy/Octopus/Octopus.Server.exe.nlog') do
it { should be_file }
its(:content) { should match /<add assembly="Seq.Client.NLog" \/>/ }
its(:content) { should match /<logger name="\*" minlevel="Info" writeTo="seqbufferingwrapper" \/>/ }
its(:content) { should match /<target name=\"seqbufferingwrapper\" xsi:type=\"BufferingWrapper\" bufferSize=\"1000\" flushTimeout=\"2000\">/}
its(:content) { should match /<target name="seq" xsi:type="Seq" serverUrl="http:\/\/localhost\/seq" apiKey="MyMagicSeqApiKey">/ }
its(:content) { should match /<property name="Application" value="Octopus" \/>/ }
its(:content) { should match /<property name="Server" value="MyServer" \/>/ }
end

#dsc overall status
describe windows_dsc do
it { should be_able_to_get_dsc_configuration }
Expand Down
15 changes: 0 additions & 15 deletions Tests/Spec/tentacle_scenario_01_install_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -239,21 +239,6 @@
its(:content) { should match /Tentacle\.Communication\.TrustedOctopusServers.*#{config['OctopusServerThumbprint']}/}
end

#seq logging
describe file('C:/Program Files/Octopus Deploy/Tentacle/Seq.Client.NLog.dll') do
it { should be_file }
end

describe file('C:/Program Files/Octopus Deploy/Tentacle/Tentacle.exe.nlog') do
it { should be_file }
its(:content) { should match /<add assembly="Seq.Client.NLog" \/>/ }
its(:content) { should match /<logger name="\*" minlevel="Info" writeTo="seqbufferingwrapper" \/>/ }
its(:content) { should match /<target name=\"seqbufferingwrapper\" xsi:type=\"BufferingWrapper\" bufferSize=\"1000\" flushTimeout=\"2000\">/}
its(:content) { should match /<target name="seq" xsi:type="Seq" serverUrl="http:\/\/localhost\/seq" apiKey="MyMagicSeqApiKey">/ }
its(:content) { should match /<property name="Application" value="Octopus" \/>/ }
its(:content) { should match /<property name="Server" value="MyServer" \/>/ }
end

### DSC success
describe windows_dsc do
it { should be_able_to_get_dsc_configuration }
Expand Down

0 comments on commit 17183a7

Please sign in to comment.