Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Planned auction for tender #4

Merged
merged 4 commits into from
Oct 30, 2015
Merged

Planned auction for tender #4

merged 4 commits into from
Oct 30, 2015

Conversation

leits
Copy link
Member

@leits leits commented Sep 30, 2015

No description provided.

@leits leits changed the title Add MESSAGE_ID and skipped_days Planned auction for tender Sep 30, 2015
@leits
Copy link
Member Author

leits commented Sep 30, 2015

MESSEGE_ID відправляється і логується skipped XX full days, якщо такі існують.

@leits
Copy link
Member Author

leits commented Sep 30, 2015

Добавив відправлення номеру потоку.

Змінна stream, яка передає своє значення у атрибут функції, що відправляється у лог, завжди дорівнює 1
Можливо, вона зовсім не змінюється.

@@ -162,7 +167,10 @@ def check_tender(tender, db):
except ResourceConflict:
planned = False
auctionPeriod = randomize(auctionPeriod).isoformat()
LOG.info('Planned auction for tender {} to {}'.format(tender['id'], auctionPeriod))
skipped_days_str = ''
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Не бачу потреби в суфіксі _str

@myroslav
Copy link
Member

Вона має стати 2 після планування 10 аукціонів на один і той самий день

On Wednesday, 30 September 2015, Andrew Leitsius notifications@github.com
wrote:

Добавив відправлення номеру потоку.

Змінна stream, яка передає своє значення у атрибут функції, що
відправляється у лог, завжди дорівнює 1
Можливо, вона зовсім не змінюється.


Reply to this email directly or view it on GitHub
#4 (comment)
.

....................................................................................................................................
Myroslav Opyr ▪ CTO ▪ Quintagroup ▪ +1.917.475.4725 ▪
http://quintagroup.com
˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙˙

@@ -0,0 +1,16 @@
[buildout]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Давай або тут в файлі, або в readme допишемо, для чого цей journal.cfg, і як будувати екземпляр для перевірки роботи журналювання

@leits leits force-pushed the master branch 17 times, most recently from b698851 to 066d9a9 Compare October 19, 2015 07:17
@leits leits force-pushed the master branch 4 times, most recently from 5725cec to c7485bc Compare October 21, 2015 14:00
@leits
Copy link
Member Author

leits commented Oct 26, 2015

Чи потребує новостворене логування 'Planned auction for lot {} of tender {} to {}' передачі в екстра таких же параметрів, як і попередні логування? ( 'PLANNED_DATE', 'PLANNED_STREAM', 'PLANNED_DAYS_SKIPPED')

@kroman0
Copy link
Contributor

kroman0 commented Oct 26, 2015

Так і ще додатково LOT_ID

@leits leits force-pushed the master branch 13 times, most recently from ee32879 to 7998460 Compare October 29, 2015 18:16
@leits
Copy link
Member Author

leits commented Oct 29, 2015

@kroman0 усі останні зміни у комітax "Add logging_context" i "Remove JournalHandler"

@leits
Copy link
Member Author

leits commented Oct 30, 2015

MESSAGE_ID
'Planned_auction_for_lot_of_tender' -> 'planned_auction_lot'
'Planned_auction_for_tender' -> 'planned_auction_tender'

kroman0 added a commit that referenced this pull request Oct 30, 2015
Planned auction for tender
@kroman0 kroman0 merged commit f9c740f into openprocurement:master Oct 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants