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

[enterprise-3.11] Fix bug where cannot create new role and new rolebinding in single operation using template #3116

Merged
merged 1 commit into from
Feb 26, 2019

Conversation

rhamilto
Copy link
Member

@rhamilto rhamilto commented Feb 6, 2019

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1673109

Big assist from @spadgett on this one. Thank you!

@openshift-ci-robot openshift-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 6, 2019
@rhamilto rhamilto changed the title [WIP] Fix bug where cannot create new role and new rolebinding in single operation using template Fix bug where cannot create new role and new rolebinding in single operation using template Feb 7, 2019
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 7, 2019
app/scripts/directives/processTemplate.js Outdated Show resolved Hide resolved
app/scripts/directives/processTemplate.js Outdated Show resolved Hide resolved
Copy link
Member

@spadgett spadgett left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 7, 2019
@rhamilto
Copy link
Member Author

rhamilto commented Feb 7, 2019

/test

@spadgett
Copy link
Member

spadgett commented Feb 7, 2019

/retest

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 7, 2019
@spadgett
Copy link
Member

spadgett commented Feb 7, 2019

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 7, 2019
@rhamilto
Copy link
Member Author

/retest

1 similar comment
@rhamilto
Copy link
Member Author

/retest

@openshift-ci-robot openshift-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 18, 2019
@rhamilto
Copy link
Member Author

/test

let routesPage = new RoutesPage(project);
routesPage.visit();
expect(element(by.cssContainingText('td', 'nodejs-mongodb-example')).isPresent()).toBe(true); // TODO: use fixture
return browser.sleep(3000).then(() => {
Copy link
Member Author

Choose a reason for hiding this comment

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

@spadgett, I took a guess this would resolve the failing test issue. Mind taking a look?

Copy link
Member

Choose a reason for hiding this comment

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

This seems reasonable. Ideally we would wait for the page to update instead of an arbitrary sleep, but I think it's OK if it unblocks CI.

Copy link
Member

@spadgett spadgett left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 26, 2019
@openshift-merge-robot openshift-merge-robot merged commit 4df3ce8 into openshift:enterprise-3.11 Feb 26, 2019
@spadgett spadgett changed the title Fix bug where cannot create new role and new rolebinding in single operation using template [enterprise-3.11] Fix bug where cannot create new role and new rolebinding in single operation using template Feb 26, 2019
@rhamilto rhamilto deleted the bz1673109 branch February 27, 2019 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants