inventory cvs importer documentation incorrect

RESOLVED FIXED

Status

--
minor
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: hwine, Assigned: juber)

Tracking

Details

(Reporter)

Description

6 years ago
A couple of slight corrections needed there. Some that gave me some challenges sorting things out include:

 - docs show header filters with surrounding white space. e.g.:
    hostname, systemrack % id
    foobaz.mozilla.com, 8 
   wouldn't work - the header needs to be "systemrack %id"

 - it would be great to note that it is "okay" to find attribute "id" values by looking for the "value" attribute value in the html form. (by "okay" I mean stable - I believe the value used is the database key for that text)

If the doc examples can be copy/pasted into the web form, I think that will be a great "tutorial" for using the system. The preview is awesome, and once I got past the above issues, it's been smooth sailing.
(Assignee)

Updated

6 years ago
Assignee: server-ops-infra → juber
(Assignee)

Comment 1

6 years ago
Indeed there was a bug with white space and header values, and I think I have a patch that fixes it.

I do mention that 'id' can be used to specify a related object. It's in the paragraph above http://inventory.mozilla.org/en-US/csv/format/#Meta-Attributes .

I took your advice and tried copying every example into the web form. This lead to other bugs revealing themselves, so I took some time to fix them and added some new features along the way.

I plan on pushing these changes some time today. 
https://github.com/mozilla/inventory/pull/20
Status: NEW → ASSIGNED
(Assignee)

Comment 2

6 years ago
This pull request was merged and deployed. Reopen if there are still issues.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Component: Server Operations: Infrastructure → Infrastructure: Other
Product: mozilla.org → Infrastructure & Operations
You need to log in before you can comment on or make changes to this bug.