Add test cases for
- KubernetesInternalRuntime, when creating from deployments
- UniqueNamesProvisioner, to ensure deployments and configmaps are
handled correctly
Signed-off-by: Angel Misevsk <amisevsk@redhat.com>
Remove podName parameter from addPod method and simply get it from the
pod's metadata.
Additional cleanup and documentation changes:
- Make some log messages level 'info' instead of 'debug'
- Modify argument order in messages to to make more sense
- Add docs for KubernetesEnvironment methods about how/when pods map can
be modified
- Fix issue where pods referencing nonexistent configmaps would emit
unreadable error messages due to configmap renaming.
- Clean up code according to PR Quality Review
Signed-off-by: Angel Misevski <amisevsk@redhat.com>
Add validation and processing on the dashboard side for recipes
containing deployments and/or configmaps. This allows users to create
new stacks using these items in recipes.
Signed-off-by: Angel Misevski <amisevsk@redhat.com>
- Rename method in KubernetesInternalRuntime to make its meaning more
clear
- Fix some documentation issues
- Refactor KubernetesEnvironment to set podData in constructor instead
of through builder, avoiding inconsistencies in podData when using
constructor directly.
- Rename KubernetesEnvironment#getPodData to getPodsData to better
represent the data that is returned
- Fix issue where pod-based recipes were not correctly getting
deployment name label and so were failing to start
- Fix issue where deployments were not given a unique name by
provisioner.
Signed-off-by: Angel Misevsk <amisevsk@redhat.com>
Allow configmaps to be specified in Kubernetes/OpenShift recipes. Also
changes UniqueNamesProvisioner to rewrite configmap names across the
environment to avoid name collisions.
Signed-off-by: Angel Misevski <amisevsk@redhat.com>
Changes necessary to make tests compile and pass after introduction of
deployments into Kubernetes/OpenShift recipes.
Signed-off-by: Angel Misevski <amisevsk@redhat.com>
Add support of deployments in Kubernetes and OpenShift recipes.
Important changes:
- Since a (Kubernetes|OpenShift)Environment now can contain *both* pods
and deployments simultaneously, we need to:
- Introduce a wrapper class, PodData, which stores spec and metadata
for either a pod or deployment, allowing provisioners to look in one
place for specs that need to be modified
- Disallow modifying the results of KubernetesEnvironment.getPods()
directly, as this would ignore any deployments in the environment
- Add method KubernetesEnvironment.getDeployments(), to obtain
deployment objects directly when needed.
- Modify workspace start process to start all deployments in environment
as well as all pods.
Note: In an effort to keep the changes in this PR readable, nothing is
done to fix broken tests due to the changes above. This commit will not
compile unless test compilation is skipped. The next commit fixes tests.
Signed-off-by: Angel Misevski <amisevsk@redhat.com>
* CHE-8909 fix add a new machine flow
Signed-off-by: Oleksii Orel <oorel@redhat.com>
* fix machine name in WorkspaceDetailsMachineActionsTest selenium test
* fix the regular expression for a container name
Signed-off-by: Oleksii Orel <oorel@redhat.com>
* Fix getting existed test workspace id
* Fix workspace name in CreateWorkspaceOnDashboardTest
* Remove error-prone command from LoginExistedUserWithOpenShiftOAuthTest
* Fix LoginNewUserWithOpenShiftOAuthTest selenium test
Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
* Upgrade version of selenium chromenode to 3.141.59-dubnium
* Fix test timeouts to comply with new chrome-node version
* Fix NewWorkspacePageTest selenium test
* Fix ImportProjectFromGitHubTest, AuthorizeOnGithubFromDashboardTest
Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
* Don't bind 'user' role to test user
Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
* Remove role 'user' from keycloak
Signed-off-by: Dmytro Nochevnov <dnochevn@redhat.com>
Yevhen Vydolob