Modify

Ticket #1150 (closed defect: fixed)

Opened 8 years ago

Last modified 8 years ago

migrate_info.py ignores perms variable

Reported by: https://www.google.com/accounts/o8/id?id=AItOawlJa6QJcUnwqaUiOAgop4U4wANp_y0hDu4 Owned by: desai
Priority: major Milestone: Bcfg2 1.3.2 Release (Bugfix)
Component: bcfg2-server Version: 1.0
Keywords: Cc:

Description

If I run migrate_info.py in a v1.2 bcfg2 repository the script keeps the owner and group info in the new info.xml files but ignores perms attribute. I think it should keep the perms attribute too.

Attachments

Change History

comment:1 Changed 8 years ago by https://www.google.com/accounts/o8/id?id=AItOawnSjgovXZr-_V3vGkvMSR0pc5LDykRc1Nc

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

It looks like migrate_info.py must be run *after* migrate_perms_to_mode.py. migrate_info.py preserves the 'mode' attribute, but not 'perms', since 'perms' is no longer supported by Bcfg2 1.3.

comment:2 Changed 8 years ago by https://www.google.com/accounts/o8/id?id=AItOawlJa6QJcUnwqaUiOAgop4U4wANp_y0hDu4

  • Status changed from closed to reopened
  • Resolution invalid deleted

You can't use migrate_info.py in a 1.2 repo because info files aren't XML.

I think that INFO_REGEX should include the following re at least in the migrate_info.py script:

'perms:(\s)*(?P<perms>\w+)|'

That way the script will keep the perms attribute and then you can use migrate_perms_to_mode.py to convert perms to mode.

comment:3 Changed 8 years ago by https://www.google.com/accounts/o8/id?id=AItOawnSjgovXZr-_V3vGkvMSR0pc5LDykRc1Nc

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

WARNING! You need to establish a session before you can create or edit tickets. Otherwise the ticket will get treated as spam.

View

Add a comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.