Skip to content
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

Consume version 0.19.0 of conjur-authn-k8s-client #1352

Merged
merged 1 commit into from Oct 8, 2020

Conversation

orenbm
Copy link
Member

@orenbm orenbm commented Oct 8, 2020

This version introduces some changes that we can benefit from, especially these:

@orenbm orenbm requested a review from a team as a code owner October 8, 2020 09:32
@orenbm orenbm force-pushed the consume-authn-client-0.19.0 branch 4 times, most recently from afc65b2 to fe02ac1 Compare October 8, 2020 09:50
CHANGELOG.md Outdated
### Added
- The `vault` provider now supports loading secrets from the KV Version 2 secret
engine. Reference a secret in Vault using the right path and a field
navigation in the Secretless configuration.
[cyberark/secretless-broker#1331](https://github.com/cyberark/secretless-broker/issues/1331)

### Changed
- Update k8s authenticator client version to
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trailing spaces

CHANGELOG.md Outdated
### Changed
- Update k8s authenticator client version to
[0.19.0](https://github.com/cyberark/conjur-authn-k8s-client/blob/master/CHANGELOG.md#0190---2020-10-08),
which adds some fixes around cert injection failure (see also changes in
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trailing spaces

CHANGELOG.md Outdated
### Added
- The `vault` provider now supports loading secrets from the KV Version 2 secret
engine. Reference a secret in Vault using the right path and a field
navigation in the Secretless configuration.
[cyberark/secretless-broker#1331](https://github.com/cyberark/secretless-broker/issues/1331)

### Changed
- Update k8s authenticator client version to
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lists should be surrounded by blank lines

This version introduces some changes that we can benefit from, especially these:
- Errors in the certificate injection process on login are now printed to the client logs.
  [cyberark/conjur-authn-k8s-client#/170](cyberark/conjur-authn-k8s-client#170)
- When authentication fails, the exponential backoff retry is correctly reset so
  that it will continue to attempt to authenticate until backoff is exhausted.
  [cyberark/conjur-authn-k8s-client#158](cyberark/conjur-authn-k8s-client#158)
- Wait slightly for the client certificate file to exist after login before
  raising an error.
  [cyberark/conjur-authn-k8s-client#119](cyberark/conjur-authn-k8s-client#119)
@orenbm orenbm force-pushed the consume-authn-client-0.19.0 branch from fe02ac1 to f141282 Compare October 8, 2020 10:13
### Added
- The `vault` provider now supports loading secrets from the KV Version 2 secret
engine. Reference a secret in Vault using the right path and a field
navigation in the Secretless configuration.
[cyberark/secretless-broker#1331](https://github.com/cyberark/secretless-broker/issues/1331)

### Changed
- Update k8s authenticator client version to
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lists should be surrounded by blank lines

@codeclimate
Copy link

codeclimate bot commented Oct 8, 2020

Code Climate has analyzed commit f141282 and detected 2 issues on this pull request.

Here's the issue category breakdown:

Category Count
Style 2

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 50.2% (0.0% change).

View more on Code Climate.

Copy link
Contributor

@izgeri izgeri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@izgeri izgeri merged commit be6c26b into master Oct 8, 2020
@izgeri izgeri deleted the consume-authn-client-0.19.0 branch October 8, 2020 13:34
BradleyBoutcher pushed a commit that referenced this pull request Oct 16, 2020
Consume version 0.19.0 of conjur-authn-k8s-client
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants