Lucas_Werkmeister_WMDE removed Lucas_Werkmeister_WMDE as the assignee of this 
task.
Lucas_Werkmeister_WMDE moved this task from In Development to Ready for Peer 
Review on the Wikidata Dev Team (Wikidata.org Slice) board.
Lucas_Werkmeister_WMDE added a comment.


  The clone path seems to resolve to `/srv/analytics-wmde/wdcm/src/`:
  
    lucaswerkmeister-wmde@stat1007:~$ sudo -u analytics-wmde git -C 
/srv/analytics-wmde/wdcm/src/ remote -v
    origin      https://gerrit.wikimedia.org/r/analytics/wmde/WDCM (fetch)
    origin      https://gerrit.wikimedia.org/r/analytics/wmde/WDCM (push)
    lucaswerkmeister-wmde@stat1007:~$ sudo -u analytics-wmde git -C 
/srv/analytics-wmde/wdcm/src/ show -U0
    commit aa40182c322410584e91b93b9327adc017a218e9 (HEAD -> master, 
origin/master, origin/HEAD)
    Author: GoranMilovanovic <[email protected]>
    Date:   Thu Feb 8 20:08:09 2018 +0100
    
        Minor
        
        Change-Id: I5329a17b7b82b88958945f90b804195838e536b7
    
    diff --git a/wdcmStructure_Update.R b/wdcmStructure_Update.R
    index dcf4b79..1d2b102 100644
    --- a/wdcmStructure_Update.R
    +++ b/wdcmStructure_Update.R
    @@ -291,0 +292,9 @@ write.csv(myWD$counts, "wdcmStructure_Counts.csv")
    +### --- updateReport File
    +updateReport <- as.character(Sys.time())
    +upY <- substr(updateReport, 1, 4)
    +upM <- as.numeric(substr(updateReport, 6, 7))
    +upM <- month.name[upM]
    +upD <- substr(updateReport, 9, 10)
    +updateReport <- paste0(upY, " ", upM, " ", upD)
    +write(updateReport, "updateReport.txt")
    +
  
  Note that this commit is 23 commits behind the actually current master, if 
I’m not mistaken. But given that the repository thinks it’s up to date with 
`origin/master`, it seems the last `git fetch` was somewhere between February 
2018 <https://gerrit.wikimedia.org/r/c/analytics/wmde/WDCM/+/409117> (commit 
shown above) and June 2018 
<https://gerrit.wikimedia.org/r/c/analytics/wmde/WDCM/+/440112> (first commit 
after that). I assume that’s because the Puppet spec has `ensure => 'present'` 
for the git clone, rather than `ensure => 'latest'` as seen in e.g. 
`graphite.pp` (which clones the analytics/wmde/scripts repo); the TODO below 
would agree with that:
  
    git::clone { 'analytics/wmde/WDCM':
        # TODO do we want a similar latest & production branch here? Or just 
manually pulling? scap?
        # Currently when we update the code in the repo we will have to pull 
the updates ourselves.
        ensure    => 'present',
  
  Given that one of the missing commits is called Security patch 
<https://gerrit.wikimedia.org/r/c/analytics/wmde/WDCM/+/441750>, I sure hope 
this outdated code never actually runs…
  
  None of the systemd timers seem to do anything with WDCM:
  
    lucaswerkmeister-wmde@stat1007:~$ systemctl cat -- $(systemctl list-timers 
--no-legend --full | awk '{print $NF}') | grep -i wdcm | wc -l
    0
  
  (`awk {print $NF}` selects the last field of the `systemctl list-timers` 
output, which is the name of the activated service, so this searches through 
all the timer-activated services.)
  
  Unfortunately, it looks like I can’t check `analytics-wmde`’s crontab:
  
    lucaswerkmeister-wmde@stat1007:~$ sudo -u analytics-wmde crontab -l
    crontabs/analytics-wmde/: fopen: Permission denied
  
  
  
  > Also, all the related cron-jobs that we know about 
<https://docs.google.com/spreadsheets/d/1w2f_ndQa6Lo2BBfPJ88sJLSg2RJeTQKFNOPd0zjiB4I/edit#gid=0>
 seem to run against a personal user directory, not something related to the 
`analytics-wmde` user?
  
  If we trust that this spreadsheet is complete, I think it’s almost certain 
that this repository isn’t used (now that we know what its path is, and can see 
that the path doesn’t show up in the spreadsheet); otherwise I guess we could 
ask someone with more permissions to check some more crontabs?
  
  (The other analytics clients 
<https://wikitech.wikimedia.org/wiki/Analytics/Systems/Clients> – stat1004, 
stat1005, stat1006, stat1008, stat1009 – don’t have a `/srv/analytics-wmde` 
directory at all, so I think this repository is limited to stat1007.)

TASK DETAIL
  https://phabricator.wikimedia.org/T350252

WORKBOARD
  https://phabricator.wikimedia.org/project/board/6751/

EMAIL PREFERENCES
  https://phabricator.wikimedia.org/settings/panel/emailpreferences/

To: Lucas_Werkmeister_WMDE
Cc: Lucas_Werkmeister_WMDE, Manuel, AndrewTavis_WMDE, Aklapper, Michael, 
Danny_Benjafield_WMDE, Astuthiodit_1, karapayneWMDE, Invadibot, maantietaja, 
ItamarWMDE, Akuckartz, Nandana, Lahi, Gq86, GoranSMilovanovic, QZanden, 
LawExplorer, _jensen, rosalieper, Scott_WUaS, Wikidata-bugs, aude, Mbch331
_______________________________________________
Wikidata-bugs mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to