-
Notifications
You must be signed in to change notification settings - Fork 932
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
1a154ab
commit 568161c
Showing
2 changed files
with
94 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,91 @@ | ||
# Writing good issues | ||
If you're experiencing a problem with Oxidized or need a new feature, you can | ||
[submit an issue on github](https://github.com/ytti/oxidized/issues). We have | ||
a great community where users help each other through the issue system. | ||
|
||
This guide provides tips on writing your issue to make it easier for the | ||
community and developers to understand and respond effectively. | ||
|
||
Why write good issues? | ||
- A clear and detailed issue improves the chances of getting your problem resolved. | ||
- By spending time to write a good issue, you save developers time, contributing | ||
to Oxidized’s progress without writing a line of code. | ||
|
||
## Submit to the correct project | ||
Choose the appropriate GitHub project based on your issue: | ||
|
||
- For issues with the web frontend or REST API, go to | ||
[oxidized-web](https://github.com/ytti/oxidized-web/). | ||
- For issues with oxidized-script, use | ||
[oxidized-script](https://github.com/ytti/oxidized-script). (note: as of | ||
November 2024, oxidized-script is not actively maintained). | ||
- For issues with third-party software relying on Oxidized, open an issue in | ||
that specific project. | ||
- For issues with Oxidized itself, go to | ||
[oxidized](https://github.com/ytti/oxidized). | ||
|
||
## Format your issue | ||
- Use [GitHub Markdown](https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax) to format your issue. | ||
- Preview your text before submitting to ensure it renders correctly. | ||
- Avoid screenshots of text. Instead, use [code formating](https://docs.github.com/en/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax#quoting-code) for any relevant code snippets. | ||
|
||
## Choose your title well | ||
Keep the title brief yet descriptive. Aim to summarize the main issue or request in a few words. | ||
|
||
## Provide detailled informations | ||
Include as many relevant details as possible. At a minimum, specify: | ||
|
||
- Oxidized version and operating system. | ||
- Relevant parts of your Oxidized configuration and a brief explanation of your setup. | ||
- Output of the error, if relevant. | ||
- For issues related to specific devices, consider creating a YAML Simulation file (instructions below). | ||
|
||
Also, provide clear steps to reproduce the issue, if applicable. | ||
|
||
## Making feature requests | ||
Feature requests are welcome, but please understand that unaddressed requests | ||
may be closed after some time. If you need a feature urgently, consider | ||
contributing code via a pull request (PR) or hiring a developer. | ||
|
||
## Sumbit a YAML Simulation File | ||
To help developers troubleshoot device-specific issues, you may be asked to submit a | ||
[YAML simulation file](https://github.com/ytti/oxidized/blob/master/examples/device-simulation/README.md#creating-a-yaml-file-with-device2yamlrb) for your device. | ||
|
||
Here's a brief overview how to do it, you can find more details in the link | ||
above. | ||
- Fork Oxidized on github | ||
- Install dependencies (git and Ruby's Net::SSH): | ||
``` | ||
# Adapt when not using a debian-based distro | ||
sudo apt install git ruby-net-ssh | ||
``` | ||
- Clone your forked Oxidized repository: | ||
``` | ||
git clone [email protected]:<your github user>/oxidized.git | ||
``` | ||
- run the device2yaml.rb script (you’ll be provided with the command set and | ||
output filename to use) | ||
``` | ||
cd oxidized/examples/device-simulation | ||
# Replace user and devicename to appropriate values | ||
./device2yaml.rb user@devicename -c cmdsets/ios -o yaml/asr900_26.8.1b.yaml | ||
``` | ||
- The script waits 5 seconds between commands, and outputs the response of the | ||
device. You can press "ESC" if you see the prompt and want to pass to next | ||
command without waiting for the timeout. | ||
- The result will be stored in `oxidized/examples/device-simulation/yaml/`. | ||
- Replace any sensitive information with placeholder values in the output file. | ||
- Commit & push the file to github | ||
``` | ||
git add yaml/asr900_26.8.1b.yaml | ||
git commit -m "Device simulation for ASR900" | ||
git push | ||
``` | ||
- Create a pull request (PR) in GitHub, referencing the issue number (e.g., | ||
"YAML simulation file for issue #1234"). | ||
|
||
|
||
|
||
|
||
|
||
|