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

Idempotency check fixes #178

Merged

Conversation

juliogonzalez
Copy link
Member

@juliogonzalez juliogonzalez commented Oct 4, 2018

What does this PR change?

  • When testing a PR, use files from the PR, instead of testing since a fixed version (that was failing as the path for the Gitarro JSON file was not correct)
  • Check for trialing backslash at schema-dir argument
  • Remove unneeded checks for arguments with default value
  • Improve messaging

GUI diff

No difference.

  • DONE

Documentation

  • No documentation needed: Internal fix

  • DONE

Test coverage

  • No tests: Fixing tests

  • DONE

Links

Fixes #
Tracks # add downstream PR, if any

  • DONE

@juliogonzalez
Copy link
Member Author

gitarro no changelog needed !!!

@juliogonzalez juliogonzalez force-pushed the idempotency-pr-fix branch 29 times, most recently from 6cdc1e3 to 74c9c2c Compare October 4, 2018 14:41
@juliogonzalez juliogonzalez changed the title WIP: Idempotency fix: when testing a PR, use files from the PR WIP: Idempotency fixes Oct 4, 2018
@juliogonzalez juliogonzalez changed the title WIP: Idempotency fixes WIP: Idempotency check fixes Oct 4, 2018
arguments with default value, remove trialing backslases from schema-dir and improve messaging
@juliogonzalez juliogonzalez changed the title WIP: Idempotency check fixes Idempotency check fixes Oct 4, 2018
@juliogonzalez
Copy link
Member Author

Waiting for the tests, and then porting to the PRs for internal branches.

@juliogonzalez juliogonzalez merged commit 3c5aa74 into uyuni-project:master Oct 4, 2018
@juliogonzalez juliogonzalez deleted the idempotency-pr-fix branch October 4, 2018 15:02
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.

2 participants