Opened 7 years ago

Closed 6 years ago

#2708 closed defect (duplicate)

[kean] b10-cmdctl-usermgr should default to installed accounts file

Reported by: jelte Owned by: UnAssigned
Priority: high Milestone: Sprint-20131015
Component: ~cmd-ctl (obsolete) Version:
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: Medium
Sub-Project: DNS Feature Depending on Ticket:
Estimated Difficulty: 2 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description

By default, b10-cmdctl-usermgr will use (or create) a file in the current working directory.

I propose we make it default to the same file as b10-cmdctl defaults to.

We should probably also print the complete file path at the start.

Subtickets

Change History (9)

comment:1 Changed 7 years ago by shane

  • Defect Severity changed from N/A to Medium
  • Milestone changed from New Tasks to Next-Sprint-Proposed

Agreed, the current behavior is non-optimal and confusing.

comment:2 Changed 7 years ago by shane

  • Milestone changed from Previous-Sprint-Proposed to Next-Sprint-Proposed

comment:3 Changed 7 years ago by shane

  • Milestone changed from Previous-Sprint-Proposed to Next-Sprint-Proposed

comment:4 Changed 7 years ago by shane

  • Milestone changed from Previous-Sprint-Proposed to Next-Sprint-Proposed
  • Priority changed from medium to high

comment:5 Changed 6 years ago by muks

  • Milestone set to Sprint-20130903
  • Summary changed from b10-cmdctl-usermgr should default to installed accounts file to [kean] b10-cmdctl-usermgr should default to installed accounts file

comment:6 Changed 6 years ago by kean

  • Owner set to kean
  • Status changed from new to accepted

comment:7 Changed 6 years ago by kean

  • Owner changed from kean to UnAssigned
  • Status changed from accepted to reviewing

This bug seems to no longer be valid. There is clearly code in the user manager to use the default location, and a simple test verifies that this is indeed the case:

$ pwd
/home/kean/isc/B10/bin
$ ../sbin/b10-cmdctl-usermgr add kean foobar
Using accounts file: /home/kean/isc/B10/etc/bind10/cmdctl-accounts.csv
$ ls -l cmdctl-accounts*
ls: cannot access cmdctl-accounts*: No such file or directory
$ ls -l ../etc/bind10/cmdctl-accounts*
-rw-rw-r--. 1 kean kean 112 Nov 18 06:45 ../etc/bind10/cmdctl-accounts.csv

comment:8 Changed 6 years ago by jreed

Fixed in Trac #2713, git 9925af3b3f4daa47ba8c2eb66f556b01ed6f0502. Feel free to close this.

comment:9 Changed 6 years ago by kean

  • Resolution set to duplicate
  • Status changed from reviewing to closed
Note: See TracTickets for help on using tickets.