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.

Sign up for GitHub

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

Jump to bottom

Fix copr project name for testing farm and add response url #499

Merged
merged 2 commits into from
Mar 17, 2020
Merged

Fix copr project name for testing farm and add response url #499

merged 2 commits into from
Mar 17, 2020

Conversation

lachmanfrantisek
Copy link
Member

  • Fix the copr-repo-name in testing-farm payload.
  • Add response url to testing-farm payload.

@lachmanfrantisek lachmanfrantisek added the area/testing-farm Related to Testing Farm integration. label Mar 17, 2020
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

lbarcziova
lbarcziova approved these changes Mar 17, 2020
View reviewed changes
Copy link
Member

@lbarcziova lbarcziova left a comment

Choose a reason for hiding this comment

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

LGTM

thrix
thrix approved these changes Mar 17, 2020
View reviewed changes
Copy link
Contributor

@thrix thrix left a comment

Choose a reason for hiding this comment

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

Also lgtm :)

@lachmanfrantisek lachmanfrantisek added the mergeit When set, zuul wil gate and merge the PR. label Mar 17, 2020
@lachmanfrantisek
Add response url to testing-farm payload
d73687f 
Signed-off-by: Frantisek Lachman <flachman@redhat.com>
@lachmanfrantisek
Fix the copr-repo-name in testing-farm payload
6ee4104 
Signed-off-by: Frantisek Lachman <flachman@redhat.com>
@lachmanfrantisek lachmanfrantisek force-pushed the fix-copr-project-name-for-testing-farm-and-add-response-url branch from 6494b4d to 6ee4104 Compare March 17, 2020 11:57
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 508fc8e into packit:master Mar 17, 2020
@lachmanfrantisek lachmanfrantisek deleted the fix-copr-project-name-for-testing-farm-and-add-response-url branch March 17, 2020 13:40
@jpopelka jpopelka mentioned this pull request Mar 17, 2020
Closed
@jpopelka
Copy link
Member

jpopelka commented Mar 17, 2020 •
edited
Loading

Not sure it's related with this PR, but I still see Pending — Tests are running ... in packit-service-testing-organisation/source-git-rpm#1 even after prod was redeployed with this PR.

EDIT: It's #504

@lachmanfrantisek
Copy link
Member Author

Not sure it's related with this PR, but I still see Pending — Tests are running ... in packit-service-testing-organisation/source-git-rpm#1 even after prod was redeployed with this PR.

EDIT: It's #504

Not related, but thanks for checking that. The fix is in #505.

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

@thrix thrix thrix approved these changes

@lbarcziova lbarcziova lbarcziova approved these changes

Assignees
No one assigned
Labels
area/testing-farm Related to Testing Farm integration. mergeit When set, zuul wil gate and merge the PR.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@lachmanfrantisek @jpopelka @thrix @lbarcziova