Update sigulsign_unsigned.py for Python3 #6996
No reviewers
Labels
No labels
after freeze
automation
backlog
blocked
change-ack
change-nak
change-noreleng
changes
Closed As
Can't Fix
Closed As
Duplicate
Closed As
Fixed
Closed As
Fixed with Explanation
Closed As
Get back later
Closed As
Grooming
Closed As
Insufficient data
Closed As
Invalid
Closed As
It's all good
Closed As
taiga
Closed As
upstream
day-to-day
dev
docs
easyfix
epel
f26
f27
f28
f29
f30
f31
f32
f33
f34
f35
f36
f37
f38
f39
f40
f41
f42
f43
f44
f45
fedora
groomed
high-gain
high-trouble
in-progress
in-review
investigation
legal
low-gain
low-trouble
mass rebuild
medium-gain
medium-trouble
meeting
mini-initiative
new_artifact
ops
pdc_retirement
rawhide
RCA
review
script
sidetarget
sprint-0
sprint-1
sprint-2
sprint-3
sprint-4
sprint-5
unfrozen
waiting on external
Backlog Status
Needs Review
Backlog Status
Ready
chore
documentation
points
01
points
02
points
03
points
05
points
08
points
13
Priority
High
Priority
Low
Priority
Medium
Sprint Status
Blocked
Sprint Status
Done
Sprint Status
In Progress
Sprint Status
Review
Sprint Status
To Do
Technical Debt
Work Item
Bug
Work Item
Epic
Work Item
Spike
Work Item
Task
Work Item
User Story
No milestone
No project
No assignees
4 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
releng/tickets!6996
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "convert-to-cccolutils"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
This is a series of 3 patches designed to update the sigulsign_unsigned for Python3.
I converted from fedora_cert to cccolutils based on @puiterwijk 's recommendations.
The linter has a lot of issues with the script, but in the interest of time I caught only a few; the rest will take more time and we can work those out after the upcoming deadline for changes.
@mohanboddu - would you please test this out per our conversation?
@puiterwijk - the update was pretty trivial, however, if there's anything in here you'd like done differently let me know.
rebased
This needs to be an argument so it can be tested in staging.
You can just import this at the top of the file.
using a cli flag for staging would be best
rebased onto
02c6c362b3@ausil and @puiterwijk I have made the requested changes.
Pull-Request has been merged by mohanboddu