-
Notifications
You must be signed in to change notification settings - Fork 356
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
[HAMMER] Dropbox removed, in exchange for using support tool #6340
[HAMMER] Dropbox removed, in exchange for using support tool #6340
Conversation
cc @simaishi |
cc @juliancheal |
Soo... what this change does is allow users to edit their existing Red Hat Dropbox log settings. |
@himdel Ah sorry for the confusion. We were doing two things with one PR. RH Dropbox has been removed from downstream, and replaced with the file depo, with the settings for that not being editable. :) |
Ah, thanks, ok, as long as the backend is dealing with this.. :) 👍 (we probably should fix those remaining mentions in master, but that's for another pr :)) |
Hold off on merging, I'm pushing one other change to match the changes in #6341 |
Make both red hat dropbox and support tool read-only https://bugzilla.redhat.com/show_bug.cgi?id=1765635
f7dc107
to
855d7a1
Compare
Checked commit jrafanie@855d7a1 with ruby 2.4.6, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0 app/views/ops/_log_collection.html.haml
|
Manual backport of #6330 and #6341
Make both red hat dropbox and support tool read-only
https://bugzilla.redhat.com/show_bug.cgi?id=1765635
Note, there's still test files referencing the changed dropbox strings but #6330 passed tests and didn't include them. If needed, we can do another upstream PR for master/hammer to get them: