Skip to content

k8s: add a locator for image objects#3525

Merged
nicks merged 1 commit into
masterfrom
nicks/locators
Jun 29, 2020
Merged

k8s: add a locator for image objects#3525
nicks merged 1 commit into
masterfrom
nicks/locators

Conversation

@nicks

@nicks nicks commented Jun 29, 2020

Copy link
Copy Markdown
Contributor

Hello @landism, @jazzdan,

Please review the following commits I made in branch nicks/locators:

d6af15a (2020-06-29 12:46:56 -0400)
k8s: add a locator for image objects
This is moving towards #3427,
but also is a good model of how to write new kinds of image locators

Code review reminders, by giving a LGTM you attest that:

  • Commits are adequately tested
  • Code is easy to understand and conforms to style guides
  • Incomplete code is marked with TODOs
  • Code is suitably instrumented with logging and metrics

@nicks
nicks requested review from jazzdan and landism June 29, 2020 16:47
This is moving towards #3427,
but also is a good model of how to write new kinds of image locators
@nicks
nicks merged commit 6892080 into master Jun 29, 2020
@nicks
nicks deleted the nicks/locators branch June 29, 2020 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants