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

Upgrade api calls to work with OpenShift 4.x #55

Open
wants to merge 34 commits into
base: develop
Choose a base branch
from

Conversation

singularo
Copy link
Contributor

  • Imagestream creation working.

@singularo singularo changed the title WIP: Upgrade api calls to work with OpenShift 4.x Upgrade api calls to work with OpenShift 4.x May 4, 2022
@singularo singularo requested a review from pingers May 4, 2022 02:06
@singularo
Copy link
Contributor Author

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 ?
Copy link
Contributor

Choose a reason for hiding this comment

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

Great todo.

@pingers
Copy link
Contributor

pingers commented May 17, 2022

While batch api v1beta works, we should move to stable.
Networkpolicies are in the same boat - these will be getting reinstated due to the namespace changes.

@pingers
Copy link
Contributor

pingers commented May 17, 2022

Until shepherd is merged, we want to keep 4.x separate from 3.x... so lets leave this branch open for now
OR we lock commit on shepherd 3.x repo
OR we release the 3.x version with a tag and keep shepherd 3.x on that.

singularo and others added 11 commits May 24, 2022 16:49
…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.
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