search

ceph / pulpito

A dashboard for Ceph tests
23 stars 29 forks source link

scrape button added #88

Open hkaur008 opened 3 years ago

hkaur008 commented 3 years ago

Fixes: https://tracker.ceph.com/issues/53048

Scrape button added #83 Signed-off-by: Hargun Kaur hargunalone@gmail.com

Checklist

Show available Jenkins commands - `jenkins retest this please` - `jenkins test classic perf` - `jenkins test crimson perf` - `jenkins test signed` - `jenkins test make check` - `jenkins test make check arm64` - `jenkins test submodules` - `jenkins test dashboard` - `jenkins test dashboard cephadm` - `jenkins test api` - `jenkins test docs` - `jenkins render docs` - `jenkins test ceph-volume all` - `jenkins test ceph-volume tox`
hkaur008 commented 3 years ago

@ideepika can you check?

hkaur008 commented 3 years ago

Cool

On Thu, 14 Oct, 2021, 4:29 PM Deepika Upadhyay, @.***> wrote:

@.**** commented on this pull request.

In pulpito/templates/run.html https://github.com/ceph/pulpito/pull/88#discussion_r728869658:

@@ -80,6 +80,8 @@

@hkaur008 https://github.com/hkaur008 the url I passed was an example and what I was intending to say that we need to url link based on the job for pulpito examination please inspect https://pulpito.ceph.com developer tools and see Log file,

job.log_href looks like: http://qa-proxy.ceph.com/teuthology/ideepika-2021-10-14_07:14:29-rbd:iscsi-wip-nizamial09-testing-2021-10-11-1540-distro-default-smithi/6442773/teuthology.log you can achieve something like that using scrape_log = " http://qa-proxy.ceph.com/{{ job['name'] }}/scrape.log"

place to look at: pulpito/templates/run_jobs_table.html

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ceph/pulpito/pull/88#pullrequestreview-779613531, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANOWNVAOO6PJKVCCZEEY6JDUG2Z2XANCNFSM5F4FMOTA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

kshtsk commented 3 years ago

If scrape is something that runs at the end of the whole run finished, maybe make the link visible when the scrape output close to be available, I mean not from the very begging of job.

That does not neglect that it should not be a button from job list filter group, maybe just another reference above the list?

ideepika commented 3 years ago

@hkaur008 let us know if you need any help, thanks!

ljflores commented 3 years ago

I created a Tracker associated with this PR: https://tracker.ceph.com/issues/53048. @hkaur008 would you please add this to your PR description and to your commit?

You can edit your PR description by clicking the three dots next to the emoticon and clicking "Edit." You can edit your commit by typing git commit --amend in your terminal and following the prompts. Add this line to both: Fixes: https://tracker.ceph.com/issues/53048.

Let us know if you need any help!

hkaur008 commented 3 years ago

Sure

On Tue, 26 Oct, 2021, 10:52 PM Laura Flores, @.***> wrote:

I created a Tracker associated with this PR: https://tracker.ceph.com/issues/53048. @hkaur008 https://github.com/hkaur008 would you please add this to your PR description and to your commit?

You can edit your PR description by clicking the three dots next to the emoticon and clicking "Edit." You can edit your commit by typing git commit --amend in your terminal and following the prompts. Add this line to both: Fixes: https://tracker.ceph.com/issues/53048.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ceph/pulpito/pull/88#issuecomment-952151201, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANOWNVAXCBY7VXDUTSIKUJTUI3PWTANCNFSM5F4FMOTA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

hkaur008 commented 2 years ago

Sorry, I got busy with my exams will start working on this pull request again!

hkaur008 commented 2 years ago

image Whenever i try to run it I get in this error , any leads would be helpful

hkaur008 commented 2 years ago

@ljflores I have added the code, Can you have a look? If things are okay!

ljflores commented 2 years ago

Thanks @hkaur008! I'll see if @kamoltat can take a look.

hkaur008 commented 2 years ago

Cool

On Tue, 4 Jan, 2022, 4:15 AM Laura Flores, @.***> wrote:

Thanks @hkaur008 https://github.com/hkaur008! I'll see if @kamoltat https://github.com/kamoltat can take a look.

— Reply to this email directly, view it on GitHub https://github.com/ceph/pulpito/pull/88#issuecomment-1004407630, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANOWNVG3WGSO6NMH4RC3OLDUUIRHNANCNFSM5F4FMOTA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you were mentioned.Message ID: @.***>