Skip to content

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.

$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.

3 participants