Closed
Bug 871078
Opened 12 years ago
Closed 12 years ago
inventory cvs importer documentation incorrect
Categories
(Infrastructure & Operations :: Infrastructure: Other, task)
Infrastructure & Operations
Infrastructure: Other
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: u429623, Assigned: juber)
Details
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•12 years ago
|
Assignee: server-ops-infra → juber
Assignee | ||
Comment 1•12 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•12 years ago
|
||
This pull request was merged and deployed. Reopen if there are still issues.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
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.
Description
•