-
Notifications
You must be signed in to change notification settings - Fork 2
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
Upgrade api calls to work with OpenShift 4.x #55
base: develop
Are you sure you want to change the base?
Conversation
singularo
commented
Apr 12, 2022
- Imagestream creation working.
Tests run passing again. Ignoring the old ClientTest.php as it requires a running shepherd to curl against. Would prefer to mock the guzzle responses, but its not been working for ages, so... ignore for now. |
@@ -1497,7 +1456,7 @@ public function updateReplicationControllers($name, $label = NULL, $replica_coun | |||
|
|||
$repControllers = $this->getReplicationControllers($name, $label); | |||
if ($replica_count === NULL) { | |||
// @todo - err what ? | |||
// @todo err what ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great todo.
While batch api v1beta works, we should move to stable. |
Until shepherd is merged, we want to keep 4.x separate from 3.x... so lets leave this branch open for now |
…ervice-accounts sapp 2505 - multiple service accounts
* Add support for updating storage size. * Add missing token var, comments.
* Add labels functionality to route object. * Remove unneeded label code, add missing annotation class, update test. * Wrong one. * Dangling comma. * Remove duplicated annotations and put all in the ObjectBase.