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

Run script in command line (without redis) #6529

Closed
DanSheps opened this issue Jun 3, 2021 · 7 comments · Fixed by #7677
Closed

Run script in command line (without redis) #6529

DanSheps opened this issue Jun 3, 2021 · 7 comments · Fixed by #7677
Assignees
@DanSheps
Labels
status: accepted This issue has been accepted for implementation type: housekeeping Changes to the application which do not directly impact the end user

Comments

@DanSheps
Copy link
Member

DanSheps commented Jun 3, 2021

Proposed Changes

Add command to run script in command line without redis.

Justification

Useful for testing, developing and debugging scripts
@DanSheps DanSheps added type: housekeeping Changes to the application which do not directly impact the end user status: accepted This issue has been accepted for implementation labels Jun 3, 2021
@DanSheps DanSheps self-assigned this Jun 3, 2021
@DanSheps
Copy link
Member Author

DanSheps commented Jun 3, 2021

Was discussed previously with the maintainers and just forgot to add in a issue for it.

DanSheps added a commit that referenced this issue Jun 3, 2021
@DanSheps
#6529 - Preiminary work on runscript command
e606709
@jeremystretch jeremystretch added status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation and removed status: accepted This issue has been accepted for implementation labels Jul 1, 2021
@github-actions
Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. NetBox is governed by a small group of core maintainers which means not all opened issues may receive direct feedback. Please see our contributing guide.

@github-actions github-actions bot added the pending closure Requires immediate attention to avoid being closed for inactivity label Aug 31, 2021
@DanSheps
Copy link
Member Author

I am going to get back on track with this. Almost have a command wrapper for this I believe done (using it locally)

@DanSheps DanSheps added status: accepted This issue has been accepted for implementation and removed status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation pending closure Requires immediate attention to avoid being closed for inactivity labels Sep 28, 2021
@DanSheps DanSheps self-assigned this Sep 28, 2021
DanSheps added a commit that referenced this issue Oct 28, 2021
@DanSheps
#6529 - Add CLI to run scripts
0a62f75
@DanSheps DanSheps mentioned this issue Oct 28, 2021
Merged
@WillIrvine
Copy link
Contributor

Looks good Dan, will this also work with reports?

@jeremystretch
Copy link
Member

@WillIrvine There's a runreport management command already.

@WillIrvine
Copy link
Contributor

@jeremystretch That still runs the report via redis doesn't it?

@jeremystretch
Copy link
Member

Ah yeah, I think it does.

DanSheps added a commit that referenced this issue Nov 2, 2021
@DanSheps
#6529 - Streamline code and resolve some issues
b7c0e8b
DanSheps added a commit that referenced this issue Nov 2, 2021
@DanSheps
#6529 - Adjusted the arguments. Added documentation
7c3318d
DanSheps added a commit that referenced this issue Nov 3, 2021
@DanSheps
#6529 - Adjusted the arguments. Fixed documentation
19bacc9
jeremystretch added a commit that referenced this issue Nov 3, 2021
@jeremystretch
Merge pull request #7677 from netbox-community/6529-command-line-run-…
2e20d7f 
…scripts

#6529 - Add CLI to run scripts
jeremystretch added a commit that referenced this issue Nov 3, 2021
@jeremystretch
Changelog for #6529
7085fe7
@jeremystretch jeremystretch mentioned this issue Nov 3, 2021
Merged
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@DanSheps DanSheps

Labels
status: accepted This issue has been accepted for implementation type: housekeeping Changes to the application which do not directly impact the end user
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#6529 - Add CLI to run scripts netbox-community/netbox
3 participants
@DanSheps @jeremystretch @WillIrvine