Skip to content

Conversation

@akartsky
Copy link
Contributor

@akartsky akartsky commented May 14, 2021

Update runtime to 0.2.0 which includes Namespace scope feature

tested positive and negative case for namespace locally

go.sum Outdated
github.com/asaskevich/govalidator v0.0.0-20180720115003-f9ffefc3facf/go.mod h1:lB+ZfQJz7igIIfQNfa7Ml4HSf2uFQQRzpGGRXenZAgY=
github.com/asaskevich/govalidator v0.0.0-20190424111038-f61b66f89f4a/go.mod h1:lB+ZfQJz7igIIfQNfa7Ml4HSf2uFQQRzpGGRXenZAgY=
github.com/aws-controllers-k8s/runtime v0.1.2 h1:prqFY6tlmgsGMZizHvHqDqT1zECRqllepqplOHet4D4=
github.com/aws-controllers-k8s/runtime v0.1.2/go.mod h1:xA2F18PJerBHaqrS4de1lpP7skeSMeStkmh+3x5sWvw=
Copy link
Contributor

Choose a reason for hiding this comment

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

go mod tidy

@surajkota
Copy link
Contributor

surajkota commented May 14, 2021

could you add testing info to pr description

@ack-bot
Copy link
Collaborator

ack-bot commented May 18, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: akartsky, mbaijal, surajkota

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [akartsky,mbaijal,surajkota]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@surajkota
Copy link
Contributor

/lgtm

@surajkota surajkota merged commit 353dbee into aws-controllers-k8s:main May 18, 2021
ryansteakley pushed a commit to ryansteakley/sagemaker-controller that referenced this pull request May 18, 2021
…ontrollers-k8s#30)

* Update runtime to 0.2.0 which includes Namespace scope feature

* go mod tidy

Add samples for each CRD

Update README
ryansteakley pushed a commit to ryansteakley/sagemaker-controller that referenced this pull request May 19, 2021
…ontrollers-k8s#30)

* Update runtime to 0.2.0 which includes Namespace scope feature

* go mod tidy
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.

4 participants