Clean up remaining references to influxdb

RESOLVED FIXED

Status

Release Engineering
Buildduty
P3
normal
RESOLVED FIXED
9 months ago
9 months ago

People

(Reporter: catlee, Assigned: aobreja)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

(Reporter)

Description

9 months ago
https://dxr.mozilla.org/build-central/search?q=influxdb_cre&redirect=false still shows a few places where we're referencing influx, even though we don't use it any more. We should clean up those references to avoid confusion.
Assignee: nobody → aobreja
Created attachment 8876061 [details] [diff] [review]
bb.patch

patch for buildbot-config
Attachment #8876061 - Flags: review?(spacurar)
Created attachment 8876062 [details] [diff] [review]
puppet.patch

Puppet patch to remove influxdb
Attachment #8876062 - Flags: review?(spacurar)
Created attachment 8876063 [details] [diff] [review]
mozharness.patch

For mozharness I deleted entire class InfluxRecordingMixin
This class is imported in (1) and use in class BuildScript,I'm not sure how much impacts if I remove this class,I didn't see function defined in InfluxrecordingMixin to be used in BuildScript.


(1) https://dxr.mozilla.org/build-central/source/mozharness/mozharness/mozilla/building/buildbase.py
Attachment #8876063 - Flags: feedback?(catlee)
Created attachment 8876064 [details] [diff] [review]
relabs-puppet

This patch should delete references from relabs-puppet repository,I see that this repository is not used to much,latest pushes are from 2 mounths,should I delete also the references from here?
Attachment #8876064 - Flags: feedback?(catlee)
Attachment #8876062 - Flags: review?(spacurar) → review+
Attachment #8876061 - Flags: review?(spacurar) → review+
Attachment #8876062 - Flags: checked-in+
Attachment #8876061 - Flags: checked-in+
(Reporter)

Updated

9 months ago
Attachment #8876063 - Flags: feedback?(catlee) → feedback+
(Reporter)

Updated

9 months ago
Attachment #8876064 - Attachment is patch: true
Attachment #8876064 - Attachment mime type: text/x-patch → text/plain
(Reporter)

Updated

9 months ago
Attachment #8876064 - Flags: feedback?(catlee) → feedback+
(Reporter)

Comment 5

9 months ago
(In reply to Andrei Obreja [:aobreja][:buildduty] from comment #4)
> Created attachment 8876064 [details] [diff] [review]
> relabs-puppet
> 
> This patch should delete references from relabs-puppet repository,I see that
> this repository is not used to much,latest pushes are from 2 mounths,should
> I delete also the references from here?

It wouldn't hurt.

Maybe Amy would know if relabs-puppet is still in use.
Attachment #8876063 - Flags: checked-in+
Attachment #8876064 - Flags: checked-in+
Cleaning done
Status: NEW → RESOLVED
Last Resolved: 9 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.