Skip to content
This repository was archived by the owner on Aug 5, 2022. It is now read-only.

Don't check disk size during rpm transaction when creating multiple part...#2

Open
biskett wants to merge 5 commits intointel:develfrom
biskett:devel
Open

Don't check disk size during rpm transaction when creating multiple part...#2
biskett wants to merge 5 commits intointel:develfrom
biskett:devel

Conversation

@biskett
Copy link
Copy Markdown
Contributor

@biskett biskett commented Feb 11, 2015

...itions

rpm check only "/" partition size before installing packages

Change-Id: Ic00e937530f926f58ee4099f7a079c640fc721a5
Signed-off-by: Yeongil Jang [email protected]

…artitions

rpm check only "/" partition size before installing packages

Change-Id: Ic00e937530f926f58ee4099f7a079c640fc721a5
Signed-off-by: Yeongil Jang <[email protected]>
@biskett
Copy link
Copy Markdown
Contributor Author

biskett commented Feb 12, 2015

If total package's size is larger than rootfs filesystem size, it makes errors even though second partition has extra partition's size.
So we find other general solution, but if not rpm consider all partition size and calculate file's size, i think there are no other solution for this.

@yongfengdu
Copy link
Copy Markdown

could you list the real case and number?
disk size, how many partition, each partition size, mount point for each partiton, package size

@biskett
Copy link
Copy Markdown
Contributor Author

biskett commented Feb 13, 2015

Sorry. I can't list partitions and package information of our products.
But we use various partitions for performance, storage efficiency and product issues like /boot, /opt and so on.
So if you want to test this problem, please fix rootfs size like 256Mb, and mount partition like /var or /usr or /opt as partition including large data.

Change-Id: I3dbb06c4682f0973f32ee94bd8db09c26733915e
Signed-off-by: Yeongil Jang <[email protected]>
Change-Id: I0df96b6ec576d46524d586ebfafb69b61952b73e
Signed-off-by: Yeongil Jang <[email protected]>
Change-Id: I7c1461bc37ddd93c8e54d15c26bbe4aee6d9668b
Signed-off-by: Yeongil Jang <[email protected]>
if use_mic_in_bootstrap option is true, don't sync mic into bootstrap

Change-Id: Ib4e17474541db77e61681ecd828a4bd2193cc606
Signed-off-by: Yeongil Jang <[email protected]>
@biskett
Copy link
Copy Markdown
Contributor Author

biskett commented Mar 12, 2015

Is there any additional modification point or alternatives?
And I added 4 commits. please review those patches too.

@yongfengdu
Copy link
Copy Markdown

I have no more comments, but now we are moving the mic development work from internal to tizen.org. (github is not the place we merge patches) so please resubmit the patch after the tizen.org repository is stable. I'll ping you when tizen.org is ready to receive patches.

@biskett
Copy link
Copy Markdown
Contributor Author

biskett commented Apr 29, 2015

How far are you on repository stabilization?
and please confirm below repository path.
(https://review.tizen.org/gerrit/tools/mic)

@yongfengdu
Copy link
Copy Markdown

Yes, the path is correct.

Although we haven’t try the release process from tizen.org, the above code repository should be good for accepting new requests for devel branch. So feel free to submit your change there.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants