Warning: Can't synchronize with repository "(default)" (Unsupported version control system "svn": No module named svn). Look in the Trac log for more information.

Ticket #1218 (closed defect: fixed)

Opened 11 years ago

Last modified 9 years ago

tg-admin: add option to quickstart project without identity without prompting

Reported by: jeffk Owned by: faide
Priority: normal Milestone: 1.1
Component: tg-admin (non-toolbox) Version: 1.0.4b3
Severity: normal Keywords: quickstart nosetests tg-admin options
Cc:

Description

It is possible that my system and dependency configuration is overly sensitive, described in #1214, but it would seem that a good test would be to run tg-admin quickstart with various combination of options, change into the directory, and run nosetests. I'm encountering bugs such as #1214, #1216, #1217 with that simple usage. It would be good if these runs were part of the test suite.

Adding --no-identity option would preempt the unwanted prompt "Do you need Identity (usernames/passwords) in this project? [no]".

Other --no-(flag) options would be good for consistent and predictable tg-admin syntax.

Attachments

noidentity.patch Download (1.2 KB) - added by matiu 9 years ago.
Patch file for --no-identity

Change History

comment:1 Changed 10 years ago by jorge.vargas

  • Milestone set to __future__

this all seem like good ideas, if you provide patches they can be applied but I don't think anyone has time to work on them unless they are needed.

comment:2 Changed 10 years ago by Chris Arndt

  • Milestone changed from __future__ to 1.1
  • Version changed from trunk to 1.0.4b3
  • Summary changed from tg-admin: add --no-identity for unattended quickstart, make quickstart nosetesting a test to tg-admin: add option to quickstart project without identity without prompting

Changed summary to better reflect first issue mentioned here.

I will open another ticket about automating the quickstart tests.

comment:3 Changed 10 years ago by Chris Arndt

I hi-jacked ticket #792 for the second issue.

comment:4 Changed 9 years ago by matiu

  • Owner changed from anonymous to matiu
  • Status changed from new to assigned

Changed 9 years ago by matiu

Patch file for --no-identity

comment:5 Changed 9 years ago by matiu

Added --no-identity (in attached patch file)

Can't see any other quickstart options that need --no-x alternatives. -s -e and -o are all mutually exclusive or implicit and have defaults already that are not prompted for.

comment:6 Changed 9 years ago by matiu

  • Status changed from assigned to new
  • Owner changed from matiu to faide

Hi faide, I'm reassigning this to you, because I don't know who else to send it to. It just needs that little patch adding. Sorry for no test case, I tested it my manually and works well. Wouldn't hurt to give a retest before commiting though if you have a 'mo.

comment:7 Changed 9 years ago by faide

Héhé seems a ticket was dumped on me without getting enough attention :-)

comment:8 Changed 9 years ago by Chris Arndt

We should maybe create a collection ticket for issues related to tg-admin refactoring for TG 1.1.

comment:9 Changed 9 years ago by Chris Arndt

  • Priority changed from normal to high
  • Severity changed from normal to major

comment:10 Changed 9 years ago by Chris Arndt

  • Priority changed from high to normal
  • Severity changed from major to normal

Sorry, changed the priority of the wrong ticket. Setting it back to normal again.

comment:11 Changed 9 years ago by faide

  • Milestone changed from 1.5 to 1.1

comment:12 Changed 9 years ago by faide

Applied in 5454. Thanks.

comment:13 Changed 9 years ago by faide

  • Status changed from new to closed
  • Resolution set to fixed

comment:14 Changed 9 years ago by Chris Arndt

  • Status changed from closed to reopened
  • Resolution fixed deleted

This patch breaks quickstart projects which use the --no-identity option, because the identity configuration is not generated correctly.

comment:15 Changed 9 years ago by Chris Arndt

  • Status changed from reopened to closed
  • Resolution set to fixed

Fixed in r5479 and also ported it to the 1.5 branch.

Note: See TracTickets for help on using tickets.