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

Openshift: other goodies from CE-Arq #814

Closed
Ladicek opened this Issue Oct 3, 2017 · 6 comments

Comments

Projects
None yet
3 participants
@Ladicek
Contributor

Ladicek commented Oct 3, 2017

There's #467 for porting @Template from CE-Arq and #468 for porting @OpenShiftResource. These are probably the most important bits, but there's a bunch of other things in CE-Arq that could be valuable. See https://github.com/jboss-openshift/ce-arq/tree/master/api/src/main/java/org/jboss/arquillian/ce/api

I'm specifically thinking about:

  • @AddRoleToServiceAccount
  • @ExternalDeployment (though not sure what exactly that is)
  • @MountSecret
  • @Replicas (again not exactly sure what this is)
  • @RoleBinding (I wonder how @AddRoleToServiceAccount compares to this)
  • @RunInPod (again not exactly sure what this is)
  • @RunInPodDeployment (again not exactly sure what this is)

I'm filing this for tracking, as the CE-Arq bits seem quite useful.

CC @alesj

@alesj

This comment has been minimized.

Show comment
Hide comment
@alesj

alesj Oct 3, 2017

Collaborator

@Replicas - explicitly set more pod relicaps
@RunInPod -- similar to @RunAsClient, but it deploys and runs the tests in a side-car / pod
@RunInPodDeployment -- you can explicitly construct an archive with the tests

Collaborator

alesj commented Oct 3, 2017

@Replicas - explicitly set more pod relicaps
@RunInPod -- similar to @RunAsClient, but it deploys and runs the tests in a side-car / pod
@RunInPodDeployment -- you can explicitly construct an archive with the tests

@lordofthejars

This comment has been minimized.

Show comment
Hide comment
@lordofthejars

lordofthejars Oct 3, 2017

Member
Member

lordofthejars commented Oct 3, 2017

@Ladicek

This comment has been minimized.

Show comment
Hide comment
@Ladicek

Ladicek Oct 3, 2017

Contributor

I might have been unclear, but I currently don't have the capacity to implement/migrate this myself. So please don't expect PRs from me :-)

Contributor

Ladicek commented Oct 3, 2017

I might have been unclear, but I currently don't have the capacity to implement/migrate this myself. So please don't expect PRs from me :-)

@lordofthejars

This comment has been minimized.

Show comment
Hide comment
@lordofthejars

lordofthejars Oct 3, 2017

Member
Member

lordofthejars commented Oct 3, 2017

@Ladicek

This comment has been minimized.

Show comment
Hide comment
@Ladicek

Ladicek Oct 3, 2017

Contributor

Yea I think so. But #467 and #468 are more important IMHO.

Contributor

Ladicek commented Oct 3, 2017

Yea I think so. But #467 and #468 are more important IMHO.

@Ladicek

This comment has been minimized.

Show comment
Hide comment
@Ladicek

Ladicek Oct 16, 2017

Contributor

I split this into #820, #821, #822, #823, #824, #825 and #826.

Contributor

Ladicek commented Oct 16, 2017

I split this into #820, #821, #822, #823, #824, #825 and #826.

@Ladicek Ladicek closed this Oct 16, 2017

@lordofthejars lordofthejars added this to the 1.9.3 milestone Nov 29, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment