#110 Fix issue when Github Full Name is unknown
Closed by sidpremkumar. Opened by sidpremkumar.
sidpremkumar/sync-to-jira github-unknown-user-name-fix  into  develop

Download 110.patch

Issue when Github user did not provide their full name, we would get None under reporter/comment user/etc. This PR attempts to fix that.

rebased onto 1bc43b15a9dccfe840a3daca4a32e9c68a1e44f1

FYI, you can do this in shorthand with 'author': comment.user.name or comment.user.login,

Same here.

One suggestion above, otherwise :+1:.

rebased onto e7feb7d3616984d48745e39113e2b13f1268e383

rebased onto 167d7ffd291e680ab4585bf91f06f5c77caf8991

153 new commits added

  • Remove converting markdown by default, now optional flag that needs to be placed in the updates array
  • Add comment when attempting to transition to CUSTOM_STATE
  • Adding converter to change github markdown to plaintext
  • Fixing cutstom labels downstream feature and updating README
  • Adding tests to cover new functions added in PR#97
  • Updating docs to reflect 'develop' param in config
  • Only send failure email if we are not developing
  • Add support to perform inital JIRA query to check status of JIRA server
  • Add 75% min coverage and added tests accordingly
  • Adding min coverage to tox
  • Updating downstream tests and some elements in main.py to be complient with flake8
  • Fixing small issues in qa-contact
  • Adding support for email to be sent with traceback in case of failure
  • Fixing change status doc string
  • Adding QA contact field support
  • Refactored status transition and updated README
  • Added configuration for custom ticket status
  • Adding check for JIRA issue fields before checking
  • Adding check for JIRA issue fields before checking
  • Editing readme to link to documentation
  • Add ReadTheDocs documentation
  • Adding overwrite field to assignee
  • Removing hours from comments, and removing hours from status
  • Configuring tests to work with new commet format
  • Changing year to date in comment
  • Changing from utc epoch, to just from epoch
  • Add more accurate logs to update title
  • Adding regex and upstream reporter to title update
  • adding regex matches to all description updates
  • adding another catch for updating descrtiption
  • Adding jinja2 to DockerFile
  • Removing mailer class and replacing with static function
  • adding admin support for emails
  • Mailer gets config values from enviormental variables
  • Adding support to email owners in the situation that duplicate issues are created
  • Adding catch for PR comment
  • adding more tests for username and comment functions
  • Nevermind. Unnecessary.
  • Try fedora 29.
  • condenced function to find recently updated issue in matching
  • Removing hard-coded user in comment parsing, now pull from config dict
  • issue matching will always return a value now
  • adding appropriate test cases
  • Added title matching for Pagure
  • Fixing issue when ID is interpreted as String
  • Adding more checks for matching issues
  • added duplicate issue comment catching
  • Fixed issue where user has no updates array
  • PyGithub is a dep.
  • Reset the UPDATE_DATE.
  • Ignore xunit-tests.
  • changing xunit-tests from json to xml
  • Drop py35.
  • Bring back nose as a dep.
  • Make sure tox is available in the slave.
  • Fix up dependencies in the specfile.
  • No need to test on python2 any more.
  • Remove python2 sub-package. No longer needed.
  • requests is actually okay at any version.
  • Pull specfile from repo.
  • changing test case command in template
  • Add more sync features
  • Make sure to copy the specfile into the Dockerfile.
  • Added support for custom fields and custom ticket type for repository
  • Handle one more transition for RHV.
  • Handle this second exception.
  • Try closing issues with a specific resolution.
  • Slightly more flexible duplicate handling.
  • Apply filters for comment messages.
  • Github pull requests are on the same topic as issues. Distinguish!
  • Only synchronize issues when projects opt-in.
  • Simpler logging configuration.
  • Update default jira connection config to match code.
  • Refactor and simplify broken code block.
  • New log statement.
  • Flake8.
  • Remove unnecessary client.
  • Added tests for comments sync
  • Made jira and github/pagure comments intersection more understandable, return logging level to previous value and remove print
  • Bring mocks in line with latest comment sync changes.
  • PEP8 futzing.
  • Remove print statement.
  • Made jira and github/pagure comments intersection more understandable, return logging level to previous value and remove print
  • Added support comment sync for pagure
  • Split line in _comment_matching
  • Add comments synchronization for Github
  • Remove pylint.
  • Add flake8.
  • downstream: Add ability to use different jira instances
  • Some notes from working with Bohdan Iakymets
  • Optionally listen to the message bus.
  • Remove old ansible config.
  • Need git.
  • Pass the argument all the way through.
  • Try setting this ENV explicitly.
  • Correct deps.
  • Correct path for pip3.
  • Verbose, please.
  • No need for this anymore.
  • Ensure command is present, and leave some other content around for debugging.
  • Pylint.
  • Try deleting the bc a different way.
  • Restore flake checks.
  • Get container build working.
  • Track latest fedora.
  • Fix xunit declaration.
  • Ignore rpmbuild output.
  • Use specfile for builddeps.
  • rpmbuild tools.
  • Pull in the spec file.
  • Use f28.
  • Install builddeps.
  • Add back missing variables for usernames to work.
  • nosetests
  • Version.sh.
  • Use older syntax to work with default jenkins master.
  • Revert "Restore docs building even thought it doesn't make sense for now."
  • Restore docs building even thought it doesn't make sense for now.
  • Only quay, thank you.
  • Messing with openshift.
  • Add in syncing for coreos/ignition
  • Add upstream fedora CoreOS repo syncing
  • Add "cl" label to issues imported from coreos/bugs
  • Handle various pagure message formats.
  • Typofix.
  • add syncing for openshift/os repo
  • Make "sst_coreosct" and "upstream" different labels
  • Rename estuary_updater to estuary-updater
  • Enable syncing issues for the Factory estuary_updater project
  • add upstream_cl and upstream_atomic labels
  • Atomic Teams now use COREOS project.
  • More labels For COREOS issue syncing
  • Component name: estuary.
  • Added the CoreOS bugs Repo and a JIRA label to use on this repo.
  • New bodhi milestone to track.
  • Set a component for finishline.
  • Enable syncing issues for ralphbean/finishline
  • Also, purview-api.
  • Add purview.
  • Remove botas.
  • Add botas.
  • Turn off BPO syncing.
  • Ignore deploy.retry.
  • Components for a few FACTORY projects.
  • Drop appstream.
  • Update JIRA label for fedora-ci/AtomicCi
  • sync2jira.py: Change github atomic label to sst_atomic_host
  • sync2jira.py: Change pagure atomic label to sst_atomic_host
  • Clean up pagure tag extraction.
  • Careful, now.
  • Seems that this needs to be status instead of state.
  • Add the upstream label back to atomic items.
  • Merge branch 'release/1.7'

rebased onto 8ada93e0a01771440362751a87fc6714b4571cdc

Accidently rebased on master :100: going to close this and open another PR, whoops

Pull-Request has been closed by sidpremkumar

Metadata