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

Retrieve the events list for a project #1047

Closed
wants to merge 2 commits into from
Closed

Retrieve the events list for a project #1047

wants to merge 2 commits into from

Conversation

boozec
Copy link
Contributor

@boozec boozec commented Oct 18, 2021

This PR adds a new command:

Manage events on Sentry.

USAGE:
    sentry-cli events [OPTIONS] <SUBCOMMAND>

OPTIONS:
    -h, --help                     Prints help information
        --log-level <LOG_LEVEL>    Set the log output verbosity. [possible values:
                                   trace, debug, info, warn, error]
    -o, --org <ORG>                The organization slug
    -p, --project <PROJECT>        The project slug
    -t, --tags                     Include tags into the list.
    -u, --user                     Include user' info into the list.

SUBCOMMANDS:
    help    Prints this message or the help of the given subcommand(s)
    list    List all events for an project.

Using events -o <org> -p <project> list it shows all the events associated with that project of that organization. Using -t extend the table with the tags, and using -u extend the table with the user informations.

@kamilogorek
Copy link
Contributor

Hey @dcariotti, thanks for the contribution. I'm leaving for a long vacation Today, so I won't be able to merge this right now, but I will definitely get back to this PR once I'm back. Cheers!

@github-actions
Copy link

This pull request has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@kamilogorek
Copy link
Contributor

Hey @dcariotti, I'm currently in the process of releasing 2.x, which brings lots of refactoring into the codebase.
It should be released this week. Once its out, I'll redo the refactoring to your PR and get it into the CLI.

From the functional standpoint, I see some problems though:

  • there's no limit implemented (it'll loop for as long as it queries everything)
  • we need per-page or similar argument to control number of events
  • the same goes for limit to prevent very large outputs/request flood
  • we could add additional info command, that queries details of a single event id (as we do for releases)

@kamilogorek
Copy link
Contributor

I'll close this PR for now, as it has to be completely reworked. Current code does not fit new commands structure, as well as it simply wont work for projects with hundreds of events, as it will loop forever.

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

Successfully merging this pull request may close these issues.

None yet

2 participants