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 • 2 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 06 '24 19:02 rnhan

Pull Request Test Coverage Report for Build 7805082681

  • 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.313%

Totals Coverage Status
Change from base Build 7778692681: 0.0%
Covered Lines: 2285
Relevant Lines: 2881

💛 - Coveralls

coveralls avatar Feb 06 '24 19:02 coveralls

please recreate this PR directly from a branch off the official master branch to allow required checks/gates to run

cameronmeissner avatar Feb 13 '24 16:02 cameronmeissner

feel free to reopen without using a fork if needed

cameronmeissner avatar Aug 28 '24 23:08 cameronmeissner