#2072 make sure that graphs actually generate, remove old files
Merged 2 years ago by kevin. Opened 2 years ago by smooge.
fedora-infra/ smooge/ansible fix_webgraph  into  main

@@ -363,18 +363,6 @@ 

       '/var/www/html/csv-reports/mirrors/mirrorsdata-all.csv' using 1:($33) title 'rawhide' w filledcurves x1 lc rgb "#FF0000"

  unset output

  

- set output "/var/www/html/csv-reports/images/fedora-rev-latest-stacked.png"

- set title "Fedora Yum Unique IPs"

- plot ["2018-01-01":"2024-12-31"] \

-      '/var/www/html/csv-reports/mirrors/mirrorsdata-all.csv' using 1:36 title 'Fedora' with lines lw 3,\

-      '/var/www/html/csv-reports/mirrors/mirrorsdata-all.csv' using 1:($33) title 'rawhide' with filledcurves x1

-      '/var/www/html/csv-reports/mirrors/mirrorsdata-all.csv' using 1:($33+$79) title 'fed39' with filledcurves x1,\

-      '/var/www/html/csv-reports/mirrors/mirrorsdata-all.csv' using 1:($33+$79+$78) title 'fed38' with filledcurves x1,\

-      '/var/www/html/csv-reports/mirrors/mirrorsdata-all.csv' using 1:($33+$79+$78+$77) title 'fed37' with filledcurves x1,\

-      '/var/www/html/csv-reports/mirrors/mirrorsdata-all.csv' using 1:($33+$79+$78+$77+$76) title 'fed36' with filledcurves x1,\

- unset output

- 

- 

  set output "/var/www/html/csv-reports/images/fedora-select-stacked.png"

  set title "Fedora Yum Unique IPs"

  plot ["2007-05-17":"2024-12-31"] \

@@ -54,6 +54,11 @@ 

    tags:

      - web-data

  

+ - name: clean out non-useful images

+   ansible.builtin.file: path=/var/www/html/csv-reports/images/{{item}}

+   state: absent

+   with_items: [ hotspot-all.png, fedora-rev-latest-stacked.png ]

+ 

  - name: scripts to condense data down for further processing

    copy: src={{item}} dest=/usr/local/bin/ mode=0755

    with_items: [condense-mirrorlogs.sh]

This removes files which are no longer generated
It also cleans up a badly generated image which was stopping other pictures from generating.

Signed-off-by: Stephen Smoogen ssmoogen@redhat.com

Yeh, seems like a typo. Not sure what an unknown state does, FWIW here are the valid ones: https://docs.ansible.com/ansible/latest/collections/ansible/builtin/file_module.html#parameter-state

@smooge Would it help if someone else fixed it up?

1 new commit added

  • Fix spelling erorr in playbook
2 years ago

1 new commit added

  • Remove tabs and trailing whitespace.
2 years ago

rebased onto decfe3e

2 years ago

rebased onto decfe3e

2 years ago

Pull-Request has been merged by kevin

2 years ago