Skip to content

Commit

Permalink
Merge pull request os-autoinst#20646 from lpalovsky/sdaf_remove_worka…
Browse files Browse the repository at this point in the history
…rounds

SDAF remove soft failures
  • Loading branch information
lpalovsky authored Nov 25, 2024
2 parents 19177d6 + f7f4889 commit 7928021
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 17 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -574,13 +574,6 @@ sub prepare_sdaf_project {
single_branch => 'yes',
output_log_file => log_dir() . '/git_clone_templates.log');

# Workaround for SDAF bug https://github.com/Azure/sap-automation/issues/617
record_soft_failure 'gh#Azure/sap-automation#617';
file_content_replace(
"$deployment_dir/sap-automation/deploy/terraform/terraform-units/modules/sap_landscape/providers.tf",
'>= 3.23' => '3.116.0'
);

assert_script_run("cp -Rp sap-automation-samples/Terraform/WORKSPACES $deployment_dir/WORKSPACES");
# Ensure correct directories are in place
my %vnet_codes = (
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -74,16 +74,6 @@ sub run {
az_login();

for my $playbook_options (@execute_playbooks) {
# Package 'fence-agents-azure-arm' is not yet installed by SDAF, therefore a workaround has to be applied
if ($playbook_options->{playbook_filename} eq 'playbook_04_00_00_db_install.yaml') {
record_soft_failure("bsc#1226671 - New package 'fence-agents-azure=arm' has to be installed to prevent HA setup failure");
my @cmd = ('ansible', 'QES_DB',
"--private-key=$sdaf_config_root_dir/sshkey",
'--inventory=' . get_required_var('SAP_SID') . '_hosts.yaml',
'--module-name=shell',
'--args="sudo zypper in -y fence-agents-azure-arm"');
assert_script_run(join(' ', @cmd));
}
$sles4sap::sap_deployment_automation_framework::basetest::serial_regexp_playbook = 1;
sdaf_execute_playbook(%{$playbook_options}, sdaf_config_root_dir => $sdaf_config_root_dir);
$sles4sap::sap_deployment_automation_framework::basetest::serial_regexp_playbook = 0;
Expand Down

0 comments on commit 7928021

Please sign in to comment.