AgentBaker icon indicating copy to clipboard operation
AgentBaker copied to clipboard

feat: add iscsi-initiator-utils and libiscsi packages for iscsi csi d…

Open rnhan opened this issue 1 year ago • 1 comments

What type of PR is this?

/kind feature

What this PR does / why we need it:

Add iscsi-initiator-utils and libiscsi packages to AzureLinux base install

Which issue(s) this PR fixes:

Fixes #

Requirements:

Special notes for your reviewer:

Update from previous pull request of @croomes , instead adding into different location for AzureLinux and as advised by @hbeberman

Adds the /usr/sbin/iscsiadm binary, required by https://github.com/kubernetes-csi/csi-driver-iscsi

The package is installed on Ubuntu AKS nodes, and would allow https://learn.microsoft.com/en-us/azure/storage/elastic-san/elastic-san-connect-aks to work on Azure Linux.

Release note:

none

rnhan avatar Feb 14 '24 18:02 rnhan

Pull Request Test Coverage Report for Build 8207919057

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 79.39%

Totals Coverage Status
Change from base Build 8199595115: 0.0%
Covered Lines: 2292
Relevant Lines: 2887

💛 - Coveralls

coveralls avatar Feb 14 '24 18:02 coveralls

we now install iscsi-initiator-utils for mariner v2 / azure linux, feel free to reopen if we still need libiscsi

cameronmeissner avatar Aug 28 '24 21:08 cameronmeissner